Add static method to generated code to return fully qualified protobuf service name

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok for us to run on our infrastructure.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok for us to run on our infrastructure.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok for us to run on our infrastructure.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok for us to run on our infrastructure.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok for us to run on our infrastructure.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok for us to run on our infrastructure.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok for us to run on our infrastructure.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok for us to run on our infrastructure.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok for us to run on our infrastructure.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok for us to run on our infrastructure.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok for us to run on our infrastructure.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok for us to run on our infrastructure.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok for us to run on our infrastructure.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok for us to run on our infrastructure.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok for us to run on our infrastructure.

jenkins: this is ok to test

No significant performance differences

No significant performance differences

I'm not sure how my PR could cause these failures. Are these tests flaky?

Known test failures: #10839 #9088 #9037 #5450 #10894

Thanks Yang!


image attribute can't tranformed to entity

image attribute can't tranformed to entity

@chengzhenping : try something like this to copy entity map also.


How to introduce a reasonable delay before Orchestrator executes automated failover

I have my cluster set up for auto-recovery via the RecoverMasterClusterFilters configuration.
This works perfectly save for one issue: my 'primary' MariaDB instance is actually a two-node Pacemaker cluster. From Orchestrator's view it looks like one server, but in the background, Pacemaker manages the two nodes in an active/passive topology.
The challenge is that if I (or Pacemaker automatically via some failure detection) moves the active MariaDB service to the other Pacemaker node, the database connection is down for 15-30 seconds, and from Orchestrator's point of view during that transition, it is a failed master, which results in the automated recovery feature of Orchestrator kicking in and moving the master elsewhere.
What I'm looking for is some way to introduce a grace period into Orchestrator's automated failover workflow. If I could delay Orchestrator deciding on failure for about a 60 second grace period, then the movements of the service amongst Pacemaker nodes would not result in spurious master failovers.
I've looked at all of the config options, and don't see anything that does what I'm looking for.
I've also considered adding a PreFailoverProcesses hook that simply sleeps 60 seconds, then checks the Pacemaker cluster to see if the service is alive again, and if so, returns a non-zero exit code, thus cancelling the rest of the failover process -- but I'm unsure of the side effects of such an approach.
So, questions:

Have I missing a config setting that will solve my problem?
If not, will the PreFailoverProcesses hook approach work, and do so without compromising Orchestrator's ability to initiate future automated failovers in the event of an actual outage?
If neither of those, would you consider adding some kind of config setting that introduces an optional grace period in automated failover?

First, I'm confused on why you need orchestrator if you have pacemaker for your failovers. Or, conversely, why you need pacemaker if you use orchestrator for your failovers.
The PreFailoverProcesses is the correct way to go. Rather, it's the only way to go right now if you want delayed failovers. FYI if a PreFailoverProcesses hook returns with non-zero exit code, the failover is aborted. That can also prove useful.
I'm not keen on a graceful period in automated failover. But you're the 2nd to ask. If more people present with use cases, I may reconsider.

I use Orchestrator to manage my MariaDB instances between regions, and Pacemaker to manage my data nodes (MariaDB, ProxySQL, NFS) in my 'primary' region.
I suppose I could chose to have Orchestrator manage all the MariaDB nodes, but that actually seems more complex given my redundancy needs in my primary region.
Thx for the clarification on PreFailoverProcesses.

I'd be interested in a grace period as well. I've got all databases running in OVH infrastructure, and I've seen network issues come and go for very short periods of time. Nothing wrong with the actual servers, but although not often, I have seen that the internal openstack networking for some reason decided on taking a very small break.
I run databases and latency sensitive applications in the same datacenter (DC). If that DC fails, it's OK if both databases and applications start up in another DC. But it would be a pity if the master database only "moves" to another DC just because the network went on a small break. By introducing a grace period, that would probably save me from having to deal with that scenario.


java.lang.NoSuchMethodError while trying to initialize

Any reason why this could happen?
Using Akka 2.5.4, also tried 2.4.14 but none of it works
val config: Config = ConfigFactory.load()
val token: String = config.getString("slack.bot.token")
val client = SlackApiClient(token)

Exception in thread "main" java.lang.NoSuchMethodError: play.api.libs.json.JsonConfiguration$.defaultConfiguration()Lplay/api/libs/json/JsonConfiguration;
	at slack.api.SlackApiClient$.<init>(SlackApiClient.scala:20)
	at slack.api.SlackApiClient$.<clinit>(SlackApiClient.scala)
	at Boot$.delayedEndpoint$com$cryptoknight$Boot$1(Boot.scala:26)
	at Boot$delayedInit$body.apply(Boot.scala:11)
	at scala.Function0.apply$mcV$sp(Function0.scala:34)
	at scala.Function0.apply$mcV$sp$(Function0.scala:34)
	at scala.runtime.AbstractFunction0.apply$mcV$sp(AbstractFunction0.scala:12)
	at scala.App.$anonfun$main$1$adapted(App.scala:76)
	at scala.collection.immutable.List.foreach(List.scala:389)
	at scala.App.main(App.scala:76)
	at scala.App.main$(App.scala:74)

After some investigation I realized that it has to do with installing play ws standalone.
If I remove those two lines everything works:
"com.typesafe.play" %% "play-ahc-ws-standalone" % wsVersion,
  "com.typesafe.play" %% "play-ws-standalone-json" % wsVersion,

Yep, I'm guessing this is caused specifically by the conflicting version of play-json being included. You should be able to use slack-scala-client with play-ws if you use the current play version: 2.6.

I am using the standalone version of ws and it is indeed the newest version
2.6.3. Also tried other versions as well, but it didn't work.
On Wed, Sep 13, 2017 at 1:20 AM Bryan Gilbert<EMAIL_ADDRESS>wrote:

Yep, I'm guessing this is caused specifically by the conflicting version
of play-json being included. You should be able to use slack-scala-client
with play-ws if you use the current play version: 2.6.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/gilbertw1/slack-scala-client/issues/59#issuecomment-329000722,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AB5qHcfGT_zj9qW72vM0xVQ0_JiTgAuqks5shwOVgaJpZM4PSAPE
.

Actually, it's been a while since I've pushed a release to maven central, and I believe that the last release was using a milestone play version: 2.6.0-M1. This might be the cause of the problem. I just merged a few pull requests and updated all of the project dependencies, including play-json. I'll do a little testing to verify that the updated dependency fixes the problem, and push a new version out to maven central.

Amazing! waiting for it

Hey, I just pushed the a 0.2.2 release to maven central. It should be available in the next few hours.

@idoshamun Have you had a chance to check if the 0.2.2 release fixes your problem?

Sorry I totally forgot, will try to get to it soon, thanks! :)
On Tue, Sep 26, 2017 at 6:33 PM Bryan Gilbert<EMAIL_ADDRESS>wrote:

@idoshamun https://github.com/idoshamun Have you had a chance to check
if the 0.2.2 release fixes your problem?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/gilbertw1/slack-scala-client/issues/59#issuecomment-332237863,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AB5qHfHqEoSqU68uP0fjLx3per45xJbqks5smRlLgaJpZM4PSAPE
.

Works even with wsVersion=1.1.1


exception is thrown at startup wscat

Hi! i need help about thrown exception when execute :
wscat -c "ws://localhost:8088/ari/events?api_key=asterisk:asterisk&app=hello-world"
Ari Rest asterisk  thrown this
/usr/lib/node_modules/wscat/node_modules/ws/lib/websocket.js:347
...options
^^^
SyntaxError: Unexpected token ...
at createScript (vm.js:56:10)
at Object.runInThisContext (vm.js:97:10)
at Module._compile (module.js:549:28)
at Object.Module._extensions..js (module.js:586:10)
at Module.load (module.js:494:32)
at tryModuleLoad (module.js:453:12)
at Function.Module._load (module.js:445:3)
at Module.require (module.js:504:17)
at require (internal/module.js:20:19)
at Object. (/usr/lib/node_modules/wscat/node_modules/ws/index.js:3:19)
please you can help me how to resolve this problem..!

What branch/image is this for? Why are you opening this on this repo at all?

hi i have the same issue on centos

This still gives me none of the informatimo I was looking for.


Performance issue with many concatenations in CXXRTL

Version
Yosys 0.29+11 (git sha1 f7ffd12bf, gcc 7.5.0-6ubuntu2 -fPIC -Os)
On which OS did this happen?
Linux
Reproduction Steps
Hi there!
I noticed that for designs many wide concatenations, CXXRTL produces a C++ source that is extremely long to compile.
For example, I gave up on this top.sv after more than an hour.
The commands are the usual as below, with an arbitrarily simple tb.cxxrtl:
commands.sh
yosys -p "read_verilog -sv top.sv; write_cxxrtl top.cpp"
g++ -g -O3 -std=c++14 -I `yosys-config --datdir`/include tb_cxxrtl.cc -o tb
./tb

The Yosys pass is incredibly fast, but g++ seems to take hours.
Here is the resulting top.cpp if you would like to have a quick look.
Interestingly, all designs with similarly large and many concatenations do not seem to be affected equally by this performance issue.
I understand that this does not sound like the most pressing issue (and certainly isn't).
Here is a longer design with the same observation.
I'm using g++ (Ubuntu 10.5.0-1ubuntu1~20.04) 10.5.0.
Thanks!
Expected Behavior
g++ terminates in reasonable time (seconds or minutes)
Actual Behavior
g++ runs for hours without an end in sight.

I’ve seen the same issue. Try it with clang++. It’s orders of magnitude faster on some template based code. And CXXRTL is nothing but templates.

Yeah, this is basically expected. I think this could be handled by splitting expressions into subexpressions when they reach a maximum depth.


Mock CAF::Path

Integration of CAF::Path with Test::Quattor, e.g. creating a mocked file using mocked FileWriter will result in a mocked basedir.
Based on #77, requires https://github.com/quattor/CAF/pull/142

@ned21 remarks addressed and renamed to CAF::Path

@jrha other typo fixed

test this please

@ned21 @jrha tests are ok now


feat: Move main.go into root for direct install.

Motivation
TL;DR: make it easier to run go install github.com/upsidr/merge-gatekeeper@v${GATEKEEPER_VERSION}
Docker Hub rate limiting makes repeatedly pulling the Golang image to build this action from GitHub actions problematic.
In my use case, a private registry and cache for the action runners is already available, to which I'm storing the amd64 build of the following:
# syntax=docker/dockerfile:1

FROM golang:1.16-alpine AS go

ARG GATEKEEPER_VERSION="1.2.1"

RUN apk add --no-cache git \
    && go get github.com/upsidr/merge-gatekeeper@v${GATEKEEPER_VERSION} \
    && cd ${GOPATH}/pkg/mod/github.com/upsidr/merge-gatekeeper@v${GATEKEEPER_VERSION} \
    && GO111MODULE=on LANG=en_US.UTF-8 CGO_ENABLED=0 go build ./cmd/merge-gatekeeper \
    && mv merge-gatekeeper /go/bin

FROM alpine:3.17 AS alpine

COPY --from=go /go/bin/merge-gatekeeper /usr/local/bin/merge-gatekeeper

ENTRYPOINT ["/usr/local/bin/merge-gatekeeper"]

Using go install would simplify this a lot.
Changes

Moved main.go and version.txt into the repository root.
Updated version.txt to the latest tagged version of your repo.

Side note: propose leveraging Actions and a Semantic release tool (example) to automate version changes.
I'm keen to contribute other changes in line with my use-case, but this one is pretty fundamental to my use case.  Thanks in advance!

Side note: propose leveraging Actions and a Semantic release tool (example) to automate version changes.

Thanks for the note, and we understand that the current setup is quite manual and error prone. We are absolutely open to ideas!
In the meantime, I'm getting this change merged 🥰


[jvm-package] jvm-packages/xgboost4j/src/native/./xgboost4j.h:2:17: fatal error: jni.h: No such file or directory

Environment:
Linux CENTOS 6.5
gcc++ 4.8.2
maven 3.5.4
java 1.8.0_181
spark cluster: 1.6.1
scala 2.10
Hello,all!
Actually I am a newcomer in xgboost in Spark. I got lots of dramatical mistakes during past two weaks (https://github.com/dmlc/xgboost/issues/3770). Finally, I believe the root reason why I get wrong is the missing of "libxgboost4j.so" file.
Thus, I cross through the directory to ../xgboost/jvm-packages and mannually run "./create_jni.sh". The interesting mistake occurs as follows【The reason why I call it interesting is due to the fact that I certainly set the JAVA_HOME not only in '~/.bashrc_profile', but also in '~/.bashrc'. In addition, the file "/etc/bashrc" and "/etc/profile" also have been set. Just like the issue https://github.com/dmlc/xgboost/issues/1011 did】:
[root@weiweiLinux jvm-packages]# ./create_jni.sh
build java wrapper
g++ -std=c++0x -Wall -O3 -msse2  -Wno-unknown-pragmas -funroll-loops -Iinclude   -Idmlc-core/include -Irabit/include -fPIC -fopenmp -I/include -I./java -I/include/linux -shared -o jvm-packages/lib/libxgboost4j.so jvm-packages/xgboost4j/src/native/xgboost4j.cpp build/learner.o build/logging.o build/c_api/c_api.o build/c_api/c_api_error.o build/common/common.o build/data/data.o build/data/simple_csr_source.o build/data/simple_dmatrix.o build/data/sparse_page_dmatrix.o build/data/sparse_page_raw_format.o build/data/sparse_page_source.o build/data/sparse_page_writer.o build/gbm/gblinear.o build/gbm/gbm.o build/gbm/gbtree.o build/metric/elementwise_metric.o build/metric/metric.o build/metric/multiclass_metric.o build/metric/rank_metric.o build/objective/multiclass_obj.o build/objective/objective.o build/objective/rank_obj.o build/objective/regression_obj.o build/tree/tree_model.o build/tree/tree_updater.o build/tree/updater_colmaker.o build/tree/updater_histmaker.o build/tree/updater_prune.o build/tree/updater_refresh.o build/tree/updater_skmaker.o build/tree/updater_sync.o dmlc-core/libdmlc.a rabit/lib/librabit.a -pthread -lm  -fopenmp -lrt  -lrt
In file included from jvm-packages/xgboost4j/src/native/xgboost4j.cpp:18:0:
jvm-packages/xgboost4j/src/native/./xgboost4j.h:2:17: fatal error: jni.h: No such file or directory
#include <jni.h>
^
compilation terminated.
make: *** [jvm-packages/lib/libxgboost4j.so] Error 1
move native lib
mv: cannot stat `lib/libxgboost4j.so': No such file or directory
complete
Does anyone know the reason?
Best Wishes

Does ${JAVA_HOME}/include directory exist?

Does ${JAVA_HOME}/include directory exist?

YES,It does! The result as follows :
[root@weiweiLinux ~]# ls ${JAVA_HOME}
bin  COPYRIGHT  include  javafx-src.zip  jre  lib  LICENSE  man  README.html  release  src.zip  THIRDPARTYLICENSEREADME-JAVAFX.txt  THIRDPARTYLICENSEREADME.txt
[root@weiweiLinux ~]# ls ${JAVA_HOME}/include
classfile_constants.h  jawt.h  jdwpTransport.h  jni.h  jvmticmlr.h  jvmti.h  linux

g++ -std=c++0x -Wall -O3 -msse2 -Wno-unknown-pragmas -funroll-loops -Iinclude -Idmlc-core/include -Irabit/include -fPIC -fopenmp -I/include -I./java [...]

See the second to last word -I/include, which corresponds to line
https://github.com/dmlc/xgboost/blob/4a8d63b6c8711fb839c71e26c659936252df1eb5/Makefile#L46
So for some reason ${JAVA_HOME} is empty. Maybe run
JAVA_HOME=/path/to/java ./create_jni.sh

or
JAVA_HOME=/path/to/java make jvm

?

g++ -std=c++0x -Wall -O3 -msse2 -Wno-unknown-pragmas -funroll-loops -Iinclude -Idmlc-core/include -Irabit/include -fPIC -fopenmp -I/include -I./java [...]

See the second to last word -I/include, which corresponds to line
xgboost/Makefile
Line 46 in 4a8d63b
export JAVAINCFLAGS = -I${JAVA_HOME}/include -I./java
So for some reason ${JAVA_HOME} is empty. Maybe run
JAVA_HOME=/path/to/java ./create_jni.sh

or
JAVA_HOME=/path/to/java make jvm

?

YES! It actually makes sense!!
[root@weiweiLinux jvm-packages]# JAVA_HOME=/usr/weiweiLinux/jdk1.8.0_181 ./create_jni.sh
build java wrapper
g++ -std=c++0x -Wall -O3 -msse2  -Wno-unknown-pragmas -funroll-loops -Iinclude   -Idmlc-core/include -Irabit/include -fPIC -fopenmp -I/usr/qudian/jdk1.8.0_181/include -I./java -I/usr/qudian/jdk1.8.0_181/include/linux -shared -o jvm-packages/lib/libxgboost4j.so jvm-packages/xgboost4j/src/native/xgboost4j.cpp build/learner.o build/logging.o build/c_api/c_api.o build/c_api/c_api_error.o build/common/common.o build/data/data.o build/data/simple_csr_source.o build/data/simple_dmatrix.o build/data/sparse_page_dmatrix.o build/data/sparse_page_raw_format.o build/data/sparse_page_source.o build/data/sparse_page_writer.o build/gbm/gblinear.o build/gbm/gbm.o build/gbm/gbtree.o build/metric/elementwise_metric.o build/metric/metric.o build/metric/multiclass_metric.o build/metric/rank_metric.o build/objective/multiclass_obj.o build/objective/objective.o build/objective/rank_obj.o build/objective/regression_obj.o build/tree/tree_model.o build/tree/tree_updater.o build/tree/updater_colmaker.o build/tree/updater_histmaker.o build/tree/updater_prune.o build/tree/updater_refresh.o build/tree/updater_skmaker.o build/tree/updater_sync.o dmlc-core/libdmlc.a rabit/lib/librabit.a -pthread -lm  -fopenmp -lrt  -lrt
move native lib
complete

[root@weiweiLinux jvm-packages]# find / -name "libxgboost4j.so"
/usr/weiweiLinux/project/xgboostv2/jvm-packages/xgboost4j/src/main/resources/lib/libxgboost4j.so

g++ -std=c++0x -Wall -O3 -msse2 -Wno-unknown-pragmas -funroll-loops -Iinclude -Idmlc-core/include -Irabit/include -fPIC -fopenmp -I/include -I./java [...]

See the second to last word -I/include, which corresponds to line
xgboost/Makefile
Line 46 in 4a8d63b
export JAVAINCFLAGS = -I${JAVA_HOME}/include -I./java
So for some reason ${JAVA_HOME} is empty. Maybe run
JAVA_HOME=/path/to/java ./create_jni.sh

or
JAVA_HOME=/path/to/java make jvm

?

Thanks for your help!!!


Oracle maximum name length reached.

All,
In certain cases, especially in nested documents, the package will create unique indexes with names that are greater than the allowed 32 length.  Of course, this is a more general issue, but when the package creates database objects on behalf of the user, it should be sure that it can before trying to do so I think.
Thanks,
Spencer

Yes, the hard limit of 32 characters is a problem and currently the package lets Oracle to fail the command. Please note, the current implementation of nested documents in the package is experimental and we don't recommend to it at this moment. Without nested data structures the limit of 32 characters should be sufficient most of the time.


Update sw-precache-webpack-plugin to lastest version

Updating sw-precache-webpack-plugin to v0.11.3 will fix the following related issues:
Windows path: https://github.com/facebookincubator/create-react-app/issues/2235
Duplicate builds: https://github.com/facebookincubator/create-react-app/pull/1728#issuecomment-301998584
I have tested by running /bin/react-scripts.js build and asserting that the build log message is not emitted twice. The problem was discovered in this https://github.com/goldhand/sw-precache-webpack-plugin/pull/78 we were calling the webpack done callback twice.
I do not have a windows machine to test the windows fix but the fix is exactly the same as in this configuration. The fix is here and has been published for a couple weeks now with no issue.
In addition to these fixes, updating to 0.11.3 will allow chunk's with hashes to be "precached" and used in importScripts.

Looks great. Thanks.


Add VXLAN properties required to support an external control plane (BGP-EVPN)

A work in progress state and request for feedback.
Will squash, etc. once any TODOs/issues have been fixed.
Add support for a configuration where a VXLAN interface is created in a non-learning mode without predefined remote VTEPs. MAC learning and VTEP configuration are handled by an external process (e.g. a BGP daemon like FRR for BGP-EVPN setups).
In this configuration the VXLAN interface might also not have an explicit parent interface, and instead traffic could go over any number of physical interfaces based on host routes (ECMP over equal links).
I've briefly tested these modifications on Fedora and been running them on CentOS 8 Stream for a while now (https://copr.fedorainfracloud.org/coprs/varesa/nmstate-git/package/nmstate/). I'll look at the project internal test suite to see how the VXLAN side is currently tested there and if it needs adding. And how to run the test suite, of course :)

/ok-to-test

Not stale :-)

@ffmancera could you comment on the few comments marked TODO in libnmstate/ifaces/vxlan.py ?

libnmstate

Sure, let me do it. Thanks!


[CELEBORN-849][DOC] Document on Master

What changes were proposed in this pull request?
As title.
Why are the changes needed?
As title.
Does this PR introduce any user-facing change?
No.
How was this patch tested?
Manual test.

cc @cfmcgrady @pan3793 @FMX

Thanks, merging to main


Add http keep alive for RPC callback. Fixes #1125

Assuming this is what https://github.com/nanocurrency/raiblocks/issues/1125 is referring to?
Not sure on the actual timeout value itself, should I make it configurable?

Not quite, the keepalive needed is at the HTTP level, not the TCP level as this PR sets.

2nd part is the http keep alive or am I in the wrong place altogether?
req->insert (boost::beast::http::field::connection, "keep-alive");
req->insert (boost::beast::http::field::keep_alive, "timeout=120");

The callback isn't a persistent connection, it makes a new connection on every observed block.

That tells the server that we support HTTP keepalive, except we don't, which would cause problems with servers that actually support HTTP keepalive.

Ahhh I get you, was looking for a quick issue from the list to get my feet wet, I guess this is not one :smiley:


NeuralNetwork() fails with multiclass problems

Issue Summary
NeuralNetwork class fails with multi class problems. Seems like this is an outstanding issue from hiive.
Steps to Reproduce / Describe the Request
Using multiclass dataset on NeuralNetwork renders an error when performing .predict.
Additional Information

Sklearn one hot encodes the labels: https://github.com/scikit-learn/scikit-learn/blob/5c4aa5d0d90ba66247d675d4c3fc2fdfba3c39ff/sklearn/neural_network/_multilayer_perceptron.py#L1124

The same was added into hiive: https://github.com/hiive/mlrose/commit/2a6d48b62aeb3ed932fba785c8265cdec2a1387f

Proposed Solution (Optional)
Will need to debug this using a toy dataset.

same issues here


wrap objectBrowserWidget with FormFieldWrapper

fixes #1969

@tiberiuichim I have used grid from FormFieldWrapper, but using this inline style scares me https://github.com/plone/volto/blob/6fd62cb2860bc7cf3cb7c36ea86bfd8bd03247d9/src/components/manage/Widgets/FormFieldWrapper.jsx#L127

@tiberiuichim I think its ready now, You can have a second look!

@sneridagh its ready , can merge it.

@sneridagh its ready , can merge it.


localStorage and sessionStorage doesn't work for browsers in "private mode"

Hi,
I see there are following user stores in the code: WebStorageStateStore and InMemoryWebStorage. When users are browsing in private mode, localStorage and sessionStorage are not accessable. InMemory is not usable since a reload would clear that one instantly as well.
What is your proposed workaround for that, so that I still can use your library?
Thanks in advance!
/Stjepan.
Sames as #269

I don't have a solution for you. Is it just localStorage, or is sessionStorage also broken in these scenarios?

If you don't want to have it persist in the browser, as that is what it is sounding like, you could fallback to anysort of database of your liking... Else just change it to public mode...?!?!

@brockallen localStorage and sessionStorage are basically the same, but in one case and that is sessionStorage clears out as soon as you close tab/browser.
@GDehnke-Turpin The last options is cookies, which would work, but there is no official userStore in the solution. I know other fixed it that way, or by preserving it on the server, as you mention.

Yea, I'm fully aware. My point was that the incognito window could provide localStorage that was actually implemented similar to sessionStorage -- that would be better than it not working at all.

I got it solved by implementing Secure Cookie Storage, if no other possible.
When in Private Mode, in Safari (both iOS and MacOS), localStorage and sessionStorage have length of 0, which will effectivelly throw an error as soon as you try to store anything.
MDN has a nice feature-detection function that can be used before you try writing to it.
https://developer.mozilla.org/en-US/docs/Web/API/Web_Storage_API/Using_the_Web_Storage_API

Closing. If you still have issues, feel free to reopen.

👍🏻
The current solutions of yours doesn't work when in private/incognito mode. My solution (with a help from another user) is by impl. secure cookie storage.

@stjepan would you mind pointing me in the direction of your solution?

@nbrady-techempower Sure!
import Cookies from 'js-cookie';

const options = {
    secure: true
};

export default class CookieStorage {
    setItem = (key, value) => Cookies.set(key, value, options);

    getItem = key => Cookies.get(key, options);

    removeItem = key => Cookies.remove(key, options);

    key = index => {
        let allKeys = Object.keys(Cookies.getJSON());
        return index > -1 && index <= allKeys.length
            ? allKeys[index]
            : '';
    };

}

const storageAvailable = type => {
    try {
        var storage = window[type],
            x = '__storage_test__';
        storage.setItem(x, x);
        storage.removeItem(x);
        return true;
    }
    catch(e) {
        return e instanceof DOMException && (
            // everything except Firefox
            e.code === 22 ||
            // Firefox
            e.code === 1014 ||
            // test name field too, because code might not be present
            // everything except Firefox
            e.name === 'QuotaExceededError' ||
            // Firefox
            e.name === 'NS_ERROR_DOM_QUOTA_REACHED') &&
            // acknowledge QuotaExceededError only if there's something already stored
            storage.length !== 0;
    }
}

export default storageAvailable;

import CookieStorage from './CookieStorage';
import storageAvailable from './storageAvailable';

export default class StorageFactory {

    static getStorage = () => storageAvailable('sessionStorage')
        ? sessionStorage
        : storageAvailable('localStorage')
            ? localStorage
            : new CookieStorage();

}

import { createUserManager } from 'redux-oidc';
import { WebStorageStateStore } from 'oidc-client';
import StorageFactory from './StorageFactory';

const userManagerConfig = {
    client_id: "your client id",
    redirect_uri: "your redirect uri",
    response_type: 'id_token',
    scope: 'openid',
    authority: "your authority",
    ui_locales: "your supported locales",
    userStore: new WebStorageStateStore({ store: StorageFactory.getStorage() }),
    stateStore: new WebStorageStateStore({ store: StorageFactory.getStorage() }),
};

const userManager = createUserManager(userManagerConfig);

export default userManager;

So, make sure you configure your userManager by overriding userStore and stateStore with the newly StorageFactory. It will check (with help of storageAvailable function, that I got from MDN) if client has support of session or local storage. If not, then it will fallback to the newly defined cookie storage.
Good luck!

Relevant: https://twitter.com/rmondello/status/910276107978412034

Hello @stjepan,
Is this still working for you? #631
I tried to change storage similar like you did, but I have an issue with userManager reading from cookie storage.
Cheers

After clearing the storage in chrome -> all options checked -> clear site date, the local/session storage is working again also in incognito mode (at least in chrome 69 ).


About _sinkhorn_iter in XLM-Align

Describe
Model I am using XLM-Align:
hello, i found that the implementation of XLM-Align define a method called _sinkhorn_iter at
here. But it seems that it has never been called. i am not sure where and how the sinkhorn algorithm is implemented?

Thank you for opening this issue! It seems a mistake emerged during code refactoring. I will fix it soon.

@zhangliang-04 The issue has been fixed by https://github.com/microsoft/unilm/pull/413


[Runtime] Allow aborting fetchNDArray through AbortSignal

This PR changes the interface of function fetchNDArrayCache of TVM runtime to accept an additional optional parameter of signal: AbortSignal to allow clients to use AbortController to abort a given fetch.
https://developer.mozilla.org/en-US/docs/Web/API/AbortController
Related issues:
https://github.com/mlc-ai/web-llm/issues/499

cc. @tqchen


Issues when .graphql files are contained in directory with a "\s" character in their name.

My project has a directory structure like so:
My Project
  |- Screens
      |- Main
          |- Change Password
          |- Advanced Settings
          |- Camera
          |- ...

When running the apollo-codegen generate $(find . -name '*.graphql') --schema path/to/schema.json --output path/to/output.swift, I get an error (customized to reflect the example above):
Error: ENOENT: no such file or directory, open '/path/to/MyProject/MyProjectSource/Screens/Main/Change'
    at Object.fs.openSync (fs.js:584:18)
    at Object.fs.readFileSync (fs.js:491:33)
    at inputPaths.map.inputPath (/usr/local/lib/node_modules/apollo-codegen/src/loading.js:28:21)
    at Array.map (native)
    at loadAndMergeQueryDocuments (/usr/local/lib/node_modules/apollo-codegen/src/loading.js:27:30)
    at generate (/usr/local/lib/node_modules/apollo-codegen/src/generate.js:15:20)
    at Object._yargs2.default.command.command.argv [as handler] (/usr/local/lib/node_modules/apollo-codegen/src/cli.js:110:9)
    at Object.self.runCommand (/usr/local/lib/node_modules/apollo-codegen/node_modules/yargs/lib/command.js:170:22)
    at parseArgs (/usr/local/lib/node_modules/apollo-codegen/node_modules/yargs/yargs.js:920:28)
    at Object.get [as argv] (/usr/local/lib/node_modules/apollo-codegen/node_modules/yargs/yargs.js:860:16)
Command /bin/sh failed with exit code 1

It looks like the script isn't considering directories/folders with "\s" in their name.
Is there an easy fix?

You could try using the Node glob support instead of letting the shell expand the pattern.
Something like: apollo-codegen generate '**/*.graphql' --schema path/to/schema.json --output path/to/output.swift

Thanks!


pytorch cannot find libopen-orted-mpir.so

What's the issue, what's expected?:
pytorch cannot find libopen-orted-mpir.so
Log message or shapshot?:
ERROR: libopen-orted-mpir.so: cannot open shared object file: No such file or directory
Traceback (most recent call last):
File "/usr/local/lib/python3.8/dist-packages/knack/cli.py", line 233, in invoke
cmd_result = self.invocation.execute(args)
File "/usr/local/lib/python3.8/dist-packages/knack/invocation.py", line 224, in execute
cmd_result = parsed_args.func(params)
File "/usr/local/lib/python3.8/dist-packages/knack/commands.py", line 146, in call
return self.handler(*args, **kwargs)
File "/usr/local/lib/python3.8/dist-packages/knack/commands.py", line 253, in _command_handler
result = op(client, **command_args) if client else op(**command_args)
File "/usr/local/lib/python3.8/dist-packages/superbench/cli/_handler.py", line 208, in exec_command_handler
executor.exec()
File "/usr/local/lib/python3.8/dist-packages/superbench/executor/executor.py", line 247, in exec
context = BenchmarkRegistry.create_benchmark_context(
File "/usr/local/lib/python3.8/dist-packages/superbench/common/utils/lazy_import.py", line 42, in getattr
self._import()
File "/usr/local/lib/python3.8/dist-packages/superbench/common/utils/lazy_import.py", line 31, in _import
self._callback()
File "/usr/local/lib/python3.8/dist-packages/superbench/benchmarks/init.py", line 15, in
'superbench.benchmarks.registry', 'BenchmarkRegistry', lambda: list(
File "/usr/lib/python3.8/importlib/init.py", line 127, in import_module
return _bootstrap._gcd_import(name[level:], package, level)
File "", line 1014, in _gcd_import
File "", line 991, in _find_and_load
File "", line 975, in _find_and_load_unlocked
File "", line 671, in _load_unlocked
File "", line 848, in exec_module
File "", line 219, in _call_with_frames_removed
File "/usr/local/lib/python3.8/dist-packages/superbench/benchmarks/model_benchmarks/init.py", line 7, in
from superbench.benchmarks.model_benchmarks.pytorch_bert import PytorchBERT
File "/usr/local/lib/python3.8/dist-packages/superbench/benchmarks/model_benchmarks/pytorch_bert.py", line 6, in
import torch
File "/usr/local/lib/python3.8/dist-packages/torch/init.py", line 191, in
_load_global_deps()
File "/usr/local/lib/python3.8/dist-packages/torch/init.py", line 153, in _load_global_deps
ctypes.CDLL(lib_path, mode=ctypes.RTLD_GLOBAL)
File "/usr/lib/python3.8/ctypes/init.py", line 373, in init
self._handle = _dlopen(self._name, mode)
OSError: libopen-orted-mpir.so: cannot open shared object file: No such file or directorynon-zero return code
additional information
Ubuntu 20.04, python3.8；OpenMPI 4.04；
I have libopen-orted-mpir.so in /usr/lib/x86_64-linux-gnu/openmpi/lib/libopen-orted-mpir.so
and I have written the path in ~/.bashrc;
I have tried to ask chatgpt4, and he cannot solve this issue.

Can't reproduce. Are you still suffering from this issue?

Close this issue due to no response.


feat: typescript migration

The results of today's Coding Badly stream: the snake now uses TypeScript!

Let the technology list GROW


Give the lang search string first preference

Fixes #97.
The ?lang=locale option has no use if it is not the first option to be
selected since the localStorage option is otherwise persistent.  By
giving it first preference, here are the possible interaction
scenarios:

User visits using the search string and gets the desired language

Sees desired language, hence happy

User browses to another page

Sees language persist despite losing the search string, hence happier

User changes language for whatever reason

Sees selected language, even happier

User browses to another page

Sees language selection persist, happiest!

Pushing this since it's better than the current state.  We can improve upon it further if needed.


Custom range selecting more than two monts

In the custom range how can one select a range of more than two months
Eg: between jan1 and may 30.
I have implemented it as in this plunker

You should be able to navigate the months and select any range by clicking on the 'chevron' arrows. See image below.
If you are unable to see the icons try adding custom icons with css. Currently icons are only supported via bootstrap 3 or font-awesome


FIX-#4945: Return empty pandas.Index instead of list in get_indices

What do these changes do?

[x] commit message follows format outlined here
[x] passes flake8 modin/ asv_bench/benchmarks scripts/doc_checker.py
[x] passes black --check modin/ asv_bench/benchmarks scripts/doc_checker.py
[x] signed commit with git commit -s
[x] Resolves #4945
[x] tests added and passing
[x] module layout described at docs/development/architecture.rst is up-to-date
[x] added (Issue Number: PR title (PR Number)) and github username to release notes for next major release

It's difficult to write a test for this because the reproducer actually shows another bug after this one is fixed!

I think it's better to show than tell. You can take a look at my changes in this PR @anmyachev

I'll go ahead and close this PR if https://github.com/modin-project/modin/pull/4951 gets merged

I'll go ahead and close this PR if #4951 gets merged

Should it be closed?

Yup let's close this @anmyachev


Proofreading wanted

Hi, my open-source buddies!
Unfortunately, my coding skills are way better than my English 😅
The README became huge and it would be cool if somebody checks and improves the texts there.
@molefrog and @rschristian hope you guys have time to help me a bit 🙏

I think it reads great! Just a couple small items stood out, not sure they need to be changed, but I'll open a PR for discussion

Thanks, guys! ❤


[bfgroup-lyra] Bump to 1.6.1

[x] Changes comply with the maintainer guide
[x] SHA512s are updated for each updated download
[x] The "supports" clause reflects platforms that may be fixed by this new version
[x] Any fixed CI baseline entries are removed from that file.
[x] Any patches that are no longer applied are deleted from the port's directory.
[x] The version database is fixed by rerunning ./vcpkg x-add-version --all and committing the result.
[x] Only one version is added to each modified port's versions file.

Note: I will be converting your PR to draft status. The above suggested changes are only recommendations. If you are willing to adopt them, please click "ready for review" after making the modifications. If you do not wish to make any changes, please click "ready for review" directly. That way, I can be aware that you've responded since you can't modify the tags.

Usage test pass with following triplets:
x86-windows
x64-windows
x64-windows-static

Thanks for the update!


怎么使用master节点gpu

请问怎么配置让集群能使用master节点的资源，包括gpu，内存，cpu，磁盘

we do not recommend running master components on worker. this could lead to issues like system instability.

got it,thank you


Output const/static functions for parameters in form header files

Discussed in https://github.com/KeyWorksRW/wxUiEditor/discussions/841
As per the discussion, the idea is to provide something in the base class header file that can be used to represent each parameter. The goal is so that you can override a parameter in separate instances of the form without requiring finding and re-implementing each parameter before the one that needs changing. For example, if the only thing changing in a wxDialog is the window name, you have to provide 5 parameters before you can specify the different window name. See end of discussion for an example of how this could be implemented.

Much as I'm reluctant to add yet another C++ property (there's already 10), there should be a way to disable it. Adding the property defaulting to true will mean the functionality is discoverable by those who really need it, and those who really don't want the extra 5+ lines in the generated header file can simply uncheck the property (if they can find it ).

I understand your comment, if I were you, I wouldn't be afraid of options.
The more your RAD tool will be flexible, refferring to generated code, the more it will be appreciated by developers.
As I see it a RAD tool shouldn't force developers to adapt using its way to generate code, on the countrary a RAD tool should be as mach as flexible and versatile as possible.


Request zipballs from GitHub over HTTPS

GitHub will redirect from HTTP requests to HTTPS, so we should avoid having to follow a redirect for every request.
Also, the error in #163 mentions things about TLS and ciphers, so I didn't know if this might reduce the chance of that error popping up (at least until someone who actually knows Haskell is able to implement a retry).
Thanks!

Duplicate of https://github.com/elm-lang/elm-package/pull/195. Closing here, but maybe add any comments you have over there.


Core6: Assertion failed (column validation on link query)

When using link query, the column validation doesn't happen on the correct table, here's a repro scenario:
InMemoryTestFile config;
        config.automatic_change_notifications = false;

        auto r = Realm::get_shared_realm(config);
        r->update_schema({
            {"Owner", {
                {"name", PropertyType::String},
                {"pet", PropertyType::Object|PropertyType::Nullable, "Cat"}
            }},
            {"Cat", {
                {"name", PropertyType::String}
            }}
        });

        auto coordinator = _impl::RealmCoordinator::get_coordinator(config.path);
        auto table_owner = r->read_group().get_table("class_Owner");
        auto table_cat = r->read_group().get_table("class_Cat");

        auto col_owner_pet = table_owner->get_column_key("pet");
        auto col_cat_name = table_cat->get_column_key("name");

        table_owner->link(col_owner_pet);
        table_owner->link(col_cat_name);

        Query query(table_owner->where());
        query.and_query(table_owner->column<Link>(col_cat_name).count() == 0);//💣 This will throw invalid column

stack
realm-core-6.0.0-alpha.17/include/realm/table.hpp:1134: [realm-core-6.0.0-alpha.17] Assertion failed: m_current_table->valid_column(ck)
0   tests                               0x000000010c38cf15 _ZN5realm4utilL18terminate_internalERNSt3__118basic_stringstreamIcNS1_11char_traitsIcEENS1_9allocatorIcEEEE + 37
1   tests                               0x000000010c38d91c _ZN5realm4util9terminateEPKcS2_lOSt16initializer_listINS0_9PrintableEE + 1580
2   tests                               0x000000010b865127 _ZN5realm9LinkChain3addENS_6ColKeyE + 119
3   tests                               0x000000010b85db9d _ZNK5realm5Table4linkENS_6ColKeyE + 61
4   tests                               0x000000010b79d86f _ZL31____C_A_T_C_H____T_E_S_T____244v + 49679

But the query is wrong. You should use col_owner_pet - the link column key. I assume that the query should find all persons with no cats.

@jedelbo
You get the same error using col_owner_pet, and I think the original query is correct since we want to find cats with an empty name (not finding persons with no cats)

That query would be something like table_owner->link(col_owner_pet).column<String>(col_cat_name) == StringData();

Ok, got it. Yes, we changed the behavior. We should probably have emphasized this a bit more, but we did not have to modify a single unit test in core, so we were not that aware. The change is done because of the need to be able to make a query in a "frozen" transaction, where you cannot modify the table object.

Fixed, by using LinkChain.


0.72 : Xiaomi Philips Light Platform not setting up

Home Assistant release with the issue:
0.72, 0.72.1
Last working Home Assistant release (if known):
0.71
Operating environment (Hass.io/Docker/Windows/etc.):
Ubuntu 18.04 on Rock64
Python 3.6.5
Component/platform:
Xiaomi Philips Light [https://www.home-assistant.io/components/light.xiaomi_miio/]
Description of problem:
The Xiaomi Philips Lightbulbs are not working with 0.72. In the logs I see python errors
Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant):
light:
  - platform: xiaomi_miio
    name: Luucht1
    host: <IP_ADDRESS>
    token: !secret xiaomi_miio_token_1

  - platform: xiaomi_miio
    name: Luucht2
    host: <IP_ADDRESS>
    token: !secret xiaomi_miio_token_2

Traceback (if applicable):
Error while setting up platform xiaomi_miio
Traceback (most recent call last):
  File "/srv/homeassistant/lib/python3.6/site-packages/homeassistant/helpers/entity_platform.py", line 129, in _async_setup_platform
    SLOW_SETUP_MAX_WAIT, loop=hass.loop)
  File "/usr/lib/python3.6/asyncio/tasks.py", line 358, in wait_for
    return fut.result()
  File "/srv/homeassistant/lib/python3.6/site-packages/homeassistant/components/light/xiaomi_miio.py", line 106, in async_setup_platform
    from miio import Device, DeviceException
  File "/srv/homeassistant/lib/python3.6/site-packages/miio/__init__.py", line 2, in <module>
    from miio.airconditioningcompanion import AirConditioningCompanion
  File "/srv/homeassistant/lib/python3.6/site-packages/miio/airconditioningcompanion.py", line 7, in <module>
    from .click_common import command, format_output, EnumType
  File "/srv/homeassistant/lib/python3.6/site-packages/miio/click_common.py", line 45, in <module>
    class ExceptionHandlerGroup(click.Group):
AttributeError: module 'click' has no attribute 'Group'

Additional information:
I made sure that python-miio version is updated to 0.4
Edit: Also happens if "model: philips.light.bulb" is in configuration.yaml
Edit 2: Tested with 0.72.1 -> same error

It looks like a old version of the "click" package is installed. Please try to update click:
su - homeassistant
source /srv/homeassistant/bin/activate
pip3 install click -U

And restart HA. If this doesn't solve the issue please update click again and try to start HA with the "--skip-pip" parameter to prevent a possible click downgrade for testing.

Tried updating click, says it's latest version (6.7
'''
pip3 install click -U
Requirement already up-to-date: click in ./lib/python3.6/site-packages (6.7)
'''

I downloaded click from https://github.com/pallets/click and placed it in /srv/homeassistant/ lib/python3.6/site-packages/click
Restarted HA, and the lights came back online (although with a "_2" appended to their entity name)
Thank you for pointing me in the right direction!


Implement startup_probe configuration

startup_probe is missing as possible inputs in the module.
This prevents some applications to start correctly.

I've created a PR to allow startup and liveness probes to be specified: https://github.com/GoogleCloudPlatform/terraform-google-cloud-run/pull/120


Unify src directory layout

This fixes #41 by creating a sub-directory for each tested Stdlib module (Atomic, Domain, ...) and library (Lockfree, Domainslib, ...)

Merging as the MacOSX CI failures are unrelated.


Reforge error

Importing via the weakaura export feature.
{"talents":"0003000000000000000000-230330221120121213231-0300000000000000000","glyphs":{"minor":["Glyph of Arcane Brilliance","Glyph of Conjuring","Glyph of Slow Fall"],"major":["Glyph of Dragon's Breath","Glyph of Blast Wave","Glyph of Evocation"],"prime":["Glyph of Molten Armor","Glyph of Fireball","Glyph of Pyroblast"]},"class":"mage","race":"Troll","name":"Athzine","gear":{"items":[{"enchant":4207,"gems":[68780,59478,59493],"id":59449},{"id":56288,"gems":[],"reforging":165},{"enchant":4199,"gems":[52208],"reforging":115,"id":56324},{"enchant":4115,"gems":[],"reforging":115,"id":56434},{"enchant":4063,"gems":[52217,52217],"reforging":144,"id":60244},{"enchant":4089,"gems":[],"reforging":165,"id":59475},{"enchant":4068,"gems":[52207],"reforging":165,"id":60247},{"id":59217,"gems":[52208,52239],"reforging":115},{"enchant":4113,"gems":[52208,52217],"reforging":167,"id":63507},{"enchant":2940,"gems":[52217],"id":62450},{"id":56333,"gems":[],"reforging":144},{"id":59501,"gems":[],"reforging":165},{"id":62047,"gems":[],"reforging":165},{"id":62465,"gems":[],"reforging":138},{"id":63536,"gems":[],"reforging":165},{"id":57923,"gems":[]},{"id":56296,"gems":[],"reforging":167}]},"professions":[{"name":"Engineering","level":525},{"name":"Tailoring","level":525}],"level":85,"spec":"","realm":"Golemagg"}
Getting this error.

@pappabgren We are not the authors of the WowSims Weakaura. The last update to the Weakaura was the 28th of May and we've fixed quite a lot of issues after that.
Make sure to install the WowSimsExporter addon to have the most up to date export feature.


Add field_access_terms to media migrations & media tests

The Access Control PR  (Role configuration) and the Media migrations happened sort of back to back. The Access Control PR doesn't account for field_access_terms in media migrations because the migrations weren't finished yet.
This ticket is to complete that work by:

[x] Migrations: adding field_access_terms to all the idc media migrations in codebase/config/sync

[x] audio
[x] document
[x] extracted text
[x] file
[ ] fits no fits migration
[x] image
[x] remote video
[x] video

[x] Verification Tests:

[x] update the csv files for media migrations to include field_access_terms in them
[x] update files containing expected json
[x] update the verification test to include checking for values in field_access_terms

Resolved with: Address field_access_terms tickets and a few minor things #178


subcommand description missing: az ad sp create-for-rbac

az ad sp create-for-rbac -h

Command
    az ad sp create-for-rbac

Arguments
    --name -n  : An unique uri. If missing, the command will generate one.
    --secret   : The secret used to login. If missing, command will generate one.
    --years    : Years the secret will be valid.  Default: 1.

Global Arguments
    --debug    : Increase logging verbosity to show all debug logs.
    --help -h  : Show this help message and exit.
    --output -o: Output format.  Allowed values: json, tsv, list, table, jsonc.  Default: json.
    --query    : JMESPath query string. See http://jmespath.org/ for more information and examples.
    --verbose  : Increase logging verbosity. Use --debug for full debug logs.

would be nice to have a sentence describing what this subcommand does.

The description will be create a service principal that can access or modify resources. It is more or less borrowed from the msdn doc

@yugangw-msft : description sounds good

#781 was merged


fix(slots): correctly respond to slot changes

Alaska Airlines Pull Request
Fixes: #32
Summary:
Update template to handle slot change events. This change is a mirror of the already released change to auro-accordion - https://github.com/AlaskaAirlines/auro-accordion/pull/21
Type of change:
Please delete options that are not relevant.

[ ] New capability
[x] Revision of an existing capability
[ ] Infrastructure change (automation, etc.)
[ ] Other (please elaborate)

Checklist:

[x] My update follows the CONTRIBUTING guidelines of this project
[x] I have performed a self-review of my own update

By submitting this Pull Request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
Pull Requests will be evaluated by their quality of update and whether it is consistent with the goals and values of this project. Any submission is to be considered a conversation between the submitter and the maintainers of this project and may require changes to your submission.
Thank you for your submission!
-- Auro Design System Team

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

@jason-capsule42 do we also need to run these lines on a slotchange?
https://github.com/AlaskaAirlines/auro-radio/blob/4f8e0ce907fefae0e8f5849f34b35258b46b4bb0/src/auro-radio-group.js#L43-L51
With the current implementation, I don't think added radios will have required/disabled properly applied.


Shorten file paths containing cloudfoundry/cloud-foundry

this helps in windows with limited file path length

Codecov Report

Merging #4466 into master will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master    #4466   +/-   ##
=======================================
  Coverage   58.89%   58.89%
=======================================
  Files         895      895
  Lines       29203    29203
  Branches     4219     4219
=======================================
  Hits        17199    17199
  Misses      12004    12004


itest: continued itest refactor and fix - III

Please check #6825 for more context.
Depending on #6822, this PR continues the fix of the tests.
This PR starts at commit lntemp+itest: refactor testEtcdFailover

@bhandras: review reminder
@sputn1ck: review reminder

@sputn1ck: review reminder

@sputn1ck: review reminder

@positiveblue: review reminder
@sputn1ck: review reminder

Looks like all of the new tests (all but one) are failing at send_to_route_error_propagation.


[AMBARI-24128] Add notification to the Alert Groups not working

What changes were proposed in this pull request?
Add notification to the Alert Groups not working.
STR:

Navigate to alerts page
Open Manage Alert Groups
Select an Alert Group
Add Notification by clicking Add and selecting a 'pre created' Alert Notification
Click Save and close the popup, the alert group should be saved with selected notification type, but only an empty String is being saved.

E.g API being invoked:
PUT https://<host>:<port>/api/v1/clusters/<cluster>/alert_groups/4

{"AlertGroup":{"name":"AMBARI_INFRA_SOLR","definitions":[14],"targets":[4]}}:

Notes:

Observed that the auto fill for the notification is case sensitive, which is not ideal.
Also observed trying to add another notification after performing the steps in STR, a 500 server error is thrown.

How was this patch tested?
UI unit tests:
21802 passing (26s)
48 pending

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/2744/
Test PASSed.


ExecutionContext has been deprecated and Immutable removed

Upstream Issue: https://issues.jboss.org/browse/WFLY-9627
Issue: https://issues.jboss.org/browse/JBEAP-13801

The compile problem in the Linux - elytron test is likely a CI environment problem.

retest this please

@msve or @mchoma is the change in the first file ok? This affects a test fixture I don't pretend to understand. :)

@msve or @mchoma is the change in the first file ok? This affects a test fixture I don't pretend to understand. :)

Me too! I actually removed the fixture and run the full tests. As nothing broke, I thought I was safe enough to at least propose thru this PR.
I could find what we are supposed to use instead of ExecutionContext.HTTP_PROXY_HOST. My gut feeling is that now, this is somehow handle transparently by the framework, but I have yet to find a proof of this intuition...

If ExecutionContext.HTTP_PROXY_HOST still exists and this gets delayed in review more than a couple weeks and that's impacting some deadline, then splitting it up and just getting rid of Immutable in one PR will unblock the upgrade.


[ADDITION] Proton Pass

Category
Password Managers
Software to be added
I want Proton Pass to be added, since it has a privacy-friendly policy, and is open-source.
Supporting Material
https://proton.me/pass
https://github.com/protonpass
Association Disclosure
I do not have any association with Proton Passm aside from using it.
Would you like to submit a PR?
Maybe?
Please tick the boxes

[X] You have filled out this form accurately, and to the best of your knowledge
[X] You have indicated weather or not you are associated with the suggested addition
[X] A similar submission has not already been opened for this software/ service
[X] You agree to the code of conduct

+1 I agree
Would you be up for submitting a PR? Otherwise I can add it later on.

+1 I agree Would you be up for submitting a PR? Otherwise I can add it later on.

I can submit one, but I'm a bit lazy.

Added.


Error: Driver.cpp:6304 - Failed to Read Network Key

I'm trying to use homeassistant 0.12 on a rPI 2 with z-wave USB adapter.
I have installed python-zwave etc with the script etc.
When I start homeassistant after a while I get this error and the process quits:
Error: Driver.cpp:6304 - Failed to Read Network Key

What can be the problem? I had the same problem with version 0.11.1 too.

I am currently trying to setup zwave. Without luck. It's working a little bit.
try adding a network key in:
/usr/local/share/python-openzwave/config/options.xml
there is a tag
<Option name="NetworkKey" value="...
I've also excluded all devices before. Got the Error back when I tried to include a device which also supports security class.

I had this issue too, and traced it back to the method you use for adding to the network. If you add a device that support security class with normal add it will fail like this. You should add the device with secure mode. Then everything should work fine.

Solution was provided by @turbokongen for this issue.

Why does home-assistant not work with secure devices added as normal devices? Other programs work fine with it.
And even if this is not going to be supported, could the error message perhaps get a bit more clear?


(docs): Add inline left join examples

i was doing some searching today on complex joins and this was really helpful!

LGTM!

Not sure this is worth the maintenance

Let me know if this can/will go forward then I'll add the generated SQL and adapt all other query builders with left joins.

Yeah the examples were left out on purpose to reduce copy-pasta. There's a link to the innerJoin docs from all other join methods. The API is exactly the same for all join methods. Copy-pasting the examples to all methods doesn't make sense to me. They will diverge.

Should we make it more clear that you can find more documentation behind the link? Something like For more info see the innerJoin docs?


Make yellow clues prefer to match on the same word.

Currently if you have the same letter in an incorrect position in both of your guess words and there is only one in the answer, the horizontal word's letter will get the yellow clue as it is evaluated first. However, this is just an implementation detail, and it would be nicer if the yellow clue was given to the word containing that letter. E.g. assuming you guessed
  W
CROSS
  R
  D

If there was only an R in the second word, it should give the yellow clue to the R in WORD, rather than the one in CROSS.

Similar issue: if there is a single instance of a given letter across both words, and the player guesses two words that each contain this letter once, both instances show as yellow, leading to an incorrect inference that the word pair contains two of this letter.


Add tslib to dependencies

Ref https://github.com/dmtrKovalenko/material-ui-pickers/issues/718
/cc @dmtrKovalenko

@dmtrKovalenko Could you make a release with this fix?

Ok I will release a patch today


Seamless poster transitions: configure model to fit desired CSS pixel rect

One of my favourite 3D examples on the web is from Panic: https://panic.com/transmit/

It's the perfect progressive enhancement. It starts off as an image of the truck, and then when the 3D model is loaded it swaps out the image for the 3D and you don't notice it at all. I'd like to explore what that would look like for model-viewer.
The main issue I see is with supporting different aspect ratios. The aspect ratio can make it tricky to line the poster up seamlessly with the rendered model.
@cdata suggests that it is theoretically possible with a combination of "background-size: cover" and "camera-orbit". You would need to adjust the "camera-orbit" attribute to ensure that the model is the same pixel size regardless of aspect ratio.
Let's see if it's possible to add a public method to make adjusting the camera orbit based on desired CSS pixel size a thing.

cc @elalish

I like this idea, though I'm not entirely following the method of solution. I could imaging that you take a screenshot of the render on a transparent background with a tight-ish bounding box, then place and scale that image appropriately within the runtime-sized model-viewer box for alignment, while rendering the background behind it normally. I think we could automatically determine the placement values based on the width and height of the input image, since that would match the model-viewer state when it was captured.

@cdata I would love to include this for 1.0. I understand if it doesn't make the cut, but it seems doable.

Would be great to have this included as a milestone before 1.0 as we'll be relying heavily on it.


Fix typo

Fix typo in SSH fault

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://66c7435f2680382bec8e0c31--harness-developer.netlify.app


Small crafted files take over 10 seconds to decode

These files were found by cargo-fuzz. They take a disproportionately long time to decode, up to 10 seconds for files of under 2 kilobytes. This slows down fuzzing, and there could be some DoS potential here.
jxl-long-decode-times.tar.gz
Tested on commit 9704179c079c73fdba06fd138426a21e02158db5

The root cause of slow decoding is large MA trees (#170, mitigrated by #173). jxl-oxide CLI exits immediately with error when I set --approx-memory-limit, so it might be a problem with fuzzing harness. I'll take a look later today.

It turns out that MA tree limit (proportional to the number of decoded samples) is applied too late. I moved those checks into MA tree decoding in #180.


Call to a member function relationLoaded() on null

Laravel Version: 5.6
Voyager Version:1.1
PHP Version:7.2.4
Database Driver & Version:

Description:Call to a member function relationLoaded() on null
Steps To Reproduce:

Please search the repo before opening issues.  This has been asked and answered multiple times already.


Implement @ToolMemoryId support

Currently, the io.quarkiverse.langchain4j.runtime.tool.QuarkusToolExecutor#execute does not include the memoryId in the arguments for a tool - it should include it if a parameter is annotated with @ToolMemoryId. The dev.langchain4j.agent.tool.DefaultToolExecutor does this, I think the code could be copied/adapted pretty easily.

Thanks for reporting!
I'll check this out this week if all goes well


Change the order of PostProcessors (Or add another PostProcessor stage)

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

[X] I understand that I will be blocked if I intentionally remove or skip any mandatory* field

Checklist

[X] I'm requesting a feature unrelated to a specific site
[X] I've looked through the README
[X] I've verified that I have updated yt-dlp to nightly or master (update instructions)
[X] I've searched known issues and the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue

Provide a description that is worded well enough to be understood
Looking at the code,
https://github.com/yt-dlp/yt-dlp/blob/111b61ddef305584d45a48e7b7c73ffcedf062a2/yt_dlp/__init__.py#L598-L725
Custom post-processors are set to execute before built-in post-processors.
I wonder if it would be possible (aka, not a massive breaking change) if the user supplied post-processors could execute after the built-ins. If not possible, would a new when stage (?) be acceptable e.g. post_post_process or before_move?
For example, here:
https://github.com/yt-dlp/yt-dlp/blob/111b61ddef305584d45a48e7b7c73ffcedf062a2/yt_dlp/YoutubeDL.py#L3715-L3722
Why?
So right now, removing segments with sponsorblock causes artifacts in the resulting video file (e.g. sound with no video), as mentioned in https://github.com/yt-dlp/yt-dlp/issues/3766 (FAQ). The ultimate fix is to recreate valid keyframes (hence, --force-keyframes-at-cuts), if any keyframes were in the removed segments (if I understand FFmpeg's docs correctly).
Since yt-dlp can't possibly know this, yt-dlp adds the ModifyChapters post-processor to re-encode the video with keyframes at the timestamps known to yt-dlp.
While the solution is completely valid, it can double post-processing time, if say, you are using the FFmpegCopyStream post-processor to convert the downloaded video into a usable format (say, downloading YouTube for Plex, which requires videos using the mainline profile, which requires both audio and video re-encoding, in most-cases). Given the order mentioned above, the cutting occurs after converting to the final format - causing a double encode.
To my knowledge, re-encoding after removing segments is all that's needed for this use-case (ignoring the use-case of ensuring chapters are along keyframe "lines"), as FFmpeg should generate keyframes as appropriate (to avoid the generation loss problem). Preferably, this could be done in a single pass - on the file with segments removed, aka, running the FFmpegCopyStream post-processor after cutting.
Provide verbose output that clearly demonstrates the problem

[ ] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[ ] If using API, add 'verbose': True to YoutubeDL params instead
[ ] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
No response

I'm not a Python dev, but I can make a PR, if ether of these suggestions are acceptable:

Change the order of the post-processors to execute user supplied PP's after built-in PP's.
Or, add a new PP stage e.g. post_post_process or before_move, that executes after the normal post_process stage.

Both paths seem rather straight forward - although, I might need help in writing the tests.


Metacello / Iceberg locks up when requesting credentials

Metacello and Iceberg can go in to a deadlock if credentials are not pre-set and the operation is initiated from the UI process.
This is because libgit takes a callback to get the credentials (LGitCredAcquireCallback).  In current Pharo 8 the callback may prompt the user for their username and password through a dialog, however because the UI process is waiting for the original git call to return the dialog never opens, leaving the system deadlocked.

We have a workaround for now. The ultimate solution requires notifications support in the native windows, and will be addressed when we will work on that.


Add rbacApiVersion and use separate resources instead of a list

Closes #2602
See #2602 for more info
Signed-off-by: Doug Davis<EMAIL_ADDRESS>ping @kramvan1

/test pull-service-catalog-xbuild

/test pull-service-catalog-integration

/test pull-service-catalog-integration

/approve

/lgtm


uses deprecated registry.terraform.io/hashicorp/template v2.2.0

Describe the Bug
The yaml config module makes use of the template_file data resource instead of the templatefile function. Since there is no registry.terraform.io/hashicorp/template v2.2.0 provider for Apple M1 systems any attempts to use this module will fail.
Initializing provider plugins...
- Finding latest version of hashicorp/template...
╷
│ Error: Incompatible provider version
│
│ Provider registry.terraform.io/hashicorp/template v2.2.0 does not have a package available for your current platform, darwin_arm64.
│
│ Provider releases are separate from Terraform CLI releases, so not all providers are available for all platforms. Other versions of this provider may have different platforms supported.
╵

Expected Behavior
Module initializes without the need for the deprecated registry.terraform.io/hashicorp/template provider.
Steps to Reproduce
Steps to reproduce the behavior:

Perform terraform init on a MacBook with a M1 chip (darwin_arm64)
Error above will be displayed.

Environment (please complete the following information):
Anything that will help us triage the bug will help. Here are some ideas:

OS: macOS
Version: 13
Chip: Apple M1 Pro

See https://github.com/hashicorp/terraform/issues/30055

This was closed in #25


fix: Fix generating dependencies for release-please commits

The previous code tried to walk to the previous commit; the new code uses the parent of the HEAD commit instead.

OwlBot post-processor ran successfully, which should mean the fix works :)


Update source link to rrdtool

Hi.
I actually try to use your meta layer (dunfell release) for building image and take error with fetchind rrdtools-1.0.50.
I checked link in your recepy:
SRC_URI = "http://oss.oetiker.ch/rrdtool/pub/rrdtool-1.0.x/rrdtool-${PV}.tar.gz"
Actual link is:
https://oss.oetiker.ch/rrdtool/pub/**archive**/rrdtool-1.0.x/
So they move old releases to archieve folder.

Thank you. Fixed it.


Add test summaries to the PR in a comment on completion

It would be nice to have an opt-in option that would comment a summary of the tests to the PR (or otherwise would just paste a link to the test run on datadog).
This would make it easier to navigate from a PR to its test summary

Hi @dmattia thank you for the suggestion. Adding GitHub PR comments and check runs support is something we have in the roadmap. It is my hope that we can get this done in the near future.

I am closing this as we are working on this separately and it will be part of the Datadog product but not the agent action.


Migrate java module to interop

Purpose

Describe the problems, issues, or needs driving this feature/fix and include links to related issues.

Fixes #20178
Approach

Describe how you are implementing the solutions along with the design details.

Samples

Provide high-level details about the samples related to this feature.

Remarks

List any other known issues, related PRs, TODO items, or any other notes related to the PR.

Check List

[ ] Read the Contributing Guide
[ ] Updated Change Log
[ ] Checked Tooling Support (#)
[ ] Added necessary tests

[ ] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

Shall we deprecate 'fromString' ?

I am closing this fix as there are few test failures with the fix. we will fix this properly later.


feat(searchBox): ability to be non-instant

If instant is set to false, the search is only triggered if ENTER is pressed.

Added some tests on even registration
Removed an outdated version of the searchbox that was at the root level of the widgets/ directory.
Updated jsdoc

Fix #458

I like the feature but I am unsure about instant API option. The library is called "instantsearch" but we have an option to disable the "instant": it was my first reaction.

Along with this option we would need a submit button to be generated (as an option too), because otherwise it makes sense only on desktop

I like the feature but I am unsure about instant API option. The library is called "instantsearch" but we have an option to disable the "instant": it was my first reaction.

Any idea what could be a better naming? searchAsYouType: true/false was not nice :/

Along with this option we would need a submit button to be generated (as an option too), because otherwise it makes sense only on desktop

Hmmm, it is the goal responsibility of the searchbox? Because having a submit button means you have a wrapping <form> as well. Good point but not sure how to fix it :/

Hmmm, it is the goal responsibility of the searchbox? Because having a submit button means you have a wrapping  as well. Good point but not sure how to fix it :/

Well we can have a wrapping form or not, that's not mandatory. You can add just a button type submit and handle the submit on it. Or we can have a wrapping form, we have one for example in the priceRanges widget.
But that's for later

Any idea what could be a better naming? searchAsYouType: true/false was not nice :/

searchOnEnterKeyPressOnly exhaustive he!

Updated @vvo

Sorry I'm late, but as @vvo asked, why do we need that? Shouldn't we push the instant search as the only real way to do intuitive search? I feel like integration this small option (which is in opposition with the main goal of the library) will create an large array of side effects.


doc/user/migrating-existing-apis.md: document API version migrations

Description of the change: From #1541:

A document describing best practices, tips, gotchas, and any other helpful information about making various API changes in an existing SDK project.

Note: for Go projects only. Ansible and Helm have TODO's.
Motivation for the change: see #1541.
Closes #1541
PTAL @awgreene @dmesser

Sorry for the hold up. I'll finish my review on this hopefully in a day or so.

/test e2e-aws-go
/test e2e-aws-ansible

/test e2e-aws-ansible

/test e2e-aws-ansible


[BUG] Guthix Rest Make-All has incorrect timing

Describe the bug
The action time for making Guthix Rests through the Make-All option is a 3 tick action, while the action progress is treating it as a 1 tick action. Having skimmed the code, it looks like the plugin was never updated to handle the new Make-All capabilities, and is just using the timing for Cup of Water + One of the herbs. I'm assuming it'd be difficult to be able to tell whether the user was making guthix rests via the "make-all" option, or if they were doing it the old method, but I'm not sure how useful action progress ever was for the old method, given you can only do 1 action at a time anyway.
To Reproduce
Steps to reproduce the behavior:

Grab 10 guams, 5 harralanders, 5 marrentills, and 5 cups of hot water from bank.
Use a cup on any herb.
Click to make all.
Notice the action progress bar completes much earlier than the actual inventory of guthix rests.

Expected behavior
I'd expect the plugin to treat the action of creating all guthix rests to count 3 ticks, rather than 1 tick per cup.
Screenshots
Here is a recording, with the visual metronome plugin on to show exactly how many ticks are passing between making each rest.
https://github.com/user-attachments/assets/a1d89df6-95e1-4b08-a56c-e61a6c722d01

It's indeed coded as a 1-tick action. The make-all detection should be implemented
https://github.com/guillaume009/runelite-plugin-action-progress/blob/6ba02959d00fafe83a69073485be216b07942f1c/src/main/java/com/github/calebwhiting/runelite/data/Herblore.java#L56-L59


Can't save a material to the preset library

Can not save a material to the preset library
Have attempted
• write permission
• different places for "RenderManAssetLibrary"
Please see the attached image with the error report.

Also, I do have the latest addon version.

So wierd. Have had issues with this for some time. Tried another Blender version 2.78 and then 2.78c. Suddeny it works.

Ok, I see that simple networks will be saved but not complex ones.

gdd_a.blend.zip

export_network_to_presets_issue.blend.zip

Fixed.  The issue was the float -> color connections.


Unable to send second email on the same email

Whenever I try to send another email without killing the application it don't send another email meanwhile it shows some error message like this
D/EGL_emulation(25261): app_time_stats: avg=14839.96ms min=3.09ms max=252025.66ms count=17
I/flutter (25261): ClientException: HTTP request failed. Client is already closed., uri=https://api.emailjs.com/api/v1.0/email/send

Hey, are you using our SDK?

Yeah I am using the emailjs sdk meanwhile I found a way to send emails
again after sending one with the init method
On Mon, Jul 10, 2023, 7:48 AM xr0master @.***> wrote:

Hey, are you using our SDK?
—
Reply to this email directly, view it on GitHub
https://github.com/emailjs-com/emailjs-flutter/issues/1#issuecomment-1627994047,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ARSIFYMRL2NDGR36YAF6M23XPNUQJANCNFSM6AAAAAA2DUHL5E
.
You are receiving this because you authored the thread.Message ID:
@.***>

Yes, this is indeed a bug, thanks.

@Muneeb-Ahmed2002 A new version 1.3.0 should be good now. Please try.

I have 2 email, email A and email B
after upgrading to version 1.3.0, I'm able to send the mail content to multiple email on first try but when I send for the 2nd time to multiple email, only the first email (email A) is able to receive the mail content, but the other email (email B) doesn't receive the mail content.
The EmailJSResponseStatus returns 200.
Please help out.

@puahhhh Please try to debug and find the issue. If it's into SDK, let us know.

i have the same issue when trying to send another email after sending one.
I/flutter ( 7445): HTTP request failed. Client is already closed.

We will investigate it, thank you.

Unfortunately, we cannot reproduce this issue. Our example in the project works without errors.
What is the flutter version? Platform?
Can you share the code where you are calling the EmailJS.send.

I use flutter 3 and android platform for debugging.
Here's the code :
`try {
await emailJs.EmailJS.send(
'someServiceCode',
'someTemplateCode',
templateParams(
namaPembeli: userData.username!,
namaPenjual: cartItems.products!.profiles!.username,
emailPembeli: userData.email,
emailPenjual: cartItems.products!.profiles!.email,
),
const emailJs.Options(
publicKey: 'somePublicKey',
privateKey: 'somePrivateKey',
),
);
print('SUCCESS SEND EMAIL');
} catch (e) {
print(e.toString());
}
  if (cartItems.products!.category == 'Produk Fisik') {
    try {
      await emailJs.EmailJS.send(
        'service_mzp1eji',
        'template_ajcpfnk',
        adminShipmentTemplateParams(
          namaPembeli: userData.username!,
          namaPenjual: cartItems.products!.profiles!.username,
          emailPembeli: userData.email,
          emailAdmin<EMAIL_ADDRESS>        ),
        const emailJs.Options(
          publicKey: 'somePublicKey',
          privateKey: 'somePrivateKey',
        ),
      );
      print('SUCCESS SEND EMAIL TO ADMIN');
    } catch (e) {
      print(e.toString());
    }
  }
}`

I got an error that client have been closed in the IF section. Assume that the IF section are meet the criteria to run.

I use flutter 3.7.11, with android for debugging. Here's the code that i run :

@geniusvale thank you, we will check it on Andriod.
Do you have the same issue in the web mode? (on Chrome)

I don't develop on web and never tried it, so i can't confirm that issues

We tested our example on Android and could not reproduce the issue.
Please be sure you are using new versions of SDK and Flutter.

I still have this issue.
'HTTP request failed. Client is already closed.'
I am using it on web => Chrome.

@cc-bilal can you provide a small project where we can see the issue?


Debug entry during hardware vectoring.

Specification says that:

In CLIC mode, the dpc CSR additionally may hold the faulting address if breakpoints are allowed to trap on the table fetch during hardware vectoring. If breakpoints are allowed to trap on the table read, dret should honor xinhv.

However, I don't think xinhv can be used for this purpose because as well as being set during the implicit vector table fetch  it will also be set if debug mode is entered while in the middle of executing a fault handler on a failed vector table fetch. In this situation xinhv will be set but dpc will not be pointing to a trap vector table entry.
If debug mode can be entered with dpc pointing to a trap vector table entry I think the debug state needs its own inhv bit to indicate that.

In addition, this section should clarify the behaviour when single stepping, in particular, unless a debug inhv bit is added to the architecture, when stepping through an interrupt vectoring operation it is not possible to stop between the return and the vector table fetch. However, if the vector table fetch traps this violates the  Debug Spec's definition of a single step:  "execute a single instruction or trap and then re-enter Debug Mode"


Not usable in NextJs Typescript after building

Works as expected when running next dev but after building and running next start, it dosent work at all.
"@use-gesture/react": "10.2.27",
"next": "13.4.12",
"react": "18.2.0",
"react-dom": "18.2.0",

Typescript project

I have the same issue

I won't look at this until it follows the proper issue template.

Yep, sure. I test a few things and come back by tmr!

Created this issue: https://github.com/pmndrs/use-gesture/issues/618#issue-1859155914
Tell me if there are any info missing, thanks a lot for looking at it ✨


[added] a --babel-config CLI option - a configurable alternative to .babelrc

The --babel-config CLI option lets you set a specific configuration file to use for babel.
It's an alternative to .babelrc, but it can be located anywhere, so one does not have to rely on Babel searching the file system for it.
Supports .yaml, .yml, .json and .js files.

Just had one comment @thealjey

@jmcriffey please check now

Cool, LGTM. Thanks!

Is there an example / documentation of how we can use this? Is it an option that we can add to the istanbul.yml? I can't seem to get it working with something like: ./node_modules/.bin/babel-istanbul --babel-config conf/babel.json

@danjford --babel-config is a CLI option, you specify it like so:
--babel-config conf/babel.json
or
--babel-config conf/babel.yml
or
--babel-config conf/babel.js
The configuration object is passed directly to Babel.
What you need to check is the version of Babel that babel-istanbul uses and whether or not the configuration you provide to it is supported by that version of Babel.
What do you mean it's not working, does it not do anything or do you see an error?
What are the contents of conf/babel.json?

I tried using the command ./node_modules/.bin/babel-istanbul --babel-config conf/babel.json ./node_modules/mocha/bin/_mocha to cover my mocha tests. In this case it gave an error saying that conf/babel.json wasn't a command.

My babel.json contained:
{
  "presets": [ "es2015" ]
}

The reason I was trying to change where the babel configurations were due to having a conflict with another library that didn't want "presets": [ "es2015" ] in the .babelrc.
As I couldn't get the --babel-config option working with babel-istanbul I've just put a work around into the .babelrc so that it doesn't conflict with the other library.

@danjford
duh, try this
babel-istanbul cover --babel-config conf/babel.json ./node_modules/mocha/bin/_mocha

Running ./node_modules/.bin/babel-istanbul would be the same as babel-istanbul except one is installed to a project and the other is installed globally :p
I'm looking back and I've tried replicating the issue but for some the command that I mentioned in the previous comment works now... so never mind lol :+1:

@danjford, that's not the point, you noticed that there's babel-istanbul instead of ./node_modules/.bin/babel-istanbul (which is basically the same fking thing), but you didn't notice that cover was right after it.
That's the reason :laughing:

Ah my mistake, I think it's actually more just that I left out the = as it would work as ./node_modules/.bin/babel-istanbul --babel-config=conf/babel.json cover ./node_modules/mocha/bin/_mocha.


Support static arrays with reverse Enzyme

Codecov Report
Attention: Patch coverage is 0% with 41 lines in your changes missing coverage. Please review.

Project coverage is 11.19%. Comparing base (94f9bc5) to head (7285690).

Files with missing lines
Patch %
Lines

...ifferentiationInterfaceEnzymeExt/reverse_onearg.jl
0.00%
34 Missing :warning:

...ace/ext/DifferentiationInterfaceEnzymeExt/utils.jl
0.00%
6 Missing :warning:

DifferentiationInterface/test/Back/Enzyme/test.jl
0.00%
1 Missing :warning:

:exclamation:  There is a different number of reports uploaded between BASE (94f9bc5) and HEAD (7285690). Click for more details.
HEAD has 40 uploads less than BASE

Flag
BASE (94f9bc5)
HEAD (7285690)

DI
33
1

DIT
8
0

Additional details and impacted files
@@             Coverage Diff             @@
##             main     #585       +/-   ##
===========================================
- Coverage   97.54%   11.19%   -86.36%
===========================================
  Files         110       76       -34
  Lines        5222     3235     -1987
===========================================
- Hits         5094      362     -4732
- Misses        128     2873     +2745

Flag
Coverage Δ

DI
11.19% <0.00%> (-87.41%)
:arrow_down:

DIT
?

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

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


Integration with Circuit Breaker

Are there any guidelines or recommendations regarding integration with Circuit Breaker?
For example, the number of database connections reaches a limit and new connections repeatedly succeed and fail. Even though the service is unstable, the health checks sometimes return successes, causing the load balancer to send traffic to the unstable instance. Therefore, I would like to integrate circuit breaker to make the health check fail until it stabilizes.
In an article written four years ago, it is stated as follows, but I cannot find what has happened since then.

We’re working with the authors of Polly to try and integrate health checks for Polly’s circuit breakers.

Thanks in advance.

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

ID: c0cbb094-29e9-740a-9fd6-db372641d753
Version Independent ID: 52d8efc9-a336-71d9-6d90-98173cbf6bf4
Content: Health checks in ASP.NET Core
Content Source: aspnetcore/host-and-deploy/health-checks.md
Product: aspnet-core
Technology: aspnetcore-hostdeploy
GitHub Login: @Rick-Anderson
Microsoft Alias: riande

@JuergenGutsch @glennc

@Rick-Anderson Thanks. If I should not combine circuit breakers with health checks in ASP.NET Core, please tell me so.
Indeed, as in the example I mentioned, excluding instances with failed health checks from the load balancer from the traffic forwarding destination could affect all instances in the event of a dependent service failure. Under some conditions, circuit breakers may be best used exclusively in the data plane, not in health checks.
If this is not recommended, developers would be happy to see a reason in the documentation.

@torumakabe I'm so sorry, I missed that mention completely :-(
I'm sure there is no recommendation not to implement circuit breakers in health checks.
You are right, some kind of resilience and transient-fault-handling should be installed every time you connect to a resource that might not accessible. On the other hand, this implementation is completely decoupled from health checks.  This means, there is no reason not to implement circuit breakers.
To try to use Polly’s circuit breakers makes absolute sense. The only difficulty is registering an instance CircuitBreakerPolicy as a singleton within the application that hosts the health checks because the checks are executed on demand or periodically and the state of the circuit breakers needs to be tracked over multiple scopes. See: https://github.com/App-vNext/Polly/wiki/Circuit-Breaker
@Rick-Anderson I'm not sure if this topic needs a section in the docs because it is really specific and the sample will be more complex than the usual topics. If you like to have a doc about it, I would be happy to create one or at least I write a blog post and you add a link to it instead of the comment mentioned above.

@Rick-Anderson I'm not sure if this topic needs a section in the docs because it is really specific and the sample will be more complex than the usual topics. If you like to have a doc about it, I would be happy to create one or at least I write a blog post and you add a link to it instead of the comment mentioned above.

@JuergenGutsch  Thanks, I think this issue is all we need for now.

@JuergenGutsch @Rick-Anderson Thanks!


Add recast to downloadable playgrounds

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


WIP: Add types to facilitate release of memory via finalizers.

I started to implement the suggestion @stevengj gave in #26. There are new types KinsolHandle, CVodeHandle, IdaHandle and NVector with finalizers. I hope this doesn't break any existing code which doesn't use the simplified interface? IDAS and CVodes are not covered yet.

Bump. Can we get this merged?

Unless I forgot anything to make NVector <: DenseVector this should be it (apart from squashing) ... I am not sure what we should do with asarray. Whenever we get a N_Vector from Sundials we might need it since we don't want those vectors to be released by the finalizer, right?

Coverage decreased (-8.86%) to 55.91% when pulling 56a16b9be5874b3f50133c8affaf2728a1554289 on acroy:types into 053a3a775b21724546768fbc4f83f6270a3c2b7d on JuliaLang:master.

Coverage decreased (-8.86%) to 55.91% when pulling 56a16b9be5874b3f50133c8affaf2728a1554289 on acroy:types into 053a3a775b21724546768fbc4f83f6270a3c2b7d on JuliaLang:master.

Regarding asarray, I think this is something that the user should never see, nor should the user ever have to call cfunction.   See now the NLopt wrapper does it: the user's callback function should be wrapped by an internal Sundials callback function (with cfunction called on the latter), and the wrapper callback should translate arrays between N_Vector and Julia arrays, with only Julia arrays passed to the user's callback.

I got two questions concerning the new type NVector.

Looking at CVodeInit(mem, cfunction(cvodefun, Int32, (RealType, N_Vector, N_Vector, Function)), t[1], NVector(y0)) we keep no reference to the newly created NVector(y0). When will the finalizer free the memory and how do we assure this does not happen before we finish solving?
How do we handle Vectors of NVector, as e.g. needed in CVodeSensInit, which takes (N Vector *) as an argument?

@axsk : Since we keep a reference to y0 in the NVector, memory will be freed when y0 is not needed anymore. For your second question, I would think that you create a bunch of (Julia) vectors, turn them into NVectors and pass an array of pointers (stored in NVector) to the function.

Is this still needed now that #67 is merged?

@ChrisRackauckas #67 was derived from this one, so I guess it could be closed. I would not delete the branch, however, as there are some code comments and discussions that could be useful for fine tuning the master.

Got it. Closing.


Avoid running the security pipeline on every PR

The security pipeline defined in azure_pipelines/code-security-analysis.yml is scheduled to run weekly as required by Microsoft policy. However, it also seems to run on all PRs, which is unnecessary. This PR changes the pipeline's configuration to only do the scheduled runs and not run as part of CI.

I don't see any problems in this PR, but I also have no way to check that this accomplishes what we intend it to accomplish. If you have looked into it then I will trust you Kris!

I can't be 100% sure, but if you look at the checks ran on this PR, the security pipeline (called microsoft.syntheseus) is no longer there, while it was there on all previous PRs, so hopefully my changes are doing the right thing 🙂


Add CRT auth package

This pulls in CRT auth abstractions from the develop branch in order to have a starting to point to pull in future updates related to CRT auth. Note the code does not actually use any of the CRT auth abstractions when making API requests. The main difference in porting the package is that because CRT is a required dependency there is no branching logic for conditionally using CRT auth. Eventually, we should remove the pure-Python sigv4 auth abstractions and completely rely on the CRT auth abstractions for sigv4 signing.
For reference the changes should follow closely to the original PR that added CRT auth to the develop branch: https://github.com/boto/botocore/pull/2350

Codecov Report

Merging #2417 (13fecf7) into v2 (c2efcec) will decrease coverage by 0.00%.
The diff coverage is 95.65%.

@@            Coverage Diff             @@
##               v2    #2417      +/-   ##
==========================================
- Coverage   97.62%   97.61%   -0.01%
==========================================
  Files          56       57       +1
  Lines       10685    10808     +123
==========================================
+ Hits        10431    10550     +119
- Misses        254      258       +4

Impacted Files
Coverage Δ

botocore/auth.py
97.95% <90.90%> (-0.03%)
:arrow_down:

botocore/crt/auth.py
96.87% <96.87%> (ø)

botocore/httpsession.py
92.55% <0.00%> (-0.47%)
:arrow_down:

botocore/utils.py
96.81% <0.00%> (-0.16%)
:arrow_down:

botocore/credentials.py
98.55% <0.00%> (-0.11%)
:arrow_down:

botocore/compat.py
69.88% <0.00%> (+2.27%)
:arrow_up:

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 c2efcec...13fecf7. Read the comment docs.


Failed to execute operation: No such file or directory

This error its a new fresh installation using the command:
wget https://raw.githubusercontent.com/freeacs/freeacs/master/scripts/install.sh -v -O install.sh && chmod +x install.sh && sudo ./install.sh;
This happen on Ubuntu 16.04
core installation start
Failed to execute operation: No such file or directory
Selecting previously unselected package freeacs-core.
(Reading database ... 93800 files and directories currently installed.)
Preparing to unpack freeacs-core_2.0.2_all.deb ...
Unpacking freeacs-core (2.0.2) ...
Setting up freeacs-core (2.0.2) ...
Creating system group: freeacs
Creating system user: freeacs in freeacs with freeacs-core daemon-user and shell /bin/false
Created symlink from /etc/systemd/system/multi-user.target.wants/freeacs-core.service to /lib/systemd/system/freeacs-core.service.
core installation complete
stun installation start
Failed to execute operation: No such file or directory
Selecting previously unselected package freeacs-stun.
(Reading database ... 93938 files and directories currently installed.)
Preparing to unpack freeacs-stun_2.0.2_all.deb ...
Unpacking freeacs-stun (2.0.2) ...
Setting up freeacs-stun (2.0.2) ...
Created symlink from /etc/systemd/system/multi-user.target.wants/freeacs-stun.service to /lib/systemd/system/freeacs-stun.service.
stun installation complete
tr069 installation start
Failed to execute operation: No such file or directory
Selecting previously unselected package freeacs-tr069.
(Reading database ... 94072 files and directories currently installed.)
Preparing to unpack freeacs-tr069_2.0.2_all.deb ...
Unpacking freeacs-tr069 (2.0.2) ...
Setting up freeacs-tr069 (2.0.2) ...
Created symlink from /etc/systemd/system/multi-user.target.wants/freeacs-tr069.service to /lib/systemd/system/freeacs-tr069.service.
tr069 installation complete
syslog installation start
Failed to execute operation: No such file or directory
Selecting previously unselected package freeacs-syslog.
(Reading database ... 94207 files and directories currently installed.)
Preparing to unpack freeacs-syslog_2.0.2_all.deb ...
Unpacking freeacs-syslog (2.0.2) ...
Setting up freeacs-syslog (2.0.2) ...
Created symlink from /etc/systemd/system/multi-user.target.wants/freeacs-syslog.service to /lib/systemd/system/freeacs-syslog.service.
syslog installation complete
web installation start
Failed to execute operation: No such file or directory
Selecting previously unselected package freeacs-web.
(Reading database ... 94341 files and directories currently installed.)
Preparing to unpack freeacs-web_2.0.2_all.deb ...
Unpacking freeacs-web (2.0.2) ...
Selecting previously unselected package freeacs-webservice.
Preparing to unpack freeacs-webservice_2.0.2_all.deb ...
Unpacking freeacs-webservice (2.0.2) ...
Setting up freeacs-web (2.0.2) ...
Created symlink from /etc/systemd/system/multi-user.target.wants/freeacs-web.service to /lib/systemd/system/freeacs-web.service.
Setting up freeacs-webservice (2.0.2) ...
Created symlink from /etc/systemd/system/multi-user.target.wants/freeacs-webservice.service to /lib/systemd/system/freeacs-webservice.service.
web installation complete
webservice installation start
Removed symlink /etc/systemd/system/multi-user.target.wants/freeacs-webservice.service.
(Reading database ... 94623 files and directories currently installed.)
Preparing to unpack freeacs-webservice_2.0.2_all.deb ...
Unpacking freeacs-webservice (2.0.2) over (2.0.2) ...
Setting up freeacs-webservice (2.0.2) ...
Created symlink from /etc/systemd/system/multi-user.target.wants/freeacs-webservice.service to /lib/systemd/system/freeacs-webservice.service.
webservice installation complete
shell installation start
Failed to execute operation: No such file or directory
Selecting previously unselected package freeacs-shell.
(Reading database ... 94623 files and directories currently installed.)
Preparing to unpack freeacs-shell_2.0.2_all.deb ...
Unpacking freeacs-shell (2.0.2) ...
Setting up freeacs-shell (2.0.2) ...
Failed to restart freeacs-shell.service: Unit freeacs-shell.service not found.
shell installation complete
monitor installation start
Failed to execute operation: No such file or directory
Selecting previously unselected package freeacs-monitor.
(Reading database ... 94688 files and directories currently installed.)
Preparing to unpack freeacs-monitor_2.0.2_all.deb ...
Unpacking freeacs-monitor (2.0.2) ...
Setting up freeacs-monitor (2.0.2) ...
Created symlink from /etc/systemd/system/multi-user.target.wants/freeacs-monitor.service to /lib/systemd/system/freeacs-monitor.service.
monitor installation complete
dpkg-query: no packages found matching nginx
Reading package lists... Done
Building dependency tree
Reading state information... Done

Is something failing? It just means that the service is not already installed. Its trying to disable a service that does not exist

Good day @jarlah
Nothing fails, but the message caught my attention and that is why I post it.
Thanks for the clarification.

Closing.