API Request DrawIO Export returns 400 Bad request

Describe the bug
When trying to use API for export DrawIO diagram according to doc    I get 400 Bad request error with the description:
{"errors":[{"start":{"line":1,"column":1,"offset":0},"error":" should have required property 'command'","path":""}]}
To Reproduce
Steps to reproduce the behavior:

Make request to API:
curl --location --request POST 'https://<your-api-gateway-id>.execute-api.<deployment-region>.amazonaws.com/Prod/resources' \ --header 'Authorization: Bearer <your-token> \ --header 'Content-Type: text/plain' \ --data-raw '{"elements":{"nodes":[], "edges": []}}'

Get response 400 Bad request error with the description:
{"errors":[{"start":{"line":1,"column":1,"offset":0},"error":" should have required property 'command'","path":""}]}

Expected behavior
Receive a URL that when clicked will open up DrawIO in the browser and show my graph.
Browser (please complete the following information):

curl 7.55.1 (Windows) libcurl/7.55.1 WinSSL

Hi @traublmak
Thanks for opening an issue
When you tried to export a DrawIO diagram, did you have any items visualised on the canvas, or was it blank?

Hi @traublmak
Thanks for opening an issue
When you tried to export a DrawIO diagram, did you have any items visualised on the canvas, or was it blank?

Hello. This issue about REST API. I tried to export diagram using API, not UI

Ok, I understand. Let me investigate, I will get back to you tomorrow.

I think the example in the README is incorrect. I am sorry for the trouble caused.
Could you try using a format similar to the one below? I have got this working successfully.
Note that the nodes and edges elements in the payload needs to be populated for components to appear in the DrawIO diagram. The example below will open an empty DrawIO diagram.
API=""
AWS_REGION=""
AUTH=""

curl -X POST "https://${API}.execute-api.${AWS_REGION}.amazonaws.com/Prod/resources" \
    --header 'Content-Type: text/plain' \
    -H "Authorization: Bearer ${AUTH}"  \
    --data-raw '{"elements":{"nodes":[], "edges": []}}'

Hi @traublmak, the issue is that you need to use a different API. If you go the aws-perspective-<accountNo><region>-ClientAPIStack stack in the CloudFormation console you should see in Outputs a parameter called DrawioRootUrl and that is the URL to use. We will update the docs to reflect this.

thank you @svozza, @ConnorKirk

Looks like you had success, @traublmak.  Good to know.
If you don't mind I'm going to re-open this so we don't forget to add the documentation fixes. :)

I am working to improve the README section for using the DrawIO API directly. This will be in the next release.


Bump diagnostics to a8faafc

Issue

Link:
Summary:

How

Screenshots

References

Checklist

[ ] Tests added
[ ] Cleaned up commit history (rebase!)
[ ] Documentation added
[ ] Thought about variable and method names

Superseded by #774


Scaling up: Response time too long

Hello,
We've been using the barefoot project for a few month now, and it has been an amazing discovery, and of a great help. But we're stuck with an issue of scaling our project.
We were using a bfmap of France (~1.4G), and now that we want to use the system of multiple countries at a time, we've made a larger bfmap(~4.2GB). But taking over 20GB or RAM and 100% or 16-core processors, it takes over 10 minutes for an average trip (~10km) to be mapmatched. Whereas with the bfmap of France, and with a smaller server, it used to take about 30 seconds to be mapmatched, for a same trip.
So we tried to work on your spark jobs implementation, but it seems that it only allows to mapmatch many trips at a time by dispatching the multiple trips to multiple spark slave machines. So It seems that it would not fix our latency issue, which needs to be reduced for EACH trip.
Would you have an idea or an advice to make the Barefoot work with a map of multiple countries (and ultimately with the entire world) ? must we use docker for each country ? a different server ?
Thank you very much for your help

How much memory (RAM) do the machines have? It sounds like it's swapping to disk, which you should definitely avoid.
The way you can avoid that for larger maps is a spatial sharding of your map region. To support this, the library allows to load only a region of the map that is stored in the database. You must then assign regions to different servers. I can give you some more hints on how to proceed on that, but please check first if the latency problem is caused by swapping.

Thank you for your answer.
Yes it is definitely a RAM issue. We're using 32GB RAM servers. And a matcher server of only France fills 18GB. So we we've started spatial sharding and it seems a good solution. Do you have other hints for such sharding ? Thank you !


Remove profile validation that disallows mod info if moderation is turned off

Please remove this validation, which was added recently:
invalid profile: moderator information is provided but moderator is not enabled
Moderators turn moderation off temporarily all the time, from anecdotal evidence.
The desktop client UX was designed to support providing and changing moderation info independent of whether moderation is off or on:

If the server wants to put in validation preventing that, I think we need to have a larger meeting where Wolf could revisit the designs and in general we could discuss whether the proposed benefits of this change justify the added scope.

Agree @rmisio
We shouldn't be introducing any new validation that breaks patterns we've introduced in the client for years... I saw this not to assign blame or anything, we have a big app and sometimes it's hard to know when a seemingly logical validation will cause problems for one of the client implementations.
I think the most prudent thing to do now is remove that validation until we have the time to discuss this further post-ETH lauch.

What branch is this on @rmisio I can't find it in the eth-master branch of go.


Batch Delete does not work with DateTimeOffset condition.

Hi All,
I have a project that is running on .Net Core 2.1, EF Core 2.1 and PostgreSql.  I am using EntityFramework-plus to perform a batch delete on one of my Entity that has a DateTimeOffset column.  I am trying to Delete all rows that has a saved time that is less than current time.  Instead of checking the condition all rows are deleted.  I recently updated to EF Core 2.1 and change the column to use DateTimeOffset instead of Datetime and it was working before.  The following code is what I am trying todo.
context.Entity
.Where(x => x.ExpirationTime <= DateTimeOffset.UtcNow)
.Delete();
Please let me know if you need anymore detail.
Sam

Thanks for the quick reply.

Hello @SamHuo213 ,
This issue will be closed since it has been answered.
Feel free to reopen it if you feel otherwise.
Best Regards,
Jonathan


fix: ignoring totalRegisters property

Thank you! I check too that I don't use the others props like page and onPageChange, and fix the types.
Check it


[TypeSpec Validation] Notify users if we suspect they might be using a different version of TypeSpec

Not a big problem anymore


support albion east

i tried to get it working, but im unsure how to change the api. https://east.albion-online-data.com/api/

Done


org: handover example-content repo to RENCI

[x] update issues & readme (CCV)
[x] walkthrough (CCV)
[x] check docs (RENCI)
[x] update docs (CCV)
[x] discuss issues (CCV)
[x] discuss any CCV-specific things and how to convert to RENCI infrastructure
[x] check repo settings (permissions, branch protection rules)
[x] discuss any special build steps (e.g. docker)
[x] discuss any changes

Closing this now the handover is complete.


Windows app not closing properly

After the latest update, Netron remains open taking up memory and CPU after closing the program. I must close it through task manager each time. I am on Windows 10

Is this happening reliably with3.3.2 or can you provide specific steps to reproduce this? Opening the app, loading a model and closing on Windows 10 1903 seems to work.

It happens reliably with 3.3.2 if I open more than one model at the same time. I tried reinstalling to no avail. I am on Windows 10 1803 and can't update due to it being a work computer.

Open App
File > Open... > select file
File > Open... > select file
File > Exit
Open Task Manager > Details > Netron is not not listed

Are you doing anything different?

Generally, I just double click the files to launch the app but if I use the above method it still remains open. Here is a screenshot with multiple instances of Netron open in task manager but the app is not open.

I also met this problem in windows10，and i installed the latest version.

I also met this problem in windows10,wtih 3.3.6.
And I'd love to try 3.4.1.
Thank you very much.

I'm also having this issue with Netron 4.5.4 and Windows 10 2004. I'd be happy to help you reproduce this behavior, if possible.

Hi, i have the same issue on Windows 10 using Netron through conda. It looks like the netron.stop() function stays  in an infinite loop.

@pArtheum this is a different issue. Can you investigate and submit a fix?

I am also experiencing the netron greedly consume the memory. When the issue is happen, it will consume all the dram. By adding dram from 8 to 16, it still try to consume all 16 and then consume the virtual memory.
os is win 10.
Also, I usually use netron to veiw two or three model at the same time.

i have the same issue, is there any solutions

I have the same issue.
I also have rogue processes taking up RAM even if I close Netron

No Netron icon open on my desktop (v4.6.4). Originally launched by double clicking on a file. Memory isn't growing at any radical pace, but CPU staying up around 14-20%. My fans were going on my laptop, so I was wondering what was going on.

I decided to create a dump, but without symbols it makes it hard to see what is going on below the Windows parts.
ntdll.dll!RtlpFreeHeapInternal()	Unknown
ntdll.dll!RtlFreeHeap()	Unknown
Netron.exe!00007ff75dc005f9()	Unknown
Any chance we could get symbols to help debug this?

@amirebrahimi DEVELOPMENT.md has instructions to run the Electron app locally. setting-up-symbol-server for getting Electron symbols.

Thanks @lutzroeder. I've never developed with Electron, so I'm seeing nowt his may be an issue with Electron then.

This could also be the issue: https://github.com/atom/atom/issues/17672#issuecomment-736758841
Perhaps try replace https://github.com/lutzroeder/netron/blob/main/source/app.js#L891 with fs.writeFile() (async version)

Or rather, change https://github.com/lutzroeder/netron/blob/main/source/app.js#L67
to 'before-quit'

@amirebrahimi can you try locally if this fixes the issue? To create an unsigned release build run npx electron-builder --win.

I'd be happy to when I can find a solid repro, which you've asked for :) I tried a variety of things, such as launching multiple files at once and closing very quickly (ctrl-w), closing apps from the taskbar while not focused, closing with the X button.

Oddly enough. Just after I wrote the previous comment above I then noticed I had a rogue process, so I had a hunch and finally was able to boil it down to a simple repro:

Double click on one .onnx file to open in Netron
Double click on a different .onnx file to open in Netron
Switch to another app (I had task manager open to verify the rogue process)
Close the Netron apps directly from the taskbar (i.e. without focusing)

The main issue is that even though a second instance is supposed to quit (via electron.app.quit) it continues past that statement   and executes remaining code, which I think fires up the render sub-process, but then there is nothing to close the app except the task manager because there is no window. By returning immediately from the constructor this doesn't happen.


Camera flash

Steps to Reproduce
When device dont have flash specially in case of IOS , Ipad and open camera
Actual results: <CameraException(setFlashModeFailed, Device does not have flash capabilities)>

Code sample

CameraException(setFlashModeFailed, Device does not have flash capabilities)
unable to capture image in this scenario and on capture it do nothing just keep showing the camera view

@avinashMobile
Which camera are you testing (front or back camera)? And which device are you having this issue (model, iOS version)? Does flash work well with device's camera app?
Please provide a completed and minimal reproducible code sample so that we may verify this. Also, please provide the output of flutter doctor -v as well.
Thank you!


Leaderboard: Update my time & position after #356

Update my time & position after #356. Just to help out a bit since it's a lot of manual work for you

Thx. I apply them in batches, that's why it wasn't reflected here yet.


[backport] Bump ReadTheDocs configuration to version 2

Backport of #5491 by @kmaehashi

[automatic post] Jenkins, test this please.

Merging as RTD test passed

Jenkins CI test (for commit a3f0e337ea22d3fa48a9627f9cc0a90916f9bf5c, target branch v9) succeeded!


Projected middleware should return serialized data.

Some APIs return JSON data, which is de-serialized and returned to the caller.
Such APIs should be modified to return the JSON data directly.

Fixed.


Linux installation instructions linked for installing Singularity does not install correct version

The instructions for installing on Linux provided from the docs:

For Linux, I suggest using the provided packages, not building from source.
Details are available here.
Be sure to use version 3 of Singularity, as it's the only version that supports the Singularity library!

The instructions lead to installing 2.5.x or 2.6.x. There is currently no stable packages for Singularity 3.0 so it is not possible to install from packages. Only option is building from source or using CSIF.

Thanks! @JulianToya, could you update the docs here to state that if you install singularity you must build from source?


I have followed every step correctly and reviewed the same , but i am not getting any report in my s3 bucket .

I have followed every step correctly and reviewed the same , but i am not getting any report in my s3 bucket .
There is no errors or issue that is been triggered, request you to assist us asap.

In this I have changed the LogArchive Account to one of my Member Account, where i created the S3 and would like to get the logs there.


first implementation of citeproc-ruby

This is the first implementation of citeproc-ruby in ruby-cff. This pull request also adds support for three citation styles (apa, harvard, ieee), includes their citationstyles.csl, and adopts the existing tests.
This pull request addresses #64.
While this is working in principle, more work is needed to map all CFF properties to citeproc.json, fix all failing tests, add more tests for utility methods, checks that one of the included styles is used, etc.

Coverage decreased (-5.0%) to 90.588% when pulling 4e15e08efe8e57efd547f392f776a4546c6eb870 on front-matter:main into 8676ecbcd724908dc447a02163cc2f79ac2de5f9 on citation-file-format:main.

I have added more tests, and support for name suffix and -particles.

Many thanks for this @mfenner, it looks great so far.
Just to keep you up to speed with immediate plans: we're going to cut a 0.8 release of this gem over the weekend so that we can get all the bug-fixes and additions so far up into GitHub, then we'll look to merge this work in for the 1.0 release when we'll have a stable schema as well.

https://mainnet.rpc.rarichain.org/http


Heritage-177-links

Ticket URL:  https://national-archives.atlassian.net/browse/HERITAGE-177
About these changes
Changing the link styles on the home page to be black.
How to check these changes
Run branch locally, check styles on home page
Before assigning to reviewer, please make sure you have

[x] Checked things thoroughly before handing over to reviewer
[x] Checked PR title starts with ticket number as per project conventions to help us keep track of changes
[x] Ensured that PR includes only commits relevant to the ticket
[x] Waited for all CI jobs to pass before requesting a review
[x] Added/updated tests and documentation where relevant

Merging PR guidance
Follow docs\developer-guide\contributing.md
Deployment guidance
Follow docs\infra\environments.md

closing, incorrect code base.


Restyling and restructuring of Nav bar

I changed a little after I compared with how Rachel did (https://marvelapp.com/3f8d0id/screen/59789980)

Thanks!
On Mon, Oct 14, 2019 at 9:46 AM Claudio Del Valle<EMAIL_ADDRESS>wrote:

Closed #388 https://github.com/openforge/OPENFORGE.IO/pull/388.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/openforge/OPENFORGE.IO/pull/388?email_source=notifications&email_token=AF7TNH7UVPT3AESSXHALWU3QORZ2TA5CNFSM4I7EKSLKYY3PNVWWK3TUL52HS4DFWZEXG43VMVCXMZLOORHG65DJMZUWGYLUNFXW5KTDN5WW2ZLOORPWSZGOUGHNBUY#event-2710491347,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AF7TNH6OXQTEH5CHVKAWRHLQORZ2TANCNFSM4I7EKSLA
.


Update code_transformers

I just released a new version of code_transformers which doesn't include the test dependency, and doesn't have any tools for testing transformers included.
There is now a separate transformer_test package which includes the things that code transformers used to, so you can add that as a dev dependency if needed.

Cool, thanks!


Spelling error of RSA test command

There is a spelling error in the following command:
openssl rsautl -sign -in msg.txt -inkey prikey.pem -out sgined.txt -engine uadk
The "sgined.txt" should be "signed.txt".

send patch pls

send patch pls

OK


no matches for kind "Prometheus" in version "monitoring.coreos.com/v1alpha1"

What did you do?
I'm following the steps on the blog at https://coreos.com/blog/the-prometheus-operator.html
And ran kubectl create -f https://coreos.com/operators/prometheus/latest/prometheus-k8s.yaml
And got:
service "prometheus-k8s" created
error: unable to recognize "https://coreos.com/operators/prometheus/latest/prometheus-k8s.yaml": no matches for kind "Prometheus" in version "monitoring.coreos.com/v1alpha1"

What did you expect to see?
Not sure
What did you see instead? Under which circumstances?
service "prometheus-k8s" created
error: unable to recognize "https://coreos.com/operators/prometheus/latest/prometheus-k8s.yaml": no matches for kind "Prometheus" in version "monitoring.coreos.com/v1alpha1"

Environment

Prometheus Operator version:
insert image tag or Git SHA here

Kubernetes version information:

Client Version: version.Info{Major:"1", Minor:"10", GitVersion:"v1.10.4", GitCommit:"5ca598b4ba5abb89bb773071ce452e33fb66339d", GitTreeState:"clean", BuildDate:"2018-06-06T08:13:03Z", GoVersion:"go1.9.3", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"10", GitVersion:"v1.10.1", GitCommit:"d4ab47518836c750f9949b9e0d387f20fb92260b", GitTreeState:"clean", BuildDate:"2018-04-12T14:14:26Z", GoVersion:"go1.9.3", Compiler:"gc", Platform:"linux/amd64"}

Kubernetes cluster kind:
bare metal.
Not sure how it was installed. I have recently inherited it.

The very top of the blog post says:

Note: The instructions in this post are out of date. To try out the Prometheus Operator, view the latest Prometheus docs for an up-to-date guide to get started.

I recommend the getting started guide right from this repo: https://github.com/coreos/prometheus-operator/blob/master/Documentation/user-guides/getting-started.md
Let us know how it goes! :slightly_smiling_face:

Please consider make the deprecation notice more prominent.
Changing to red color might be a good idea.
Or the entire text should be hidden behind a zippy button, stating something like:
"This is deprecated, to read, click this button"
Which then shows the text after clicking the button.


Convert Docker Volumes to mountPoints

A Docker image can specify Volumes which are akin to mountPoints in the
appc spec. This commit converts Volumes to mountPoints allowing the user
to mount volumes with rkt's --volume option.
Since Docker doesn't name Volumes but the appc spec requires a name, we
generate a name by appending the path to "volume-". That is, if a Volume
has "/var/tmp" as path, the resulting mountPoint name will be
"volume-/var/tmp". A mountPoint name must be a valid ACName so any
invalid characters will be replaced by a "-".

Could you document this, please?

Done.

@iaguis :+1:

LGTM modulo question about duplicates.

Updated.

LGTM.

:smiley:


CVXPY defaults to MOSEK instead of Clarabel

Describe the bug
This page says the default solver is Clarabel. However, sometimes I get MOSEK instead:
$ py -c "import cvxpy as cp; x = cp.Variable(); prob = cp.Problem(cp.Maximize(x), [x <= 1]); print(prob.solve())"
mosek.Error: rescode.err_missing_license_file(1008): License cannot be located. The default search path is ':/Users/carlos/mosek/mosek.lic:'.
$ py -c "import cvxpy as cp; x = cp.Variable(); prob = cp.Problem(cp.Maximize(x), [x <= 1]); print(prob.solve(solver='CLARABEL'))"
1.0

Version

OS: macOS 14.6.1
CVXPY:

$ py -c "import cvxpy as cp; print(cp.__version__)"
1.7.0.dev0+92.14124e1

The default solver depends on what you have installed. It is based on a subjective judgment of reliability/speed:
https://github.com/cvxpy/cvxpy/blob/master/cvxpy/reductions/solvers/defines.py#L77
MOSEK is ranked higher than Clarabel if MOSEK is installed.

@SteveDiamond I think it'd be nice for users if CVXPY detected the missing license error and skipped to Clarabel. WDYT?

@rileyjmurray There are some corner cases, for example a user can have a Mosek license only for LP+MIP but not covering conic problems, and also with floating licenses with limited tokens the concept of being licensed may not be invariant in time. Also one has to be careful to clean up the licenses checked out in the test. And if the license is floating the test will introduce a tiny delay.
I assume we are talking about a situation when the user has some solver installed, but does not want to use it, for example because they have no license for it, or whatever other reason. In that case it might make sense if CVXPY just allowed the user to provide a list of solvers they want to blacklist.

Is there anything stopping Problem.solve from using try… except… internally to fall back to Clarabel if there's any error with the default solver (e.g. the mosek.Error above)? That would handle all such errors at solve time.


:bug: Wait until the resources is ready to start controller-manager

Summary
The controller runtime manager won't start if it watches a resource that hasn't existed since version 0.15.  So if we want to run the global hub operator under a bare kube environment. It needs to detect the necessary resources to be ready to start the manager.
Solution

We will first start a crd controller to detect the necessary resources, like ACM or Kafka.
To avoid the operator memory being overused. Instead of watching CRD directly, let the  controller use WatchesMetadata
Once the ACM resources are ready, then the crd controller starts the addon, and global hub controllers
The crd controller also records the states of Kafka resources and the triggered addon and global hub controllers.

Related issue(s)
References: https://github.com/kubernetes-sigs/controller-runtime/issues/2456
Fixes # https://issues.redhat.com/browse/ACM-10955

/hold

/retest

/unhold

/retest

/retest

/override ci/prow/sonarcloud
/override "SonarCloud Code Analysis"

/cc @clyang82

If the crd resources aren't ready. the operator will be running with the following log:
oc logs -f -l name=multicluster-global-hub-operator -n multicluster-global-hub
2024-04-18T05:25:41.385Z        INFO    setup   Go OS/Arch: linux/amd64
2024-04-18T05:25:41.402Z        INFO    setup   starting manager
2024-04-18T05:25:41.402Z        INFO    controller-runtime.metrics      Starting metrics server
2024-04-18T05:25:41.402Z        INFO    starting server {"kind": "health probe", "addr": "[::]:8081"}
2024-04-18T05:25:41.402Z        INFO    controller-runtime.metrics      Serving metrics server  {"bindAddress": ":8080", "secure": false}
I0418 05:25:41.402653       1 leaderelection.go:250] attempting to acquire leader lease multicluster-global-hub/multicluster-global-hub-operator-lock...
I0418 05:25:41.407270       1 leaderelection.go:260] successfully acquired lease multicluster-global-hub/multicluster-global-hub-operator-lock
2024-04-18T05:25:41.408Z        INFO    Starting EventSource    {"controller": "CRDController", "source": "kind source: *v1.PartialObjectMetadata"}
2024-04-18T05:25:41.408Z        INFO    Starting Controller     {"controller": "CRDController"}
2024-04-18T05:25:41.510Z        INFO    Starting workers        {"controller": "CRDController", "worker count": 1}

/hold

/hold

Did you test the operator in pure OpenShift cluster?

Yes. It works now.

/retest

/retest

/retest

override "ci/prow/sonarcloud"

/override ci/prow/sonarcloud

When start the operator in a OCP without ACM:
root@myan workspace/hub-of-hubs (br_wait_ready) » oc logs -f -l name=multicluster-global-hub-operator -n multicluster-global-hub
2024-04-26T16:01:56.522Z        INFO    setup   Go OS/Arch: linux/amd64
2024-04-26T16:01:56.540Z        INFO    setup   starting manager
2024-04-26T16:01:56.540Z        INFO    controller-runtime.metrics      Starting metrics server
2024-04-26T16:01:56.540Z        INFO    starting server {"kind": "health probe", "addr": "[::]:8081"}
2024-04-26T16:01:56.540Z        INFO    controller-runtime.metrics      Serving metrics server  {"bindAddress": ":8080", "secure": false}
I0426 16:01:56.540571       1 leaderelection.go:250] attempting to acquire leader lease multicluster-global-hub/multicluster-global-hub-operator-lock...
I0426 16:01:56.545371       1 leaderelection.go:260] successfully acquired lease multicluster-global-hub/multicluster-global-hub-operator-lock
2024-04-26T16:01:56.545Z        INFO    Starting EventSource    {"controller": "CRDController", "source": "kind source: *v1.PartialObjectMetadata"}
2024-04-26T16:01:56.545Z        INFO    Starting Controller     {"controller": "CRDController"}
2024-04-26T16:01:56.646Z        INFO    Starting workers        {"controller": "CRDController", "worker count": 1}

After installing ACM. It will start all the other controllers:
I0426 16:07:29.302458       1 utils.go:241] Wait MulticlusterGlobalHub created
2024-04-26T16:07:29.302Z        INFO    Starting EventSource    {"controller": "addonInstaller", "source": "kind source: *v1.ManagedCluster"}
2024-04-26T16:07:29.302Z        INFO    Starting EventSource    {"controller": "addonInstaller", "source": "kind source: *v1alpha1.ManagedClusterAddOn"}
2024-04-26T16:07:29.302Z        INFO    Starting EventSource    {"controller": "addonInstaller", "source": "kind source: *v1alpha1.ClusterManagementAddOn"}
2024-04-26T16:07:29.302Z        INFO    Starting EventSource    {"controller": "addonInstaller", "source": "kind source: *v1.Secret"}
2024-04-26T16:07:29.302Z        INFO    Starting Controller     {"controller": "addonInstaller"}
2024-04-26T16:07:29.303Z        INFO    Starting Controller     {"controller": "multiclusterglobalhub", "controllerGroup": "operator.open-cluster-management.io", "controllerKind": "MulticlusterGlobalHub"}
2024-04-26T16:07:29.303Z        INFO    Starting workers        {"controller": "multiclusterglobalhub", "controllerGroup": "operator.open-cluster-management.io", "controllerKind": "MulticlusterGlobalHub", "worker count": 1}
2024-04-26T16:07:29.303Z        INFO    Starting EventSource    {"controller": "conditionController", "controllerGroup": "operator.open-cluster-management.io", "controllerKind": "MulticlusterGlobalHub", "source": "kind source: *v1alpha4.MulticlusterGlobalHub"}
2024-04-26T16:07:29.303Z        INFO    Starting EventSource    {"controller": "conditionController", "controllerGroup": "operator.open-cluster-management.io", "controllerKind": "MulticlusterGlobalHub", "source": "kind source: *v1.Deployment"}
2024-04-26T16:07:29.303Z        INFO    Starting Controller     {"controller": "conditionController", "controllerGroup": "operator.open-cluster-management.io", "controllerKind": "MulticlusterGlobalHub"}
2024-04-26T16:07:29.303Z        INFO    Starting EventSource    {"controller": "multiclusterglobalhub", "controllerGroup": "operator.open-cluster-management.io", "controllerKind": "MulticlusterGlobalHub", "source": "kind source: *v1alpha4.MulticlusterGlobalHub"}
2024-04-26T16:07:29.303Z        INFO    Starting EventSource    {"controller": "multiclusterglobalhub", "controllerGroup": "operator.open-cluster-management.io", "controllerKind": "MulticlusterGlobalHub", "source": "kind source: *v1.Deployment"}
2024-04-26T16:07:29.303Z        INFO    Starting EventSource    {"controller": "multiclusterglobalhub", "controllerGroup": "operator.open-cluster-management.io", "controllerKind": "MulticlusterGlobalHub", "source": "kind source: *v1.StatefulSet"}
2024-04-26T16:07:29.303Z        INFO    Starting EventSource    {"controller": "multiclusterglobalhub", "controllerGroup": "operator.open-cluster-management.io", "controllerKind": "MulticlusterGlobalHub", "source": "kind source: *v1.Service"}
2024-04-26T16:07:29.303Z        INFO    Starting EventSource    {"controller": "multiclusterglobalhub", "controllerGroup": "operator.open-cluster-management.io", "controllerKind": "MulticlusterGlobalHub", "source": "kind source: *v1.ServiceAccount"}
2024-04-26T16:07:29.303Z        INFO    Starting EventSource    {"controller": "multiclusterglobalhub", "controllerGroup": "operator.open-cluster-management.io", "controllerKind": "MulticlusterGlobalHub", "source": "kind source: *v1.Secret"}
2024-04-26T16:07:29.303Z        INFO    Starting EventSource    {"controller": "multiclusterglobalhub", "controllerGroup": "operator.open-cluster-management.io", "controllerKind": "MulticlusterGlobalHub", "source": "kind source: *v1.Role"}
2024-04-26T16:07:29.303Z        INFO    Starting EventSource    {"controller": "multiclusterglobalhub", "controllerGroup": "operator.open-cluster-management.io", "controllerKind": "MulticlusterGlobalHub", "source": "kind source: *v1.RoleBinding"}
2024-04-26T16:07:29.303Z        INFO    Starting EventSource    {"controller": "multiclusterglobalhub", "controllerGroup": "operator.open-cluster-management.io", "controllerKind": "MulticlusterGlobalHub", "source": "kind source: *v1.Route"}
2024-04-26T16:07:29.303Z        INFO    Starting EventSource    {"controller": "multiclusterglobalhub", "controllerGroup": "operator.open-cluster-management.io", "controllerKind": "MulticlusterGlobalHub", "source": "kind source: *v1.MutatingWebhookConfiguration"}
2024-04-26T16:07:29.303Z        INFO    Starting EventSource    {"controller": "multiclusterglobalhub", "controllerGroup": "operator.open-cluster-management.io", "controllerKind": "MulticlusterGlobalHub", "source": "kind source: *v1.ConfigMap"}
2024-04-26T16:07:29.303Z        INFO    Starting EventSource    {"controller": "multiclusterglobalhub", "controllerGroup": "operator.open-cluster-management.io", "controllerKind": "MulticlusterGlobalHub", "source": "kind source: *v1.Namespace"}
2024-04-26T16:07:29.303Z        INFO    Starting EventSource    {"controller": "multiclusterglobalhub", "controllerGroup": "operator.open-cluster-management.io", "controllerKind": "MulticlusterGlobalHub", "source": "kind source: *v1.ClusterRole"}
2024-04-26T16:07:29.303Z        INFO    Starting EventSource    {"controller": "multiclusterglobalhub", "controllerGroup": "operator.open-cluster-management.io", "controllerKind": "MulticlusterGlobalHub", "source": "kind source: *v1.ClusterRoleBinding"}
2024-04-26T16:07:29.303Z        INFO    Starting EventSource    {"controller": "multiclusterglobalhub", "controllerGroup": "operator.open-cluster-management.io", "controllerKind": "MulticlusterGlobalHub", "source": "kind source: *v1.Secret"}
2024-04-26T16:07:29.303Z        INFO    Starting EventSource    {"controller": "multiclusterglobalhub", "controllerGroup": "operator.open-cluster-management.io", "controllerKind": "MulticlusterGlobalHub", "source": "kind source: *v1alpha1.ClusterManagementAddOn"}
2024-04-26T16:07:29.303Z        INFO    Starting EventSource    {"controller": "multiclusterglobalhub", "controllerGroup": "operator.open-cluster-management.io", "controllerKind": "MulticlusterGlobalHub", "source": "kind source: *v1.ManagedCluster"}
2024-04-26T16:07:29.303Z        INFO    Starting EventSource    {"controller": "multiclusterglobalhub", "controllerGroup": "operator.open-cluster-management.io", "controllerKind": "MulticlusterGlobalHub", "source": "kind source: *v1.ServiceMonitor"}
2024-04-26T16:07:29.303Z        INFO    Starting EventSource    {"controller": "multiclusterglobalhub", "controllerGroup": "operator.open-cluster-management.io", "controllerKind": "MulticlusterGlobalHub", "source": "kind source: *v1alpha1.Subscription"}
2024-04-26T16:07:29.303Z        INFO    Starting EventSource    {"controller": "backupController", "controllerGroup": "operator.open-cluster-management.io", "controllerKind": "MulticlusterGlobalHub", "source": "kind source: *v1alpha4.MulticlusterGlobalHub"}
2024-04-26T16:07:29.303Z        INFO    Starting EventSource    {"controller": "backupController", "controllerGroup": "operator.open-cluster-management.io", "controllerKind": "MulticlusterGlobalHub", "source": "kind source: *v1.Secret"}
2024-04-26T16:07:29.303Z        INFO    Starting EventSource    {"controller": "backupController", "controllerGroup": "operator.open-cluster-management.io", "controllerKind": "MulticlusterGlobalHub", "source": "kind source: *v1.ConfigMap"}
2024-04-26T16:07:29.303Z        INFO    Starting EventSource    {"controller": "backupController", "controllerGroup": "operator.open-cluster-management.io", "controllerKind": "MulticlusterGlobalHub", "source": "kind source: *v1.PersistentVolumeClaim"}
2024-04-26T16:07:29.303Z        INFO    Starting EventSource    {"controller": "backupController", "controllerGroup": "operator.open-cluster-management.io", "controllerKind": "MulticlusterGlobalHub", "source": "kind source: *v1.MultiClusterHub"}
2024-04-26T16:07:29.303Z        INFO    Starting Controller     {"controller": "backupController", "controllerGroup": "operator.open-cluster-management.io", "controllerKind": "MulticlusterGlobalHub"}
2024-04-26T16:07:29.534Z        INFO    Starting workers        {"controller": "addonInstaller", "worker count": 1}
2024-04-26T16:07:29.534Z        INFO    Starting workers        {"controller": "backupController", "controllerGroup": "operator.open-cluster-management.io", "controllerKind": "MulticlusterGlobalHub", "worker count": 1}
2024-04-26T16:07:29.535Z        INFO    Starting workers        {"controller": "conditionController", "controllerGroup": "operator.open-cluster-management.io", "controllerKind": "MulticlusterGlobalHub", "worker count": 1}


MAINT refactor feature_perturbation in Tree explainers

The switch between "interventional" to "tree_path_dependent" being silent is quite surprising.
Instead, I would propose to have an "auto" method that would switch to the method depending if the background dataset provided.
Then when providing a specific algorithm, we will in the future raise an error if the conditions are not met to run the algorithm. For the moment, I propose to raise a FutureWarning to ask the user to change to "auto" if this is really what they want (and therefore keep the code backward compatible).
I find this way more explicit, knowing that "tree_path_depedent" might have an important impact on the symmetric axiom expected by the users.

Hi @glemaitre,
Thanks for the PR!
To provide a bit of context for why this has taken so long: unfortunately repo was largely unmaintained 2022-23. However, as of last year we now have a very small group of maintainers who are working through the issue and PR backlog. More context here if you're interested: Join the team of Contributors & Maintainers!
(Side note: it would be great to have you as a contributor or even maintainer, especially with your experience from the scientific python ecosystem!)
I think your refactor above would be a good improvement. I'll take a look and sort out the merge conflicts with master.

Ouch, I forgot about this PR. Let me try to have another look at it :)


Joystick found on boot but not usable

If a joystick is plugged into the computer on startup it does not appear to be usable until it is unplugged and plugged back in.  The device is visible in dmesg and appears in the list of devices connected to the computer.  Calling devices.gamepads[0].read() however blocks indefinitely and does not seem to get events.
Restarting udev or triggering it does not solve the problem.
Not 100% sure that this is a problem with inputs but it's between that and the OS.
Ubuntu 16.04 minimal install on Neousys 2510-VTC.

Just found out about evtest.  Running that showed no events being created so it's not inputs.  I'll close the issue.


Add Firefox-extension link and mention in ReadMe

Hello!
Just discovered your extension, and was a bit saddened you don't have FF extension, but eventually found you do!
https://addons.mozilla.org/en-US/firefox/addon/marinawallet/?utm_source=addons.mozilla.org&utm_medium=referral&utm_content=search
If you can confirm it's made by you, it would be cool to add it on the readme!
Thanks!

Hello @Donach! Yes Marina is available on Firefox, the link is correct. But you opened your issue on the wrong repo :)

Oh sorry, was on phone and overlooked this 😆


[INFRA-2346] Deploy websdk

@yntema @rubinsingh @hubt
This now has everything except for npm publish

npm publish also enabled, just not tested yet since we need to deploy for that

👍

👍


Noranda

Can you take the 3D model from the ZTEM inversion and plot it as a cube with one corner extracted so that the 3D geology can be seen. Also make it clear where 275 m depth slice is.  Maybe plot the model from the surface to about 1km (or less)

I'll try to locate the models on Elliot's old computer.

@dougoldenburg Thoughts on this?

shows about 1300 m vertically
top shows depth slice at -225
Black like shows where -475 m is


style: remove unnecessary blank identifiers from assignments

This patch fixed the lint error which is https://staticcheck.io/docs/checks#S1005.

This change is

Codecov Report

Merging #51 (fe895b7) into main (1550f30) will increase coverage by 0.06%.
The diff coverage is 25.00%.

@@            Coverage Diff             @@
##             main      #51      +/-   ##
==========================================
+ Coverage   63.09%   63.15%   +0.06%
==========================================
  Files         124      124
  Lines       16711    16711
==========================================
+ Hits        10543    10554      +11
+ Misses       5631     5627       -4
+ Partials      537      530       -7

Impacted Files
Coverage Δ

tests/it/testenv.go
33.64% <25.00%> (ø)

internal/storagenode/logstream/sequencer.go
89.65% <0.00%> (-2.88%)
:arrow_down:

...metarepos/dummy_storagenode_client_factory_impl.go
60.40% <0.00%> (-1.35%)
:arrow_down:

internal/metarepos/report_collector.go
77.94% <0.00%> (+0.27%)
:arrow_up:

internal/metarepos/storage.go
81.06% <0.00%> (+0.46%)
:arrow_up:

pkg/mrc/mrconnector/mr_connector.go
77.86% <0.00%> (+4.58%)
:arrow_up:

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


[FEA] Quick integration tests

Is your feature request related to a problem? Please describe.
The integration tests take 45+ min
Describe the solution you'd like
Have a way to tag some tests as essential that devs can run as part of their pre-commit. This way the commit won't take 45+ min as it is today.
Describe alternatives you've considered
NA
Additional context
NA

@revans2 I believe this is done as part of your PR

My PR took a different approach and ran the tests in parallel.  This does speed things up a lot, but I expect us to add more tests especially as we add in more data types and more operators.  I don't necessarily want to drop the idea of tagging tests too, and that could help speed up the tests even more.  I am going to drop the Release and put it back into need triage so we can look at it again.

First step is to apply parallelism to all integration tests.  Second is to consider other alternatives like tagging tests for each environment, or dynamically selecting tests based on the changes.


[CHA-RL5a1] Ignore suspended channel during channel windDown operation

As per CHA-RL5a1, As many chat features share channels, the equality of contributors when deciding not to detach is based on their realtime channel, and not the contributor themselves. i.e. if two features share a realtime channel, and that channel is suspended, then that channel should not be detached.
So, following code snippet should be updated from
https://github.com/ably/ably-chat-js/blob/533c4d0314bcce942a30b4d7d6a012c3b081420d/src/core/room-lifecycle-manager.ts#L630-L637

to
    if (contributor.channel === except?.channel && this._lifecycle.status !== RoomStatus.Failed) {
      return;
    }

Thanks! We've got this one covered already on https://ably.atlassian.net/browse/CHA-700


[Bug]: OpenAIAgent should work with Azure OpenAI's 2023-07-01-preview api version

Bug Description
Azure OpenAI now has function calling and llama-index's OpenAIAgent almost works with it out of the box, passing an AzureOpenAI as the llm argument, with one exception.  There is no content attribute in the response, so the call fails trying to create a chat message from the response.
https://learn.microsoft.com/en-us/azure/ai-services/openai/how-to/function-calling
Here is an example response from the above link
{
  "role": "assistant",
  "function_call": {
    "name": "search_hotels",
    "arguments": "{\n  \"location\": \"San Diego\",\n  \"max_price\": 300,\n  \"features\": \"beachfront,free breakfast\"\n}"
  }
}

Version
0.7.11.post1
Steps to Reproduce
In addition to setting appropriate environment variables I did the following
from llama_index.tools import FunctionTool
from llama_index.llms import  AzureOpenAI
from llama_index.agent import  OpenAIAgent

# define sample Tool
def multiply(a: int, b: int) -> int:
    """Multiple two integers and returns the result integer"""
    return a * b

multiply_tool = FunctionTool.from_defaults(fn=multiply)

azure_openai = AzureOpenAI(engine=AZURE_OPENAI_CHATGPT_DEPLOYMENT, model=AZURE_OPENAI_CHATGPT_MODEL, temperature=0.0)

agent = OpenAIAgent.from_tools([multiply_tool], llm=azure_openai, verbose=True)
agent.query("what is 221 * 9")

Relevant Logs/Tracbacks
Here is where the first exception would be thrown (my temp fix on the next line)

backend_env\Lib\site-packages\llama_index\llms\openai_utils.py:220, in from_openai_message_dict(message_dict)
    218 """Convert openai message dict to generic message."""
    219 role = message_dict["role"]
--> 220 content = message_dict["content"]
    221 # content = message_dict.get("content", None)
    223 additional_kwargs = message_dict.copy()

KeyError: 'content'

Thanks for raising this! Will put in a fix!


delete mwork for pair MSA

related to PR https://github.com/stolostron/cluster-backup-operator/pull/443
adding a deletion of the custom manifestwork for the pair MSA , managedserviceaccount-import-pair
This pair MSA is created at half time the lifespan of the initial MSA to increase the likelihood of a valid MSA token being available with any backup
oc get ServiceAccount -A | grep auto-import-account
open-cluster-management-addon-observability        auto-import-account                                1         11d
open-cluster-management-addon-observability        auto-import-account-pair                           1         6d2h

/cherry-pick release-2.8

/cherry-pick release-2.8

/cherry-pick release-2.7


[ReleaseNotes] add musicardio3.0 changes

Add some changes from MUSIC 2.4 to MUSICardio 3.0. Do not hesitate to tell me if i forgot something, or if you want a line to be changed.
:m:

NB: this "release notes" PR can be reviewed/merged and you can add other notes in a new PR if needed


Use version v0.5.3 of CoreDataModules

No other changes in this PR

@as2388

No other changes in this PR

There seemed to be ICR code changes in messages.py - can you confirm if they were intentional?

@lukechurch Thanks for catching that. That was an experiment which slipped in when it shouldn't have. I will remove it

@lukechurch fixed


[8.x] Add Validator::excludeArrays() to exclude array keys that aren't included in the validation rules

Given the following rules:
return request()->validate([
    'users' => 'required|array',
    'users.*.name' => 'required',
]);

The validator will return the users attribute with all its children:
array:1 [▼
  "users" => array:1 [▼
    1 => array:2 [▼
      "name" => "mohamed"
      "job" => "dev"
    ]
  ]
]

You can see that the job key was returned even though it wasn't included in the validator rules.
This PR adds a new Validator::excludeArrays() that when added to a boot method of a service provider, will instruct the validator to exclude any array keys that weren't validated:
array:1 [▼
  "users" => array:1 [▼
    1 => array:1 [▼
      "name" => "mohamed"
    ]
  ]
]

This behavior of the validator has caused me trouble in the past too. The validate method returning data that wasn't validated should be considered a bug if you ask me. What about making this fix the default in Laravel 9, and removing the need for the Validator::excludeArrays() call?

@SjorsO that's the plan for Laravel 9. But since it's a breaking change, people must opt-in.

If anyone has good ideas for the final method name of this method let me know... /cc @derekmd

@SjorsO that's the plan for Laravel 9. But since it's a breaking change, people must opt-in.

If anyone has good ideas for the final method name of this method let me know.

It isn't easy to name something that fixes such a specific problem. Maybe something like:

Validator::excludeDataWithoutValidationRules()
Validator::excludeNestedDataWithoutValidationRules()
Validator::excludeUnvalidatedNestedData()

excludeArrayValuesNotValidated()

I ran into this issue a little over a month ago too, but as I was working on a fix I found the (at the time undocumented) feature that you could add the keys behind the array rule, like array:name,job which validates that no other array keys are included on that array. Given that this rendered my fix unnecessary for the use-case I was trying to fix, I didn't move forward with it.
Of course, validating the absence of invalid keys is not the same as not returning them in the validated() method if they don't have a rule.
One additional feature I made in my fix was allowing array:*, which would ignore the flag and return everything for that array, as I could see a situation where in the validator you have two arrays: (1) should only return the validated keys and nothing more; and (2) where you may want all the keys even if they don't have their own rules.
https://github.com/laravel/framework/compare/8.x...bonroyage:validator-arrays

@bonroyage I forgot you can do that... 👀

I've run into this a few times as well. Nice work, Mohamed!

Validator::preventAccidentalArrayKeys()
Validator::skipUnattendedArrayKeys()

@bonroyage @taylorotwell
array:one,two,... has two problems. First, it throws a validation error when different keys are provided. That's not the desired behaviour in most cases. Second, it requires that you define all the keys you accept. So you have to define them once as parameters to the array rule, and another to define validation rules for each of the keys.

And it gets even trickier with more levels of nesting, like users.*.jobs.*.title.
I'm not sure if the current approach offers a way to restrict keys for both arrays users and jobs.

Until Laravel 9 you can make this the default using:
app()->resolving(\Illuminate\Validation\Factory::class, fn($factory) => $factory->excludeUnvalidatedArrayKeys());


[FEATURE] New UI Design for signup and Login Page

Description
I want To implement a new Design for the login and signup page. My Design sample is ready you can have a look at below picture
Screenshots
Log in

Sign up

I want to implement a design similar to the above for both the signup and login pages.  This sample design is from one of my personal projects. I will implement better UI and UX and use as many website design resources as required
Request
If you find this issue worth implementing please assign me this under gssoc label.
Additional information
No response

@ZapeeoSheikh can you please review this issue?


az image copy - Not working between RGs in the same subscription

This is autogenerated. Please review and update as needed.
Describe the bug
Command Name
az storage container create
Errors:

Authentication failure. This may be caused by either invalid account key, connection string or sas token value provided for your storage account.

To Reproduce:
Steps to reproduce the behavior. Note that argument values have been redacted, as they may contain sensitive information.

Put any pre-requisite steps here...
az storage container create --name {} --account-name {} --output {}

Expected Behavior
Environment Summary
Linux-5.15.0-50-generic-x86_64-with-glibc2.31, Ubuntu 20.04.5 LTS
Python 3.10.5
Installer: DEB

azure-cli 2.41.0

Extensions:
image-copy-extension 0.2.9

Dependencies:
msal 1.20.0b1
azure-mgmt-resource 21.1.0b1

Additional Context

Have you run az login before az storage container create?
If you don't specify --account-key or --sas-token or --connection-string, you need to login first for authentication

Hi @evelyn-ys, Actually I was trying to copy a VM image to another resource group using the command az image copy. I'm logged into the azure account by az login
The run command was. az image copy --source-resource-group ***** --source-object-name ****-Image-20221017 --target-location eastus --target-resource-group *****
Te following error was reached:
Getting OS disk ID of the source VM/image
Creating source snapshot
Getting sas url for the source snapshot with timeout: 3600 seconds
Target location count: 1
eastus - Creating target storage account (can be slow sometimes)
eastus - Creating container in the target storage account
command failed: ['/usr/bin/../../opt/az/bin/python3', '-m', 'azure.cli', 'storage', 'container', 'create', '--name', 'snapshots', '--account-name', 'eastuses6wjpfw26crv7qib6', '--output', 'json']
output: ERROR:
Authentication failure. This may be caused by either invalid account key, connection string or sas token value provided for your storage account.

The command failed with an unexpected error. Here is the traceback:
Command '['/usr/bin/../../opt/az/bin/python3', '-m', 'azure.cli', 'storage', 'container', 'create', '--name', 'snapshots', '--account-name', 'eastuses6wjpfw26crv7qib6', '--output', 'json']' returned non-zero exit status 1.
Traceback (most recent call last):
  File "/opt/az/lib/python3.10/site-packages/knack/cli.py", line 233, in invoke
    cmd_result = self.invocation.execute(args)
  File "/opt/az/lib/python3.10/site-packages/azure/cli/core/commands/__init__.py", line 663, in execute
    raise ex
  File "/opt/az/lib/python3.10/site-packages/azure/cli/core/commands/__init__.py", line 726, in _run_jobs_serially
    results.append(self._run_job(expanded_arg, cmd_copy))
  File "/opt/az/lib/python3.10/site-packages/azure/cli/core/commands/__init__.py", line 697, in _run_job
    result = cmd_copy(params)
  File "/opt/az/lib/python3.10/site-packages/azure/cli/core/commands/__init__.py", line 333, in __call__
    return self.handler(*args, **kwargs)
  File "/opt/az/lib/python3.10/site-packages/azure/cli/core/commands/command_operation.py", line 121, in handler
    return op(**command_args)
  File "/home/dtovar/.azure/cliextensions/image-copy-extension/azext_imagecopy/custom.py", line 158, in imagecopy
    create_target_image(cmd, location, transient_resource_group_name, source_type,
  File "/home/dtovar/.azure/cliextensions/image-copy-extension/azext_imagecopy/create_target.py", line 81, in create_target_image
    run_cli_command(cli_cmd)
  File "/home/dtovar/.azure/cliextensions/image-copy-extension/azext_imagecopy/cli_utils.py", line 45, in run_cli_command
    raise ex
  File "/home/dtovar/.azure/cliextensions/image-copy-extension/azext_imagecopy/cli_utils.py", line 21, in run_cli_command
    cmd_output = check_output(cmd, stderr=STDOUT, universal_newlines=True)
  File "/opt/az/lib/python3.10/subprocess.py", line 420, in check_output
    return run(*popenargs, stdout=PIPE, timeout=timeout, check=True,
  File "/opt/az/lib/python3.10/subprocess.py", line 524, in run
    raise CalledProcessError(retcode, process.args,
subprocess.CalledProcessError: Command '['/usr/bin/../../opt/az/bin/python3', '-m', 'azure.cli', 'storage', 'container', 'create', '--name', 'snapshots', '--account-name', 'eastuses6wjpfw26crv7qib6', '--output', 'json']' returned non-zero exit status 1.

Seems an auth error but az cli is authenticated with an administratos user (used for IaC)
Regards!

UPDATE
I tried also using ubuntu 18.04 and got the same error as 22.04.
Then I tried using CentOS 7 and finally I was able to copy the image using the next environment:
Environment Summary
Linux-5.15.0-50-generic-x86_64-with-centos-7.5.1804-Core, CentOS Linux 7 (Core)
Python 3.6.8
Installer: RPM

azure-cli 2.38.0 *

Extensions:
image-copy-extension 0.2.9

Dependencies:
msal 1.18.0b1
azure-mgmt-resource 21.1.0b1

From my perspective, seems an error with python 3.10, could you please take a look, @evelyn-ys?
Regards!

Hi, any update on this?

@zhoxing-ms Can you take a look? These are all commands from VM module

Traceback (most recent call last):
File "/opt/az/lib/python3.10/site-packages/azure/core/_enum_meta.py", line 59, in getattr
return cls.member_map[name.upper()]
KeyError: 'IMPORT_ENUM'

This should be the same issue as https://github.com/Azure/azure-cli/issues/24263


Observers are not removed after teardown

0.7.1:
http://jsfiddle.net/z7ytqLhc/
0.6.1:
http://jsfiddle.net/5xmhL0g3/
As you can see in the console of the examples above, the 0.7.1 version keeps calling the observers from the old instances.

My last opened issue(#1861) can also be related to this.

confirmed with 0.7.1

and child r-14 in my case observe still stuck around even though the view is torn down.

Fixed in 0.7.2. Thanks for the test case @blackgate - you'll find that the observer is called immediately before teardown (true -> false), but after that it's neutered.


Object literal shorthand rule

Closes #304

Thanks for your interest in palantir/tslint, @danvk! Before we can accept your pull request, you need to sign our contributor license agreement - just visit https://cla.palantir.com/ and follow the instructions. Once you sign, I'll automatically update this pull request.

@danvk can we make the name a little more verbose, object-literal-shorthand? I don't really like eslint's name for this rule.
also thanks for signing the CLA. as you can see that's a new thing :)

some of my comments got collapsed in an outdated diff but they're still relevant -- please click through

Thanks for the quick review @adidahiya. I've addressed all your comments. Good catch on named function expressions. eslint allows them and now I do, too.

At the risk of side-tracking this PR, I am curious what you guys think of https://github.com/eslint/typescript-eslint-parser as an alternative way of running lint rules like this one that aren't TS-specific. It would be great to use the AirBnb style guide with TS.

ping @adidahiya @JKillian anything else you need from me on this?

Looks good, thanks @danvk! I haven't given the ESLint parser a try yet, have you had any luck with it? I believe it strips all TS-specific elements from the AST?

@JKillian I believe that's correct. You should really take typescript-eslint-parser for a spin. It's quite easy to set up and gives a mixture of great and less-great results.

For style rules (e.g. object-curly-spacing, no-extra-semi, space-infix-ops, indent, padded-blocks, ...) it's perfect.
Other rules like no-undef, no-unused-vars and dot-notation, it's hopeless. You can import symbols exclusively to use them as types, which it will never be able to understand.
Rules that are purely about ES6 semantics also work well, e.g. guard-for-in, no-restricted-syntax and prefer-arrow-callback.

I think some sort of combination of eslint+typescript-eslint-parser and tslint could be very powerful. There's no sense in replicating all the eslint rules that are independent of TS (like space-infix-ops). But there are many rules that eslint can't get right in a TypeScript context.

Wrapping ESLint in some way and using its style rules is an interesting idea. Do the style rules actually work that well though? For example, does the AST that ESLint builds include interface definitions or generic constructs, both elements that wouldn't be normally found in an ES AST?

The rules that are purely about style (whitespace / indentation / variable naming) mostly work great.
But unless their plan is different than what I think, there will always be some rules they can't implement because they require type information, e.g. no-undef. tslint will always be needed for those sorts of rules.
Another nice thing about the typescript-eslint-parser approach is that you get eslint --fix for free.

@danvk there's discussion going on here about this: https://github.com/eslint/typescript-eslint-parser/issues/61

Thanks so much for linking to that issue, @adidahiya, and thanks for trying out the parser @danvk!
Currently, that thread is definitely the best place to learn about how we feel the combination of the typescript-eslint-parser and eslint-typescript-plugin will be able to facilitate full linting of TypeScript code with ESLint.
I appreciate that it is not necessarily very clear from the existing documentation what the scope of the project is. We will definitely rectify that once we get closer to a full "beta".
Please note that there is no limit to the ambition here, so for example @danvk:

But unless their plan is different than what I think, there will always be some rules they can't implement

...it most definitely is! You can see that we have already debunked the false preconceptions about what is possible, and not possible, in that issue.
@JKillian:

For example, does the AST that ESLint builds include interface definitions or generic constructs

Yes, it already does! You can see how the AST from https://github.com/eslint/typescript-eslint-parser/blob/master/tests/fixtures/typescript/basics/class-with-implements-generic-multiple.src.ts includes typeParameters here: https://github.com/eslint/typescript-eslint-parser/blob/master/tests/fixtures/typescript/basics/class-with-implements-generic-multiple.result.js#L107
Things are progressing very well, it is simply early on in the process.
At this stage, we are able to parse all stable TypeScript (i.e. currently 1.8.10) syntax (including TSX). In addition to that, we do already have some type information recorded in the ESTree-compatible AST (matching Flow's terminology, and as linked above) and I am most of the way through adding full decorator support.
We are also currently spec'ing out how we can elegantly support things such as the full TypeScript TypeChecker being used within rules (you can already see a POC of what those kind of rules could do in that thread).
Thanks again for taking the interest in the parser, and for your work on TSLint. Everybody in the TypeScript community is going to benefit from having both TSLint and ESLint available to them!

Thanks for the detailed response @JamesHenry! I responded in part over in https://github.com/eslint/typescript-eslint-parser/issues/61 - it seemed easiest to keep discussion in one thread over there


feat: build a macOS Bundle

This PR prepares the code to create versioned releases of Ref Studio 🚀

Instructions for the versioning rules and release were added to the root README file
Update version to match the first version agreed on #311  (23.1.0)
Add yarn build scripts to generate release files
Fix the editor toolbar icons (replace images with react-icons).
Add extra action in the editor toolbar to "Clear Formatting"

Note: After merging this into main we should tag main with 23.1.0 and create a release with the binaries.
Closes #311

Is it easy to change the icon to be something specific to Ref Studio and not the Tauri icon? Also what's the best way to test out the .dmg file before we cut a release?

I think we should just use the RS logo you put in the empty editor section on startup. I spoke with the design team today and they said explicitly they won't be doing logo design in this initial engagement.

I think we should just use the RS logo you put in the empty editor section on startup. I spoke with the design team today and they said explicitly they won't be doing logo design in this initial engagement.

Ok, I will make one with the RS letters 🤞 😄

@hammer this is the new logo I've created and how it looks.


🌐 Add Chinese translation for docs/zh/docs/advanced/testing-events.md

As title

📝 Docs preview for commit b64c4c3dd963734ed11717f24c572551252a0a12 at: https://649a08f605f6af08cdd6cf44--fastapi.netlify.app

Thanks! @jaystone776 😎👏
And thanks for the help @zy7y 🚀


Support defining proxies only for specific hosts

Please have a look at the individual commit messages for the details.

@sschuberth  Can you update ORT docs with an example how to use this new feature?

@sschuberth Can you update ORT docs with an example how to use this new feature?

We currently have no docs about proxy support at all. And while I agree it would be useful to have, I'd prefer to do that in a separate PR, probably one that moves the whole configuration section from README.md to a separate file, and then add a proxy section to it.


Error occurred during deleting layout

Hi,
I just found an issue.
When we delete a layout from a page, it will lead to an error.

Hi, do you have some errors in the browser ?


Update PackageDescription.md

Update the internal documentation.
Motivation:
Follow-up to: apple/swift-package-manager#3079
Modifications:

Rename the second target method to executableTarget.
Update the CLanguageStandard and CXXLanguageStandard enums.

Result:

Internal documentation is improved.
External documentation is still outdated.

thanks @benrimmington will merge when we are back next week

thanks @benrimmington will merge when we are back next week

@swift-ci please smoke test

@swift-ci please smoke test

Since @tomerd has approved, I will merge as soon as tests pass.

Since @tomerd has approved, I will merge as soon as tests pass.


Related Content Link

I think it would be helpful were the "Related Content" section to have a link to:
xml (Transact-SQL)
https://docs.microsoft.com/en-us/sql/t-sql/xml/xml-transact-sql

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: ed5014a4-6cfd-670a-b5ee-c75669fa5b5b
Version Independent ID: eae2dd35-ff1b-6edf-aa0e-df7942d147e4
Content: XML Data (SQL Server) - SQL Server
Content Source: docs/relational-databases/xml/xml-data-sql-server.md
Product: sql
Technology: xml
GitHub Login: @MightyPen
Microsoft Alias: genemi

@JediSQL  Thank you for the feedback.
The documentation has been updated.


jQuery drop-in?

Hi and thanks!
Is it supposed to be a jQuery drop-in replacement? if so, what would I do to enable the "$" thing for a project that already built with jQuery?
Thanks

Cash doesn't implement everything that jQuery does, but if you only use things implemented also in cash it should be a drop-in replacement.

if so, what would I do to enable the "$" thing for a project that already built with jQuery?

Just load cash instead of jQuery.


Add new rules engine for section enabled rules

What is the context of this PR?
This adds new rules engine for the section enabled element of schemas, based on element type it will now evaluate when rules using old or new when rule evaluation. All schemas that use that used old type of when rules in section enabled has been updated and added to schemas with new prefix.
How to review
Run new test schemas manually or run unit tests.
Checklist

[ ] New static content marked up for translation
[ ] Newly defined schema content included in eq-translations repo

 Runner to use new rules engine for section enabled rules (3 SP)

All the tests are referencing new schemas now except one that I had to keep in order to get 100% coverage (old evaluator engine code was missing tests).


pkcs5: optimize pbkdf2_hmac with larger passphrases and iteration counts

This significantly optimizes the mbedtls_pkcs5_pbkdf2_hmac function by
caching and reusing the initial HMAC digest state across all iterations.
This is especially evident when using passphrases longer than the digest
block size, which must first be digested for shortening before they can be
used a a key for the HMAC algorithm.
On an Intel Core i5-4690 CPU, using extreme parameters of SHA256 as the
digest, a passphrase length of 4096 bytes, a salt length of 128 bytes, an
iteration count of 1000000, and a derived key length of 128 bytes, the
runtime was reduced from 76 seconds to 3.75 seconds (a 95% reduction).  This
is comparable to OpenSSL's runtime of 2.75 seconds using the same
parameters.
The mbedtls_pkcs5_self_test() function, which uses SHA1 with much shorter
passphrases, salts, iterations, and keys, still had its runtime reduced from
10.1ms to 7.3ms (a 28% reduction).

This has been done in https://github.com/ARMmbed/mbed-crypto/pull/277.


Bug - Beta v24 - QRouteTab no longer binding router specific attributes to HTML tag

Using QRouteTab is no longer binding router-specific classes and attributes to the actual HTML element.
For example, on the regular version of Quasar, my QRouteTabs will each have classes "q-router-link", "q-router-link--active" (etc), and a href="" attribute for SEO purposes, however after switching to this branch, all these properties no longer exist on the QRouteTab attributes.
Clicking the actual tab still correctly redirects to the defined to prop, but as mentioned, the actual HTML source doesn't display the href=' attribute making it invalid for SEO.
(This problem doesn't exist for QTab acting as a route - they still correctly bind the attributes)

Can you check if you really had q-router-link class on the tabs (and it is not added by you) - I'm not talking about q-router-link--active?

Yep, I compared the running code on our production server and I certainly didn't add the classes manually. Code was identical, except one was running on beta v24.

q-router-link?

That was what I was asking, because there was no q-router-link before and I needed to confirm your:
my QRouteTabs will each have classes "q-router-link", "q-router-link--active" (etc)

Oh my bad then, sorry about that. I should clarify that the q-router-link--active classes etc aren't being bound anymore, but instead using q-tab--active. Plus href not being bound either.
The q-router-link--active one I suppose can be bound using the exact-active-class prop (unless it is actually a bug).

All the rest is done :) but that class q-router-link puzzled me because it was nowhere in original Q1

Haha sorry again. And thanks a lot for fixing the three bugs. I will continue testing the version tomorrow and report back if I find anything else. Rest seems solid so far.

https://github.com/pdanpdan/quasar/releases/tag/quasar-pdan-v1.16.0-beta.25


POD annotations are dropped with the reconcile of CHK STS

Keeping the POD annotations is essential to run the workload in EKS, where the fargate profile is the default one.
Dropping essential annotations, such as "eks.amazonaws.com/compute-type" = "ec2" will cause the POD to be unschedulable due to the fact that:

you can't mount a volume to a fargate node.
you can't possibly satisfy the nodeSelector & nodeAffinity rules with fargate profile in case you ask for dedicated EC2 node for billing purposes.

Example manifest:
apiVersion: "clickhouse-keeper.altinity.com/v1"
kind: "ClickHouseKeeperInstallation"
metadata:
  name: clickhouse-keeper
spec:
  configuration:
    clusters:
      - name: chk
        layout:
          replicasCount: 3
  templates:
    podTemplates:
      - name: clickhouse-keeper
        metadata:
          annotations:
            eks.amazonaws.com/compute-type: "ec2"

Interestingly, first POD of the 3 replicas starts with correct annotation, but then, the second doesn't as the annotations are dropped from the underlying statefulset.
Note that I also see this in the log of the operator, which is possibly the result of this behavior:
E0802 07:02:22.310975       1 reconciler.go:299] err: Operation cannot be fulfilled on clickhousekeeperinstallations.clickhouse-keeper.altinity.com "chk": the object has been modified; please apply your changes to the latest version and try again

Please see this pull request to the branch 0.24.0 🙏🏿 .

I encountered a similar issue when adding additional annotations for Datadog agent metrics scraping.
Here are the details:

Defining annotations in the podTemplates for a CHK manifest works successfully when creating the CHK for the first time.
However, modifying the annotations block later on causes the reconciliation process to drop all annotations.

We are also randomly seeing reconciler errors on some deploys. since we are using annotations in our env, i would suspect it's the same issue as above mentioned for datadaog
 1 reconciler.go:299] err: Operation cannot be fulfilled on clickhousekeeperinstallations.clickhouse-keeper.altinity.com "keeper": the object has been modified; please apply your changes to the latest version and try again

@g-marius do you use something like Flux or ArgoCD?

I believe the fix is released already as part of 0.24.0


Error: pull model manifest

What is the issue?
When I try ollama pull mistral, I get the Error: pull model manifest.
pulling manifest
Error: pull model manifest: Get "https://registry.ollama.ai/v2/library/mistral/manifests/latest": tls: failed to verify certificate: x509: certificate has expired or is not yet valid: current time 2024-03-25T13:08:04+05:30 is after 2024-03-25T07:17:47Z
What did you expect to see?
I expect that this command would be able to download the model locally.
Steps to reproduce
Download Ollama, and run  ollama pull mistral
Are there any recent changes that introduced the issue?
No response
OS
Linux
Architecture
x86
Platform
No response
Ollama version
No response
GPU
Nvidia
GPU info
TU104 [GeForce RTX 2060]
CPU
AMD
Other software
No response

Can not pull any model now..


Support additional parameter value during mapping

Is your feature request related to a problem? Please describe.
I have DTO to Domain object mappings that also have additional context required for the domain to function correctly
Describe the solution you'd like
I'd like to be able to hand off additional parameters to the mapping function for use within the mapping
[Mapper]
public partial class CarMapper
{
    public partial Car ToCar(CarDto car, ICarContext carContext);
}

public record CarDto(int Wheels);

public class Car
{
    public Car(int wheels, ICarContext carContext)
    {
        Wheels = wheels;
        _carContext = carContext;
    }
    public int Wheels { get; }
    private readonly ICarContext _carContext;
}

// Relevant new / adjusted API surface
    public partial class CarMapper
    {
        [global::System.CodeDom.Compiler.GeneratedCode("Riok.Mapperly", "<IP_ADDRESS>")]
        public partial global::Car? ToCar(global::CarDto? carDto, ICarContext carContext)
        {
            if (carDto == null)
            {
                return default;
             }
             var target = new global::Car(CarDto.Wheels, carContext);
             return target;
        }

Describe alternatives you've considered
I don't know what alternatives there might be other than a manual mapping, I'm new to this library.
Additional context
N/A

Duplicate of https://github.com/riok/mapperly/issues/103


[Network] Order of message sent is incorrect.

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Expected Result:
The message should be sent in order as the user sent them.
Actual Result:
The order of sent messages is changed randomly.
Action Performed:
First way

Open Expensify.cash app.
Open any report.
Go to airplane mode.
Send few messages.
Turn off Airplane mode.
Observe the order of messages.

Second way

Open Expensify.cash app.
Open any report.
Send a video that is big in size.
Send another video that is very small in size compared to the first one.
Observe the order of messages after videos are uploaded. (This proves that we don't track the order of messages).

Workaround:
Send messages one by one.
Platform:
Where is this issue occurring?
Web :white_check_mark:
iOS :white_check_mark:
Android :white_check_mark:
Desktop App :white_check_mark:
Mobile Web :white_check_mark:
Version Number: Latest
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
Expensify/Expensify Issue URL:
https://user-images.githubusercontent.com/24370807/117268155-9a1d8f80-ae74-11eb-9e8c-dd2a0126f8ae.mp4
View all open jobs on Upwork

Not sure if there's anything we need to do about this right now. Some videos can be slower to upload than other things like comments. Not sure we should block the comment because the video is still uploading. I think this would need to be addressed in the backend if we wanted to do something about it.

Mostly I am concerned about the messages that are sent after user comes back online. We can't impose sequencial queue processing as it would be slow. But I think we should store the message based on timestamp or something to retain the order.

@mallenexpensify. Sorry I didn't pulled it on Slack. Let me just raise the flag there.

I wonder if this effect might be mitigated by the changes here?

How do you mean @roryabraham ?

Well, if I upload a large attachment, then 1 second later send a text message, then the text message may added to the report in the back-end before the attachment. So from my perspective I sent the attachment first, but then the text message will appear first in the report history.
But under the new paradigm, we'll often be sending attachments and text "at the same time", at least from the user's perspective. So then it won't feel incorrect that the text appears first.
Anyways, it doesn't actually solve this issue, just perhaps mitigates the effect it may have on the user.
Ultimately I think the long-term solution I would probably advocate for is to 🔪 sequence numbers entirely and display report actions in the order in which they were created, referencing a timestamp attached to the report action when it's created on the client (not the server).

under the new image-sending paradigm, we'll often be sending attachments and text "at the same time", at least from the user's perspective. So then it won't feel incorrect that the text appears first.

Got it thanks! Yeah, I'm still unsure if this is a problem or not. If your attachment shows up after a message because it takes a bit longer that could be a feature (we just prioritize message speed over preserving the order of messages).

Ultimately I think the long-term solution I would probably advocate for is to 🔪 sequence numbers entirely and display report actions in the order in which they were created, referencing a timestamp attached to the report action when it's created on the client (not the server).

That could work. A lot of things are dependent on sequenceNumber, but it might be worth an investigation.

@mallenexpensify was able to reproduce this today on Desktop  v1.0.85-9
Message showing out of order. Desktop
Action performed:

Sent (brb gonna make coffee)
Closed comp lid/computer put to sleep
Opened comp
Sent new Yes, Correct message
old brb message is stuck at bottom

Logs:
Failed to load resource: net::ERR_NETWORK_IO_SUSPENDED
app-3421065….bundle.js:210
[API] Handled error when making request TypeError: Failed to fetch
app-3421065….bundle.js:210
Uncaught (in promise) Error: session.offlineMessageRetry
    at app-3421065….bundle.js:210
    at app-3421065….bundle.js:210
app-3421065….bundle.js:210
[Pusher] Reconnecting to Pusher
9
app-3421065….bundle.js:210
GetIOUReport returned a reportID of 0, not fetching IOU report data
app-3421065….bundle.js:210
[Report] Fetching reportActions for reportIDs:  Object
app-3421065….bundle.js:210
[Pusher] Reconnecting to Pusher
9
app-3421065….bundle.js:210
GetIOUReport returned a reportID of 0, not fetching IOU report data
app-3421065….bundle.js:210
[Report] Fetching reportActions for reportIDs:  Object
2
app-3421065….bundle.js:210
[LOCAL_NOTIFICATION] No notification because comment is from the currently logged in user
DevTools failed to load SourceMap: Could not parse content for app://-/app-3421065….bundle.js.map: Unexpected end of JSON input
DevTools failed to load SourceMap: Could not parse content for app://-/pdf.worker.entry-b22bab8….bundle.worker.js.map: Unexpected end of JSON input

This seems to be happening semi-regularly.  I'll post more here when it does.
The issue I have right now is...
Was on unstable internet this morn that was slow, spotty and occasionally cut out.  Sent a message to Monte, What's your morn looking like?, and it got 'stuck' at the bottom. Even after clearing cookies and storage and signing back in, the message is still stuck at the bottom

I've experienced this as well, where it most easily happen when you go offline and send 3 or 4 messages in a single chat then resume connectivity.
When you resume connectivity all 4 messages are sent simultaneously and they would rarely be registered in the correct chronological order on the backend
I guess something similar can happen on a dodgy connection

Even after clearing cookies and storage and signing back in, ...

This particular problems seems related to: https://github.com/Expensify/App/issues/2642
Even though you log out some actions continue to run and add data to storage (like the optimistic message you see on the bottom)
When you sign in again you still see items you expected to be cleared by a log out
A work around to avoid this would be to

Log out
Close the app/browser window (this terminates any leftover rouge actions / network queue)
Open App again
Clear cookies and storage (removes any leftover data added by rogue actions)
Log in

Some thoughts on ordering the messages and the UX aspect of the matter

Let's say we're offline and write some messages at 9:00am they get persisted
Our recipient manages to write us before we're back online at 9:05am

What's the order we expect to see when connection is restored?
Our messages getting inserted before theirs
It will be confusing to insert our messages into the conversation
Our messages we're indeed earlier, bu it would be confusing for our recipient - they'll see there are new messages for the conversation but when they open it they'll see they are the last person to comment and there are no new messages after their comment, they might not notice some messages are inserted before theirs.
Chronological order is preserved but it might not make sense
If the recipient saw our messages from 9:00 they might not have sent their message at. 9:05, or send a different message
If we go back to this conversation later a day or a week after, lacking context that we were on a dodgy connection the conversation might not be making much sense, unless there' a comment like "Hey there was a mix up because I was briefly offline"
Their messages come before ours
This seems would cause less problems - our recipient will be informed of new messages and see them as such
The roles are reversed - we'll be seeing a conversation that has new messages, but when we open it we'll see we are the last person to comment. But because we're the one experiencing networking issues we are more likely to realizes the other party replied while we were offline and their comments are above ours
There's again the problem of one of the parties changing their mind about the message - this time it's us - seeing our correspondent's message we might wish we didn't send ours
I think these UX reasons are the reasons why Slack does not resend messages automatically but let's you decide - "message wasn't sent - [try again]"
This way you can alter the message or not, but it's clear it will take place now and will be the new last message of the conversation
In case new messages arrived you would see them as such - they will not be inserted before your own
I think Slack deals with the UX problem well by offering the choice to retry the message or discard it
We can do something similar and it would also solve the ordering problem
Each failed message would have contextual actions to retry or discard - the user will be manually selecting which ones to retry, you'll have to be really fast to make 2 requests start at the same time, so messages will pretty much be ordered by user interaction
In comparison to current behavior we send up to 5 messages at the same time and their order depends on when they arrive to the backend (networking)

Another case - persisted messages are sent later - it could be a hour, a day a week etc.

We're mobile
We're offline
We write some messages
We quit the app - current logic would persist them to be send later
We open the desktop app
Our messages are not there as they weren't sent
We either write them again or continue the conversation in another way
On the day after we open the mobile app
Current logic would run and post the persisted messages

Problems

in that particular case the user probably doesn't want to send the messages at all
where would these message appear in the conversation? In the middle at the end?
Should the recipient be notified about them?

The same solution where we don't send messages automatically but offer the user a choice, would address the problems here as well

Can't reopen. This is necessary...

@parasharrajat, this Monthly task hasn't been acted upon in 6 weeks; closing.
If you disagree, feel encouraged to reopen it -- but pick your least important issue to close instead.

@parasharrajat, this Monthly task hasn't been acted upon in 6 weeks; closing.
If you disagree, feel encouraged to reopen it -- but pick your least important issue to close instead.

@parasharrajat, this Monthly task hasn't been acted upon in 6 weeks; closing.
If you disagree, feel encouraged to reopen it -- but pick your least important issue to close instead.

I think this one is still relevant - it happens when you've write a couple of message while offline (or spotty connection) - all of them are sent at once and the order is random

Thanks for commenting @kidroca , I'll reopen

It is important to have so I request to reopen this. May be put some label so that Melvin doesn't close this.

The network request queue now runs in sequence so this should be solved. If there is a specific case where messages are not showing up in order it's probably related to Pusher and outside of our control.


Sporadic Error when using CLI

@David-Fowler I am getting this error when I try and use the CLI in R. I have gotten the same error sporadically over the last few weeks. Yesterday I got this error a few times. I restarted R Studio a few times and then it went away and all was well again. I was able to make API calls and get data back.
Any ideas please?

It just failed a whole bunch of times in a row. Left it 20 minutes and tried it again and now it works.... Sorry, I know that this is the worst kind of bug report.

This will be fixed in the update pushed later tonight. It is mixed in with other fixes regarding the new API. This particular issue I believe revolves around credentials and key timeouts but I am not 100% certain. I have added a unit test to attempt to catch this type of issue from here on out.

@philipbaileynar if this is still occurring for you can you try the following to narrow the issue down for me. I have already checked the credential timeout code as well as a few other targeted sections with no luck in regenerating the issue. Closing R studio and reopening (unless you have cached your workspace) should have set the package state similar to second command below. If you could test the cli() function after each of the following lines of code it may help.
NAMCr::diagnose_connection()
# Make sure the output reads:
# Internet, API and oAuth2 provider are all available and online.

NAMCr::reinitialize()
# test the cli()
cli()

NAMCr::reinitialize(clear_credentials = TRUE)
# test the cli()
cli()

Well that is no good... looks like I never fully hooked up the diagnose_connection function. The latest version, 0.5.5, has the fix for the diagnose_connection function. This was just pushed.

I upgraded to 0.5.5.
CLI worked. I will report back here if the "vector size" error happens again.


Add support for reCAPTCHA (and others)

This PR:

Adds support for reCAPTCHA (closes #20)
Add redirectError option

There is a redirect option which, when present, redirects successful form submissions to the specified URL instead of returning a JSON response. The new redirectError option does the same thing but for failed submissions.

Ensure path config option doesn't have trailing slash (#68)
Improves error handling

Error messages are now more consistent and Staticman now supports Sentry for error reporting. There is a new sentryDSN config option to enable that.

Removes the allowedOrigins config option

This was an experiment as part of the development of the email notification feature to offer another level of protection against spoofed form submissions, but quite frankly I'm not sure where I was going with this. Perhaps we allow API instances to only accept email notifications if reCAPTCHA is used.

@zburgermeiszter could you have a look and help me test this?
Thanks!

@eduardoboucas
I'm glad this feature reached the point where you are ready to merge it to master.
Please have a look at my PR, which is now outdated, but I created a separate reCaptcha class, in order to improve the code readability.

Thanks @zburgermeiszter. I'll definitely look into it, and I'll address that in a later release. Before that, though, I want to make sure the feature is ready to roll out to production and all issues have been covered.

Merged! 🎉
@zburgermeiszter many thanks for your effort!


add model workflows

adding merlinite & granite model workflow

One thing you should also do is include is more targeted on policy to specify what we want to test for. Here is an of what this looks like for our chatbot workflow
It should look something like this:
on:
  schedule: # schedule the job to run at 12 AM daily
   - cron: '0 0 * * *'

  pull_request:
    branches:
      - main
    paths:
      - ./models/Makefile
      - .github/workflows/model_image_build_push.yaml
  push:
    branches:
      - main
    paths:
      - ./models/Makefile
      - .github/workflows/model_image_build_push.yaml

It was determined we want to push ahead and get these images in Quay ASAP. I will follow on with a refactor PR adressing the concerns laid out above.

@Gregory-Pereira is going to refactor this and make it 1000% better, I'm merging this as/is to populate these images


Add first module

Module for launching ec2 instances with IAM role(s) and security group(s)

post merge +1


Oneironaut 0.2.1+ breaks Upgraded Spellbook from HexLink

Mods
hexcasting-fabric-1.19.2-0.10.3
hexgloop-1.19.2-0.2.0-fabric
hexlink-2.2.11
oneironaut-fabric-1.19.2-0.2.0
+all required dependencies

Upgraded Spellbook can store iotas like Spellbook and cast them on right-click like Artifact. But when oneironaut version is more 0.2.0, it open Hex on every cast
https://github.com/beholderface/oneironaut/assets/5018813/b87dcb7e-84ae-4bf5-8570-1571316219a9
https://github.com/beholderface/oneironaut/assets/5018813/6f729a22-ed50-4ae0-a526-b314bff1b1c5

Looks like this is an interaction with my code that allows you to designate non-staff things to act as staves by adding the hexcasting:staves tag to them, which the upgraded spellbook has for some reason.

I've tested it after speaking to jempasam (hexlink creator) on discord, and determined that the upgraded spellbook works fine without the tag. Here is a datapack that removes it.
spellbooktagremoval.zip
Note that this datapack removes the staves tag from the upgraded spellbook by replacing the entire list of items that have the tag. I've added all the base hex staves, and those added by Oneironaut and Hexlink, but if you have any other addons that add things to the tag (HexGloop, Hextended Staves, Hexical, maybe others idk) you'll have to add them by editing the staves.json file in this datapack.


fix(binaries): use plain http

Opened as an experiment, but closed because it would be insecure.

Good decision! :+1:


GetWorkflowStateAsync "error from internal actor: no such instance exists"

Hello, when we use the GetWorkflowStateAsync method it is expected that if the workflow does not exist it should indicate that it does not exist. Then it generates an exception of type: Grpc.Core.RpcException with status: Status(StatusCode="Unknown", Detail="error from internal actor: no such instance exists"). This can be unexpected because if the workflow breaks. This is terminated and if you use the method to check if it is active then you can have an exception. That then forces you to implement patches to be able to validate if the error that you are having is because the workflow does not exist.

Transferred issue to the runtime repo.
cc @cgillum

Is the expectation that no gRPC error is observed or is the expectation that the gRPC error makes it easy to distinguish between "not found" and other errors, for example, by using appropriate status codes (NotFound instead of Unknown)?

well reviewing the existing implementation in the sdk it seems to me that they expect a null.

My question was whether you the application developer have an expectation on the ideal behavior. But I assume based on the screenshots you shared that you're okay with the behavior that the SDKs were designed for.
Looking at the durabletask-go code, it seems the gRPC API implementation is consistent with the SDKs, as shown here: https://github.com/microsoft/durabletask-go/blob/60e42f2bb25034de2b20201a3139f5fcc159200e/backend/executor.go#L268-L278. In other words, it expects a nil response from the backend implementation (defined by Dapr in this case) to indicate that an instance doesn't exist, in which case it will return a "doesn't exist" response to the gRPC client, which the SDKs are designed to handle gracefully.
I think the simplest way to fix this is to update the Dapr runtime code here: https://github.com/dapr/dapr/blob/d4f06c2b699bc76dd337dfc1329380abb7c1eb7a/pkg/runtime/wfengine/backends/actors/backend.go#L224-L230. Specifically, this internal actor code should check for an error result indicating that the workflow wasn't found and then return a nil response rather than an error response.
Alternatively, we could update durabletask-go to export and check for a well-known ErrInstanceNotFound error, and then have the Dapr runtime return this value instead of nil if we determine that a queried workflow doesn't exist.

+1

. Specifically, this internal actor code should check for an error result indicating that the workflow wasn't found and then return a nil response rather than an error response.
Alternatively, we could update durabletask-go to export and check for a well-known ErrInstanceNotFound error, and then have the Dapr runtime return this value instead of nil if we determine that a queried workflow doesn't exist.

@cgillum Of the two, I'd vote for the first option, returning nil instead of an error.  IMO, "non existence" isn't (necessarily) an error.  Or rather, whether it's an error or not should be determined by the calling application and not the object store.  Returning an error value, even a very specific "not found" value, will tend to get treated as a generic error on the other side of gRPC boundary.  In the case of .NET, that will be an exception, which are tedious to handle as well as impact performance.  (I wish the gRPC libraries had the option of treating invocations more more like raw HTTP invocations where the result could be observed before any action is taken.)

any update guys??

hi guys, @cgillum any progress?

Hey guys, don't we have any news about this? @cgillum, @philliphoff, @yaron2

@cgillum is this required for Stable in 1.15?


Ampersand in a string is causing experimental features to not be installed properly

🧐 Have you fully read the README and viewed similar issues?

[X] I have read the README
[X] I have read the FAQ
[X] I have checked previous issues, open and closed
[X] I have completed all troubleshooting steps and the issue is not present on stock but returns after re-applying SpotX-Bash

Сountry of your account
Canada
What is your Spotify Plan?
Premium
Do you have Spicetify installed?
No
ℹ Computer information
- Spotify version: <IP_ADDRESS>3
- OS: Linux Mint 21.3 Cinnamon
- Shell: bash

📝 Description

Ran the installer, run.sh, which in turn ran the spotx.sh bash file (With or without options or arguments)
???
And you're done!
...

There's an unintended ampersand & in the middle of a string used for the experimental features Array, expEx. Specifically, on line 865.
Change it to the word 'and', and the script will run fine.
🖥 Terminal Input/Output
Searching for Spotify directory...

Found Spotify Directory: /usr/share/spotify

cat: /etc/upstream-release: Is a directory
Debug: Linux Mint 21.3
Debug: x86_64 detected
Debug: APT detected
Debug: flatpak detected
Debug: Cache directory: ~/.cache/spotify

Latest supported version: <IP_ADDRESS>8
Detected Spotify version: <IP_ADDRESS>3

✔ Created backup
✔ Applied free-tier plan patches
Unquoted string "web" may clash with future reserved word at -e line 1.
String found where operator expected at -e line 1, near "web",default:\K!1}"
Unknown regexp modifier "/t" at -e line 1, near "+= "
Can't find string terminator '"' anywhere before EOF at -e line 1.
✔ Enabled experimental features
✔ Enabled new UI
✔ Finished

📸 Screenshots

Sorry about that. Good catch. Fixed.