[release/2.8] Fix panic in inmemory driver

Signed-off-by: Shengjing Zhu<EMAIL_ADDRESS>

Hm, right so looks like this fails because it's installing "github.com/vbatts/git-validation" "latest", which installs a version of golang.org/x/sys that's incompatible with go1.16
cd /tmp && go get -u github.com/vbatts/git-validation

....

# golang.org/x/sys/unix
Error: /home/runner/work/distribution/distribution/src/golang.org/x/sys/unix/syscall.go:83:16: undefined: unsafe.Slice
Error: /home/runner/work/distribution/distribution/src/golang.org/x/sys/unix/syscall_linux.go:1018:20: undefined: unsafe.Slice
Error: /home/runner/work/distribution/distribution/src/golang.org/x/sys/unix/syscall_linux.go:2297:9: undefined: unsafe.Slice
Error: /home/runner/work/distribution/distribution/src/golang.org/x/sys/unix/syscall_unix.go:118:7: undefined: unsafe.Slice
Error: /home/runner/work/distribution/distribution/src/golang.org/x/sys/unix/sysvshm_unix.go:33:7: undefined: unsafe.Slice
Error: Process completed with exit code 2.


TagHelpers not highlighted?

HEy All,
When I fire up a new web application project for ASP NET Core 1 none of the Tag helpers seem to light up and I do not get intellisense in my tags for asp-. Any ides on how to get that back? For example adding an asp-for to a div tag does not turn the asp-for the purple color it was I think in RC1.

@chassq Have you built the project? That's currently a requirement for TagHelper IntelliSense.

Same issue on my side

@Pictuel for a new web application unchanged?

I actually can see the highlighting and intellisense on a new project but on an upgraded project from RC1 I do not see the highlighting.

no, my application is a port from rc1, taghelpers seems to works fine when the app is running but are not higlighted nor intellisensed on vs

For an upgraded app you need to ensure the following: https://github.com/aspnet/Tooling/issues/484#issuecomment-219807702

still nothing for me, everything was set as you said except for:
"Microsoft.NETCore.App": {
"type": "platform",
"version": "1.0.0-*"
},

@Pictuel what dotnet are you using and what version of Microsoft.NETCore.App is being resolved?

Can find out the dotnet version by typing dotnet --info in the package manager console window in Visual Studio.
Cna find out the Microsoft.NETCore.App version by looking in your project.lock.json file.

my dotnet -v is 1.0.0-preview1-002702 (this is the version also set in my global.json)
and Microsoft.NETCore.App seems to be 1.0.0-rc2-3002702 (now set in project.json)

These entries look correct in my project.

Could both of you push a minimal repro to GitHub so I could take a look?

Hey, paired down a simple web site I had converted from RC1 to RC2. See attached zip file. You should see the issue if you navigate to the Views->Account-Login.cshtml file. At least when I open it the tag helpers are not highlighted nor do I get intellisense for asp-* attributes. Hope this helps!
PS Took out bower and node module folders so it will have to restore them.
SampleWebSite.zip

I'm afraid I don't have any project to share.

Somehow mine is highlighted now, probably a reboot made it work again, weird.

@chassq So I looked at your project and it seems your issue is tied to this bug which I have a PR out for.
Ultimately, to fix your issue change your @addTagHelper statements to be unquoted.
Aka @addTagHelper *, Microsoft.AspNetCore.Mvc.TagHelpers

Yep, that did the trick!! Thanks!!


[Question]  Listen multiple sockets

I need to run 3 sockets in one worker thread. So i have 3 sockets with different protocols:

income port (tcp) used to connect devices with custom protocol.
status socket (unix) used by monitoring service threads
command socket used by admin to send data to devices (unix)

I can run 3 workers, but i need to run them in single thread to have simple communication between sockets. When device change its state i can send message to all status clients, and when admin send commands via command socket them simply redirected to some connected device on main socket.
here is the suggested pseudo code:
$this->workers[$name] = $worker = new Worker( 'tcp://<IP_ADDRESS>:60291' );
$worker->protocol = DeviceProtocol::class;
$worker->name = 'main';
$app = $this->createApplication('income);
//here we set onConnect, onMessage, onError, onClose handlers.
$app->attach($worker);

$status_server = new TcpConnection('unix:///run/device_status.sock');
$status_server->protocol = MonitorProtocol::class;
$status_app = $this->createApplication('status');
//here we set onConnect, onMessage, onError, onClose handlers.
$status_app->attach($status_server);
$status_app->connect($app);

$command_server = new TcpConnection('unix:///home/admin/device_admin.sock');
$command_server->protocol = Jsonp::class;
$command_app = $this->createApplication('command');
//here we set onConnect, onMessage, onError, onClose handlers.
$command_app->attach($conn);
$command_app->connect($app);

$worker->onWorkerStart = function() use ($status_server, $command_server){
	$status_server->listen();
	$command_server->listen();
}

But i`ve found that all connection classes are client side only, is there any way to create listening sockets using Workerman framework or should i use other libs like reactphp for this?

Yes, you can do that like this.
$worker = new Worker( 'tcp://<IP_ADDRESS>:60291' );
$worker->onWorkerStart = function() {
    global $status_server, $command_server;
    $status_server = new Worker('unix:///run/device_status.sock');
    $status_server->onMessage = function (){....};
    $status_server->listen();

    $command_server = new Worker('unix:///home/admin/device_admin.sock');
    $command_server->onMessage = function (){....};
    $command_server->listen();
};
$worker->onMessage = function(){....};

Nice, thank you!

Hi Walkor,
How can we implement this in gateway worker solution.
Can you send me the example.
Thank you


Where is the example to generate the ckpt file?

@liuliu  i want to not use the sqllite file, but i have the weights in memroy, how do i set the weights of the model, if i already have the weights in the memory?

In examples/unet/main.swift has an example.


Managing branches and environments

Which Category is your question related to?*
all of them
Amplify CLI Version
4.10.0
I'm following the "Environments and Teams" part of the documentations.
When I add a function, for example, i want to add it to another branch once my testing is done. So I check out the branch using git, then i merge the branch where i have the new function into the "new" branch. I don't see the new function listed in amplify status before or after the push. The functions are not in the team provider info json file either.
My maind development branch is built using the amplify console but my secondary branch is pushed via CLI.
How should I manage this use case?

@kldeb After you merge the changes to your git branch, did you do amplify env checkout <env-name>?  The functions are in backend-config.json

After amplify env checkout <env-name>, the functions don't show up in the amplify-meta.json file?

@kaustavghosh06 I just ran env checkout now and it shows up (after the push) and now the status indicates that the functions will be created.

@kldeb Okay, cool. I just tried re-producing this to see if this is an issue but the functions showed up after amplify env checkout <env-name in another env.
Let me know if you're able to reproduce it with any other resource or category.

Closing this issue. Please let me know if you're able to reproduce.


Error response code not compliant to RFC 6749

What version of UAA are you running?
76.22.0 (latest)
How are you deploying the UAA?
as part of a commercial Cloud Foundry distribution
What did you do?
I am trying to generate access tokens using the token endpoint - {authenticationUrl}/oauth/token
What did you expect to see? What goal are you trying to achieve with the UAA?
with invalid client credentials, the error response was expected to be as follows:

Code: 401 Unauthorized
Body:
- “error”: “invalid_grant”

“error_description”:  “Bad credentials”

What did you see instead?
with invalid client credentials, the error response is as follows:

Code: 401 Unauthorized
Body:
- “error”: “unauthorized”

“error_description”:  “Bad credentials”

Question 1: in RFC6749 section 5.2 -  “unauthorized” is not in the list of allowed ‘error codes’. We should use an error response code from this available list of codes.
Question 2: in RFC6749 section 5.2 - suggests a 400 unless specified otherwise. Any reason why we use a 401?

Is there any update on the same?

Hi @neelalex
Ok, I simply checked client credential agains some others.
Others reply with
401
{"error":"invalid_client","error_description":"Client authentication failed"}
Azure tells a long story in error_description, but also invalid_client in error
So yes, we should fix it in UAA.
Are you able to create a fixing PR ? This would increase the time until we get the fix.

@Tallicia FYI, BadCredential in UAA is not from OAuth2Exception , so we should move to own Exception . In spring-security-oauth2 library there would be an extra Exception -> BadClientCredentialsException, but this is depcrecated because the library is depcrecated... so we need to do it on our own

Hi team,
I can see there is a Pull Request open, when can we expect the fix to be available and in which version?

yes the PR is in review, it is small but we need some time. We will change only the error text, but if a client would use exsisting error text, it would fail

created https://github.com/cloudfoundry/uaa/pull/2596 as alternative

@neelalex we will fix this soon with PR https://github.com/cloudfoundry/uaa/pull/2596 , please check if this would solve it for you


review: conservative_discrete_laplacian_tail_to_alpha

Close #2061

#2124  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @Shoeboxam and the rest of your teammates on  Graphite


Add start and stop commands to Docker task

Task name:
DockerV2
Description:
Adds a stop and start command for containers
Documentation changes required: (Y/N)
Y
Added unit tests: (Y/N)
Y
Attached related issue: (Y/N)
N
Checklist:

[x] Task version was bumped - please check instruction how to do it
[x] Checked that applied changes work as expected

cc: @ajinkya599 to take a look at this PR. Also while you're at it could you check with our PM as well?


[KOGITO-951] Refactor KubernetesServiceDiscoveryIntegrationTest to us…

…e mocked OpenShift client
https://issues.redhat.com/browse/KOGITO-951
Signed-off-by: Karel Suta<EMAIL_ADDRESS>

Jenkins retest this

@spolti Can you please take a look and possibly merge?

@spolti  Good catch, I completely overlooked that :)
PR adjusted


Upgrade to Android 11

Update the samples to target Android 11, as well as the templates.
Things to do:

[ ] Re-enable Maps sample in AllControls
[ ] Ensure all samples fully work
[ ] Ensure templates work as expected on both macOS and Windows with the latest IDEs updates
[ ] Ensure F# 5 can be used

/azp run full build

@TimLariviere what is wrong with fabulousweather?

@SergejDK It was complaining that Xamarin.Forms.targets was imported 2 times when it was not. I got frustrated after fighting with it for 1 hour. 😅
I'll readd it after I'm done with the rest.

/azp run full build

@TimLariviere does this maybe have something to do with the other issue and android? maybe this is a bug, too from Xamarin.Android and the tooling?
Do you still have the older VS version installed and can check this ? Looks similar to me...

I don't think it's linked to the issue with VS.
I'm working on macOS with the latest versions of VS Mac and Rider.
I think it's due to an invalid state in the Fabulous solution that forces the IDEs to load the FabulousWeather project 2 times.

/azp run full build

/azp run full build

/azp run full build

/azp run full build

The hosted build agents are not yet updated to build .NET 5.0 apps...
Apparently, they will be updated next week.
https://developercommunity.visualstudio.com/content/problem/1255480/building-net-50-in-a-pipeline-doesnt-work.html
I will revert that change in the meantime.

/azp run full build

/azp run full build

/azp run full build

/azp run full build


Add AWSSigv4Signing

Issue #, if available:
PR for partial changes - raising to avoid delays during my OOTO

Using jetpack.io for aws-request-signing-apache-interceptor because there is no maven published jar - refer https://github.com/awslabs/aws-request-signing-apache-interceptor/issues/2
There is no simple Junit as it requires credentials and works fine (on individual machines as most will have .aws/Credentials configured on their local - TODO Use the default chain to include a junit.
ToDo: Testing

Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Closing this PR for now, raised PR against the v0.7.x branch for patch release.

Closing this PR for now, raised PR against the v0.7.x branch for patch release.


SOCManager present animation speed

Version: SlideOverCard 2.0.0, Xcode 13.3
Issue: Unnaturally fast animation when displaying a card using SOCManager.present vs  .slideOverCard(isPresented.
I would expect both versions to display the card at the same animation speed. The SOCManager presents the card so quickly that it feels off.
Potential Quick Fix:
Give the SOCManager the same animation speed as the slideOverCard
Potential Enhancement:
Add and option to change the speeds between slow, medium, fast, and or passing in custom animation speeds
Code snippets:
Button {
    showLogIn = true

    SOCManager.present(isPresented: $showLogIn, onDismiss: {
       // code
    }, options: [.disableDrag, .disableDragToDismiss, .hideExitButton]) {
      SomeView(isPresented: $showLogIn)
    }
  }
}

.slideOverCard(isPresented: $showView, options: [.disableDrag, .disableDragToDismiss, .hideExitButton]) {
  SomeView(isPresented: $showSummary)
}

Thank you for your time.

same problem here

I have the same problem with SOCManager.present, which is presented too fast. Are there plans to fix this issue?
Environment:
Xcode version: 15.0.1
SlideOverCard version: 2.1.0


chore: (sample app) enable switching user and tag during runtime.

Changes

No need to restart the sample app after changing the user, tag
Update the demo app following up the documents
Add a picker for selecting difference types of variants for testing (int, boolean, string, double, JSON)

@cre8ivejp please help me to take a look.


Did you plan to used force-riscv to generate random-instruction in Ibex's dv env?

Hi, as I known, force-riscv can generate random-instruction more faster than riscv-dv because of its base on C++. Did you have a plan to used force-riscv to generate random-instruction in your dv env next up?  It can coexist with riscv-dv. We can only replace the part of instruction generator.

I hadn't come across force-riscv before, just had a brief look and it does look like a capable instruction generator.
We have no plans to move away from riscv-dv, in particular speed of instruction generation isn't much of a concern, riscv-dv is fast enough for our purposes and supports what we require for Ibex. However force-riscv has the advantage of not requiring a full system verilog simulator to run, allowing its use with a verilator based testbench. So it may be of interest to us in the future.
Our current DV environment isn't intimately tied to riscv-dv, so using force-riscv in its place is certainly possible though not entirely trivial.

Ok. I have got it. Thanks!


fix: Fixed zia_dlp_web_rules new parent_rule and sub_rules attributes

fix: Fixed zia_dlp_web_rules new parent_rule and sub_rules attributes

Codecov Report
Attention: Patch coverage is 66.66667% with 18 lines in your changes are missing coverage. Please review.

Project coverage is 71.77%. Comparing base (dd7f7d5) to head (65abccb).

Additional details and impacted files

@@            Coverage Diff             @@
##           master     #345      +/-   ##
==========================================
- Coverage   72.19%   71.77%   -0.42%
==========================================
  Files          79       79
  Lines       12558    12468      -90
==========================================
- Hits         9066     8949     -117
- Misses       2864     2904      +40
+ Partials      628      615      -13

Files
Coverage Δ

zia/data_source_zia_admin_roles.go
96.19% <66.66%> (+5.11%)
:arrow_up:

...data_source_zia_application_services_group_lite.go
89.18% <66.66%> (+11.91%)
:arrow_up:

zia/data_source_zia_application_services_lite.go
89.18% <66.66%> (+11.91%)
:arrow_up:

...data_source_zia_cloud_browser_isolation_profile.go
90.69% <50.00%> (+10.69%)
:arrow_up:

zia/data_source_zia_device_group.go
94.02% <66.66%> (+7.54%)
:arrow_up:

zia/data_source_zia_fw_filtering_time_window.go
91.11% <66.66%> (+10.34%)
:arrow_up:

zia/data_source_zia_dlp_web_rules.go
92.58% <20.00%> (-1.29%)
:arrow_down:

zia/resource_zia_dlp_web_rules.go
47.33% <75.00%> (-10.45%)
:arrow_down:

... and 1 file with indirect coverage changes


etcdserver: get rid of unreliable WaitSchedule

In this case, we know we are waiting for an action happened on
storage. We can do a busy wait instead of calling waitSchedule.
The test previously failed on CI with no observed actions.
Fix #3793.

/cc @gyuho @eyakubovich @jonboulle

LGTM.


feat(applications-service-api): get applications searching

Ticket: https://pins-ds.atlassian.net/browse/ASB-1752
Adds searchTerm parameter to search for PromoterName and ProjectName in get applications endpoint.

:tada: This PR is included in version 2.15.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


remove duplicate comment

Forgive me for the late merge @nrgwsth! I don't know why or how, but the notification for this PR wen't by unnoticed and I missed it.
Sorry for that! (mea culpa)


Is there any way tho edit an existing query?

I'm creating a CRUD, for the user to edit a previous saved query.
Placing the query on the $scope.option.query variable is not working. Is there any other way I can populate the queryBuilder to create the elements automatically?

IMO, you could get the value of scope.options._query and feed it on your needs.
Feel free to create a PR if needed.


Fixed issue when registry creds are provided for ACRs using containerapp up command

Description
This PR aims to fix an issue with current container app up command in the scenario where user provides --registry-server, --registry-password, and --registry-username for ACRs. In the past, up command will treat it as ACRs in the current subscription and apply them. However, this results in issues when user starts to use ACRs cross subscriptions.
Before

After

Related command

az containerapp up
General Guidelines

[x] Have you run azdev style <YOUR_EXT> locally? (pip install azdev required)
[x] Have you run python scripts/ci/test_index.py -q locally? (pip install wheel==0.30.0 required)
[ ] My extension version conforms to the Extension version schema

For new extensions:

[ ] My extension description/summary conforms to the Extension Summary Guidelines.

About Extension Publish
There is a pipeline to automatically build, upload and publish extension wheels.
Once your pull request is merged into main branch, a new pull request will be created to update src/index.json automatically.
You only need to update the version information in file setup.py and historical information in file HISTORY.rst in your PR but do not modify src/index.json.

Thank you for your contribution! We will review the pull request and get back to you soon.

The change looks good to me overall. However, could you please clarify: was this a regression? If yes, do you know when/how it was introduced?

Another question: could you please clarify if this fix should be in the extension (this repo), or in the base Azure CLI?
https://github.com/Azure/azure-cli/blob/main/src/azure-cli/azure/cli/command_modules/containerapp/_up_utils.py
At this point some of our changes are checked in in the Extensions repo (mostly for preview features), while all the base features are in the base Azure CLI repo so that customers can get these features without installing an extra extension.

@daniv-msft
From just checking the file history, it was marked as TODO but just left there when the method was first created. Please let me know if I understand and answered the question correctly.
Meanwhile, I think it would be better to open the PR in base repo since this is just a bug fix on an existing public feature rather than a preview feature. I'll close this PR for now and move it to the az cli repo. Thanks for the reminder!

Please update history.rst

Hi @harryli0108
The following test is failed because of the api-version of containerapp has been update from 2023-08-01-preview to 2023-11-02-preview, please rerun the test. Thanks.
FAILED src/containerapp/azext_containerapp/tests/latest/test_containerapp_up_on_connected_env.py::ContainerAppUpConnectedEnvImageTest::test_containerapp_up_on_arc_e2e

 No match for the request (<Request (GET) [https://management.azure.com/subscriptions/00000000-0000-0000-0000-000000000000/resourceGroups/clitest.rg000001/providers/Microsoft.App/containerApps?api-version=2023-11-02-preview>](https://management.azure.com/subscriptions/00000000-0000-0000-0000-000000000000/resourceGroups/clitest.rg000001/providers/Microsoft.App/containerApps?api-version=2023-11-02-preview%3E)) was found.
E           Found 9 similar requests with 1 different matcher(s) :
E
E           1 - (<Request (GET) [https://management.azure.com/subscriptions/00000000-0000-0000-0000-000000000000/resourceGroups/clitest.rg000001/providers/Microsoft.App/containerApps?api-version=2023-08-01-preview>).](https://management.azure.com/subscriptions/00000000-0000-0000-0000-000000000000/resourceGroups/clitest.rg000001/providers/Microsoft.App/containerApps?api-version=2023-08-01-preview%3E).)
E           Matchers succeeded : ['method', 'scheme', 'host', 'port', 'path']
E           Matchers failed :
E           _custom_request_query_matcher - assertion failure :
E           None

Hi @harryli0108
Please merge the latest code from main branch.
FAILED src/containerapp/azext_containerapp/tests/latest/test_containerapp_commands.py::ContainerappServiceBindingTests::test_containerapp_dev_service_binding_customized_keys_e2e
FAILED src/containerapp/azext_containerapp/tests/latest/test_containerapp_commands.py::ContainerappServiceBindingTests::test_containerapp_dev_service_binding_customized_keys_yaml_e2e
FAILED src/containerapp/azext_containerapp/tests/latest/test_containerapp_commands.py::ContainerappServiceBindingTests::test_containerapp_dev_service_binding_e2e
FAILED src/containerapp/azext_containerapp/tests/latest/test_containerapp_commands.py::ContainerappServiceBindingTests::test_containerapp_dev_service_binding_none_client_type
FAILED src/containerapp/azext_containerapp/tests/latest/test_containerapp_up_on_connected_env.py::ContainerAppUpConnectedEnvImageTest::test_containerapp_up_on_arc_e2e

These tests failed:
FAILED src/containerapp/azext_containerapp/tests/latest/test_containerapp_commands.py::ContainerappServiceBindingTests::test_containerapp_dev_service_binding_customized_keys_e2e
FAILED src/containerapp/azext_containerapp/tests/latest/test_containerapp_commands.py::ContainerappServiceBindingTests::test_containerapp_dev_service_binding_customized_keys_yaml_e2e
FAILED src/containerapp/azext_containerapp/tests/latest/test_containerapp_commands.py::ContainerappServiceBindingTests::test_containerapp_dev_service_binding_e2e
FAILED src/containerapp/azext_containerapp/tests/latest/test_containerapp_commands.py::ContainerappServiceBindingTests::test_containerapp_dev_service_binding_none_client_type
FAILED src/containerapp/azext_containerapp/tests/latest/test_containerapp_up_on_connected_env.py::ContainerAppUpConnectedEnvImageTest::test_containerapp_up_on_arc_e2e
============ 5 failed, 107 passed, 36 skipped in 496.07s (0:08:16) ============

I will handle the others but do not include test_containerapp_up_on_arc_e2e.
test_containerapp_dev_service failed because command az containerapp service is deprecated.

Where is test test_containerapp_up_artifact_with_bullseye_buildpack_e2e?

Hi @wangzelin007
Can you help to merge this PR? Thanks.


[Question]Is there a version of tensorflow 1.15 with a merlin-tensorflow image of sok installed?

This template is for generic questions that a user may have in using HugeCTR.
Note: Before filing an issue, you may want to check out our compiled Q&A list first.

Hi @recklessnolove ，Merlin has not released the TF1 image for a long time and does not have the latest image for installing SOK. You need to manually install SOK(https://github.com/NVIDIA-Merlin/HugeCTR/tree/main/sparse_operation_kit#installing-sok-from-source) based on the TF1 image(nvcr.io/nvidia/tensorflow:23.03-tf1-py3) from NVCR.


RPC methods that lookup block by hash will now return error if no block found

Previously were returning null in this case eg ethGetCode
New behavior matches geth 1.10.25
Do we consider this a breaking change?
Documentation

[ ] I thought about documentation and added the doc-change-required label to this PR if
updates are required.

Changelog

[x] I thought about the changelog and included a changelog update if required.

Furthermore this touches on endpoints that use the "default block parameter."  Querying by Hash is not a valid option.  The data type is "BlockNumberOrTag" - https://github.com/ethereum/execution-apis/blob/main/src/eth/state.yaml#L60
And the type does not accept hashes - https://github.com/ethereum/execution-apis/blob/main/src/schemas/block.yaml#L92-L108
So if there was movement towards the spec conformance direction that step would be to not return answers when looking up by hash even when one is found.

I argue this (and geth) are out of spec
https://ethereum.org/en/developers/docs/apis/json-rpc/#eth_getblockbyhash

Returns
Object - A block object, or null when no block was found:
[... description of fields ...]

This change was made here https://github.com/hyperledger/besu/pull/1784 https://github.com/ethereum/EIPs/blob/master/EIPS/eip-1898.md

There has been further conversation here https://discord.com/channels/595666850260713488/595701319793377299/1041758135821533266 indicating that it should be part of the spec.
EIP-1898 - changing status from stagnant to draft https://github.com/ethereum/EIPs/pull/5980 and adding relevant parts to the spec https://github.com/ethereum/execution-apis/pull/326

Update on this

EIP 1898 has moved (from stagnant) back to draft https://github.com/ethereum/EIPs/blob/master/EIPS/eip-1898.md
EIP 1898 requires EIP 1474 - PR here to move it to draft also https://github.com/ethereum/EIPs/pull/5986 - waiting on review
changes to spec to allow block hash param also waiting on review https://github.com/ethereum/execution-apis/pull/326 | raised in Eth R&D discord https://discord.com/channels/595666850260713488/804019759934078987

This has been merged https://github.com/ethereum/execution-apis/pull/326 so next step is to verify that (with this PR) besu matches the spec

The official execution spec is disturbingly void of standard error handing. I'll approve this on the presumption that this is not at odds with how other clients handle such an exception.

Thanks @shemnon - this PR actually arose out of an effort to standardise error handling across clients https://github.com/ethereum/execution-apis/issues/286


Community Chat

As Hauler moved from RGS org to hauler-dev, I believe there was a Hauler slack inside rancher-users.slack.com. Is it planned to have a separate slack or group to chat within?
Docousarus docs config still references rancher-users.slack.com.

Thank you for pinging us about this... definitely overdue!
#hauler has been created in rancher-users --> https://rancher-users.slack.com/archives/C07F009GNRY


cjxl generates 16-bit float from 32-bit float exr input

Describe the bug
cjxl generates 16-bit float from 32-bit float exr input.  pfm  files are not affected.
To Reproduce

Convert image to 32-bit float in GIMP and export as an exr file.
Confirm image is 32-bit float using exrheader.
Use cjxl to compress file in both lossy and lossless modes.
Use jxlinfo to examine bits_per_sample and exponent_bits_per_sample.

Result is (16, 5).  Expected is (32, 8).

Expected behavior
jxl file should have the same pixel format as the original files (32-bit float in this case).
Environment

OS: Ubuntu 21.04; Linux 5.11.??
Compiler version: gcc 10.3.0
CPU type: x86_64
cjxl/djxl version string: 0.6.0 810ecc3

Additional context
It appears codec_exr.cc has 16-bits hard coded for exr files, rather than detecting the number of bits.
Note: I'm just playing with jxl capabilities.  I don't have any need for 32-bit float files.

Looks like what is needed here is to fix codec_exr.cc to detect the correct float format and pass that to the encoder. It works for pfm so it should also be possible to get it to work for exr.

I did some searching, and it looks like the OpenEXR API call that's being used supports only float16 RGB.  To get float32 or grayscale image data requires using a more complicated API call.

That's annoying. Well, I guess as a workaround you can use pfm?

@jonsneyers  With pfm, there's the issue I opened then closed (#463).  Different programs treat the data differently.  GIMP treats it as linear data.  Imagemagick treats it as non-linear data.  Whether the ICC profile is used doesn't make any difference.
Maybe someone more familiar with the OpenEXR API will eventually fix this?


I think that this needs a change..

Our staking portal does not work with the current version of lucid, I believe that this change is what is necessary in order to return functionality, we will simply offset tx bounds as required for now.
I could not (in a short time) find where these offsets can actually be referenced in any literature, but https://github.com/ADAOcommunity/round-table/blob/main/src/cardano/utils.ts uses a time similar to the one I have used in this PR.

This PR actually has a mistake as well, I am going to try to find the correct number. I am still convinced that there is a change required here.


[WIRES] Make qchem templates wire independent

Context: Part of the overall wire management refactor.
The new qchem templates use a logic of consecutive wires. For example, given two wires p, h they generate a range(p, h) and expect that every entry in that range is also a valid wire. If we want to make PennyLane independent of this assumption the templates need to be refactored.
Description of the Change:
Change the input arguments of the templates to take the entire list of wires that the template acts on.
Benefits: Makes templates more flexible.

@agran2018 I would need your help adapting the docstrings and naming the arguments in a sensible manner :)

I don't know why the test file has so many code format changes, I didn't black it...sorry for that.


Fix readme

Coverage remained the same at 98.864% when pulling 0417a9420e6da4b896a86bd0ca94208d3876ea4a on test into cec18b54950bce9ee802bf0b4ebe6d1d7e7b825b on master.


[AutoParallel] Add FLAGS_enable_fused_ffn_qkv_v2 for llama

PR types

Others
PR changes

Others
Description

Add FLAGS_enable_fused_ffn_qkv_v2 for llama

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


(EAI-598) [UI] Investigate further code splitting

Jira: (EAI-598) [UI] Investigate further code splitting
Changes

Adds some additional code splitting & lazy loading to some components inside of the chat window view
Note the build output includes more, smaller, chunks. There is still some significant heft to the Message component that I think comes from the LG Chat markdown. Need to look upstream for a fix there.

Before (current main):
...
build/es/ConversationProvider.js            67.30 kB │ gzip:  16.89 kB
build/es/lodash.js                         121.75 kB │ gzip:  30.95 kB
build/es/index3.js                         173.01 kB │ gzip:  35.20 kB
build/es/index7.js                         181.52 kB │ gzip:  54.19 kB
build/es/messageLinks.js                   210.30 kB │ gzip:  41.70 kB
build/es/index2.js                         313.03 kB │ gzip:  83.32 kB
build/es/ui-text.js                        328.27 kB │ gzip:  66.02 kB
build/es/ChatWindow.js                   2,060.51 kB │ gzip: 418.91 kB
✓ built in 5.17s

After (this PR):
...
build/es/ConversationProvider.js            67.33 kB │ gzip:  16.89 kB
build/es/ChatWindow.js                      75.68 kB │ gzip:  13.25 kB
build/es/ChatMessageFeed.js                 77.13 kB │ gzip:  16.42 kB
build/es/lodash.js                         120.77 kB │ gzip:  30.56 kB
build/es/index13.js                        131.94 kB │ gzip:  44.06 kB
build/es/index6.js                         160.88 kB │ gzip:  31.62 kB
build/es/index15.js                        202.81 kB │ gzip:  39.37 kB
build/es/InputBar.js                       305.09 kB │ gzip:  61.32 kB
build/es/index2.js                         313.00 kB │ gzip:  83.31 kB
build/es/MessageRating.js                  612.44 kB │ gzip: 116.19 kB
build/es/MessageContent2.js              1,201.38 kB │ gzip: 262.81 kB
✓ built in 5.02s

what do you think is the actual performance impact of this

Honestly pretty minimal, though we'd need to test to get actual numbers. Should have some small effect though.


nan after auto_symbolic and training

I train a KAN using 19 inputs, 5 hidden neurons, and 1 output (I know that I only need a subset of this input, and I was hoping the KAN will tell me which).
I train by refining the grid 5, 10, 20, 50 with sub-optimal results:
train loss: 6.33e+01 | test loss: 6.56e+01 | reg: 2.68e+02 : 100%|██| 50/50 [00:56<00:00,  1.13s/it]
train loss: 6.23e+01 | test loss: 6.81e+01 | reg: 2.95e+02 : 100%|██| 50/50 [01:03<00:00,  1.26s/it]
train loss: 6.14e+01 | test loss: 7.05e+01 | reg: 3.03e+02 : 100%|██| 50/50 [01:27<00:00,  1.76s/it]
train loss: 6.59e+01 | test loss: 2.32e+02 | reg: 6.24e+02 : 100%|██| 50/50 [01:55<00:00,  2.31s/it]

I run auto_symbolic as shown in the tutorial, and get all of the nodes "fixed"
fixing (0,0,0) with x^2, r2=0.9999875381797473
fixing (0,0,1) with log, r2=0.9995437326827541
fixing (0,1,0) with tanh, r2=0.9190223510394673
fixing (0,1,1) with abs, r2=0.591284704723076
fixing (0,2,0) with tanh, r2=0.9859092016712654
fixing (0,2,1) with tanh, r2=0.2644378732312754
fixing (0,3,0) with abs, r2=0.8094657967145165
fixing (0,3,1) with sin, r2=0.20524626203198593
fixing (0,4,0) with sin, r2=0.8596364823055564
fixing (0,4,1) with abs, r2=0.7073148948571956
fixing (0,5,0) with sin, r2=0.8451701555686204
fixing (0,5,1) with tanh, r2=0.12906659532236966
fixing (0,6,0) with sin, r2=0.5258884938324261
fixing (0,6,1) with sin, r2=0.4188728101754649
fixing (0,7,0) with sin, r2=0.9013293203443674
fixing (0,7,1) with tanh, r2=0.7815785647587229
fixing (0,8,0) with tanh, r2=0.9598249300435913
fixing (0,8,1) with sin, r2=0.874354100172932
fixing (0,9,0) with sin, r2=0.8688442004450128
fixing (0,9,1) with sin, r2=0.8079476861665728
fixing (0,10,0) with sin, r2=0.511931841317213
fixing (0,10,1) with abs, r2=0.5395735955860584
fixing (0,11,0) with abs, r2=0.8789512062476048
fixing (0,11,1) with sin, r2=0.9865673967925179
fixing (0,12,0) with x^2, r2=0.9392742452969135
fixing (0,12,1) with sin, r2=0.7497273750201916
fixing (0,13,0) with sin, r2=0.8426054543213266
fixing (0,13,1) with tanh, r2=0.27823616049107963
fixing (0,14,0) with tanh, r2=0.9149009145289683
fixing (0,14,1) with sin, r2=0.8631999116146144
fixing (0,15,0) with sin, r2=0.40641405950097254
fixing (0,15,1) with sin, r2=0.7259583807317198
fixing (0,16,0) with tanh, r2=0.3861879500950182
fixing (0,16,1) with tanh, r2=0.7091527502138572
fixing (0,17,0) with x^2, r2=0.7360096867746375
fixing (0,17,1) with sin, r2=0.9544954283288226
fixing (0,18,0) with sin, r2=0.8808563153924848
fixing (0,18,1) with sin, r2=0.8349780876453031
fixing (1,0,0) with tanh, r2=0.9440596493618962
fixing (1,1,0) with tanh, r2=0.5857742023383322

But then when training again, I immediately get nan:
train loss: nan | test loss: nan | reg: nan :  10%|█▊                | 5/50 [00:08<01:15,  1.67s/it]

Intel MKL ERROR: Parameter 6 was incorrect on entry to DGELSY.

Traceback (most recent call last):
  File "/Users/amitmoryossef/dev/sign-language-processing/mediapipe-hand-crop-fix/mediapipe_crop_estimate/train_kan.py", line 90, in <module>
    model.train(dataset, opt="LBFGS", steps=50)
  File "/opt/homebrew/anaconda3/lib/python3.9/site-packages/kan/KAN.py", line 913, in train
    self.update_grid_from_samples(dataset['train_input'][train_id].to(device))
  File "/opt/homebrew/anaconda3/lib/python3.9/site-packages/kan/KAN.py", line 243, in update_grid_from_samples
    self.act_fun[l].update_grid_from_samples(self.acts[l])
  File "/opt/homebrew/anaconda3/lib/python3.9/site-packages/kan/KANLayer.py", line 220, in update_grid_from_samples
    self.coef.data = curve2coef(x_pos, y_eval, self.grid, self.k)
  File "/opt/homebrew/anaconda3/lib/python3.9/site-packages/kan/spline.py", line 136, in curve2coef
    coef = torch.linalg.lstsq(mat.to('cpu'), y_eval.unsqueeze(dim=2).to('cpu')).solution[:,:,0] # sometimes 'cuda' version may diverge
RuntimeError: false INTERNAL ASSERT FAILED at "/Users/runner/work/pytorch/pytorch/pytorch/aten/src/ATen/native/BatchLinearAlgebra.cpp":1539, please report a bug to PyTorch. torch.linalg.lstsq: (Batch element 0): Argument 6 has illegal value. Most certainly there is a bug in the implementation calling the backend library.

If I only train once, with 5 grids, it does train even after auto_symbolic

I'm having the same problem bro.QAQ

It retrain after symbolic regression indeed can be tricky sometime. because of this line
fixing (0,0,1) with log, r2=0.9995437326827541

logairthim is not defined if its input is <=0.
I mentioned this problem in this examples, and it is unclear to me how to fix this in general. However, two hacky ways to fix this is: if you don't have strong reasons why to keep those potentially singular functions, you could
(1) inspect and replace those functions that potentially lead to singularity (log,sqrt, x^-1etc.) viafix_symbolic(0,0,1,f)wherefcan be any function from the top of the list if you runsuggest_symbolic(0,0,1). For example, if you found x^2can also fit (0,0,1) quite well, you may runfix_symbolic(0,0,1,'x^2')`.
(2) you may remove singular functions from your symbolic library in the first place, e.g., you confine your symbolic formulas only to sin, squared, and exp.
lib = ['sin', 'x^2', 'exp']
model.auto_symbolic(lib=lib)

I had the same problem when I tried to run hellokan.ipynb, but it worked with the optimizer set to Adam.
model.train(dataset, opt="Adam", steps=50);

I think the nan values come from overflow values after a few iterations under some setups, for example if a large k is used, the following lines would report that there is an illegal value:
[c:\...\pykan\venv\Lib\site-packages\kan\KANLayer.py:220](file:///C:/Desktop/School/Papers/extra/pykan/venv/Lib/site-packages/kan/KANLayer.py:220), in KANLayer.update_grid_from_samples(self, x)
    [218] grid_uniform = torch.cat([grid_adaptive[:, [0]] - margin + (grid_adaptive[:, [-1]] - grid_adaptive[:, [0]] + 2 * margin) * a for a in np.linspace(0, 1, num=self.grid.shape[1])], dim=1)
    [219] self.grid.data = self.grid_eps * grid_uniform + (1 - self.grid_eps) * grid_adaptive
--> [220] self.coef.data = curve2coef(x_pos, y_eval, self.grid, self.k, device=self.device)
...
    [134] mat = B_batch(x_eval, grid, k, device=device).permute(0, 2, 1)
--> [135] coef = torch.linalg.lstsq(mat.to('cpu'), y_eval.unsqueeze(dim=2).to('cpu')).solution[:, :, 0]  # sometimes 'cuda' version may diverge
    [136] return coef.to(device)

Not sure how this can occur though, there are way too many code to go through xd