Update bundle, cloudwatch, sdk-core, ses, sns, ... to 2.26.9

About this PR
📦 Updates

software.amazon.awssdk:bundle
software.amazon.awssdk:cloudwatch
software.amazon.awssdk:sdk-core
software.amazon.awssdk:ses
software.amazon.awssdk:sns
software.amazon.awssdk:sqs
software.amazon.awssdk:ssm

from 2.26.0 to 2.26.9
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "software.amazon.awssdk" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "software.amazon.awssdk" }
}]

labels: library-update, early-semver-patch, semver-spec-patch, commit-count:1

Superseded by #3261.


Snow scuttling integration

Use Snow to scuttle all future realms

update core deps to install snow (also ignore snow on depcheck since it isn't consumed programatically)
export snow source code from core package
update lavapack to inject snow source code at the beginning of runtime.js if option includeSnow is true
update kernelTemplate and kernelCoreTemplate to define snow (falling back to a dummy callback if snow intentionally is left out)
update kernelCoreTemplate to use snow for scuttling
regenerate runtime.js

we kinda decided that we should not squash commits in a monorepo but that means having more meaningful commit messages

suggestion
rebase -i HEAD~15 before rebasing origin/master may save you time

decided on a different approach #462


User test in meteringpointClient in BFF

[ ] Add helper class to generate token to be used in integration tests (Partially completed)
User test for meteringpointClient in BFF

Was originally part of https://app.zenhub.com/workspaces/mighty-ducks---the-outlaws-6193fe815d79fc0011e741b1/issues/energinet-datahub/the-outlaws/150

Hey @mknic would it be possible for us to get Daniels assistance on this story?

@Christian-Meyer sure. Would it be in the iteration starting February 21?
@frontend-specialisten this might tap into the issue you've solved by having a test user as we discussed on planning?

@mknic Yes, next iteration starting feb. 21

@mknic Tasks have been updated. It is the first step we need help with. (He needs to do it) Currently we cannot login as users without a UI.

@djorgensendk please have a look at bullet one in the description. Would you mind creating an issue on our board if you know what it's all about?

Look at issue linked by Dan, and check if this solves this issue.


Logging Operation and Batch

Sometimes it is useful to have a decorator for operations and batches which log every operation before being executed.

done, #203


MM-27559 - Clicking 'Create a playbook' button does nothing on MM-Des…

Summary
Allowing any internal-mattermost link to open within the app.
Ticket Link
Fixes https://mattermost.atlassian.net/browse/MM-27559
Screenshots

/update-branch

@ali-farooq0 FYI that Jelena is OoO for now (just mentioning in case this is time sensitive for v0.6)

Thanks @amyblais. No, not time sensitive, however I'd like to get this in asap. How is covering for Jelena?

Unfortunately I'm not sure who is covering her, maybe Linda or Dean can help.

Hey @stevemudie are you able to help out with the QA request above as a backup to Jelena?


option to set filename of output, defaulting to what quarto uses

quarto_titlepages looks good, but it interferes with my workflow, because it uses the title of the book as the filename for output, rather than the default index.tex, index.pdf, ... used by quarto. Messes up Makefiles, ....
This happened after I had installed the extension, but did not use it in my book project.
I'm going to delete the extension until this is sorted out.

This is not from the extension, per se, but because this is a Quarto book project. For Quarto book project, by default it uses title yaml for the file names for the PDFs but you can change this by adding output-file to the yaml like so. Note, this isn't specific to the quarto_titlepages extension. Any Quarto book project has the same behavior.
book:
  title: "Here is the really long title that I don't want to use as a file name."
  output-file: "index"


BugFix: Camera pictures are not accessible when storage permission has not been given.

Issue:
Camera pictures are not accessible when storage permission has not been given.
Description and steps:
Let the user use the camera when storage permission has not been given. Currently, the app lets users open the camera but doesn't let them proceed as storage permission is denied.
Steps:

Open image picker functionality
Deny storage permission
Click on the camera icon
Capture the image and proceed further
Nothing happens after that

Expected behavior:
Let the user proceed further with the captured image even when storage permission is denied
Issue recording:
https://user-images.githubusercontent.com/20037228/196465424-49d6e17e-c267-4721-af81-baeed1856127.mp4
Fixed issue recording:
https://user-images.githubusercontent.com/20037228/196465874-17e40c15-ed4a-4fbb-803f-44312ee70349.mp4
Please let me know if you have any feedback.
Thank you!

Hey, @kavitamp thanks for opening the PR for this. While the PR fixes the issue, I think it's not touching the root of the issue, which is permission.
AIP image capture code is changing so much from the first time it was written, at the time we need permission for getting the image. For now, I'm not sure. Do we need the storage permission at all?

Do we need storage permission at all?

We need it actually. https://github.com/esafirm/android-image-picker/pull/431 should fix the issue.


Add support for fastboot Continue feature

Signed-off-by: Eramox<EMAIL_ADDRESS>

Replace by PR:140


Add ATT workaround to docs

Summary
Added workaround for App Tracking Transperency on iOS 15.0 to the documentation.

Motivation: ... as discussed in this comment
Test Plan

No testing needed, only docs have changed.
Checklist

[ ] I have tested this on a device and a simulator
[X] I added the documentation in README.md
[ ] I mentioned this change in CHANGELOG.md
[ ] I updated the typed files (TS and Flow)
[ ] I added a sample use of the API in the example project (example/App.js)

I merge it and will add a few changes since I want it to be in the README for the next release (npm one is not synced with GitHub README). Thanks a lot 🙂


Bug: Star menu covers up flyout

Describe the bug:
When the Start Menu is present the flyout gets lost behind. The windows default one is always on top in any moment.
OS Version:

Windows 10 Version: 1909

ModernFlyouts Version:

Version:  v<IP_ADDRESS>

Same as #32.
I'll merge this with #32 & close this issue as a duplicate of it.
You can keep up with that issue.
Please don't get offended, it'll be better for us to keep the issue count as low as possible in order to track and fix them.

Same as #32.
I'll merge this with #32 & close this issue as a duplicate of it.
You can keep up with that issue.
Please don't get offended, it'll be better for us to keep the issue count as low as possible in order to track and fix them.

Haha, of course im not offended, anything to help!


Revert changes for RDK-25879

Moving the changes for reverting RDK-25879 from the 0001-DELIA-45155-4.4_p1v-build-fail-wit-revert-RDK-25879.patch into main branch.

https://jenkins.code.rdkcentral.com/job/github-triggered/2968/ started

https://jenkins.code.rdkcentral.com/job/build-component-rdk-next/19917/ started.

build-rpi-mc-wpe-rdk-next Console Log uploaded to Gist URL: https://gist.github.com/0b84a217ccb4c553d988a1abad6a9726

build-rpi-hybrid-wpe-rdk-next Console Log uploaded to Gist URL: https://gist.github.com/d46758c81d64128fcc409eb8072c1612

build-rpi-hybrid-refapp-rdk-next Console Log uploaded to Gist URL: https://gist.github.com/5f53a5b7ad55fd0f68a005a68f8b9f9a

https://jenkins.code.rdkcentral.com/job/github-triggered/2973/ started

https://jenkins.code.rdkcentral.com/job/build-component-rdk-next/19919/ started.

build-rpi-mc-wpe-rdk-next Console Log uploaded to Gist URL: https://gist.github.com/4171f6c599994f07def2b5d8ae8b07e7

build-rpi-hybrid-wpe-rdk-next Console Log uploaded to Gist URL: https://gist.github.com/b1fe2f0716cd23544aa574edbc46558d

build-rpi-hybrid-refapp-rdk-next Console Log uploaded to Gist URL: https://gist.github.com/d2d7401b9cfce3387667b97827c9344a


SPIKE: Ingesting Marysville CVE data

@carriebrinkley commented on Thu Oct 31 2019
Background:
While exploring how we will go about ingesting the Columbus CVE data when it becomes available, we learned that Marysville is using the same vendor (Kapsch) for their sensor data aggregation.  While at a luncheon, Mandy teed up an introduction for us with an administrator in Marysville, Mike, who said that he would love for their CVE data to be in the OS.  Additionally, there is a research lab at OSU (lead by a man named Levent) that would like to have access to this data.
After speaking with Kapsch about how the data will get into the OS, the following was decided:
1.) Kapsch is going to provide Smart Columbus with a Docker image that will allow Smart Columbus to set up their own instance to receive the CVE data via push.
2.) Data will be in JSON with an encoded payload and metadata
3.) The data will be pushed into Docker and then pushed to Smart Columbus
The sample data has been reviewed by @ramboyapati , has been approved, and is attached to this card.
Scope of this card:
1.) Review the following from Rob at Kapsch, and set it up for testing:
Please use this link https://urldefense.proofpoint.com/v2/url?u=https-3A__kess-2Deu.kapsch.net_-5Flayouts_download.aspx-3FSourceUrl-3Dhttps-3A__kess-2Deu.kapsch.net_3eb9aceb5c1643f2af84f9dcbf6943d0_cmcc-2Ddata-2Dpublisher-2D1.0.tar&d=DwMGaQ&c=eIGjsITfXP_y-DLLX0uEHXJvU8nOHrUK8IrwNKOtkVU&r=XrNCvsK8oGSPn2nKVSZNgTCrrZTyBZgwZUWdXGz02QM&m=vvQwOl6fJLRM-noyl3o07LsfcmBkUij6708YScaEmY8&s=11One0JqrVnrbYmmDWboYp1Jk3kuXvEMDqnLDWMRnDk&e= to download a docker image for publishing CMCC data.  It is rather large at over 400MB so please be patient and try to do it from a high speed connection if possible.
I am not sure of your familiarity with Docker so here is a quick summary.

To load the image into a Docker environment use this command: docker load --input cmcc-data-publisher-1.0.tar
To create a container for the image that runs in the background run this command: docker run -d  --net=host -e cmccdatapublisher.udpPublishHost=<IP_ADDRESS> -e cmccdatapublisher.udpPublishPort=5555  --name=cmcc-data-publisher docker.gde.kapschtraffic.com/connected-mobility-platform/cmcc-data-publisher:1.0
You can of course change the name of the image and the host and ports for publishing in the command above.
To check the logs run: docker logs cmcc-data-publisher and if it is working you should see something like these lines as the last ones:
INFO: Starting CMCC data publishing from production.etranssystems.com:255 to <IP_ADDRESS>:5555
Oct 29, 2019 5:54:43 PM net.kapsch.CMCCCommServerManager setDataTypes
INFO: Getting data of type: BSM
Oct 29, 2019 5:54:43 PM net.kapsch.UDPPublisher
INFO: Publishing data to-> <IP_ADDRESS>:5555
Oct 29, 2019 5:54:43 PM net.kapsch.CMCCCommServerManager sendDataRequest
INFO: Setting up subscription for RSU: PMW01718
Oct 29, 2019 5:54:43 PM net.kapsch.CMCCCommServerManager sendDataRequest
INFO: Sending data request
Oct 29, 2019 5:54:43 PM com.etranssystems.commmgr.client.CommServerManager getOrCreateConnection
INFO: Setting up connection to: CONN1 via: production.etranssystems.com:255
Oct 29, 2019 5:54:47 PM net.kapsch.ApplicationManager logStarted
INFO: Started ApplicationManager in 5.034 seconds (JVM running for 5.412)

There are some parameters which are defaulted in the app configuration that we can change later if needed but I think this should be enough to get you started.
2.) Assess how we would build a push gateway with the following criteria, and discuss with @carriebrinkley , @jeffgrunewald , and @jmorris-pillar :

@jeffgrunewald please complete this section

Fake issue - testing functionality.


cross validation

E:\Anaconda folder\lib\site-packages\sklearn\cross_validation.py:44: DeprecationWarning: This module was deprecated in version 0.18 in favor of the model_selection module into which all the refactored classes and functions are moved. Also note that the interface of the new CV iterators are different from that of this module. This module will be removed in 0.20.
"This module will be removed in 0.20.", DeprecationWarning)
plz solve my problem

you should use the updated version at https://github.com/amueller/scipy-2017-sklearn/

dear sir.
i need data dictionary of the customer churn dataset
i extremely need it .because i want to know why these fields(state, area
code,phone number ,churn) are removed in customer churn dataset .so that i
will be able to get the information about dataset.
so please sir send me data dictionary as soon as possible. i will be very
thankful to you.
On Mon, Jul 17, 2017 at 9:25 PM, Andreas Mueller<EMAIL_ADDRESS>wrote:

you should use the updated version at https://github.com/amueller/
scipy-2017-sklearn/
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/amueller/scipy_2015_sklearn_tutorial/issues/61#issuecomment-315797854,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AXuJEyUSl92k2H4i0o5JZDc_p3Iec-YDks5sO4mvgaJpZM4OZn-C
.

There was no churn data in this tutorial as far as I know.


fail to recover network chaos

networkchaos  was not working halfway

chaos-daemon error log
2020-01-06T02:37:41.045Z        INFO    chaos-daemon-server     Delete netem    {"Request": "container_id:\"docker://3b45b159bfddcf83602149939bdde8e89643ef47003e1f1b3b15da5ce8564c2e\" "}
2020-01-06T02:37:41.047Z        INFO    chaos-daemon-server     Cancel netem on PID     {"pid": 121865}
2020-01-06T02:37:41.047Z        ERROR   chaos-daemon-server     failed to remove Qdisc  {"error": "invalid argument"}
github.com/go-logr/zapr.(*zapLogger).Error
        /home/pingcap/go/pkg/mod/github.com/go-logr/zapr@v0.1.0/zapr.go:128
github.com/pingcap/chaos-mesh/pkg/chaosdaemon.Cancel
        /home/pingcap/go/src/github.com/pingcap/chaos-mesh/pkg/chaosdaemon/netem.go:98
github.com/pingcap/chaos-mesh/pkg/chaosdaemon.(*Server).DeleteNetem
        /home/pingcap/go/src/github.com/pingcap/chaos-mesh/pkg/chaosdaemon/netem_server.go:51
github.com/pingcap/chaos-mesh/pkg/chaosdaemon/pb._ChaosDaemon_DeleteNetem_Handler
        /home/pingcap/go/src/github.com/pingcap/chaos-mesh/pkg/chaosdaemon/pb/chaosdaemon.pb.go:625
google.golang.org/grpc.(*Server).processUnaryRPC
        /home/pingcap/go/pkg/mod/google.golang.org/grpc@v1.24.0/server.go:995
google.golang.org/grpc.(*Server).handleStream
        /home/pingcap/go/pkg/mod/google.golang.org/grpc@v1.24.0/server.go:1275
google.golang.org/grpc.(*Server).serveStreams.func1.1
        /home/pingcap/go/pkg/mod/google.golang.org/grpc@v1.24.0/server.go:710

You could exec into this pod and use tc to get currently existing Qdisc. I suspect that this error occurs because qdisc was not added to it successfully.
But why it was not added successfully (though it had successfully reconciled)? Maybe we can find out the reason in chaos-daemon's log (during related time).

From the chaos-daemon log, the chaos-dameon did not receive the command to apply netem, so we cancel this netem was failed.

From the chaos-daemon log, the chaos-dameon did not receive the command to apply netem, so we cancel this netem was failed.

means there is something abnormal in crd and chaos-daemon?


Task: lazy load editor component

Description

Please include a summary of the changes and which issue has been fixed. Please also include relevant motivation
and context. List any dependencies that are required for this change.

Fixes # (issue)
Lazy load CodeEditor using lazy and suspense
Checklist:

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes

/ok-to-test sha=6dfe0e2


Create an "unstable" feature

Users can opt into the unstable feature in order to gain access to items and modules that may not be fully baked yet.
A good candidate for this would be the RouteInjector component, because that has usability issues at the moment that might require changes.

This doesn't make sense at the current time - because everything is unstable. This can be revisited whenever new functionality is added after things have settled down for a bit.


docs: extend readme

Description of your changes
Extend / correct README.md to include the submodules step.
I have:

[x] Read and followed Crossplane's contribution process.
[x] Run make reviewable test to ensure this PR is ready for review.

How has this code been tested
Ran make submodule as the first command before buildung

Thanks!


Don't call next next dup on destroyed mime field mloc.

plugins/background_fetch/headers.cc
plugins/cache_range_requests/cache_range_requests.cc
plugins/experimental/access_control/headers.cc
plugins/experimental/cache_fill/background_fetch.cc
plugins/header_rewrite/operators.cc
plugins/prefetch/headers.cc

[approve ci Clang-Analyzer]

Cherry-picked to v9.0.x branch.
Cherry-picked to v9.1.x branch.

@ezelkow1 This may be a candidate for 8.1.x as well ?

@ezelkow1 This may be a candidate for 8.1.x as well ?

Agreed, @ywkaras you mind doing a backport PR for 8.1?

OK.  I assume we need to backport https://github.com/apache/trafficserver/pull/7825 also.


[fix] copy filter_scopes with duplicate charts

CATEGORY
Choose one

[x] Bug Fix
[ ] Enhancement (new features, refinement)
[ ] Refactor
[ ] Add tests
[ ] Build / Development Environment
[ ] Documentation

SUMMARY
in code refactor PR #9224, it introduced a bug:
https://github.com/apache/incubator-superset/blob/7a91498cf1a9e56d4b3d7b3805076137525ea277/superset/views/core.py#L1293
here we should use old slice's id as key instead of sid which is new or copied slice's id
TEST PLAN
fixed unit test.
ADDITIONAL INFORMATION

[x] Has associated issue: #9224

REVIEWERS
@serenajiang @john-bodley

Codecov Report

Merging #9271 into master will decrease coverage by 0.02%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##           master   #9271      +/-   ##
=========================================
- Coverage   58.93%   58.9%   -0.03%
=========================================
  Files         373     373
  Lines       12014   12026      +12
  Branches     2945    2953       +8
=========================================
+ Hits         7080    7084       +4
- Misses       4755    4763       +8
  Partials      179     179

Impacted Files
Coverage Δ

...t-frontend/src/dashboard/actions/dashboardState.js
30.66% <0%> (-1.06%)
:arrow_down:

...uperset-frontend/src/views/chartList/ChartList.tsx
62.72% <0%> (-0.67%)
:arrow_down:

superset-frontend/src/chart/Chart.jsx
10.41% <0%> (ø)
:arrow_up:

...erset-frontend/src/dashboard/components/Header.jsx
41.79% <0%> (ø)
:arrow_up:

...d/src/dashboard/components/SliceHeaderControls.jsx
11.62% <0%> (ø)
:arrow_up:

.../src/dashboard/components/gridComponents/Chart.jsx
67.41% <0%> (ø)
:arrow_up:

...-frontend/src/dashboard/components/SliceHeader.jsx
24% <0%> (ø)
:arrow_up:

...nd/src/dashboard/containers/DashboardComponent.jsx
92% <0%> (ø)
:arrow_up:

...ashboard/components/gridComponents/ChartHolder.jsx
79.66% <0%> (ø)
:arrow_up:

superset-frontend/src/chart/chartAction.js
43.87% <0%> (+0.53%)
:arrow_up:

... and 1 more

Continue to review full report at Codecov.

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


Explore "BLoC"

In this video from the DartConf, one of the spokesmans was saying about this "BLoC" – Bussiness Logic Component, which has really pedantic rules to follow, but provides us a nice way how to separate a logic of an app from views (angular, flutter).

BLoC designing guidelines:

Inputs and outputs are simple Streams/Sinks only
Dependencies must be injectable and platform agnostic
No platform branching allowed
Implementation can be whatever you want if you follow the previous rules

UI design guidelines:

Each "complex enough" component has a corresponding BLoC
Components should send inputs "as is"
Components should show outputs as close as possible to "as is"
All branching should be based on simple BLoC boolean outputs

TL;DR

Move bussiness logic to BLoCs
Keep UI components simple

The task is to explore the video, try to underestand the concept and make some example app with this.

Because our source is a video and we do not wanna go through it over and over again, I did some screenshots as shown below.
Basically, to get this to work, we "only" have to separate UIs and business logic. If some platform dependent API is needed, it should be passed as a parameter and the API should be derivation of API's interface.
On the UI side, all inputs and outputs are provided through the business logic. Because Flutter app needs to be reloaded on each change in order to redraw, it's a good idea to use StreamBulder, in Angular, there is | async pipe.
The business logic therefore uses inputs (Sink), works with (platform dependent) APIs and generates some outputs (Stream).

Image 1

Image 2

Image 3

Image 4

Image 5

Image 6

To ensure everything will work good, the developers must strictly follow "aggressively pedantic rules":

Inputs and outputs are simple Streams/Sinks only
Dependencies must be injectable and platform agnostic
No platform branching allowed
Implementation can be whatever you want if you follow the previous rules

And of course, not everything has to have its own BLoC, only "complex enough" component should have one:

Each "complex enough" component has a corresponding BLoC
Components should send inputs "as is"
Components should show outputs as close as possible to "as is"
All branching should be based on simple BLoC boolean outputs

TL;DR:

Move bussiness logic to BLoCs
Keep UI components simple
Design rules aren't negotiable

I think this is a quite good way and it worth a try. As an output of this project, we may do some review of this BLoC.


Implement full ICE

This is still a WIP, but here is the overall architecture.
ICEAgent is a member of network/manager. (All of the state in network/manager could go directly on RTCPeerConnection. I think we should keep the code in the top-level as terse as possible. If it isn't relevant for a user to browse/read it should be in internal or pkg.)
Each port push/pulls from ICEAgent the STUN traffic for send/recv. The ICEAgent itself has all the timers/selected-pair.
When data comes from the RTCPeerConnection it queries the ICEAgent for the selected pair, and if it has one it will admin the proper src/dest. Otherwise it will return nothing and the traffic will be dropped/queued.
I would say glance over this now, but I hope to be done by this weekend.

Pull Request Test Coverage Report for Build 352

52 of 259 (20.08%)  changed or added relevant lines in 11 files are covered.
8 unchanged lines in 5 files lost coverage.
Overall coverage increased (+1.8%) to 27.157%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

internal/network/port-receive.go
0
1
0.0%

internal/sdp/jsep.go
0
2
0.0%

rtcpeerconnection.go
8
12
66.67%

rtcconfiguration.go
1
6
16.67%

internal/network/port-send.go
0
8
0.0%

internal/network/util.go
19
28
67.86%

signaling.go
0
14
0.0%

pkg/ice/candidate.go
0
14
0.0%

internal/stun/stun.go
0
35
0.0%

pkg/ice/agent.go
11
47
23.4%

Files with Coverage Reduction
New Missed Lines
%

internal/sdp/jsep.go
1
24.0%

signaling.go
1
4.43%

internal/network/port-send.go
2
0.0%

rtcpeerconnection.go
2
14.29%

internal/network/manager.go
2
15.24%

Totals

Change from base Build 348:
1.8%

Covered Lines:
1067

Relevant Lines:
3929

💛  - Coveralls


a bug in m-array _insert_v() method?

I'm facing something that looks a bug. I need to make space inside an array-based list , so I'm using _insert_v method to create some new initialized entries. As I understand from the documentation and from a quick view on the code: it should manage the case when the reserved/allocated space in the list is insufficient to support the insertion.
Something like this works:
my_list_resize(list, 201);
my_list_reserve(list, 512);
my_list_insert_v(list, 200, 300);

Instead this doesn't:
my_list_resize(list, 201);
my_list_reserve(list, 512);
my_list_insert_v(list, 200, 600);

Is it a bug?
In my original code I'm facing assertion violations on the contract clause: v->size <= v->alloc.

I reproduced the issue. It was due to an overflow.
It should be fixed in master.
Thanks for your report.


chore: rename the 'analyse' task to 'analyze'

This PR renames the 'analyse' task to 'analyze'. Closes #67
Can you please review this?

Thanks a lot for taking care of this 😊


Configurable storage for results

The storage volume for the results should be configurable such that the user can choose the most appropriate settings depending on his env (cloud or local).
So far, we've had the following challenges:

ReadWriteMany volumes work on local k3d clusters but not on GKE;
Standalone NFS server works on GKE but doesn't work on k3d (due to DNS issues in kubelet).

What we should do instead is exposing the relevant storage settings in values.yaml (we already expose storageClass) such that we can configure Adelphi to use AWS S3 or Google Cloud Storage.

The original goal of this ticket was to provide a way to write results in parallel to some external storage service (NFS) mainly to work around the ReadWriteMany limitation in GKE. It turns out that setting up an additional NFS server (such as Filestore) beforehand would add more complexity to the overall process and that approach wouldn't cover all bases (some users might choose to store the results in a different way).
Instead, the approach we went with is exposing the result through nginx such that the files can be easily download and moved anywhere else.
The way we work around the ReadWriteMany limitation in GKE is by having separate PVCs per job (gemini, nosqlbench, cassandra-diff). Since these PVCs can't be mounted simultaneously, we wait until the Argo workflow is done to start the results webserver. The results server has a separate lifecycle from the workflow, it will continue running even after the workflow exits such that we are able to download the data when everything is done. But once the Helm chart is uninstalled, then the results server will be terminated too.

The original goal of this ticket was to provide a way to write results in parallel to some external storage service (NFS) mainly to work around the ReadWriteMany limitation in GKE. It turns out that setting up an additional NFS server (such as Filestore) beforehand would add more complexity to the overall process and that approach wouldn't cover all bases (some users might choose to store the results in a different way).
Instead, the approach we went with is exposing the result through nginx such that the files can be easily download and moved anywhere else.
The way we work around the ReadWriteMany limitation in GKE is by having separate PVCs per job (gemini, nosqlbench, cassandra-diff). Since these PVCs can't be mounted simultaneously, we wait until the Argo workflow is done to start the results webserver. The results server has a separate lifecycle from the workflow, it will continue running even after the workflow exits such that we are able to download the data when everything is done. But once the Helm chart is uninstalled, then the results server will be terminated too.

I'll update the title of this ticket to reflect the new approach and  the actual work done here.
When we actually have the need for pluggable storage, we can revisit this.

I'll update the title of this ticket to reflect the new approach and  the actual work done here.
When we actually have the need for pluggable storage, we can revisit this.


fetchUser method doesn't work

Version
@nuxtjs/strapi: v0.1.0
nuxt: v2.13.0
Reproduction Link
https://github.com/nuxt-community/strapi-module/blob/master/lib/plugin.js#L88
Steps to reproduce

Install @nuxt/strapi
Log in console this.$strapi.user
Try to login withthis.$strapi.login() method
User object appears in console
Refresh the page
User undefined

What is Expected?

Install @nuxt/strapi
Log in console this.$strapi.user
Try to login withthis.$strapi.login() method
User object appears in console
Refresh the page
User still appears in console

What is actually happening?
In the library, you using fetchUser method, what calling this.getEntry('users', 'me') but it's not a function.
Code goes to catch block and calling this.clearToken()

@myfailemtions Thanks for your report, should be fixed in v0.1.1.


Question on how to generate coco dataset

I am able to generate COCO style json files by using  fathomnet-generate with the option --format coco.
In your Colab tutorial you explain how to use image.to_pascal_voc to generate Pascal_VOC dataset. Is there an equivalent way to generate a COCO dataset?

The COCO format describes a full dataset, whereas Pascal VOC describes annotations for an image. Therefore, something like image.to_coco wouldn't work.
Check the fathomnet-generate source to get an idea of how the --format coco flag works. I wrote a small library (coco-lib) with COCO data models as Python dataclasses to facilitate this process.


User can hide file in Warcraft III maps. Just add ~ in filename.

Or non printable filenames, can't see in MPQ Editor
Problem with ~ in filename:
https://github.com/ladislav-zezula/StormLib/blob/adac4aa0f586c7aa2a3c19aa4e687d827dc86195/src/SFileListFile.cpp#L192
Please fix it :)

I'll fix this together with https://github.com/ladislav-zezula/StormLib/issues/119.

Not fixed. I check latest MPQ Editor.
Try this archive:  ( With full internal listfile )
TestBadCharInPath.zip

Dear friend, please, learn to read.
Repeat: I'll fix this together with #119. The "I'll" part means "I will", implicating that it's gonna happen in the future.

Sorry i bad english language, i translate only "fix" and "#119" words :))))

Fixed in commit e549b5db4c8a02b8a4b9cce9477720c103d8e339


Allow "refs/" prefix in parameter to GHRepository.getRef()

Description
Allows having the "refs/" prefix in parameter to GHRepository.getRef(). This is optional, so "refs/heads/master" will work, as well as "heads/master".
Fixes #518.

There is a CI failure about "addCollaborators". From the log:
[ERROR] addCollaborators(org.kohsuke.github.GHRepositoryTest)  Time elapsed: 0.046 s  <<< ERROR!
I don't think I've changed this method, though.


Time Ordering Option on Small-Result-Set Scan Queries

This PR addresses point 2 of issue #6088 .  For scan queries where the limit is less than a configurable threshold (default 100K), users have the option to time-order their results (either ascending or descending).  Users also have the option to skip time-ordering by setting timeOrder to none, in which case the existing implementation of scan will run.  There's a limit on the number of records that can be ordered because this ordering currently occurs in memory.  There are plans to include on-disk sorting for larger result sets in the future but those changes are outside of the scope of this PR.

Closing in favour of the refactored PR: https://github.com/apache/incubator-druid/pull/7133


ERROR: point to the same directory in the file system.

I have this problem in Android studio when syncing my project.
ERROR: The modules ['@react-native-community_datetimepicker', 'react-native-datetimepicker'] point to the same directory in the file system.
Each module must have a unique path.
I can run my app anyway but when I create apk Release this error appears:

Task :@react-native-community_checkbox:verifyReleaseResources FAILED
FAILURE: Build failed with an exception.

Is there any solution??

I found this provisional solution if you people don't know how to create a apk still
insted of:
gradlew assembleRelease
do this:
gradlew app:assembleRelease

I found this provisional solution if you people don't know how to create a apk still
insted of:
gradlew assembleRelease
do this:
gradlew app:assembleRelease

I have the same issue in gradle build step while running with android studio.
not in gradlew assembleRelease

In my case, it was due to the same two lines defined in  the following files :

android/app/build.graddle, dependencies :
->  "implementation project(':react-native-datetimepicker')" was declared twice
android/app/src/main/java/com/ ProjectName / MainApplication.java
-> "new RNDateTimePickerPackage()" was declared twice

then :

Stop "react-native start"
clean build using "./gradlew cleanBuildCache"
run "react-native start"
run "react-native run-android"

You should check this issue : https://github.com/react-native-community/react-native-datetimepicker/issues/27
Apparently the README is incorrect 😬


Prim's Algorithm, Connected components and Articulation point in JAVA, Python and C++

Hi I am a participant at GSSOC'21.
And I felt important to raise this issue., as these have been missing under the topic Graphs.
Prim's Algorithm is used to find the minimum spanning tree of a graph.
and
To list the Connected components and Articulation points.
I'll give the code in Python, C++ and JAVA.
@plazzy99 @rudrakshi99 @aaadddiii @anubhavitis @dheerajkotwani @tarun26091999
could you please assign this issue to me.

If possible , i would like to contribute in python :)


ROS release

Are you planning to release this package (ideally from humble onwards)? We are planning to use it for foxglove_bridge.

That's cool!
Currently I'm planning to release it from Jazzy onwards because my backport PR (ros2/rosidl_typesupport#149) that fixes the introspection for services and actions was not merged for humble yet.

With the upstream backport complete on the humble side, what efforts would be required to support humble long term?

None to my knowledge. I have already released it for humble. If it's not available yet, it should be after the next sync.

Released for all current LTS versions.


Pods stuck in "Terminating" state on Windows nodes when deleting the pod after docker restart

We are using MCR version 20.10.9 (https://docs.mirantis.com/mcr/20.10/release-notes/20-10-9.html) and we have seen that after restarting the docker service and deleting a pod, the pod gets stuck at "Terminating" state. Describing the pod shows:
container ecddba6213ecd857171e6af11988979b5f0d16629509ad6f42385147ab24f2d9: driver "windowsfilter" failed to remove root filesystem: failed to detach VHD: failed to open virtual disk: The process cannot access the file because it is being used by another process.: rename C:\ProgramData\docker\windowsfilter\ecddba6213ecd857171e6af11988979b5f0d16629509ad6f42385147ab24f2d9 C:\ProgramData\docker\windowsfilter\ecddba6213ecd857171e6af11988979b5f0d16629509ad6f42385147ab24f2d9-removing: Access is denied.

Kubelet logs:
I0309 16:38:47.933233    2040 kuberuntime_container.go:661] "Killing container with a grace period override" pod="default/windows-log-test-6c9956b86d-t5jcp" podUID=9afb7084-8a8d-4239-8d7c-18176c80ae21 containerName="winloggertest" containerID="docker://8a26af90e71ec6eb8b8015253d6b76001d5efde3bd26e587eef9b8a8218ed2e2" gracePeriod=30
I0309 16:38:47.933773    2040 fake_topology_manager.go:47] "RemoveContainer" containerID="8a26af90e71ec6eb8b8015253d6b76001d5efde3bd26e587eef9b8a8218ed2e2"
E0309 16:38:47.945199    2040 remote_runtime.go:296] "RemoveContainer from runtime service failed" err="rpc error: code = Unknown desc = failed to remove container \"8a26af90e71ec6eb8b8015253d6b76001d5efde3bd26e587eef9b8a8218ed2e2\": Error response from daemon: container 8a26af90e71ec6eb8b8015253d6b76001d5efde3bd26e587eef9b8a8218ed2e2: driver \"windowsfilter\" failed to remove root filesystem: failed to detach VHD: failed to open virtual disk: The process cannot access the file because it is being used by another process.: rename C:\\ProgramData\\docker\\windowsfilter\\8a26af90e71ec6eb8b8015253d6b76001d5efde3bd26e587eef9b8a8218ed2e2 C:\\ProgramData\\docker\\windowsfilter\\8a26af90e71ec6eb8b8015253d6b76001d5efde3bd26e587eef9b8a8218ed2e2-removing: Access is denied." containerID="8a26af90e71ec6eb8b8015253d6b76001d5efde3bd26e587eef9b8a8218ed2e2"
E0309 16:38:47.945199    2040 kuberuntime_gc.go:146] "Failed to remove container" err="rpc error: code = Unknown desc = failed to remove container \"8a26af90e71ec6eb8b8015253d6b76001d5efde3bd26e587eef9b8a8218ed2e2\": Error response from daemon: container 8a26af90e71ec6eb8b8015253d6b76001d5efde3bd26e587eef9b8a8218ed2e2: driver \"windowsfilter\" failed to remove root filesystem: failed to detach VHD: failed to open virtual disk: The process cannot access the file because it is being used by another process.: rename C:\\ProgramData\\docker\\windowsfilter\\8a26af90e71ec6eb8b8015253d6b76001d5efde3bd26e587eef9b8a8218ed2e2 C:\\ProgramData\\docker\\windowsfilter\\8a26af90e71ec6eb8b8015253d6b76001d5efde3bd26e587eef9b8a8218ed2e2-removing: Access is denied." containerID="8a26af90e71ec6eb8b8015253d6b76001d5efde3bd26e587eef9b8a8218ed2e2"

Using kubectl delete pod --force did clean the pod from the cluster but according to kubelet logs, it keeps trying to remove the old container with no luck:
I0309 16:50:48.263635    2040 kuberuntime_container.go:661] "Killing container with a grace period override" pod="default/windows-log-test-6c9956b86d-t5jcp" podUID=9afb7084-8a8d-4239-8d7c-18176c80ae21 containerName="winloggertest" containerID="docker://8a26af90e71ec6eb8b8015253d6b76001d5efde3bd26e587eef9b8a8218ed2e2" gracePeriod=30
I0309 16:50:48.264169    2040 fake_topology_manager.go:47] "RemoveContainer" containerID="8a26af90e71ec6eb8b8015253d6b76001d5efde3bd26e587eef9b8a8218ed2e2"
E0309 16:50:48.275910    2040 remote_runtime.go:296] "RemoveContainer from runtime service failed" err="rpc error: code = Unknown desc = failed to remove container \"8a26af90e71ec6eb8b8015253d6b76001d5efde3bd26e587eef9b8a8218ed2e2\": Error response from daemon: container 8a26af90e71ec6eb8b8015253d6b76001d5efde3bd26e587eef9b8a8218ed2e2: driver \"windowsfilter\" failed to remove root filesystem: failed to detach VHD: failed to open virtual disk: The process cannot access the file because it is being used by another process.: rename C:\\ProgramData\\docker\\windowsfilter\\8a26af90e71ec6eb8b8015253d6b76001d5efde3bd26e587eef9b8a8218ed2e2 C:\\ProgramData\\docker\\windowsfilter\\8a26af90e71ec6eb8b8015253d6b76001d5efde3bd26e587eef9b8a8218ed2e2-removing: Access is denied." containerID="8a26af90e71ec6eb8b8015253d6b76001d5efde3bd26e587eef9b8a8218ed2e2"
E0309 16:50:48.275910    2040 kuberuntime_gc.go:146] "Failed to remove container" err="rpc error: code = Unknown desc = failed to remove container \"8a26af90e71ec6eb8b8015253d6b76001d5efde3bd26e587eef9b8a8218ed2e2\": Error response from daemon: container 8a26af90e71ec6eb8b8015253d6b76001d5efde3bd26e587eef9b8a8218ed2e2: driver \"windowsfilter\" failed to remove root filesystem: failed to detach VHD: failed to open virtual disk: The process cannot access the file because it is being used by another process.: rename C:\\ProgramData\\docker\\windowsfilter\\8a26af90e71ec6eb8b8015253d6b76001d5efde3bd26e587eef9b8a8218ed2e2 C:\\ProgramData\\docker\\windowsfilter\\8a26af90e71ec6eb8b8015253d6b76001d5efde3bd26e587eef9b8a8218ed2e2-removing: Access is denied." containerID="8a26af90e71ec6eb8b8015253d6b76001d5efde3bd26e587eef9b8a8218ed2e2"

Searching for handles on this path shows that System process holds sandbox.vhdx:
System	4	File	C:\ProgramData\docker\windowsfilter\8a26af90e71ec6eb8b8015253d6b76001d5efde3bd26e587eef9b8a8218ed2e2\sandbox.vhdx

Environment:

Kubernetes version: v1.21.5
OS version: Windows Server 2019 Datacenter   10.0.17763.2565
Docker version: docker://20.10.9

We are still experiencing this issue. Could you reproduce this issue?

Hi, just following up to let you know we are currently investigating a similar issue. Nothing more to report right now though.

Hey @ilanitr, we're in the middle of a fix for this and tracking a similar issue internally (internal ID: 31661934)

No updates to share yet, but we're making progress on the fix for this.

We've already worked on a fix, but we don't have timelines to share yet of when this will be available.

@fady-azmy-msft you were the last to comment on this a while ago - has there been any progress on this item?

@jeffwmiles yes, the fix was validated internally and we expect the fix to roll out soon. I don't have a timeline to share unfortunately since this depends on our backporting priority and process but I will update the thread when it comes out.

I took am looking forward to a fix.

Any update on this? We're closing in on 2 years since a fix was developed, and it seemingly has yet to be released.

Hey all, thank you for your patience on this. We shipped the fixed for this issue with the 2024 Feb release of Windows Server 2022. Can you confirm that the issue has been solved on your end?

^ Bumping this.

Closing due to lack of response.


Correct the capitalistion of 'Wellcome' in the footer

Per the commit message.
“Wellcome” is a proper noun and should be capitalised as such.

I had to quickly verify that this isn't part of the style given to us by Design for the lockup. You're right @alexwlchan - it should be W

Merged - but let me know if you want to release etc.

@jamesgorrie I should probably learn how to release at some point. I’ll have a look through the README and see how much I understand.


[VS Code] Removing Terminal: Cannot read property 'terminalId' of undefined

Error:
Removing Terminal: Cannot read property 'terminalId' of undefined
Steps to Reproduce:

Version Data

extensionName
VSLS

extensionVersion
0.3.954

protocolVersion
2.2

applicationName
VSCode

applicationVersion
1.29.1

platformName
Windows

platformVersion
10.0.17763

Thanks for reporting the issue, to help us diagnose the problem, please attach the liveshare logs by running "Export liveshare logs" command from command palette.

log.zip

@brandonbayliss8 Did you click "Remove Terminal" on the Live Share viewlet (the panel to the left of VSCode window)?

My terminal dont work via vsc it used to but when i reset my pc it stopped working and thet poped up

We fixed Live Share bug which could result in Cannot read property 'terminalId' of undefined error message.
The other one, where PowerShell is not starting properly, looks like not related to Live Share as it repros when you're not in a collaboration session. Can you reproduce it if you uninstall Live Share extension?

I’m going to close this as resolved, since we fixed this error. However, if you’re still running into issues, please let us know and/or reactivate this bug. Thanks!


Fix broken link to Normalizing State Shape article and composite primary key example

Updating broken link to article: Redux Normalizing State Shape documentation.
From: http://redux.js.org/docs/recipes/reducers/NormalizingStateShape.html
To: https://redux.js.org/recipes/structuring-reducers/normalizing-state-shape/

Uhhh thank you so much!

Glad to be able to help! 😄


Add delete_key functionality and improve API

Fixes #7.

This isn't quite ready to be merged yet because of some sort of Travis CI problem, but otherwise the code should be good, I think.

Can we remove the hardcoded tmp?
This works better I think:

import tempfile
with tempfile.NamedTemporaryFile() as temp:
    temp.write(script content.......)
    temp.flush()
    exec(['/bin/sh',temp.name])

Current coverage is 54.78% (diff: 5.55%)

Merging #13 into master will decrease coverage by 2.05%

@@             master        #13   diff @@
==========================================
  Files            12         12
  Lines           431        449    +18
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits            245        246     +1
- Misses          186        203    +17
  Partials          0          0

Powered by Codecov. Last update 24dcb0a...8cc143e

The only trouble with doing that is that, tempfile is for creating a temporary file on the local file system, whereas we want a temporary file on the remote file system.
I could use a uuid4 instead, so at least the name should not collide with any others. Would that be better?


Add encryptionKey for deploy cloudhub with maven

What happened 👀
To use secure properties in CloudHub, we need to set the encryption key when deploy so that the CloudHub can use the value to decrypt in the runtime.
Insight 📝
add encryptionKey as the inputs and deployed flag -DencryptionKey="${encryptionKey}"

wrong branch name


[IMPROVEMENT] Need Build Instructions + Better Build Method

Please confirm these before moving forward.

[X] I have searched for my feature proposal and have not found a work-in-progress/duplicate/resolved/discarded issue.
[X] This improvement refers to an existing feature. If you want to suggest a new feature, please use this template.
[X] This improvement is not a bug. If you want to report a bug, please use this template.

Describe the improvement
There's build_release.bat in the main dir of the repo, but there's no instructions on anything beyond that. There's no information on prerequisite libraries required, no manual compilation steps, nothing. Just a batch file that executes a ton of nested Python code. Having an actual build method in like Clang or something would be way better.
Describe how this improvement could help users
The current build method executes a bunch of nested Python code. There's no way to tell what that code is doing at a glance. Having a standard build method with detailed instructions ensures that the build can be reproduced, is secure, and can be contributed to in the future (you know, because open source).

There's build_release.bat in the main dir of the repo, but there's no instructions on anything beyond that. There's no information on prerequisite libraries required, no manual compilation steps, nothing. Just a batch file that executes a ton of nested Python code. Having an actual build method in like Clang or something would be way better.

This is used to build the final build, with the installed.
For test builds, visual studio or jetbrains rider should be plug and play. Just open the solution and the editor should figure everything out, since UniGetUI is built with .NET projects

However, I agree this needs clarification

There's an instructions page on your website, it just requires a lot of digging to find (and appears to be for old WingetUI, that ran on Python).

yes, documentation on my web page is quite outdated, I have to work on it...


[stable/k8s-spot-termination-handler] Merge [incubator/kube-spot-termination-notice-handler] into this chart

What this PR does / why we need it:
Apparently, there are 2 charts for the same products in stable and incubator:

incubator/kube-spot-termination-notice-handler added by https://github.com/helm/charts/pull/1908
stable/k8s-spot-termination-handler added by https://github.com/helm/charts/pull/8187

This PR merges useful changes from [incubator/kube-spot-termination-notice-handler] to [stable/k8s-spot-termination-handler]:

Switches to the image kubeaws/kube-spot-termination-notice-handler which has better support and versioning.
Adds the support of Slack integration
Allows to customize EC2 metadata poll interval
Improves the RBAC policies, so there should be no issues for the k8s-spot-termination-handler to drain the node (the fix https://github.com/helm/charts/pull/4730 ported)
Updates k8s labels according to the latest guidelines of Helm (https://github.com/helm/helm/blob/master/docs/chart_best_practices/labels.md#standard-labels)
Deletes incubator/kube-spot-termination-notice-handler chart. Considered it as promoted to stable by this PR.

Checklist

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md

cc: @egeland , @kierranm
Closes #10270 as superseded

/assign @mattfarina

/ok-to-test
/lgtm


Can't load images from external source.

Whenever I have an img tag inside a template I get Error: Navigation failed because browser has disconnected! and no image appears.
If I use page.goto instead of page.setContent it will load images, but I need to load my own template.
Using  "puppeteer": "^5.2.1"
async function printPDF() {
    const browser = await puppeteer.launch();

    const template = ejs.compile(fs.readFileSync("./src/template.ejs", "utf8"));
    const html = template();

    const page = await browser.newPage();

    page.setContent(html);

    await page.emulateMediaType('screen');

    const pdf = await page.pdf({
        format: "A4",
        printBackground: true,
        margin: { top: "1cm", right: "1cm", bottom: "1cm", left: "1cm" },
    });

    await browser.close();

    return pdf;
}

Currently only tested this locally on my Mac. Any ideas what's wrong?

I think you need an await in front of page.setContent().  That would help it to wait until the content is loaded. But also, you may need to include another instruction to allow all of the network activity to resolve before rendering your PDF.  Look for something to which you can pass an option of { waitUntil: 'networkidle0' }

hello David, did you solve it？

I'm facing this same issue. anyone with solution to this?

Same issue here... Any updates?

@icheered  Maybe you can try this as it worked for me
await page.setContent(html, {       waitUntil: ["load", "networkidle0", "domcontentloaded"],     });

Try to include image in  base64-encoded string format. you can use the following function to convert the image URL to a base64 string
async function getBase64FromUrl(url) {
  const response = await axios.get(url, { responseType: 'arraybuffer' });
  const buffer = Buffer.from(response.data, 'binary');
  return buffer.toString('base64');
}

const logo = await getBase64FromUrl('/images/logo-placeholder.png');
Once you have the base64 string, you can pass this to your header or footer template and use it as follows
src="data:image/jpeg;base64,<%= logo %>"
Its work for me. I hope its help you too.


Sequence not running all children nodes while a child is in the RUNNING state

Godot version: 4.0.0 Stable
Describe the bug
While a Sequence's child is in the RUNNING status, it will not process any other children every tick until the RUNNING process returns either SUCCESS or FAILURE
To Reproduce
Steps to reproduce the behavior:

Create a Sequence Node
Create at least 2 children nodes
Set one node to be in the RUNNING state every tick
Print something in the second node and it will not print anything until after the first time

Expected behavior
The Sequence should still process other children nodes while a child is in the RUNNING state.
Screenshots

"PLAYER WAS NOT FOUND" should be printed every tick instead of just once.
Desktop (please complete the following information):

OS: Windows 10, in engine, not a built application

This behavior is expected, as described here:

"Tick again" means that the next time the sequence is ticked, the same child is ticked again. Previous sibling, which returned SUCCESS already, are not ticked again.

When a sequence node encounters a running child node, it will pause execution of the sequence and wait for the running child node to complete before continuing with the next child node. So, after the running child node returns a running status, the sequence node will not immediately proceed to the next child node. Instead, it will re-tick the running child node on the next update and wait for it to complete.
What you are looking for instead is the parallel node https://github.com/bitbrain/beehave/issues/58 that will execute all its children in order (even if some of them are currently running). That node will return SUCCESS when all children returned SUCCESS eventually. However, that node is not implemented yet. Feel free to raise a pull request.


Update export list after installing a plugin

If you have the export window open, and install a plugin, that plugin is not available (you need to close the window and open again)
we should somehow make it update so the plugin is usable when it's installed.

You mean the "editor" window, right?


Use "RoleBinding" instead of ClusterRoleBindings, so it works with namespaces

Based on @yangguan suggestion.

Jenkins job istio/presubmit passed


[Gally]: master <- dev

Automatically created by Git-Ally

:tada: This PR is included in version 2.5.25 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Support list all datasets for a container

Codecov Report
Patch coverage has no change and project coverage change: +14.07 :tada:

Comparison is base (0bfd8b8) 0.00% compared to head (7b1ae46) 14.07%.

Additional details and impacted files
@@              Coverage Diff              @@
##             master    #2476       +/-   ##
=============================================
+ Coverage          0   14.07%   +14.07%
- Complexity        0     2360     +2360
=============================================
  Files             0      779      +779
  Lines             0    62217    +62217
  Branches          0     7643     +7643
=============================================
+ Hits              0     8754     +8754
- Misses            0    52955    +52955
- Partials          0      508      +508

Impacted Files
Coverage Δ

...com/github/ambry/account/CachedAccountService.java
0.00% <0.00%> (ø)

.../com/github/ambry/account/MySqlAccountService.java
0.00% <0.00%> (ø)

...ava/com/github/ambry/account/mysql/AccountDao.java
0.00% <0.00%> (ø)

.../github/ambry/account/mysql/MySqlAccountStore.java
0.00% <0.00%> (ø)

.../java/com/github/ambry/account/AccountService.java
0.00% <0.00%> (ø)

...github/ambry/config/MySqlAccountServiceConfig.java
0.00% <0.00%> (ø)

...src/main/java/com/github/ambry/rest/RestUtils.java
0.00% <ø> (ø)

...hub/ambry/frontend/FrontendRestRequestService.java
0.00% <0.00%> (ø)

...com/github/ambry/frontend/ListDatasetsHandler.java
0.00% <0.00%> (ø)

.../com/github/ambry/account/InMemAccountService.java
6.28% <0.00%> (ø)

... and 769 files with indirect coverage changes

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


Feature/apns

Adding Apns to SDK.

We dediced to add this code after refactoring.
Closing PR as unmerged.


Add LICENSE file.

Hi. Please add MIT License file. Currently the only source of truth about license type is:
https://www.getzola.org/themes/blow/
I used "selection of language", "sidebar" and "deployment pipeline" from this repository in my theme and I want to include proper information in README and LICENSE to be precise about author and source of these pieces of code.

Proposed PR: https://github.com/tchartron/blow/pull/10

I'm going to merge this, thank you for adding License and thank you for being so careful about the pieces of code you use and how it's owner would want to share it, that is great respect 👍🏼


P1S - Aetherflail during Last Shining Cells

It shows the wrong safe tiles during the final phase of the fight.

Weird, didn't see that happen. Next time you do the fight, please record the log before pulling (General settings -> Dump world state events), and then attach it to the issue, so that I can investigate.

Can confirm this has happened to me a few times, I'll keep the option ticked and will provide the log if I ever come across it again!

Outdated.


duplicate identifier errors

Lots of  duplicate identifier errors due to conflict between phaser and pixi. E.g.
phaser.comments.d.ts and pixi.comments.d.ts etc.
Steps to Reproduce
Install latest version of TS (2.3+)
Create an empty asp.net visual studio project in VS2017
Add the collection of latest Phaser typescript definitions to project
errors... lots of them

Thank you for taking the time to work on this and open your pull request. However, Phaser 2 is now maintained in the Phaser CE GitHub repo. If the issue still exists in Phaser CE then please submit your issue there and I'm sure the community will be able to help out or offer guidance.


Segmentation fault

when I import threestudio from root folder, I run into this error.
Anyone know why? I follows the instruction for installation closely

Hi @Cli98 ,
Could you provide more detailed information about this error?


[Enhancement]Optimize eventmesh-connectors module structure

Search before asking

[X] I had searched in the issues and found no similar issues.

Enhancement Request

IMO,The module structure needs to be optimized
Describe the solution you'd like

Optimize as shown in the above figure,
Are you willing to submit PR?

[X] Yes I am willing to submit a PR!

We want to separate the two modules of source/sink at the deployment, operation and development levels.

OK, The development of the Connector will be based on the situation after the changes


Allow calling for Physics step from code

fixes #24769
exposes a function that calls a physics tick with the given delta type in parameter
also allows to set Engine.physics_ticks_per_second to 0
My particular use case is training AI in a sped-up, not-real-time environment.
related to #76462 but with a different approach

Out of curiosity, why not use the --fixed-fps command line argument for this purpose? If you need to toggle it at run-time, there's a PR for that 🙂

Interesting, I knew about headless but not that there was a  way to do non-real-time simulation 🤔
In most deep learning environments, it is the training script that calls each simulation step (not the simulation environment) ; It would still make interfacing existing code easier if there was a native way to call a physics update in godot from code, but your suggestion is a valid solution. Thanks ! ☺
I guess it's up to whichever reviewer to choose to add my PR in godot.
There's probably a more elegant/flexible way to achieve the same thing, but it is serviceable enough for things like forcibly speeding jumping or rewinding time (the latter will need more than just this function though).

closing since I force-pushed to my master branch thoughtlessly

C# build fails on the methods I added


Reset password not working

Describe the bug

To be filled
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Configuration

Python version : 3.10.4
FastAPI version : 0.65.2
FastAPI Users version : 10.0.7

FastAPI Users configuration
# Please copy/paste your FastAPI Users configuration here.

Additional context
Add any other context about the problem here.

Could you give more details about the steps you are following and the configuration you have? A screenshot is not sufficient to help you.
From what I see, the token is probably expired.


Improve error message on firewall controller image not found.

Sometimes looks like this:
Last Operation:
TIME                    STATE   PROGRESS        DESCRIPTION
2021-08-02T09:24:21Z    Error   32% [Reconcile] Flow "Shoot cluster reconciliation" encountered task errors: [task "Waiting
                                                until shoot infrastructure has been reconciled" failed: admission webhook
                                                "validation.metal.provider.extensions.gardener.cloud" denied the request: controllerVersion:
                                                Required value: firewall controller version:438931e was not found in available versions: [{89c80cb
                                                https://images.metal-stack.io/firewall-controller/pull-requests/82-dns-based-egress-policies/firewall-controller
                                                0xc00092e9e0} {4e19541
                                                https://images.metal-stack.io/firewall-controller/pull-requests/109-fix-reload/firewall-controller 0xc00092ea00}
                                                {v1.0.5 https://images.metal-stack.io/firewall-controller/v1.0.5/firewall-controller 0xc00092ea10} {v1.0.9
                                                https://images.metal-stack.io/firewall-controller/v1.0.9/firewall-controller 0xc00092ea60} {v1.0.10
                                                https://images.metal-stack.io/firewall-controller/v1.0.10/firewall-controller 0xc00092eac0} {v1.1.2
                                                https://images.metal-stack.io/firewall-controller/v1.1.2/firewall-controller 0xc00092eaf0}]] Operation will be
                                                retried.

Or:
Error: [POST /v1/cluster][500] updateCluster default  admission webhook "validation.metal.provider.extensions.gardener.cloud" denied the request: controllerVersion: Required value: firewall controller version:43
8931e was not found in available versions: [{89c80cb https://images.metal-stack.io/firewall-controller/pull-requests/82-dns-based-egress-policies/firewall-controller 0xc000929d10} {92777f9 https://images.metal-s
tack.io/firewall-controller/pull-requests/82-dns-based-egress-policies/firewall-controller 0xc000929d20} {4e19541 https://images.metal-stack.io/firewall-controller/pull-requests/109-fix-reload/firewall-controlle
r 0xc000929d40} {v1.0.5 https://images.metal-stack.io/firewall-controller/v1.0.5/firewall-controller 0xc000929d60} {v1.0.9 https://images.metal-stack.io/firewall-controller/v1.0.9/firewall-controller 0xc000929d8
0} {v1.0.10 https://images.metal-stack.io/firewall-controller/v1.0.10/firewall-controller 0xc000929da0} {v1.1.2 https://images.metal-stack.io/firewall-controller/v1.1.2/firewall-controller 0xc000929db0}] (500)

Error message now looks like:
could not process: controllerVersion: Required value: firewall controller version \"foo\" was not found in available versions: [2d64a16 v1.2.2 v1.3.0 v2.0.0 v2.0.1 v2.0.2 v2.0.3 v2.0.4 v2.0.5]"


Linter fixups

What problem does the pull request solve?
The IntelliJ linter found some things it objected to. See individual commits for details.
Checklist

[x] I’ve used the pull request template
[n/a] I’ve written unit tests for these changes
[n/a] I’ve update the documentation (in DOCUMENTATION.md)
[ ] I’ve bumped the version number (in src/main/resources/application.properties)
and run the update_version.sh script

@govuk-notify-jenkins test this please

Done


Cannot auto-register commands in non-default namespace with multiple namespaces

Laravel Version
9.52.7
PHP Version
8.1.10
Database Driver & Version
Postgres 14
Description
Hello!
The App\Console\Kernel::console() does not register commands that are outside of the default namespace---this is due to a bug in the class name parsing algorithm which assumes commands are always in the App namespace and produces a 'class' that looks like:
"App\<namespace_path>\Support\Commands\EventReminders"

This can be fixed by replacing the logic in Kernel::load():
https://github.com/laravel/framework/blob/c60191f33a88f91685634b157b4dc515381fccd1/src/Illuminate/Foundation/Console/Kernel.php#L337-L341
with the logic from DiscoverEvents::classFromFile():
https://github.com/laravel/framework/blob/c60191f33a88f91685634b157b4dc515381fccd1/src/Illuminate/Foundation/Events/DiscoverEvents.php#L88-L97
which the EventServiceProvider uses to auto-register listeners (and does work with listeners in non-default namespaces).
Thanks!
Steps To Reproduce

Register 2 different namespace in application
Place command in non-default namespace
Attempt to auto-register command in Kernel

You can create a service provider for that namespace and register the commands within that
<?php

namespace CustomNamespace

use Illuminate\Support\ServiceProvider;

class CustomNamespaceServiceProvider extends ServiceProvider
{
    /**
     * The commands to be registered.
     *
     * @var array
     */
    protected $commands = [
        FooCommand::class,
    ];

    /**
     * Register the service provider.
     */
    public function register(): void
    {
        $this->commands(array_values($commands));
    }
}

Then just add the service provider to your projects ./config/app.php file, inside the providers array:
    'providers' => [
        // ...

        \CustomNamespace\ CustomNamespaceServiceProvider::class,

        // ...
    ]

@CalebDW the only real way to know if it will be accepted is to submit a pull request and find out

This isn't supported sorry. We expect commands to be in the App namespace.


Gradle dependency resolution issues with parent POM

build.gradle:
  compile group: 'com.facebook', name: 'ktfmt', version: '0.12'

Leads to an error of:
Could not resolve com.facebook:ktfmt:0.12.

Possible solution:
 - Declare repository providing the artifact, see the documentation at https://docs.gradle.org/current/userguide/declaring_repositories.html

It has to do with the parent pom not being available by name on the published repo (https://repo1.maven.org/maven2/com/facebook/ktfmt/0.12/).
Gradle is essentially traversing the pom tree and trying to resolve ktfmt-parent artifact:
https://repo.jfrog.org/artifactory/libs-release-bintray/com/facebook/ktfmt-parent/0.12/ktfmt-parent-0.12.pom?referrer
Maybe we can rename the parent pom artifact to simply ktfmt and then rename the child modules to ktfmt-core and ktfmt-vendor?

Just for additional context, I ran into this issue while adding a spotless plugin integrating ktfmt into spotless for ease of use.
The soon-to-be PR ready diff is available here: https://github.com/diffplug/spotless/compare/master...RanadeepPolavarapu:feat/ktfmt.
Ran into this issue when running the integration tests which has Gradle resolve ktfmt from Maven Central and that causes gradle to traverse POM tree and thus leading to this issue.

Exciting!!!
I'll try to look at it ASAP. Thanks for the report.

So, the 'parent' thing is due to us building ktfmt against a local checkout of google-java-format. It's done so we don't need to wait for GJF releases after we make changes to it that we need for ktfmt.
I'll see if we still need it, and whether it's actually a problem (GJF might be ok with releasing for us)
cc @cushon

@RanadeepPolavarapu, can you give it another try? use the coordinates com.facebook:ktfmt:0.13.

@cgrushko Perfect, thanks for the update. I can confirm gradle resolution is working under version 0.13.
Opened diffplug/spotless#569, hopefully should be available via Spotless soon. Will contribute the IntelliJ plugin once I finish the spotless one.

Nice!!! I'll release 0.13 then.


Signal notifier numeric boundaries

As analyzed in issue https://github.com/ngxtension/ngxtension-platform/issues/428#issue-2360862664

Thanks @leonelvsc this seems sensible - my understanding is that it will wrap v back to<PHONE_NUMBER> and then it has the space to go all the way back up to<PHONE_NUMBER>740991 then it will wrap back to<PHONE_NUMBER> again which is fine, the main consideration is that it starts on 0 and doesn't go back to 0 again because this has special meaning in terms of providing the ability to not run effects on init
Maybe there is a consideration here as to whether people will use the version number in some logic, but considering this is a reasonably unlikely case in the first place maybe we can just add a note to the docs about behaviour with large numbers

Thanks @leonelvsc this seems sensible - my understanding is that it will wrap v back to<PHONE_NUMBER> and then it has the space to go all the way back up to<PHONE_NUMBER>740991 then it will wrap back to<PHONE_NUMBER> again which is fine, the main consideration is that it starts on 0 and doesn't go back to 0 again because this has special meaning in terms of providing the ability to not run effects on init
Maybe there is a consideration here as to whether people will use the version number in some logic, but considering this is a reasonably unlikely case in the first place maybe we can just add a note to the docs about behaviour with large numbers

I've created a simple stackblitz showing the behaviour it will never reach MAX_SAFE_INT because of the >>> 0 shift it will bounce between 1 and<PHONE_NUMBER> ( 2 ** 32 - 1 =<PHONE_NUMBER> (32 bits all 1 ones then + 1 >>> 0 makes all the 32 bits 0 again)
https://stackblitz.com/edit/stackblitz-starters-rwqgze?file=src%2Fmain.ts

So another simplier solution is to initialize signal with null then on notify set "Symbol()" it will also work, if you need numeric values another solution could be Math.random() + 1

It doesn't specifically need to be numeric for the use cases outlined in the docs, just truthy/falsey works for the purpose of the init stuff so in that sense null/Symbol would work fine. But the version number is accessible and people could be relying on these incremental version numbers for something, so I think any change to that behaviour would be considered a breaking change
But the bit shift approach still seems fine to me (unless I'm missing something), since it would be:

0, 1, 2, ...,<PHONE_NUMBER>, 1, 2...

Which still satisfies the falsey/truthy requirement and keeps things incremental + infinite and non breaking

Would you mind adding a test case in the PR for the overflow behaviour?

Would you mind adding a test case in the PR for the overflow behaviour?

It woud implicate the call of 2 ** 32 times to notify() it i don't mofiy the createNotifier signature to allow an initial value, is there any other way to test this ?
for (let i = 0; i < 2 ** 32; i++) {
  trigger.notify();
}

problem here i can't mock the internal signal because is defined as const can't patch metods and have a reference to that const signal

Good point, we could separate out the update function, e.g:
e.g:
export function createNotifier(){}

export function notifierUpdateFn(version: number){}

But I'm not a fan of modifying the API for the sake of tests. We can probably just consider it an implementation detail and leave it untested, if the existing tests pass we're good imo. I'll leave it up to you though and other reviewers can weigh in if they want.

Well i think is better if we leave it untested but it's just my opinion. Test with the loop would look like:
it('should bounce back to version 1 after called 2 ** 32 + 1 times', () => {
    TestBed.runInInjectionContext(() => {
      const trigger = createNotifier();

      let triggerVersion: number = 0;

      const utilFn = () => {
        trigger.notify();
        triggerVersion = trigger.listen();
      };

      for (let i = 0; i < 2 ** 32; i++) {
        trigger.notify();
      }

      utilFn();
      expect(triggerVersion).toBe(1);
    });
  });

but it takes 478 seconds to complete the test
 PASS   ngxtension  libs/ngxtension/create-notifier/src/create-notifier.spec.ts (478.04 s)
  createNotifier
    ✓ should trigger on init (20 ms)
    ✓ should allow not triggering until explicitly called (4 ms)
    ✓ should continue to trigger when additional calls are made (3 ms)
    ✓ should bounce back to version 1 after called 2 ** 32 + 1 times (476559 ms)

Test Suites: 1 passed, 1 total
Tests:       4 passed, 4 total
Snapshots:   0 total
Time:        478.586 s
Ran all test suites matching /create-notifier/i.

We can skip that test I guess.


Introduce /itchysats/order protocol over libp2p

The new design follows the proposal in #1770. This means that
the taker now generates an OrderId when placing the order. We
currently reuse the same type for the offer (legacy) and the taker's
order, which is a bit confusing (but hard to avoid, because our model
wants an OrderId). With the new protocol the taker ignores the
OrderId associated with the offer, but we have to keep it around for
legacy connections.
One actor test got removed - with libp2p we're not changing the offers after
one was picked; the OrderId is generated on the taker side.
Co-authored-by: Lucas Soriano del Pino<EMAIL_ADDRESS>

I have slightly modified @luckysori 's PR:

removed Contracts changes, as they are orthogonal to the network migration, and the discussion on github is not resolved yet - moreover, they caused grief during rebasing, when types got migrated).
addressed some leftover TODOs
made the tests pass
removed one actor level test (we no longer remove the offer after an order got taken)

I like separating offer and order more clearly.
We might also want to add tests that show that one taker (or different takers for that matter) can take the same offer_id twice now (resulting in different order_ids.

I have rebased the protocol; it's not ready yet for another review as I need to add:
TODO:

[ ] re-add identity (we're using it in grafana now)
[ ] validate the offer by tying order and offer id together (as discussed with @da-kami )

Marking as draft until then.

I rebased on latest master for now, will see to work in the changes of the taker sending the offer id when creating an order instead of all the parameters.
@klochowicz what is the second TODO about? - i.e.:

re-add identity (we're using it in grafana now)

the TODO was about re-adding the identity - @luckysori has hardcoded a random "identity" field as it wasn't used, but we're now using it for some reporting.

I rebased on latest master for now, will see to work in the changes of the taker sending the offer id when creating an order instead of all the parameters.
@klochowicz what is the second TODO about? - i.e.:

re-add identity (we're using it in grafana now)

the TODO was about re-adding the identity - @luckysori has hardcoded a random "identity" field as it wasn't used, but we're now using it for some reporting.

Sorry, but could you provide a bit more context? What "reporting" war we talking about? Is this important for the future as well?
This identity is only relevant for the old connection as far as I understand, how would we do the "reporting" once we remove it?

@da-kami @luckysori I've added in total 2 commits into this PR  that bring Identity (taker_id and maker_id) into Order protocol. please have a look at these too :)

Interesting: installing an using a new sqlx-cli to prepare the db can indeed lead to the changes in the sqlx-data.json file. I just had such an instance, where the CI complained and I could only reproduce it after installing the new version.

bors r+


docs(adr): ADR-005: Support multisig chain link

Description

This PR introduces the draft of the 6th ADR related to Support multisig chain link in #633.

Author Checklist
All items are required. Please add a note to the item if the item is not applicable and
please add links to any relevant follow up issues.
I have...

[ ] included the correct type prefix in the PR title
[ ] added ! to the type prefix if API or client breaking change
[ ] targeted the correct branch (see PR Targeting)
[ ] provided a link to the relevant issue or specification
[ ] followed the guidelines for building modules
[ ] included the necessary unit and integration tests
[ ] added a changelog entry to CHANGELOG.md
[ ] included comments for documenting Go code
[ ] updated the relevant documentation or specification
[ ] reviewed "Files changed" and left comments if necessary
[ ] confirmed all CI checks have passed

Reviewers Checklist
All items are required. Please add a note if the item is not applicable and please add
your handle next to the items reviewed if you only reviewed selected items.
I have...

[ ] confirmed the correct type prefix in the PR title
[ ] confirmed ! in the type prefix if API or client breaking change
[ ] confirmed all author checklist items have been addressed
[ ] reviewed state machine logic
[ ] reviewed API design and naming
[ ] reviewed documentation is accurate
[ ] reviewed tests and test coverage
[ ] manually tested (if applicable)

Can we change the number of this ADR to be 005 as per #690?


pip install lightning-flash[image] results in torch downgrade to 1.8.1

🐛 Bug
One of the dependencies of lightning-flash requires torch version <= 1.7.1, this is untenable for my use case and really needs to be fixed. I believe it's an issue with icevision. As you can see icevision requires torch version strictly less than 1.9.
To Reproduce
Have torch > 1.9 installed in a conda environment. Attempt to install lightning-flash[image]. Obeserve at some point pip attempting to downgrade torch. This has now occurred on two separate machines with separate architectures, which isn't a huge issue on Linux (my requirement of 1.9 is on Windows), but on Windows it replaces the GPU version of pytorch with the CPU version.
Environment

PyTorch Version (e.g., 1.0): 1.9
OS (e.g., Linux): Linux + Windows
How you installed PyTorch (conda, pip, source): conda
Build command you used (if compiling from source):
Python version: 3.7 and 3.8

Additional context
I can trick it a bit if I install it with no dependencies. This is what I've had to do and then manually install each missing package as it comes up.

Hi @dlangerm yeah, this seems to be caused by the install of icevision. I think we can do a small refactor here to make this work, just move icevision to a fully optional dependency for a handful of tasks.

@dlangerm Interesting, haven't encountered that before, I'll keep an eye out for it, but let us know if you encounter it again or have any more info :smiley:

@ethanwharris Any status on making icevision optional?

Hi @dlangerm whoops, meant to close this issue. IceVision is now a fully optional dependency in flash 0.5.0 :smiley:


Parser

Implementation status:

Statement
Status

line
❌

lineWithEmote
❌

lineWithText
❌

lineWithEmoteAndText
❌

emote
❌

leave
❌

set
❌

if
❌

elseIf
❌

else
❌

menu
❌

label
❌

goTo
❌

background
❌

include
❌

Additionally this is missing exception handling.


does not support "Lint Files on Save"

Hello,
I've been using this extension, and it works well when opening new files.
However, when saving files after changes, the linter does not update its output. This is even though I've turned on "View -> Lint Files on Save" in the main menu.
Thanks

Hi, something must be wrong with your Brackets installation or configuration. I use this extension on a daily basis and Lint Files on Save works properly as it should. Can you get me a list of extensions, that you have installed? Thanks

$ ls -l ~/.config/Brackets/extensions/user/
brackets-coffeelint
brackets-compare
brackets-editorconfig
brackets-integrated-development
brackets-npm-registry
camden.csslint
camden.htmlhint
camden.jshint
garthdb.brackets-stylus
giovannicalo.brackets-coffeescript
globexdesigns.brackets-jscs
gruehle.markdown-preview
hirse.beautify
ivogabe.icons
jade
jshint.configurator
phoenix.brackets.phplinter
zaggino.brackets-git

here's my configuration for linting in brackets.json:
  "linting.collapsed": true,
  "linting.enabled": true,
  "language": {
    "javascript": {
        "linting.prefer": [
            "JSHint",
            "JSCS"
        ],
        "linting.usePreferredOnly": true
    }
  },

I noticed there's nothing in brackets.json about linting on save, should there be?

No, you shouldn't need to add anything there. You don't have any errors in the console at all? (F12 to open or Cmd+Opt+I on Mac)

There are some warning, no errors.
A lot of these:
Use brackets.getModule("thirdparty/CodeMirror2/lib/codemirror") instead of global CodeMirror.

And one of this:
Deprecated: Do not use $().on/off() on Brackets modules and model objects. Call on()/off() directly on the object without a $() wrapper.

I've tried this on three separate computers now and it works fine on all of them. Sorry, but I've got no idea what might be wrong with yours. Have you tried using my brackets-eslint ? It's internals are pretty much the same as the ones in brackets-coffeelint - it might be worth checking out, if it has the same problem.

@zaggino
Thanks for all your testing.
After your comment, I tried re-installing Brackets, and later re-installing all my extensions in sequence.
I found that the culprit is brackets-phplinter. I have posted an issue on the subject.
Thanks again, now it works and I love it!


IsarError: Could not initialize IsarCore library -- not the 32-bit issue

Steps to Reproduce
Hello,
I'm building a macos app and I'm getting the following basic error (details are below) - IsarError: Could not initialize IsarCore library.
I've read the following issue and discussion which report a similar error but cannot figure out how to resolve this:

https://github.com/isar/isar/issues/48
https://github.com/isar/isar/discussions/101

This is NOT related to the 32-bit issue.
pubspec.yaml:
name: api
description: A Dart API Client for the Builder API.
version: 1.0.0+1

environment:
  sdk: '>=2.17.0-222.0.dev <3.0.0'

dependencies:
  flutter:
    sdk: flutter
  flutter_lints: ^1.0.0
  http: ^0.13.0
  json_annotation: ^4.4.0
  path_provider: ^2.0.9
  flutter_secure_storage: ^5.0.2
  isar: ^2.2.1
  isar_flutter_libs: 2.2.1 # contains the binaries

dev_dependencies:
  flutter_test:
    sdk: flutter
  isar_generator: 2.2.1
  build_runner: ^2.1.8

Imports:
import 'package:isar/isar.dart';
import 'package:path_provider/path_provider.dart';

Code:
    final dir = await getApplicationSupportDirectory(); // path_provider package
    final isar = await Isar.open(
      schemas: [UserSchema],
      directory: dir.path,
      inspector: true, // if you want to enable the inspector for debug builds
    );

Error:
Exception has occurred.
IsarError (IsarError: Could not initialize IsarCore library. If you create a Flutter app, make sure to add isar_flutter_libs to your dependencies.)

Debug console:
flutter: Invalid argument(s): Failed to load dynamic library 'libisar.dylib': dlopen(libisar.dylib, 1): no suitable image found.  Did find:
	[/Users/me/Documents/Projects/the_thing/thing_desktop/build/macos/Build/Products/Debug/thing_desktop.app/Contents/MacOS/../Frameworks/isar_flutter_libs.framework/Resources/libisar.dylib](): code signature in ([/Users/me/Documents/Projects/the_thing/thing_desktop/build/macos/Build/Products/Debug/thing_desktop.app/Contents/MacOS/../Frameworks/isar_flutter_libs.framework/Resources/libisar.dylib]()) not valid for use in process using Library Validation: mapped file has no cdhash, completely unsigned? Code has to be at least ad-hoc signed.
[ERROR:flutter[/lib/ui/ui_dart_state.cc]()(198)] Unhandled Exception: IsarError: Could not initialize IsarCore library. If you create a Flutter app, make sure to add isar_flutter_libs to your dependencies.

Version

Platform: Mac, 11.6.4
Flutter version: 2.10.3
Isar version: ^2.2.1
Isar Generator version: 2.2.1

More:
% flutter doctor -v
[✓] Flutter (Channel stable, 2.10.3, on macOS 11.6.4 20G417 darwin-x64, locale en-US)
    • Flutter version 2.10.3 at /Users/me/development/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 7e9793dee1 (5 weeks ago), 2022-03-02 11:23:12 -0600
    • Engine revision bd539267b4
    • Dart version 2.16.1
    • DevTools version 2.9.2

[✓] Android toolchain - develop for Android devices (Android SDK version 32.1.0-rc1)
    • Android SDK at /Users/me/Library/Android/sdk
    • Platform android-32, build-tools 32.1.0-rc1
    • Java binary at: /Applications/Android Studio.app/Contents/jre/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.11+0-b60-7590822)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 13.2.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • CocoaPods version 1.11.3

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2021.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.11+0-b60-7590822)

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

[✓] Connected device (2 available)
    • macOS (desktop) • macos  • darwin-x64     • macOS 11.6.4 20G417 darwin-x64
    • Chrome (web)    • chrome • web-javascript • Google Chrome 100.0.4896.75

[✓] HTTP Host Availability
    • All required HTTP hosts are available

• No issues found!

Please let me know how to resolve this.
And just to confirm, does Isar support Mac, Windows and Linux or is it best to use Hive for now?
Also, I got the same errors when running flutter channel master (2.13.0-0.0.pre.429).
Thank you!

same problem

same problem here

@siberiusfr please use an x64 emulator. For now only x86 devices are supported. x86 emulators will be supported in the next version :+1:

Had the same problem, had to run melos bootstrap.
My project was divided on multiple modules/flutter apps and module which starts whole app, don't have isar_flutter_libs in pubspec.yaml.
So I had to either add it to pubspec or just run melos bs like I should with my project structure.

Thanks @bziembanski, I'll try that.

same problem :frowning_face:

@leisim
Error message:
IsarError: Could not initialize IsarCore library. If you create a Flutter app, make sure to add isar_flutter_libs to your dependencies: Invalid argument(s): Failed to load dynamic library 'libisar.so': libisar.so: cannot open shared object file: No such file or directory
package:isar/src/native/isar_core.dart 74:5      initializeIsarCore
package:isar/src/native/open.dart 81:3           openIsarSync
package:isar/src/native/isar_native.dart 119:12  _IsarNative.openSync
package:isar/src/isar.dart 105:23                Isar.openSync

flutter --version
Flutter 3.1.0-0.0.pre.672 • channel master • https://github.com/flutter/flutter.git
Framework • revision 1ea7d24e3d (7 days ago) • 2022-05-12 15:14:08 -0400
Engine • revision db84036d27
Tools • Dart 2.18.0 (build 2.18.0-106.0.dev) • DevTools 2.13.1

Isar version 2.5.0 (isar and isar_flutter_libs)
Where exactly does Isar expect the dynamic library to reside? I was thinking of manually downloading the dynamic library from the isar-core repository to circumvent the issue I am having.

Please try Isar 3.0.0. The libraries are in the isar_flutter_libs package. Do you build for 32 or 64 bit Ubuntu?

Hi @leisim
I'm trying this again on a new app MacOs app and I'm getting this error when I call Isar.open():
Exception has occurred.
ArgumentError (Invalid argument(s): Failed to load dynamic library '/Users/me/Library/Containers/com.example.isar_test_app/Data/libisar.dylib': dlopen(/Users/me/Library/Containers/com.example.isar_test_app/Data/libisar.dylib, 1): image not found)

I'd be grateful if you could tell me how to get Isar working.
Thank you.

By the way I'm using:
dependencies:
  flutter:
    sdk: flutter
  flutter_lints: 2.0.1
  go_router: 3.0.3
  flutter_localizations:
    sdk: flutter
  flutter_riverpod: 2.0.0-dev.7
  flutter_secure_storage: ^5.0.2
  dio: ^4.0.6
  rxdart: ^0.27.3
  isar: ^3.0.0-dev.0
  isar_flutter_libs: ^3.0.0-dev.0
  path_provider: ^2.0.10

dev_dependencies:
  flutter_test:
    sdk: flutter
  isar_generator: ^3.0.0-dev.0
  build_runner: ^2.1.11

@leisim I tried Isar 3.0.0, and I found the issue, but it only occurs when running tests and not the actual application. I think flutter handles running the test program differently than the application. Anyways, for testing my issue was that Isar tries to locate the dynamic library in the project directory, which is not where it is located by default. I can circumvent this issue by making a copy of the libisar.so in the project directory.

@holdenhinkle Maybe your issue is related to mine?

@jon-dez regarding tests, please check the instructions at the very bottom of the readme 👍

How silly of me... thanks!

@holdenhinkle Maybe your issue is related to mine?

Thanks @jon-dez .
It might be. But there doesn't seem to be a .dylib in isar-3.0.0-dev.0.
Also, I'm kind of new to Flutter and haven't run into a package when I had to manually copy files into a directory to get it working. Something about that seems wrong to be, but I could be wrong.
Any help or thoughts would be very much appreciated. I'd really like to use Isar for this new project (MacOs, Linux, Windows).
Thanks.

Fixed in 3.0.0-dev.7

Great news. Can’t wait to check it out. Thank you.
On Sat, Jul 16, 2022 at 4:56 AM Simon Leier @.***>
wrote:

Fixed in 3.0.0-dev.7
—
Reply to this email directly, view it on GitHub
https://github.com/isar/isar/issues/330#issuecomment-1186124802, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ACHE5J2A6L4DPHP7IEXJQKTVUJ2KXANCNFSM5S7TTSWQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
Sent from Gmail Mobile

Same problem with me. In a pure Dart project.

@catalunha For a pure Dart project you need to take care of the dynamic library and provide it. Check out this: https://github.com/isar/isar#unit-tests

Thanks for the quick response but check it out.

The method is not available in 2.5.0. If you don't want to use the newest dev version you need to download the dynamic libraries yourself :)

Where can I find the library to run Isar on Linux with pure Dart.

https://github.com/isar/isar-core/releases/tag/2.5.0

Congratulations Leisim for the Isar project.
But I'm not able to put an Isar project to run in flutter at all. Not a basic. It always gives libs error. You would have a complete base project working for us to just change Contacts through our templates.

@catalunha did you try the latest dev version? What error do you get exactly on which device? Thanks 🙂

Again Leisim thanks for the prompt service and the fantastic product.
I just connected a Contact class with ObjectBox in flutter and it went smoothly. A simple test app I'm working on.
But in Isar as much as I try the simplest way generates a lot of errors from libs and others. And the 2.5.0 documentation has a lot of differences with 3.0.0-dev.13
You would have a minimal example of Isar in flutter. Being able to save a simple Contact class in storage.
Well, it's really hard to do the least.

It's a fantastic product. I have to see it work.

For you guys that are having trouble while running tests. Deleting libisar.dylib worked for me.

i got this from crashlitycs from this device, but the app work fine:
Brand:condor
Model:Plume P6 Pro LTE
Orientation: Portrait
RAM free: 567.28 MB
Disk free: 345.93 MB
Version:Android 6.0
Orientation: Portrait
Rooted:No

Fatal Exception: io.flutter.plugins.firebase.crashlytics.FlutterError: IsarError: Could not initialize IsarCore library for processor architecture "android_arm64". If you create a Flutter app, make sure to add isar_flutter_libs to your dependencies.
Invalid argument(s): Failed to load dynamic library 'libisar.so': dlopen failed: library "libisar.so" not found
#00 pc 0xb385cf com.my.app (initializeCoreBinary [isar_core.dart:78])
#01 pc 0x7223bf com.my.app (openIsarSync [open.dart:127])
#02 pc 0x72236b com.my.app (Isar.openSync [isar.dart:137])
#03 pc 0xb37e67 com.my.app (OcDataBase.openDB [oc_database.dart:22])
#04 pc 0xa92b0f com.my.app (_SuspendState._createAsyncCallbacks.thenCallback [async_patch.dart:344])
#05 pc 0xa9be2f com.my.app (_rootRunUnary [zone.dart:1399])
#06 pc 0x2e899f com.my.app (_rootRunUnary [zone.dart:1397])
#07 pc 0x2e8d3b com.my.app (_FutureListener.handleValue [zone.dart:1300])
#08 pc 0xa9c087 com.my.app (Future._propagateToListeners.handleValueCallback [future_impl.dart:766])
#09 pc 0x2e7723 com.my.app (Future._propagateToListeners [future_impl.dart:795])
#10 pc 0x2e7233 com.my.app (Future._completeWithValue [future_impl.dart:566])
#11 pc 0xa921c3 com.my.app (_SuspendState._returnAsyncNotFuture [async_patch.dart:500])
#12 pc 0xa92b0f com.my.app (_SuspendState._createAsyncCallbacks.thenCallback [async_patch.dart:344])
#13 pc 0xa9be2f com.my.app (_rootRunUnary [zone.dart:1399])
#14 pc 0x2e899f com.my.app (_rootRunUnary [zone.dart:1397])
#15 pc 0x2e8d3b com.my.app (_FutureListener.handleValue [zone.dart:1300])
#16 pc 0xa9c087 com.my.app (Future._propagateToListeners.handleValueCallback [future_impl.dart:766])
#17 pc 0x2e7723 com.my.app (Future._propagateToListeners [future_impl.dart:795])
#18 pc 0x2e7233 com.my.app (Future._completeWithValue [future_impl.dart:566])
#19 pc 0xa921c3 com.my.app (_SuspendState._returnAsyncNotFuture [async_patch.dart:500])
#20 pc 0xa92b0f com.my.app (_SuspendState._createAsyncCallbacks.thenCallback [async_patch.dart:344])
#21 pc 0xa9be2f com.my.app (_rootRunUnary [zone.dart:1399])
#22 pc 0x2e899f com.my.app (_rootRunUnary [zone.dart:1397])
#23 pc 0x2e8d3b com.my.app (_FutureListener.handleValue [zone.dart:1300])
#24 pc 0xa9c087 com.my.app (Future._propagateToListeners.handleValueCallback [future_impl.dart:766])
#25 pc 0x2e7723 com.my.app (Future._propagateToListeners [future_impl.dart:795])
#26 pc 0x2e7233 com.my.app (Future._completeWithValue [future_impl.dart:566])
#27 pc 0xa922f3 com.my.app (_SuspendState._returnAsync [async_patch.dart:481])
#28 pc 0xa92b0f com.my.app (_SuspendState._createAsyncCallbacks.thenCallback [async_patch.dart:344])
#29 pc 0xa9be2f com.my.app (_rootRunUnary [zone.dart:1399])
#30 pc 0x2e899f com.my.app (_rootRunUnary [zone.dart:1397])
#31 pc 0x2e8d3b com.my.app (_FutureListener.handleValue [zone.dart:1300])
#32 pc 0xa9c087 com.my.app (Future._propagateToListeners.handleValueCallback [future_impl.dart:766])
#33 pc 0x2e7723 com.my.app (Future._propagateToListeners [future_impl.dart:795])
#34 pc 0x2e7233 com.my.app (Future._completeWithValue [future_impl.dart:566])
#35 pc 0x2e9277 com.my.app (Future._asyncCompleteWithValue. [future_impl.dart:639])
#36 pc 0xa9c5bf com.my.app (_rootRun [zone.dart:1391])
#37 pc 0x2e8eef com.my.app (_rootRun [zone.dart:1382])
#38 pc 0x96631b com.my.app (_CustomZone.run [zone.dart:1293])
#39 pc 0xb5cb17 com.my.app (_CustomZone.runGuarded [zone.dart:1201])
#40 pc 0x96480f com.my.app (_CustomZone.bindCallbackGuarded. [zone.dart:1241])
#41 pc 0x2e6e9f com.my.app (_microtaskLoop [schedule_microtask.dart:40])
#42 pc 0xa9ad2b com.my.app (_startMicrotaskLoop [schedule_microtask.dart:49])
#43 pc 0x2e6e13 com.my.app (_startMicrotaskLoop [schedule_microtask.dart:44])

Did you solve the error?
I also encountered this problem.  in ios and in release mode

Crashlytics reported this issue on Samsung Galaxy A22 (5G), Android 13.

i am having the same issue

Hi guys,
Am having the same error.

Any help please

Launching lib/main.dart on sdk gphone64 arm64 in debug mode...
Running Gradle task 'assembleDebug'...
FAILURE: Build failed with an exception.

What went wrong:
A problem occurred configuring project ':isar_flutter_libs'.

Could not create an instance of type com.android.build.api.variant.impl.LibraryVariantBuilderImpl.
Namespace not specified. Specify a namespace in the module's build file. See https://d.android.com/r/tools/upgrade-assistant/set-namespace for information about setting the namespace.

 If you've specified the package attribute in the source AndroidManifest.xml, you can use the AGP Upgrade Assistant to migrate to the namespace value in the build file. Refer to https://d.android.com/r/tools/upgrade-assistant/agp-upgrade-assistant for general information about using the AGP Upgrade Assistant.

Try:

Run with --stacktrace option to get the stack trace.
Run with --info or --debug option to get more log output.
Run with --scan to get full insights.
Get more help at https://help.gradle.org.

BUILD FAILED in 2s
Error: Gradle task assembleDebug failed with exit code 1


How to Run SleekDB

I'm New to SleekDB. How to run the DB?
Please Can someone give me Assistance with learning in sleekdb?

Hi @SmpManeth welcome to sleekdb :)
I would recommend to join our discord server. There are many veterans who love to help newcomers :)
https://discord.gg/WJGqUFPtZw
Regardless of that, you just need to follow the documentation 😊
And if you encounter problems you are welcome to ask for help on discord.

I just set this up today and found it extremely easy. The documentation is easy to follow here: https://sleekdb.github.io/.

I'm New to SleekDB. How to run the DB? Please Can someone give me Assistance with learning in sleekdb?

Just include store.php
and add names spaces above the PHP file
use SleekDB\Store, SleekDB\Query;
as mentioned above, its super easy, just follow the documentation
https://sleekdb.github.io/
regards

Thank you So much guys. I found the Documentation very usefull.