random java.lang.NullPointerException: networkMode was not specified

In our ci builds we see random failures every other build saying a container can't be started. Our scenario is the following:

We have a container running our application
We have another container (BrowserWebDriverContainer) to run chrome
We have specified a network for both
The chrome container depends on the application container.

The code looks like this:
private void createContainers() {
	Network network = Network.newNetwork();
	GenericContainer<?> applicationContainer = new ApplicationContainer();
	applicationContainer.withNetwork(network).withNetworkAliases("app").withExposedPorts(8080, 8443);

	BrowserWebDriverContainer<?> chrome = new BrowserWebDriverContainer<>(
			DockerImageName.parse("selenium/standalone-chrome-debug:3.141.59-20210713")
					.asCompatibleSubstituteFor("selenium/standalone-chrome-debug"));
	chrome.withNetwork(network).withRecordingMode(VncRecordingMode.SKIP, null).dependsOn(applicationContainer);

	chrome.start();
}

In this scenario, ApplicationContainer extends GenericContainer<ApplicationContainer>. The code works most of the time. As said, every now and then we see the following error which makes the test run fail.
We are using testcontainers 1.15.3 right now.
java.lang.RuntimeException: com.github.dockerjava.zerodep.shaded.org.apache.hc.core5.http.NoHttpResponseException: localhost:2375 failed to respond
	at com.github.dockerjava.zerodep.ApacheDockerHttpClientImpl.execute(ApacheDockerHttpClientImpl.java:165) ~[docker-java-transport-zerodep-3.2.8.jar:?]
	at com.github.dockerjava.zerodep.ZerodepDockerHttpClient.execute(ZerodepDockerHttpClient.java:8) ~[docker-java-transport-zerodep-3.2.8.jar:?]
	at org.testcontainers.shaded.com.github.dockerjava.core.DefaultInvocationBuilder.execute(DefaultInvocationBuilder.java:228) ~[testcontainers-1.15.3.jar:?]
	at org.testcontainers.shaded.com.github.dockerjava.core.DefaultInvocationBuilder.post(DefaultInvocationBuilder.java:124) ~[testcontainers-1.15.3.jar:?]
	at org.testcontainers.shaded.com.github.dockerjava.core.exec.CreateNetworkCmdExec.execute(CreateNetworkCmdExec.java:27) ~[testcontainers-1.15.3.jar:?]
	at org.testcontainers.shaded.com.github.dockerjava.core.exec.CreateNetworkCmdExec.execute(CreateNetworkCmdExec.java:12) ~[testcontainers-1.15.3.jar:?]
	at org.testcontainers.shaded.com.github.dockerjava.core.exec.AbstrSyncDockerCmdExec.exec(AbstrSyncDockerCmdExec.java:21) ~[testcontainers-1.15.3.jar:?]
	at org.testcontainers.shaded.com.github.dockerjava.core.command.AbstrDockerCmd.exec(AbstrDockerCmd.java:35) ~[testcontainers-1.15.3.jar:?]
	at org.testcontainers.containers.Network$NetworkImpl.create(Network.java:92) ~[testcontainers-1.15.3.jar:?]
	at org.testcontainers.containers.Network$NetworkImpl.getId(Network.java:63) ~[testcontainers-1.15.3.jar:?]
	at org.testcontainers.containers.GenericContainer.applyConfiguration(GenericContainer.java:813) ~[testcontainers-1.15.3.jar:?]
	at org.testcontainers.containers.GenericContainer.tryStart(GenericContainer.java:359) ~[testcontainers-1.15.3.jar:?]
	at org.testcontainers.containers.GenericContainer.lambda$doStart$0(GenericContainer.java:325) ~[testcontainers-1.15.3.jar:?]
	at org.rnorth.ducttape.unreliables.Unreliables.retryUntilSuccess(Unreliables.java:81) ~[duct-tape-1.0.8.jar:?]
	at org.testcontainers.containers.GenericContainer.doStart(GenericContainer.java:323) ~[testcontainers-1.15.3.jar:?]
	at org.testcontainers.containers.GenericContainer.start(GenericContainer.java:311) ~[testcontainers-1.15.3.jar:?]
	at java.util.concurrent.CompletableFuture.uniRun(CompletableFuture.java:719) [?:1.8.0_281]
	at java.util.concurrent.CompletableFuture$UniRun.tryFire(CompletableFuture.java:701) [?:1.8.0_281]
	at java.util.concurrent.CompletableFuture$Completion.run(CompletableFuture.java:456) [?:1.8.0_281]
	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1149) [?:1.8.0_281]
	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624) [?:1.8.0_281]
	at java.lang.Thread.run(Thread.java:748) [?:1.8.0_281]
Caused by: com.github.dockerjava.zerodep.shaded.org.apache.hc.core5.http.NoHttpResponseException: localhost:2375 failed to respond
	at com.github.dockerjava.zerodep.shaded.org.apache.hc.core5.http.impl.io.DefaultHttpResponseParser.createConnectionClosedException(DefaultHttpResponseParser.java:87) ~[docker-java-transport-zerodep-3.2.8.jar:?]
	at com.github.dockerjava.zerodep.shaded.org.apache.hc.core5.http.impl.io.AbstractMessageParser.parse(AbstractMessageParser.java:243) ~[docker-java-transport-zerodep-3.2.8.jar:?]
	at com.github.dockerjava.zerodep.shaded.org.apache.hc.core5.http.impl.io.AbstractMessageParser.parse(AbstractMessageParser.java:53) ~[docker-java-transport-zerodep-3.2.8.jar:?]
	at com.github.dockerjava.zerodep.shaded.org.apache.hc.core5.http.impl.io.DefaultBHttpClientConnection.receiveResponseHeader(DefaultBHttpClientConnection.java:187) ~[docker-java-transport-zerodep-3.2.8.jar:?]
	at com.github.dockerjava.zerodep.shaded.org.apache.hc.core5.http.impl.io.HttpRequestExecutor.execute(HttpRequestExecutor.java:175) ~[docker-java-transport-zerodep-3.2.8.jar:?]
	at com.github.dockerjava.zerodep.HijackingHttpRequestExecutor.execute(HijackingHttpRequestExecutor.java:50) ~[docker-java-transport-zerodep-3.2.8.jar:?]
	at com.github.dockerjava.zerodep.shaded.org.apache.hc.core5.http.impl.io.HttpRequestExecutor.execute(HttpRequestExecutor.java:218) ~[docker-java-transport-zerodep-3.2.8.jar:?]
	at com.github.dockerjava.zerodep.shaded.org.apache.hc.client5.http.impl.io.PoolingHttpClientConnectionManager$InternalConnectionEndpoint.execute(PoolingHttpClientConnectionManager.java:596) ~[docker-java-transport-zerodep-3.2.8.jar:?]
	at com.github.dockerjava.zerodep.shaded.org.apache.hc.client5.http.impl.classic.InternalExecRuntime.execute(InternalExecRuntime.java:215) ~[docker-java-transport-zerodep-3.2.8.jar:?]
	at com.github.dockerjava.zerodep.shaded.org.apache.hc.client5.http.impl.classic.MainClientExec.execute(MainClientExec.java:107) ~[docker-java-transport-zerodep-3.2.8.jar:?]
	at com.github.dockerjava.zerodep.shaded.org.apache.hc.client5.http.impl.classic.ExecChainElement.execute(ExecChainElement.java:51) ~[docker-java-transport-zerodep-3.2.8.jar:?]
	at com.github.dockerjava.zerodep.shaded.org.apache.hc.client5.http.impl.classic.ExecChainElement$1.proceed(ExecChainElement.java:57) ~[docker-java-transport-zerodep-3.2.8.jar:?]
	at com.github.dockerjava.zerodep.shaded.org.apache.hc.client5.http.impl.classic.ConnectExec.execute(ConnectExec.java:181) ~[docker-java-transport-zerodep-3.2.8.jar:?]
	at com.github.dockerjava.zerodep.shaded.org.apache.hc.client5.http.impl.classic.ExecChainElement.execute(ExecChainElement.java:51) ~[docker-java-transport-zerodep-3.2.8.jar:?]
	at com.github.dockerjava.zerodep.shaded.org.apache.hc.client5.http.impl.classic.ExecChainElement$1.proceed(ExecChainElement.java:57) ~[docker-java-transport-zerodep-3.2.8.jar:?]
	at com.github.dockerjava.zerodep.shaded.org.apache.hc.client5.http.impl.classic.ProtocolExec.execute(ProtocolExec.java:172) ~[docker-java-transport-zerodep-3.2.8.jar:?]
	at com.github.dockerjava.zerodep.shaded.org.apache.hc.client5.http.impl.classic.ExecChainElement.execute(ExecChainElement.java:51) ~[docker-java-transport-zerodep-3.2.8.jar:?]
	at com.github.dockerjava.zerodep.shaded.org.apache.hc.client5.http.impl.classic.ExecChainElement$1.proceed(ExecChainElement.java:57) ~[docker-java-transport-zerodep-3.2.8.jar:?]
	at com.github.dockerjava.zerodep.shaded.org.apache.hc.client5.http.impl.classic.HttpRequestRetryExec.execute(HttpRequestRetryExec.java:93) ~[docker-java-transport-zerodep-3.2.8.jar:?]
	at com.github.dockerjava.zerodep.shaded.org.apache.hc.client5.http.impl.classic.ExecChainElement.execute(ExecChainElement.java:51) ~[docker-java-transport-zerodep-3.2.8.jar:?]
	at com.github.dockerjava.zerodep.shaded.org.apache.hc.client5.http.impl.classic.ExecChainElement$1.proceed(ExecChainElement.java:57) ~[docker-java-transport-zerodep-3.2.8.jar:?]
	at com.github.dockerjava.zerodep.shaded.org.apache.hc.client5.http.impl.classic.ContentCompressionExec.execute(ContentCompressionExec.java:128) ~[docker-java-transport-zerodep-3.2.8.jar:?]
	at com.github.dockerjava.zerodep.shaded.org.apache.hc.client5.http.impl.classic.ExecChainElement.execute(ExecChainElement.java:51) ~[docker-java-transport-zerodep-3.2.8.jar:?]
	at com.github.dockerjava.zerodep.shaded.org.apache.hc.client5.http.impl.classic.ExecChainElement$1.proceed(ExecChainElement.java:57) ~[docker-java-transport-zerodep-3.2.8.jar:?]
	at com.github.dockerjava.zerodep.shaded.org.apache.hc.client5.http.impl.classic.RedirectExec.execute(RedirectExec.java:116) ~[docker-java-transport-zerodep-3.2.8.jar:?]
	at com.github.dockerjava.zerodep.shaded.org.apache.hc.client5.http.impl.classic.ExecChainElement.execute(ExecChainElement.java:51) ~[docker-java-transport-zerodep-3.2.8.jar:?]
	at com.github.dockerjava.zerodep.shaded.org.apache.hc.client5.http.impl.classic.InternalHttpClient.doExecute(InternalHttpClient.java:178) ~[docker-java-transport-zerodep-3.2.8.jar:?]
	at com.github.dockerjava.zerodep.shaded.org.apache.hc.client5.http.impl.classic.CloseableHttpClient.execute(CloseableHttpClient.java:67) ~[docker-java-transport-zerodep-3.2.8.jar:?]
	at com.github.dockerjava.zerodep.ApacheDockerHttpClientImpl.execute(ApacheDockerHttpClientImpl.java:161) ~[docker-java-transport-zerodep-3.2.8.jar:?]
	... 21 more
ERROR our-image:latest] - Could not start container
java.lang.NullPointerException: networkMode was not specified
	at java.util.Objects.requireNonNull(Objects.java:228) ~[?:1.8.0_281]
	at com.github.dockerjava.api.command.CreateContainerCmd.withNetworkMode(CreateContainerCmd.java:294) ~[docker-java-api-3.2.8.jar:?]
	at org.testcontainers.containers.GenericContainer.applyConfiguration(GenericContainer.java:813) ~[testcontainers-1.15.3.jar:?]
	at org.testcontainers.containers.GenericContainer.tryStart(GenericContainer.java:359) ~[testcontainers-1.15.3.jar:?]
	at org.testcontainers.containers.GenericContainer.lambda$doStart$0(GenericContainer.java:325) ~[testcontainers-1.15.3.jar:?]
	at org.rnorth.ducttape.unreliables.Unreliables.retryUntilSuccess(Unreliables.java:81) ~[duct-tape-1.0.8.jar:?]
	at org.testcontainers.containers.GenericContainer.doStart(GenericContainer.java:323) ~[testcontainers-1.15.3.jar:?]
	at org.testcontainers.containers.GenericContainer.start(GenericContainer.java:311) ~[testcontainers-1.15.3.jar:?]
	at java.util.concurrent.CompletableFuture.uniRun(CompletableFuture.java:719) [?:1.8.0_281]
	at java.util.concurrent.CompletableFuture$UniRun.tryFire(CompletableFuture.java:701) [?:1.8.0_281]
	at java.util.concurrent.CompletableFuture$Completion.run(CompletableFuture.java:456) [?:1.8.0_281]
	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1149) [?:1.8.0_281]
	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624) [?:1.8.0_281]
	at java.lang.Thread.run(Thread.java:748) [?:1.8.0_281]
ERROR our-image:latest] - Could not start container

Thanks for reporting this issue @hstaudacher.
I have a couple of questions, that might help us in triaging this bug further:

Could you ever reproduce this issue on your local machine?
Can you reproduce the error in CI, if you omit the BrowserWebDriverContainer and just start your single ApplicationContainer directly?
Can you reproduce the error in CI, if you use a GenericContainer (not your custom ApplicationContainer) with some simple image, e.g. Redis?
How is your Docker set up in CI? From the logs, it looks as if the Docker daemon is not responding when Testcontainers tries to create the network. This might be from the load on the daemon in high congestion shared CI scenarios. We might look into making the network creation more resilient.

Could you ever reproduce this issue on your local machine?
-> No

Can you reproduce the error in CI, if you omit the BrowserWebDriverContainer and just start your single ApplicationContainer directly?
-> No, we have several other builds that use the same "ApplicationContainer". The scenarios are similar e.g. with a JdbcContainer but we haven't seen the error in such cases.

Can you reproduce the error in CI, if you use a GenericContainer (not your custom ApplicationContainer) with some simple image, e.g. Redis?
-> We haven't tried this because it only happens 1 out of 10 runs or so.

How is your Docker set up in CI? From the logs, it looks as if the Docker daemon is not responding when Testcontainers tries to create the network. This might be from the load on the daemon in high congestion shared CI scenarios. We might look into making the network creation more resilient.
-> Not sure what you mean with how is it setup, what is the informatiom you are looking for. High load is a good pointer. We are executing the described scenario a couple times in parallel with forked surefire test vms. We can observe the when the error happens it only happens on one forked vm. All others pass.

Regarding the Docker setup question:
Is there a Docker daemon installed on the same node that executes the build? Is it accessed of the default Unix socket (I see localhost:2375 in the logs)? Is the same Docker daemon shared by multiple CI builds at the same time? Is the Docker daemon running on another server? Is the build running Docker-in-Docker, etc.
IMO we should be able to reproduce this scenario locally by running something like a load test of multiple Testcontainers processes trying to create containers with a new network.

We are using aws spot fleet instances for our builds and use one instance per job at a time. So, the job itself has all the hardware for itself.
On jenkins we are building inside a docker container. The container has mounted the /var/run/docker.sock as a volume from the host.

In theory, what's the upper bound on the number of networks that your test suite could be creating? e.g. assuming all your tests ran as concurrently as possible, how many networks roughly would that entail?

We have one network per forked jvm because we are holding the containers as static variables. We have configured a fork count of 6. So, 6 networks I think.

OK, that shouldn't be enough to exhaust the IP address space that Docker uses for networks, unless there's a particularly bad overlap between the docker defaults and your AWS VPC subnets.
I think we need to see why creation of the Network is failing, rather than the container that relies on the Network. Are any other exceptions logged before this?

Sorry, no exceptions before. Also, the aws instances are killed after the job ran. So, every job starts with a fresh address space. I will check the AWS VPC subnets with our IT.

Besides understanding the root cause of this issue in the case of @hstaudacher, I think we also have a bug (or rather oversight) in our code path, that leads to Testcontainers not retrying the creation of the network was part of the container startup retry.
This is the code in question:
https://github.com/testcontainers/testcontainers-java/blob/2be4c2a927abbed84e2a6cfcea974f821bfe3628/core/src/main/java/org/testcontainers/containers/Network.java#L62
If network creation failed (for whatever reason), the initialized will be set and Testcontainers won't try the creation again as part of the container startup retry block.


How to set a line length with python?

Is it possible to set the line length in the python style?

Yep. Link to applicable Options documentation: https://github.com/Glavin001/atom-beautify/blob/master/docs/options.md#python---max-line-length

Thanks, that works for me.


What's difference between PyTorch:Softmax & openfold/utils/kernel/csrc/softmax

they both calculate softmax on cuda, process similar operations. so why dont we import torch.nn.Softmax

torch.nn.softmax consumes massive amounts of memory. See https://github.com/pytorch/pytorch/issues/72450. Our custom CUDA kernel uses five times less.


WIP: Include devfile/registry to renovate-runner config

What does this PR do?
This PR completes the work for https://github.com/devfile/registry/pull/205 and adds the https://github.com/devfile/registry repo to the config-other.js.
Which issue(s) this PR fixes:
Partially fixes https://github.com/devfile/api/issues/1139

I've added the WIP label, as we have also to invite the bot to the registry repo

I've added the WIP label, as we have also to invite the bot to the registry repo

NP. I merged the PR.  LMK when the invite is issued.


Updated README

Made this library sound more official by fixing a spelling oops of the word 'official'

I signed the CLA

Codecov Report

Merging #906 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #906   +/-   ##
=======================================
  Coverage   93.99%   93.99%
=======================================
  Files          85       85
  Lines        3599     3599
  Branches      568      568
=======================================
  Hits         3383     3383
  Misses         90       90
  Partials      126      126

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 34a7f41...e4c305f. Read the comment docs.


config: Extract to own file.

Extract the Config struct to its own file.
Reasoning: Currently the src/raft.rs file is very long, it can be reduced by extracting some unrelated things, like the configuration.

I rebased this due to #87. PTAL.

PTAL @BusyJay


aimbot

Did also anyone have this "BUG" ?
https://youtu.be/13gp_QyLE14
And I also saw this in console:
CEngineClient::SetViewAngles:  rejecting invalid value [inf -inf -nan(ind)]
CEngineClient::SetViewAngles:  rejecting invalid value [-inf -inf -nan(ind)]
CEngineClient::SetViewAngles:  rejecting invalid value [-inf -inf -nan(ind)]
CEngineClient::SetViewAngles:  rejecting invalid value [-inf -inf -nan(ind)]
CEngineClient::SetViewAngles:  rejecting invalid value [-inf -inf -nan(ind)]
CEngineClient::SetViewAngles:  rejecting invalid value [-inf -inf -nan(ind)]
CEngineClient::SetViewAngles:  rejecting invalid value [-inf -inf -nan(ind)]
CEngineClient::SetViewAngles:  rejecting invalid value [-inf -inf -nan(ind)]
CEngineClient::SetViewAngles:  rejecting invalid value [-inf -inf -nan(ind)]
CEngineClient::SetViewAngles:  rejecting invalid value [-inf -inf -nan(ind)]
CEngineClient::SetViewAngles:  rejecting invalid value [-inf -inf -nan(ind)]
CEngineClient::SetViewAngles:  rejecting invalid value [-inf inf -nan(ind)]
CEngineClient::SetViewAngles:  rejecting invalid value [-inf -inf -nan(ind)]
CEngineClient::SetViewAngles:  rejecting invalid value [-inf -inf -nan(ind)]
CEngineClient::SetViewAngles:  rejecting invalid value [-inf -inf -nan(ind)]
CEngineClient::SetViewAngles:  rejecting invalid value [-inf -inf -nan(ind)]
CEngineClient::SetViewAngles:  rejecting invalid value [-inf -inf -nan(ind)]
CEngineClient::SetViewAngles:  rejecting invalid value [-inf -inf -nan(ind)]
CEngineClient::SetViewAngles:  rejecting invalid value [-inf -inf -nan(ind)]
CEngineClient::SetViewAngles:  rejecting invalid value [-inf -inf -nan(ind)]
CEngineClient::SetViewAngles:  rejecting invalid value [-inf -inf -nan(ind)]
CEngineClient::SetViewAngles:  rejecting invalid value [-inf -inf -nan(ind)]
CEngineClient::SetViewAngles:  rejecting invalid value [-inf -inf -nan(ind)]
CEngineClient::SetViewAngles:  rejecting invalid value [-inf -inf -nan(ind)]
CEngineClient::SetViewAngles:  rejecting invalid value [-inf -inf -nan(ind)]
CEngineClient::SetViewAngles:  rejecting invalid value [-inf -inf -nan(ind)]
CEngineClient::SetViewAngles:  rejecting invalid value [-inf -inf -nan(ind)]
CEngineClient::SetViewAngles:  rejecting invalid value [-inf -inf -nan(ind)]
CEngineClient::SetViewAngles:  rejecting invalid value [-inf -inf -nan(ind)]
CEngineClient::SetViewAngles:  rejecting invalid value [-inf -inf -nan(ind)]
CEngineClient::SetViewAngles:  rejecting invalid value [-inf -inf -nan(ind)]
CEngineClient::SetViewAngles:  rejecting invalid value [-inf -inf -nan(ind)]
CEngineClient::SetViewAngles:  rejecting invalid value [-inf -inf -nan(ind)]
CEngineClient::SetViewAngles:  rejecting invalid value [-inf -inf -nan(ind)]
CEngineClient::SetViewAngles:  rejecting invalid value [inf -inf -nan(ind)]
CEngineClient::SetViewAngles:  rejecting invalid value [inf -inf -nan(ind)]
CEngineClient::SetViewAngles:  rejecting invalid value [inf -inf -nan(ind)]
CEngineClient::SetViewAngles:  rejecting invalid value [-inf -inf -nan(ind)]
CEngineClient::SetViewAngles:  rejecting invalid value [-inf -inf -nan(ind)]
CEngineClient::SetViewAngles:  rejecting invalid value [-inf -inf -nan(ind)]
CEngineClient::SetViewAngles:  rejecting invalid value [inf inf -nan(ind)]
CEngineClient::SetViewAngles:  rejecting invalid value [inf -inf -nan(ind)]

Check your aimbot settings. I think you put some invalid numbers.

@cojo1989 Okay, I fixed it....I had to make new config
I was been still loading old config


Release 0.5.0

@SuperQ Perhaps you could document what's needed to tag and build a release?

Sure, we usually refer to the upstream Prometheus docs. Although we don't typically do release branches for exporters.


Add raven.async.queue.overflow option

Cleanup of https://github.com/getsentry/raven-java/pull/257

Cool, I actually built this on top of the other PR that pulled out DSN options, so will merge after: https://github.com/getsentry/raven-java/pull/256


Promblems importing Adventurer's League statblocks

First of all, great module!  This will save me tons of prep time!
I'm trying to import a statblock from the Adventurer's League Oracle of War module DDAL-EB-10.  Here's an image of the statblock, and the copy n' pasted text version...

Big Bara
Medium humanoid (warforged), neutral evil
Armor Class 18 (natural armor, Imposing Majesty)
Hit Points 117 (18d8 + 36)
Speed 30 ft.
STR
 DEX
 CON
 INT
 WIS
 CHA
14 (+2)
 17 (+3)
 15 (+2)
 13 (+1)
 16 (+3)
 18 (+4)
Saving Throws Con +6, Wis +7
Skills Perception +7, Survival +7
Damage Immunities poison
Condition Immunities charmed, frightened, poisoned
Senses darkvision 60 ft., passive Perception 17
Languages Common
Challenge 9 (5,000 XP)
Imposing Majesty. Big Bara adds her Charisma bonus to her AC
(included above).
Warforged Resilience. Big Bara is immune to disease and magic
can’t put her to sleep.
Actions
Multiattack. Big Bara makes two attacks, either with her
shortsword or armbow.
Shortsword. Melee Weapon Attack: +7 to hit, reach 5 ft., one
target. Hit: 6 (1d6 + 3) piercing damage plus 13 (3d8) poi-
son damage.
Armbow. Ranged Weapon Attack: +7 to hit, range 30/120 ft.,
one target. Hit: 10 (2d6 +3) piercing damage plus 13 (3d8) poi-
son damage.
Poisonous Cloud (2/Day). Poison gas fills a 20-foot-radius
sphere centered on a point Big Bara can see within 50 feet of
her. The gas spreads around corners and remains until the start
of Big Bara’s next turn. Each creature that starts its turn in the
gas must succeed on a DC 16 Constitution saving throw or be
poisoned for 1 minute. A creature can repeat the saving throw
at the end of each of its turns, ending the effect on itself on
a success.

After importing, the actor looks like this...

There are a few problems with it...

The ability scores are all 0, apart from Cha, which is 14 instead of 18.
The saving throws are wrong for Con and Wis
The perception and survival skills are wrong
None of the damage immunities, Condition immunities, senses or languages have been imported.
Poisonous Cloud is missing completely
Creature type and alignment are missing
The armbow does not roll an attack roll, it just rolls damage
I am unable to edit the "Shortsword" and "Warforged Resilience" items.  Simply opening these items and then closing the window is enough to trigger the error.  If I attempt to make changes, then they are lost after re-opening the item.

I also just noticed that the poison damage is missing from the Shortsword and Armbow damage rolls

Wonderful thanks so much for sharing, I'm able to duplicate your error and I have a very strong suspicion I know what is causing it, I should be able to get it patched tonight!

@cs96and The issues you described should have been resolved in this commit: bf22c2ce4f78264baaa022eebd6505927a5bd296
Update to version 0.2.2 and let me know if your problems are fixed!
One note: I think the error thrown when trying to edit weapons is a more pervasive bug, I've noticed it in a few other modules that create monsters as well. I'm going to spend some time investigating it but wanted to get all of the other concerns addressed as quickly as possible. If everything else is fixed, I'll close this issue out and open another issue for that specifically.

Thanks!
It might be worth asking on Discord about the embedded ID problem.

I'm going to spend a little more time looking into it and if I can't figure it out I'll tag in discord, I'm going to close this, feel free to open another ticket if you have issues.
Thanks again for the detailed ticket and extra data, part of my goal for publishing this was to start getting a variety of other input to harden my solution, so it's very much appreciated.

Sorry only just got round to testing the new version.  Works great now, apart from one minor thing... I'll submit another report.  Thanks.


Layered text with sokol_fontstash.h

Cheerio,
I've been trying to achieve layered text rendering with sokol_fontstash.h and couldn't quite get it to work out of the box.
My approach has been to preallocate a pool of FONScontext's with sfons_create(...) and hand out one each time text needed to be rendered in the frame call - and then "re-cycle" them at the end of each frame.
But calling sfons_flush(FONScontext* ctx) followed by sgl_draw() multiple times per frame yields:
sg_update_buffer: only one update allowed per buffer and frame
^^^^  SOKOL-GFX VALIDATION FAILED, TERMINATING ^^^^

The FONScontext in each flush call is unique and only used once per frame. So something in the above call sequence is calling sg_update_buffer more than once, apparently.
I can work around this by commenting out:
https://github.com/floooh/sokol/blob/efc3a2b3d89d1d6371e130dd2d1046c976a4a9cc/sokol_gfx.h#L14712
and
https://github.com/floooh/sokol/blob/efc3a2b3d89d1d6371e130dd2d1046c976a4a9cc/sokol_gfx.h#L15944
Which enables me to render text in a layered way:

(The above GIF is a run where the desired layered rendering is achieved, by applying the "patch")
I'm not sure how intentional it is to not be able to render layered text, maybe I'm doing something wrong?
If not - is there any other way to achieve, layered text rendering?

My approach has been to preallocate a pool of FONScontext's with sfons_create(...) and hand out one each time text needed to be rendered in the frame call - and then "re-cycle" them at the end of each frame.

This sounds about right, but:
The problem is most likely not sokol_fontstash.h (because that doesn't call sg_update_buffer(), but instead sokol_gl.h's sgl_draw() call (which does an sg_update_buffer() call, and must only be called once per frame).
You may need to create one sokol-gl context per fontstash-context (so that each call to sfons_flush() is paired with a call to sgl_context_draw()).
The sgl-context-sapp sample demonstrates how to use multiple contexts with sokol-gl:
https://github.com/floooh/sokol-samples/blob/master/sapp/sgl-context-sapp.c
(however not together with layered text rendering, but for rendering into different render targets), also see this section of the sokol_gl.h documentation: https://github.com/floooh/sokol/blob/efc3a2b3d89d1d6371e130dd2d1046c976a4a9cc/util/sokol_gl.h#L384-L457)

Beautiful :heart: - it works a charm without any nasty workarounds.
Thanks a lot for your time and in-depth answer!


list routes upon start up

list all routes, methods, and handlers upon startup

Waiting for log package.


Enable Continuous, Static and Dynamic Batching

Refactor PredictorDeployment.__call__
Support Continuous Batching (use vllm), Static Batching and Dynamic Batching
TODO: This PR only support --simple, need to consolidate OpenAI support in a separate PR.

check better version in #153


v4: Dropdown toggle not working after hiding

I've used $().dropdown('toggle') to hide the dropdown but after that the dropdown won't show anymore.

Bug reports must include a live demo of the problem. Per our contributing guidelines, please create a reduced test case via JS Bin and report back with specific browser and OS details.

@mdo Hi man! Can you check out this link https://jsbin.com/ranapahihi/edit?html,css,js,console,output

I cannot say for sure if the behaviour is intended but here's a run down of what's happening.

I think the current dropdown toggle function expects that only the button inside .dropdown to trigger it. So...
When you execute the toggle() function using the Hide Dropdown button, that button's reference gets passed to toggle function as if it was the button inside .dropdown.
To be specific, this is where things get messed up. The var parent = Dropdown._getParentFromElement(this); gets the div.p-4.show as the parent and not the .dropdown.
After you have clicked the button, the div.p-4 gets a class show and then on it just returns false assuming the dropdown is already shown.

The $().dropdown('toggle') has to be called on the dropdown trigger. See https://v4-alpha.getbootstrap.com/components/dropdowns/#via-javascript
It works If you change your js function for:
function hideDropdown() {
  $('#dropdownMenuButton').dropdown('toggle');
}

See https://jsbin.com/civimovoxa/edit?html,js,output

Closing per latest comment's fix. Lemme know if we need to revisit.


op.uri does not update, getmarketHistory does not accept count argument

Hey nomad,
I encountered two issues working with your api.
I try to set the count for the getMarketHistory method to anything but 20 but it does not accept the argument and always defaults to 20.
This method seems broken.
Further your op.uri does not update the iteratee in process and always hangs at BTC-ZRC market name as can be seen here at least for the console.log at the end of your pipe.
The iteration itself happens but op.uri does not change. Weird is that the console.log has the count argument correctly in it.
Last, the duration at the end does not seem to be seconds but milliseconds.
streamed from https://bittrex.com/api/v1.1/public/getmarkethistory?market=BTC-ZRC&count=50 in: 4.664s
streamed from https://bittrex.com/api/v1.1/public/getmarkethistory?market=BTC-ZRC&count=50 in: 4.677s
streamed from https://bittrex.com/api/v1.1/public/getmarkethistory?market=BTC-ZRC&count=50 in: 4.697s
streamed from https://bittrex.com/api/v1.1/public/getmarkethistory?market=BTC-ZRC&count=50 in: 4.708s
Your help would be much appreciated in fixing those issues.
Thank you very much for this library.
Sven

I can't see a problem in node.bittrex.api, as you can see request URI is set correctly in both request and stream:
requested from https://bittrex.com/api/v1.1/public/getmarkethistory?market=BTC-ZRC&count=50 in: 0.252s
streamed from https://bittrex.com/api/v1.1/public/getmarkethistory?market=BTC-ZRC&count=50 in: 0.261s

but i still count 20 objects that the Bittrex API returns:
bittrex.getmarkethistory( { market : 'BTC-ZRC', count : 50 }, function( data ) {
  console.log( data.result.length );
});

so, the problem seems to be on the side of bittrex, they simply ignore the 'count' parameter and always return a set of 20 objects.
You could write an email to<EMAIL_ADDRESS>and simply ask/tell them about that issue (if they aren't aware of this already) and post the answer here.

Hey thanks for your response.
Yes indeed it is a bittrex issue and I filled an issue on their support page.
I will let you know what comes out of it.

The response from bittrex was that they updated the api and did not yet update the api doc section about.
The getmarkethistory method no longer allows for a custom count argument.
Regards,
Sven

thanks,
i've updated the readme and i will close this issue as soon as Bittrex updates their API docs.

Good to know nomad ;)
Can I ask you a question?
The issue arose for me trying to get request the tradehistory for all
market names on bittrex.
With those api changes I am currently only able to fetch the default 20
last trades for BTC-NXT ( for instance ) and de-duplicate those with 20
trades I already have saved in my db.
The whole iteration takes around 7 seconds. So if I update trades of quite
actively traded markets on a given moment I am not able to capture all the
trades that happened in between those 7 seconds for some market names.
My question is do you happen to know a strategy for collecting all the
trades that happen on bittrex that is effective?
Thanks in advance and for the exchange of information so far.
Kind regards,
Sven
On 12 August 2015 at 11:53, Adrian Soluch<EMAIL_ADDRESS>wrote:

thanks,
i've updated the readme and i will close this issue as soon as Bittrex
updates their API docs.
—
Reply to this email directly or view it on GitHub
https://github.com/n0mad01/node.bittrex.api/issues/10#issuecomment-130242742
.

hi,
first off, this doesn't belong in this thread and should be a dedicated one because it's another issue.
you could run every trading-pair in a separate instance, then every iteration should last only milliseconds, there aren't that many trades in the meanwhile, so you would'nt lose much data (as every trade has it's own, iterative and numeric id) - but that would get too abusive on the API and you may get blocked for that.
if you need large datasets for historical analysis you could ask bittrex or look elsewhere:
https://www.cryptocoincharts.info/tools/api
in case you need fresh data for predictions you don't need absolutey all data, it's always possible that some trades 'get swallowed', but if you don't miss whole minutes and therefore tens or hundreds of trades then you're good to make predictions.


Documentation "enum RecipeBonusType" vs. usage in XML

Issue Description:
There seem to be two ways of controlling the bonus of item processing: either e.g. exclude the balls () or directly specify it directly in the recipe (<recipe name="iron_gravel_to_crushed" energyCost="500" bonusType="chance_only" >
(I prefer the later)
I searched the code on github to find how "bonusType" gets parsed, but I couldn't find it.

Is bonusType supported at all?
If it is not, are the affected default recipes broken / in need of adjustment?
If it is, it needs documentation, especially for specifying other values than "chance_only"

PS: Typo in SAGMillRecipes_Core.xml: Missing string terminator:
<dumpRegistry modObjects="true/>
Affected Versions (Do not use "latest"):
EnderIO-1.10.2-3.1.183.jar
EnderCore-1.10.2-<IP_ADDRESS>-beta.jar

The github search is not the best. The value is parsed, put into the job and finally used:
  protected void taskComplete() {
    IPoweredTask ct = currentTask;
    super.taskComplete();
    // run it again if the ball says so
    if (gb != null && useGrindingBall && ct != null) {
      if (ct.getBonusType() == RecipeBonusType.MULTIPLY_OUTPUT) {
        float chance = random.nextFloat();
        float mul = gb.getGrindingMultiplier() - 1;
        while (mul > 0) {
          if (chance <= mul) {
            currentTask = ct;
            super.taskComplete();
          }
          mul--;
        }
      }
    }
  }

In comparison, theis is what the grinding ball exclusion does:
  @Override
  protected IPoweredTask createTask(IMachineRecipe nextRecipe, float chance) {
    PoweredTask res;
    useGrindingBall = false;
    if (gb != null) {
      useGrindingBall = !SagMillRecipeManager.getInstance().isExcludedFromBallBonus(getRecipeInputs());
      if (useGrindingBall) {
        res = new PoweredTask(nextRecipe, chance / gb.getChanceMultiplier(), getRecipeInputs());
        res.setRequiredEnergy(res.getRequiredEnergy() * gb.getPowerMultiplier());
      } else {
        res = new PoweredTask(nextRecipe, chance, getRecipeInputs());
      }
    } else {
      res = new PoweredTask(nextRecipe, chance, getRecipeInputs());
    }
    return res;
  }

However, you are right, the value "RecipeBonusType.CHANCE_ONLY" is not used at all.
Is this broken? I have no idea, I don't claim to understand the recipe system. However, as the bonus type "NONE" is also not used by any recipe, it seems to me that the bonus type actually only has 2 values "chance only" and "multiply".
Let me write down how it works for reference:
So, if "multiply" is specified (default): The whole recipe output is duped for grindingMultiplier times (meaning you get one output per whole number and the remainder of the number is the chance for another output, e.g. 7,39 = 7 outputs and a 39% chance for an 8th). If it is set to "chance only" (or "none"), the grindingMultiplier has no effect and you only ever get one recipe output from a recipe.
If a recipe is not excluded from the grinding ball bonus: The chances for the outputs are multiplied with the chanceMuliplier. So if an output is given with a chance of e.g. 0.5 in the recipe and a grinding ball with a chance multiplier of 1.6 is used, you have a 75% (0.5*1.5) chance of getting that output. Note that here chances above 100% do not give multiple outputs, this is a hit-or-miss check. If it is excluded, the chance is used as-is and the "chance only" behavior from above is also enforced.
That is interesting. It seems that the grinding ball exclusion list---which is a relatively recent addition---is actually not needed. The multiplication of outputs can already be disabled and the higher chances are not something one usually would want to prevent.
Thanks for bringing this up, I'll have a look into making "NONE" work again and doing the same thing the exclusion list does now.
tl;dr: bonusType="chance_only" works. bonusType="none" is broken and should do what the exclusion list now does.

Update: Looking at the history, it seems I was wrong. The exclusion list is the older of those 2 features. My mistake.

My ¢¢: The exclusion list should just overwrite bonusType then, that will reduce the the amount of code dealing with exceptions.

Nah, I just removed the exclude list. The only thing remaining is the automatic detection of "nugget", "ingot" and "block" recipes (e.g. iron ingot to iron dust).
This goes into 1.11 because it's a breaking change in the config file.


Support v4.0.0-alpha.2

Hi,
It'd be great if you could add support for bootstrap v4.0.0-alpha.2 please.
Thanks

Hey @boosh, if you want to submit a PR, I'll add as a separate branch for now.

Ah sorry, I switched to bootstrap-loader instead. Thanks though.


C-components of G' generates empty set in Example use case line 51

Running the Example Use Case notebook line 51 results in an error:
print(graph_test.idc_star_alg(do_gam,do_del,[],obs_del))

should result in
# Ans: [sum_{W}P_{Z,D,W*}(Y,X)P_{X*}(W)]/[sum_{Y}[sum_{W}P_{Z,D,W*}(Y,X)P_{X*}(W)]]

Instead, it results in the following:
---------------------------------------------------------------------------
ValueError                                Traceback (most recent call last)
<ipython-input-5-b9b46cc37970> in <module>
     49 # Ans: sum_{W}P_{Z*,W*}(Y,X)P_{X*}(W)
     50
---> 51 print(graph_test.idc_star_alg(do_gam,do_del,[],obs_del))
     52 # Ans: [sum_{W}P_{Z,D,W*}(Y,X)P_{X*}(W)]/[sum_{Y}[sum_{W}P_{Z,D,W*}(Y,X)P_{X*}(W)]]

~/Projects/Proposals/MARS/Pathogenesis/Covid-19-Causal-Reasoning/bel2scm/bel2scm/causal_graph.py in idc_star_alg(self, do_in, do_delta, obs_in, obs_delta, graph_in)
    927
    928
--> 929         if self.id_star_alg(do_delta,obs_delta,graph_in) == '0':
    930             #print('IDC* Step 1')
    931             return 'UNDEFINED'

~/Projects/Proposals/MARS/Pathogenesis/Covid-19-Causal-Reasoning/bel2scm/bel2scm/causal_graph.py in id_star_alg(self, do_in, obs_in, graph_in)
    849                     #print(do_in_temp)
    850
--> 851                     str_temp = self.id_star_alg(do_in_temp,[],graph_temp)
    852
    853                     # make sure that all the variables being summed over don't have asterisks

~/Projects/Proposals/MARS/Pathogenesis/Covid-19-Causal-Reasoning/bel2scm/bel2scm/causal_graph.py in id_star_alg(self, do_in, obs_in, graph_in)
    779             s_sets = [item for item in s_sets if item[0] not in do_vars_temp]
    780             if len(s_sets) == 0:
--> 781                 raise ValueError(f"S' is empty {list(nx.connected_components(conf_out.to_undirected()))} after removing do variables {do_vars_temp}")
    782
    783             #print('C sets')

ValueError: S' is empty [] after removing do variables []

Note that I did a very minor cleanup of the code so that importing classes would work correctly. I also added the ValueError exception so that you could easily see where the problem occurs.

After speaking with @cbakker2 his latest code has not been pushed to the updated_code_craig branch and this branch has not been merged to master branch

Code updated and now functioning


How to use config service within app.module.ts

I have the following folder structure (as given in the getting started section of the readme)
/src
├── app.module.ts
├── config
│   ├── app.ts
...

In my app.module.ts, I have
import { Module } from '@nestjs/common';
import { AppController } from './app.controller';
import { AppService } from './app.service';
import { ConfigModule, ConfigService } from 'nestjs-config';

@Module({
  imports: [ConfigModule.load()],
  controllers: [AppController],
  providers: [AppService],
})
export class AppModule {
  constructor(private readonly config: ConfigService) {
    this.config = config;
    console.log(config);  // ConfigService { helpers: {} }
    AppModule.host = this.config.get('app.url');
    console.log( 'App URL is',  this.config.get('app.url') ); // App URL is undefined
  }
...
}

What could I be doing wrong?

By the time the AppModule Constructor runs, it hasn't initialized ConfigModule via Dependency Injection yet.
Have you tried OnModuleInit lifecycle hook?

I think this is related to #45 where using .load without a path resolves cwd and not src. I haven't really decided what the best fix would be.

#35

@bashleigh. Yeah, I eventually figured it out after a while. Will you consider updating the readme to reflect this. I spent alot of time trying to figure out what I was doing wrong and even went the route suggested by @natrc without knowing this is a bug that has already been reported.

I think instead I'm going to remove the empty load example and do a complete refactor and clean up for the main service and also see if I can improve the decorator usage. Plus keep this issue open until I've done so.

It will also be good to show how to make use of this in the bootstrap/main.ts itself. Given that access to the port is needed for example.
Ideally, config service should be one of the first things loaded in an app before even the server is started, once loaded it should be available globally always.
Also, it would be nice that if it can easily (read seamlessly) be injected as middleware so it reloads at runtime and only if files changed for perf reasons.
See : [https://github.com/kelseyhightower/envconfig]( Golang Env Config) for some ideas. I have used this go module previously.


sparssified yolov5s does not speed up rather to yolov5s .

I'd use below code to transfer learning yolov5 on my custom dataset
!sparseml.yolov5.train --data {dataset.location}/data.yaml  --cfg models_v5.0/yolov5s.yaml \                        --hyp data/hyps/hyp.finetune.yaml \                        --recipe zoo:cv/detection/yolov5-s/pytorch/ultralytics/coco/pruned-aggressive_96?recipe_type=transfer --batch 16\                        --epochs 100  --imgsz  416 --save-period 5
and after that,  export to onnx
!sparseml.yolov5.export_onnx --weights epoch60.pt                              --simplify
Expected behavior
I expect this onnx file give faster  inference than  yolov5s traind on same custom data which exported to onnx.
but i get this result for sparssified yolov5s :
on CPU: 338 ms
on GPU: 13 ms
and this for yolov5s :
on CPU: 345 ms
on GPU: 12 ms

Hi @mpanahish, you'll want to add the argument --weights zoo:cv/detection/yolov5-s/pytorch/ultralytics/coco/pruned-aggressive_96?recipe_type=transfer to your training command. You're on the right track by using sparse transfer learning to train your sparse model. The recipe that you're using assumes you're starting with an already sparse model and maintains the sparsity level while fine-tuning on your dataset (the recipe_type=transfer query argument specifies this), so adding the argument above will set your starting model as the sparse one.
Closing the issue, but feel free to re-open if you're seeing any more speedup issues!

Hi @mpanahish, you'll want to add the argument --weights zoo:cv/detection/yolov5-s/pytorch/ultralytics/coco/pruned-aggressive_96?recipe_type=transfer to your training command. You're on the right track by using sparse transfer learning to train your sparse model. The recipe that you're using assumes you're starting with an already sparse model and maintains the sparsity level while fine-tuning on your dataset (the recipe_type=transfer query argument specifies this), so adding the argument above will set your starting model as the sparse one.
Closing the issue, but feel free to re-open if you're seeing any more speedup issues!

I did your comment and retrain my model but nothing changed !
time performance not changed !
my training code :
!sparseml.yolov5.train --data {dataset.location}/data.yaml  --cfg models_v5.0/yolov5s.yaml \                        --weights zoo:cv/detection/yolov5-s/pytorch/ultralytics/coco/pruned-aggressive_96?recipe_type=transfer \                        --hyp data/hyps/hyp.finetune.yaml \                        --recipe zoo:cv/detection/yolov5-s/pytorch/ultralytics/coco/pruned-aggressive_96?recipe_type=transfer --batch 16\                        --epochs 100  --imgsz  640 --save-period 5\                        --project /content/drive/MyDrive/--name=exp


what is the current PDK_HASH

when i try to enable sk13o using the following commads:
PDK_HASH = 'fa87f8f4bbcc7255b6f0c0fb506960f531ae2392'
PDK_ROOT = '/root/.volare/volare/sky130/versions'
pip install --upgrade --no-cache-dir volare
volare enable --pdk sky130 {PDK_HASH}
i got following error:
Version {PDK_HASH} not found either locally or remotely.
Try volare build {PDK_HASH}.
when tried to build {PDK_HASH} it gives:  Cannot update paths and switch to branch 'current' at the same time.\nDid you intend to checkout '{fa87f8f4bbcc7255b6f0c0fb506960f531ae2392}' which can not be resolved as commit?\n"
what does that mean??

Volare is a project build and maintained by @efabless and not Google.

You will want to report the issue to https://github.com/efabless/volare


Catch error when citydb schema(s) are all empty

If the citydb has been correctly installed but it is empty, or there are multiple citydb schemas and they are all empty, the user client will raise an error.

Solved in v. 0.6.0


pgvector: Include comments on exported symbols

This includes comments on the exported symbols.

Thanks @tmc!


Frontend docker build fails with compile error

Hey hey 👋
I've been running the following command to build a docker image of the near-explorer frontend:
docker build -f frontend/Dockerfile .

and it's failing with the following error:
[+] Building 300.9s (18/28)
 => [internal] load build definition from Dockerfile                                                                                0.0s
 => => transferring dockerfile: 1.23kB                                                                                0.0s
 => [internal] load .dockerignore                                                                                0.0s
 => => transferring context: 34B                                                                                0.0s
 => [internal] load metadata for docker.io/mhart/alpine-node:16                                                                                0.5s
 => [internal] load build context                                                                                0.1s
 => => transferring context: 1.45MB                                                                                0.1s
 => CACHED [stage-1 1/9] FROM docker.io/mhart/alpine-node:16@sha256:c9014e9e5b33f29d47c867ea548edc0235ba71677f40456409a44c278d8a8e01                                                                                0.0s
 => [stage-1 2/9] RUN apk add --no-cache git                                                                                2.2s
 => CACHED [build  2/16] WORKDIR /near-explorer                                                                                0.0s
 => CACHED [build  3/16] COPY ./package-lock.json ./package.json ./tsconfig.json ./                                                                                0.0s
 => CACHED [build  4/16] COPY ./patches/ ./patches/                                                                                0.0s
 => CACHED [build  5/16] COPY ./common/package.json ./common/tsconfig.json ./common/                                                                                0.0s
 => [build  6/16] COPY ./frontend/package.json ./frontend/tsconfig.json ./frontend/                                                                                0.0s
 => [build  7/16] COPY ./backend/package.json ./backend/tsconfig.json ./backend/                                                                                0.0s
 => [build  8/16] RUN npm clean-install                                                                                175.1s
 => [stage-1 3/9] WORKDIR /near-explorer                                                                                0.0s
 => [build  9/16] COPY ./common ./common                                                                                0.0s
 => [build 10/16] COPY ./frontend ./frontend                                                                                0.0s
 => [build 11/16] COPY ./backend ./backend                                                                                0.0s
 => ERROR [build 12/16] RUN npm run -w frontend build                                                                                125.0s
------
 > [build 12/16] RUN npm run -w frontend build:
#18 2.034
#18 2.034 ><EMAIL_ADDRESS>build
#18 2.034 > next build
#18 2.034
#18 26.19 warn  - You have enabled experimental feature(s).
#18 26.20 warn  - Experimental features are not covered by semver, and may cause unexpected or broken application behavior. Use them at your own risk.
#18 26.20
#18 26.28 Attention: Next.js now collects completely anonymous telemetry regarding usage.
#18 26.28 This information is used to shape Next.js' roadmap and prioritize features.
#18 26.28 You can learn more, including how to opt-out if you'd not like to participate in this anonymous program, by visiting the following URL:
#18 26.28 https://nextjs.org/telemetry
#18 26.28
#18 26.94 info  - Checking validity of types...
#18 124.8 Failed to compile.
#18 124.8
#18 124.8 ./src/pages/_app.tsx:315:16
#18 124.8 Type error: Property 'props' does not exist on type '{ ctx?: NextPageContext | undefined; }'.
#18 124.8
#18 124.8   313 |     const networkName =
#18 124.8   314 |       "props" in info
#18 124.8 > 315 |         ? info.props.pageProps.networkName
#18 124.8       |                ^
#18 124.8   316 |         : getNearNetworkName(info.ctx.query ?? {}, info.ctx.req?.headers.host);
#18 124.8   317 |     const isSsr = typeof window === "undefined";
#18 124.8   318 |     const httpUrl = getBackendUrl(networkName, "http", isSsr);
#18 124.9 npm ERR! Lifecycle script `build` failed with error:
#18 124.9 npm ERR! Error: command failed
#18 124.9 npm ERR!   in workspace<EMAIL_ADDRESS>#18 124.9 npm ERR!   at location: /near-explorer/frontend
------
executor failed running [/bin/sh -c npm run -w frontend build]: exit code: 1

Not super familiar with FE code in general, so I'd rather get some expert opinion before I dig into this more. Any idea what might be going on there?

@luixo it is reproducible on my end. Could you take a quick look if it is something obvious that can be fixed quickly?

@gbouv thanks for catching that one!
The problem was that some patches migrated from root dir to frontend dir and were not copied inside the docker context.
The fix is in the master.

Yes I can confirm issue is fixed! Thanks for the fix!


Padding over \sqrt and Paths for frac-line

This replaces two earlier PRs, #1131 Change frac-line from SVG line to SVG path, and #1136 Add 0.08em padding above \sqrt line.
It was easier to create a substitute branch than it was to repair my messed up history.

@kevinbarabash Thank you. In my frustration, I got hasty.

I've regenerate the screenshots in https://github.com/ronkok/KaTeX/pull/1.  It's incredible how many of our screenshots involve fractions.  Many of them probably don't need fractions to test what they're testing.

It's incredible how many of our screenshots involve fractions.

I know that I contributed to that. When I was testing stretchy arrows, fractions were a cheap way to test vertical alignment and vertical kerns used for clearances.

When I was testing stretchy arrows, fractions were a cheap way to test vertical alignment and vertical kerns used for clearances.

I guess you need some way to test clearance on top.  It's either \frac or \sqrt I guess.

Well, I'm out of date with regard to a submodule. I tried running git submodule update --init --recursive but have gotten no relief. I'm stuck.

@ronkok I'll check out your branch tomorrow and see if I can resolve the conflict.

@kevinbarabash I am not able to view the Travis log, but I believe that I have this PR to the point where it passes all tests except screenshotter.

@ronkok thanks for figuring out the merge.  I've been slammed at work.  Will try to regenerate the screenshots tonight.

Having a coverage target for patches is silly.  I'm going to try to get rid of that check.


wait for failed jobs

What this PR does / why we need it: closes #9285
Special notes for your reviewer:
If applicable:

[ ] this PR contains documentation
[*] this PR contains unit tests
[ ] this PR has been tested for backwards compatibility

This branch has conflicts that must be resolved

@aCLr can you  rebase ?

@antonio-antuan If this is still relevant, could you fix the conflicts? If not, can you close it?

@antonio-antuan If this is still relevant, could you fix the conflicts? If not, can you close it?

ok closed


[FEATURES] ActivateOpenWindow, OpenWindowDetection, ChildLockEnabled

This merge requests adds following features
Child Lock
You can now enable/disable the child lock by setting the childLockEnabled property.
Open Window Detection
You can now modify the settings tado.[Z].[X].Rooms.[Y].openWindowDetection.enabled and tado.[Z].[X].Rooms.[Y].openWindowDetection.timeoutInSeconds.
Note: I did not change state attribute write on enabled in the state_attr.js, as the parameter is used in other places as well. I guess, you can set it via code nonetheless?
ActivateOpenWindow
When the thermostat detects an open window, tado will send you a notification and in the app appears a button to power off the radiator/thermostat (for the period defined in tado.[Z].[X].Rooms.[Y].openWindowDetection.timeoutInSeconds). The new button tado.[Z].[X].Rooms.[Y].activateOpenWindow can trigger this app call.
Using the button you can create a blockly automation like this:

This basically is the same the "Auto-Assist" does.
Note that triggering tado.[Z].[X].Rooms.[Y].activateOpenWindow without a detected open window by tado has no effect.

Hi @ilueckel ! Thanks for the PR! I just recognized the PR today and will have a look on it in the next few days!

Hi @HGlab01,
I was wondering if there is anything left or needs to be changed in handling the states from your point of view?

Really nice implementations!
I hope it was not to hard to understand the general approaches for your additional implementations!
@state enabled:
I did the following "trick" for the attribute celsius:
https://github.com/DrozmotiX/ioBroker.tado/blob/dde43947766ba063d7905322ef699ae836e74e6e/main.js#L821-L824
With this I renamed the JSON attribute before creating states. I think you can do this here as well.
Maybe not extremely beautiful but it works ;-)

Great - yeah this would be a good possibility. I renamed openWindowDetection.enabled to openWindowDetection.openWindowDetectionEnabled and added the matching state.

Thanks for all the work! Let's merge to main and create an alpha version including your PR!

One additional remark/wish:
If you add additional things please deactivate sentry in your test installation. Otherwise I have a lot of issues in Sentry which makes me nervous until I recognize where it comes from ;-)

Enjoy!
https://github.com/DrozmotiX/ioBroker.tado/archive/refs/tags/v0.3.15-alpha.2.tar.gz


Confirm that CheckPoint CSPM can run/compare reports before and after an event (like a patch)

Describe the issue
We have too many test frameworks.
The user persona of the test that Carles is aiming for at end of Sept is a Platform Admin who is performing an SEA patch on ECF Test and intended for ECF Prod.
We want to try to leverage CSPM so that we can hopefully snapshot state before the patch is run, snapshot after, compare and alert if different (assuming that the state changes in fail condition).
Additional context

Assume that it is easy to get reports out of CSPM
Assume that it is unlikely to do the comparison within CSPM

Definition of done

Compare the output of a report pointed at an AWS workload environment to the output from the AWS CLI (check for completeness)
Come up with a preliminary process using CSPM and perhaps even a manual step like running two output files in Meld (an opensource compare tool).
Maybe CSPM has the ability to detect a state change and notify out.  That would make the next part below irrelevant.
In scope, detect a difference before after.
Out of scope, create automation for detecting the difference (this could look like an docker image to be defined and run locally with a Meld/diff compare, a test-auto framework, or some other approach.  Then even later, we host that as a service on AWS and run our tests there, notify out)

the following curl command lists all the roles, in this case AWS roles
curl -X GET
https://api.dome9.com/v2/CloudIamRole
-H 'Accept: /'
-H 'Accept-Encoding: gzip, deflate'
-H 'Authorization: Basic YzE1ZjQyOGUtOTE1Yy00OWIwLWJhN2YtYTdlYTkxMzgwNWQ2Onh1bmNxaXc5a205aGU2ODViNnBveG90aQ=='
-H 'Cache-Control: no-cache'
-H 'Connection: keep-alive'
-H 'Cookie: visid_incap_1095475=5zkS+5mSSCOH3hYXv1Lrj9O7p2IAAAAAQUIPAAAAAAA+sXvGbBuzh5XDpc1rtOML; nlbi_1095475=P1i2HKLrrQd4vv5qQ7IvqQAAAADLwCYtsTYZ/MvKw+8xEWVS; incap_ses_8078_1095475=AgCkTxnachJAFVLbp9IacL/Bp2IAAAAA/wfsnDAAFICA0asDcvVHDA=='
-H 'Host: api.dome9.com'
-H 'Postman-Token: b0bd8f7e-ceb4-4b13-b843-98c9705cd2d4,b7145329-965a-47f3-9d2a-101058a0c744'
-H 'User-Agent: PostmanRuntime/7.19.0'
-H 'cache-control: no-cache'

Compare the output of a report pointed at an AWS workload environment to...
CheckFront Cloudguard does not seem to have the possibility to create reports, however, the API collection allows to download the appropriate information. And from here reports can be built

The AWS IAM CLI function also allow to extract useful information for each account. Hopefully it will also allow to extract the information for all account associated to user when login as such user

Come up with a preliminary process using CSPM and perhaps even a manual ....
See attached file

Maybe CSPM has the ability to detect a state change and notify out.
Going through the interface it does not seem this feature exists. I am also login to an instance with expired license and not sure if it is up to date.

In scope, detect a difference before after.
Currently this is implemented by comparing selected fields from the files obtained using e the Cloudguard API and the AWS CLI (see attached file)
Not sure how the information may change after a SEA upgrade/update.  i will need to check after such operation. Hopefully there may be no changes in artifacts that exist before and after the upgrade and the differences will be limited to known changes

Out of scope, create automation for detecting the difference (this could look like....
Currently the steps are manual. However, with a long live AWS account all the steps can be part of an automated test scripthttps://app.zenhub.com/files/241742911/8911afe0-28b3-4820-95fa-2e9a92834e05/download


/notifications link has double http:// resulting in a random website on click

Template,
What kind of issue is it? (question, bug, suggestion, etc.)
Bug
In which Minecraft Server Software did it occur, Spigot or BungeeCord?
Spigot
In which version did it occur?
Beta 27
How would you describe the issue?
When I make a report, and then do /notifications and click the link in chat, it opens a link "http://https://MYWEBSITE.com/user/alerts/?view=XXX
Do you have (relevant) resources about this issue (Logs, pictures, etc.)?
http://prntscr.com/nyn53d

The plugin does not add http:// afaik, it might be a website issue

It's also possible that you mistyped "http://" at the beginning of api-url in config.yml


Admin user couldn't see the active sessions of created user.

Describe the issue:
Once the super admin user creates a new user under a super organization with admin privileges, the newly created user can log in to the console. So once the newly created user login to the console, there will be an active session and the admin user should be able to see the user's session. However, the active sessions of the newly created users are not visible to the admin user. Please refer to the screen cast below:
https://user-images.githubusercontent.com/13362482/180153945-e18ec142-2a61-4219-8778-888775da15da.mov
How to reproduce:

Log in to the console (https://localhost:9443/console) as an admin user
Create a user with admin privileges under super organization.
Now login to console with newly created admin privileges user.
Logout and log in again to the console as an admin user.
Navigate to the Manage -> Users section and edit the newly created user and verify the active sessions.

Expected behavior:
Admin users should be able to see the details of the active session.
Environment information (Please complete the following information; remove  any unnecessary fields) :

Product Version: [e.g., IS 5.10.0, IS 5.9.0]
OS: [e.g., Windows, Linux, Mac]
Database: [e.g., MySQL, H2]
Userstore: [e.g., LDAP, JDBC]

Optional Fields
Related issues:

Suggested labels:

The issue is not reproducible on wso2is-6.1.0-m1-SNAPSHOT as shown in below figure where "sadilc" is an admin priviladged user and currently logged in via admin user. Though active sessions shown in super organization level users, the console app for sub organizations level hide the active session view.


docs: add Decision-Record about removing SFTP extensions

WHAT
Adds a decision record about removing SFTP.
WHY
documenting the decision
FURTHER NOTES
List other areas of code that have changed but are not necessarily linked to the main feature. This could be method signature changes, package declarations, bugs that were encountered and were fixed inline, etc.
Closes # <-- insert Issue number if one exists

At least ZF would like to have an FTP Dataplane and we are also working on it together with Cofinity-X. We would like to propose a working dataplane as a contribution, but as usual other things have more priority on our side at the moment.
The reason why we would like the EDC to support another protocol is that S3 (and also Azure Blob) is a proprietary protocol and therefore does not fulfil the needs of companies that want to exchange big data without being forced to use one of the big cloud providers as the storage provider.
\cc @dr-markus

Of course we could "copy/paste" the extensions into our EDC build (which we have already done during testing/development). But we're not just thinking of ourselves. Many small and medium sized businesses won't have a cloud subscription or want to host a full-blown MinIO instance. Most of them have almost no IT department. However, for us as Tractus-X, it makes perfect sense to offer SFTP, which is also an already established standard for data storage/exchange worldwide. Refusing to have this in our "standard" Tractus-X EDC is a big mistake for me and @dr-markus.
Of course I totally agree with you that the current implementation is not sufficiently maintained! But instead of saying "let's get rid of it" or "you can provide it yourselves" we should think about how we can get rid of the maintenance issue and how we can make sure it is maintained as a core Tractus-X EDC protocol in the future.

Hosting MinIO is not harder than hosting (and properly configuring!) an SFTP server, so I fail to see the validity in that argument.
My point is that for almost a year, no-one that uses our standard distribution in Tractus-X could have been using SFTP, because we didn't package it. And we have received exactly 0 requests to that effect. Your mileage may vary, which is why I think that functionality is better suited for a ZF/Cofinity variant.
We absolutely cannot keep those modules in the Tx-EDC code base in their current form, because they are poorly implemented and violate several coding and architectural principles, and they pose a potential security hole.
We discussed removing them already, about a year ago, and the decision back then was to keep them, but to rewrite and improve them, which obviously has not happened. Since it is exclusively you who wants to keep them around, and no-one (including you) has picked up that baton, I can only assume that the priority for you is not high enough.
I think one year is enough of a grace period.
The "you can provide it yourself" is key aspect and architectural principle of EDC.

Of course we could "copy/paste" the extensions into our EDC build (which we have already done during testing/development). But we're not just thinking of ourselves. Many small and medium sized businesses won't have a cloud subscription or want to host a full-blown MinIO instance. Most of them have almost no IT department. However, for us as Tractus-X, it makes perfect sense to offer SFTP, which is also an already established standard for data storage/exchange worldwide. Refusing to have this in our "standard" Tractus-X EDC is a big mistake for me and @dr-markus.
Of course, I totally agree with you that the current implementation is not sufficiently maintained! But instead of saying "let's get rid of it" or "you can provide it yourselves," we should think about how we can eliminate the maintenance issue and ensure that it is maintained as a core Tractus-X EDC protocol in the future.

I don't think that contradicts what this DR proposes. A separate repository can be established in TX to maintain and release the extension properly, which would appear to fulfill your requirements. As it stands now, the current code is not properly maintained or up to the standards required for general use.
I also don't think the use case for small and medium-sized businesses is common. If a business has limited IT staff, it is much cheaper (on the pennies!), less technically challenging, and more secure to use a cloud provider's object storage than to host and properly maintain (S)FTP servers. Properly operating scalable, reliant (S)FTP servers is challenging and typically not something an SMB would invest in.
The core TX-EDC team has limited resources, and we are trying to deploy them judiciously to cover high-priority use cases. Unfortunately, limited resources force us to make choices. The prioritization seems clear based on the fact that the SFTP extensions have not been maintained or included in a TX distribution for a long time, nor have they been proposed as a C-X standard.

Hi, @paullatzelsperger @jimmarino,
Actually, we just prepared a contribution to complete the SFTP extension, so that it can be used in a dataplane. It is a planned feature of Dataspace OS and we (Cofinity-X in association with ZF) are willing to maintain the feature in the future. Of course it could be maintained in a different repository, but I would strongly suggest to keep it where it is to simplify the consistency of the extension.
Btw., Minio is in productive use a limited option, due to legal issues and efforts to fulfill legal constraints.

Tx-EDC is not a toolbox of random extensions, but a reference implementation. I thought that was understood.

If the Tractus-X repository is dedicated for the task of providing the "official" distribution of the Tractus-X EDC, I understand and support the deletion of the piece. On the other hand, I would see the necessity to have a place to share quality extensions within the community, so that end users can make use of the feature easily. What do you propose as place for something like that? Does this already exist?
Actually, with simpler I meant, that an own repository comes with additional efforts in setting the repository up to provide the necessary quality infrastructure, visibility is decreased, so people who might need the feature won't find it, and finally it is an additional repository to be maintained by the overall project infrastructure.

This could very well be a separate tractusx repo, that is maintained by ZF/Cof. It is very easy to set that up. The EF has a tool called Otterdog where additional repos can be requested, and we even provide a template repo for that exact purpose, that comes complete with all required legal documents, quality processes etc. It does not get any easier than that.

Tx-EDC is not a toolbox of random extensions, but a reference implementation. I thought that was understood.

If the Tractus-X repository is dedicated for the task of providing the "official" distribution of the Tractus-X EDC, I understand and support the deletion of the piece. On the other hand, I would see the necessity to have a place to share quality extensions within the community, so that end users can make use of the feature easily. What do you propose as place for something like that? Does this already exist?
Actually, with simpler I meant, that an own repository comes with additional efforts in setting the repository up to provide the necessary quality infrastructure, visibility is decreased, so people who might need the feature won't find it, and finally it is an additional repository to be maintained by the overall project infrastructure.

A place to share quality extensions or a directory would be a very useful thing to have. We could do something similar to upstream EDC's Known Friends Directory.
We already have a Github template project that does most of the heavy lifting to set up an extension repository. We could continue to develop the template project to automate this process even further.

That sounds good, we will discuss the approach and move forward. Thanks, Paul, Jim.

For me and @dr-markus it totally makes sense to provide sftp as a standard protocol in the tractus-x edc.

the official Tx-EDC distribution and repository can only contain officially sanctioned and standardized C-X extensions, which SFTP is not.

@paullatzelsperger Where is it documented which EDC extensions are officially standardised by C-X and where is the process documented for proposing an extension to be officially standardised by C-X? And who decides on this?
In the past it was (more or less) decided by us developers or not?
Please don't misunderstand my comments - I agree with you that we should get rid of unused code! I just want to clarify that there is a need for SFTP and that we should clarify how we can proceed with including SFTP as a official supported protocol in the Tractus-X EDC. So for me it would be perfectly fine if we get rid of the existing code if we have a plan for the future.

I think a good way forward would be to spin up a community or "contrib" repository as outlined in earlier comments, where people can develop and maintain their extensions, and if a strong need arises to adopt one into the standard distribution of EDC, we decide that on a case-by-case basis. We can even do so without moving any code, provided that a Maven publication exists.
I do think that certain acceptance criteria and rules have to be established for that.

For me and @dr-markus it totally makes sense to provide sftp as a standard protocol in the tractus-x edc.

the official Tx-EDC distribution and repository can only contain officially sanctioned and standardized C-X extensions, which SFTP is not.

@paullatzelsperger Where is it documented which EDC extensions are officially standardised by C-X and where is the process documented for proposing an extension to be officially standardised by C-X? And who decides on this?
In the past it was (more or less) decided by us developers or not?

If you want to standardize SFTP, you should join the CX Standardization Working Group and discuss it there. Keep in mind that standardization would require all implementations of the Catena-X wire protocol (including EDC) and all participants to support the feature, so the bar for standardization is high.


CAM for binary class

for example, the task is to classify dog or not, and we label dog as class 1.
when comes a dog image, class 1 focus on any part except dog;
on the contrary, class 0 focus right on dog.

I found the reason. Cuz colorspace is  BGR in OPENCV !!!. This can be closed...


Ask Mma to only compute one eigenvalue instead of all

Currently, the spectral approach asks Mma to compute all eigenvalues, and then selects one based on a guess of the ordering:
https://github.com/BlackHolePerturbationToolkit/SpinWeightedSpheroidalHarmonics/blob/2d6d87fa88ede0cc7dc8eed849d6bea1a3bb2cf8/Kernel/SpinWeightedSpheroidalHarmonics.m#L154-L156
If the sparse matrix is large, Mma will convert it to dense for the computation of all e'vals. This is inefficient. Instead, we should ask it to only compute 1 e'val, whichever is closest to a certain guessed value. This avoids making the matrix dense, and it also gives a speedup because it uses sparse multiplies and doesn't attempt to compute all n e'vals.
I recommend adding an optional argument ClosestTo->(lambdaGuess) for the spectral method. If it is not specified, it should be taken to be the analytically-known e'val for the SWSphericalH of the same s,l,m.
With a guess lambdaGuess, finding the e'val that is closest in the complex plane involves computing the smallest e'val of shiftedMatrix = M - lambdaGuess * SparseIdentityMatrix[n], via
shiftedLambda = Eigenvalues[shiftedMatrix, -1];

and then shifting back the resulting e'val by lambdaGuess.

The choice of which eigenvalue is the "right" one is tricky and in some cases the existing code gets it wrong - see the plots in the most recent comment on issue #5. In particular, the eigenvalues sometimes swap over and certainly the "right" one might not be the closest to the spherical eigenvalue. As an alternative, we could compute an initial guess using a small matrix, and then refine it using iteration.

I think the eval/vec pair should probably be chosen to make them continuous with \gamma -- so maybe doing some low-precision walking from \gamma=0 to the desired \gamma in the complex plane, and then refining?

That would certainly be robust, but I wonder if it would be overly expensive? Also, what would happen in cases where two eigenvalues cross over?

It does sound expensive... But I think this is the type of approach taken by libraries like arb that try to guarantee correctness. It would help if we knew ahead of time where in the complex gamma plane are any potential e'val collisions. Are these branch points?


Drupal form elements

#1
This PR attempts to provide scrape_element.js to scrape renderable elements under web/core/lib/Drupal/Core/Render/Element and stores as element.json snippet for the ease of form/render element usage.
All elements rendered will automatically append properties from docblock of each file.
Additionally, FormElement will have four extra properties by default.
Gif demos are provided in README.md

Oh, wonderful! I'll review this ASAP!

I merged the refactor PR, so please make sure to merge those changes in here.

Refactored the scrape_element.js into index.js and separated into relative files by your clear comments!

Thanks for the refactor! I'll take a look at this soon.

Changes are made, thanks for giving such helpful suggestions.
Made changes so it's testable locally, awaiting for your responses, thanks 👍


Tests: refactor trainer dataloaders

What does this PR do?
Fixes # (issue).
PR review
Anyone in the community is free to review the PR once the tests have passed.
If we didn't discuss your PR in Github issues there's a high chance it will not be merged.
Did you have fun?
Make sure you had fun coding 🙃

Hello @Borda! Thanks for opening this PR.

In the file tests/base/eval_model_train_dataloaders.py:

Line 22:26: W292 no newline at end of file

In the file tests/base/eval_model_valid_dataloaders.py:

Line 16:65: W292 no newline at end of file

Do see the Hitchhiker's guide to code style

no idea why it fails here with
>   val_loss_mean = torch.stack([x['val_loss'] for x in outputs]).mean()
E   TypeError: list indices must be integers or slices, not str

but locally all pass fine...


Derive shared services namespace from Authentication

Addresses the following:

https://github.ibm.com/IBMPrivateCloud/roadmap/issues/57489
https://github.ibm.com/IBMPrivateCloud/roadmap/issues/57490

Changes:

Perform an intelligent guess on where the shared services namespace is based upon where the Operator is able to list an Authentication CR. There should only ever be one present among the namespaces the Operator can list. If no or more than one Authentication CR is found, an error is thrown as this is an unsupported usage of the Authentication CR.
Use the shared services namespace for lookups of ConfigMaps, Secrets, etc.
Revert use of namespace-to-ClientControllerConfig map to just a single ClientControllerConfig on ReconcileClient because the resources being looked up should always be in the shared services namespace, thus no longer requiring a map structure.
When handling URLs that the code knows are based upon Service names, always convert them to the format <Service name>.<Namespace of the Service>.svc.

/lgtm


Does ajv-cli tool validates the format type of specification.

Hello All,
Given below is the specification in which we have type and format of fields. I am able to validate the specification with type parameter with ajv-cli  but I also want to validate with format parameter. Is there a way to validate specification with format parameter?
"schema"{
"properties": {
"startDate": {
"type": "string",
"format": "date"
},
"storage": {
"type": "number",
"format": "float"
}
}

it supports the same as Ajv - so it does.


Add GUID to struct returned from individual response

Signed-off-by: Ross Pavitt<EMAIL_ADDRESS>

Spotted that the GUID set on the account isn't being added to the object returned from the Get call. You could argue that its known already, but I'm hoping it will make passing around the object easier.


Updated test template

Updated example as per #143 - this updated test template reflects the discussion from that issue around the structuring of unit tests for DSC resources, focusing on making them more scenario/functionality based as opposed to being based on the structure of the code.

This change is

Hi @BrianFarnhill, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by Microsoft and real humans are currently evaluating your PR.
TTYL, MSBOT;

Review status: 0 of 1 files reviewed at latest revision, 1 unresolved discussion.

Tests.Template/unit_template.ps1, line 75 [r1] (raw file):

        #TODO: Update the assertions below to align with the expected results of this state
        It "Should return a specific value in the get method" {

How about stating the name of the method first?
e.g. "Set method should return a specific value"
this way we don't need to read the whole "it" section to figure out what given test case is focusing on.

Comments from Reviewable

@nzspambot What type of example have you got in mind for that one? I haven't gotten around to adding localization support to my SharePointDsc module yet so I haven't got as much context of what would be useful to test in that regard.

Reviewed 1 of 1 files at r2.
Review status: all files reviewed at latest revision, 1 unresolved discussion.

Tests.Template/unit_template.ps1, line 75 [r1] (raw file):
Previously, KarolKaczmarek wrote…
> How about stating the name of the method first?
> e.g. "Set method should return a specific value"
>
> this way we don't need to read the whole "it" section to figure out what given test case is focusing on.

Done.

Comments from Reviewable

@nzspambot So you're essentially looking to test that the expected verbose messages came out as well as the result then, yea? @KarolKaczmarek your thoughts?
Personally I'm not convinced on that - if I'm getting the desired output from the input then I'm happy with the code path I'm testing there. What value does checking the verbose messages add? Happy to be convinced if there is something I'm missing here though - just wanted to have the discussion

There are a number of cases I've seen where the only way to see if you've hit the right block of if statements is to check the verbose message, as it just returns a value otherwise. xWebApplication has a number of examples of this.

Another area where you may want to check against localized message are negative test cases and verifying we got the expected error message (another approach is to check for FullyQualifiedErrorId).
That being said, I'm not sure we should cover all these cases in template. I'd keep them as lightweight and possible and place examples how to test localized data etc. here: https://github.com/PowerShell/DscResources/blob/master/TestsGuidelines.md
We could mention this link in the template though.

I'd definitely like to see a section added to the Test Guidelines that we can add common test patterns to and keep the template as slim as possible. The common test patterns should also link to the real examples in other DSC repos where possible. The templates are looking good @BrianFarnhill.

Review status: all files reviewed at latest revision, 1 unresolved discussion.

Comments from Reviewable

Review status: all files reviewed at latest revision, 1 unresolved discussion.

Comments from Reviewable

Reviewed 1 of 1 files at r2.
Review status: all files reviewed at latest revision, 1 unresolved discussion.

Comments from Reviewable


Fixed comment

Wrong return value in comment

Thank you for noticing the issue.  Fixed in the commit above (note that OpenSSL states that all values except one indicates an error).


Automatic URI generation

As per the documentation it's possible to automatically generate URIs for documents inserted by leaving out the 'uri' property of the document descriptor.
Where can I find the code that does the automatic URI generation?

Either the uri or the extension property are required (but not both).  The extension identifies the document format and mime type via the mime type mapping table (which can be viewed through the Admin UI on the server).

so let's assume that I specify the extension property to be 'json'. How does the actual URI get generated?

The suffix is appended to a random unsigned long.  If the optional directory property is also set, that's prepended.

By the way, it would be a reasonable request to use sem:uuid() instead of the random long.  There may even be an RFE on the REST API for that option.

So is this random unsigned long generated via some code in the REST API or by the Node.js API?

In the REST API.  That's the usual approach so that different clients don't have to reimplement the same features in a consistent way.

Thanks @ehennum 🥇


Set RntbdContextRequest request time out

In Rntbd layer, generally we have 5-10s time limit for each request. But there is no time limitation on how long we will spend on RntbdContextRequest.

This is done.


Let there be scroll bars

Description
Add Scrollbars to Gallery. Most scrollables without sliver navigations now have a scroll bar. All  changes are trivial. The GalleryOptionsPage now has a hard coded bottom padding.
Related Issues
#5525
Tests
No tests yet.
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I signed the CLA.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I updated/added relevant documentation (doc comments with ///).
[x] All existing and new tests are passing.
[x] The analyzer (flutter analyze --flutter-repo) does not report any problems on my PR.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Does your PR require Flutter developers to manually update their apps to accommodate your change?

[ ] Yes, this is a breaking change (Please read Handling breaking changes). Replace this with a link to the e-mail where you asked for input on this proposed change.
[x] No, this is not a breaking change.

I was a bit concerned about unconditionally adding scrollbars everywhere, but that seems to be OK.


camera: Cleanup memory handling of CameraData

I was starting to look into the Camera plugin to understand a bit of segfaults, and I noticed that the handling of data in CameraData was a bit complex. In particular, the only used CameraData instance was a local variable of the gzyarp::Camera gz-sim plugin, that then is passed via the setCameraData function to gzyarp::CameraDriver YARP device, where the m_imageBuffer  buffer was allocated. To simplify everything, I just moved all the initialization to a init method of CameraData, and I called init in the method of  gzyarp::Camera gz-sim plugin.

Merging 🚀


"Dămienești", "Călugăreni", "Drăgești", "Pădureni" are villages in "Dămienești " common, "BACĂU" county

Description : When user is typing the villages in the search bar, the locations should appear in the dropdown menu correctly spelled and with localities information (as in Law nr. 2 din 16 februarie 1968)
Preconditions : the website (https://peviitor-ro.github.io/ui.orase/) is up and runing.
After every step delete the text you typed from the search bar.

Steps 1: Click on the search bar and write "Dămienești "
Expected result: In the drop down menu you find "Dămienești , Bacău"

Steps 2: Click on the search bar and write "Călugăreni"
Expected result: In the drop down menu you find "Călugăreni, Bacău (Dămienești )"

Steps 3: Click on the search bar and write "Drăgești"
Expected result: In the drop down menu you find "Drăgești, Bacău (Dămienești )"

Steps 4: Click on the search bar and write "Pădureni"
Expected result: In the drop down menu you find "Pădureni, Bacău (Dămienești)"


[boost-core:x64-linux] build failure

Operating system
linux
Compiler
G++ 10
Steps to reproduce the behavior
Host Environment:
- Azure Pipelines
  - Pool: Azure Pipelines
  - Image: ubuntu-20.04
  - Agent: Hosted Agent
- Initialization:
  - Agent name: 'Hosted Agent'
  - Agent machine name: 'fv-az95-189'
  - Current agent version: '3.239.1'
  - Operating System
  - Runner Image
  - Runner Image Provisioner
  - Current image version: '20240526.1.0'
  - Agent running as: 'vsts'
  - Prepare build directory.
  - Downloading task: CMake (1.231.1)
  - Downloading task: ArchiveFiles (2.231.1)
  - Downloading task: UniversalPackages (0.239.1)
- vcpkg.json

{
  "$schema": "https://raw.githubusercontent.com/microsoft/vcpkg/master/scripts/vcpkg.schema.json",
  "name": "redacted",
  "version": "redacted",
  "dependencies": [
    "xerces-c",
    "boost-uuid"
  ]
}

### Failure logs

[DEBUG] Feature flag 'binarycaching' unset
[DEBUG] Feature flag 'compilertracking' unset
[DEBUG] Feature flag 'registries' unset
[DEBUG] Feature flag 'versions' unset
[DEBUG] Feature flag 'dependencygraph' unset
[DEBUG] Detected CI environment: Azure_Pipelines
[DEBUG] 1000: execute_process(curl --fail -L https://github.com/boostorg/core/archive/boost-1.85.0.tar.gz --create-dirs --output /usr/local/share/vcpkg/downloads/boostorg-core-boost-1.85.0.tar.gz.3063.part)
[DEBUG] 1000: cmd_execute_and_stream_data() returned 0 after   351754 us
[DEBUG] Trying to hash /usr/local/share/vcpkg/downloads/boostorg-core-boost-1.85.0.tar.gz.3063.part
[DEBUG] /usr/local/share/vcpkg/downloads/boostorg-core-boost-1.85.0.tar.gz.3063.part has hash a1e774c98b537dd42018742af68f5889af1afd4e6c9809399987124e56a3b9ae2f20e8173170bc6fc0ad76fb95e2da7b9d385a4037e9ee12011fc02bf8f7e009
error: Failed to download from mirror set
error: File does not have the expected hash:
url: https://github.com/boostorg/core/archive/boost-1.85.0.tar.gz
File: /usr/local/share/vcpkg/downloads/boostorg-core-boost-1.85.0.tar.gz.3063.part
Expected hash: 3a7be75e52f5c20830fccb9e7391a1e4556ebb072e6324df95b1ba38bed46e24f4c9f27a62a1099eddc90f2ac1ede083f0c850e2dc27fd42375d028516f675b3
Actual hash: a1e774c98b537dd42018742af68f5889af1afd4e6c9809399987124e56a3b9ae2f20e8173170bc6fc0ad76fb95e2da7b9d385a4037e9ee12011fc02bf8f7e009
[DEBUG] /source/src/vcpkg/base/downloads.cpp(997):
[DEBUG] Time in subprocesses: 351754us
[DEBUG] Time in parsing JSON: 2us
[DEBUG] Time in JSON reader: 0us
[DEBUG] Time in filesystem: 333us
[DEBUG] Time in loading ports: 0us
[DEBUG] Exiting after 354 ms (353332us)

CMake Error at scripts/cmake/vcpkg_download_distfile.cmake:32 (message):

### Additional context

may be tied to vckpg version; builds locally under ubuntu 20-04 without issue.

Expected hash: 3a7be75e52f5c20830fccb9e7391a1e4556ebb072e6324df95b1ba38bed46e24f4c9f27a62a1099eddc90f2ac1ede083f0c850e2dc27fd42375d028516f675b3
Actual hash: a1e774c98b537dd42018742af68f5889af1afd4e6c9809399987124e56a3b9ae2f20e8173170bc6fc0ad76fb95e2da7b9d385a4037e9ee12011fc02bf8f7e009

Already fixed in https://github.com/microsoft/vcpkg/pull/38979/files#diff-ab4a5faa5d3d52b3dbb67c00dd34ef155b681084b2ae93703f45f8de0e568232.


The parameter 'std' keeps decreasing

Thanks for your amazing work!
I had some data tested with the HEMM method and the result of subgroup prediction is abnormal. With further evaluation, I found that the parameter 'std' of the Gaussian distribution keeps decreasing and fell below zero while it supposed to converge to a positive value.
What's the cause of this phenomenon and how can I fix it? Is this about parameter initialization?

Thank you for using causallib and taking the time to report this problem.
I'll admit models in the contrib module come with limited warranty, but I'll do my best to assist.
First, could you please provide a minimal code example that reproduces the problem?
Second, I'll ping @chiragnagpal (Hi Chirag! 👋 🙃 ), the paper's first author and the person who implemented the model, to see if he can find the time to take a look and see what he can make of it.

Thanks for your reply and help!
Here is the example code:

from causallib.contrib.hemm import HEMM
import causallib.contrib.hemm.gen_synthetic_data as gen_synthetic_data
import causallib.contrib.hemm.hemm_outcome_models as hemm_outcome_models
import numpy as np
import pandas as pd
def generate_traindata():
d = 100
X, T, Y, Z, mu1, mu0 = gen_synthetic_data.gen_data(n=50000, d=d)
HTE = mu1 - mu0
data = np.column_stack((X, Y, T, HTE, mu1, mu0, Z))

cols = ['x' + str(i) for i in range(d+1)]

output_train_data = pd.DataFrame(data, columns=cols + ['Y', 'T', 'HTE', 'Y0', 'Y1', 'Z'])
return output_train_data

train_data = generate_traindata()
hemm=HEMM(D_in=D_in,K=K,bc=bc,lamb=lamb,mu=mu,std=std,response=response,metric='AuROC',outcome_model=hemm_outcome_models.genMLPModule(D_in=D_in, H=2, out=2))
losses=hemm.fit(train_data.iloc[:,:100].values,train_data['T'].values,train_data['Y'].values)

K=2, batch_size = 30, other parameters remain default.
I think it may be a overfitting problem, but still don't know the reason that 'std' kept decreasing and fell below zero.

Hi @ehudkr thanks for connecting me.. ! It's been good, just trying to finish wrapping up my thesis and move on to newer things : )
@R2Bb1T I looked at the code, and it indeed seems like the model isn't constraining the std variable to be positive. One simple way around this  is to add a relu activation on the std parameter.
I can try pushing that fix to the code, but its prolly faster for you to fix it at your end before waiting for it to be reflected on the next release.

@chiragnagpal Thanks for your reply! The solution you mentioned worked! The result of ITE seems right. But the result of subgroup predicted does not match the truth subgroup very well. I kept the data d=2 to simulate the experiment in the paper. And I found that even all the hyperparameters are kept same and randomly generate similar data to experiment several times, the result can have a huge difference. The visualization varies a lot, sometimes even cannot form a circle, if do the center and the radius seems deviated, and I calculated the AUC between subgroup 'z' predicted and the truth 'Z', it also varies from 0.5 to 0.7. Is any procedure I conducted wrong or it does have the problem?
This time I change the d to 2, n to 1000, batchsize to 10, lamb to 0.1 and the threshold of posen and negen to 0.25.


kubectl error for large cluster

I set up large cluster (90 nodes). Few times I've noticed the following error:
$ ./cluster/kubectl.sh get pods
current-context: "ivory-honor-865_e2e-test-filipg"
Running: ./cluster/../cluster/gce/../../cluster/../_output/dockerized/bin/linux/amd64/kubectl get pods
F0327 12:25:31.913390   26283 get.go:70] Error: Get http://<IP_ADDRESS>:4001/v2/keys/registry/pods/default?quorum=false&recursive=true&sorted=true: dial tcp <IP_ADDRESS>:4001: cannot assign requested address
@wojtek-t observer similar error for 50-node cluster. I haven't seen for small ones.

This is not related to kubectl - while running soem performance tests, I'm observing that error in Kubelet logs, e.g.:
E0327 13:41:40.496608    4712 kubelet.go:1729] error updating node status, will retry: error getting node "e2e-test-wojtekt-minion-0jxm.c.groovy-sentry-504.internal": Get http://<IP_ADDRESS>:4001/v2/keys/re
gistry/minions/e2e-test-wojtekt-minion-0jxm.c.groovy-sentry-504.internal?quorum=false&recursive=false&sorted=false: dial tcp <IP_ADDRESS>:4001: cannot assign requested address

BTW, the 4001 port suggests this might be a problem with etcd

As @fgrzadkowski suggested, this issue might be related to saturated network within a single machine (connections between etcd and apiserver). Decreasing the load on etcd from apiserver might help (e.g. #6349)

Has this problem gone away? If so, please close this issue.

Please run 'kubectl clusterinfo' when you run into this issue? I noticed that the context you have is talking about "ivory-honor-865_e2e-test-filipg", but sounds like you tried to retrieve information from different cluster: much larger ones?

This is still happening:
wojtekt@wojtekt-work:~/kubernetes$ cluster/kubectl.sh get rc
current-context: "groovy-sentry-504_e2e-test-wojtekt"
Running: cluster/../cluster/gce/../../cluster/../_output/dockerized/bin/linux/amd64/kubectl get rc
Error: Get http://<IP_ADDRESS>:4001/v2/keys/registry/controllers/default?quorum=false&recursive=true&sorted=true: dial tcp <IP_ADDRESS>:4001: cannot assign requested address

A while back I put a lot of effort into creating large GCE clusters (1,000 nodes) and this resulted in many PRs (I worry that some of them are being undone without due concern about large clusters). So I think the issues around bringing up large clusters is something we need to keep an eye on and keep constantly ensuring is working. I had adapted how we assigned IP addresses to ensure nodes in large clusters got good IP addresses.

@dchen1107 My e2e cluster was just much larger (if it's name that suggests that it's small)
@satnam6502 I agree that we should be careful, but for now let's only make sure that 100 node cluster works, not 1'000.

@fgrzadkowski : OK. It's a shame to throw away the work I did for big clusters.

@satnam6502 : I agree! That's why I said we should be careful :) But I don't think it's reasonable to expect that everybody will test something on 1000 node cluster. Let's just look for PRs that can break what you have fixed!

Well some of what I "fixed" has been obsoleted anyway (for the better) e.g. by the move to M.I.G. The remaining things relate to routes, firewalls, IP allocation, rate of calls to the apiserver etc.

@fgrzadkowski and @wojtek-t say this is fixed.

I am sorry, so how does this issue actually fixed?