GRL - instancing support

Please make sure to label your PR with "bug", "new feature" or "breaking change" label(s).
To prevent this PR from going to the changelog marked it with the "skip changelog" label.

Snapshot stored with reference name:
refs/pull/14460/merge
Test environment:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/14460/merge/index.html
To test a playground add it to the URL, for example:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/14460/merge/index.html#WGZLGJ#4600
Links to test babylon tools with this snapshot:
https://playground.babylonjs.com/?snapshot=refs/pull/14460/merge
https://sandbox.babylonjs.com/?snapshot=refs/pull/14460/merge
https://gui.babylonjs.com/?snapshot=refs/pull/14460/merge
https://nme.babylonjs.com/?snapshot=refs/pull/14460/merge
To test the snapshot in the playground with a playground ID add it after the snapshot query string:
https://playground.babylonjs.com/?snapshot=refs/pull/14460/merge#BCU1XR#0

@Popov72
Is this correct? Can I drop these three uniorms?
                uniforms: [
                    // "worldViewProjection",
                    // "worldView",
                    // "view",
                    "world",
                    "viewProjection",
                    "projection",

Yes (see my answer in the comments).

Visualization tests for webgl1 have failed. If some tests failed because the snapshots do not match, the report can be found at
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/14460/merge/testResults/webgl1/index.html
If tests were successful afterwards, this report might not be available anymore.

Visualization tests for webgl2 have failed. If some tests failed because the snapshots do not match, the report can be found at
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/14460/merge/testResults/webgl2/index.html
If tests were successful afterwards, this report might not be available anymore.

Visualization tests for webgl1 have failed. If some tests failed because the snapshots do not match, the report can be found at
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/14460/merge/testResults/webgl1/index.html
If tests were successful afterwards, this report might not be available anymore.

Visualization tests for webgl1 have failed. If some tests failed because the snapshots do not match, the report can be found at
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/14460/merge/testResults/webgl1/index.html
If tests were successful afterwards, this report might not be available anymore.


Automated cherry pick of #5784

Cherry pick of #5784 on release-5.25.

#5784: update error handling from server

/cc  @sbishel

Will try to auto merge this PR once all tests and checks are passing. This might take up to an hour.

Trying to auto merge this PR.

Pull Request successfully merged
SHA: 3d077991a12ffdbb52b25b2edac8f05c027b060e


Nested Visitors to Reduce Node State

We should investigate using nested visitors to reduce the state being attached to JSX Elements and the transformed iDOM Calls.
Implementing the main visitor to just look for root JSX elements, then launching a nested visitor for everything in that function scope.
Re: https://github.com/babel-plugins/babel-plugin-incremental-dom/pull/28#issuecomment-145404028, https://github.com/babel-plugins/babel-plugin-incremental-dom/pull/28#issuecomment-145604800

Closed by #33.


Refactor conditions to enable shortcuts

Fixes #132

@banacorn Would it be possible to please merge this PR? Or alternatively, could you give me (or someone else within the Agda dev team) permission to merge PRs here?

Hello, I also had trouble with this behaviour. I took the liberty of publishing an extension with this fix, because otherwise it can be difficult to install in remote vscode.
It is available here: https://marketplace.visualstudio.com/items?itemName=GuilhermeEspada.agda-mode-fork
Let me know if there is any other things I should merge into it.
Espada

I will contact him privately. Please hold for a moment.

I am able to merge this PR now (@jespercockx may also have received the invitation to own this repo as well), but I am not familiar with VS code extension development.
Is there anything I need to do after merging this PR to update this extension?

I would follow this procedure:

Merge to master
Wait for CI to succeed
Tag master with v0.3.12 and push the tag
Wait for CI to publish

Looking at https://github.com/banacorn/agda-mode-vscode/blob/master/.github/workflows/release.yml that's all it seems to need

@GUIpsp what did you mean by removing the SVG from the readme..?

MS banned svg images from their readmes out of security concerns. This may be project/tool specific, but I had to remove them to publish the fork. Look here: https://github.com/GUIpsp/agda-mode-vscode/commit/a65d0975f1b347f0d6d8627027512a3e693c4e51#diff-b335630551682c19a781afebcf4d07bf978fb1f8ac04c6bf87428ed5106870f5L3

Right, thanks for letting me know!

Looks like we'll need @banacorn to refresh the Personal Access Token to the vscode marketplace.
Also, you might want to bump publish-vscode-extension to v1 to get rid of this warning.

Is it viable to move this to the Agda organization? It's probably good we don't burden @banacorn with extra maintenance burden.

I would be very happy to move this into the Agda organization! Do you know if it's possible to move a repository on github or do we need to create a new repo?

https://github.com/banacorn/agda-mode-vscode/transfer

I will discuss this with @banacorn when he is available and see how to proceed.

This is still not published.

0.3.12 should now be published. I now have permission to manage the extension and will try to maintain it on behalf of @banacorn at least before we officially transfer this extension to the Agda development team.


[Feature] Self-hosted CodeLlama LLM for code autocompletion

Summary
This pr provides a solution to use CodeLlama for self-hosted code autocompletion.

A standalone LLM is required to run separately and expose a RESTful API to access. The most straight-forward way is to use llama.cpp, in which you can host the LLM either on a Mac laptop or a GPU-installed machine.

Please follow the instructions of llama.cpp to run the LLM locally or in a cloud, also feel free to let me know if a detailed tutorial is needed.

A Sidebar setting is added to enable/disable the copilot
tRPC seems not very well support multiple providers in React, ref, thus I put the Route in api/src/server, initially we talked about to create an standalone server service to the copilot-related services.
There are a couple of follow-up patches to add, e.g., vite bug fix, monitoring the copilot service connection status, similar to the sync statue in the top of Sidebar, adding infilling mode in addition to autocomplete mode. This PR also aims to collect some early feedback regarding the overall design and architecture.

Test

First, enable the RESTful API on the llama.cpp, assuming the IP address is x.x.x.x, port is 9090
On the local machine, open the terminal,

cd codepod/api/
pnpm dev --copilotIP x.x.x.x --copilotPort 9090

Note that, the screenshot below intends to the demonstrate the functionality, the quality of the auto-completion might be low due to the 4-bit quantized llama-7b model.

tRPC seems not very well support multiple providers in React, ref,

I also came across multiple providers the other day, and it is well supported: trpc#3049. I have implemented multiple providers in https://github.com/codepod-io/codepod-cloud/pull/11

But it could actually be better and simpler to leave it in api/ routers, so that the frontend always has one API to talk to. We can let the api/ forward the request to the actual LLM service internally through tRPC or gRPC.

Also, there's an uncaught exception in the console for canceled API call. I'd like to catch it and display a canceling message to keep the console clean.

monitoring the copilot service connection status

This isn't that critical. We can assume that the service is up.

adding infilling mode in addition to autocomplete mode

This is quite important. It is quite often that we edit code in the middle.

After the discussion, we decide to leave this PR as a reference to integrate the self-hosted copilot and address the comments in the codepod-cloud repo.


:seedling: Backport container build image wf to release-1.7

This backports https://github.com/metal3-io/ip-address-manager/pull/789 to release-1.7 branch

/cc @tuminoid @adilGhaffarDev

/override metal3-ubuntu-e2e-integration-test-release-1-7


Private Forks

Hi,
I have this idea for a while and i thought that it might be a good fit for Coolify,
I will try try to explain it with an example,
An awesome app called audiobookshelf
Maybe i would love to make fundamental changes to ui or something else but the owner / community doesn't agree, if I fork the repo it will always forces it to be public and if I clone it and republish the repo privately it will be a hassle to manage the updates ( especially for multiple projects ) as explained is this Gist :
https://gist.github.com/0xjac/85097472043b697ab57ba1b1c7530274
So what i'm suggesting is the possibility to manage something like this,
You just provide the repo you want to clone, coolify copies it into your profile privately and whenever there is a new update from the orignal repo coolify magically notifies you and you have the ability to push the new update to the private repo, and if there is a conflict it creates a new branch etc ...
What do you guys think ?

repo linking and CI CD can be done in coolify. the rest of the things like syncing up repo with upstream and tracking conflicts should be done manually. I think that's the standard. I have been doing this for several open source products. But when it comes to syncing and updating my fork I'd never do it automatically because I don't know what is coming in updates and in the case of enterprise where there are live users this is never advisable.
I believe it will complexify the flow in which coolify currently setting up projects. If you can do it with simple toggle button in setting or something like it then it would be great. Feel free to create a PR or to discuss further here.


Determine Disk Type

Currently udevadm does not detect device tags in docker.
Workaround required.

Possible alternatives:
cdrkit
cdrecord


Packit downstream automation workshop: Brno/EMEA online edition

Let's do an online workshop about downstream automation using Packit.

[x] Pick the date at least a month ahead. (Agree with the team and people needed to run the workshop.)
[x] Create a calendar invite and book the resources (e.g. a room) if needed.
[x] Prepare a sign-up sheet. (limit the number of people to something manageable like 20, promise repetition if full)
[x] Send the essential information to the relevant channels, in this case:

[x] Brno/EMEA mailing lists
[x] Brno/EMEA Google rooms
[x] #fedora Slack channel
[x] #team-packit Slack channel

[x] Clone and update the slide deck from the previous run.
[x] Prepare the environment.
[x] Send a calendar invite to the attendees (ideally, once they sign up).
[x] Send reminder a week ago

[x] To the attendees with the instructions on how to prepare.
[x] To the channels with the announcement if we have a free capacity.

[x] Agree about the responsibilities within a team.
[x] Run the workshop:

[x] Provide a welcoming space for people to ask for help.
[x] Don't forget about recording! not applicable to Gather

[x] Send a feedback form after the meeting.
[x] Send a recording and info about the next runs.

Happened on October 25.


Custom class name

Original question:
Hi! How can i set my custom class to table to tr tag. I what to make possible to change background of custom rows.
Acceptance criteria:

to each design system component add a css class with admin-bro_ prefix like admin-bro_Button etc.

classes are now handled by styled-components and are generated automatically. I will change the description of this task to add some custom class names to all design-system components

after this being done you will be able to use regular CSS selectors to change particular elements.

until then you can try to inject your custom css: https://softwarebrothers.github.io/admin-bro-dev/AdminBroOptions.html#assets and select those elements via css selectors which don't require class names (you can utilise data-testid css property)

closed in verson 2.1.0-beta.7

@wojtek-krysiak how do we inject the css file for custom css?
I tried this:
  const adminBro = new AdminBro({
    assets: {
      styles: [AdminBro.bundle('./index.css')],
    },
    ....
  }

but not working

Nevermind figured it out using this answer: https://stackoverflow.com/a/58046155/4297337


[ios] Notifications only work when complied on specific laptop

For some reason, devices only receive messages when i compile on my laptop. When compiled on coworker's laptop the messages never go though.
I've tried downloading and installing the production and development APNs certificate, swapping out his .entitlements file with mine, uninstalling, cleaning xcode, and running the app again on both an ipod Touch and Iphone 7. Every time it works on mine but will not work on his. Our code is the same on Git, so i'm pretty certain it's an Xcode issue. We just restarted and i replaced his GoogleServiceInfo.plist and Info.plist with mine and i'll update if that fixes it, but i'm not holding my breath :(
Package info:
"react": "^16.0.0-alpha.12",
"react-native": "^0.45.1",
"react-native-fcm": "^6.2.3",

It worked after just giving him my entire iOS folder. I can look more into the solution if anyone really needs it in the future


Support Github enterprise

Is your feature request related to a problem? Please describe.
I want to use this tool on Github enterprise, but this tool connect API URL didn't work on Github enterprise,
Github

API: https://api.github.com
GraphQL endpoint: https://api.github.com/graphql

Github enterprise

API: https://[hostname]/api/v3
GraphQL endpoint: https://[hostname]/api/graphql

Describe the solution you'd like
Add API URL and GraghQL URL option on this tool
Describe alternatives you've considered
N/A

Hi @bcoe,
yes, I can do it.
I uploaded a screenshot of the execution process on #1083


Provide config option to define default sort field and order

Right now, OBJECTID is used to sort the table and is always shown in the table.
OBJECTID is a field that makes sense to the database but doesn't provide any useful context to the end users. Users may want to use another field to sort their data, such as the Request ID field below.
In that case, users may not want to to see the OBJECTID field at all, and have a preset sorting set on the Request ID field

cc @azizaparveen @chris-fox

@avezina, we definitely have the ability to configure which fields display in the table as something in the backlog for the app. This is an important requirement that we had previously in Crowdsource Manager.
With regard to setting a default sort field with the app loads, our initial thought was to have it behave like it did previously in Crowdsource Manager where the default sort order was OBJECTID descending. This was done to always have the most recent created features display at the top of the table. Users could still change the sort order at runtime, but the reverse sort on OBJECTID worked well for most cases and I am not aware of any user feedback requesting the ability to specify a different default sort order.
With this in mind I am inclined to have this same behavior for beta and wait for user feedback on if they require a different default sort order per layer.

Looked at the possibility of sorting based on OID as well as hiding the OID field and it does not appear that we can sort on a hidden field. The only way I saw it work was to add a small delay after the call to sort that would then hide the OID field. But if the OID field was marked not visible when creating the instance of the table or even immediately after the call to sort it would not work.

We are providing config option for field order in R1 release, next release we will work on the default sort field.

add the objectid at the end of the table, and can we sort based on any field, add toggle for sort button

@chris-fox @avezina, in today's meeting, we discussed about showing the sort button on hover, I am wondering if we can show the sort button always and mimic the table behavior. The table shows ascending and descending button by default,  once clicked it shows ascending icon at first, change the color of the icon and text to blue that gives a clear indication that particular field is used for sorting the table, clicked once more, it shows the descending arrow with text and icon color in blue, only thing I don't like is the icon but for consistency I am open to use the same icon.

@azizaparveen Makes sense to me. Not sure there would be a Default view though? There would always have to be a field defined for sorting - probably ObjectID by default?
Other icon options  https://developers.arcgis.com/calcite-design-system/icons/?query=sort

@avezina we now sort the table by ObjectID by default.
These icons are also good alternatives, will send these to Lucy to get her opinion

logged an issue in Template config repo
https://devtopia.esri.com/WebGIS/arcgis-template-configuration/issues/4447

IA team suggested that we use dropdown to select default field and sort order, their concern with the our approach is the lack of clarity in indicating to users that clicking on the sorting icons establishes the default sorting field. Here is their suggestion.

@azizaparveen
This needs changes in Instant app for configuration, once those chanes are implemenetd we can implement it in the component
@jmhauck - please confirm

@sumitzarkar correct this will require changes from IA team in the repo listed above.

@sumit, this is installed in devext, Interactivity tab ->Table Options -> clicking on the Fields button

Looks like this should already come through with mapInfo.layerOptions but I will confirm.

@azizaparveen @avezina
Dev Checklist

Items
Is required
Status
Comments

Required NLS changes?
No

Required Backward compatibility
Yes

Required 508
No

Need Sanitize?
No

Impact Analysis Report

Impacted Areas
Comments

Have Configuration changes?
No

Have Runtime changes?
Yes

Impact on RTL
No

Impact at runtime sorting feature functionality
Features in the layer table should be sorted based on the configured sort field and sorting order

For backward compatibility, by default features should be sort by OBJECTID field in descending order

@jmhauck
https://github.com/Esri/solutions-components/issues/308#issuecomment-2085812344

Correct, we are getting this info through mapInfo. we don't need seperate props on manager component.
So we have removed this props from manager and updated the layer-tables to use the mapinfo to fetch the configured sortOrder and sortField

We will deliver this in the next code drop

@sumitzarkar sorting is not working in the config preview but it works as expected at the runtime.

@jmhauck

sorting is not working in the config preview but it works as expected at the runtime.

Please let us know if any changes required for this in manager component

I was having issues running the config app locally so I couldn't fully confirm but I did add a call to sort with mapInfo changes in layer-table.

It looks good in devext.


On will destroy event set object/array refs to null

Uses willDestroy hook to set complex properties to null

We were debugging a memory leak in our Acceptance tests and are curious about removing references on teardown. The complex objects (not primitives, Objects and Arrays) have references to various other objects. The thought here is that it may be good practice to remove any references between objects in the store and the store itself when the application is torn down between tests.

@pixelhandler sounds good - if you slap a quick test in here I'll merge it up and publish

@toranb I added a test :)

@toranb without the addition of the willDestroy the destroyed object still has properties with references to other objects, e.g. objects in the store. (That was my curiosity regarding breaking the references, to be sure the clean up leaves no references to other objects.) Perhaps this is overly cautious but does not hurt anything, attempting to clean up after itself between tests.

@pixelhandler this is now included in the v5.2.0 release
thanks for taking the time to update this and submit!!!!

@toranb you're welcome :)


Adding LastInsertId for Go

#1459 adds support for sql.Result.LastInsertId()
:execlastinsertid is calls .LastInsertId() rather than .RowsAffected() with :execrows.
This will avoid the sql.Result database package leaking across the boundary of the pkg.Querier interface generated by sqlc when using MySQL & MariaDB

Can you share more about why :execrows was a mistake?

In this case, you're right that it's not much more work to support. Also, there are only two methods on sql.Result, so we aren't going to have to add any more in the future.

I tried to add the python generation example, but when I changed into the internal/endtoend/testdata/exec_lastid/python_postgresql folder and run it against my local sqlc it complained:
$ ../../../../../sqlc2 generate
> error parsing sqlc.json: unknown target langauge "python"

I was using the same code from the exec_rows example.
I also wasn't sure what the PGX (go_postgresql_pgx) should be since this isn't something for PostgreSQL - so I left it.

It's okay that you weren't able to add the Python support, can do that in a subsequent PR.
You're also right that we don't need to add support for pgx, as it's Result type does not have this method.


[Frontend][MXNet] Add support for MXNet GroupNorm

Supports the GroupNorm op in mxnet frontend.
Since MXNet's groupnorm implementation is bugged when num_groups != num_channels, I disabled a few unit tests for now. MXNet fixed the bugs in MXNet 2.0 https://github.com/apache/incubator-mxnet/pull/18199

@junrushao1994 @kevinthesun @icemelon9 Could you please review?

CC: @sxjscience if you would like a second look. Thanks!

This looks good to me.


[webview_flutter_wkwebview] Implement one callback method for review of the design

Implementation of WKNavigationDelegate.didFinishNavigation. This changes the callback to be required at instantiation.
The FlutterApi for NSObject was also created to show how inheritance will be handled.

Since Objective-C uses reference counting, extra steps had to be taken to avoid circular references. In Objective-C, the addition of the FlutterApi lead to a circular reference as shown in the diagram below. S arrows means an instance holds a strong reference to the instance it points to. W means the equivalent, but with a weak reference.
NavigationDelegate--S-->FlutterApi--S-->BinaryMessenger--S-->HostApi--W-->InstanceManager
^-----------------------------------------------S------------------------------------------/
As shown above, HostApi with classes containing callback methods will hold only a weak reference to InstanceManager to break the circular reference.
I included a test to ensure the circular reference is broken and verified the circular reference exists if HostApi had a strong reference to InstanceManager.
Unlike Android, the iOS engine doesn't actually keep a direct reference to any plugins that are registered. So, a strong reference InstanceManager can be stored in an empty MethodChannel implementation passed to the registrar.
No version change:
Part of https://github.com/flutter/flutter/issues/93732 and doesn't make any changes to the current implementation.
No CHANGELOG change: Incremental unused code doesn't need to be noted in the CHANGELOG.
Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the relevant style guides and ran the auto-formatter. (Unlike the flutter/flutter repo, the flutter/plugins repo does use dart format.)
[x] I signed the CLA.
[x] The title of the PR starts with the name of the plugin surrounded by square brackets, e.g. [shared_preferences]
[x] I listed at least one issue that this PR fixes in the description above.
[ ] I updated pubspec.yaml with an appropriate new version according to the pub versioning philosophy, or this PR is exempt from version changes.
[ ] I updated CHANGELOG.md to add a description of the change, following repository CHANGELOG style.
[x] I updated/added relevant documentation (doc comments with ///).
[x] I added new tests to check the change I am making, or this PR is test-exempt.
[x] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

Note: https://github.com/flutter/flutter/issues/103644

At the cost of making the PR larger, I went ahead and added the final implementations of InstanceManager for Dart and Objective-C and updated the tests for them. The WeakReference and Finalizer usage in the Dart InstanceManager were also commented out since they still require Dart version 2.17. And unfortunately, the releasing of Dart WeakReferences and weak references in Objective-C NSMapTables are non-deterministic and I wasn't sure how to test them. I did however write tests that the strong references are released.
The WKNavigationDelegate  serves as the standard that all the other classes will be formatted to in the followup PR. Note the .detached constructor and copy() method.
The InstanceManager change could probably be its own PR, but I wanted to show what everything would look like together.


ERROR (ArgumentError): Unknown boot_config subcommand clear

kamal proxy boot_config clear
ERROR (ArgumentError): Unknown boot_config subcommand clear

kamal proxy boot_config reset

https://github.com/basecamp/kamal/blob/7b48648bf2a380bfb44f89d357cf5ec975c7b18b/lib/kamal/cli/proxy.rb#L47-L50

Interesting. In the Kamal documentation, it's correct, but in the code it's not.

https://kamal-deploy.org/docs/commands/proxy/

Once this PR is merged, and documentation is updated, it should be fine.


Discussion: ox as Formula

Hi,
I recently tried to install  ox and was wondering why it wasn't in homebrew/core. Now I first wanted to start a discussion whether or not this is something we'd want to have in core or if the tool is too old/outdated to put in here.
I'd be happy to open a PR if it's something we'd want to have!
Thanks in advance,
Daniel

It seems like it may be too outdated but if it compiles and installs without patching or inreplace we may accept it anyway. Thanks!


Use max AF - max of variant.frequency - for variants display.

Pop_freq_max in the annotations files - or from the variant adaptor - would be faster, but this is kind of fun. Also use ExAC_MAF from VEP vcf if available.

:+1:  been thinking about to change this for a while...


fix(md-tabs): reset click event in refresh method to allow dynamically added tabs

to fire the custom tabSelected event

Thanks. 👍
Can you change the commit message to use our commit message convention, please? It's used to generate the changelog. 😃
Although I could add to the changelog manually this time.

That's what I get for hurrying :)  Does editing the PR commit do it or do I need to submit another PR with correct verbiage?

I'm not sure, tbh. Maybe there is a way in Github/git to change it..
It's possible before pushing to change the commit message.
You did it while I was writing. 😄

Yes, it wasn't as painful as I thought

How did you do it?

https://help.github.com/articles/changing-a-commit-message/#rewriting-the-most-recent-commit-message
git commit --amend
git push --force

Awesome, thanks! 👍
I will save the link for later (and I hope I don't forget that I saved it -,-).


500 Error after ListAllNotes section

Hi,
So far everything has ran well and passed all the tests. I could upload my files as well and I see them in my DynamoDB. However, I am currently getting a 500 error when I load my page after implementing the renderNotesList function in Home.js. When inspecting and clicking on /prod url, it says ""error": "One or more parameter values were invalid: Condition parameter type does not match schema type"". I looked through the discourse and the only thing I could find was to check DynamoDB to make sure I have the correct attribute names, which I do. Any help is appreciated,
Thanks.

Instead of manually creating my DB, I redeployed my stack but used the DynamoDB yml file under resources and it fixed it.


Low cpu usage when parallel tuning

Hi, first of all thanks for Your kindly reply and support.
I got a low cpu uasge wen doing parallel tuning with blendsearch. My code and parameters as below:
from flaml import AutoML
automl = AutoML()
automl.fit(X_train, y_train,
        task="classification",
        estimator_list=["lgbm"],
        metric='roc_auc',
        time_budget=600,
        n_jobs=8,
        n_concurrent_trials=2,
        hpo_method='bs')

Here , X_train is a ndarray with size [20000000, 39] and dtype 'float64' ,  the y_train's  dtype is 'int32'.
The task is executed in a container with 16CPU, 180G memory resources
This got a very low cpu uasge, only around 200%, not the expected 1600%. So the training trails are vary slow.
Supplementary Note：
During trails running, the following warning occurred:
WARNING trial_runner.py:886 -- Trial Runner checkpointing failed: cannot serialize a bytes object larger than 4 GiB
when I cut the size of X_train to [10000000, 39], everything looks normal, the cpu uasge reached close to 1600%.
It seems to be related to the data size of X_train, for
20000000 x 39 x 8/(1024^3) \approx 5.8GiB > 4 GiB
10000000 x 39 x 8/(1024^3) \approx 2.9GiB < 4 GiB
Please could You kindly help me, Thanks!

@Yard1 This seems an issue caused by ray. Do you have an idea?

Can you point me to where you are specifying how to pass the data to Tune? @sonichi

Can you point me to where you are specifying how to pass the data to Tune? @sonichi
It appears that the data is serialized in the checkpoint, which shouldn't happen unless it's passed as a part of the config, or as a part of the reported metrics. Using with_parameters should not cause this behavior. This may also be caused if the searcher, scheduler or trial runner have the data as an attribute, as the entire object will be pickled on checkpoint. I'd imagine the low CPU utilization is due to serialization overheads of the data being passed, which causes the trial runner loop to get bottlenecked.

Here it is: https://github.com/microsoft/FLAML/blob/b78d0b57c498ab852aec7f8d30bc782d7914ff69/flaml/automl/automl.py#L2144

@shufan007 It it possible for you to provide a fully self contained reproducible example?

@shufan007 It it possible for you to provide a fully self contained reproducible example?

I put the example here: https://github.com/shufan007/flaml_example
Thanks!

@Yard1 , @sonichi
This behavior is probably to be caused by here:
https://github.com/microsoft/FLAML/blob/90aea9c28b6100faf86f2c204e53a68e87f10c66/flaml/automl/automl.py#L3128
Data can be passed to ‘search_alg’ through ‘self._state’ as as an attribute.
I did a test, when set 'config_constraints=None', the problems mentioned above do not exist.

Thanks! I'll be looking at this myself later today, this is very helpful.

@shufan007 PR to fix this issue is up ^

Ok, thank you!


Another couple of improvements for the APIM release process

Description

use the same settings.xml file in every job
reformat and standardize all maven command
remove useless code
reuse existing commands
re-implement nexus-staging job and remove the dependency to gravitee orb

📚  View the storybook of this branch here

🚀 CI was able to deploy the build of this PR, so you can now try it directly here
Notes: The deployed app is linked to the management API of the Element Zero team's environment.

Closing this because most of the important things have been done with  https://github.com/gravitee-io/gravitee-api-management/pull/1509


Change create wallet flow, to achieve state full steps

On the current flow, when a user is creating a Wallet and the pipeline is broken for wherever reason (refresh, etc), the creation flow needs to be restarted.
The possible improvement to do this is to re-order the flow;

Ask the user for a password
Show the seed phrase
2.1 Store the encrypted data on storage
If the users refresh or anything happen
3.1 Show the password unlock page
Show seed phrase
Finish flow

This issue is being closed due to inactivity and can be recreated if necessary. This action is been done to ensure the correct prioritization of issues.


⚠ Roblox Devforum has degraded performance

In 9266406, Roblox Devforum (https://devforum.roblox.com) experienced degraded performance:

HTTP code: 200
Response time: 25316 ms

Resolved: Roblox Devforum performance has improved in 12553d2 after 31 minutes.


Fix erroneous upload handling

Second try, now with unit test, and target "develop": Fix https://github.com/zendframework/zend-psr7bridge/issues/21.

Could you rebase so only your commits shows in this PR, you can do with the following steps:

Ensure the upstream/origin set

git remote add upstream https://github.com/zendframework/zend-psr7bridge.git
git remote add origin https://github.com/Pittiplatsch/zend-psr7bridge

Process rebase

git checkout develop && git pull upstream develop
git checkout fix_erroneous_upload_handling && git rebase develop
git push --force origin fix_erroneous_upload_handling

Done.
Thx for detailed instructions 👍

@Pittiplatsch thanks for your contribution!

You're welcome :-)


Self closing tag whitelist vs blacklist

First of all, this is awesome. It lets me work on my electron app with jsx-like code without having any build step. It's great!
I'm just curious as to why self closing tags are whitelisted.
Because right now when I'm a bit lazy (or rushed) I just want to write
 when I know it won't have any children, but then it breaks the dom by not closing it. The fix is easy of course, just close it like

 but it still gets me every time.
I'm sure there's a reason, obviously there is a nice opportunity to be able to validate the code, so we don't try to put children into a  etc.
Another case which I'm currently playing around with an idea in my app to have custom component tags by registering the components (as opposed to the suggestion in #9) and replacing tagName with it in the rendering function, like this:
// hx.js
const {h} = require('preact')
const render = hyperx(hr)
render.register = register
module.exports = render
const components = {}
function hr(tagName, attrs, children) {
  return h(components[tagName] || tagName, attrs, children)
}
function register(component) {
  components[component.name] = component
  return component
}

// Component.js
const hx = require('./hx')
const Component =({onClick}) => hx`<div onClick=${props.onClick} />` // <-- this i would like to be able to "self close" like this
register(Component)

// Parent.js
const hx = require('./hx')
const Parent = props => hx`<Component onClick=${props.onClick} />` // <-- this i would like to be able to "self close" like this

The bad part about this right now it that I have to require all the components somewhere, not necessarily where they're used (unlike when using it like in #9 or in React).
If there's some interest in flipping the whitelist to a blacklist instead (not allowing children for elements which don't support children) I'd love to give a PR a try.

When writing some tests for this I realised I've mistaken the issue, it does not seem to be related to a whitelist of self closing tags. But instead is seems to be an issue with multiple self closing elements.

Closing this for #34


ci: Update for main and Go 1.18

Minor updates to the CI, and bumps the supported Go version to 1.18.
Also fixes some minor lints.

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@0b5598b). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #150   +/-   ##
=======================================
  Coverage        ?   52.70%
=======================================
  Files           ?       16
  Lines           ?     1958
  Branches        ?        0
=======================================
  Hits            ?     1032
  Misses          ?      825
  Partials        ?      101


lint: enable export documentation check

With the revive linter replacing golint within golangci-lint, the check for exports having documentation has been disabled by default.
This should be re-enabled as all exports should have documentation and reviews are having to check for this unnecessarily.
linters:
  enable:
    - revive

linters-settings:
  revive:
    rules:
      - name: exported
        arguments:
          - disableStutteringCheck

issues:
  include:
    - EXC0012  # EXC0012 revive: Annoying issue about not having a comment. The rare codebase has such comments
    - EXC0014  # EXC0014 revive: Annoying issue about not having a comment. The rare codebase has such comments

See https://www.pixelstech.net/article/1650069870-golangci-lint-to-enable-comment-check-for-exported-functions

blocked by #1321

blocked by #1321

or...enables #1321 haha
probably should be done together since it would validate that we hit them all.


Cannot add Facebook Messenger

If I try to add Messenger service stucks on Verify account on Facebook screen

OS: Windows 7 SP1

Same thing happens with Ferdi. It seems to be "something" to do with Facebook

had the same issue. only way i could sign in was to disable 2 factor auth , sign in again and then re-enable the 2 factor auth

Same issue, @andyrea76's suggestion worked for me as well.

same issue, and same workaround worked


Response listener gets added multiple times in a ReactPHP environment

Right now the listener assumes the EventDispatcher will be rebooted after every request as per the normal PHP way of handling requests, not the case in a ReactPHP app, where all services are long-lived and serve many requests. Therefore, on every CORS request, the response listener is added again to the EventDispatcher queue. Not sure if there's an easy way to prevent that without a considerable refactor.

This was fixed.. some time ago :D


(GH-1358) Add yaml plugin

Ship a yaml plugin that reads yaml data from a specified file path and returns the contents. Enhance documentation with examples from this very simple concept.

CLA signed by all contributors.


Updated NextJS project dependencies

NextJS project was updated to the most recent versions of dependencies. Minor edit in Readme.

LGTM, thanks.


Using JACK for recording

I want to record my game with my headset mic, and the first method (using PulseAudio) nearly blew my ears out. I want to uninstall PulseAudio to get JACK, but I'm not sure if it is safe. Here: http://www.maartenbaert.be/simplescreenrecorder/recording-game-audio/#recording-game-audio-and-microphone-at-the-same-time

That should be fine, although you may have to manually start JACK after installation.

Do I look at the documentation for that? QJackCtl didn't work because of
ALSA whatever.
 ___      .__   __. .___________. __    __    ______   .__   __.

/   \     |  \ |  | |           ||  |  |  |  /  __  \  |  \ |  |

\   \     /   /
/  ^  \    |   |  | ---|  |----|  ||  | |  |  |  | |   |  |
\   \   /   /
/  /_\  \   |  .  |     |  |     |   __   | |  |  |  | |  .  |
_    _/
/  _____  \  |  |\   |     |  |     |  |  |  | |  `--'  | |  |\   |     |  |
//     _\ |__| _|     ||     ||  ||  _/  || __|     |_|
On Sun, Feb 22, 2015 at 8:12 PM, David Schneider<EMAIL_ADDRESS>wrote:

That should be fine, although you may have to manually start JACK after
installation.
—
Reply to this email directly or view it on GitHub
https://github.com/dnschneid/crouton/issues/1434#issuecomment-75487376.

Get this log:
15:56:42.653 Patchbay deactivated.
15:56:42.655 Statistics reset.
15:56:42.683 Could not open ALSA sequencer as a client. ALSA MIDI patchbay will be not available.
ALSA lib seq_hw.c:457:(snd_seq_hw_open) open /dev/snd/seq failed: Permission denied
15:56:50.090 D-BUS: Service is available (org.jackaudio.service aka jackdbus).
Cannot connect to server socket err = No such file or directory
Cannot connect to server request channel
jack server is not running or cannot be started
15:57:54.863 D-BUS: JACK server could not be started. Sorry
Cannot connect to server socket err = No such file or directory
Cannot connect to server request channel
jack server is not running or cannot be started
Mon Feb 23 15:57:54 2015: Starting jack server...
Mon Feb 23 15:57:54 2015: JACK server starting in realtime mode with priority 10
Mon Feb 23 15:57:54 2015: ERROR: Cannot lock down 82274202 byte memory area (Cannot allocate memory)
Mon Feb 23 15:57:54 2015: ERROR: control open "hw:0" (No such file or directory)
Mon Feb 23 15:57:54 2015: ERROR: control open "hw:0" (No such file or directory)
Mon Feb 23 15:57:54 2015: creating alsa driver ... hw:0|hw:0|1024|2|44100|0|0|nomon|swmeter|-|32bit
Mon Feb 23 15:57:54 2015: ERROR: control open "hw:0" (No such file or directory)
Mon Feb 23 15:57:54 2015: ERROR: ALSA: Cannot open PCM device alsa_pcm for playback. Falling back to capture-only mode
Mon Feb 23 15:57:54 2015: ERROR: Cannot initialize driver
Mon Feb 23 15:57:54 2015: ERROR: JackServer::Open failed with -1
Mon Feb 23 15:57:54 2015: ERROR: Failed to open server
Mon Feb 23 15:57:56 2015: Saving settings to "/home/firestar115/.config/jack/conf.xml" ...
15:58:00.551 Could not connect to JACK server as client. - Overall operation failed. - Unable to connect to server. Please check the messages window for more info.
Cannot connect to server socket err = No such file or directory
Cannot connect to server request channel
jack server is not running or cannot be started

Returns error.

Running:
$ aplay -D pcm.jack ~/Downloads/Black\ Mesa\ EBS.mp3
jack_client_new: deprecated
Cannot connect to server socket err = No such file or directory
Cannot connect to server request channel
jack server is not running or cannot be started
aplay: main:722: audio open error: No such file or directory

Try killing of cras (sudo stop cras in the crosh shell), then add yourself to the audio-hw group in crouton (sudo adduser $USER audio-hw), exit the chroot, enter the chroot, and try again.

adduser: The group `audio-hw' does not exist.
 ___      .__   __. .___________. __    __    ______   .__   __.

/   \     |  \ |  | |           ||  |  |  |  /  __  \  |  \ |  |

\   \     /   /
/  ^  \    |   |  | ---|  |----|  ||  | |  |  |  | |   |  |
\   \   /   /
/  /_\  \   |  .  |     |  |     |   __   | |  |  |  | |  .  |
_    _/
/  _____  \  |  |\   |     |  |     |  |  |  | |  `--'  | |  |\   |     |  |
//     _\ |__| _|     ||     ||  ||  _/  || __|     |_|
On Mon, Feb 23, 2015 at 4:45 PM, David Schneider<EMAIL_ADDRESS>wrote:

Try killing of cras (sudo stop cras in the crosh shell), then add
yourself to the audio-hw group in crouton (sudo adduser $USER audio-hw),
exit the chroot, enter the chroot, and try again.
—
Reply to this email directly or view it on GitHub
https://github.com/dnschneid/crouton/issues/1434#issuecomment-75675541.

Argh, hwaudio, sorry.

It kind of worked...
https://www.youtube.com/watch?v=OMOlU5ajP9E

WAIT! I CAN'T HEAR ANY SOUNDS ON MY CHROMEBOOK! :anguished:

Yes, when you tell crouton to take over the hardware audio device, Chromium OS audio will stop working.  You need to stop JACK and then do a sudo start cras in Chromium OS to get it back.

Worked!


Fix Numpy VisibleDeprecationWarning in indexing tests

NumPy is emitting VisibleDeprecationWarning in these tests because of dimension mismatch with boolean index.
/home/niboshi/w/repos/cupy/tests/cupy_tests/core_tests/test_ndarray_adv_indexing.py:283: VisibleDeprecationWarning: boolean index did not match indexed array along dimension 0; dimension is 2 but corresponding boolean dimension is 1
  a[self.indexes] = self.value

LGTM!


[stable/airflow] Mount password when Postgres or Redis are enabled

What this PR does / why we need it:
Fixes the if condition introduced  in PR #15712.
If we install the chart with the default values like helm install stable/airflow --name=airflow the web, scheduler and worker pods won't have the Redis and Postgres passwords mounted. This will cause the workers to fail on spawning:
consumer: Cannot connect to redis://airflow-redis-master:6379/1: NOAUTH Authentication required.. Trying again in X seconds...
Which issue this PR fixes

fixes #15896

Special notes for your reviewer:
The issue will still persist if postgresql.enabled or redis.enabled are set to false and the existingSecret is not set, which should be ok.
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Title of the PR starts with chart name (e.g. [stable/chart])

@gsemet Can you please have a look? This should fix this open issue #15896
Thanks!

/assign @gsemet

/lgtm
i guess this would not be enough, since i do not think this allow to specify the key name with the pg password in existing secret

@gsemet Yes that is true, it will only read the keys postgres-password and redis-password.
I can open another PR to enable the user to specify the keys too and have it in values.yaml, wdyt?

Yes please


pat: add prefix pattern

fixes: #62
Signed-off-by: Tim Bray<EMAIL_ADDRESS>

Codecov Report
Base: 95.93% // Head: 95.83% // Decreases project coverage by -0.10% :warning:

Coverage data is based on head (ddd5671) compared to base (c1b07bf).
Patch coverage: 91.48% of modified lines in pull request are covered.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #190      +/-   ##
==========================================
- Coverage   95.93%   95.83%   -0.10%
==========================================
  Files          17       17
  Lines        2067     2114      +47
==========================================
+ Hits         1983     2026      +43
- Misses         60       62       +2
- Partials       24       26       +2

Impacted Files
Coverage Δ

value_matcher.go
91.77% <85.71%> (-1.31%)
:arrow_down:

pattern.go
96.59% <100.00%> (+0.41%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

 That breaks if there's a * in the prefix. OK, am happy with this now, un-draft-i-fying it, would appreciate a review

Just nits left, I'll approve and let you decide whether you want to make small adjustments based on the review. At least reverting the default scenario seems like a zero-waste but good engineering practice.


only allow skip, mute segment types

downstream of https://github.com/mchangrh/sb-slash/issues/63

has some use for jumping to the time


seed bundle tests

DEPENDS ON https://github.com/holochain/lair/pull/48

adds unit and fixture implementation tests

Just to make sure, do these tests do a roundtrip of the custom deserialization you wrote?

@maackle

Just to make sure, do these tests do a roundtrip of the custom deserialization you wrote?

Yep, test_pwhash_cipher does a round trip of a bundle containing a pwhash cipher, and test_security_questions_cipher does a round trip of a bundle containing a security questions cipher.


Deal with edge case of zero length paths.

When cropping larger OSM datasets sometimes paths with zero nodes are formed.
Such paths break path -> svg code.
Checking for len < 2 resolves this problem.

Checks had failed mostly because of style issues. The project uses Black style formatter. To apply Black style, one have to simply run
black -l 80 map_machine

with the Black installed (pip install black).


feat: add version to built file (resolve #610)

@fi3ework note that this won't suffice, in the publish flow the build is run first, and after that the version number is updated, so some swapping around is needed there as well :)


Accesslog bufferingSize parameter not applying

[X] Yes, I've searched similar issues on GitHub and didn't find any.
[X] Yes, I've searched similar issues on the Traefik community forum and didn't find any.

What did you do?
Specified bufferingSize: 100 parameter in traefik.yml static configuration file. Expected Traefik AccessLog to be updated only every 100 lines.
Extra information:

Setting bufferingSize: foo raises appropriate error: "command traefik error: strconv.ParseInt: parsing "Foo": invalid syntax"
Setting --accesslog.bufferingsize=100 or TRAEFIK_ACCESSLOG_BUFFERINGSIZE=100 under command or environment sections respectively in docker compose.yml does nothing.
All other settings in traefik.yml apply appropriately.

What did you see instead?
bufferingSize parameter did not apply. AccessLog file is updated on each log entry.
What version of Traefik are you using?
Traefik version 2.9.10 built on 2023-04-06T16:15:08Z
What is your environment & configuration?
Docker Compose.yml, Traefik section.
services:
    reverse-proxy:
      image: traefik:v2.9
      container_name: traefik
      restart: unless-stopped
      depends_on:
        - socket-proxy
      ports:
        - "443:443"
      networks:
        - default
        - traefik_proxy
      volumes:
        - ./traefik/traefik.yml:/traefik.yml:ro
        - ./traefik/acme.json:/etc/traefik/acme.json
        - ./traefik/traefik-provider.yml:/traefik-provider.yml
        - /run/log/traefik:/run/log/
        - /etc/localtime:/etc/localtime:ro
        - /etc/timezone:/etc/timezone:ro
      labels:
        - "traefik.enable=true"
        - "traefik.http.routers.dashboard.entrypoints=https"
        - "traefik.http.routers.dashboard.rule=Host(`REDACTED`)"
        - "traefik.http.routers.dashboard.service=api@internal"
        - "traefik.http.routers.dashboard.tls=true"
        - "traefik.http.routers.dashboard.tls.certresolver=letsencrypt"
        - "traefik.http.routers.dashboard.tls.domains[0].main=REDACTED"
        - "traefik.http.routers.dashboard.tls.domains[0].sans=REDACTED"

Traefik.yml, Static Configuration
global:
  checkNewVersion: false

api:
  dashboard: true

entryPoints:
  https:
    address: ":443"
    http:
      tls:
        certResolver: letsencrypt
        domains:
          - main: "REDACTED"
            sans: "REDACTED"

certificatesResolvers:
  letsencrypt:
    acme:
      dnsChallenge:
        provider: REDACTED
      storage: "/etc/traefik/acme.json"

providers:
  docker:
    endpoint: "tcp://socket-proxy:2375"
    watch: true
    exposedbydefault: false
  file:
    filename: "/traefik-provider.yml"

log:
  level: "DEBUG"
  filePath: "/run/log/traefik.log"

accessLog:
  filePath: "/run/log/traefikaccess.log"
  bufferingSize: 100
  fields:
    names:
      StartUTC: drop

If applicable, please paste the log output in DEBUG level
time="2023-06-25T18:56:16-07:00" level=debug msg="Static configuration loaded {\"global\":{},\"serversTransport\":{\"maxIdleConnsPerHost\":200},\"entryPoints\":{\"https\":{\"address\":\":443\",\"transport\":{\"lifeCycle\":{\"graceTimeOut\":\"10s\"},\"respondingTimeouts\":{\"idleTimeout\":\"3m0s\"}},\"forwardedHeaders\":{},\"http\":{\"tls\":{\"certResolver\":\"letsencrypt\",\"domains\":[{\"main\":\"REDACTED\",\"sans\":[\"REDACTED\"]}]}},\"http2\":{\"maxConcurrentStreams\":250},\"udp\":{\"timeout\":\"3s\"}}},\"providers\":{\"providersThrottleDuration\":\"2s\",\"docker\":{\"watch\":true,\"endpoint\":\"tcp://socket-proxy:2375\",\"defaultRule\":\"Host(`{{ normalize .Name }}`)\",\"swarmModeRefreshSeconds\":\"15s\"},\"file\":{\"watch\":true,\"filename\":\"/traefik-provider.yml\"}},\"api\":{\"dashboard\":true},\"log\":{\"level\":\"DEBUG\",\"filePath\":\"/run/log/traefik.log\",\"format\":\"common\"},\"accessLog\":{\"filePath\":\"/run/log/traefikaccess.log\",\"format\":\"common\",\"filters\":{},\"fields\":{\"defaultMode\":\"keep\",\"names\":{\"StartUTC\":\"drop\"},\"headers\":{\"defaultMode\":\"drop\"}},\"bufferingSize\":100},\"certificatesResolvers\":{\"letsencrypt\":{\"acme\":{\"caServer\":\"https://acme-v02.api.letsencrypt.org/directory\",\"storage\":\"/etc/traefik/acme.json\",\"keyType\":\"RSA4096\",\"certificatesDuration\":2160,\"dnsChallenge\":{\"provider\":\"REDACTED\"}}}}}"

Hello,
you cannot use file and CLI flags or env vars at the same time to define the static configuration.

There are three different, mutually exclusive (i.e. you can use only one at the same time), ways to define static configuration options in Traefik:

In a configuration file
In the command-line arguments
As environment variables

https://doc.traefik.io/traefik/getting-started/configuration-overview/#the-static-configuration

Hello,
you cannot use file and CLI flags or env vars at the same time to define the static configuration.

Hello,
I am using exclusively config file for static configuration. The attempts at using CLI and Env parameters were included only for completeness.

Hello @GittyBruce,
There is a mistake in the documentation.
The buffering size is not the number of logs Traefik keeps in memory before writing.
Traefik does not wait for the buffer to be full before writing.
It is only a mechanism to allow asynchronous writes of the log lines.

Hello @GittyBruce,
There is a mistake in the documentation. The buffering size is not the number of logs Traefik keeps in memory before writing. Traefik does not wait for the buffer to be full before writing. It is only a mechanism to allow asynchronous writes of the log lines.

Looking at logger.go that does seem to be true. If I'm understanding this correctly, any bufferingSize parameter >= 1 is the same? So bufferingSize=1 is identical to bufferingSize=100 is identical to bufferingSize=9,223,372,036,854,775,808?

Hello @GittyBruce,

Looking at logger.go that does seem to be true. If I'm understanding this correctly, any bufferingSize parameter >= 1 is the same? So bufferingSize=1 is identical to bufferingSize=100 is identical to bufferingSize=9,223,372,036,854,775,808?

No, the value could have an effect depending on the load.
A >0 value would indeed make Traefik start writing log lines asynchronously.
Otherwise, the buffering size is only a buffer size and not a chunk/block size to batch write the log lines. The greater it is the less are chances for the log lines write operation to be blocking a request.


The old version(KyranRana/cloudflare-bypass) doesn't work

https://github.com/KyranRana/cloudflare-bypass/commit/97f4af8c1fed1ee43e902a0a4abad7654ae72365 Only this version works, and the old version can no longer bypass cloudflare.
The bypass mechanism has changed, but has not yet been released see https://github.com/KyranRana/cloudflare-bypass/issues/146.
maybe we can change the composer.json file to depends on the master branch node?
Just a suggestion

That's a relevant suggestion but not possible yet due to the release process adopted by Kyran on its repository for now. I already opened an issue (https://github.com/KyranRana/cloudflare-bypass/issues/157) to get rid of this and being able to be autonomous.


Introduction of result set selection

Every type of calculation (nominal, U-I, study) adds specific "set_name"-column to result DataFrame
Selection based on calculation type:
Single Calculation:

U-I-Curve:

Study

Study with U-I

resolves #47


Build error that seems to have not impact

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
I am occasionally seeing an error in the maven build that looks alarming but does not seem to cause a problem

I am raising this issue in case others see it too.  This is the error in text
Could not find artifact org.osgi:org.osgi.core:pom:8.0.0 in 1-MavenCentral (https://repo1.maven.org/maven2/)

Expected Behavior
No errors in the build
Steps To Reproduce
No response
Environment
- Egeria:
- OS:
- Java:
- Browser (for UI issues):
- Additional connectors and integration:

Any Further Information?
No response

Does this only occur with intellij? Ie is command line clean?
If only intellij, What intellij version
Is it only on a clean clone where no clue build has been done?
May be related to https://youtrack.jetbrains.com/issue/IDEA-301612/Importing-a-gradle-project-into-intelliJ-gives-errors-after-prepareKotlinBuildScriptModel-Build-is-working

Checked jetbrains issue. no update. chasing. Probably worth concentrating on the gradle variant in future and fix up these glitches there (as much as we can).
It may also be worth adding a note in the docs on this scenario

removing maven tag. will leave open to verify for gradle only

I am going to suggest we close this if it's only affecting maven as we will move fully to gradle.
The project init issue in #6893 will be left open as gradle is impacted


Xcode 15 fails to build iOS 17 Flutter app

Is there an existing issue for this?

[X] I have searched the existing issues
[X] I have read the guide to filing a bug

Steps to reproduce

Open Flutter Project
Run flutter build

Actual results
It errors out with the following message:

Error
Could not build the precompiled application for the device.
Error (Xcode): Cycle inside Runner; building could produce unreliable results.
Cycle details:
→ Target 'Runner': CodeSign /Users/thilo/Documents/notely/build/ios/Debug-iphoneos/Runner.app
○ That command depends on command in Target 'Runner': script phase “[CP] Embed Pods Frameworks”
○ That command depends on command in Target 'Runner': script phase “Thin Binary”
○ Target 'Runner' has process command with output '/Users/thilo/Documents/notely/build/ios/Debug-iphoneos/Runner.app/Info.plist'
○ Target 'Runner' has copy command from '/Users/thilo/Documents/notely/build/ios/Debug-iphoneos/ScheduleWidgetExtension.appex' to
'/Users/thilo/Documents/notely/build/ios/Debug-iphoneos/Runner.app/PlugIns/ScheduleWidgetExtension.appex'

Raw dependency cycle trace:

target:  ->

node: <all> ->

command: <all> ->

node: /Users/thilo/Documents/notely/build/ios/Debug-iphoneos/Runner.app/_CodeSignature ->

command: P0:target-Runner-18c1723432283e0cc55f10a6dcfd9e0288a783a885d8b0b3beb2e9f90bde3f49-:Debug:CodeSign
/Users/thilo/Documents/notely/build/ios/Debug-iphoneos/Runner.app ->

node: /Users/thilo/Documents/notely/build/ios/Debug-iphoneos/Runner.app/Frameworks/FBLPromises.framework/ ->

directoryTreeSignature: b ->

directoryContents: /Users/thilo/Documents/notely/build/ios/Debug-iphoneos/Runner.app/Frameworks/FBLPromises.framework ->

CYCLE POINT ->

node: /Users/thilo/Documents/notely/build/ios/Debug-iphoneos/Runner.app/Frameworks/FBLPromises.framework ->

command: P2:target-Runner-18c1723432283e0cc55f10a6dcfd9e0288a783a885d8b0b3beb2e9f90bde3f49-:Debug:PhaseScriptExecution [CP] Embed Pods Frameworks
/Users/thilo/Library/Developer/Xcode/DerivedData/Runner-bzzryhtvqztgomayixlvocnhlfff/Build/Intermediates.noindex/Runner.build/Debug-iphoneos/Runner.build/Script-7B327
4101575EF670BE0E37C.sh ->

node:
/Users/thilo/Library/Developer/Xcode/DerivedData/Runner-bzzryhtvqztgomayixlvocnhlfff/Build/Intermediates.noindex/Runner.build/Debug-iphoneos/Runner.build/InputFileLis
t-7B3274101575EF670BE0E37C-Pods-Runner-frameworks-Debug-input-files-22ff9ee76a45f44ef92677a5f9f6deea-resolved.xcfilelist ->

command: P2:target-Runner-18c1723432283e0cc55f10a6dcfd9e0288a783a885d8b0b3beb2e9f90bde3f49-:Debug:WriteAuxiliaryFile
/Users/thilo/Library/Developer/Xcode/DerivedData/Runner-bzzryhtvqztgomayixlvocnhlfff/Build/Intermediates.noindex/Runner.build/Debug-iphoneos/Runner.build/InputFileLis
t-7B3274101575EF670BE0E37C-Pods-Runner-frameworks-Debug-input-files-22ff9ee76a45f44ef92677a5f9f6deea-resolved.xcfilelist ->

node: <target-Runner-18c1723432283e0cc55f10a6dcfd9e0288a783a885d8b0b3beb2e9f90bde3f49--fused-phase4-thin-binary> ->

command: P0:::Gate target-Runner-18c1723432283e0cc55f10a6dcfd9e0288a783a885d8b0b3beb2e9f90bde3f49--fused-phase4-thin-binary ->

node:
<execute-shell-script-18c1723432283e0cc55f10a6dcfd9e02f1eee2015e8ff5ebcd27678f788c2826-target-Runner-18c1723432283e0cc55f10a6dcfd9e0288a783a885d8b0b3beb2e9f90bde3f49-
> ->

command: P2:target-Runner-18c1723432283e0cc55f10a6dcfd9e0288a783a885d8b0b3beb2e9f90bde3f49-:Debug:PhaseScriptExecution Thin Binary
/Users/thilo/Library/Developer/Xcode/DerivedData/Runner-bzzryhtvqztgomayixlvocnhlfff/Build/Intermediates.noindex/Runner.build/Debug-iphoneos/Runner.build/Script-3B06A
D1E1E4923F5004D2608.sh ->

node: /Users/thilo/Documents/notely/build/ios/Debug-iphoneos/Runner.app/Info.plist/ ->

directoryTreeSignature: L ->

directoryContents: /Users/thilo/Documents/notely/build/ios/Debug-iphoneos/Runner.app/Info.plist ->

node: /Users/thilo/Documents/notely/build/ios/Debug-iphoneos/Runner.app/Info.plist ->

command: P0:target-Runner-18c1723432283e0cc55f10a6dcfd9e0288a783a885d8b0b3beb2e9f90bde3f49-:Debug:ProcessInfoPlistFile
/Users/thilo/Documents/notely/build/ios/Debug-iphoneos/Runner.app/Info.plist /Users/thilo/Documents/notely/ios/Runner/Info.plist ->

node: /Users/thilo/Documents/notely/build/ios/Debug-iphoneos/Runner.app/PlugIns/ScheduleWidgetExtension.appex ->

command: P0:target-Runner-18c1723432283e0cc55f10a6dcfd9e0288a783a885d8b0b3beb2e9f90bde3f49-:Debug:Copy
/Users/thilo/Documents/notely/build/ios/Debug-iphoneos/Runner.app/PlugIns/ScheduleWidgetExtension.appex
/Users/thilo/Documents/notely/build/ios/Debug-iphoneos/ScheduleWidgetExtension.appex ->

node: <target-Runner-18c1723432283e0cc55f10a6dcfd9e0288a783a885d8b0b3beb2e9f90bde3f49--fused-phase5--cp--embed-pods-frameworks> ->

command: P0:::Gate target-Runner-18c1723432283e0cc55f10a6dcfd9e0288a783a885d8b0b3beb2e9f90bde3f49--fused-phase5--cp--embed-pods-frameworks ->

node: /Users/thilo/Documents/notely/build/ios/Debug-iphoneos/Runner.app/Frameworks/FBLPromises.framework

Error launching application on iPhone (23).

Logs
No response
Flutter Doctor output

Doctor output
[✓] Flutter (Channel stable, 3.10.5, on macOS 14.0 23A5257q darwin-arm64, locale en-CH)
    • Flutter version 3.10.5 on channel stable at /Users/thilo/fvm/versions/stable
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 796c8ef792 (2 days ago), 2023-06-13 15:51:02 -0700
    • Engine revision 45f6e00911
    • Dart version 3.0.5
    • DevTools version 2.23.1

[✗] Android toolchain - develop for Android devices
    ✗ Unable to locate Android SDK.
      Install Android Studio from: https://developer.android.com/studio/index.html
      On first launch it will assist you in installing the Android SDK components.
      (or visit https://flutter.dev/docs/get-started/install/macos#android-setup for detailed instructions).
      If the Android SDK has been installed to a custom location, please use
      `flutter config --android-sdk` to update to that location.

[✓] Xcode - develop for iOS and macOS (Xcode 15.0)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 15A5160n
    • CocoaPods version 1.12.1

[!] Android Studio (not installed)
    • Android Studio not found; download from https://developer.android.com/studio/index.html
      (or visit https://flutter.dev/docs/get-started/install/macos#android-setup for detailed instructions).

[✓] VS Code (version 1.79.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.66.0

[!] Connected device
    ! Error: Browsing on the local area network for iPhone (23). Ensure the device is unlocked and attached with a cable or associated with the same local area
      network as this Mac.
      The device must be opted into Developer Mode to connect wirelessly. (code -27)

[✓] Network resources
    • All expected network resources are available.

! Doctor found issues in 3 categories.

Hi @thilojaeggi. Does this occur with a new project or an existing one? If existing project, can you create a new flutter project and run flutter build? If the issue persists, kindly provide your full flutter run --verbose. If the logs are too long, you may attach them as txt file.
Thank you

Removing the widget did the trick. Idk why though tbh.

Ok. I guess it solved now. Thanks for confirming.

@thilojaeggi So I can’t have a widget? Or just remove it and make it again?

I honestly haven't tried adding a new one yet, maybe there are some settings specific to widgets that need to be changed.

I've just experienced this same issue. Deleting the widgets from the app gets the app to build & run, but of course this is not a real solution.

Hi @emawby.
Can try the steps in https://github.com/flutter/flutter/issues/128739#issuecomment-1590619555 if it helps resolve your issue.
If it doesn't, kindly provide a minimal complete code sample that doesn’t include 3rd party plugins or complex production code and step by step process to reproduce this issue.
Thank you

@thilojaeggi
This was locked by the bot, it is unlocked, you should be able to reply now. Thanks

Totally forgot to comment after this, the proposed solution didn't work in my case as ENABLE_USER_SCRIPT_SANDBOXING was disabled by default.

Thanks for your response and patience. Currently, we are unable to replicate this due to lack of latest config (Xcode 15 and iOS 17), but based on the report, there seems to be an already open issue similar to this https://github.com/flutter/flutter/issues/128531 and a workaround was provided on this issue per https://github.com/flutter/flutter/issues/128531#issuecomment-1587984546. Can you confirm this suits your use case?

@dam-ease
Seems like this issue happens when Xcode 15 tries to sign the app. I was able to replicate this issue by adding Widget Extension as a new target to an iOS Runner workspace.

If you create an empty flutter project, and just add the default notification extension in xcode, the app won't build. This is demonstrated in this repository: https://github.com/bobby-beckmann/FlutterTest
Attached output of flutter run --verbose
output.log

This seems to be happening with an extension on Xcode 14 too.

Op seems to have disappeared so I’ve opened https://github.com/flutter/flutter/issues/131586
please comment there to get attention.

Thanks @bobby-beckmann. Since the new issue is clearer on this issue, closing this as a duplicate of the newly opened issue, kindly comment, upvote and follow up on that issue for updates.
Thank you!


History back button doesn't work

@tomasbedrich commented:
I am not 100% sure it is the issue of this repository, so sorry in advance, if not.
The problem appears when I go to API documentation of (for example) a concatMap() method and then navigate to another method on the same page, for example a concatMapTo() method without reloading the page (i.e. using a search).
Then I am not able to use a back button in my browser to get back to a concatMap() method. Only the URL changes, but the page doesn't scroll to the right method.
(Copied from ReactiveX/reactivex.github.io#311)

There is appears to be some sort of bug in ESDoc involving anchor tag links that's causing this. @staltz do you have any input here? You've worked with the docs more than anyone.


Add support for Number

BTW, I've added support for requirements to https://github.com/sharkdp/bat/pull/2361. And I use your https://github.com/raimon49/requirements.txt.vim/tree/master/examples/requirements.txt as its test file. TIA. And if you like, you can also add it to Similar Projects.

Great, I'm a lover of the bat command. 👍


Version 2.1.1

Landed in https://github.com/metarhia/metasql/pull/256/commits/9fa8f0505abf355b082e388a98567b17a14f6770


python3Packages.georss-client: init at 0.13

Motivation for this change
Python library for accessing GeoRSS feeds
https://github.com/exxamalte/python-georss-client
Those are Home Assistant dependencies.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 125834 run on x86_64-linux 1

20 packages built:

python38Packages.aio-georss-client
python38Packages.aio-georss-gdacs
python38Packages.georss-client
python38Packages.georss-generic-client
python38Packages.georss-ign-sismologia-client
python38Packages.georss-ingv-centro-nazionale-terremoti-client
python38Packages.georss-nrcan-earthquakes-client
python38Packages.georss-qld-bushfire-alert-client
python38Packages.georss-tfs-incidents-client
python38Packages.georss-wa-dfes-client
python39Packages.aio-georss-client
python39Packages.aio-georss-gdacs
python39Packages.georss-client
python39Packages.georss-generic-client
python39Packages.georss-ign-sismologia-client
python39Packages.georss-ingv-centro-nazionale-terremoti-client
python39Packages.georss-nrcan-earthquakes-client
python39Packages.georss-qld-bushfire-alert-client
python39Packages.georss-tfs-incidents-client
python39Packages.georss-wa-dfes-client

Result of nixpkgs-review pr 125834 at fcc81afc run on aarch64-linux 1

20 packages built successfully:

python38Packages.aio-georss-client
python38Packages.aio-georss-gdacs
python38Packages.georss-client
python38Packages.georss-generic-client
python38Packages.georss-ign-sismologia-client
python38Packages.georss-ingv-centro-nazionale-terremoti-client
python38Packages.georss-nrcan-earthquakes-client
python38Packages.georss-qld-bushfire-alert-client
python38Packages.georss-tfs-incidents-client
python38Packages.georss-wa-dfes-client
python39Packages.aio-georss-client
python39Packages.aio-georss-gdacs
python39Packages.georss-client
python39Packages.georss-generic-client
python39Packages.georss-ign-sismologia-client
python39Packages.georss-ingv-centro-nazionale-terremoti-client
python39Packages.georss-nrcan-earthquakes-client
python39Packages.georss-qld-bushfire-alert-client
python39Packages.georss-tfs-incidents-client
python39Packages.georss-wa-dfes-client


Adding missing e2e tests and reordering tests based on alphabetical order

Codecov Report

Merging #1311 (dba29ee) into master (fc9d55f) will increase coverage by 0.48%.
The diff coverage is n/a.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files

Flag
Coverage Δ

packit-e2e
74.59% <ø> (+0.48%)
:arrow_up:

testsuite
11.09% <ø> (ø)

unittests
1.92% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

keylime/keylime_agent.py
73.12% <0.00%> (+0.57%)
:arrow_up:


Optimizes internal trace context related code, especially extra

After some trial and error with benchmarking, I found
MutableTraceContext is not enough a win unless we change api, which we
can't until Brave 6. Meanwhile, this optimizes as much as possible,
notably eliminating as many arraylist creations (from extra) as
possible.
Alternative to #755

here are the results, btw
before:
Benchmark                                                                                Mode      Cnt      Score      Error   Units
TracerBenchmarks.joinWithSpanInScope:joinWithSpanInScope·p0.999                                               sample              21.216              us/op
TracerBenchmarks.joinWithSpanInScope:·gc.alloc.rate.norm                                                      sample       15    791.518 ±    1.755    B/op
TracerBenchmarks.joinWithSpanInScope_extra:joinWithSpanInScope_extra·p0.999                                   sample              23.872              us/op
TracerBenchmarks.joinWithSpanInScope_extra:·gc.alloc.rate.norm                                                sample       15    782.750 ±    6.044    B/op
TracerBenchmarks.joinWithSpanInScope_unsampled:joinWithSpanInScope_unsampled·p0.999                           sample               1.100              us/op
TracerBenchmarks.joinWithSpanInScope_unsampled:·gc.alloc.rate.norm                                            sample       15     40.008 ±    0.005    B/op
TracerBenchmarks.joinWithSpanInScope_unsampled_extra:joinWithSpanInScope_unsampled_extra·p0.999               sample              31.611              us/op
TracerBenchmarks.joinWithSpanInScope_unsampled_extra:·gc.alloc.rate.norm                                      sample       15     40.055 ±    0.004    B/op
TracerBenchmarks.newChildWithSpanInScope:newChildWithSpanInScope·p0.999                                       sample              28.770              us/op
TracerBenchmarks.newChildWithSpanInScope:·gc.alloc.rate.norm                                                  sample       15   1336.272 ±    0.096    B/op
TracerBenchmarks.newChildWithSpanInScope_extra:newChildWithSpanInScope_extra·p0.999                           sample              20.769              us/op
TracerBenchmarks.newChildWithSpanInScope_extra:·gc.alloc.rate.norm                                            sample       15   1712.572 ±   37.532    B/op
TracerBenchmarks.newChildWithSpanInScope_unsampled:newChildWithSpanInScope_unsampled·p0.999                   sample               1.682              us/op
TracerBenchmarks.newChildWithSpanInScope_unsampled:·gc.alloc.rate.norm                                        sample       15     96.010 ±    0.003    B/op
TracerBenchmarks.newChildWithSpanInScope_unsampled_extra:newChildWithSpanInScope_unsampled_extra·p0.999       sample              16.352              us/op
TracerBenchmarks.newChildWithSpanInScope_unsampled_extra:·gc.alloc.rate.norm                                  sample       15    424.346 ±    0.046    B/op
TracerBenchmarks.nextWithSpanInScope:nextWithSpanInScope·p0.999                                               sample              11.152              us/op
TracerBenchmarks.nextWithSpanInScope:·gc.alloc.rate.norm                                                      sample       15   1384.177 ±   37.565    B/op
TracerBenchmarks.nextWithSpanInScope_extra:nextWithSpanInScope_extra·p0.999                                   sample              14.320              us/op
TracerBenchmarks.nextWithSpanInScope_extra:·gc.alloc.rate.norm                                                sample       15   1760.513 ±    0.033    B/op
TracerBenchmarks.nextWithSpanInScope_unsampled:nextWithSpanInScope_unsampled·p0.999                           sample               1.756              us/op
TracerBenchmarks.nextWithSpanInScope_unsampled:·gc.alloc.rate.norm                                            sample       15     96.007 ±    0.001    B/op
TracerBenchmarks.nextWithSpanInScope_unsampled_extra:nextWithSpanInScope_unsampled_extra·p0.999               sample              13.182              us/op
TracerBenchmarks.nextWithSpanInScope_unsampled_extra:·gc.alloc.rate.norm                                      sample       15    565.759 ±    4.174    B/op
TracerBenchmarks.startScopedSpanWithParent:startScopedSpanWithParent·p0.999                                   sample              11.742              us/op
TracerBenchmarks.startScopedSpanWithParent:·gc.alloc.rate.norm                                                sample       15   1296.177 ±    0.031    B/op
TracerBenchmarks.startScopedSpanWithParent_extra:startScopedSpanWithParent_extra·p0.999                       sample              15.318              us/op
TracerBenchmarks.startScopedSpanWithParent_extra:·gc.alloc.rate.norm                                          sample       15   1672.493 ±   37.556    B/op
TracerBenchmarks.startScopedSpanWithParent_unsampled:startScopedSpanWithParent_unsampled·p0.999               sample               1.746              us/op
TracerBenchmarks.startScopedSpanWithParent_unsampled:·gc.alloc.rate.norm                                      sample       15    104.010 ±    0.006    B/op
TracerBenchmarks.startScopedSpanWithParent_unsampled_extra:startScopedSpanWithParent_unsampled_extra·p0.999   sample              17.600              us/op
TracerBenchmarks.startScopedSpanWithParent_unsampled_extra:·gc.alloc.rate.norm                                sample       15    440.389 ±   12.483    B/op

after:
Benchmark                                                                                Mode      Cnt      Score      Error   Units
TracerBenchmarks.joinWithSpanInScope:joinWithSpanInScope·p0.999                                               sample              23.616             us/op
TracerBenchmarks.joinWithSpanInScope:·gc.alloc.rate.norm                                                      sample       15    735.714 ±   4.159    B/op
TracerBenchmarks.joinWithSpanInScope_extra:joinWithSpanInScope_extra·p0.999                                   sample              23.392             us/op
TracerBenchmarks.joinWithSpanInScope_extra:·gc.alloc.rate.norm                                                sample       15    722.041 ±   3.148    B/op
TracerBenchmarks.joinWithSpanInScope_unsampled:joinWithSpanInScope_unsampled·p0.999                           sample               2.274             us/op
TracerBenchmarks.joinWithSpanInScope_unsampled:·gc.alloc.rate.norm                                            sample       15     40.009 ±   0.006    B/op
TracerBenchmarks.joinWithSpanInScope_unsampled_extra:joinWithSpanInScope_unsampled_extra·p0.999               sample              25.632             us/op
TracerBenchmarks.joinWithSpanInScope_unsampled_extra:·gc.alloc.rate.norm                                      sample       15     40.053 ±   0.004    B/op
TracerBenchmarks.newChildWithSpanInScope:newChildWithSpanInScope·p0.999                                       sample              11.216             us/op
TracerBenchmarks.newChildWithSpanInScope:·gc.alloc.rate.norm                                                  sample       15   1288.179 ±  75.129    B/op
TracerBenchmarks.newChildWithSpanInScope_extra:newChildWithSpanInScope_extra·p0.999                           sample              12.048             us/op
TracerBenchmarks.newChildWithSpanInScope_extra:·gc.alloc.rate.norm                                            sample       15   1512.306 ±  12.507    B/op
TracerBenchmarks.newChildWithSpanInScope_unsampled:newChildWithSpanInScope_unsampled·p0.999                   sample               0.779             us/op
TracerBenchmarks.newChildWithSpanInScope_unsampled:·gc.alloc.rate.norm                                        sample       15     96.007 ±   0.002    B/op
TracerBenchmarks.newChildWithSpanInScope_unsampled_extra:newChildWithSpanInScope_unsampled_extra·p0.999       sample               9.920             us/op
TracerBenchmarks.newChildWithSpanInScope_unsampled_extra:·gc.alloc.rate.norm                                  sample       15    336.123 ±   0.008    B/op
TracerBenchmarks.nextWithSpanInScope:nextWithSpanInScope·p0.999                                               sample              11.040             us/op
TracerBenchmarks.nextWithSpanInScope:·gc.alloc.rate.norm                                                      sample       15   1336.182 ±   0.012    B/op
TracerBenchmarks.nextWithSpanInScope_extra:nextWithSpanInScope_extra·p0.999                                   sample              11.808             us/op
TracerBenchmarks.nextWithSpanInScope_extra:·gc.alloc.rate.norm                                                sample       15   1544.300 ±  45.141    B/op
TracerBenchmarks.nextWithSpanInScope_unsampled:nextWithSpanInScope_unsampled·p0.999                           sample               0.816             us/op
TracerBenchmarks.nextWithSpanInScope_unsampled:·gc.alloc.rate.norm                                            sample       15     96.006 ±   0.001    B/op
TracerBenchmarks.nextWithSpanInScope_unsampled_extra:nextWithSpanInScope_unsampled_extra·p0.999               sample              16.866             us/op
TracerBenchmarks.nextWithSpanInScope_unsampled_extra:·gc.alloc.rate.norm                                      sample       15    448.328 ±  12.505    B/op
TracerBenchmarks.startScopedSpanWithParent:startScopedSpanWithParent·p0.999                                   sample               9.600             us/op
TracerBenchmarks.startScopedSpanWithParent:·gc.alloc.rate.norm                                                sample       15   1248.145 ±  37.564    B/op
TracerBenchmarks.startScopedSpanWithParent_extra:startScopedSpanWithParent_extra·p0.999                       sample              10.336             us/op
TracerBenchmarks.startScopedSpanWithParent_extra:·gc.alloc.rate.norm                                          sample       15   1496.261 ±  50.079    B/op
TracerBenchmarks.startScopedSpanWithParent_unsampled:startScopedSpanWithParent_unsampled·p0.999               sample               0.660             us/op
TracerBenchmarks.startScopedSpanWithParent_unsampled:·gc.alloc.rate.norm                                      sample       15    104.006 ±   0.001    B/op
TracerBenchmarks.startScopedSpanWithParent_unsampled_extra:startScopedSpanWithParent_unsampled_extra·p0.999   sample               5.975             us/op
TracerBenchmarks.startScopedSpanWithParent_unsampled_extra:·gc.alloc.rate.norm                                sample       15    352.117 ±  12.520    B/op

running bench after latest. will merge in the morning and move on unless someone screams

using flags directly is measurably quicker vs helpers. I will revert to using them

You would expect jvm to inline the helpers.

@yurishkuro we can revisit later as it is internal code. np

by revisit, what I mean is that places which are using bitwise to quick check on something.. someone can run the bench only changing this specific line. The problem is that the benchmarks take a very long time to execute (in sampled mode), line 20m to run just TracerBenchmarks. It is possible some lines can be same as if using dispatch, but anyway can't personally afford to spend too much more time on this right now, so good enough basically.


Update UG

Update UG for all commands
Update some javadocs
Update HelpCommand and ClearCommand

Codecov Report

Merging #106 (99cc2a3) into master (65c893a) will increase coverage by 0.09%.
The diff coverage is 100.00%.

:exclamation: Current head 99cc2a3 differs from pull request most recent head c4aeeed. Consider uploading reports for the commit c4aeeed to get more accurate results

@@             Coverage Diff              @@
##             master     #106      +/-   ##
============================================
+ Coverage     31.79%   31.88%   +0.09%
- Complexity      390      391       +1
============================================
  Files           145      145
  Lines          2837     2838       +1
  Branches        343      343
============================================
+ Hits            902      905       +3
+ Misses         1890     1888       -2
  Partials         45       45

Impacted Files
Coverage Δ

...ain/java/jarvis/logic/commands/AddNoteCommand.java
0.00% <ø> (ø)

...jarvis/logic/commands/AddParticipationCommand.java
0.00% <ø> (ø)

.../java/jarvis/logic/commands/DeleteNoteCommand.java
0.00% <ø> (ø)

...java/jarvis/logic/commands/MarkStudentCommand.java
0.00% <ø> (ø)

...ava/jarvis/logic/commands/MasteryCheckCommand.java
0.00% <ø> (ø)

...ava/jarvis/logic/commands/UnmarkLessonCommand.java
0.00% <ø> (ø)

...va/jarvis/logic/commands/UnmarkStudentCommand.java
0.00% <ø> (ø)

src/main/java/jarvis/ui/HelpWindow.java
0.00% <ø> (ø)

.../main/java/jarvis/logic/commands/ClearCommand.java
100.00% <100.00%> (ø)

src/main/java/jarvis/model/ModelManager.java
57.69% <0.00%> (+1.92%)
:arrow_up:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Add calibration comparison command

This provides a way to compare a trained model against a collection of images which have been annotated with their actual distance values in the same way as the calibration inputs have been.

Calibration is no longer a thing.


Use same size for initial capacity of Builder in transformations

Hi @daniel-rusu!
Great idea! While most Kotlin applications certainly don't need to worry about squeezing out the efficiency of using something like ImmutableArray, it is undoubtedly a valuable tool to add to the Kotlin ecosystem. Your implementation is great!
This PR changes the use of Builder() to Builder(size) based on the idea that some of these operations would typically result in arrays of around the same size as the original array, while other operations will never result in an array exceeding the size of the original array, and as such having an initial capacity of the same size means we can avoid repeatedly resizing the array of the builder. E.g. for something like filter and mapNotNull we know that the result is at most an array of the same size as the original array, so by having an initial capacity capable of holding all the elements, we will see at most 2 array allocations.

[X] Have you followed the contributing guidelines?
[X] Have you added tests that validate the new capability or bug fix?
[X] Does your submission align with the current code style?
[X] Have you checked that there aren't other open pull requests for the same change?

Looking forward to your feedback!

Hi @daniel-rusu!
Great idea! While most Kotlin applications certainly don't need to worry about squeezing out the efficiency of using something like ImmutableArray, it is undoubtedly a valuable tool to add to the Kotlin ecosystem. Your implementation is great!
This PR changes the use of Builder() to Builder(size) based on the idea that some of these operations would typically result in arrays of around the same size as the original array, while other operations will never result in an array exceeding the size of the original array, and as such having an initial capacity of the same size means we can avoid repeatedly resizing the array of the builder. E.g. for something like filter and mapNotNull we know that the result is at most an array of the same size as the original array, so by having an initial capacity capable of holding all the elements, we will see at most 2 array allocations.

Hi @Nillerr , thanks so much for your suggestion!  I thought a lot about whether to use some sort of initial capacity estimate for the builders when the resulting size isn't known in advance.  There are some gotchas that would make the performance worse while also increasing memory consumption:

For operations like flatMap, we generally have something like a list of orders with each order having a list of products, and we want to flatten it into a large list of all products from all orders.  Note that if you have less than 10 orders with each order having a bunch of products, setting the initial builder capacity to the number of orders will increase the number of resizing operations.

For operations like groupBy, we generally have something like a list of people that we want to group based on each person's favorite sport.  Using the size of the initial array as the default capacity for each group will multiply the memory consumption from N elements to N X G elements where G is the number of groups.  Note that G can be as high as N depending on the criteria for group determination so the memory consumption could jump to N^2 during this operation.

When filtering a very large array (filter / filterNotNull / mapNotNull / etc.), it's highly unlikely for all elements to pass the condition and it's also possible for only a tiny percentage to pass. Creating a builder with the original very large capacity will make it more difficult for the memory subsystem to locate sufficient contiguous memory resulting in more checks as it looks through the list of free memory chunks.  Additionally, these large memory allocations are typically allocated directly in the tenured memory zone due to the weak-generational hypothesis which assumes that large objects will stick around a long time.  The tenured space is typically only inspected during major garbage collections which last hundreds of times longer than minor collections.  So adding memory directly into the tenured space will increase the rate of major GCs and negatively affect application performance potentially quite significantly.

So overall, I think that most developers would be disappointed if memory consumption increased after swapping read-only lists with Immutable Arrays.  My goal is that  Immutable Arrays would have the same or lower memory consumption while also improving performance.
Having said all that, I think we can approach this from a different angle and perhaps achieve the seemingly impossible result of getting the best of both worlds.  Stepping back, the real reason we're trying to avoid re-sizing operations is because each resize has to copy all the current elements into the new larger array.  I'm tinkering with builder enhancements that give us the best of both worlds by starting with low capacity and also avoiding any copying when resizing.  The idea is for the builder to store a linked list of increasing arrays so that any new capacity would just create another link and leave the current elements as is.  Copying from the links into the final array shouldn't be worse as this allows us to copy only a single time instead of each time it's resized.  This should also minimize large memory requests as the builder will have the elements split into chunks instead of one much larger array.  I haven't submitted any code yet as I'm currently writing throw-away benchmarks to measure different ideas.

The filter method with initial size set allows to remove the builder with its capacity checks completely. That gave around 25% performance gain on my system for the test scenario you use. That's significant. May it is an option to define a size value for what is considered as "small" array and then use different implementations for "small" and "big" arrays.

I also tested another 'branchless' version that could be done when there is no builder. The results speak for itself.

Thats the code without builder and branchless loop:
@Suppress("UNCHECKED_CAST")
public inline fun filter(predicate: (element: T) -> Boolean): ImmutableArray<T> {
    val buffer = arrayOfNulls<Any?>(size) as Array<T>
    var bufferSize = 0

    for (element in values) {
        buffer[bufferSize] = element
        bufferSize += predicate(element).asInt
    }

    return when (bufferSize) {
        0 -> EMPTY
        size -> this
        else -> ImmutableArray<T>(buffer.copyOf(bufferSize) as Array<T>)
    }
}

 /** @returns 1 if true otherwise 0 */
public inline val Boolean.asInt : Int get() = 1 and (hashCode() shr 1)

(pls let me know if this the wrong place for my comments)

Regarding:
The idea is for the builder to store a linked list of increasing arrays so that any new capacity would just create another link and leave the current elements as is
Thats an Unrolled linked list, i talked about here.


Luckperms don't reconnect mysql database

I run minecraft server in Server A,  mysql database on Server B, when server B down (offline, broken internet) for 3, 4 minutes and after that get online back. But luckperms don't reconnect database, so players can't join. Other plugins using mysql still reconnect successfull.
Sorry my english bad.

If Server B goes down and then up whilst Server A is online, then it will reconnect when Server B comes back up.
However, if Server B is offline while Server A starts up for the first time, it will not reconnect. This is the intended behaviour.

No, in my case: Server A and B both online -> B down, A still online -> B comeback, Lucperms on A don't connect B.

LuckPerms uses HikariCP to handle connection pooling.
https://github.com/brettwooldridge/HikariCP
It's unlikely that there's an issue there. More likely that your database isn't accepting new connections, or the old ones aren't being completely dropped.
Either way, it's definitely not an LP issue.


Correct setup instructions for a monorepo w/ independent versions

The setup instructions for a monorepo with independently released versions mentions that the version of the root package needs to be formatted a certain way. This was true when we were first developing this tool, but we have changed this requirement since.

Had a lint violation 🤦🏻


(WIP) feat(react-await): add awaitOptions

Overview

awaitOptions (experimental)
add test for awaitOptions

PR Checklist

[ ] I did below actions if need

I read the Contributing Guide
I added documents and tests.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 84.15%. Comparing base (6e3deeb) to head (1bb549c).

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #1059      +/-   ##
==========================================
+ Coverage   84.09%   84.15%   +0.05%
==========================================
  Files          49       50       +1
  Lines         547      549       +2
  Branches      117      118       +1
==========================================
+ Hits          460      462       +2
  Misses         82       82
  Partials        5        5

Components
Coverage Δ

@suspensive/react
97.03% <ø> (ø)

@suspensive/react-query
75.28% <ø> (ø)

@suspensive/react-query-4
0.00% <ø> (ø)

@suspensive/react-query-5
0.00% <ø> (ø)

@suspensive/jotai
0.00% <ø> (ø)

@suspensive/react-await
100.00% <100.00%> (ø)

@suspensive/react-image
80.39% <ø> (ø)


Improvements to SuggestResult

It would be nice to see better parsing of the SuggestResult JSON so that things like the text, score and payload data within the options array could be accessed using conventional Java bean style methods.
Thanks,
Tom

Suggest API is removed and same functionality can be retrieved using the Search API.


Enable asset distance test

Moved the test to SolrSpec (which is where it belongs) and reenabled it.

LGTM


Can we highlight the color of the stacked bar?

First of all, thank you for this awesome package. Loved it. Anyway, I was wondering if we can highlight the bar in stacked bar chart, whenever user pressed it ?

Hi @prakashawal 👋
I guess you want to highlight (change the color of) a particular section of a bar as shown in this gif-

You can use the property onPress inside the stacks array of the data.
Here's the code for the above chart-
import React, {useState} from 'react';
import {BarChart} from 'react-native-gifted-charts';

const App = () => {
  const onFocusSection = (barIndex, sectionIndex, originalColor) => {
    const focusedColor = 'red';
    const newStackData = [...stackData];
    newStackData[barIndex].stacks[sectionIndex].color =
      newStackData[barIndex].stacks[sectionIndex].color === originalColor
        ? focusedColor
        : originalColor;
    setStackData(newStackData);
  };
  const data = [
    {
      stacks: [
        {
          value: 10,
          color: 'orange',
          onPress: () => onFocusSection(0, 0, 'orange'),
        },
        {
          value: 20,
          color: '#4ABFF4',
          marginBottom: 2,
          onPress: () => onFocusSection(0, 1, '#4ABFF4'),
        },
      ],
      label: 'Jan',
      // add onPress here for whole bar here
    },
    {
      stacks: [
        {value: 10, color: '#4ABFF4', onPress: () => onFocusSection(1, 0, '#4ABFF4'),},
        {value: 11, color: 'orange', marginBottom: 2},
        {value: 15, color: '#28B2B3', marginBottom: 2},
      ],
      label: 'Mar',
    },
    {
      stacks: [
        {value: 14, color: 'orange'},
        {value: 18, color: '#4ABFF4', marginBottom: 2},
      ],
      label: 'Feb',
    },
    {
      stacks: [
        {value: 7, color: '#4ABFF4'},
        {value: 11, color: 'orange', marginBottom: 2},
        {value: 10, color: '#28B2B3', marginBottom: 2},
      ],
      label: 'Mar',
    },
  ];

  const [stackData, setStackData] = useState(data);

  return (
    <BarChart width={340} rotateLabel noOfSections={4} stackData={stackData} />
  );
};

The onPress property can be used for a particular section of a bar or the whole bar.
I hope it helps.


Reverse conversation message order

Sort messages before displaying in chat read so time progresses down.
$ keybase chat read frank
Found CHAT conversation: frank,gil
fetching conversation frank,gil ...

[1]  [frank 25m] the beginnings of a conversation
[2]    [gil 24m] what a fine beginning it is
[3]  [frank 23m] shall we say more?
[4]    [gil 23m] sure. what's the latest message you've sent?
[5]  [frank 23m] this one!

The sort method is called ascending, but don't be fooled.
r? @mmaxim || @songgao

I agree probably not many people would use any non-default behavior either way.
Thanks for taking on updating your tests! Sorry for the terrible timing there.

NP!