[Bug]: Selfhosted Postgresql could not obtain lock on row in relation "current_states"

Preflight Checklist

[X] I could not find a solution in the documentation, the existing issues or discussions
[ ] I have joined the ZITADEL chat

Environment
Self-hosted
Version
v2.49.1
Database
PostgreSQL
Database Version
postgres:16-alpine
Describe the problem caused by this bug
Functionality seems to be not affected, but I wonder what this means or breaks.
I.e. I was not able to use triggers and add static claims as stated within documentation... but this could be something else.
Using opague tokens with service user (basic auth + jwt private key)
Thank you!
db-1       | 2024-04-XX 03:55:47.384 UTC [658] ERROR:  could not obtain lock on row in relation "current_states"
db-1       | 2024-04-XX 03:55:47.384 UTC [658] STATEMENT:  SELECT
db-1       |        aggregate_id
db-1       |        , aggregate_type
db-1       |        , "sequence"
db-1       |        , event_date
db-1       |        , "position"
db-1       |        , filter_offset
db-1       |    FROM
db-1       |        projections.current_states
db-1       |    WHERE
db-1       |        instance_id = $1
db-1       |        AND projection_name = $2
db-1       |    FOR UPDATE NOWAIT;
zitadel-1  | time="2024-04-XXT03:55:47Z" level=info msg=activity caller="/home/runner/work/zitadel/zitadel/internal/activity/activity.go:105" domain="https://iam.mcp.buscher.xyz" grpcStatus= httpStatus= instance=<id> isSystemUser=false method= org=<id>path=/oauth/v2/token requestMethod=POST trigger=accessToken user=<id>

To reproduce
Using opague tokens with service user (basic auth + jwt private key) and request a token for a serviceuser
POST {{baseUrl}}/oauth/v2/token HTTP/1.1
Content-Type: application/x-www-form-urlencoded
Authorization: Basic {{user}}:{{password}}

grant_type=client_credentials&scope=openid profile urn:zitadel:iam:org:project:id:{projectId}:aud

Screenshots
No response
Expected behavior
No response
Operating System
No response
Relevant Configuration
Tried recreate with --init-projections=true (without deleting data)
Additional Context
No response

have the same issue with PostgreSQL 15 (Cloud SQL on Google Cloud).
Here is a list of errors that I can observe in the pod log

time="2024-04-09T14:59:12Z" level=info msg="process events failed" caller="/home/runner/work/zitadel/zitadel/internal/eventstore/handler/v2/handler.go:416" error="sql: transaction has already been committed or rolled back" projection=projections.apps6

time="2024-04-09T14:59:16Z" level=error msg="statement execution failed" caller="/home/runner/work/zitadel/zitadel/internal/eventstore/handler/v2/handler.go:629" error="ID=CRDB-pKtsr Message=exec failed Parent=(sql: transaction has already been committed or rolled back)" projection=projections.apps6

time="2024-04-09T14:59:16Z" level=warning msg="unable to get failure count" aggregate=261811373993033834 caller="/home/runner/work/zitadel/zitadel/internal/eventstore/handler/v2/failed_event.go:53" error="ID=CRDB-Unnex Message=unable to update failure count Parent=(sql: transaction has already been committed or rolled back)" instance=261811373992837226 projection=projections.apps6 sequence=14

time="2024-04-09T14:59:16Z" level=warning msg="unable to update state" caller="/home/runner/work/zitadel/zitadel/internal/eventstore/handler/v2/state.go:96" error="sql: transaction has already been committed or rolled back" projection=projections.apps6

Thanks for raising this. Actually we have spotted such errors as well on our GCP infra.
Let us look into this, but ATM we assume that it does not have an immediate harm.

@fforootd the harm is it rockets the cpu usage, makes it unusable

@pot-code what version are you using with what config?

@fforootd
OS: windows11 Home Edition 23H2
Env: docker
Zitadel: ghcr.io/zitadel/zitadel:latest
DB: postgres:16-alpine
docker compose file was pulled from Zitadel doc.
it happens after updating the zitadel image from v2.41 (if i remember it correctly), then docker recreted the container and boom
after a reboot the problem is gone and now i'm struggling with the FATAL:  role "root" does not exist and could not obtain lock on row issues

Same problem here

@fforootd guess i figured it out, it may be caused by the double render in React (when StrictMode is turned on).
the lock error is gone after removing it

Do you both see the same if you deploy a fresh v2.54.2 instance?

@fforootd reproduced in v2.53.3, on macOS
2024-06-14 01:07:43.793 UTC [1296] ERROR:  could not obtain lock on row in relation "current_states"
2024-06-14 01:07:43.793 UTC [1296] STATEMENT:  SELECT
	    aggregate_id
	    , aggregate_type
	    , "sequence"
	    , event_date
	    , "position"
	    , filter_offset
	FROM
	    projections.current_states
	WHERE
	    instance_id = $1
	    AND projection_name = $2
	FOR UPDATE NOWAIT;
2024-06-14 01:07:50.773 UTC [1802] FATAL:  role "root" does not exist
2024-06-14 01:08:00.836 UTC [2189] FATAL:  role "root" does not exist
2024-06-14 01:08:10.880 UTC [2344] FATAL:  role "root" does not exist

As far as we are aware the error `could not obtain lock on row in relation "current_states"``happens from time to time and should not be worry some.
But FATAL:  role "root" does not exist is a weird one... can you share your zitadel config?

the config is simply pulled from zitadel self-host example. i could get ride of the error by setting another PGUSER environment variable

@pot-code ok thanks for sharing. Re: CPU, can you try 2.53.6 that contains a fix for a CPU problem we had  (or 2.54.3)
The CPU might relate to this issue #8059

@fforootd that's a relief! thank you

This is still not fixed for me. This looks like it might be related: https://github.com/jackc/pgx/issues/831

I am having the same issue in my Docker setup. I only just noticed the issue even, though I haven't changed anything ( maybe the server could have restarted, but I don't think so ).
zitadel:
image: 'ghcr.io/zitadel/zitadel:v2.55.2'

postgres:
image: postgres:16-alpine

I was also getting the FATAL:  role "root" does not exist, but then I fixed that by adding user: postgres to the Docker compose service for the database. Apparently the database wants to run with the same system user as the database or something.
But that seems unrelated to the zitadel issue now.
I also had the issue with zitadel version ghcr.io/zitadel/zitadel:v2.48.2, but I upgraded to v2.55.2 to try and fix the issue.
Here's my error:
2024-07-07 13:43:15.106 UTC [2303] ERROR:  could not obtain lock on row in relation "current_states"
2024-07-07 13:43:15.106 UTC [2303] STATEMENT:  SELECT
	    aggregate_id
	    , aggregate_type
	    , "sequence"
	    , event_date
	    , "position"
	    , filter_offset
	FROM
	    projections.current_states
	WHERE
	    instance_id = $1
	    AND projection_name = $2
	FOR UPDATE NOWAIT;

I got the same error since I've started with Zitadel (2.54.3) . I've updated to 2.56.2 this morning.
The error is still there.
I see It always occur, when somebody is login or logout.
So far I haven't seen any severe problems.

same issue each time in each deployment when I added a user ( through URL/v2/users/human). no matter what I do, I always end up with :
zdb         | 2024-09-02 21:01:02.372 GMT [1556] ERROR:  could not obtain lock on row in relation "current_states"
zitadel     | time="2024-09-02T21:01:02Z" level=debug msg="unable to query current state" caller="/home/runner/work/zitadel/zitadel/internal/eventstore/handler/v2/state.go:70" error="ERROR: could not obtain lock on row in relation "current_states" (SQLSTATE 55P03)" projection=auth.users3
have been trying for the past 2 months at this point, but I can't make zitadel work.

same issue each time in each deployment when I added a user ( through URL/v2/users/human). no matter what I do, I always end up with : db | 2024-09-02 21:01:02.372 GMT [1556] ERROR: could not obtain lock on row in relation "current_states" zitadel | time="2024-09-02T21:01:02Z" level=debug msg="unable to query current state" caller="/home/runner/work/zitadel/zitadel/internal/eventstore/handler/v2/state.go:70" error="ERROR: could not obtain lock on row in relation "current_states" (SQLSTATE 55P03)" projection=auth.users3
have been trying for the past 2 months at this point, but I can't make zitadel work.
(latest version of docker, docker compose and Fedora)

Usually, even if this message appears zitadel still works.
Can you share the exact version you use (zitadel prints that during startup) and how zitadel looks like when you try to access it?

We found the source of this errors was the milestones. More info can be found here: https://github.com/zitadel/zitadel/issues/8741. It should be fixed with: https://github.com/zitadel/zitadel/pull/8788.


Query For Cloud Stack Management Server install

I have one query for my private cloud only LAN. Can I install  Cloud Stack Management Server access for my local LAN without public access.
How can I setup it. I am able to normally setup and access vm's using host as XEN server with include WAN Ethernet (public IP).

Please ask questions on the users ML instead - http://cloudstack.apache.org/mailing-lists.html
You may also refer the docs and the quick install guide - http://docs.cloudstack.apache.org
Short answer - yes  @alamintech
Closing as this is not a bug or a feature request.

Thanks for your reply.
I got this link before but I am not understand which exact mail address. If possible please give me exact mail address where I send my query.
Another Q. for you which part I will remove from http://docs.cloudstack.apache.org/ this guide for my query. I am successfully install it using host as XEN server.
Thanks!


Branch protection with specified team exceptions

Scenario
I have a repository that only contains documentation—markdown files and images. The core team who manages the repo (codeowners) makes updates to the documentation frequently.
Goal
We've decided that we don't want to require pull request approval to make changes to doc files. But, we still want to require pull request approval for all outside (non-codeowner) contributors.
Attempted Solution
I attempted to accomplish my desired outcome with the the following configuration.
branches:
  - name: master
    protection:
      required_pull_request_reviews:
        required_approving_review_count: 0
        require_code_owner_reviews: false
      restrictions:
        teams:
          - example-team-name

However, the configuration did not accomplish my goal.
How do I enable branch protection while allowing specific individuals or teams to change files without review?

I solved this issue by setting the users setting to an empty list.
branches:
  - name: master
    protection:
      required_pull_request_reviews:
      restrictions:
        teams:
          - example-team-name
        users: []

thanks for the report, @ryanbrookepayne. it seems the api for this is very particular about all details being provided in order to successfully configure anything. we've already noticed other similar issues, like #150.
i'm going to keep this report open in the hopes that we can provide some better defaults before calling the api in cases like this.


Orders Page

Hey, I've started working on this as mentioned in issue #12. 🙂

Orders page should have a single item view with these related components (according to designs):

[ ] Product-view
[ ] Cart-view
[ ] Checkout


Cleanup project output/intermediate dirs

Summary of the Pull Request

set separate intermediate dir for vcxprojs
set separate intermediate dir for csprojs
CopyToOutputDirectory only when necessary, so we don't rebuild stuff every time
set output dir prefix to "modules" for all modules

PR Checklist

[x] Applies to #1612

Detailed description

this reorganization helps to keep all build artifacts in one place out of src folder
it's also easier to see what files each project produces and remove all build artifacts manually in one move

We still observe some .lib/.pdb/.../ stuff in the output folder and can't just tell the installer to simply package everything (*) in a x64\Release folder, because that would require introducing too many custom rules for each new project and is kinda error-prone anyway (e.g. including unwanted build artifacts to the installer by accident)
Validation Steps Performed

Built MSI, installed it and used ImageResizer + FZ
Built MSIX

@crutkas Could you please try building from this branch to verify if it works as you expect?

F5 doesn't work from a total clean solution.  Stuff is much cleaner but F5 still doesn't work correctly out of gate.
Getting stuff to work:

debug folder deleted, full rebuild, F5: Settings loads, all PTs load.  133 total files in modules.  Root dir looks way cleaner now.
debug folder deleted, F5: Modules only has 41 items in it, no lang folders.  Only FZ, WW, and SG loaded

Files being left behind after a clean solution command too.

From a full compile, modules has 133 items with lang folders
From a full compile then cleaning, modules has 50 items with lang folders.  debug folder has no files

maybe this is the .NET dev in me, but I'm use to below.  I do think this would effect build scripts and the installer however

root/bin/ProcessorType/Files
root/obj/ProcessorType/Files

img of artifacts after a clean solution.

Yes, I'm also getting 133 and 41 files from the steps you've described. Getting 41 files on pressing F5 is expected, since most projects do not explicitly depend on the runner:

So on pressing F5, msbuild only build direct runner dependencies. I don't think we should explicitly set all projects to depend on the runner, since usually work on a single PT at a time and don't need to wait for everything to be built.
Regarding the clean task issue - reading as how it's implemented internally, it looks like msbuild cannot track files produced by nuget references. For example, here's my PowerToys\x64\Debug\obj\FancyZonesEditor\FancyZonesEditor.csproj.FileListAbsolute.txt contents:
list of tracked artifacts for FZE
S:\Programming\utilities\PowerToys\x64\Debug\modules\FancyZonesEditor.exe.config
S:\Programming\utilities\PowerToys\x64\Debug\modules\FancyZonesEditor.exe
S:\Programming\utilities\PowerToys\x64\Debug\modules\FancyZonesEditor.pdb
S:\Programming\utilities\PowerToys\x64\Debug\obj\FancyZonesEditor\FancyZonesEditor.csprojAssemblyReference.cache
S:\Programming\utilities\PowerToys\x64\Debug\obj\FancyZonesEditor\FancyZonesEditor.exe.config
S:\Programming\utilities\PowerToys\x64\Debug\obj\FancyZonesEditor\CanvasEditor.baml
S:\Programming\utilities\PowerToys\x64\Debug\obj\FancyZonesEditor\WindowLayout.baml
S:\Programming\utilities\PowerToys\x64\Debug\obj\FancyZonesEditor\CanvasEditor.g.cs
S:\Programming\utilities\PowerToys\x64\Debug\obj\FancyZonesEditor\GridEditor.g.cs
S:\Programming\utilities\PowerToys\x64\Debug\obj\FancyZonesEditor\GridResizer.g.cs
S:\Programming\utilities\PowerToys\x64\Debug\obj\FancyZonesEditor\LayoutPreview.g.cs
S:\Programming\utilities\PowerToys\x64\Debug\obj\FancyZonesEditor\MainWindow.g.cs
S:\Programming\utilities\PowerToys\x64\Debug\obj\FancyZonesEditor\EditorOverlay.g.cs
S:\Programming\utilities\PowerToys\x64\Debug\obj\FancyZonesEditor\GridEditorWindow.g.cs
S:\Programming\utilities\PowerToys\x64\Debug\obj\FancyZonesEditor\WindowLayout.g.cs
S:\Programming\utilities\PowerToys\x64\Debug\obj\FancyZonesEditor\CanvasEditorWindow.g.cs
S:\Programming\utilities\PowerToys\x64\Debug\obj\FancyZonesEditor\CanvasZone.g.cs
S:\Programming\utilities\PowerToys\x64\Debug\obj\FancyZonesEditor\GridZone.g.cs
S:\Programming\utilities\PowerToys\x64\Debug\obj\FancyZonesEditor\App.g.cs
S:\Programming\utilities\PowerToys\x64\Debug\obj\FancyZonesEditor\GeneratedInternalTypeHelper.g.cs
S:\Programming\utilities\PowerToys\x64\Debug\obj\FancyZonesEditor\FancyZonesEditor_MarkupCompile.cache
S:\Programming\utilities\PowerToys\x64\Debug\obj\FancyZonesEditor\FancyZonesEditor_MarkupCompile.lref
S:\Programming\utilities\PowerToys\x64\Debug\obj\FancyZonesEditor\App.baml
S:\Programming\utilities\PowerToys\x64\Debug\obj\FancyZonesEditor\GridEditor.baml
S:\Programming\utilities\PowerToys\x64\Debug\obj\FancyZonesEditor\GridResizer.baml
S:\Programming\utilities\PowerToys\x64\Debug\obj\FancyZonesEditor\LayoutPreview.baml
S:\Programming\utilities\PowerToys\x64\Debug\obj\FancyZonesEditor\MainWindow.baml
S:\Programming\utilities\PowerToys\x64\Debug\obj\FancyZonesEditor\EditorOverlay.baml
S:\Programming\utilities\PowerToys\x64\Debug\obj\FancyZonesEditor\GridEditorWindow.baml
S:\Programming\utilities\PowerToys\x64\Debug\obj\FancyZonesEditor\CanvasEditorWindow.baml
S:\Programming\utilities\PowerToys\x64\Debug\obj\FancyZonesEditor\CanvasZone.baml
S:\Programming\utilities\PowerToys\x64\Debug\obj\FancyZonesEditor\GridZone.baml
S:\Programming\utilities\PowerToys\x64\Debug\obj\FancyZonesEditor\FancyZonesEditor.g.resources
S:\Programming\utilities\PowerToys\x64\Debug\obj\FancyZonesEditor\FancyZonesEditor.Properties.Resources.resources
S:\Programming\utilities\PowerToys\x64\Debug\obj\FancyZonesEditor\FancyZonesEditor.csproj.GenerateResource.cache
S:\Programming\utilities\PowerToys\x64\Debug\obj\FancyZonesEditor\FancyZonesEditor.csproj.CopyComplete
S:\Programming\utilities\PowerToys\x64\Debug\obj\FancyZonesEditor\FancyZonesEditor.exe
S:\Programming\utilities\PowerToys\x64\Debug\obj\FancyZonesEditor\FancyZonesEditor.pdb

So when we invoke Clean task, files from that list are getting removed. As you can see, there're no e.g. HtmlAgilityPack.dll, GalaSoft.MvvmLight.dll etc., so they survive the clean. I'm not sure it could be solved easily if we don't nuke the whole build folder from each project.

root/bin/ProcessorType/Files
root/obj/ProcessorType/Files

So you'd like to have PowerToys\bin\x64\Debug and PowerToys\obj\x64\Debug\ instead of the current PowerToys\x64\Debug and PowerToys\x64\Debug\obj? Yeah, it will most likely require changing build scripts, but it could be done.

I don't think we should explicitly set all projects to depend on the runner

We will have to do it since it's important.

I don't think we should explicitly set all projects to depend on the runner

We will have to do it since it's important.

Stuff that we are shipping 100% needs to be part of F5 inside master.  If someone wants to speed up their build and disable that, that is up to them, but master, F5 needs to represent the what we are shipping.

Folder structure is a future work item, rather have bugs / features done.

reasons why i feel F5 needs to represent the project

I strongly believe most people's work flows (for .NET at least) would F5 the project versus solution build.
Doing a full compile includes tests (added time to just test)
If someone just ran the project, it looks like half our stuff doesn't work and it is a more complex build to do contribute and add in new features.

done, PTAL.

on a fresh clone, Runner defaults to AnyCPU

Past that quirk that, this looks good to me.

on a fresh clone, Runner defaults to AnyCPU

That is not something that is persisted in git, and we actually shouldn't anyway.

it is the WinAppDriver project that is causing this.

Than can be change for sure. I'll take care of it.

@enricogior removed AnyCPU, could you please check if win-app-driver still works?


refactor: added stage name to iam role statement name

 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.

 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.


Initial wget script doesn't seem to work

No idea why, but I'm stuck on the very first hurdle!

~/Downloads: cd ~/Downloads && wget -N -O solidworks-install.sh https://raw.githubusercontent.com/cryinkfly/SOLIDWORKS-for-Linux/main/files/builds/stable-branch/bin/install.sh && mv install.sh solidworks-install.sh && chmod +x solidworks-install.sh && ./solidworks-install.sh
WARNING: timestamping does nothing in combination with -O. See the manual
for details.

--2024-09-24 11:19:02--  https://raw.githubusercontent.com/cryinkfly/SOLIDWORKS-for-Linux/main/files/builds/stable-branch/bin/install.sh
Loaded CA certificate '/etc/ssl/certs/ca-certificates.crt'
Resolving raw.githubusercontent.com (raw.githubusercontent.com)... <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, ...
Connecting to raw.githubusercontent.com (raw.githubusercontent.com)|<IP_ADDRESS>|:443... connected.
HTTP request sent, awaiting response... 200 OK
Length: 28053 (27K) [text/plain]
Saving to: ‘solidworks-install.sh’

solidworks-install.sh         100%[===============================================>]  27.40K  --.-KB/s    in 0.002s

2024-09-24 11:19:02 (10.9 MB/s) - ‘solidworks-install.sh’ saved [28053/28053]

mv: cannot stat 'install.sh': No such file or directory

Update: I managed to get past this by chmodding the solidworks-install.sh and starting it ... but still, this might trip up users.

this was introduced with an incompatibility with pulls #57 and #48. Either remove mv install.sh solidworks-install.sh && or -O solidworks-install.sh when initally running the command and it will launch as expected.


Render button when useGoggleLogin

Hi,
Using the GoogleLogin component renders a button (as far as I can tell from the code, the button is rendered by the Google scripts).
I need to use the Authorazation flow, and as such I am using useGoogleLogin. My issue is that the example (as well as the code) seems to not support rendering a button, but I need to roll my own.

Is there an easy way to get a "standard" rendered button, so that I don't need to roll my own?
Is there a reason that when implementing the library it was implemented like this?
If there is an easy answer to (1), and there is no good answer to (2), would a PR that adds this feature be approved?
Thanks!

Hello,
For authorization flows Google. Doesn't render any buttons via any script just give us a function to call to open the login process
And useGoogleLogin is just small wrapper around this
It's our own responsibility to create the button styles in any way you would like

Is there any library that you would recommend that renders the button in react? So that I don't need to deal with it?

I would recommend styling it by yourself It would be easy and small in size
Or you can use something like this https://www.npmjs.com/package/react-social-login-buttons to render any social button you want

Thanks


Updated vue-fusioncharts to v3.0.3 from github

The fix for multi-series on time axis has been fixed in the latest version of vue-fusioncharts in v3.0.3. Please do a npm install or npm install vue-fusioncharts@latest after merging this pull request.

It seems to be working perfectly now.


Upgrade dependencies and fix error that stemmed from upgrade

This pull request updates the following dependencies:

TypeScript
TypeORM
Prettier
NestJS (@nestjs/*)
class-transformer
class-validator
Postgres (pg)
Jest
RxJS
cross-env
supertest
nodemon
ts-loader
ts-node
tsconfig-paths

These changes required a small change to server/src/main.ts as well, but the change is minimal.

The changes look good and everything works as it does currently.


Bugfix: credentialSubject.id, iss, and sub should be DID itself

[OID4VCI]
credentialSubject.id, iss, and sub should be DID, not DID URLs pointing to verification methods.

Could you point me to relevant documentation on this? Thanks!

@dbluhm
um, probably here?
https://www.w3.org/TR/vc-data-model/#identifiers

The value of the id property MUST be a single URI. It is RECOMMENDED that the URI in the id be one which, if dereferenced, results in a document containing machine-readable information about the id.

I'm a beginner in this field, so I'm not quite sure, but I've never seen before that 'credentialSubject.id' and 'issuer' values are DID URLs pointing to Verification Methods, and it feels uncommon to me. Is there any rational reason for this?

Thanks! And welcome to the field! I think you must be right. I'm trying to think back to why we chose to use DID URLs to verification method IDs at the time. I might have to spend some time looking deeper at this. Unfortunately, that might be a bit before I can look too much closer.
Would you be able to look into the test failures? DCO especially will need to be taken care of. The Details link should provide some pretty detailed instructions on how to handle that. Thanks!

Thanks.

Would you be able to look into the test failures?
Yes sure. I'll take care of it soon.


Update spack submodule tag for release 1.1.0

Update spack submodule tag in container recipes and .gitmodules for release 1.1.0. The spack submodule tag was created yesterday and points to the same hash as the head of branch release/1.1.0.

I will let those CI tests run to completion before merging.


Blog app - controllers specs

In this PR we:

[x] Added RSpec as a gem
[x] Created a Request spec file for all controllers
[x] We added a test for users and posts controllers
[x] We added tests for all actions of controllers

Thanks @mikethreels


The manager does not save last installed configuration

Description

Right now manager always tries to uninstall Serverless based on the configuration saved in the Serverless CR. It does mean that if a user would edit Serverless CR and pass a configuration that causes an error (new configuration will not be installed) and uninstall the CR then the manager will prepare components to uninstall based on the updated CR which is not fully installed.
Expected result

The manager should uninstall components based on the lastly applied configuration, not
Steps to reproduce

This problem is masked by this ticket. After fixing issue 117 it will be able to reproduce when a user would update Serverless CR from:
spec:
  dockerRegistry:
    enableInternal: true

to:
spec:
  dockerRegistry:
    enableInternal: false
    secretName: "does-not-exist" // this should cause an error state

Even after resolving ticket 117 the manager will not remove docker-registry, because it does not propagate changes (uninstall docker-registry) before the error occurred and to uninstallation, it will use the Serverless CR to generate objects to remove.

ideas:

use "last-applied-spec" annotation

I see one possible issue in the ideas described by the @kwiatekus. What if we would like to upgrade the serverless-manager? In this case, when the code is different and we would provide changes in the chart, a new serverless-manager generates a manifest based on (for example) the last-applied-spec annotation using new code and a new chart. The result of this operation would be omitting some resources (not templated by from the new chart version). I think we should think about caching the whole manifest, not last-applied-spec. Additional ideas:

create a secret in the serverless-manager's namespace with installation info
cache in the manager's memory (problem with manager pod restart)

I would propose implementing a secret with a manifest inside. It gives us huge possibilities to store more info in the future or even to replace this logic with something else if we decided that it's somehow blocking us

Could the resources managed by manager have a label so that regardless of manager's code version there would be a cleanup part that tries to lookup for some orhpaned legacy resources and clean them up

Generally, we thought about searching based on labels or keeping manifest in configmap instead of secret. But finally, we had a problem with listing all resources with the specific label because Kubernetes client expect specific resource GroupVersionKind for listing so we can't simply list all resources with a label. Secret is much better than configmap because rendered manifest may contain secrets, keys, tokens, and other sensitive information that we should keep in more safe resources like secrets.


hector_slam: 0.5.2-2 in 'noetic/distribution.yaml' [bloom]

Creating the PR with bloom didn't finish correctly.
So here's the last step manually.
I'm using an oauth token and the first time it complained about needing the workflow permission.
After adding that it failed with:
To https://github.com/StefanFabian/rosdistro.git
 * [new branch]          bloom-hector_slam-3 -> bloom-hector_slam-3
Failed to open pull request: GithubException - HTTP Error 404: Not Found (https://api.github.com/repos/ros/rosdistro/pulls): None

Sure, here you go:
Increasing version of package(s) in repository hector_slam to 0.5.2-4:

upstream repository: https://github.com/tu-darmstadt-ros-pkg/hector_slam.git
release repository: https://github.com/tu-darmstadt-ros-pkg-gbp/hector_slam-release.git
distro file: noetic/distribution.yaml
bloom version: 0.10.6
previous version for package: 0.5.1-1

hector_compressed_map_transport
* Fix out of bounds access
  Fixes #52 <https://github.com/tu-darmstadt-ros-pkg/hector_slam/issues/52> and #70 <https://github.com/tu-darmstadt-ros-pkg/hector_slam/issues/70>
* Contributors: Stefan Fabian

hector_geotiff
* Refactored hector_geotiff dependencies.
* Contributors: Stefan Fabian

hector_geotiff_launch

No changes

hector_geotiff_plugins

No changes

hector_imu_attitude_to_tf

No changes

hector_imu_tools

No changes

hector_map_server

No changes

hector_map_tools

No changes

hector_mapping
* Remove tf_conversions as a dependency (#93 <https://github.com/tu-darmstadt-ros-pkg/hector_slam/issues/93>)
* Add reset mapping (and pose) service (#87 <https://github.com/tu-darmstadt-ros-pkg/hector_slam/issues/87>)
  * add service to reset the mapping with a new initial pose
* Reorganize scanCallback and add comments (#89 <https://github.com/tu-darmstadt-ros-pkg/hector_slam/issues/89>)
  * Refactor, reformat and comment scanCallback
  * make rosPointCloudToDataContainer void
  * make rosLaserScanToDataContainer void
* Add pause and reset services to hector_mapping (#86 <https://github.com/tu-darmstadt-ros-pkg/hector_slam/issues/86>)
  * Add pause and reset services to Hector
* Contributors: Marcelino Almeida

hector_marker_drawing

No changes

hector_nav_msgs

No changes

hector_slam

No changes

hector_slam_launch

No changes

hector_trajectory_server

No changes


Floating toolbar

A floating toolbar (ala Office 2007+ style) that allows you to bold/italic/header/list/etc the selection would help with feature discoverability.

Added a pull request with an implementation for this:
https://github.com/Code52/DownmarkerWPF/pull/44


[Platform][ASR] add asr582x & asr595x support

add asr platform support
brief:
ASR582X is a low-power, high-performance, highly integrated combo SoC supporting 1x1 IEEE 802.11 b/g/n and BLE 5.1. The chip integrates RF transceiver, 802.11/BLE PHY + MAC, Arm China STAR-MC1 Processor, multiple peripheral interfaces, AOA/AOD, Real Time Counter (RTC) and a complete power management module.
ASR595X is a low-power, high-performance, highly integrated combo SoC supporting 1x1 IEEE 802.11 b/g/n/ax and BLE 5.1. Thanks to the uniquely designed power management circuitry, this chip significantly reduces power consumption and peripheral components.

https://github.com/project-chip/connectedhomeip/pull/26295

Nice, keep go on

https://github.com/project-chip/connectedhomeip/pull/26361


Remove dependency on internal angular method ($el.scope()) (issue #255)

Fix #255 Drag and drop gives an error with Angular 1.3.* when you have debugInfo disabled.
Plunker
Breaking change
Now html markup require ng-model to be used with the ng-sortable
Example:

{{item}}

{{item}}

{{item}}

Alas, there is no time to deal with this problem. Those who want to use with disbled debugInfo, can take this patch.

Try dev-branch.
Example: http://jsbin.com/yugoropuzo/1/edit?js,output


Update shell-ui for new search design and minimized suggestions

There is a fair amount of commented code and general cruftiness that I left in these modules just because we've been shuffling around the ui elements, turning things on and off. If we settle in on this setup in the short term, I will push a cleanup PR.

I forgot to update the Selenium tests to deal with the UI changes. Will do that asap.


Suspected timing issue around temporary cache and garbage collection of resources

Bug Report
Keycloak tests in the same class run in the same namespace and typically just reuse the same parent cr name. The cleanup between tests ensures that the dependent statefulset is gone, but doesn't check for other dependents such as secrets or services.
In a rare case (1% of runs?) there is a timing issue happening where the state of the temporary cache is out of sync with the informer cache in such a way that does not resolve on its own. In particular the temporary cache will show a resource as existing, while the informer cache / server has it as non-existent.
What did you do?
It's difficult to reproduce, turning up logging made it even less likely to occur - with either fabric8 informer or operator sdk debug logging turned on for the reflector / informer it failed to reproduce.
Instead I resorted to some logic that checked the state of getSecondarResource vs a listing of what was in cache when were'nt fully ready after the main resolving method. That confirmed there was a temporary cache entry, but nothing in the informer cache nor on the server.
What did you expect to see?
The temporary cache state in sync.
What did you see instead? Under which circumstances?
In an extremely rare case, and seemingly related to the garbage collection of resource with the same name, the state was out-of-sync.
Somehow the temporary cache is referencing the garbage collected entry - or one that it thought it created shortly after garbage collection, but that the api server mistakenly deletes.
A possible explanation: a create of the resource seems successful so a resource is being returned for that operation. The client then receives an added and delete event in rapid succession - prior to the operator setting the state of the temporary cache. Then once it sets the temporary cache, it is no longer valid nor will there be an event that causes it to correct.
Environment
v4.8.0, Minikube v1.32.0, Kubernetes v1.27.10
Possible Solution
I'm not sure yet. I'd like to capture the full sequence of events to better understand. It could possibly invole tombstoning the temporary cache so that it won't set an entry if it's "too soon" after a delete event.
Additional context

Ok, I think the full sequence goes something like this:
Test run finishes and deletes the parent keycloak, but doesn't wait for the delete event nor all of the dependents to be deleted.
The next run of the test will start off by using the keycloak that has been marked for deletion as the operator hasn't received that event yet, even though the next test has created a new version of the keycloak resource.
A reconciliation of the old keycloak happens at a point in time such that the old secret does not exist, so a new one is created but it's owner reference is still set to the one to be deleted.
Then prior to the operator setting the temporary cache state, it receives the add and delete events for that secret.
If this is correct the operator should be able to see that the owner reference is wrong on the secret in temporary cache - I'll add some logging about that to confirm this is the case.
@csviri The secrets in question are using ResourceDiscriminators for their lookup, so that may also be an issue - should that be checking for the owner reference as well, and not just the name?

just to clarify:

The next run of the test will start off by using the keycloak that has been marked for deletion as the operator hasn't received that event yet, even though the next test has created a new version of the keycloak resource.

The KeyCloak controller does not seem to be working with finalizers, by "marked for deletion" you, mean it is actually deleted? or there is a finalizer from another controller?

A reconciliation of the old keycloak happens at a point in time such that the old secret does not exist, so a new one is created but its owner reference is still set to the one to be deleted.

So we start a reconciliation for the old keycloak, it will create a secret and put it into the cache with an owner ref to the old (non existing keycloak).
What does not adds to me (what can happen I guess but very unlikely)  is how is that the during the reconciliation the Keycloak resource is in the cache but Secret is not. That mean that somebody delete the Keycloak, the GC deleted the Secret - what can be quite a big time delay, but we did not receive the event from Keycloak but received for the Secret (from GC delete)?

Then prior to the operator setting the temporary cache state, it receives the add and delete events for that secret.

The order is strictly delete then  add if understand correctly. ? or you mean also the Delete event of the new secret that is created with a non-existing owner ?

The KeyCloak controller does not seem to be working with finalizers, by "marked for deletion" you, mean it is actually deleted? or there is a finalizer from another controller?

Yes this is a simple foreground delete. However there is a lag (at least 30 seconds) between when the operation completes and when you can still set that uid on an owner reference - even if a simple get returns a 404 for the owner.
This demonstrates that and forces the situation to happen where we resurrect an invalid entry by adding a time delay:
@KubernetesTest
public class TestSomething {

    static KubernetesClient client;

    @BeforeAll
    public static void before() throws FileNotFoundException {
        BaseOperatorTest.createCRDs(client);
    }

    @Test
    public void testGargageCollection() {
        Keycloak defaultKc = K8sUtils.getDefaultKeycloakDeployment();

        client.secrets().inform(new ResourceEventHandler<Secret>() {

            @Override
            public void onAdd(Secret obj) {

            }

            @Override
            public void onUpdate(Secret oldObj, Secret newObj) {

            }

            @Override
            public void onDelete(Secret obj, boolean deletedFinalStateUnknown) {
                System.out.println("Deleting " + obj.getMetadata().getName() + " " + obj.getMetadata().getUid() + " " + obj.getMetadata().getResourceVersion());
            }
        });

        Keycloak keycloak = null;
        for (int i = 0; i < 100; i++) {
            try {
                keycloak = client.resource(defaultKc).create();
            } catch (KubernetesClientException e) {
                if (e.getCode() != 409) {
                    throw e;
                }
            }
            client.resource(keycloak).delete();
            final var kc = keycloak;
            Secret s = new SecretBuilder().withNewMetadata().withName("x").endMetadata().build();
            s.addOwnerReference(kc);
            try {
                var result = client.resource(s).create();
                Thread.sleep(100);
                System.out.println("Setting temporary cache to " + result.getMetadata().getName() + " " + result.getMetadata().getUid() + " " + result.getMetadata().getResourceVersion());
            } catch (Exception e) {
                System.out.println("Could not create dependent, the owner is missing");
            }
        }
    }

}

If you run something like that, or substitute any other owner / dependent, you should see logs like:
Deleting x bedbfde6-05a9-4e8c-9800-dc917ba67f4e 629598
Setting temporary cache to x bedbfde6-05a9-4e8c-9800-dc917ba67f4e 629597

Showing that we've moved backwards in resourceVersion to the now deleted entry.
More than likely there's some extreme behavior being introduced here cause these minikube instances are running in GHA with the minimum of 2 cpu and already seem a little stressed for memory.

That means that somebody deleted the Keycloak, and the GC deleted the Secret - what can be quite a big time delay, but we did not receive the event from Keycloak but received the Secret (from GC delete)?

That is correct, and sense these are independent informers that's possible - just very unlikely.

The order is strictly delete then add if understand correctly. ? or you mean also the Delete event of the new secret that is created with a non-existing owner ?

It's the latter. As long as the sequence ends with a delete, then we are in trouble. If the sequence were to end in an add/update then the current add/update opertation would compare resourceVersions.

Just a side note: if I understand this correctly, the main problem is that the Delete event in this case arrives before the resource from our update is put in the secondary cache; if the delete arrives after the resource is put in the cache we are ok, since temp cache is cleared, and informer cache is leading again. Note that the former event recording algorithm kinda would cope with this scenario also (without tombstones).

Just a side note: if I understand this correctly, the main problem is that the Delete event in this case arrives before the resource from our update is put in the secondary cache

It can be any sequence that ends in a delete yes - in theory you could create and delete the resource several times in the span of time that it takes to set the temporary cache entry.

Note that the former event recording algorithm kinda would cope with this scenario also (without tombstones).

I've opened a pr to show what this fix would look like with pre/post calls. This makes it more similar to the previous logic.

@csviri Would it be possible to backport this change so it's available as part of RHBQ 3.8?

Hi @vmuzikar that is more a question for @metacosm

I've release 6.6.7 that should contain this fix. A PR has also been opened for inclusion in Quarkus 3.8.


documentation structure change - new category: advanced features of lakeFS

@johnnyaug
Here's what I've done step by step:

I created a new folder called "advanced."
I moved 4 files to this folder - for each file, I changed the parent and added a redirect link.
I created an index page with a summary of what you can find in the folder and links to all four pages.

@arouyan Seems like a lot has changed in the docs since this PR, can we close it?

@itaiad200 Absolutely, please close it. Thank you!


Improve mirror registry logics

Currently, mirror registry is implemented in low-level http transport connection layer. In such a case, it is hard to handle the scenario that the redirected URL has format foo/bar/v2/blobs/xxxx when accessing the mirror registry which does not have the resources, and 404 is returned.
We'd better move the mirror registry to a higher level and make it only work for registry storage backend.

It has been improved on latest nydusd.


[BUG] wash push to jFrog artifactory

Affected project(s)

[ ] documentation
[ ] examples
[ ] wasmCloud host
[X] wasmCloud CLI (wash)
[ ] wasmCloud dashboard UI (washboard)
[ ] capability providers
[ ] provider bindgen
[ ] control interface client
[ ] other / not sure

Describe the bug
When I trying to push any artifacts with size more then 4mb via wash, push command return success code. But when I trying to download it I got an error:
Invalid provider archive: numeric field did not have utf-8 text: *** when getting cksum for **********

I did my own investigation and came to the conclusion that the problem is in the chunked load.
Becuse artifact layer in registry have same size as last chunk.

Logs from push:
DEBUG oci_distribution::client: Pushing chunk start_byte=20971520 end_byte=21628742 blob_data_len=21628743 body_len=657223 location="my.registry.com/external/providers/http-server/blobs/uploads/06325aaf-a022-439e-ad70-c65d5ebba93b" headers={"content-range": "20971520-21628742", "content-length": "657223", "content-type": "application/octet-stream"}

With artifacts with size less then 4mb wash push works fine.
I guess issue related with https://github.com/oras-project/rust-oci-client
Steps to reproduce
wash pull ghcr.io/wasmcloud/http-server:0.23.1
wash push my.artifactory.com/providers/http-server:0.23.1 ./http-server.par.gz

wash pull my.artifactory.com/providers/http-server:0.23.1

Expected behavior
Artifact pushed to artifactory with correct layer size.
Artifact can be pulled after successful push.
Environment

OS: Linux, MacOS
Shell: bash, zsh

Screenshots / Logs / Additional context
No response

I think @joonas has been running point on fixing this upstream in rust-oci-client, once that's ready we should bring it into wash

That's right, once https://github.com/oras-project/rust-oci-client/pull/177 lands (and possibly https://github.com/oras-project/rust-oci-client/pull/176, I'm not 100% on how Artifactory behaves in this regard), we should be able to add an option to wash to do a monolithic push instead of chunked.

@ptsgr I've https://github.com/wasmCloud/wasmCloud/pull/3461 up for review, the intention is to address this issue with the new feature in wash (and updates to the underlying OCI crates).
If you have a chance to pull down my branch and compile wash locally off of that branch to try it out, I would be interested to hear if that addresses the problem you're seeing.

Since #3461 shipped as part of wash v0.37.0 I'll go ahead and close this out.
Please feel free to re-open the issue if you experience any further issues!


set window/clear when window is not open

Ned reported this.  He says that forever his scripts have made a couple of windows on the screen.  He makes a plot in the default window, window 1
plot/i=1:15/thick/color=blue 1./i

Then wants to work in a second window, so the script might do this:
set win/clear/siz=0.5 2
plot/i=1:100/thick/color=red cos(i/20)

Clearing the window is a good general step, closing viewports that might have been defined in a previous plot in that window.
What happens in the session, is
! the Window 1 plot draws correctly
yes? plot/i=1:15/thick/color=blue 1./i    ! the plot draws correctly

yes? set win/clear/siz=0.5 2
gclearws()   25 Specified workstation is not open

! The first window is wiped blank, either right away or if the windows on the screen get moved,
! and this second plot draws correctly
yes? plot/i=1:100/thick/color=red cos(i/20)

So it seems that in the case where window 2 is not yet open, the SET WINDOW/CLEAR 2  is wiping out window 1 contents.
I see this going back at least a couple years, but not in the very oldest Ferret executables we have on dunkel, but it seems we should be able to check whether the window we're asking to clear is even open and if not, don't do a call to clear anything.

This is a ferret-only bug; it has to do with xgk calls.
The sequence is to use the commands to draw both windows, move window 1 to a new location, click on window 2, then window 1 goes blank.
There's an easy fix.  The logical flag wn_open(wid) is set indicating the window numbers in use. In clear_window.F, check this flag and make the calls only if wn_open(wid)  is true (!)
I can't think of a way to test this in the benchmarks, as it requires the user moving the window, but making the change does not change benchmark outputs.


tautomer.TautomerEnumerator() function

Hi,
I am trying to enumerate tautomers using the tautomer.TautomerEnumerator() function. I am wondering if the returned list is sorted by scores so that the best-scoring tautomer in the list will have an index of zero, as it is not clear from the documentation, and if so, can I just limit to the most canonical tautomer by specifying the maximum tautomer parameter to 1
Thanks

I'm afraid that won't work. Specifying max_tautomers will actually just stop the enumeration once that many have been enumerated, so there's no guarantee that the best scoring tautomer has been found. So the only way to be sure is to enumerate fully first, and then score every enumerated tautomer.

thanks. does molvs has a function to score and sort these enumerated tautomers in an easy way. I tried to look at the documentation but could not figure out is a function like this exist. thanks

Cf. MolVS/molvs/tautomer.py
class TautomerCanonicalizer.canonicalize

Thanks. But my understanding is that the canonical tautomer is not essentially the most stable (or most energetically favorable). Other than canonicalization, is there is something else we can do to get the most energetically favorable tautomer, assuming standard conditions and neutral pH ?

Maybe score them using a force field from like MMFF94 in rdkit?

Interesting, will try that. thanks for the suggestion.


add dummy scan dropping functionality

add a flag into ciftify_subject_fmri and cifti_vis_fmri to drop n number of dummy scans from the front of the image...

this is incorporated itnto ciftify_clean_img


Added XGBoost tutorial

Hi
I pushed the scripts and files for the Tutorial: ML for Hydrological Sciences.
(https://github.com/uwhackweek/schedule-2024/issues/40)
Best,
Savalan

Thanks @savalann for putting this together! Just had a quick look and a couple suggestions:

You'll need to add your notebooks to the table of contents file if you want them to render on the website

https://github.com/geo-smart/website-2024/blob/0c6cfa505ecf4de3d3d6bad17d8c973f58ec2718/book/_toc.yml#L18-L22

This is currently 60MB, which is kind of large!

not critical, but for simple vector data consider using geojson or geopackage instead of shapefiles because it's nice to just have a single file to work with, and there are other advantages (http://www.geopackage.org)
I think you can remove 03.output folder from the PR if those are files that are generated by running your code
Consider moving your input data files to a separate repository that you copy in at the beginning of the notebook https://guidebook.hackweek.io/training/technology/data-management.html#guidelines-for-tutorials

This seems to be a lot of content for a 60min session, so be sure to do a run-though to make sure you're able to cover your main points in the allotted time

@savalann I am going to go ahead and merge this pull request, then I will make some of the above requested changes myself :)
Thanks again!

@savalann I am going to go ahead and merge this pull request, then I will make some of the above requested changes myself :)
Thanks again!

I've made those changes, see PR #9


v1.0.6alpha Release

Fixes #1291
Checklist:

[ ] I have checked for PMD and check-style issues
[x] I have read the Contribution & Best practices Guide and my PR follows them.
[x] My branch is up-to-date with the Upstream development branch.

Changes:
Screenshots for the change:

@iamareebjamal Please merge.


Increase EVENT_HANDLER_WINDOW_MOVED frequency for window drawing speed/efficiency(?)

My C is a bit rusty and I have had a cursory look through event.c and border.c as well as using yabai with verbose mode on and I see that EVENT_HANDLER_WINDOW_MOVED comes in at a steady rate no matter how fast the window is moving, would this be some API limitation or debouncing within Yabai (as I am still looking).
I'd like to experiment with increasing this if possible so if I am moving a floating window then window borders stick more closely to them.
Here's a video of what I mean (the GIF makes the framerate look weird, here's an MP4 for smoother viewing: https://i.imgur.com/hb0ZZP4.mp4)

This is a limitation of the macOS Accessibility API. Same thing applies to tracking when a window is resized.

The underlying limitation is that the borders are currently drawn as a view on a separate overlay window, which is redrawn when a window moved event comes in. You'd have to rework the border code so it draws its view on the target window itself. This is not an easy task.
As an alternative, you might wanna look into not rendering borders for windows that are currently moving: Measure the mean time delta between window moved events, then stop drawing borders until a timer using that delta expires, and refresh the timer whenever a window moved event comes in.

You'd have to rework the border code so it draws its view on the target window itself. This is not an easy task.

You could likely grab the current context of any window (through the scripting-addition), but every time that window is repainted due to some internal changes, our border would disappear. By reusing the same context we would likely occlude parts of the actual window as well - it would be an internal border, rather than external as it is now.

I figured it might be something external, all I was looking for in the code was some kind of hardcoded tick rate and when I didn't see that and saw EVENT_HANDLER_WINDOW_MOVED coming in regular intervals no matter how fast or slow I spazzed the window I felt it must've been something external.
@koekeishiya gotcha, shame it's limited so much from macOS's side.
@dominiklohmann sounds like a good enough approach to me as in all other situations apart from floating windows I almost never (99% of the time I never) see a window border lagging.

You'd have to rework the border code so it draws its view on the target window itself. This is not an easy task.

You could likely grab the current context of any window (through the scripting-addition), but every time that window is repainted due to some internal changes, our border would disappear. By reusing the same context we would likely occlude parts of the actual window as well - it would be an internal border, rather than external as it is now.

@dominiklohmann @koekeishiya would a semi-sane alternative to an outside border rendering as-fast-as-possible be to modify the SystemAppearance.bundle with Yabai such that active windows have custom assets which render a border? I don't know if it would be possible but in the asset files I was digging around in today there are a lot of shadows etc which means there might at least be some assets that bleed over the visible-content (for lack of a better term) of the window. That is the Finder window has it's view, and then the shadow that extends past it.
Tangentially today I've spent my time modifying the asset files to make window edges square instead of rounded; most of my time was spent looking for the damn files responsible hence I saw a few shadows; here's the repo: https://github.com/tsujp/square-macos-windows
Linking it because if this sounds like a semi-sane approach then the base information there might prove useful.

semi-sane approach

This sounds like it's a stupid amount of work to maintain. But if you're doing this externally anyways, feel free to add borders to your custom appearance files and link them somewhere here.
The main issue is that some applications don't respect the system appearance (FireFox, Kitty, older Electron apps etc.).
Just do something like this:
# if you have ffm disabled
yabai -m signal --add event='mouse_down' action='yabai -m window --toggle border'
yabai -m signal --add event='mouse_up' action='yabai -m window --toggle border'

# if you have ffm set to autofocus (adapt slightly for autoraise)
yabai -m signal --add event='mouse_down' action='yabai -m config focus_follows_mouse off; yabai -m window --toggle border'
yabai -m signal --add event='mouse_up' action='yabai -m window --toggle border; yabai -m config focus_follows_mouse autofocus'

The main issue is that some applications don't respect the system appearance (FireFox, Kitty, older Electron apps etc.).

 An the whole insert border colour, I don't know how that'd even be done in that now ~semi~in-sane approach.
A-ha!  Kitty is fine because it's square anyway. I wonder how I can force Firefox to behave nicely now too, the journey just keeps on going.


RHIROS-1164 implement sorting on suggested instance types table

Why do we need this change? :thought_balloon:
RHIROS-1164
Documentation requires update? :memo:

[ ] Yes
[X] No

Security Checklist :lock:
Upon raising this PR please go through RedHatInsights/secure-coding-checklist
:guardsman: Checklist :dart:

[ ] Bugfix
[X] New Feature
[ ] Refactor
[ ] Unittests Added
[ ] DRY code
[ ] Dependency Added

Additional :mega:
Feel free to add any other relevant details such as links, notes, screenshots, here.

Codecov Report
Attention: 16 lines in your changes are missing coverage. Please review.

Comparison is base (75ed454) 20.29% compared to head (2484614) 20.04%.
 Consider uploading reports for the commit adaf7e6 to get more accurate results

Files
Patch %
Lines

...nents/InstanceTypes/SuggestedInstanceTypesTable.js
0.00%
12 Missing and 1 partial :warning:

src/Utilities/api.js
0.00%
2 Missing and 1 partial :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #292      +/-   ##
==========================================
- Coverage   20.29%   20.04%   -0.26%
==========================================
  Files          47       47
  Lines         956      968      +12
  Branches      180      182       +2
==========================================
  Hits          194      194
- Misses        713      723      +10
- Partials       49       51       +2

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

@yash2189 The PR is ready for testing.

@PreetiW verified it on EE, working as expected.


Height is no longer divided by 2 in grass

Since the player Thing no longer keeps its own canvas, there's no concrete way to signify that the sprite should be drawn at half height.

Idea: keep the player full height, move it behind grass, and make the shadow half-height and above the grass.


PowerToys has detected an app... Always On Top message

Microsoft PowerToys version
0.77.0
Installation method
PowerToys auto-update
Running as admin
Yes
Area(s) with issue?
Always on Top
Steps to reproduce
Boot PC and then log into Windows 11 Pro, as an administrator.
✔ Expected Behavior
I expect to be told WHAT particular application is running with "Always On Top."
❌ Actual Behavior
When I log into Windows 11 Pro (as an administrator), the message about an app detected with Always On Top appears twice in the notification tray; WITHOUT telling me which app is involved.
Other Software
None.

We need to know which application.
We need a way to ignore that application.
We need a way to disable this notification entirely (and specifically).

Landed in this thread by looking up if there is a way to make this notification stop happening, or make it silent. It's good info, but I don't need to know that same info every time I open an app with admin, such as the task manager with Ctrl+Alt+Delete (which I imagine is a rather common thing for power users to do).
For the record, I don't get the message on startup, because I don't think I boot Windows itself as administrator like the top post says, But implementing the suggestions here would still reduce a little bit of annoyance for me, and I imagine others.

We need to know which application.

I'm particularly interested in this.
I would like to know which application is running as administrator. This could help to solve some security issues.

Just wondering if there are any updates.  I am having the same issue as well, and would like to know the program.  If I click on the notification, the counter just goes down.
Thank you!

Nope, not a thing. What I was told is that whilst there is a flag, there's nothing that allows the app to be identified. Grrr.
I suspect that it'll need to be a clever clog's programmer to do it it.
Sent from Outlook for Androidhttps://aka.ms/AAb9ysg

From: four2theizz0 @.>
Sent: Friday, November 8, 2024 2:12:58 AM
To: microsoft/PowerToys @.>
Cc: Andy @.>; Manual @.>
Subject: Re: [microsoft/PowerToys] PowerToys has detected an app... Always On Top message (Issue #30973)
Just wondering if there are any updates. I am having the same issue as well, and would like to know the program. If I click on the notification, the counter just goes down.
Thank you!
—
Reply to this email directly, view it on GitHubhttps://github.com/microsoft/PowerToys/issues/30973#issuecomment-2463618543, or unsubscribehttps://github.com/notifications/unsubscribe-auth/BEDQNUO7UFDOGK3W2WUOB4LZ7QM2VAVCNFSM6AAAAABB34YHLKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINRTGYYTQNJUGM.
You are receiving this because you are subscribed to this thread.Message ID: @.***>

I too would love to know what app is causing the notification. If, for example it's some sort of virus, that would be extra useful.


IRGen: Correctly indicate non-key same-typed generic parameters.

The correct query here is whether the generic parameter is canonical within its generic signature, not whether it's concrete.
(Note that this includes the patches from https://github.com/apple/swift/pull/15960, since without that fix this ends up crashing in IRGen elsewhere.)

@swift-ci Please test

@swift-ci Please test source compatibility

Looks good, but an execution test would be nice too!

@slavapestov I've got an execution test coming along with the bug I'm really trying to fix.


ClassAttributesSeparationFixer - Re-add omitted only_if_meta option

Since #5920, the only_if_meta option was inadvertently removed from fixer description. This PR simply re-adds it.

Coverage remained the same at 92.358% when pulling f73470dd6f9aec6c838d547ce6d776bcb4696e9d on paulbalandan:omitted-option into a339a06f3116d18050173f45f0c574651bbaa259 on FriendsOfPHP:master.

Thank you @paulbalandan.


Update TIP to 2.9.0

Fix race condition bug in downloader code
Move TIPImageViewFetchHelper to supporting UIView
Improve rendered cache
Wide gamut color support

 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.Nolan O'Brien seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Could not login to OCI registry "public.ecr.aws": login attempt to https://public.ecr.aws/v2/ failed with status: 403 Forbidden

Description
Hi,
I've been managing Karpenter with the helm chart + terraform for a while now, but I've had login issues since yesterday.
Error: could not login to OCI registry "public.ecr.aws": login attempt to https://public.ecr.aws/v2/ failed with status: 403 Forbidden
The setup is really simple:
provider "aws"{
  alias = "use1"
  region = "us-east-1"
}

data "aws_ecrpublic_authorization_token" "token" {
    # https://registry.terraform.io/providers/hashicorp/aws/latest/docs/data-sources/ecrpublic_authorization_token
    provider = aws.use1
}

resource "helm_release" "karpenter" {
  namespace           = "kube-system"

  name                = "karpenter"
  repository          = "oci://public.ecr.aws/karpenter"
  repository_username = data.aws_ecrpublic_authorization_token.token.user_name
  repository_password = data.aws_ecrpublic_authorization_token.token.password
  chart               = "karpenter"
  # https://github.com/aws/karpenter-provider-aws/releases
  version             = "0.37.0"
  wait                = false
}

I have already tried to do docker logout public.ecr.aws or helm registry logout public.ecr.aws.
Is there any other source to pull the chart from ? This ecr repo is a bit annoying. I see I'm not the only one having this issue (https://github.com/aws/karpenter-provider-aws/issues/3441#issuecomment-2106370162).
Thanks !

I'm going to guess you can't provide an authorization token to ECR public, could you try removing these lines?
data "aws_ecrpublic_authorization_token" "token" {
    # https://registry.terraform.io/providers/hashicorp/aws/latest/docs/data-sources/ecrpublic_authorization_token
    provider = aws.use1
}

The public ECR is currently the only official release source for the helm chart.

Thanks for your answer, took it from the official examples, seems there's an issue there then ! https://github.com/terraform-aws-modules/terraform-aws-eks/blob/323fb759d718ed1f295c7278ef89a2e2f060d4a1/examples/karpenter/main.tf#L147
Thanks again !

Why is this closed? I am hitting the same issue. What is the solution?

Remove repository_username and repository_password for public repositories.

Just tested today by installing Karpenter. As mentioned by @RobinFrcd, it works fine after removing repository_username and repository_password for public repositories.
Additionally, we can safely remove data "aws_ecrpublic_authorization_token" if we don't use private ECR repositories.

docker logout public.ecr.aws
This works
Reference : https://gallery.ecr.aws/karpenter/karpenter

Logout via helm cli works fine.
$ helm registry logout public.ecr.aws

Check below commands for works well.
$ helm fetch oci://public.ecr.aws/karpenter/karpenter-crd


abnormal nocifensive behavior

J:200749 talks about changes in abnormalities in nocifensive behavior after UV exposure. We do not have any terms for nocifensive behavior. I found a definition of nocifensive behavior here http://www.therapyofpain.com/nocifensive-behavior.htm. This seems like it should be under abnormal sensory capabilities/reflexes/nociception. I think we can't put it under abnormal touch/ nociception since then it would end up under integument as well as behavior.
Reported by: sbello
Original Ticket: obo/mammalian-phenotype-requests/1672

See https://books.google.com/books?id=9LpxCwGcV6oC&pg=PA334&lpg=PA334&dq=nocifensive+vs+nociceptive&source=bl&ots=ity_VclZNc&sig=7KUV2_N3ipyO4QtWa_L7JNrqUgw&hl=en&sa=X&ved=0ahUKEwiu8dbE64LPAhWD2T4KHRJcAwQQ6AEILzAC#v=onepage&q=nocifensive vs nociceptive&f=false nocifensive behaviors may be displayed in the absence of nociceptive signals, also need to distinguish nociception from pain

Currently we have nociception terms under behavior and integument but I'm not sure either of these is correct. Nociceptors may be found in tissues other than skin and most of the nociception terms are defined as ability to sense a stimulus rather than a behavioral response to a stimulus.
See PMID:27322240 http://www.mdpi.com/1420-3049/21/6/797/htm
PMID:26690128 http://www.mdpi.com/1422-0067/16/12/26151/htm
"Nociceptors innervate target tissue in muscles, joints, the digestive tract, and several internal organs, where they transduce noxious physical sensations into neural impulses that are transmitted along the cell’s axon to the dorsal horn of the spinal cord" PMID:26543372 https://www.ncbi.nlm.nih.gov/pmc/articles/PMC4624094/


drivers: flash: spi: Move to using select in Kconfig for SPI bus

Updated to the latest policy to select the SPI bus for flash controllers
Updated SPI flash samples to remove default selected symbols

Marking DNM to allow some board maintainer feedback based on @de-nordic's comment.

@galak Anyone in particular we should ping?

@galak is the DNM label still applicable?

@henrikbrixandersen, @dleach02 , @danieldegrasse can you guys take a look

@pdgendt  looks like this needs a rebase to resolve a merge conflict.

Architecture WG:

depends on <BUS> vs select <bus>
The <BUS> is only enabled if the corresponding DT_HAS_<>_ENABLED Kconfig option is enabled, which it is only if there's a Devicetree node enabled
This was done for all sensors in the tree, see this commit
@henrikbrixandersen mentions that they use another compatible whenever they extend an upstream driver, which then decouples the OoT implementation from the original one
@gmarull mentions how, before @galak's commit was merged, defconfig files were unmaintainable, and so going back to the old approach is not really maintainable
@mbolivar-nordic mentions that there are workarounds for sensors with multiple buses: https://github.com/zephyrproject-rtos/zephyr/blob/3454a2547ead7f867c75cf83b12a38a6ea4dc599/drivers/sensor/bme280/Kconfig#L11

@pdgendt I've rebased the branch


PDF output for latex research notes

The core cosmology library group is writing a research note about the library and would like to use latex (to be able to incorporate the equations that go into the library). Since the main means of publishing desc research notes is via github, it would be great if there was a facility to easily publish pdfs of notes written in latex.
Here's a link to the branch of CCL including the skeleton of the note: https://github.com/DarkEnergyScienceCollaboration/CCL/tree/research_note

@drphilmarshall I know we have discussed this need to convert between latex and pdf before.  Were we still interested in using pandoc for that or did you have another tool in mind? (My latex experience is rusty!!!)

I have a .travis.yaml for mkauthlist that does latex compilation as part of it's tests. It's a bit hairy because you need to apt-get some (but not all) of the latex packages.

Yeah, I have one of these as well. My plan is to get CCL's note working, and then show the travis solution to @tony-johnson next week. Great if you can take a look at what I do and suggest improvements!

OK, I have a working version of the PDF deployment at 0453ef9 in PR #32. I'll try and get the CCL repo up and running next.

Ah - we don't have a travis.com account, and so cannot have travis run on private repos like CCL. I can put all the things you need in the research_note branch, but they won't run until CCL goes public. Sorry, @c-d-leonard ! I'll talk to Tony about setting up Jenkins instead.


Fixing wrong arguments order in instanceof's function in pyez_rpc task

Hello,
After trying some scripts I got a weird error.
After investigations I found it and it is the order of the argument in the isinstance function in pyez_rpc task.
I fixed it and I am publishing a new release right now.
So unfortunately we need to consider release 0.2.6 as buggy.

I removed release 0.2.6 as it was buggy.


Boolean dissimilarity bug

Current state of "boolean similarity" distance metrics (such as RUSSELL_RAO, DICE, etc.) are incorrectly treating values in vectors as actual continuous values. This can yield negative distances and in algorithms like RadiusNeighbors causes a radius search in the negatives. This is a bug

This was fixed in 52127823eab2a3f25a7291afb68c87cd50497849 with the addition of the BooleanSimilarity class.


Acturator error on teapot

Description
deploying 2022 code to teapot their is an error about the actuator
Steps to Reproduce

deploy/ run sim of 2022 code to teapot

Expected behavior:
no errors when deploying the 2022 code to teapot
Actual behavior:
actuator error
Required Information

What tag/branch/release of Robot code are you using?
2022 base code

Useful Information

latest wiplib and python

obsolete and we're into 2023 code now


Add onMultiplyScrollableMonths to SingleDatePicker

Fix #1768

Coverage decreased (-0.3%) to 84.644% when pulling c926f62309760de4796b2a9a017daea707e0ff42 on mmarkelov:Add_onMultiplyScrollableMonths_to_SingleDatePicker into cc1e3397a3c75f648dbdb4fac3d07e4322937f48 on airbnb:master.

@ljharb youp! sure!

@mmarkelov it'd be ideal to add new tests instead of changing existing ones; that way it's super clear it's not a breaking change :-)


Project scope and architectural change process

This is in Draft for intentionally to prevent this from merging too soon.
Signed-off-by: Steven Royer<EMAIL_ADDRESS>

For the Architecture level items this looks good.  Should we add that an Issue will be opened for the architecture change and marked with the Architecture label?  This way we can still have the general items that are contributed be reviewed and merged without the time limit delay with the approval.


expose HVAC current_tempature as a sensor

Hello,
Would it be possible to expose the current temperature field as a sensors in HA? - for example, this is the climate entry the integeration generate for my device:
hvac_modes:
  - fan_only
  - heat
  - 'off'
  - cool
min_temp: 5
max_temp: 35
target_temp_step: 0.5
fan_modes:
  - low
  - auto
  - medium
  - high
preset_modes:
  - comfort
  - eco
current_temperature: 22
temperature: 20
fan_mode: auto
preset_mode: comfort
icon: mdi:home-thermometer
friendly_name: Name
supported_features: 25

What is the use case?  I"m sure you can access the current_temperature attribute on the climate entity in most contexts you would want this.

thanks for your reply, its pretty simple, history tracking and simple display (without templating/helpers) on a dashboard.

History tracking is already available on climate entities.  And displaying attributes in an entities card is not difficult.
I don't think duplicating information into multiple entities is a useful goal, as it will affect performance, and if added at the integration level, everyone using the integration takes the performance hit whether they wanted it or not. Whereas using a template entity if you really need the separate sensor entity rather than an attribute only affects those who need it.

thanks for taking the time looking into it...


Remove version: Ryujinx.Ryujinx.Ava version 1.1.1211

part of #179207
All InstallerUrls returned 404 Not Found

https://github.com/Ryujinx/release-channel-master/releases/download/1.1.1211/test-ava-ryujinx-1.1.1211-win_x64.zip

Microsoft Reviewers: Open in CodeFlow

URL: https://github.com/Ryujinx/release-channel-master/releases/download/1.1.1211/test-ava-ryujinx-1.1.1211-win_x64.zip
Response status code does not indicate success: 404 (Not Found).
(Automated message - build 896)


Rewrite current version defined in package.json using semver

Hi!
I just cloned your repo and I wasn't able to install the dependencies through npm install:

I realised your current master version is not compatible with npm's versioning system. Seems that a valid version is required even to pull the dependencies!
https://docs.npmjs.com/getting-started/semantic-versioning
I just set the missing patch/minor release number to 0 and now works fine :)

Had no idea this was the case. Thanks for pointing it out and even submitting a PR! :)


Copy substrait proto files by cmake script during configure time

Auto copy substrait proto files by cmake script during configure time, then we can easily  update substrait version.

cc @assignUser @kgpai, can you help to review when you have some time? thanks

Do I understand correctly that previously you had to manually download and edit the substrait proto files?

Do I understand correctly that previously you had to manually download and edit the substrait proto files?

Yes

A good way might be to allow them to supply a local archive of the substrait repo via the URL download method?

Hi @assignUser, can you give some example? Thank you!

@leoluan2009 this allows us to use arbitrary commits/refs but can be overridden to use a local archive too:
if(DEFINED ENV{VELOX_SUBSTRAIT_URL})
  set(VELOX_SUBSTRAIT_SOURCE_URL "$ENV{VELOX_SUBSTRAIT_URL}")
else()
  set(VELOX_SUBSTRAIT_VERSION 6a74f40f0d835a447a7175b20dd430c7c39c5d11)
  set(VELOX_SUBSTRAIT_SOURCE_URL
      "https://api.github.com/repos/substrait-io/substrait/tarball/${VELOX_SUBSTRAIT_VERSION}"
  )
endif()

FetchContent_Declare(
  substrait
  URL ${VELOX_SUBSTRAIT_SOURCE_URL}
  PATCH_COMMAND git apply ${CMAKE_CURRENT_LIST_DIR}/sb.patch)
FetchContent_MakeAvailable(substrait)

FetchContent_Declare(   substrait   URL ${VELOX_SUBSTRAIT_SOURCE_URL}   PATCH_COMMAND git apply ${CMAKE_CURRENT_LIST_DIR}/sb.patch)
I find PATCH_COMMAND can not work in above code, my cmake version is 3.25.2

I have tested this locally in a test project and it works, I changed the patch location and name for the test, so that is probably why it's not working for you. Try substituting your path and name for the patch and it should work.

@leoluan2009 are you still working on this? I think it is a nice change to add!


WRONG DATE SHOWING in Coratia

(Required) Version Number: 7.1.8
Description: Month starting with wrong date.
Steps To Reproduce
In Coratia, 1st of Feb is Monday, whereas in JT calender it is showing as Sunday.
Similarly, today is showing monday, but is tuesday.
In India & US, showing correct dates and days.
Expected Behavior
Additional Context
I have updated the pod today by
platform :ios, '10.0'
use_frameworks!
target '' do
pod 'JTAppleCalendar', '~>; 7.1'

https://github.com/patchthecode/JTAppleCalendar/issues/252

Thanks for your prompt reply. I am using the configuration below. Above link #252 didn't helped me solve my issue.
`func configureCalendar(_ calendar: JTAppleCalendarView) -> ConfigurationParameters {
let startDate = Date(timeIntervalSince1970: TimeInterval(Y))
    return ConfigurationParameters(
        startDate: startDate,
        endDate: Date(),
        calendar: .current)
}`

Hi, did you look at the link above?
It mentions using a DateFormatter
Are you saying that using a dateformatter doesnt resolve your issue?

Apologies.
I had hard coded the weeks in the header (Su, Mo, Tu, We, Th, Fr, Sa) but forgot to use firstDayOfWeek: .sunday in configureCalendar.


Add ability to start range at 0

Add a checkbox to enable/disable starting the bounding range at 0.

Delivered to master.
I added a property to GraphRenderer for starting the range at 0. This just works by setting the bounding range and then re-initializing, which updates everything including the current range/domain from the data again. The initialize method checks the property and sets the bounding range to start at 0 if the property is switched on.
Commit
commit 1cbfa056ca6149b799fbe85f9ef8ed28c9196703
Author: Alex Hunt<EMAIL_ADDRESS>Date:   Tue Apr 12 16:07:12 2022 -0400
Issue47, Issue48: Added checkboxes for showing/hiding tickmarks and for starting the bounding range at 0 or not.

Testing

Tested that the "Start range at 0" checkbox is initially unchecked
Tested that switching it on/off makes the range start at 0 or at the lowest point in the data and that the graph updates immediately to show this.
Tested when zoomed in the zoom resets when the checkbox is checked/unchecked back to the bounding range.


Error: x must be a string of length 1

When trying to generate the results document within the Inspection report, I get the following error:
Error: `x` must be a string of length 1
This happens both client-side and server-side.
Since I can't attach R and RDS files, here is the inspection_report.R inline:

fftempdir <- if (Sys.getenv("FFTEMP_DIR") == "") "~/fftemp" else Sys.getenv("FFTEMP_DIR")
options(fftempdir = fftempdir)

cdmDatabaseSchema <- "cdm"
resultsDatabaseSchema <- "results"
vocabDatabaseSchema <- cdmDatabaseSchema
oracleTempSchema <- NULL
runVocabularyChecks <- TRUE
runDataTablesChecks <- TRUE
runPerformanceChecks <- TRUE
runWebAPIChecks <- TRUE
sqlOnly <- FALSE
verboseMode <- TRUE
smallCellCount <- 5
baseUrl <- "http://localhost:8080/WebAPI"
outputFolder <- "/root/filip/sandbox/kcs"
outputFolderCdmI <- paste0(outputFolder, "/cdmi")
authors <- "Filip Maljkovic"
databaseId <- "HELIANT"
databaseName <- "Heliant Health"
databaseDescription <- "Heliant Health is a Health information system (HIS), with capabilities of EHR (electronic health record) and follows many hospital business processes, from patient care, financial information, to drug and material maintenance."

library(CdmInspection)

results <- CdmInspection::cdmInspection(connectionDetails, cdmDatabaseSchema = cdmDatabaseSchema, resultsDatabaseSchema = resultsDatabaseSchema, vocabDatabaseSchema = vocabDatabaseSchema, oracleTempSchema = oracleTempSchema, databaseName = databaseName, runVocabularyChecks = TRUE, runDataTablesChecks = TRUE, runPerformanceChecks = TRUE, runWebAPIChecks = TRUE, smallCellCount = smallCellCount, baseUrl = baseUrl, sqlOnly = FALSE, outputFolder = outputFolderCdmI, verboseMode = verboseMode)

CdmInspection::generateResultsDocument(results = results, outputFolder = outputFolderCdmI, authors = authors, databaseDescription = databaseDescription, databaseName = databaseName, databaseId = databaseId, smallCellCount = smallCellCount)

Here's the link for the intermediate inspection_results.rds

See the message in teams @filipmaljkovic

The problem was in the fact that the vocabulary table was loaded twice, so the row with ID "None" was in the table two times, which caused the error in question.
The problem was solved by truncating the vocabulary table and reloading the CSV file generated by Athena.


Doc Comment throws can't link to Exceptions in other Modules

Currently the throws tag only accepts unscoped identifiers, there is no way to do:
interface MyInterface {
    /// @throws ::Test2::MyException: when it breaks
    op() throws ::Test2::MyException
}

instead the parser expects this:
interface MyInterface {
    /// @throws MyException: when it breaks
    op() throws ::Test2::MyException
}

Both could work in theory.

If we allow scoped identifiers, then the parser can resolve the link using that scoped identifier.

But even if we keep the current behavior (only unscoped identifier), we can still find the exception in the exception specification and link against that. We already check to ensure that anything in an @throws tag matches up with a specified exception.

Option 2 seems more convenient, but is more 'magical' than the option 1.

I don't see how Option 2 could work. For example, the following is valid Slice:
op() throw ::M1::MyException, ::M2::MyException

Which MyException does an unqualified "@throws MyException" refer to?

Yes you're right, option 2 is unworkable.
Option 1 is the only sane choice.


Incompatibility Issue with<EMAIL_ADDRESS>on arm64 CPU Architecture

Problem Description
During the execution of the yarn install command, an error occurs indicating that the<EMAIL_ADDRESS>package is not compatible with the arm64 CPU architecture.
How can I solve this?
Logs
 yarn install
yarn install v1.22.19
[1/5] 🔍  Validating package.json...
[2/5] 🔍  Resolving packages...
warning react-native-android-location-enabler ><EMAIL_ADDRESS>The library contains critical security issues and should not be used for production! The maintenance of the project has been discontinued. Consider migrating your code to isolated-vm.
warning react-native-android-location-enabler > degenerator ><EMAIL_ADDRESS>The library contains critical security issues and should not be used for production! The maintenance of the project has been discontinued. Consider migrating your code to isolated-vm.
[3/5] 🚚  Fetching packages...
error<EMAIL_ADDRESS>The CPU architecture "arm64" is incompatible with this module.
error Found incompatible module.
info Visit https://yarnpkg.com/en/docs/cli/install for documentation about this command.

Version

react-native : 0.72.4
"react-native-android-location-enabler : 2.0.0

Additional Information
Apple M1 Pro  - macOS Ventura 13.5.2
Yarn Version: v1.22.19

Oh, didn't see that coming, I'll check asap.

I just released a new version 2.0.1 can you try with this version please ?

@Richou
It will now install well.
Thank you for your quick action.

@chlee1001 It still does not work ?

@Richou
It works fine.
Thank you 👍


Typechecking = fails

Verification of this module fails with Unsupported comparison op = with args [].
(define-keyset 'auction-admin-keyset
  (read-keyset "auction-admin-keyset"))

(module auction 'auction-admin-keyset
  (defschema state step:string)
  (deftable table:{state})

  (defun start-auction:bool ()
    (let ((step:string (at 'step (read table 'STATE_ROW))))
      (enforce (= step 'NOT_STARTED))))
)

However, if we change start-auction to:
(defun start-auction:bool ()
  (enforce (= (at 'step (read table 'STATE_ROW)) 'NOT_STARTED)))

... now it verifies successfully.
I logged the AST output by the typechecker in both cases:

Failing case
[Binding {_aNode = let0::bool, _aBindings = [("step"(let0_step1::string),App {_aNode =
at2::string, _aAppFun = FNative {_fInfo = , _fName = "at", _fTypes = (idx:integer list:[<l>]
-> <a>) :| [(idx:string object:object:<{o}> -> <a>)], _fSpecial = Nothing}, _aAppArgs =
[Prim {_aNode = string3::string, _aPrimValue = PrimLit "step"},App {_aNode =
read4::object:{auction.state [step:string]}, _aAppFun = FNative {_fInfo = , _fName = "read",
_fTypes = (table:table:<{row}> key:string -> object:<{row}>) :| [(table:table:<{row}> key:string
 columns:[string] -> object:<{row}>)], _fSpecial = Nothing}, _aAppArgs = [Table {_aNode =
auction.table5::table:{auction.state [step:string]}, _aTableName = "table"},Prim {_aNode =
string6::string, _aPrimValue = PrimLit "STATE_ROW"}]}]})], _aBody = [App {_aNode =
enforce7::bool, _aAppFun = FNative {_fInfo = , _fName = "enforce", _fTypes = (test:bool
msg:string -> bool) :| [], _fSpecial = Nothing}, _aAppArgs = [App {_aNode = =8::bool,
_aAppFun = FNative {_fInfo = , _fName = "=", _fTypes =
(x:<a[integer,string,time,decimal,bool,[<l>],object:<{o}>,keyset]>
y:<a[integer,string,time,decimal,bool,[<l>],object:<{o}>,keyset]> -> bool) :| [],
_fSpecial = Nothing}, _aAppArgs = []}]}], _aBindType = let}]

Successful case
[App {_aNode = enforce0::bool, _aAppFun = FNative {_fInfo = , _fName = "enforce",
_fTypes = (test:bool msg:string -> bool) :| [], _fSpecial = Nothing}, _aAppArgs =
[App {_aNode = =1::bool, _aAppFun = FNative {_fInfo = , _fName = "=", _fTypes =
(x:<a[integer,string,time,decimal,bool,[<l>],object:<{o}>,keyset]>
y:<a[integer,string,time,decimal,bool,[<l>],object:<{o}>,keyset]> -> bool)
:| [], _fSpecial = Nothing}, _aAppArgs = [App {_aNode = at2::string,
_aAppFun = FNative {_fInfo = , _fName = "at", _fTypes = (idx:integer list:[<l>]
-> <a>) :| [(idx:string object:object:<{o}> -> <a>)], _fSpecial = Nothing},
_aAppArgs = [Prim {_aNode = string3::string, _aPrimValue = PrimLit "step"},App
{_aNode = read4::object:{auction.state [step:string]}, _aAppFun = FNative
{_fInfo = , _fName = "read", _fTypes = (table:table:<{row}> key:string ->
object:<{row}>) :| [(table:table:<{row}> key:string columns:[string] ->
object:<{row}>)], _fSpecial = Nothing}, _aAppArgs = [Table {_aNode =
auction.table5::table:{auction.state [step:string]}, _aTableName = "table"},Prim
{_aNode = string6::string, _aPrimValue = PrimLit "STATE_ROW"}]}]},Prim {_aNode
= string7::string, _aPrimValue = PrimLit "NOT_STARTED"}]}]}]

Lightly edited version of the failing case:
App {
  _aAppFun = FNative {
    _fInfo = ...,
    _fName = "enforce"
  },
  _aAppArgs = [
    App {
      _aAppFun = FNative {
        _fInfo = ...,
        _fName = "="
      },
      _aAppArgs = []
    }
  ]
}

In particular note that the string "NOT_STARTED" is shown correctly as an argument to =` in the working case but doesn't even appear in the failing case.

cc @emilypi @slpopejoy @bts

For future reference, you can get debug output on a function using _inferTestFun in TypecheckSpec.hs

I changed the variable name to s as step is a reserved word, I'm surprised that didn't fail for you.

This was exactly the problem. Seems the issue here is really the error message.


Profile: Profile Details module - Inline Edit component

A follow-up task for https://zube.io/x-team/xp-formerly-auto/c/1956
To facilitate the functionality development, we'll need a generic inline-edit component that will support onSave and onCancel custom callbacks to do respectively saving data and reverting it to the previous version. We should keep in mind it might be used for different unrelated fields on the admin dashboard, so it should be a generic one. Also, it would be great to support saving by clicking Enter and canceling edit mode by clicking Esc.
On one of the last projects I implemented a similar component and would like to share a recording of how it works:

It was implemented the way it replicates Trello's similar component, but for our purposes, we can simplify that. I can share the code if required 😄
@mccrodp Do we want to have similar Save/Cancel button when in edit mode? Or what are your expectations of in-edit mode?

@nicksp, considering my comment on respecting the controls shown to edit fields based on their type, I'd like to choose the option that is as generic as possible for fields of simple string/text type. If this means having a textfield where clicking Enter is save and clicking out of the area equates to cancel, then that's fine also (if it preserves edits locally while in the window if you need to go back to the edit component after scrolling or clicking elsewhere on the page for example).
What do you suggest in relation to this, I'm open to options?

@mccrodp Given the current scope, we will have 3 types of in-edit controls: number field (for rate), textarea for text fields, and date picker for availability dates. To make the editing flow clear and consistent regardless of the control type, I suggest showing extra Save/Cancel buttons (as shown in the gif above) for all of the controls in edit mode. Additionally, for all types of controls, we might have a common Enter/Esc functionality to respectively Save or Dismiss changes.


New datasets

Hi, I want to extend the possible datasets which I can utilise, however as I can see there is not code in the repository to generate the hyperedges from simple graph. Can I ask you to provide the code?
Thank you

I examined the "cora" dataset closely and found that some nodes do not participate in any hyperedge (around 47% of nodes do not participate). The paper states that the hyperedges are constructed from initial pairwise interactions. I verified that every node in the original edges of the "cora" dataset has a neighbor. Can you explain what I am misunderstanding?

Hi @levtelyatnikov ,
Thanks for the question. Regarding the cora dataset, we directly obtained from Yadati et al. (2019), the HyperGCN paper, as we stated in our paper (page 8). Thus we did not generate this dataset nor include codes for generating hyperedges from a simple graph. Could you let me know where you see the claim on generating cora from simple edges in our paper?
Regarding how Yadati et al. (2019) generate their datasets, please check section 12 in their paper. I do not think they generate hypergraphs from simple graphs either.
Finally, to include your own datasets, please check how we store the data. For example, see here in our code.
Best,
Eli

Hi @levtelyatnikov ,
Since there's no more follow-up, I close the issue for now. Please feel free to reopen the issue if you have further questions.
Best,
Eli


Will this sample update to aspnetcore 2.0?

I want to use Nlog in aspnetcore 2.0 & store the logs in SQLServer.But I have no idea to do that.

Yes, as soon as I get time. the last time I checked, I was waiting for NLog at the time. @304NotModified NLog to 2.0 ready?
Greetings Damien

partly, not in RTM. But that was also the case for .NET Core 1
See also https://github.com/NLog/NLog.Extensions.Logging/wiki for .NET/ASP.NET Core 2 tutorials


If jargon is to be used, adequately defining those definitions without relying upon external links to do so.

~_~ Fat finger, I did


chore(master): release git-jira-branch 2.2.0

:robot: I have created a release beep boop
2.2.0 (2024-11-09)
Features

add tidy command that removes branches for finished tickets (#509) (e061186), closes #257
create: allow creation of branches without a type prefix (#525) (de3daa4)

Bug Fixes

deps: bump the production-dependencies group with 3 updates (#465) (cbe5d74)
deps: bump the production-dependencies group with 7 updates (#459) (5acb477)
deps: bump the production-dependencies group with 7 updates (#462) (30d2431)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/ccntrq/git-jira-branch/releases/tag/git-jira-branch-v2.2.0 :sunflower:


Compilation error on Ubuntu 20.04

I've followed the "getting started" section of the manual pretty closely, but unfortunately I get a compilation error while trying to build scopehal-apps:
[  1%] Copying shaders...
[  2%] Built target xptools
[  2%] Copying styles...
[  2%] Copying icons...
[  3%] Built target log
[  3%] Copying icons...
[  6%] Built target protocolshaders
[  6%] Copying fonts...
[ 10%] Built target ngrendershaders
[ 10%] Copying gradients...
[ 10%] Built target styles
[ 10%] Built target shaders
[ 13%] Built target rendershaders
[ 14%] Built target ngcomputeshaders
[ 14%] Built target icons
[ 14%] Built target nfd
[ 14%] Built target ngfonts
[ 14%] Built target ngicons
[ 14%] Copying protocol shaders...
[ 14%] Built target gradients
Error copying file (if different) from "/home/robbe/glscopeclient/scopehal-apps/build/lib/scopeprotocols/shaders/CMakeFiles/protocolshaders" to "/home/robbe/glscopeclient/scopehal-apps/build/src/ngscopeclient/shaders".
make[2]: *** [src/ngscopeclient/CMakeFiles/ngprotoshaders.dir/build.make:58: src/ngscopeclient/CMakeFiles/ngprotoshaders] Error 1
make[1]: *** [CMakeFiles/Makefile2:1652: src/ngscopeclient/CMakeFiles/ngprotoshaders.dir/all] Error 2
make[1]: *** Waiting for unfinished jobs....
[ 15%] Built target graphwidget
[ 34%] Built target scopehal
make: *** [Makefile:141: all] Error 2

I've tried to revert the #539 PR, but that did not fix it for me.
I'll try to figure out what goes wrong and will update this issue with my findings!
P.S.: Another (small) note I have is that it's impossible to just copy-paste the command line instructions from the PDF (chrome PDF reader -> zsh). For some reason, it seems to add spaces before and after every special character. Maybe it would be nice to have some bash script to do the bulk of the setup work instead?

From the looks of it the .spv files were already in the destination folder somehow, so I just commented out the copy commands from the CMake files, and that seemed to have worked.
No idea how it came up with the (wrong) source path for them though, there's nothing there...

The cmake from Ubuntu 20.04 is probably too old (buggy) as there is not such issue with Ubuntu 22.04 LTS or more

The cmake from Ubuntu 20.04 is probably too old (buggy) as there is not such issue with Ubuntu 22.04 LTS or more
Could you try again with your Ubuntu 20.04 on master ?

I have since upgraded to 22.04, but also haven't tried glscopeclient anymore.
Feel free to close the issue in the meantime, I'll re-open should I hit it again in the future!

I think this was a race condition between the builds of glscopeclient and ngscopeclient that I fixed a few months back. Closing, reopen if it comes back.


remove error return in ci

Goals:

fix the error in .github/workflows/publish-docarray-org.yml which causes this error https://github.com/docarray/docarray/actions/runs/3594725408/jobs/6053362924

Codecov Report
Base: 87.85% // Head: 37.42% // Decreases project coverage by -50.42% :warning:

Coverage data is based on head (a86a5f0) compared to base (ae49969).
Patch has no changes to coverable lines.

Additional details and impacted files
@@             Coverage Diff             @@
##             main     #888       +/-   ##
===========================================
- Coverage   87.85%   37.42%   -50.43%
===========================================
  Files         138      138
  Lines        7137     7137
===========================================
- Hits         6270     2671     -3599
- Misses        867     4466     +3599

Flag
Coverage Δ

docarray
37.42% <ø> (-50.43%)
:arrow_down:

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

 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.


Dex strings skip every second byte in 9.1

Describe the bug
Dex strings are displayed incorrectly in 9.1, skipping every second byte.
To Reproduce
Steps to reproduce the behavior:

Import a dex file as a single file in 9.1
Carry out analysis with default auto-analyzers.
Once complete, head over to the strings_data section.
Open up any string_data structure.
Observe that while the comment and the name of the structure show the correct string, the "data" field skips every second byte.
This effect also appears in Defined Strings window, as well as when the structure data is queried directly through the scripting API.

Expected behavior
As per 9.0.4, display the full string, with no missing bytes.
Screenshots

Attachments
Environment (please complete the following information):

OS: Windows 10
Java Version: 12.0.1
Ghidra Version: 9.1

Additional context

Likely related to #1209


feat: add e2e test case for yurtstaticset controller

What type of PR is this?

Uncomment only one /kind <> line, hit enter to put that in a new line, and remove leading whitespace from that line:
/kind bug
/kind documentation
/kind enhancement
/kind good-first-issue
/kind feature
/kind question
/kind design
/sig ai
/sig iot
/sig network
/sig storage

What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #1477
Special notes for your reviewer:

Does this PR introduce a user-facing change?

other Note

@xavier-hou PTAL

@y-ykcir  Could you add e2e test case for this new feature?
https://github.com/openyurtio/openyurt/pull/1516

@y-ykcir Could you add e2e test case for this new feature? #1516

Done. cc@xavier-hou

@rambohe-ch  @y-ykcir  lgtm

/lgtm