Cannot obtain data from a service containing a plus sign

Monitoring some Wireguard peers via CheckMK and Grafana is unable to display any peers containing a plus sign in the peer key. Appears to be an interpreting the + as a space. Below are the Grafana query and error.
{
  "request": {
    "url": "api/datasources/proxy/uid/f2e1c855-c920-4be0-826b-65f29496f2f4/cmk/check_mk/webapi.py?action=get_graph",
    "data": "request={\"specification\":[\"template\",{\"site\":\"[REDACTED]\",\"host_name\":\"[REDACTED]\",\"service_description\":\"WireGuard wg0 Peer nCnVnG/eLbqu0AS8ROTxehqQ+hO/aHNcCOyLQfN15is\",\"graph_name\":\"bandwidth_translated\"}],\"data_range\":{\"time_range\":[1709305413,1709327013]}}",
    "method": "POST",
    "hideFromInspector": false
  },
  "response": {
    "result_code": 1,
    "result": "Checkmk exception: Cannot calculate graph recipes: No matching entries found for query: GET services
Columns: perf_data metrics check_command
Filter: host_name = [REDACTED]
Filter: service_description = WireGuard wg0 Peer nCnVnG/eLbqu0AS8ROTxehqQ hO/aHNcCOyLQfN15is

Traceback (most recent call last):
  File \"/omd/sites/[REDACTED]/lib/python3/livestatus.py\", line 237, in query_row
    return result[0]
IndexError: list index out of range

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File \"/omd/sites/[REDACTED]/lib/python3/cmk/gui/plugins/metrics/graph_images.py\", line 223, in graph_recipes_for_api_request
    graph_recipes = graph_identification_types.create_graph_recipes(graph_identification)
  File \"/omd/sites/[REDACTED]/lib/python3/cmk/gui/plugins/metrics/identification.py\", line 58, in create_graph_recipes
    return type_cls().create_graph_recipes(spec_info, destination=destination)
  File \"/omd/sites/[REDACTED]/lib/python3/cmk/gui/plugins/metrics/graph_templates.py\", line 94, in create_graph_recipes
    row = get_graph_data_from_livestatus(site, host_name, service_description)
  File \"/omd/sites/[REDACTED]/lib/python3/cmk/gui/plugins/metrics/utils.py\", line 967, in get_graph_data_from_livestatus
    info = dict(zip(labels, sites.live().query_row(query)))
  File \"/omd/sites/[REDACTED]/lib/python3/livestatus.py\", line 239, in query_row
    raise MKLivestatusNotFoundError(
livestatus.MKLivestatusNotFoundError: No matching entries found for query: GET services
Columns: perf_data metrics check_command
Filter: host_name = [REDACTED]
Filter: service_description = WireGuard wg0 Peer nCnVnG/eLbqu0AS8ROTxehqQ hO/aHNcCOyLQfN15is

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File \"/omd/sites/[REDACTED]/lib/python3/cmk/gui/webapi.py\", line 116, in page_api
    resp = _execute_action(api_call, request_object)
  File \"/omd/sites/[REDACTED]/lib/python3/cmk/gui/webapi.py\", line 207, in _execute_action
    return _execute_action_no_lock(api_call, request_object)
  File \"/omd/sites/[REDACTED]/lib/python3/cmk/gui/webapi.py\", line 221, in _execute_action_no_lock
    \"result\": api_call[\"handler\"](request_object),
  File \"/omd/sites/[REDACTED]/lib/python3/cmk/gui/plugins/webapi/graphs.py\", line 35, in _get_graph
    return graph_spec_from_request(request)
  File \"/omd/sites/[REDACTED]/lib/python3/cmk/gui/plugins/metrics/graph_images.py\", line 235, in graph_spec_from_request
    graph_data_range, graph_recipes = graph_recipes_for_api_request(api_request)
  File \"/omd/sites/[REDACTED]/lib/python3/cmk/gui/plugins/metrics/graph_images.py\", line 225, in graph_recipes_for_api_request
    raise MKUserError(None, _(\"Cannot calculate graph recipes: %s\") % e)
cmk.gui.exceptions.MKUserError: Cannot calculate graph recipes: No matching entries found for query: GET services
Columns: perf_data metrics check_command
Filter: host_name = [REDACTED]
Filter: service_description = WireGuard wg0 Peer nCnVnG/eLbqu0AS8ROTxehqQ hO/aHNcCOyLQfN15is

"
  }
}

Hi, could you tell me which version of the plugin and which version and edition of checkmk you are using?

Apologies, I meant to include that in the first post.
CheckMK Raw 2.1.0p14
Grafana v10.3.3
CheckMK Plugin 3.2.2

I updated my CheckMK systems to 2.2.0p23 and changed the checkmk plugin to >=2.2 and it's working now. I guess the change from WebAPI to REST API helped there.
Sorry for the unnecessary issue.

Happy to read that.


"java.lang.IllegalStateException: Call fetchIfNeeded()" when calling remove() on a already null field

I noticed another Error that doesn't happens on Parse.com hosted server when trying to call "remove" on a null text field I get this error:
"java.lang.IllegalStateException: ParseObject has no data for 'alternateEmail'. Call fetchIfNeeded() to get the data."
the android code that causes this is this:
 super.remove("alternateEmail");
with the Parse hosted server I never get this error.
Can you please fix it?
Thanks!

The server shouldn't really affect this issue, and from looking at the android source, I don't see why this error could be thrown during a remove call.  https://github.com/ParsePlatform/Parse-SDK-Android/blob/f439b3d3ba02eaad52e653b6890b3a10e74168da/Parse/src/main/java/com/parse/ParseObject.java#L157
How are you getting the object?

It's a reference from the CurrentUser object.
And as I said it works well on the parse.com hosted server, so there shouldn't be a different.

any update on this?
I actually tried even calling fetchInBackground before calling "remove" but still the same error.
trying to call "put" with a null value cases a different exception (as in parse.com hosted site) so that isn't an option either.
Can you please check this?

??????

This sounds like an issue with the Android SDK. Can you open an issue over there?

How could it be an issue with android sdk?
The server is the one throwin the exception.
Also, this happens only on parse-server. Not on parse.com.

In that case can you provide some more details about what exception the server is throwing? Your original issue mentioned a java language exception, which is part of the Android SDK, not the server. If you could provide some consistent repro steps, that would be idea.

I'll do that ASAP.
But I don't think there was any exception printed to the log.

No response in many days, closing.

Dude, I don't have time for this at the moment. But this is a serious issue.
Can you try to reproduce?
IMHO you need to take a look into this.
But for now I abandoned parse-server because of all these differences from the parse.com server.

We did try to reproduce and weren't able to. If you can post a small example code that reproduces the error, or a cURL request that has different behaviour on Parse.com and Parse Server (the API Console can help with that) then we will be able to help.


post: tweeting from pull requests

Plan Error
the default workspace is currently locked by another command that is running for this pull request–wait until the previous command is complete and try again

@barbosa  I just read and it’s really good, thanks for writing.
I felt like missing a “Clojure” on the blogpost.
Maybe after the So to achieve what I had in mind, I decided to implement this one myself. you could have a new section title explaining that the article is coming to an end…
Also, if you want to make this post more technical, you could show the part of the code you changed in the workflow, so people understand what this post is all about in terms of a “technical perspective”.

@djalmaaraujo thanks!
I wanted to focus more on the process than the technical aspect of it. That's the main reason I didn't want to include code in there and leave that for those who'd be interested in knowing more to visit our public repo (on purpose).


Fixing PerfBenchmarkDriver

Due to the change in #3864, controller now registers as a helix
participant. This causes the issue with PerfBenchmarkDriver
because it separately create helixZkManager and it tries to
register controller participant with the same host and port.
This pr resolves the issue.

Codecov Report

Merging #4109 into master will increase coverage by 21.1%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #4109      +/-   ##
============================================
+ Coverage     45.74%   66.85%   +21.1%
- Complexity        0       20      +20
============================================
  Files          1037     1037
  Lines         51426    51426
  Branches       7192     7192
============================================
+ Hits          23525    34379   +10854
+ Misses        25891    14683   -11208
- Partials       2010     2364     +354

Impacted Files
Coverage Δ
Complexity Δ

...er/validation/BrokerResourceValidationManager.java
25% <0%> (-56.25%)
0% <0%> (ø)

...he/pinot/core/query/pruner/ValidSegmentPruner.java
57.14% <0%> (-28.58%)
0% <0%> (ø)

...egation/function/customobject/MinMaxRangePair.java
75.86% <0%> (-24.14%)
0% <0%> (ø)

...r/validation/RealtimeSegmentValidationManager.java
55% <0%> (-20%)
0% <0%> (ø)

.../impl/dictionary/FloatOnHeapMutableDictionary.java
82.22% <0%> (-11.12%)
0% <0%> (ø)

...e/impl/dictionary/LongOnHeapMutableDictionary.java
82.22% <0%> (-11.12%)
0% <0%> (ø)

...impl/dictionary/FloatOffHeapMutableDictionary.java
87.27% <0%> (-5.46%)
0% <0%> (ø)

...pinot/core/operator/docidsets/OrBlockDocIdSet.java
84.9% <0%> (-3.78%)
0% <0%> (ø)

...not/broker/broker/helix/ClusterChangeMediator.java
66.66% <0%> (-3.45%)
0% <0%> (ø)

...e/operator/dociditerators/MVScanDocIdIterator.java
60.6% <0%> (-3.04%)
0% <0%> (ø)

... and 521 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 26330f3...0685bd7. Read the comment docs.


FSharp.Option.Some from C# client side

Description
i have a simple type generated with Xsd type provider,
then a simple record marked CLIMutable i use as interface from c# client side.
in this way i can construct a record from c# and then use it to generate a new XmlElement to serialize.
My problem is that with the option type, referenced from C# side for some fields of the record, which have minOccur = 0, when i pass an option different from None, like FSharpOption.Some(0.00m) i get runtyme type error exceptions from within the serializer.
the error though comes from original Xml type provider:
C:\Git\FSharp.Data\src\Xml\XmlRunti
me.fs:line 219
Repro steps
Please provide the steps required to reproduce the problem

make a library from fsharp side with generated type provider from xsd.
let the xsd have some optional fields. now make a "bridge" record type (CLIMUtable) from FSHarp so that when you want to create an istance of the provided type, you create an instance of the bridge record type and pass that on to a function, let's say

let makeXml (recordType : RecType) : System.Xml.Linq.XElement
makeXml : RecType -> System.Xml.Linq.XElement

Reference the library from C# app and try to call makeXml, with all option values set to None, you'll notice no exceptions are thrown.

Try to build the recordType instance with at least one optional value set to FSHarpOption.Some(0.00m)

Expected behavior
providing FSharp.Option.Some(T value) to the RecordType constructor should give no errors to,
and i should be able to get a valid System.Xml.Linq.XElement instance as output.
Actual behavior
When i try to build the recordType instance with at least one optional value set to FSHarpOption.Some(value)
Here's the exception i get.
08:19:49,397 [12] Service - System.Exception: Unexpected value: Some(0.00)
   at Microsoft.FSharp.Core.Operators.FailWith[T](String message)
   at Microsoft.FSharp.Core.PrintfImpl.go@523-3[b,c,d](String fmt, Int32 len, FSharpFunc`2 outputChar, FSharpFunc`2 outa, b os, FSharpFunc`2 finalize,
 FSharpList`1 args, Int32 i)
   at Microsoft.FSharp.Core.PrintfImpl.run@521[b,c,d](FSharpFunc`2 initialize, String fmt, Int32 len, FSharpList`1 args)
   at Microsoft.FSharp.Core.PrintfImpl.capture@540[b,c,d](FSharpFunc`2 initialize, String fmt, Int32 len, FSharpList`1 args, Type ty, Int32 i)
   at <StartupCode$FSharp-Core>.$Reflect.Invoke@720-4.Invoke(T1 inp)
   at <StartupCode$FSharp-Data>.$XmlRuntime.serialize$cont@209(CultureInfo cultureInfo, Object v, Unit unitVar) in C:\Git\FSharp.Data\src\Xml\XmlRunti
me.fs:line 219

Known workarounds
Still have not found a workaround, i'll try to convert options to nullables form fsharp side, yet it's not very comfortable to accomplish in a fast manner.
Related information

Windows 8 64 bit
master
.NET 4.6

could not reproduce :(


API endpoints are 404

When I've signed in, the page tries to make an api/?method=foursquare.venues.search request to find nearby venues, but that comes up as a 404 Not Found page. I tried requesting api/ with no arguments too, and it was also 404.
I found I was able to fix it by copying api_rest.php to api.php so it's available as api/ also. Did I misconfigure something that should have already routed the request properly?

That sounds like MultiViews are enabled, in Apache.
Can you try turning them off and see if that fixes the problem? If it I does I'll figure out how to make that explicit in the .htaccess file.

Ahhh, yeah, not only were MultiViews enabled but I had overrides disabled, so none of the rewrite rules in www/.htaccess were in effect. Enabling the rewrite rules with AllowOverride All seems to fix it, since it doesn't have to fall back to multiviews. Thanks!


Use installer paths from dependency

Apologies if this is already a feature, but I can't get it to work for me.
I have a bunch of sites that share common modules and libraries (drupal 8) and I'm creating a composer lib that all those sites can use to keep it easier to maintain patches etc across sites.
I'm having issues getting the installer paths to be inherited from the dependency though, or is this not supported by composer/installers?
I.e:
My root package:
{
    "name": "djr/site-vanilla",
    "description": "Project template for Drupal 8 projects with composer",
    "type": "project",
    "license": "GPL-2.0+",
    "config": {
        "discard-changes" : true
    },
    "repositories": [
        {
            "type": "vcs",
            "url": "myrepo.com/djr_libs.git"
        },
        {
            "type": "composer",
            "url": "https://packagist.drupal-composer.org"
        }
    ],
    "require": {
        "djr/djr_libs": "dev-master",
        "composer/installers": "^1.2.0",
        "cweagans/composer-patches": "^1.5.0"
    },
    "extra": {
        "installer-paths": {},
        "patches": {},
        "enable-patching": true
    },
    "minimum-stability": "dev",
    "prefer-stable": true
}

And djr_libs composer file:
{
    "name": "djr/djr_libs",
    "description": "Composer library to consolidate common site functionality.",
    "type": "project",
    "config": {
        "discard-changes" : true
    },
    "repositories": [
        {
            "type": "composer",
            "url": "https://packagist.drupal-composer.org"
        }
    ],
    "require": {
        "composer/installers": "^1.2.0",
        "cweagans/composer-patches": "^1.5.0",
        "drupal/core": "8.1.*"
    },
    "minimum-stability": "dev",
    "prefer-stable": true,
    "extra": {
        "installer-paths": {
            "web/core": ["type:drupal-core"]
        },
        "patches": {
            "drupal/core": {
                "UUID node links": "https://www.drupal.org/files/issues/2353611-uuid-link.67.patch",
                "Menu link deployment": "https://www.drupal.org/files/issues/2577923-menu-link-deploy.40.patch"
            }
        }
    }
}

The patches are correctly applied, but drupal/core is installed in the root directory, not web/core.

You need to configure the installer-paths in the root composer.json.  A package vendor shouldn't have to care about the filesystem you are installing it on, and giving that power to a third party opens up opportunity for malicious acts.  Imagine a package pulled in as a dependency declares it is to be installed at the absolute path /... When the package is to be updated, the rm -rf will hurt (hopefully not, composer shouldn't be run as root).

@slbmeh fair enough, I've done that for the time being so no worries. Thanks for the response.


Gallery not working when container div is hidden (display:none)

Swiper is not working properly if initialized inside an hidden div.
I want the swiper gallery to show after clicking on a "Show the gallery" link.
My page looks like this:

<!— Swiper —>

…

<!— Add Arrows —>

<!— End Swiper —>

I tried with the @AaronOpfer's timeout solution here: https://github.com/thebird/Swipe/issues/121

I changed "slider" with "slide-event", the id of my swiper div.
I am not a javascript expert, what's wrong?
Keep getting a browser's console "Uncaught ReferenceError: Swipe is not defined".
Any suggestion?

Bad forum sorry


fix determine merge iterator considering segment num

Proposed changes
picked #29269
Issue Number: close #xxx

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall


Update install.md

Learn Build status updates of commit c569fbb:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

articles/desktop-flows/install.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

Learn Build status updates of commit 60cb187:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

articles/desktop-flows/install.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel


Update package version

3.0.1-alpha.3 (2023-10-25)
Bug Fixes

frontend: fix audio tool preview (55476d6)

:tada: This PR is included in version 3.0.1-alpha.5 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.1.0-alpha.13 :tada:
The release is available on npm package (@alpha dist-tag)
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.1.0-alpha.13 :tada:
The release is available on npm package (@alpha dist-tag)
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 3.0.1-alpha.6 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.1.0-alpha.11 :tada:
The release is available on npm package (@alpha dist-tag)
Your semantic-release bot :package::rocket:


upstream #3151 remove  arg '--max-reconcile-rate' to crossplane deplo…

…yment
Signed-off-by: Steven Borrelli<EMAIL_ADDRESS>Description of your changes
Sync upstream PR https://github.com/crossplane/crossplane/pull/3151
I have:

[ ] Read and followed Upbound's contribution process.
[ ] Run make reviewable to ensure this PR is ready for review.
[ ] Added backport release-x.y labels to auto-backport this PR, as appropriate.

How has this code been tested

Not needed. These files are generated during build.


Error when start debug in win8 using visual studio code

Debugger listening on [::]:39939
react-planner-master 1.2.7\react-planner-master\lib\components\viewer3d\libs\orbit-controls.js:18
var THREE = window.THREE || require('three');
^
ReferenceError: window is not defined
at Object. (\react-planner-master 1.2.7\react-planner-master\lib\components\viewer3d\libs\orbit-controls.js:18:26)
at Module._compile (module.js:570:32)
at Object.Module._extensions..js (module.js:579:10)
at Module.load (module.js:487:32)
at tryModuleLoad (module.js:446:12)
at Function.Module._load (module.js:438:3)
at Module.require (module.js:497:17)
at require (internal/module.js:20:19)
at Object. (\react-planner-master 1.2.7\react-planner-master\lib\components\viewer3d\viewer3d.js:25:22)
at Module._compile (module.js:570:32)

Are you using server rendering?
"window" should be always defined in browser..

i am using localhost
when run without debug it's run ok
but when start to debug this error appear
Debugger listening on [::]:39939
react-planner-master
1.2.7\react-planner-master\lib\components\viewer3d\libs\orbit-controls.js:18
var THREE = window.THREE || require('three');
^
ReferenceError: window is not defined
at Object. (\react-planner-master
1.2.7\react-planner-master\lib\components\viewer3d\libs\orbit-controls.js:18:26)
at Module._compile (module.js:570:32)
at Object.Module._extensions..js (module.js:579:10)
at Module.load (module.js:487:32)
at tryModuleLoad (module.js:446:12)
at Function.Module._load (module.js:438:3)
at Module.require (module.js:497:17)
at require (internal/module.js:20:19)
at Object. (\react-planner-master
1.2.7\react-planner-master\lib\components\viewer3d\viewer3d.js:25:22)
at Module._compile (module.js:570:32)
On Wed, Apr 26, 2017 at 3:30 PM, Alessio Carrafa<EMAIL_ADDRESS>wrote:

Are you using server rendering?
"window" should be always defined in browser..
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/cvdlab/react-planner/issues/87#issuecomment-297408461,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEh8hGeY_QO6UWQB7KtIXDrHUHrqoE1aks5rz0bkgaJpZM4NIMq5
.

--
Regards,
Emad eldin ahmad ِabdin

Hi @imadOmran,
Are you using the visual studio code debugger for JS server side? If so, as this library is only client-side, you are experiencing issues with some things that are defined in the browser (e.g window ) but are undefined for nodeJS. As suggested here to debug this library one should use a browser debugger

thanks @Danilo salvati
how i can to set element properties
let height = element.properties.get('height').get('length');
On Thu, Apr 27, 2017 at 8:37 AM, Danilo Salvati<EMAIL_ADDRESS>wrote:

Hi @imadOmran https://github.com/imadOmran,
Are you using the visual studio code debugger for JS server side? If so,
as this library is only client-side, you are experiencing issues with some
things that are defined in the browser (e.g window ) but are undefined
for nodeJS. As suggested here
https://code.visualstudio.com/docs/languages/javascript#_debugging to
debug this library one should use a browser debugger
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/cvdlab/react-planner/issues/87#issuecomment-297625235,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEh8hFXeIE6-ka4ejsdGJdMXaU3PZXEOks5r0De_gaJpZM4NIMq5
.

--
Regards,
Emad eldin ahmad ِabdin

Since the original issue is solved, I'd prefer to close it. For this new question feel free to open a new issue :smile:

sorry for this misunderstand
when debug not work for me i delay this for next time ,
so still the same (but i will work for it and i will send to you the answer)
i am so sorry for this.
thanks
On Fri, Apr 28, 2017 at 8:44 AM, Danilo Salvati<EMAIL_ADDRESS>wrote:

Closed #87 https://github.com/cvdlab/react-planner/issues/87.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/cvdlab/react-planner/issues/87#event-1061619193, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AEh8hNwh9UeFgCLBxwMqQBeJKr83bmWkks5r0YrIgaJpZM4NIMq5
.

--
Regards,
Emad eldin ahmad ِabdin

Sorry for the mistake, so, if I have understood your last message, debugging with visual studio code is not working yet...

yes
On Tue, May 2, 2017 at 10:39 AM, Danilo Salvati<EMAIL_ADDRESS>wrote:

Sorry for the mistake, so, if I have understood your last message,
debugging with visual studio code is not working yet...
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/cvdlab/react-planner/issues/87#issuecomment-298557435,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEh8hIdQiUrLC1vP9tUIUuMUnKJjGpjIks5r1uvcgaJpZM4NIMq5
.

--
Regards,
Emad eldin ahmad ِabdin


Register parameters for CMKL and override clone()

Registers MKL class parameters and handles 3rd party lib parameters in ::clone(). See #3725. I couldn't get shogun to compile with CPLEX enabled, it looks to me as if there are plenty of #ifdef USE_CPLEX...#endif blocks in shogun that actually don't compile anymore.

One question. Do we have some tests of MKL results: unit/integration? Because if not it might be dangerous to merge this....

Everything that is not registered will not be part of serialization and clone... is it fine with those here? What about initialization of the memory of them?

Sorry that was an obsolete comment. These are actually registered, W* I removed (was unused).
I'm not registering the clock. If one wanted to serialize a CMKL mid-action that might be relevant, but honestly I don't see the practical relevance. Also, CTime can't be cloned, it doesn't have it's params registered.

One question. Do we have some tests of MKL results: unit/integration? Because if not it might be dangerous to merge this....

I'm not sure how well the actual MKL logic is tested. There are tests using MKL though. And even clone it (maybe they are using xval).  Anyways, we know that the before state was wrong, I think this should be merged.

@karlnapf i'd merge this... soooo?

@vigsterkr Added the error and changed NULL to nullptr

Happy to merge once registered parameter get default values

Moved member init into register_params()
Marked clone() as virtual


Performance issues in react-native

Hey,
Just to let ya know - we've been seeing really poor performance with redux-logger enabled when used with react-native on iOS - even on new iPhone devices. Use of the logger would cause the app to freeze for seconds at a time.
I've attached a screenshot from xcode instruments which is what led us to disabling redux-logger:

Not entirely sure whether this is redux-logger's fault entirely, but may be worth looking into

If console.logs are slow this seems like something better addressed in RN itself.

redux-logger just calls console.log() in basic variant. Do you pass any options to createLogger?

@theaqua we do not.
I haven't noticed any performance issues with use of our own logging so when the problems ceased after removal of redux-logger I just assumed it was at fault :p. The weird thing is that redux-logger doesn't really seem to log too often - just on actions etc so I have no idea why performance would be so badly affected.
If I get a chance it's prob worth doing some profiling with the chrome dev tools or equivalent to see where the time is being spent within the javascript as opposed to the native code.

Besides being slow, did everything else in your app work normally @mtford90 ? I added redux-logger into my middleware chain and stuff began failing silently. Reducers stopped getting called, actions stopped being dispatched, etc. Removed redux-logger and everything went back to normal.

Yeah, I just discovered that this was the cause of our performance issues. Would highly recommend mentioning in the documentation that this should be disabled for production builds.

It definitely should! (Just like DevTools)

Would be curious to see if it has anything to do with the formatting functions (e.g. colors) involved in the logging. The only time this affected my application was when I was not debugging with chrome. When chrome's logger was used, everything was mostly smooth.

Having removed the formatters, and replaced with pure console.log functions, I'm convinced it's the formatting of the statements that causes the hang. Currently investigating how to differentiate between formattable window instances and not, but for now this is what I've done as a workaround:
const args = (global.__DEV__) ? [thunk, api, logger] : [thunk, api];
const createStoreWithMiddleware = applyMiddleware(...args)(createStore);

Actually, reading through releases, it looks like colors: false as an option might suffice. Sorry for the comment-spam. Will try and report back if no one else does before me.

You're still (quite probably) wasting memory because you're logging state objects so they (probably?) can't be GC'ed. So don't log in prod.

Oh for sure, can't see why one would want to log in a production react native app anyway.

I close this issue because for ~3 months we're still don't know what is causes problem and issue with that title scares newcomers :)
If anyone have steps to reproduce and it's precisely our problem, please file a new issue.

I really think this issue should be mentioned clearly in the docs. I run into the same problem and it took me a while to understand the terrible performance issues I was getting on iOS where just caused by redux-logger. I don't know exactly why this happens (I guess it's got to do with the GC not being able to remove the states as console.log works by reference?), but it's definitely happening.

@fabiosussetto lol I can't mention it in docs while nobody doesn't understand why it happens. See it as a newcomer: "hey, we have the nice library with a bunch of console.logs, but we (2 people of 173,301 downloads) stuck with one problem of performance". I definitely will not use that library.

@theaqua probably I didn't make my point clear enough. There's a bunch of places where people mention that using console.log causes very noticeable performance issues in react native (e.g. https://github.com/aksonov/react-native-router-flux/issues/199#issuecomment-190474938)

hey, we have the nice library with a bunch of console.logs, but we stuck with one problem of performance". I definitely will not use that library

I don't really think it's a good idea to "hide" potential issues just so that people don't get scared by them, it's not really transparent.
IMO a "Known issues" section could be easily added to the readme, mentioning something like:
"We are aware some people reported performance issues using console.log (and thus redux-logger) in React Native. If you get these issues, please consider removing redux-logger in production".
I don't see how this can harm this project, all the best project have a "Known issues" section where potential problems are mentioned while being investigated/fixed.
For me it was a big waste of time realising that the performance issues I was getting where just because of this. I would have been very happy to be aware of this potential problem from the beginning.
Then it's obviously up to you.

As a note, redux-logger is slowing down React Native for me too.

Yes this tripped me up also, very annoying. The following snippet should be added to the docs as the default configuration.
From: slava-sh https://github.com/evgenyrodionov/redux-logger/issues/6
const middleware = [
  ... other middleware ...
  __DEV__ && logger,
].filter(Boolean);
const createStoreWithMiddleware = applyMiddleware(...middleware)(createStore);

If building for an iOS device, in xCode, click the Run button whilst pressing ALT to change the build scheme (and toggle to release).
This significantly slowed down the performance of a simple app, simply disabling it fixed everything.
👍  Update Docs

Reopened and mentioned in docs. I agree we need fix this.

Hello,
I have encountered a severe slow down in my app too, especially because my state contains a lot of data. Moreover, Xcode's console outputs all state's changes and this is exactly when slow downs  occur.
If i force the use of my store.prod.js instead of .dev.js, everything works fine.

I'd also like to note that using console.log() (accidently in a release app, for example) will significantly slow down performance (this is especially true on android).
I had a bug where a console.log statement (in a release build - with no redux logger) caused slow-animations and slow-responsive touch behaviour when using the Navigator component (on android), It was due to the console.log statements - removing them all fixed everything.
This is noted in the official docs here (Keep it in mind if you're new to React Native - like me):
https://facebook.github.io/react-native/docs/performance.html#console-log-statements
I tried using babel-plugin-transform-remove-console, but this doesn't currently work with RN, there's an open issue here:
https://github.com/facebook/react-native/issues/10412
It seems, for now, remove/uncomment all console.log statements manually (and also disable the  redux-logger dev mode in release builds as noted above).
Hope this helps.

I have also notice that console.log (and console.whatever) could slow down your app.
A great alternative (instead of using another weird babel plugin) is to wrap console into you own module:
`if (!DEV) {
console.log =
console.info =
console.error =
console.warn =
console.debug =
console.trace = () => {};
}
export function log() {
_d('log', arguments);
}
export function info() {
_d('info', arguments);
}
export function warn() {
_d('warn', arguments);
}
export function error() {
_d('error', arguments);
}
export function debug() {
_d('debug', arguments);
}
export function trace() {
_d('trace', arguments);
}
function _d(type, args) {
if (!DEV) return;
if (args[0]) {
args[0] = _getHeader() + args[0];
}
Function.apply.call(console[type], console, args);
}
function _getHeader() {
var date = new Date();
var d = date.getDay();
if (d.toString().length == 1)
d = '0' + d;
var m = date.getMonth() + 1;
if (m.toString().length == 1)
m = '0' + m;
var y = date.getFullYear();
var h = date.getHours();
if (h.toString().length == 1)
h = '0' + h;
var mn = date.getMinutes();
if (mn.toString().length == 1)
mn = '0' + mn;
var s = date.getSeconds();
if (s.toString().length == 1)
s = '0' + s;
var ms = date.getMilliseconds();
var time = d + '/' + m + '/' + y + ' ' + h + ':' + mn + ':' + s + ' ' + ms;
return '[YOUR_APP_NAME ' + time + '] ';
}`

By the way, you can pass own implementation of console in options.console. For example,
const console = {
  log: () => {
    // your code
  },
  group: () => {
    // your code
  },
  // etc
};

const logger = createLogger({
  console: console,
});

How about predicate func  option.
const logger = createLogger({
  predicate: () => isDebugging,
  collapsed: true,
  duration: true,
 .....
})

can we close this?

@sibelius, not sure yet. I didn't manage to analyze the problem. Do you have any feedback?

This is not a problem of redux-logger, it is a problem with console.log being slow on react native

For those interested, here is a setting that works for me and still permits me to dev on Android:
const isVerboseLoggingPlatform = Platform.OS === 'android';

  if (__DEV__) {
    middleware.push(
      createLogger({
        stateTransformer: state => {
          return isVerboseLoggingPlatform ? state : 'state';
        },
      }),
    );
  }

I suspect shipping state objects from phone/emulator to xcode requires some expensive serialization, not much we can do about it imho

I've run into this problem as well. It's not a problem with redux-logger itself. It's a problem with console.log as it was already mentioned. Anyway I've done a workaround for this. So just sharing the ideas.
There are 3 environments you can run your React Native app:

Production
Debug remotely
Debug (without chrome)

Production
You definitely need to switch the redux-logger off (for example):
const IS_PRODUCTION = process.env.NODE_ENV === 'production'

  const middlewares = [
      !IS_PRODUCTION &&
      createLogger({
        ....
      }),
  ]

  // remove non function middlewares
  middlewares.filter(middleware => _.isFunction(middleware))

Debug remotely
The IS_PRODUCTION will be set to false. It means the redux-logger will be pushed to middlewares. In this case it works like Andrew Goodale said in this thread

It's because when you run with remote debugging, the JavaScript code is executed inside Chrome's V8 engine. That is able to do various optimizations with JIT compiling. When running without debugging, JavaScript is executed by iOS's JavaScriptCore, which cannot JIT-compile JavaScript. There are also differences in performance between the two engines. For example, I noticed that mutating Date objects is slower in JavaScriptCore than V8.

So we can leave as it is, because everything works smoothly
Debug (without chrome)
The problem was with this case for me. Everything went really slowly. And I don't really want the redux-logger in this case, because the JS runs on Device/Emulator without chrome/react-devtools. So I added this condition:
const { pathname } = window.location || {}
const IS_RUNNING_IN_CHROME = pathname && pathname.indexOf('debugger-ui')
const IS_PRODUCTION = process.env.NODE_ENV === 'production'

  const middlewares = [
      IS_RUNNING_IN_CHROME && !IS_PRODUCTION &&
      createLogger({
        ....
      }),
  ]

  // remove non function middlewares
  middlewares.filter(middleware => _.isFunction(middleware))

If anybody knows there is a better way how to check that the chrome is running - tell me please :)

@xotahal, thanks for sharing your experience!

By the way, you can pass own implementation of console in options.console. For example,
const console = {
  log: () => {
    // your code
  },
  group: () => {
    // your code
  },
  // etc
};

const logger = createLogger({
  console: console,
});

yes this worked for me to prevent VS Code debug console to stop working:
  const logger = createLogger({
    console: { log: line => console.log(line) }
  });

It may be a bit over simplified and I don't see the pretty colors anymore, but solved my problem


How to get/set the Caret Position?

I'm working at a XML Editor with Search and GoToLine. Most parts are already implemented but i cannot find a way to set the Caret position for both.

Use property CurrentPosition for getting the current position. And call method GotoPosition to set a position.

With GoToPosition(LineNumber) the Caret gets set only to the character of the given number (For example if my text has 106 Lines and I want to do to Line 50 im in Line 2 and character 50)

Sorry I don't understand you. Method GotoPosition accepts the character position not a line number.

Yes but I want to set the Caret to a specific Line with my own GoToLine Dialog

You can look at Lines property which each of them has the Position and then use GoToPosition.

I thought:
MyScintilla.Lines[23].Goto();
or
MyScintilla.GotoPosition(MyScintilla.Lines[23].Position);

@Stumpii and @Aghlara are correct. If you want to get to a specific character on a specific line, do the math that would determine what position that is, e.g. line position start + offset.
However, I might suggest that you're making it unnecessarily hard for yourself. Why isn't your search/find dialog giving you the document position directly instead of the line+offset? If it did that you could pass the value straight to GoToPosition.


Updates for astro-sst after SST v3 and Astro v5

Changes after SST v3
The astro-sst package use to be located inside the core repository in v2. I can no longer find the package in this repo (search) and assume it's still being deployed from the v2 repo.
To make contributions easier, could the astro-sst package, and presumably the others in v2/packages, be moved into this repository?
Changes after Astro v5
I'm unable to get deploys working with Astro v5.
16:52:47 [ERROR] [vite] x Build failed in 32ms
[vite:build-import-analysis] [plugin vite:build-import-analysis] @astrojs-ssr-adapter (1:14): Failed to parse source for import analysis because the content contains invalid JS syntax. If you are using JSX, make sure to name the file with the .jsx or .tsx extension.
file: @astrojs-ssr-adapter:1:14

1: export * from undefined;
                 ^

  Location:
    @astrojs-ssr-adapter:1:14

After looking at the adapter code locally, there seems to be a few incompatibilities (AstroAdapter, in particular). The package also depends on @astrojs/webapi, which appears to be deprecated.
Misc
There are a few open issues for Astro as well. I don't think they're all critical, but I'll include them here for reference.

https://github.com/sst/sst/issues/4611
https://github.com/sst/sst/issues/4949

Yeah we should do this.

Oh, I am glad to see mention of the sharp issue here.  Been struggling for 3 days trying to solve. If I set up a separate toy hello world lambda, I don't have a problem accessing the dependency.

Thanks for pulling everything here @san4d!
A few things needs :

Move astro-sst out of v2 repo
Support Astro v5
Use dynamic origins.
Context:

The serverRoutes option in astro-sst is no longer required.
It's there currently b/c CloudFront only allows for 25 behaviors by default. If we create 1 behavior per route/asset, it can easily go over limit. We ended up working around this limitation by using a default catch-all failover behavior to handle all routes/assets. But because the catch-all failover behavior doesn't support non-GET requests, we ask users to list out serverRoutes in their astro config. And the Astro component creates 1 POST behavior for each.
Recently CloudFront announced dynamic origins. We can remove this hack.

Rework edge mode.
Context:

Currently the deploying to edge use Lambda@Edge. L@E has limitations, slow to deploy, cannot be removed (has to wait 20min).
SST v3 can deploy to multiple regions cleanly & natively. I want to move away from L@E, and have the component deploy to multiple regions. ie. regions: ["us-east-1", "eu-west-1"], or regions: "all".

I'll get 1 done asap. And the community can start tackling 2, while I rework 3 & 4.

#1 done. Moved to https://github.com/sst/astro-sst

Thank you, @fwang!

Any pointers on where to start to get v5 to work?
@san4d What did you find out what needs to be done to support v5?

Started a v5 branch here:
https://github.com/sst/astro-sst/pull/2/files
I'm running into an issue with image service support for static sites, but I'm able to deploy in server mode. @mvanleest @weisisheng I'd appreciate it if you could test my branch on your projects and let me know if you run into any issues.

Clone my repo and checkout the MR branch
Install bun and build the branch
Update your package.json to point to the local version:

"astro-sst": "file:/path/to/astro-sst/packages/astro-sst/"

I'm not experienced with bleeding edge local dev.  I'm trying to build from a separate directory and getting 'Error: Failed to resolve entry for package "astro-sst".'  Does it matter that astro was build with npm and the branch built with bun?

Because of time pressures and inability to fix some obscure env / vite issues, I've been forced to switch to next.js which I am more familiar with.  Will keep watching this space and try to assist.


Jsapi: Implement missing EncoderConfig methods

rav1e_js

add EncoderConfig.setColorDescription(...)
add EncoderConfig.setMasteringDisplay(...)
encoder

Pass EncoderConfig as ref to constructor
--> This enables still using the config afterwards
Rename [FrameEncoder, VideoEncoder]::fromEncoderConfig(...) to ::new(...)
Rm ::default() from FrameEncoder, VideoEncoder

mod: pub use encoder::VideoEncoder
tests: Clean utils::simple_encoding
bump version to 0.2.1

Coverage increased (+1.6%) to 82.014% when pulling a826e4c408c673115b2478404eeeabd76b3412d2 on Urhengulas:jsapi-v0.1.1 into 99114995e8771dd923a146e5616f7474e9b33eb7 on xiph:master.

Good idea

Am 29.06.2020 um 08:12 schrieb Luca Barbato<EMAIL_ADDRESS>@lu-zero commented on this pull request.
In src/api/color.rs:

@@ -41,33 +41,34 @@ pub use v_frame::pixel::ChromaSampling;
/// Supported Color Primaries
///
/// As defined by “Color primaries” section of ISO/IEC 23091-4/ITU-T H.273
+#[wasm_bindgen]
wasm_bindgen should not require this, can you poke upstream and ask them to fix that in the macro?

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

The version should be 0.3 since you are breaking the API by removing fromEncoderConfig and default.

Alex Crichton confirmed that there is currently no better way than our current workaround for the enum-problem (https://github.com/rustwasm/wasm-bindgen/issues/2273#issuecomment-685965892).
So I would propose to actually go with it for now. Even if it is kinda not beautiful...

If this is still relevant, the PR should be ported to https://github.com/rust-av/rav1e_js


Priming fields for aggregation query builder

Feature Request

Q
A

New Feature
yes

RFC
yes

BC Break
no

Summary

Looks like right now there is no way to use fields priming for aggregation queries (with hydration enabled). It would be nice feature, I think. Or it's possible somehow?

What is your use-case? Priming was added since there's no way to use joins in queries. For aggregation pipelines, the $lookup operator provides this functionality, so I'd recommend you use that whenever you know that you're going to need referenced documents.
I'll also add that if you find yourself in the need to prime references to avoid n+1 problems, your schema might need improving, as the references should be changed to embedded documents instead. I'm aware that this does not always work or satisfy every use-case, but in general you should prefer embedded documents over references and primers.

Use-case pretty common, I think: for example, I need to calculate some statistic data per user (or just $project document to different format) and return collection of these elements like a regular document's collection with avoiding n+1 situations.
I never use hydration for aggregation query builder's before and always done it manually. I thought, since there a way to hydrate my aggregation results and with them like with regular documents, priming can be used for them too

In this case, you'd $lookup your reference and map it as @EmbedOne or @EmbedMany in the aggregation result document. This negates the need for lookup as the "priming" will be performed in the original aggregation pipeline.

@alcaeus ok, thank you for advice!


Roll engine 0816c040fbba..f9c45276a057 (17 commits)

https://github.com/flutter/engine/compare/0816c040fbba...f9c45276a057
git log 0816c040fbba..f9c45276a057 --first-parent --oneline
2020-02-03<EMAIL_ADDRESS>[fuchsia] Add additional contributing instructions for Googlers (flutter/engine#16337)
2020-02-03<EMAIL_ADDRESS>Roll src/third_party/dart ed441e48d055..c8ed304e979a (1 commits) (flutter/engine#16328)
2020-02-03<EMAIL_ADDRESS>Roll fuchsia/sdk/core/linux-amd64 from AtXwP... to Tszo5... (flutter/engine#16334)
2020-02-03<EMAIL_ADDRESS>Roll src/third_party/skia 3d05192abedf..f1e316ff39e7 (3 commits) (flutter/engine#16329)
2020-02-03<EMAIL_ADDRESS>Roll fuchsia/sdk/core/mac-amd64 from jtwQF... to 6h3IH... (flutter/engine#16332)
2020-02-03<EMAIL_ADDRESS>Change NSLog to FML_DLOG (flutter/engine#16331)
2020-02-01<EMAIL_ADDRESS>Roll src/third_party/skia 5d92ae2b56df..3d05192abedf (1 commits) (flutter/engine#16326)
2020-02-01<EMAIL_ADDRESS>Roll src/third_party/skia 776293dceefd..5d92ae2b56df (1 commits) (flutter/engine#16325)
2020-02-01<EMAIL_ADDRESS>Roll src/third_party/skia 1b80c357926b..776293dceefd (2 commits) (flutter/engine#16324)
2020-02-01<EMAIL_ADDRESS>use echo to test whether host is up (flutter/engine#16322)
2020-02-01<EMAIL_ADDRESS>Revert "Modifies accessibility bridge to populate new node fields in semantic… (#15116)" (flutter/engine#16323)
2020-02-01<EMAIL_ADDRESS>Roll fuchsia/sdk/core/mac-amd64 from x6sVj... to jtwQF... (flutter/engine#16320)
2020-02-01<EMAIL_ADDRESS>Roll src/third_party/dart 91a3ea89c8e4..ed441e48d055 (4 commits) (flutter/engine#16319)
2020-02-01<EMAIL_ADDRESS>Remove all uses of the redundant flutter_root variable. (flutter/engine#16311)
2020-02-01<EMAIL_ADDRESS>Roll src/third_party/skia 4513cdde4912..1b80c357926b (1 commits) (flutter/engine#16318)
2020-02-01<EMAIL_ADDRESS>Roll src/third_party/dart 5ae5aff6406a..91a3ea89c8e4 (22 commits) (flutter/engine#16310)
2020-01-31<EMAIL_ADDRESS>Roll fuchsia/sdk/core/linux-amd64 from VMcGV... to AtXwP... (flutter/engine#16308)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md

Trybots failed. These were the failed builds: https://github.com/flutter/flutter/runs/424271652 , https://github.com/flutter/flutter/runs/424271616 , https://github.com/flutter/flutter/runs/424271473 , https://github.com/flutter/flutter/runs/424271451 , https://github.com/flutter/flutter/runs/424271447 , https://github.com/flutter/flutter/runs/424271442

Commit queue failed; closing this roll.


adds faq

@cvan @delapuente
wanna add some questions?

@cvan r?

@cvan good for another r?

Looks good, thanks r+ merging


Bandwidth saving option doesn't persist across logins

It's causing confusion because users will most likely assume that it's about saving bandwidth between OctoPi and cloud. In fact it's about saving the bandwidth between cloud and browser/phone.
Need to come up with a better wording, or a tool tip to explain it.

It turned out to be a misunderstanding on what "bandwidth saving" toggle means. Released a version to clarify it.


Adding cluster version to telemetry elastic

Missing cluster version when posting to elastic search that is printed in the general telemetry
This is a key metric where we will be able to more look at regressions

@tsebastiani is there an easier way for the elastic and telemetry models to match? We'll always want these to be the same

@chaitanyaenr @tsebastiani any final thoughts here? I can maybe update to use the telemetry model instead of having its own elastic model in a future pr. We need this for prow runs to be able to compare the pod recovery time among the different versions

Merging this in favor of evaluating and creating PR for leveraging telemetry model for indexing if it makes sense.


Adding initial instruction for running local soak tests.

Adding initial instruction for running local soak tests.
TODO / IN-PROGRESS:

Automate soak test runs using prow jobs
Make local soak test experience as close to push button as possible
Add the ACK grafana dashboard template that service team's can import.

A great start on the documentation. I have some concerns over the sheer number of manual steps. I understand many of these are limitations of the products, but I imagine a lot of these can be set in helm values or done with scripts that would save hassle.
Also this PR seems to have pushed some .DS_Store files (Mac hidden files) - I'd recommend adding these to your global .gitignore file.

Thank you! Very good suggestions!
Based on the feedback, I am working today to minimize number of copy-paste service team have to do.
Also separating the configuration variables(export commands) into a single place. (soak-test configuration file.)

/lgtm


YouTube/YouTube Music: Wrong album name

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

[X] I understand that I will be blocked if I intentionally remove or skip any mandatory* field

Checklist

[X] I'm reporting that yt-dlp is broken on a supported site
[X] I've verified that I have updated yt-dlp to nightly or master (update instructions)
[X] I've checked that all provided URLs are playable in a browser with the same IP and same login details
[X] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[X] I've searched known issues and the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue
[X] I've read about sharing account credentials and I'm willing to share it if required

Region
Germany
Provide a description that is worded well enough to be understood
For some reason the Album name is wrong:
yt-dlp uses this playlist-link instead of the one I give as an input:
https://www.youtube.com/playlist?list=OLAK5uy_lHa3_q4J0gQo6dJLtjAajASoQgLBjziFI
The Album is called "Back in Black (Live)". But yt-dlp adds the Album name "Back in Black", which is the wrong one.
In the description of the Songs the album name is "Back in Black" but I think, that might be the wrong spot to aquire the albumname. The right one is in that box below the description:

I think that this a bug. Not sure.
yt-dlp -f bestaudio --extract-audio --parse-metadata "release_year:(?s)(?P<meta_date>.+)" --parse-metadata "playlist_index:(?s)(?P<track_number>.+)" --audio-format m4a --embed-metadata --add-metadata --embed-thumbnail --output "%(album)s/%(title)s.%(ext)s" "https://music.youtube.com/playlist?list=OLAK5uy_lHa3_q4J0gQo6dJLtjAajASoQgLBjziFI"

Provide verbose output that clearly demonstrates the problem

[X] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[ ] If using API, add 'verbose': True to YoutubeDL params instead
[X] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
[debug] Command-line config: ['-vU', '-f', 'bestaudio', '--extract-audio', '--parse-metadata', 'release_year:(?s)(?P<meta_date>.+)', '--parse-metadata', 'playlist_index:(?s)(?P<track_number>.+)', '--audio-format', 'm4a', '--embed-metadata', '--add-metadata', '--embed-thumbnail', '--output', '%(album)s/%(title)s.%(ext)s', 'https://music.youtube.com/playlist?list=OLAK5uy_lHa3_q4J0gQo6dJLtjAajASoQgLBjziFI']
[debug] Encodings: locale UTF-8, fs utf-8, pref UTF-8, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version<EMAIL_ADDRESS>from yt-dlp/yt-dlp [12b248ce6]
[debug] Python 3.12.4 (CPython x86_64 64bit) - Linux-6.9.4-201.fsync.fc40.x86_64-x86_64-with-glibc2.39 (OpenSSL 3.3.1 4 Jun 2024, glibc 2.39)
[debug] exe versions: ffmpeg 6.1.1 (setts), ffprobe 6.1.1, rtmpdump 2.4
[debug] Optional libraries: certifi-2024.06.02, mutagen-1.47.0, requests-2.32.3, secretstorage-3.3.3, sqlite3-3.46.0, urllib3-1.26.18
[debug] Proxy map: {}
[debug] Request Handlers: urllib, requests
[debug] Loaded 1820 extractors
[debug] Fetching release info: https://api.github.com/repos/yt-dlp/yt-dlp/releases/latest
Latest version<EMAIL_ADDRESS>from yt-dlp/yt-dlp
yt-dlp is up to date<EMAIL_ADDRESS>from yt-dlp/yt-dlp)
[youtube:tab] Extracting URL: https://music.youtube.com/playlist?list=OLAK5uy_lHa3_q4J0gQo6dJLtjAajASoQgLBjziFI
WARNING: [youtube:tab] YouTube Music is not directly supported. Redirecting to https://www.youtube.com/playlist?list=OLAK5uy_lHa3_q4J0gQo6dJLtjAajASoQgLBjziFI
[youtube:tab] OLAK5uy_lHa3_q4J0gQo6dJLtjAajASoQgLBjziFI: Downloading webpage
[youtube:tab] OLAK5uy_lHa3_q4J0gQo6dJLtjAajASoQgLBjziFI: Redownloading playlist API JSON with unavailable videos
[download] Downloading playlist: Album - Back In Black (Live)
[debug] Skipping writing playlist thumbnail
[youtube:tab] Playlist Album - Back In Black (Live): Downloading 5 items of 5
[download] Downloading item 1 of 5
[youtube] Extracting URL: https://music.youtube.com/watch?v=jvLtXUwvYbk
[youtube] jvLtXUwvYbk: Downloading webpage
[youtube] jvLtXUwvYbk: Downloading ios player API JSON
[youtube] jvLtXUwvYbk: Downloading ios music player API JSON
[youtube] jvLtXUwvYbk: Downloading web music client config
[youtube] jvLtXUwvYbk: Downloading player 84314bef
[youtube] jvLtXUwvYbk: Downloading web music player API JSON
[debug] [youtube] Extracting signature function js_84314bef_111
[debug] Loading youtube-sigfuncs.js_84314bef_111 from cache
[debug] Loading youtube-nsig.84314bef from cache
[debug] [youtube] Decrypted nsig pAgcP1XSbwxJ6KBGRC => Hk3aVZUNp9dozg
[debug] [youtube] Extracting signature function js_84314bef_107
[debug] Loading youtube-sigfuncs.js_84314bef_107 from cache
[debug] Loading youtube-nsig.84314bef from cache
[debug] [youtube] Decrypted nsig 5meu4FFPP7sytle1y2 => CdH36qf6FmNHng
[youtube] jvLtXUwvYbk: Downloading m3u8 information
[youtube] jvLtXUwvYbk: Downloading m3u8 information
[debug] Sort order given by extractor: quality, res, fps, hdr:12, source, vcodec:vp9.2, channels, acodec, lang, proto
[debug] Formats sorted by: hasvid, ie_pref, quality, res, fps, hdr:12(7), source, vcodec:vp9.2(10), channels, acodec, lang, proto, size, br, asr, vext, aext, hasaud, id
[debug] Searching for '(?s)(?P<meta_date>.+)' in '%(release_year)s'
[MetadataParser] Parsed meta_date from '%(release_year)s': 'NA'
[debug] Searching for '(?s)(?P<track_number>.+)' in '%(playlist_index)s'
[MetadataParser] Parsed track_number from '%(playlist_index)s': '1'
[info] jvLtXUwvYbk: Downloading 1 format(s): 251
[info] Downloading video thumbnail 47 ...
[info] Writing video thumbnail 47 to: Back In Black/Back In Black (Live).webp
[download] Back In Black/Back In Black (Live).m4a has already been downloaded
[debug] ffmpeg command line: ffprobe -show_streams 'file:Back In Black/Back In Black (Live).m4a'
[ExtractAudio] Not converting audio Back In Black/Back In Black (Live).m4a; file is already in target format m4a
[Metadata] Adding metadata to "Back In Black/Back In Black (Live).m4a"
[debug] ffmpeg command line: ffmpeg -y -loglevel repeat+info -i 'file:Back In Black/Back In Black (Live).m4a' -map 0 -dn -ignore_unknown -vn -acodec copy -write_id3v1 1 -metadata 'title=Back In Black (Live)' -metadata date=NA -metadata 'description=Provided to YouTube by IIP-DDS

Back In Black (Live) · AC/DC · Brian Johnson

Back In Black

℗ Staten Island

Released on: 2020-03-20

Music  Publisher: Copyright Control

Auto-generated by YouTube.' -metadata 'synopsis=Provided to YouTube by IIP-DDS

Back In Black (Live) · AC/DC · Brian Johnson

Back In Black

℗ Staten Island

Released on: 2020-03-20

Music  Publisher: Copyright Control

Auto-generated by YouTube.' -metadata 'purl=https://www.youtube.com/watch?v=jvLtXUwvYbk' -metadata 'comment=https://www.youtube.com/watch?v=jvLtXUwvYbk' -metadata track=1 -metadata 'artist=AC/DC, Brian Johnson' -metadata 'album=Back In Black' -movflags +faststart 'file:Back In Black/Back In Black (Live).temp.m4a'
[ThumbnailsConvertor] Converting thumbnail "Back In Black/Back In Black (Live).webp" to png
[debug] ffmpeg command line: ffmpeg -y -loglevel repeat+info -f image2 -pattern_type none -i 'file:Back In Black/Back In Black (Live).webp' -update 1 -movflags +faststart 'file:Back In Black/Back In Black (Live).png'
[EmbedThumbnail] mutagen: Adding thumbnail to "Back In Black/Back In Black (Live).m4a"

yt-dlp currently doesn't extract metadata from music.youtube.com, only from youtube.com, so the album title in the description is the best it can do for now.
Duplicate of #622

@bashonly Thanks for the fast answer.

yt-dlp currently doesn't extract metadata from music.youtube.com, only from youtube.com, so the album title in the description is the best it can do for now.

I know that. I also read that and I think that my Issue is not a duplicate. The Info of the album name is also available on YouTube. Watch the screenshot. I don't how yt-dlp works under the hood, but if it takes the album name from that Box below the description it would be right without needing to support music.youtube.com.

I'm geo-blocked, I can't see any of that on youtube.com

I'm geo-blocked, I can't see any of that on youtube.com

Oh. I will check it is a Germany feature only.

I tested some countries and this box seems to be globally available. Just that album with the wrong albumname in the description is not available in all of them.
https://youtu.be/8QX1uBEWtUw?list=OLAK5uy_lHa3_q4J0gQo6dJLtjAajASoQgLBjziFI
Here the autogenerated description is correct. But the box should be there.
You can leave the Issue closed. Maybe that wrong album name in the description was an anomaly.

I can confirm that the correct album is available on the webpage and in the initial player response.
ytInitialPlayerResponse.json

It looks like this album metadata may be newly available; if I'm not mistaken, the ViewModel fields are part of a new YT rollout (the videoAttributeVideoModel field contains what we want here). The question is: how do we determine if we actually want to extract the metadata from the cards and/or have it override the description metadata?

Depends on where the informations for both come from and how old they are.
My guess is, that the video description gets generated once and won't update automatically if metadata changes on YouTube Music. But the Box below might fetch the metadata from YouTube Music so the information is always up to date?
But I really don't know.

Related: #10576, #9913


export: Work top-down from UserMessage

This is just a writeup of an idea that @timabbott and I informally discussed after converting over some large sites to new servers this weekend using our export/import tools for data.  Neither of us are necessarily sold on this new approach yet.
Right now, when you export a realm, but only a subset of the users want to go to the new version or the new server, the following tables are tricky to filter:
Recipient/Stream/Subscription/Huddle
Message/UserMessage
Attachment
Right now our process is something like this:

use your user_id list to calculate Recipient/Stream/Subscription in a mostly straightforward way
calculate the subset of Huddles using somewhat twisty logic to exclude huddles that aren't "pure" in terms of users
calculate the subset of Message with complicated logic to create disjoint queries of messages-sent-by-exported users and messages-received-by-exported users
subset Attachment rows using a set of valid message ids
find UserMessage rows in a straightforward but processing-intensive sort of way from the list of messages

A new scheme would work like this:

Do the process-intensive work up front to generate a list of UserMessage rows that is almost exactly UserMessage.objects.filter(user_profile__in=list_of_user_ids)
Derive the subset of Message rows directly from the list of UserMessage.message ids.
Continue to derive the Attachment rows directly from the list of Message ids.
Follow this path: Message -> Recipient(Personal) -> Subscription(Personal)
Follow this path: Message -> Recipient(Stream) -> Subscription(Stream, with user filter again) -> Stream
Follow this path: Message -> Recipient(Huddle) -> Subscription(Huddle, with user filter again) -> Huddle (but re-hash)

Huddle will still unfortunately be complicated.  Let's say Alice and Bob are in your list of your users but Xavier is not.  The source DB may have huddles for both AB and ABX, but the target system will only have AB, and we may need to somehow re-visit the Message rows in our export that originally had a recipient for an ABX huddle subscription.

Closing this as unlikely to be implemented; the import/export process has been going great and feels well-tested with the current system.


Is the final model in the paper a open source？

Is the final model in the paper a open source？

We have open-sourced the code for our model, but we have not released the pre-trained weights

Great work! Do you have a timeline for releasing the final pre-trained  weights？Thank you!

Thank you for your interest. We anticipate releasing the pre-trained weights for ViT-B or Swin-T next week. The remaining weights will likely be made public after the article is submitted.

Thank you for your interest. We anticipate releasing the pre-trained weights for ViT-B or Swin-T next week. The remaining weights will likely be made public after the article is submitted.

Great JOB！Is the training data RSDataSet used by this model the sum of all open source data sets? Is there self-labeled data? Will this data be open


Current travis integration has limitations

currently only catkin cools (with ROS kinetic) is tested in travis
builds get cancelled after 50 min runtime, i.e. we cannot test both building with catkin and building with cmake due to time constraints.
builds get cancelled after too many printouts. It is difficult to find the right level of verbosity to prevent travis from cancelling while still getting out useful information.

Closed since no better viable option is on the horizon.


Clarify support (or not) for character encodings other than UTF-8

The documentation in the README.md doesn't explain what is xsv's support or policy for character encodings.  I think it really ought to.
Looking through the code for xsv and the csv crate, it looks like there isn't a consistent policy:

Most of the code reads rows with the byte_records() function.
xsv search, however, uses the records() function, which interprets the data as UTF-8.
There are a few places where the code calls str::from_utf8() on byte data.
The select module uses String to represent field names, which is UTF-8.  What happens when you try to xsv select from a file that has Latin-1 field names?

I think my intention was to support text encodings that are "ASCII compatible," which should include Latin-1. For example, in almost all cases from str::from_utf8 is used, there is an actual fallback that runs with just the raw bytes. So there shouldn't be any places where, say, a true latin-1 encoding would be a problem.
Of course, you did pick out a few! In particular:

Field name selection does appear to be limited to utf-8. Fixing that probably means moving the parser to &[u8] instead of &str.
Searching via regex required &str at the time I wrote the code, but we can switch to byte based regexes. (The search pattern must still be UTF-8, but, one can search for arbitrary bytes with hex escapes. That isn't particularly ideal, but does make latin-1 support possible...)

IMO xsv should be UTF-8 first:

supporting other charsets is not really a requirement as you can always convert from anything else to unicode very quickly... but the reverse is not true
all the rust ecosystem is very UTF8-centric for good reasons, and the performance of UTF8 regexes is stellar as you very well know ;-)
latin1 is dying at least on the web

but I guess you had specific motivations for latin1 support?

@eddy-geek I don't really understand what's motivating your comment. CSV itself doesn't have a specified character encoding, and most CSV parsers are written to be ASCII compatible. ASCII compatibility is the goal, and as a result, encodings like latin-1 wind up being supported. This is important because CSV data is often quite messy, and there's nothing worse than failing to read CSV data because of a character encoding issue.
This issue is basically "fix a few places in xsv where UTF-8 is assumed." That's it. Nothing more.

Ok I  see, sorry for the noise
On 12 Dec 2016 5:56 pm, "Andrew Gallant"<EMAIL_ADDRESS>wrote:

@eddy-geek https://github.com/eddy-geek I don't really understand
what's motivating your comment. CSV itself doesn't have a specified
character encoding, and most CSV parsers are written to be ASCII
compatible. ASCII compatibility is the goal, and as a result, encodings
like latin-1 wind up being supported. This is important because CSV data is
often quite messy, and there's nothing worse than failing to read CSV data
because of a character encoding issue.
This issue is basically "fix a few places in xsv where UTF-8 is assumed."
That's it. Nothing more.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/BurntSushi/xsv/issues/42#issuecomment-266485403, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ACpOGRdfgX_9dVE-Ti8T43HGpGKtXvUsks5rHXy4gaJpZM4J9qb5
.

Not supported.


Reusing code in /dask/dask/array/utils.py

I was looking at issue https://github.com/dask/dask/issues/2682 and saw an opportunity to reuse code in this Array util's assert_eq method.
We can move the logic for computed, dt, meta and computed value to a separate method and reuse that for both params a and b.

I have created a PR, please see if this makes sense and looks good.


buffer.expect needs to adapt to the try catch notation

Now there is a section in buffer_test.mbt:
test "expect method with matching content" {
  let buf = Buffer::make(100)
  buf.write_string("Test")
  buf.expect(content="Test")?
}

test "expect method with non-matching content" {
  let buf = Buffer::make(100)
  buf.write_string("Test")
  buf.expect(content="Test")?
}

This is because the -u parameter of moon will update the expected result, while the return value of buffer.expect is still a Result that cannot be thrown.

#609


Wrong max length for Italian phone number

Max length for Italian phone number is set to 13 instead of 10

Yes!! Italy is 10!  @jgeewax  @KANekT @tritao @lingster
Can you fix it?
it's really well done but for my italian app this is a big problem!


Add support for BigAutoField

Hi, apparently using django-postgres-copy with BigAutoFields is not possible at the moment.
I get the error:
type "bigserial" does not exist

One could resort to explicitly setting the field to be an AutoField (or directly using the latter in a per-app or per-project setting).
BigAutoFields are a major thing in Django 3.2 and seeing django-postgres-copy support it without workarounds would be much appreciated.
Thank you.

I think I got this fixed and you should find relief by installing v2.6.0 from PyPI.


Can not run "next start"

Even though I run "next build", I am not able to run "next start".
I can confirm that a .next folder is created, but I get the following error -
"Could not find a valid build in the '.next' directory! Try building your app with 'next build' before starting the server."

Uses "next": "^2.0.0-beta.22"

Can you check if your .next folder has a BUILD_IDfile?

Also, can you provide a sample repository?

Where is the BUILD_ID at?

I won't be able to standup a sample repo currently. Here is a screenshot of the output.

@khrome83 It's located right inside .next (.next/BUILD_ID). From the looks of it this .next folder is from a next dev build. Which is strange because next build should remove the .next folder and then create a new one with the production build. Can you try manually removing .next and running next build again?

Apparently it is not building a .next folder. I deleted the .next folder and confirmed that it was because of dev.
When I run npm build which in turn runs next build it just ends. It does not give me any output.
When I run ./node_modules/.bin/next build then it does produce a production .next folder with a BUILD_ID inside.
This is not a issue only locally on my machine. I can confirm, that on node 6.9.1 this is also happening on the GitLab-CI environment.

@khrome83 try npm run build instead of npm build.

Ok that works locally. It's strange because npm build does not trigger unknown NPM command, and does not trigger unknown script error

Because it's a known command 😂 😂 😂
https://docs.npmjs.com/cli/build
This is really confusing. Anyway closing now.


Why there are only positive outputs?

I think there should be negative outputs as well because of shifting by the median of disparities, but why are there only positive outputs?

During pre-processing, we also normalize the disparities to 0-1 after shifting them with the median value.


[ENHANCEMENT] Add rule system

Add a rule system with a message being send to a channel once and a role.

[ ] Message command
[ ] Verification system (Emoji/Button)

🚧 On hold, see #14.


Update pokeapi example to riverpod v2

Hello, I updated the example using the Pokemon API as it was outdated.
Please let me know if there are any mistakes in the usage of fpdart.

update freezed version
update riverpod v2 and use riverpod_generator
migrate all model to freezed class
migrate RequestStatus to AsyncValue
migrate StateNotifier to AsyncNotifier

@utamori do you think this is ready?


Request for Support: Integration of Chat GLM Model

Issue
I hope you are doing well. I'm just contacting you to ask for the addition of support for a recently released Chinese language model known as Chat GLM. This model has been gaining significant attention in the AI community due to its impressive capabilities.
Version and model info
No response

Thank you for filing this issue.
As long as LiteLLM does not support this model (e.g. it is in the LiteLLM database -> https://github.com/BerriAI/litellm/blob/main/model_prices_and_context_window.json) it will be hard for aider to support this model.
You can set up a model configuration manually, see here -> https://aider.chat/docs/config/adv-model-settings.html

@fry69 It's JSON response is fully compatible with OpenAI. It works just fine with fabric, by just readjusting the base URL to the one of glm

As this issue appears to be resolved, I'm closing it.
If any new related concerns arise, please feel free to comment, and I'll reopen the issue.

@fry69
Is that an automated message?

@fry69 Is that an automated message?

No that is a saved message, so I do not have to type this again and again. Apart from that every action here from me is 100% manual.

@fry69  Okay, but what about what I pointed about (concerning the compatible endponts)?

@fry69 Okay, but what about what I pointed about (concerning the compatible endponts)?

This solves the initial problem, together with my other remarks above, so I closed this issue. Or is there anything else I missed?

If it is OpenAI compatible then it should work fine with aider:
https://aider.chat/docs/llms/openai-compat.html
There is probably no need to mess with advanced model settings.

Advanced model settings may help to set certain parameters which aider otherwise guesses as too low by default, like context window, so this is still an important hint IMHO.

@paul-gauthier
I just tried that solution, and It did work, however, glm has different namings as well for the available models, but besides that, the rest should be the same. AFAIK those models are 'glm-4-0520', 'glm-4', 'glm-4-airx', 'glm-4-air', 'glm-4-flash'. @fry69 Sorry. It went above my head, as I couldn't see a way to modify the base url at the yaml file.

@whyihaveyou  The base URL should first be updated to https://open.bigmodel.cn/api/paas/v4/ with export OPENAI_API_BASE=https://open.bigmodel.cn/api/paas/v4/ because it's not available yet at litllm
with the following .aider.model.metadata.json format. (Feel free to update it and please notify me if so)
{
    "sample_spec": {
        "max_tokens": "set to max_output_tokens if provider specifies it. IF not set to max_tokens provider specifies",
        "max_input_tokens": "max input tokens, if the provider specifies it. if not default to max_tokens",
        "max_output_tokens": "max output tokens, if the provider specifies it. if not default to max_tokens",
        "input_cost_per_token": 0.0000,
        "output_cost_per_token": 0.000,
        "litellm_provider": "one of https://docs.litellm.ai/docs/providers",
        "mode": "one of chat, embedding, completion, image_generation, audio_transcription, audio_speech",
        "supports_function_calling": true,
        "supports_parallel_function_calling": true,
        "supports_vision": true
    },
    "openai/glm-4": {
        "max_tokens": 4096,
        "max_input_tokens": 128000,
        "max_output_tokens": 4096,
        "input_cost_per_token": 0.000005,
        "output_cost_per_token": 0.000015,
        "litellm_provider": "openai",
        "mode": "chat",
        "supports_function_calling": true,
        "supports_parallel_function_calling": true,
        "supports_vision": true
    },
    "openai/glm-4-air": {
        "max_tokens": 4096,
        "max_input_tokens": 128000,
        "max_output_tokens": 4096,
        "input_cost_per_token": 0.000005,
        "output_cost_per_token": 0.000015,
        "litellm_provider": "openai",
        "mode": "chat",
        "supports_function_calling": true,
        "supports_parallel_function_calling": true,
        "supports_vision": true
    },
    "openai/glm-4-airx": {
        "max_tokens": 4096,
        "max_input_tokens": 128000,
        "max_output_tokens": 4096,
        "input_cost_per_token": 0.000005,
        "output_cost_per_token": 0.000015,
        "litellm_provider": "openai",
        "mode": "chat",
        "supports_function_calling": true,
        "supports_parallel_function_calling": true,
        "supports_vision": true
    },
    "openai/glm-4-flash": {
        "max_tokens": 4096,
        "max_input_tokens": 128000,
        "max_output_tokens": 4096,
        "input_cost_per_token": 0.000005,
        "output_cost_per_token": 0.000015,
        "litellm_provider": "openai",
        "mode": "chat",
        "supports_function_calling": true,
        "supports_parallel_function_calling": true,
        "supports_vision": true
    },
    "openai/glm-4-0520": {
        "max_tokens": 4096,
        "max_input_tokens": 128000,
        "max_output_tokens": 4096,
        "input_cost_per_token": 0.000005,
        "output_cost_per_token": 0.000015,
        "litellm_provider": "openai",
        "mode": "chat",
        "supports_function_calling": true,
        "supports_parallel_function_calling": true,
        "supports_vision": true
    }
}


[ALLUXIO-6286] modify core/common/src/main/java/alluxio/PropertyKey.java

https://alluxio.atlassian.net/browse/ALLUXIO-6286

alluxio-bot, check this please

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/17206/
Test PASSed.

alluxio-bot, check this please


How to build jerrystandalone and run test in RPI2

Hi, what your recommended way to build jerry and run test for RPI2?
1, directly git clone codes on RPI2, then make, then run-perf-test
or
2, cross-build the jerry
or
other ways?
I often see the RPI2 memory/performance date posted under some issues or PRs

I'm also interested on this and hope to add a document in 'docs' folder.

and it fails when i directly make release.linux on RPI2
/usr/include/arm-linux-gnueabihf/bits/byteswap.h: In function ‘unsigned int __bswap_32(unsigned int)’:
/usr/include/arm-linux-gnueabihf/bits/byteswap.h:44:31: sorry, unimplemented: Thumb-1 hard-float VFP ABI
 __bswap_32 (unsigned int __bsx)
                               ^
compilation terminated due to -Wfatal-errors.

For cross building, I've committed targets-rpi2 branch so you can at least run in RPi2.
git checkout targets-rpi2
make -f targets/rpi2/Makefile.rpi2

output binary should exist as targets/rpi2/bin/jerryscript file. I think it can be merged after #733 lands.
For performance testing, hope someone will add some comments how to.

@seanshpark Thanks
1,we can remind users to apt-get install gcc-arm-linux-gnueabihf and apt-get install g++-arm-linux-gnueabihffirst
2, should add .PHONY all jerry clean in your makefile, otherwise, it may tell "all is already up-to-date" if user has file jerry in the main folder

@jiangzidong , thanks for the comments, applied to above comment and to git branch. is it ok?

@seanshpark Yes, that works.Thanks!
1,And if we set LTO = ON, then copy the final jerryscript into the RPI2.
2, then use the tools/run-perf-test.sh
the perf data is similar with those posted by @egavrin

@jiangzidong , ok, good. I haven't tested with LTO=ON but seems working :)

LTO=ON is set by default.

@jiangzidong, currently we measure performance on Rpi2 with the following steps:

build engine through make TOOLCHAIN=build/configs/toolchain_linux_armv7l-hf.cmake release.linux -j on a host
run measurement on board through setarch linux32 -R ./tools/run-perf-test.sh ./jerry.1 ./jerry.2 5 500 sunspider-1.0.2


Docker : Publish last hue release

Hi,
Can you update your docker repository with the last version of hue ?
https://hub.docker.com/r/gethue/hue/
Thanks,
Frédéric

Done: https://hub.docker.com/r/gethue/hue/


Update codeowners

👍🏼 🥲

@CarolineLaw Ok I see now, code owners need write permission on the repo, hence the errors. Sorry Caroline 😕, but please get involved in PRs if you want 😀


feat: migrate from packemon to tsup

This PR uses tsup for bundling instead of packemon. This should fix the bundling issues I came across recently.

server components: https://github.com/coinbase/onchainkit/issues/684
esm: https://github.com/coinbase/onchainkit/pull/646

TBH I have no knowledge of packemon so there might be easier solutions, I'm just introducing a tool that does the job fine for me, and maybe it will here too. An objective point though is that tsup is widely used, should be much more battle-tested and have more resources available.
tsup supports:

esm / cjs
declaration types
code splitting
tree-shaking
and more

It also keeps use client directives so this should resolve the issue related to server components. There's a caveat though where code splitting might interfere and not keep directives in chunk files.
Two solutions for that:

disable code splitting
add use client directives where components are exported i.e. index.ts like here https://github.com/radix-ui/primitives/blob/16d6e8409abeacb2b2023ab3db1c0193434f404f/packages/react/avatar/src/index.ts#L1

The second one is better but it will define all components as client components. Generally this is not ideal but this is fine for OnchainKit since everything is done on the client.
Another thing to consider, dev dependencies are inlined when bundling and not dependencies. So if you want OnchainKit to work without users having to install an external lib then you should put it in dev dependencies.

@roushou My preference is to keep using packemon. It's built by one of our past coworker and we use it internally everywhere.
If packemon doesn't work, we can push updates upstream to packemon and the maintainer is very active.
If we need further help on packemon, we can internally pull our packemon experts to help out as well.
Overall, I don't see a huge benefit of replacing packemon with tsup at the moment.

For tree-shaking, we can use knip to remove any unused code. Works super well.

@kyhyco before introducing any breaking changes, please wait my +1

Another thing to consider, dev dependencies are inlined when bundling and not dependencies. So if you want OnchainKit to work without users having to install an external lib then you should put it in dev dependencies.

For clarity, I was referring to the case where tsup is used for bundling. AFAIK other bundlers (esbuild, rollup, ...) work the other way by inlining dependencies instead of dev dependencies.

We should discuss this one further. I too think that viem, wagmi, etc. should be a dev dependencies. There could be a potential conflict between version that the user is using vs what we bundle with.

IMO it would be better to not inline Viem/Wagmi/... but declare them as peer dependencies. Given the wide range of features those libraries cover it might be hard to cover them all so users might still reach out to them and cause conflicts.

Makes sense to still use packemon. Letting this PR open in case you change your mind. Otherwise feel free to close it.

Closing since it's been a while


[Bug]: Feed back loop and citation features do not work both with JS and PY SDK

Language
Javascript/Typescript
Version
latest
Description
For JS SDK, feed back loop and citation features do not work by following the CONCEPTS doc.
For PY SDK, no docs can follow. By following similar steps as JS doc shows, these two features do not work either.
Reproduction Steps
1.
2.
3.
...

@frankqianms these features are not yet on public ring - are you using R4? It should be available within a week or two on all rings. Users would need to be using at least developer preview or lower rings.
I have tested this on JS yesterday and things are working on R0.
Otherwise, more information on what problems you are running into would be helpful

@frankqianms please provide the raw request and response of the bot activity so that we can take a look.

@corinagum Thanks for your remind. After switching to R0, feedback loop worked well both on JS and PY. But for citations, it didn't work for me. I wonder if it was related to this issue: https://github.com/microsoft/teams-ai/issues/1590.
I tested these using sample g.datasource-azureAISearch after deleting this line.


Add "Scroll to see reports" button

The form is gelling longer and I'm afraid visitors won't notice there is a repots to read at the bottom.
Adding something like ....
Share your story in the form below or  [check submitted reports](link to scroll down)

would be nice.
I'm also open to any layout change on the site if visitors can look at reports in 1st page view.

What if we made it so "Share Your Experience" form was collapsed on page load? Depending on the user's browser viewport size, that would bring the reports up higher but I'm not sure if it'll get the reports up where user's will see them on the initial page load.

yey thanks @droidenator :)
merged in #34 (sorry for delay in getting back !)


Improve etcd usability

Main goal is to add "join" command for etcd daemon:
user@mynewnode: $ etcd2 --join-cluster %existing_cluster_node_ip%

Second goal is create web interface where you can:

join existing cluster
create new cluster
if you run cluster, add node to your existing cluster

As an example you can review this couchbase doc: http://docs.couchbase.com/admin/admin/Tasks/rebalance-adding-nodes.html
p.s. din't have time to think more on implementation details
/cc @robszumski @yichengq

@kayrus Sorry we are not going to support this. See https://github.com/coreos/etcd/blob/master/Documentation/runtime-reconf-design.md#two-phase-config-changes-keep-you-safe.
We are aware of this design and intentionally do not support it.

I think first goal is not good, and current workflow is what we learn after interacting with users.
The web interface one is a good one, and it could make users to monitor and update the cluster easily.


$tree->path() does not travese

Should $tree->path() be able to traverse a path such as "subdir1/README".
ex. $tree->path('subdir1/README');
The example currently returns null yet the directory and file exists.

yap, some php-git API should be more easily to use.
but libgit2 does not support convenience features and php-git become large extension.
i want to fix these issues near the future.
please obtain each tree object resolve path at this moment.


generic table is not working inside of ng-template

this is my code:
        <ng-template *ngIf="true">

                     <generic-table #table1 [gtSettings]="gt.settings" [gtClasses]="'small'" [gtInfo]="gt.info" [gtOptions]="gtOptions" [gtFields]="gt.fields" [(gtData)]="gt.data" (gtEvent)="trigger($event)"></generic-table>

                 <gt-pagination [gtClasses]="'justify-content-center'" [genericTable]="table1"></gt-pagination>
         </ng-template>
When i remove ng-template, it's working... Is this bug ?

It might be, because it should work with ng-template so if it doesn’t we should fix it I think. I don’t have access to a computer at the moment and won’t have access to one for another week I’m afraid, but it might have something to do with the object references not being updated/set correctly when using ng-template. You could try skipping the gt object and just refer directly to settings, fields and data or declare them at the same time true becomes true  not sure it works but I think I’ve come across this before at least.

Thanks for your answer, It's not a problem for my situation. I used ngIf in my Div tag and fixed the issue. But i just wanted to tell if its really a bug or not. :)


feat: add support for version 2<IP_ADDRESS>.84

Sorry i will not add for absolutely each version of instagram a supported version, otherwise it will be horrible in app haha

I think it would be for the best if you'd allow custom class names to be hooked because I'm in the alpha channel and it gets updates literally twice a day.

He already done that, please use the second version...

Ah, I see that the update didn't get through the LSPosed repo. Thanks for the tip @ReSo7200!


modify build and serve console output

Modified chalk over console log as it seemed less obtrusive (webpack has a dependency on chalk but I could not find any uses).

The path of the CSS file isn't as expected.