Sort list of required attributes in JSON schema

We are fetching the JSON schema from OCSF server and committing it to a repo. The order of the required fields changes randomly, generating  unnecessarily large diffs.

Nice catch. I think this makes sense to do and looks good to me, but would also like @floydtree and @rmouritzen-splunk to take a look. Also along with this the server version should be updated in mix.exs following Semver guidelines.


Is there a way to access a clipped svg image to save back to the device in react-native using react-native-svg

I have a static image that i crop with a circle and now i want to save this modified image in my device's gallery, does anyone know how to do that? I am using TypeScript.
import React from "react";
import { SafeAreaView } from "react-native";
import Svg, { Circle, ClipPath, Defs, Image } from "react-native-svg";

export const App = () => {
    return (
        <SafeAreaView>
            <Svg>
                <Image
                    href={require('./apple.jpg')}
                    clipPath="url(#clip)" />
                <Defs>
                    <ClipPath id="clip">
                        <Circle cx="150" cy="150" r="150" />
                    </ClipPath>
                </Defs>
            </Svg>
        </SafeAreaView>
    );
}

export default App;

So far this solution seems to work involving another library: react-native-view-shot
import React, { RefObject, useRef } from "react";
import { Button, SafeAreaView } from "react-native";
import Svg, { Circle, ClipPath, Defs, Image } from "react-native-svg";
import ViewShot, { CaptureOptions, captureRef } from "react-native-view-shot";
import RNFetchBlob, { Encoding } from "rn-fetch-blob";

const IMAGE_FOLDER = RNFetchBlob.fs.dirs.DownloadDir + "/signalstickers";

export const App = () => {
    const viewShotRef = useRef() as RefObject<ViewShot>;

    const captureAndSaveImage = async (imageName: string, options: CaptureOptions) => {
        await RNFetchBlob.fs.mkdir("./test");
        await RNFetchBlob.fs.writeFile(IMAGE_FOLDER + "/" + imageName + "." + options.format, await captureRef(viewShotRef, options), "uri" as Encoding);
    }
    return (
        <SafeAreaView>
            <Svg>
                <Image
                    href={require('./apple.jpg')}
                    clipPath="url(#clip)" />
                <Defs>
                    <ClipPath id="clip">
                        <Circle cx="150" cy="150" r="150" />
                    </ClipPath>
                </Defs>
            </Svg>
            <Button title="Capture" onPress={() => captureAndSaveImage("pic1", { format: "jpg", quality: 1.0 })} />
        </SafeAreaView>
    );
}

export default App;

Thank you for your fast response, I will check this out :)


[WIP] Travis: Investigating macOS build problems

Download:
Environment: QT=5.12, ARCH=64, PYTHON_PATH=C:\Python35-x64
Environment: QT=5.12, ARCH=32, PYTHON_PATH=C:\Python35

Download:
Environment: QT=5.12, ARCH=32, PYTHON_PATH=C:\Python35
Environment: QT=5.12, ARCH=64, PYTHON_PATH=C:\Python35-x64


Long term support of jaxlie?

Hello! Me again. Again, this is a great library and thank you for responding to pull requests so promptly. I was wondering though its plans for long-term support, as I’m debating if I should keep this as a dependency in my library or write something home-cooked.

Hi Michael, thanks again for the nice words!
There are no new features planned, but I'm planning to support jaxlie indefinitely. I'm also using both this JAX version and ports to PyTorch + NumPy[^1] on an almost daily basis, so if you run into any issues I'd of course be very interested. 🙂
That said, jaxlie is fairly lightweight and building something yourself could make sense if you want to reduce dependencies, make breaking API changes, or are just looking for an exercise. In terms of the API, there are certainly design details that I've gone back and forth on, like:

Putting parameters for Lie groups into a single vector always. There are usability advantages of the current system, but a lot of operations could be more efficient if wxyz_xyz in SE3(), for example, was instead two vectors (wxyz, translation) or seven (qw, qx, qy, qz, tx, ty, tz).
Making each Lie group a dataclass. I could see some things like vmap usage and/or general batching within the PyTree to be confusing for people.
Related: whether the Lie group abstraction is a good idea to begin with. It's second nature for a lot of us, but I think a convincing argument could be made for those who don't care as much about the math that it ends up introducing a lot of unnecessary jargon to common transformation operations. If you want to convert a quaternion to its axis-angle representation, for example, our current syntax is axis_angle = SO3(quat).log(). We could also just implement a function like axis_angle = compute_axis_angle_from_quaternion(quat) or Rotation3.from_quat(quat).to_axis_angle().

Overall I think the specific decisions we made for these things were correct given my personal priorities, but I can see other folks having different ones!
[^1]: the PyTorch version isn't public, but there's a NumPy port that we're depending on pretty heavily for some visualization tooling: https://viser.studio/transforms/

Thank you for answering this, this is all helpful to know.

In my case, I’ve been wrapping the SO(3) object into something somewhat akin to the SE(3)  object. On this note, are you no longer than thinking about implementing #7? For my case, I may be interested in parameterizing by Euler angles and only batching over one angle. It would save some memory to do this.
Batching within the PyTree does seem to be a learning curve within JAX, but my mindset has become that it’s part of the functional programming learning curve. I’ve built my package on equinox, which has some really nice utilities for batching.
I hear this, the Lie group abstraction makes sense for this package I think! The objects I’ve been writing avoid the Lie group abstraction because it does not align with conventions of how people think about rotations in my field.

As for feature requests, if we go forward with this as a dependency at some point I would consider a PR that allows for utilities for using different euler angle conventions. However, this is easy enough to do with the package as is.
Thanks again!

Hi sorry for the late reply, this got buried by holiday things...
Yeah, I don't have plans to implement #7. This would be a breaking API change, the ergonomics of having several floating arrays now feels worse to me, and I don't think the hypothetical + likely small runtime benefits would be worth the effort.
If you have features you want I'm happy to review PRs! Euler angles are frowned upon in most of the circles I find myself in (because of singularities, the illusion of interpretability, too many ordering conventions, etc), but I do think incorporating more support for them makes sense for this package. 😛

Great, thanks for the update and no problem on the delay. The help is much appreciated!


typescript typings for vr don't match webvr-api

Babylon.js typescript typings for VR don't match with those in webvr-api, leading to failed complication of typescript programs that include both babylon.js and threejs (which brings in webvr-api typings) via npm.
See:
https://www.npmjs.com/package/@types/webvr-api
which is exported from:
https://github.com/DefinitelyTyped/DefinitelyTyped/tree/master/types/webvr-api
and imported with the "three" npm package.
The problematic typings are:
// babylon.js
declare var VRDisplay: any;

vs
// webvr-api
declare var VRDisplay: {
    prototype: VRDisplay;
    new(): VRDisplay;
};

and
// babylon.js
interface Navigator {
    getVRDisplays: () => any;
    ...
}

vs
// webvr-api
interface Navigator {
    getVRDisplays(): Promise<VRDisplay[]>;
    readonly activeVRDisplays: ReadonlyArray<VRDisplay>;
    ...
}

Good catch!
I'll fix it for next commit


[WIP] opengl example for the DK2

copied examples/opengl to examples/opengl-DK2, changed some hardcoded resolution constants and changed the shader so it looks quite nicely with my DK2.
still to be done:

automatic detection of the model
retrieval of the distortion coefficients instead of hardcoding them
fix build, it doesn't build on my machine when I do not tinker with the Makefile manually

Closing after recent modernization of the opengl example for all devices.


clusterwide not really working

I added clusterwide option and deployed the example to kube-system - but the operator never reacted to the example-etcd-cluster.yaml.. I then removed all (incl. roles that were in kube-system) and added it in default namespace - and then it worked.
It seems it could be because the create_role script creates a role that does not have access to other namespaces ?

Link to the create role script?

Link to the create role script?
https://github.com/coreos/etcd-operator/blob/master/example/rbac/create_role.sh

u should add the clusterwide annotation too

So how should this actually look, for it to work? https://github.com/coreos/etcd-operator/blob/master/example/deployment.yaml


Add-in could not be installed on MonoDevelop 5.7

I followed the instructions from
http://indomitable.github.io/monodevelop-tfs-addin/
Selected the following URL for repository:
http://indomitable.github.io/monodevelop-tfs-addin/5.0/
After clicking in Install, the following error appeared:

Same issue

"sudo apt-get install monodevelop-versioncontrol" solved this problem for me.


[SPARK-4194] [core] Make SparkContext initialization exception-safe.

SparkContext has a very long constructor, where multiple things are
initialized, multiple threads are spawned, and multiple opportunities
for exceptions to be thrown exist. If one of these happens at an
innoportune time, lots of garbage tends to stick around.
This patch re-organizes SparkContext so that its internal state is
initialized in a big "try" block. The fields keeping state are now
completely private to SparkContext, and are "vars", because Scala
doesn't allow you to initialize a val later. The existing API interface
is kept by turning vals into defs (which works because Scala guarantees
the same binary interface for those).
On top of that, a few things in other areas were changed to avoid more
things leaking:

Executor was changed to explicitly wait for the heartbeat thread to
stop. LocalBackend was changed to wait for the "StopExecutor"
message to be received, since otherwise there could be a race
between that message arriving and the actor system being shut down.
ConnectionManager could possibly hang during shutdown, because an
interrupt at the wrong moment could cause the selector thread to
still call select and then wait forever. So also wake up the
selector so that this situation is avoided.

Note the PR contains the commits from #5311. I hope once that is pushed that github will figure things out. (If not I'll rebase manually.)
With both of these PRs, I was able to run the core/ unit tests and verify that:

no executor allocator threads were left behind
no driver heartbeater threads were left behing
no "AkkaAskTimeout" exceptions show up in the unit test logs

I tried to run MiMA checks locally and they look ok, but let's see what jenkins says.

Test build #29621 has started for   PR 5335 at commit 8caa8b3.

Test build #29621 has finished for   PR 5335 at commit 8caa8b3.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.
This patch does not change any dependencies.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/29621/
Test PASSed.

Oops, borked merge, fixing...

Test build #29684 has started for   PR 5335 at commit c671c46.

Test build #29684 has finished for   PR 5335 at commit c671c46.

This patch fails Scala style tests.
This patch merges cleanly.
This patch adds no public classes.
This patch does not change any dependencies.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/29684/
Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/29683/
Test FAILed.

Jenkins, retest this please.

Test build #29685 has started for   PR 5335 at commit 6b73fcb.

Test build #29686 has started for   PR 5335 at commit 6b73fcb.

Test build #29685 has finished for   PR 5335 at commit 6b73fcb.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.
This patch does not change any dependencies.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/29685/
Test FAILed.

Test build #29692 has started for   PR 5335 at commit 2621609.

Test build #29686 has finished for   PR 5335 at commit 6b73fcb.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.
This patch does not change any dependencies.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/29686/
Test PASSed.

Test build #29692 has finished for   PR 5335 at commit 2621609.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.
This patch does not change any dependencies.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/29692/
Test PASSed.

Test build #29764 has started for   PR 5335 at commit 408dada.

Test build #29764 has finished for   PR 5335 at commit 408dada.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.
This patch does not change any dependencies.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/29764/
Test PASSed.

Ping?

I tend to trust your hand on this. This is a big change and it's hard to match up the existing logic with new logic in the diff, though it looks like that was the intent and effect from spot-checking some elements. That the tests have passed several times is a good sign. The reorganization is significantly positive since the fields, and their initialization, are clearly grouped now.
One minor style thing, why the _member naming syntax, where it's not otherwise used in Spark? just to make it crystal clear here what's a member?
I think the additional changes look reasonable, like using a Java Executor in the, well, Executor.
I favor this though weakly on the grounds that I'm mostly relying on tests for correctness. The intent is sound. @rxin @pwendell do you have any thoughts on this one?

One minor style thing, why the _member naming syntax

That's actually used in lots of places in Spark. It's used when some variable / field name conflicts with a def, which is the case in this change.

OK right, and that's true of all of them here.

ping @JoshRosen - I think he's proposed this exact change to me in the past.

See the description of PR #3121 for my previous discussion of this.  If we want to avoid introducing vars in SparkContext, one alternative would be to move the creation of these components outside of the SparkContext constructor and into a method in the companion object, putting the try / catch blocks there.  This lets us isolate the mutability into a single method, so things are mutable while we're constructing SparkContext but are immutable once the object has been fully constructed.  I'm not sure whether this approach would be easier or harder to understand than the mutable vars + getters used here.

That aproach would be a lot more complicated. The first reason why it would be complicated is that you'd need an uber-constructor in SparkContext that takes all the initialized internal values. Unless there's some fancy Scala feature I'm not aware of, that in itself is scary as hell, and would mean the other constructors would be similarly ugly in that they'd have to call the companion object.
It would also cause (even more) duplication of the declaration of these things, since they'd have to be declared in the companion object's method too.
Finally, it would complicate stop(), because it would have to either be copy & pasted into the companion object so that it cleans up after an exception, or you'd need a stop() method in the companion object that takes all arguments as parameters, so that the SparkContext class can call it.
So while I would love to simplify the code in SparkContext, the alternative suggestion, as far as I can see, does nothing towards that.
And that's why I chose private vars. It's not optimal, and I really wish Scala would allow me to initialize a val after its declaration, like Java does. But it's the easiest approach, and it doesn't expose any mutable SparkContext state that wasn't already exposes before.

I also feel that the current approach makes more sense than Josh's alternative. Changes to SparkContext get a lot of scrutiny during code review, so clear documentation, IMO is sufficient to ensure this is followed correctly (famous last words). I didn't have time to dive into this to make sure there are no correctness issues, but the broad approach looks good to me... I think it's worth fixing this up.

Seems like there's support for this change, though needs a rebase @vanzin . Any objections to proceeding after that?

Test build #30260 has started for   PR 5335 at commit 80fc00e.

Test build #30260 has finished for   PR 5335 at commit 80fc00e.

This patch fails to build.
This patch merges cleanly.
This patch adds no public classes.
This patch removes the following dependencies:

RoaringBitmap-0.4.5.jar
activation-1.1.jar
akka-actor_2.10-2.3.4-spark.jar
akka-remote_2.10-2.3.4-spark.jar
akka-slf4j_2.10-2.3.4-spark.jar
aopalliance-1.0.jar
arpack_combined_all-0.1.jar
avro-1.7.7.jar
breeze-macros_2.10-0.11.2.jar
breeze_2.10-0.11.2.jar
chill-java-0.5.0.jar
chill_2.10-0.5.0.jar
commons-beanutils-1.7.0.jar
commons-beanutils-core-1.8.0.jar
commons-cli-1.2.jar
commons-codec-1.10.jar
commons-collections-3.2.1.jar
commons-compress-1.4.1.jar
commons-configuration-1.6.jar
commons-digester-1.8.jar
commons-httpclient-3.1.jar
commons-io-2.1.jar
commons-lang-2.5.jar
commons-lang3-3.3.2.jar
commons-math-2.1.jar
commons-math3-3.1.1.jar
commons-net-2.2.jar
compress-lzf-1.0.0.jar
config-1.2.1.jar
core-1.1.2.jar
curator-client-2.4.0.jar
curator-framework-2.4.0.jar
curator-recipes-2.4.0.jar
gmbal-api-only-3.0.0-b023.jar
grizzly-framework-2.1.2.jar
grizzly-http-2.1.2.jar
grizzly-http-server-2.1.2.jar
grizzly-http-servlet-2.1.2.jar
grizzly-rcm-2.1.2.jar
groovy-all-2.3.7.jar
guava-14.0.1.jar
guice-3.0.jar
hadoop-annotations-2.2.0.jar
hadoop-auth-2.2.0.jar
hadoop-client-2.2.0.jar
hadoop-common-2.2.0.jar
hadoop-hdfs-2.2.0.jar
hadoop-mapreduce-client-app-2.2.0.jar
hadoop-mapreduce-client-common-2.2.0.jar
hadoop-mapreduce-client-core-2.2.0.jar
hadoop-mapreduce-client-jobclient-2.2.0.jar
hadoop-mapreduce-client-shuffle-2.2.0.jar
hadoop-yarn-api-2.2.0.jar
hadoop-yarn-client-2.2.0.jar
hadoop-yarn-common-2.2.0.jar
hadoop-yarn-server-common-2.2.0.jar
ivy-2.4.0.jar
jackson-annotations-2.4.0.jar
jackson-core-2.4.4.jar
jackson-core-asl-1.8.8.jar
jackson-databind-2.4.4.jar
jackson-jaxrs-1.8.8.jar
jackson-mapper-asl-1.8.8.jar
jackson-module-scala_2.10-2.4.4.jar
jackson-xc-1.8.8.jar
jansi-1.4.jar
javax.inject-1.jar
javax.servlet-3.0.0.v201112011016.jar
javax.servlet-3.1.jar
javax.servlet-api-3.0.1.jar
jaxb-api-2.2.2.jar
jaxb-impl-2.2.3-1.jar
jcl-over-slf4j-1.7.10.jar
jersey-client-1.9.jar
jersey-core-1.9.jar
jersey-grizzly2-1.9.jar
jersey-guice-1.9.jar
jersey-json-1.9.jar
jersey-server-1.9.jar
jersey-test-framework-core-1.9.jar
jersey-test-framework-grizzly2-1.9.jar
jets3t-0.7.1.jar
jettison-1.1.jar
jetty-util-6.1.26.jar
jline-0.9.94.jar
jline-2.10.4.jar
jodd-core-3.6.3.jar
json4s-ast_2.10-3.2.10.jar
json4s-core_2.10-3.2.10.jar
json4s-jackson_2.10-3.2.10.jar
jsr305-1.3.9.jar
jtransforms-2.4.0.jar
jul-to-slf4j-1.7.10.jar
kryo-2.21.jar
log4j-1.2.17.jar
lz4-1.2.0.jar
management-api-3.0.0-b012.jar
mesos-0.21.0-shaded-protobuf.jar
metrics-core-3.1.0.jar
metrics-graphite-3.1.0.jar
metrics-json-3.1.0.jar
metrics-jvm-3.1.0.jar
minlog-1.2.jar
netty-3.8.0.Final.jar
netty-all-4.0.23.Final.jar
objenesis-1.2.jar
opencsv-2.3.jar
oro-2.0.8.jar
paranamer-2.6.jar
parquet-column-1.6.0rc3.jar
parquet-common-1.6.0rc3.jar
parquet-encoding-1.6.0rc3.jar
parquet-format-2.2.0-rc1.jar
parquet-generator-1.6.0rc3.jar
parquet-hadoop-1.6.0rc3.jar
parquet-jackson-1.6.0rc3.jar
protobuf-java-2.4.1.jar
protobuf-java-2.5.0-spark.jar
py4j-<IP_ADDRESS>.jar
pyrolite-2.0.1.jar
quasiquotes_2.10-2.0.1.jar
reflectasm-1.07-shaded.jar
scala-compiler-2.10.4.jar
scala-library-2.10.4.jar
scala-reflect-2.10.4.jar
scalap-2.10.4.jar
scalatest_2.10-2.2.1.jar
slf4j-api-1.7.10.jar
slf4j-log4j12-1.7.10.jar
snappy-java-<IP_ADDRESS>.jar
spark-bagel_2.10-1.4.0-SNAPSHOT.jar
spark-catalyst_2.10-1.4.0-SNAPSHOT.jar
spark-core_2.10-1.4.0-SNAPSHOT.jar
spark-graphx_2.10-1.4.0-SNAPSHOT.jar
spark-launcher_2.10-1.4.0-SNAPSHOT.jar
spark-mllib_2.10-1.4.0-SNAPSHOT.jar
spark-network-common_2.10-1.4.0-SNAPSHOT.jar
spark-network-shuffle_2.10-1.4.0-SNAPSHOT.jar
spark-repl_2.10-1.4.0-SNAPSHOT.jar
spark-sql_2.10-1.4.0-SNAPSHOT.jar
spark-streaming_2.10-1.4.0-SNAPSHOT.jar
spire-macros_2.10-0.7.4.jar
spire_2.10-0.7.4.jar
stax-api-1.0.1.jar
stream-2.7.0.jar
tachyon-0.5.0.jar
tachyon-client-0.5.0.jar
uncommons-maths-1.2.2a.jar
unused-1.0.0.jar
xmlenc-0.52.jar
xz-1.0.jar
zookeeper-3.4.5.jar

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/30260/
Test FAILed.

Test build #30293 has started for   PR 5335 at commit 746b661.

Test build #30293 has finished for   PR 5335 at commit 746b661.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.
This patch does not change any dependencies.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/30293/
Test PASSed.


feat: port Tactic.Group

We implement group by alternating between a simp call and ring_nf (to normalize the exponents) until no progress is made.
To do this we also implement the tactic fail_if_no_progress, so that repeat (fail_if_no_progress (simp ... <;> ring_nf)) behaves as expected.

This looks good to me, and I checked locally that it works with some group invocations in #2143. Maybe it can be implemented more efficiently (e.g., because of the defeq checks in fail_if_no_progress), but I don't think we need to be particularly concerned with that right now. It seems to be fast enough.

I agree that we shouldn't worry about it before merging this tactic; I'll continue enhancing fail_if_no_progress on a separate branch, but, since it's alright for now, I don't think we should slow down the port and hold up group by perfecting it prior to merging.

Thanks :tada:
bors merge


Explicity enable route_sharing feature on tests that need it

previously tests were not stating they needed this feature enable, only a minimum version of capi
since the feature is not enabled by default, this adds a call to enable it
attempt to disable it after each test, however when tests run in parallel, this caused the tests to fight each other on turning it on and off.  Since a minimum capi version is checked and we don't have tests that explicitly test the feature being disabled, it seems ok to just leave it on

I checked the "allow edits and access to secrets by maintainers, so any approver should be able to push commits if they see small issues
Thank you for contributing to the CF CLI! Please read the following:

Please make sure you have implemented changes in line with the contributing guidelines
We're not allowed to accept any PRs without a signed CLA, no matter how small.
If your contribution falls under a company CLA but your membership is not public, expect delays while we confirm.
All new code requires tests to protect against regressions.
Contributions must be made against the appropriate branch. See the contributing guidelines
Contributions must conform to our style guide. Please reach out to us if you have questions.

Does this PR modify CLI v6, CLI v7, or CLI v8?
Modifies tests to make github actions integration test work better, so should be backported to all versions still under testing
Please see the contribution doc above or review Architecture Guide.
Description of the Change
Explicitly enable route_sharing feature on tests that need it
We must be able to understand the design of your change from this description.
Keep in mind that the maintainer reviewing this PR may not be familiar with or
have worked with the code here recently, so please walk us through the concepts.
Why Is This PR Valuable?
Will allow folks to just run the tests instead of configuring their cloud foundry install
What benefits will be realized by the code change? What users would want this change? What user need is this change addressing?
Why Should This Be In Core?
n/a
Explain why this functionality should be in the cf CLI, as opposed to a plugin.
Applicable Issues
List any applicable Github Issues here
#2326
How Urgent Is The Change?
As urgent as we want integration tests to work on github actions
Is the change urgent? If so, explain why it is time-sensitive.
Other Relevant Parties
@cloudfoundry/wg-app-runtime-interfaces-cli-approvers
@cloudfoundry/wg-app-runtime-interfaces-cli-reviewers
Who else is affected by the change?

I'm curious... didn't we have something for this in the integration tests setup? weren't we enabling some feature flags?

I'm curious... didn't we have something for this in the integration tests setup? weren't we enabling some feature flags?

The last commit here moves the enabling to a helper file which is also enabling the flags "diego_docker" and "service_instance_sharing" so adding "route_sharing" there seems to be consistent with what we've got

I'm curious... didn't we have something for this in the integration tests setup? weren't we enabling some feature flags?

The last commit here moves the enabling to a helper file which is also enabling the flags "diego_docker" and "service_instance_sharing" so adding "route_sharing" there seems to be consistent with what we've got

ohhhh Sorry!!! I didn't see which file was being modified!

One concern I thought of is "route_sharing" is not available on all versions of capi.  So trying to enable this feature on a version of capi where "route_sharing" does not exist could fail badly.  The original location in each test had a check on the version of capi so it was only being enable on a version of capi that support "route_sharing"


getrandom() initialization failed if run on boot

Steps to reproduce the problem:
I created a systemd startup file:
[Unit]
Description=Mitmproxy
After=network.target

[Service]
#Type=simple
Type=idle
EnvironmentFile=/etc/default/mitmproxy
User=mitmproxy
Group=mitmproxy
LimitNOFILE=32768
ExecStart=/usr/local/bin/mitmdump $DAEMON_ARGS --conf=$CONFFILE --cadir=/etc/mitmproxy

[Install]
WantedBy=multi-user.target

Whenever I reboot my ubuntu 16.04 VM, mitmdump runs, but throw error for SSL websites:
Cannot establish TLS with <IP_ADDRESS>:443 (sni: ssl.google-analytics.com): TlsException("SSL handshake error: Error([('', 'osrandom_rand_bytes', 'getrandom() initialization failed.')],)",)
If I log into ssh and restart the service manually it then runs fine.
System information
Ubuntu 16.04 - mitmproxy 1.0.2

This is more a general Python issue and unfortunately not directly related to mitmproxy.
Fur further information please see:
https://lwn.net/Articles/693189/
https://bugs.python.org/issue26839
You could try and experiment with systemd-random-seed.service in your service definition...

Same here,
https://discourse.mitmproxy.org/t/run-mitmdump-from-crontab-solved/373/4
I try with:
@reboot sleep 60 && script.sh
Still not working (didn't happen on mitm0.18, but 2.0)


Collect vulnerabilities and package references from cve-search (and/or via4cve)

At first the goal is to collect data exposed by the API of https://cve.circl.lu/
There is also a dump for via4 data available at https://www.cve-search.org/feeds/via4.json
Longer term, we should setup our own instance of cve-search instead of using the public site API.

Will we be exposing our  cve-search   at same  end point(proposed to  be removed by https://github.com/nexB/vulnerablecode/pull/177) ?
This actually needs  a lot more discussion as in  future we will  be adding graphql based api  and remove/keep the rest API.


Add a way to place given variable at a given address in flash at linker time

Is your feature request related to a problem? Please describe.
Impossibility right now to have two variables placed at the beginning (fixed address) and the end of the zephyr app code.
Describe the solution you'd like
The idea would be that a variable in the beginning of the program and another at the end would be used to verify its integrity when being sent to a device through an unreliable channel. If the first variable points towards an address that correctly contains a fixed magic word that is located at the very end of the image, then the app was sent in its entirety.
I would like to be able to place C variables at given addresses in the Linker at compile time so that the generated .hex already has those addresses with the correct values.
More specifically, I'd like to be able to do something like this in my C code:
PLACE_IN_SECTION("APP_END")   const uint32_t my_magic_value = 0x1234abcd;
PLACE_IN_SECTION("APP_START") const uint32_t my_magic_value_address = (uint32_t) &my_magic_value;

APP_END and APP_START would be defined in a linker or using a specific structure as follows:
SECTIONS
{
    .start_region 0x08000300:
  {
	KEEP(*(APP_START))
	. = ALIGN(4);
  } >FLASH
    .end_region :
  {
	KEEP(*(APP_END))
	. = ALIGN(4);
  } >FLASH
}

where APP_START would be forced at 0x08000300 (example fixed address at the beginning of the app) and APP_END would naturally go at the end of the app.
Describe alternatives you've considered
I tried to use the Code Relocation (https://docs.zephyrproject.org/latest/kernel/code-relocation.html) feature but with no success for a single 32-bits word of data to be put at the very beginning of the program space.

Have you seen how e.g. the NXP Kinetis series handles this? It needs a specific variable for configuration to be stored at a fixed address in flash: https://github.com/zephyrproject-rtos/zephyr/blob/main/soc/arm/nxp_kinetis/flash_configuration.c


AEM 6.5.5 + ACS commons 4.8.6 + core.wcm 2.11.0- I cannot get the Marketo form to show up in a published page

Any ideas why I cannot see the marketo form in the publisher but it's showing up in author.

In both author and publisher, I have configured the Marketo settings. (in both cases, the settings is stored in this path: /conf/myapp/settings/cloudconfigs/Marketo)

I looked around and found this piece of code (inside /apps/acs-commons/components/content/embeddable/marketo/marketo.html): . In author and publisher (using view-as-published), ${configMgr) is not null. BUT when viewing the page via the dispatcher, this variable is null/empty.

I looked at the publisher logs and found these entries. These only happens when viewing the published page via the dispatcher.
  02.06.2021 02:32:38.204 *ERROR* [<IP_ADDRESS> [1622601158058] GET /content/flinders/en/about.html HTTP/1.1] org.apache.sling.scripting.sightly.render.ObjectModel Cannot access method configuration on object com.adobe.acs.commons.marketo.impl.MarketoClientConfigurationManagerImpl@3e08245
  java.lang.reflect.InvocationTargetException: null
          at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
          at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
          at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
          at java.base/java.lang.reflect.Method.invoke(Method.java:566)
          at org.apache.sling.scripting.sightly.render.ObjectModel.invokeBeanMethod(ObjectModel.java:418) [org.apache.sling.scripting.sightly.runtime:<IP_ADDRESS>_4_0]
          at org.apache.sling.scripting.sightly.render.ObjectModel.resolveProperty(ObjectModel.java:128) [org.apache.sling.scripting.sightly.runtime:<IP_ADDRESS>_4_0]
          at org.apache.sling.scripting.sightly.render.AbstractRuntimeObjectModel.getProperty(AbstractRuntimeObjectModel.java:138) [org.apache.sling.scripting.sightly.runtime:<IP_ADDRESS>_4_0]
          at org.apache.sling.scripting.sightly.impl.engine.runtime.SlingRuntimeObjectModel.getProperty(SlingRuntimeObjectModel.java:32) [org.apache.sling.scripting.sightly:<IP_ADDRESS>_4_0]
          at org.apache.sling.scripting.sightly.render.AbstractRuntimeObjectModel.resolveProperty(AbstractRuntimeObjectModel.java:74) [org.apache.sling.scripting.sightly.runtime:<IP_ADDRESS>_4_0]
          at org.apache.sling.scripting.sightly.apps.acs__002d__commons.components.content.embeddable.marketo.marketo_html.render(marketo_html.java:93)
          at org.apache.sling.scripting.sightly.render.RenderUnit.render(RenderUnit.java:50) [org.apache.sling.scripting.sightly.runtime:<IP_ADDRESS>_4_0]
          at org.apache.sling.scripting.sightly.impl.engine.SightlyCompiledScript.eval(SightlyCompiledScript.java:60) [org.apache.sling.scripting.sightly:<IP_ADDRESS>_4_0]
          at org.apache.sling.scripting.core.impl.DefaultSlingScript.call(DefaultSlingScript.java:386) [org.apache.sling.scripting.core:2.0.56]
          at org.apache.sling.scripting.core.impl.DefaultSlingScript.eval(DefaultSlingScript.java:184) [org.apache.sling.scripting.core:2.0.56]
  --------- S N I P ------
          at java.base/java.lang.Thread.run(Thread.java:834)
  Caused by: java.lang.NullPointerException: null
          at com.adobe.acs.commons.marketo.impl.MarketoClientConfigurationManagerImpl.getConfiguration(MarketoClientConfigurationManagerImpl.java:56) [com.adobe.acs.acs-aem-commons-bundle:4.8.6]
          ... 386 common frames omitted

Any ideas on how to fix?? Thanks

@mrjayviper What file exactly are you seeing ${configMgr} go null in?

@mrjayviper What file exactly are you seeing ${configMgr} go null in?

I found the code inside /apps/acs-commons/components/content/embeddable/marketo/marketo.html
I also added some log entries I found on the publisher logs. This only shows up when viewing the published page via the dispatcher.

@mrjayviper What file exactly are you seeing ${configMgr} go null in?

I found the code inside /apps/acs-commons/components/content/embeddable/marketo/marketo.html
I also added some log entries I found on the publisher logs. This only shows up when viewing the published page via the dispatcher.

I am facing the exact same problem. Form is coming in author and publisher but when viewing via dispatcher, it is not showing up.
Did you find the solution ? could you please share it here

Hi All,
Any idea why the Marketo embed form is not showing up in Publisher but it is showing in Author. Please let me know if any solution. We are getting this below error.

Thanks,
Jeeva

@JeevaRavi25 ,
I did not find any solution to fix it. I customize and created a OSGI service to save the Marketo details and then fetch the marketo details in form.


Migrate marker display

Closes #100
This PR also adds a considerable amount of tests that are disabled for now.
It introduces convenience methods for scene graph introspection.

I'd say we wait for CI until the Pose-Display is sorted out (we'll need to rebase then anyway).

CI after rebase on master:

Linux
Linux-aarch64
macOS
Windows

I updated my repos file and everything, let's see what hapens:

Linux
Linux-aarch64
macOS
Windows

I have no idea why the build ignores the common_interfaces repository. How can we fix this @wjwwood ?

@Martin-Idel-SI we disable several of the message packages so to save time in the builds, I'll have to remove that one:
https://github.com/ros2/ci/blob/d06052d66c407722e469b04184e05d32b20f24d7/ros2_batch_job/__main__.py#L77-L88

Until that is merged, you can use reenable_visualization_msgs in the CI_SCRIPTS_BRANCH field of the Jenkins jobs.

Thanks! So I tried that, let's see where we're at:

Linux
Linux-aarch64
macOS
Windows

Next try:
Building on master in workspace /var/lib/jenkins/jobs/ci_launcher/workspace

Linux
Linux-aarch64
macOS
Windows

Windows warning is a false positive, so CI looks good, let me do a code review real quick.

It would be good, in the future, to migrate some of the "acceptance tests" from rviz, like these:

https://github.com/ros2/rviz/blob/8dfb0c0c752cc4a2756eed931c57703cbeb50317/rviz/src/test/mesh_marker_test.cpp#L1
https://github.com/ros2/rviz/blob/8dfb0c0c752cc4a2756eed931c57703cbeb50317/rviz/src/test/marker_test.cpp#L1

https://github.com/ros2/rviz/blob/8dfb0c0c752cc4a2756eed931c57703cbeb50317/rviz/src/test/marker_array_test.py#L1


Re-save all JS files now that we have Prettier

Should help out with PRs in the future.

@scooooooooby  Should this only include files within /react-components or throughout the whole system?

@M0nica Just within react-components for now, I think.

This was resolved when the component file structure was flattened in https://github.com/NYPL/nypl-design-system/pull/174


Parameter orderNumber doesn't seem to exist

I tried the code from the readme
$redsysRequest = RedsysRequest::create(
    $redsysClient,
    new RequestParameters(
        amountInCents: 123_45,
        orderNumber: '22013100005',
        currency: Currency::EUR,
        transactionType: TransactionType::Autorizacion,
        merchantUrl: 'https://example.com/redsysNotification',
        urlOk: 'https://example.com/paymentOk',
        urlKo: 'https://example.com/paymentKo',
    )
);

Creagia\Redsys\Support\Signature::calculateSignature(): Argument #2 ($order) must be of type string, null given, called in [...]]\vendor\creagia\redsys-php\src\RedsysRequest.php on line 68
https://github.com/creagia/redsys-php/blob/9135739b1e4956991e9d407866de4866388bc29d/src/Support/RequestParameters.php#L35-L37
It seems to be just order.

Created PR #29

Thanks for the PR!


Scheduler back to the main library?

With all these things that pop up over and over again like:

(double) renew
renew subscription upon the trial finishes
change plan (downgrade / upgrade) upon next renewal

I thought maybe it would be a good idea to make again a scheduler an integral part of this package?
But for wider compatibility I would resign from job batches. Actually, I would even resign from jobs altogether. Why not just make the implementation in the command ($schedule->command(...)->everyMinute()->runInBackground();)? Commands already run in background and using jobs seems to me superfluous.  This would also demand less requirements - just running the scheduler and no need to configure queue drivers or worker.
For example, upon issuing a renewal, the task could be saved into plan_subscription_schedules to be run upon ends_at (or trial_ends_at) and scheduler will pick it up upon right moment and make appropriate changes.

Just the other day I was thinking this, that I should revisit the schedule package and see if I can drop the job batch and bring it back here again here, since it's very useful.


Add table github_outside_collaborators

Steampipe already supports querying regular members of an organization through the github_organization_member table. There is another way people can access an arganization's repositories: as outside collaborator.
An outside collaborator is a person who is not a member of an organization, but has access to one or more of the organization's repositories
It would be very nice if we could query these as well using Steampipe.
References
The API documentation to query outside collaborators for an organization is here: https://docs.github.com/en/rest/orgs/outside-collaborators?apiVersion=2022-11-28

Thank you, @olafz, for bringing this matter to our attention 👍. We'll investigate it further.

Hello, @olafz, upon reviewing our current plugin code, I noticed we already have a github_repository_collaborator table. This should help us gather information on collaborators (OUTSIDE, DIRECT, and ALL) for a repository. You can specify the type of collaborators you're interested in by using the affiliation option in the where clause, with ALL being the default setting. For example: select * from github_repository_collaborator where repository_full_name = 'turbot/steampipe-plugin-aws' and affiliation = 'outside'. When querying github_repository_collaborator, providing the repository_full_name is mandatory, while specifying the affiliation parameter is optional (with possible values: direct, outside, and all).
Please try this approach and let me know if it's beneficial.
Thank you!

Hi @ParthaI,
Thanks for your feedback. Your suggestion of using the github_repository_collaborator does work, but I have an use case which makes using this table extremely inefficient.
I would like to create an overview of all external collaborators in a GitHub organization. If I were to use the github_repository_collaborator table, I would have to create a query that iterates (join) over all repositories in the GitHub organization (since repository_full_name is a mandatory parameter).
In our case - because of the size of the organization - that would mean going over thousands and thousands of repositories, where most of them don't even have an outside collaborator. It would be much more efficient if I were able to do it the other way around: first query for all outside collaborators and then go from there.
Thanks again for looking into this issue and I hope this creates some clarity. Also: if there is another way to achieve this efficiently, let me know :)

Hello, @olafz, I was trying to achieve the goal as per the comments provided in the in the issue description.
However, I am facing a few issues while achieving the goal.
I found some articles and community discussion related to my issue. I will give it a try once more.
I constructed a query to get outside collaborators at the organization level. Could you please give it a try in Github GraphQL Explorer and share your feedback with us if the following GraphQL query works as expected?
    query {
        organization(login: "<YOUR ORG NAME>") {
            url
            login

            repositories(first: 100) {
                pageInfo {
                    endCursor
                    hasNextPage
                }

                nodes {
                    name
                    collaborators(affiliation: OUTSIDE) {
                        edges {
                            permission
                        }
                        nodes {
                            url
                            login
                            name
                            email
                            company
                        }
                    }
                }
            }
        }
    }

Thank You!

Hello @ParthaI,
Thanks for the follow-up. I tested the query you provided and it seems to do the job; it looks pretty good.
See below the example from the first response for one repository with external collaborators:
{
  "data": {
    "organization": {
      "url": "https://github.com/<YOUR ORG NAME>",
      "login": "<YOUR ORG NAME>",
      "repositories": {
        "pageInfo": {
          "endCursor": "...",
          "hasNextPage": true
        },
        "nodes": [
          {
            "name": "<REPOSITORY NAME>",
            "collaborators": {
              "edges": [
                {
                  "permission": "READ"
                },
                {
                  "permission": "WRITE"
                }
              ],
              "nodes": [
                {
                  "url": "https://github.com/<SOME OUTSIDE COLLABORATOR>",
                  "login": "<SOME OUTSIDE COLLABORATOR>",
                  "name": "...",
                  "email": "...",
                  "company": "..."
                },
                {
                  "url": "https://github.com/<SOME OTHER OUTSIDE COLLABORATOR>",
                  "login": "<SOME OTHER OUTSIDE COLLABORATOR>",
                  "name": "...",
                  "email": "...",
                  "company": null
                }
              ]
            }
          },
          ...

Thank you, @olafz, for verifying that the GraphQL query works well in the GitHub GraphQL Explorer. I'll proceed with the required adjustments on the plugin side and inform you as soon as the modifications are complete.
I appreciate your cooperation!

Hello @olafz, I've introduced a new table called github_organization_collaborator in the issue-404 branch.
Table Usage:

It's necessary to include the organization value (the name of the organization) in the where clause when querying this table.
Optionally, you can specify an affiliation value in the where clause.
The default value for affiliation is ALL, with possible values being OUTSIDE, DIRECT, or ALL.

Unfortunately, I was unable to test the changes due to missing organization-level permissions. Your assistance in testing these changes on the PR branch and providing feedback would be greatly appreciated.
Testing Steps in the PR Branch:

Clone the repository: git clone https://github.com/turbot/steampipe-plugin-github.git
Navigate to the directory: cd steampipe-plugin-github
Switch to the issue-404 branch: git checkout issue-404
Execute make
Run your query.

Thank you!

Hi @ParthaI,
Awesome; thanks! I'm going to give it a try somewhere in the next couple of days.

Hello, @olafz, have you had the opportunity to try it out yet?

Hello @ParthaI,
Thank you for your patience. I've been testing this branch, but unfortunately it did not work. The following query:
> SELECT * FROM github_organization_collaborator WHERE organization = '...';

Consistently results in
Error: github: non-200 OK status code: 502 Bad Gateway body: "{\n   \"data\": null,\n   \"errors\":[\n      {\n         \"message\":\"Something went wrong while executing your query. This may be the result of a timeout, or it could be a GitHub bug. Please include `86E8:...:...:...:...` when reporting this issue.\"\n      }\n   ]\n}\n"

The GraphQL results still work, so I suspect Steampipe to do something inefficient? I've contacted GitHub support; maybe they can help figuring out what the cause is.

I've gotten a response from GitHub:

Our logs indicate that you're hitting our timeout threshold when fetching the data. All API requests, both for the REST API and GraphQL API, have a 10 second limit on execution time. If that limit is reached for a request, the request is terminated and you get back that error.

One of the things I notice, is that in other GitHub related Steampipe queries, I see a Loading results quickly after the query has been issued, increasing in number:

This goes on until there is an output. I don't see anything like that in the github_organization_collaborator query. In an attempt to further debug the issue, I ran the earlier query, but this time with a limit:
> SELECT * FROM github_organization_collaborator WHERE organization = '...' LIMIT 1;

That gave a clue:
Error: github: unable to parse hydrate item {0xc0006529f0 ADMIN {{992 MDQ6VXNlcjk5Mg== Olaf van Zandwijk} olafz  2008-02-26 21:22:23 +0000 UTC 2024-02-05 15:43:50 +0000 UTC https://github.com/olafz}} as an User (SQLSTATE HV000)

Hi @olafz, apologies for the late reply, and I greatly appreciate your patience.
I've submitted a PR to address this issue. The table should now return the results correctly.
Thanks!

Hi @ParthaI,
The new version works better, but still not good :) The query now returns results as long as I limit the results to a relatively low amount:
> SELECT * FROM github_organization_collaborator WHERE organization = '...' LIMIT 50;

Without a limit, or with a larger limit (> 100), I still see the same behavior as before:
I still observe the progress for this query differently compared to other GitHub queries. They show a progress indicator:
Other queries:

This query just shows Executing query... for a while and then fails:
> SELECT * FROM github_organization_collaborator WHERE organization = '...' LIMIT 100;

Error: github: non-200 OK status code: 502 Bad Gateway body: "..." (SQLSTATE HV000)

Hello @olafz, apologies for the snag!!
Could you please raise a separate issue as per the issue comment? I will take a look.
Please include the following details in the new issue body:
Are you aware of the rough number of collaborators within your organization? The issue might stem from the extensive datasets in the org. I'll attempt to replicate it locally by tweaking the plugin code.
Could you describe how the query SELECT * FROM github_organization_collaborator WHERE organization = '...' and affiliation = 'OUTSIDE' behaves, both with and without the limit clause?
Thanks!

I've opened a new issue here: https://github.com/turbot/steampipe-plugin-github/issues/413
Thanks for the assistance!


Incorrect  time shown on calendar on Day light time start date.

for example dst for canberra starts from April 2, and current system time zone is canberra only i.e utc+10 . so when you navigate to 2nd april on calendar it shows wrong time in calendar.

It looks like this repo isn't maintained anymore.
I am maintaining a fork, could you try it and see if you still have the same problem.


Generates verification file for Chocolatey

A verification file is now needed since this cli_pkg builds an embedded package with the software included inside the package.
Have successfully passed the approval Chocolatey approval process with the dynamically generated file in this PR.

Are you sure this is necessary? My understanding was that it was required when shipping binaries to Chocolatey, but cli_pkg doesn't ship binaries—it just generates an install script that compiles the binaries on the end user's system.

@nex3 this is the information I received from their moderators. I can try to get more info, so they can provide a technical explanation for this requirement.

That would be good—as far as I know we've been successfully releasing Sass without a verification file.

@nex3 yes I had done a successful submission also that is why it seemed like it changed.
I will try to get more info from them…

@nex3 I have some more info, seems they define this type of package as "embedding the binaries"  as per this link https://docs.chocolatey.org/en-us/create/create-packages#including-the-software-installer-in-the-package
Also, my guess what happens with Sass has a "Trusted package" status, which means it gets approved after automated checks, and won't go through moderation anymore.
I have asked some follow-up questions and can update once I hear back from them.

It sounds like that section is talking about embedding binaries, where cli_pkg ships the package source (plus an install script that compiles that source). Maybe the verifiers want to check that the source code that exists is correct?

That is the information I was sent to, but I asked that follow-up, still no response. Yes, it seems they just want to verify the source of the script in general. Sorry just being the message here but I am also confused.
I will try to submit a version for approval, without the verification file and see what the new moderator says and try to have the conversation again. This approval process feels a lot like the "App Store" .

@nex3 This their "official" response. I don't think I will push further.
I would disagree that this is not an embedded package.
Embedded means that the package contains the actual software, instead of only including scripts that download and install the software.
It does not matter if the software is in source code form or binary form.

Therefore, a verification.txt is required.

It is used to help moderators (and users as well) determine that the software included inside the package is original and unmodified from the official download location.

If you think this merge is no needed, maybe there is a way to just bring in a verification file, even though you won't have the "dynamic" approach to it, maybe that is a workaround.
What do you think?

Have an implementation chocolateyRepoUrl, but also have to build the release URL, for the verification which assumes that is on GitHub. Maybe later I can implement a verification file override or a template override.
Will check why some tests are failing soon.

Tests should be fixed by #81.

Have an implementation chocolateyRepoUrl, but also have to build the release URL, for the verification which assumes that is on GitHub. Maybe later I can implement a verification file override or a template override.

I don't know if it's worth allowing people to override the entire verification file, but I think having chocolateyRepoUrl and chocolateyReleaseUrl be customizable variables makes senes.

@leoafarias Are you planning on circling back to this?

@leoafarias Are you planning on circling back to this?

Yes, haven’t had much time this week.


pickled feature vector

When you are running the full_gurobi_parser, where does the 'feature_vectors_pickled' come from?

These are feature vectors (last layer before FC) computed using your favourite deep-learning framework,


Add EnumUtils.equalsAny

Added EnumUtils.equalsAny similar to StringUtils.equalsAny. This can help to have shorter, easier to read code.
Usage example:
if (EnumUtils.equalsAny(myObject.getMyField().getMyEnum(), Vehicle.CAR, Vehicle.TRUCK, Vehicle.MOTORBIKE) {
    // do something ...
}

If desired, I can also implement an EnumUtils.equalsNone.

Similar to (null checks not included):
EnumSet.of(searchEnums[0], searchEnums).contains(testEnum)

This code would leverage the optimised EnumSet code for any Enum that has less than 64 members (as internally it uses bit flags in a long).

How is this any different from org.apache.commons.lang3.ArrayUtils.contains(Object[], Object)?

Using this approach results in longer, less readable code, as there is no vararg parameter.
if (ArrayUtils.contains(new Vehicle[]{ Vehicle.CAR, Vehicle.TRUCK, Vehicle.MOTORBIKE}, myObject.getMyField().getMyEnum())) {
    // do something ...
}

Also, ArrayUtils.contains is not generic, so no type checking can be done. I can write code like this and the compiler will not warn me that this expression will always evaluate to false:
if (ArrayUtils.contains(new Vehicle[]{ "123", "abc"}, myObject.getMyField().getMyEnum())) {
    // do something ...
}

Similar to (null checks not included):
EnumSet.of(searchEnums[0], searchEnums).contains(testEnum)

This code would leverage the optimised EnumSet code for any Enum that has less than 64 members (as internally it uses bit flags in a long).

if (EnumSet.of(Vehicle.CAR, Vehicle.TRUCK, Vehicle.MOTORBIKE).contains(myObject.getMyField().getMyEnum())) {
    // do something ...
}

This is certainly an option, but it has two disadvantages:

less readable than EnumUtils.equalsAny.
it is not possible to: check at once if my enum is null or has one of several given values

What can I do to move this issue forward?

Hello @mfg92

What can I do to move this issue forward?

You need to gather positive consensus from the community, which I do not see ATM.

Hello @mfg92
How about using the existing API, like this (from ArrayUtilsTest):
    @Test
    public void testContainsAnyEnum() {
        assertTrue(ArrayUtils.containsAny(ElementType.values(), ElementType.ANNOTATION_TYPE));
        assertFalse(ArrayUtils.containsAny(ElementType.values(), (ElementType) null));
    }

?

Closing: No reply in over 3 weeks.


(🎁) Ability to make a non-local parent branch

I don't really like having a local checkout of my parent branch (main). Having the parent branch be a remote branch would be my preferred workflow I think.
PS: Is this the right place to raise this? Or should it go on https://github.com/VirtusLab/git-machete?

I have absolutely no need for a copy of main branch to sit around and get out of date, I would rather just link the child branches directly to the origin/main. I want to be able to update my branch in a single click, I want a single button to make by branch up to date with origin/main, if not I have to update my local main, then update my branch.

We decided to drop this idea.
The reasons are similar as for (https://github.com/VirtusLab/git-machete-intellij-plugin/issues/241), so...

It would make things too complicated (for users). In particular sync to parent status refers to the status between local branches - in the case of the remote branch, none of the statuses is applicable; theoretically, a new status would be required.

Soon we will provide options (quick actions) within the machete file (https://github.com/VirtusLab/git-machete-intellij-plugin/issues/1232) - one will be able to easily checkout the remote branch.


Create VM

Hello! Can I create a virtual machine using your driver? Could you show an example of how this feature can be used?
Many thanks!

Sorry not yet, I've been on and off with this project.. will see if I can add that sooner than later

Made some progress: https://github.com/dariusbakunas/terraform-provider-truenas/pull/5

Released: https://registry.terraform.io/providers/dariusbakunas/truenas/latest/docs/resources/vm


Ignore Windows shims for (e.g.) python3.12.exe

AppData\Local\Temp
❯ uv --version
uv 0.1.14 (c525fdf2b 2024-03-04)

AppData\Local\Temp
❯ uv venv -p 3.12 vvv
  × failed to canonicalize path `C:\Users\...\AppData\Local\Microsoft\WindowsApps\python3.12.exe`
  ╰─▶ The file cannot be accessed by the system. (os error 1920)

Originally posted by @madig in https://github.com/astral-sh/uv/issues/2105#issuecomment-1978399748

I can confirm that 0.1.15 fixes the issue :) thanks for working on it!


Fixed issue where reference to connection.digest is not longer valid

This was possibly an issue that arose with newer version of rosbags.

Hey thanks for making these updates!


Support Module aliasing in SwiftPM

Resolves rdar://87583788

@swift-ci test

@swift-ci test

@swift-ci test

@swift-ci test

@swift-ci please smoke test

https://github.com/apple/swift-llbuild/pull/758
@swift-ci smoke test

apple/swift-llbuild#758
@swift-ci smoke test

apple/swift-llbuild#758
@swift-ci Please Build Toolchain macOS Platform

@elsh in order to build a toolchain, you need to create a PR on the swift repositories and then mention all the related PR (like this one and the llbuild one) for cross repo building. iow, the SwiftPM repo is not setup to create toolchains, you have to create an empty PR on the swift toolchain to do that.

@swift-ci test

@swift-ci test

@swift-ci smoke test

probably worth adding at least one fixtures in the functional tests area for a end-to-end test of the functionality

@swift-ci smoke test

probably worth adding at least one fixtures in the functional tests area for a end-to-end test of the functionality

Planning to add that in a separate PR; lmk if you think it should accompany this PR.

@swift-ci smoke test

i think at least one end-to-end (fixture) test is in place

@swift-ci smoke test

@swift-ci smoke test

@swift-ci smoke test

@swift-ci smoke test

@swift-ci smoke test

🥳


Rename backend server to headlamp-server

The former name was too generic, which made it difficult to spot, and
could clash on Windows as well.
This patch renames it as headlamp-server but keeps as /headlamp/server
symlink in the container image, for backward compatibility.
fixes #487

Broken mac build fixed in here: https://github.com/kinvolk/headlamp/pull/498


Protractor ignores connection to DB via ssh

Hello.
I have the next tricky task: I need to test a chat and then make a query to DB and check billing.
We are using mysql over ssh connection.
I found the next library to work with:
https://www.npmjs.com/package/mysql-ssh
I tried it and it works.
But when I am trying to set it in my spec I see nothing in console.log and the specs always runs successfully even if I change sql query to non-valid:

I do not have ideas how to push it working.
I am using latest version of protractor.
Thanks.

Maybe add an catch and check if the promise got rejected, so you could get more information.
.catch(err => {
console.log(err)
})

Maybe also check for missing returns in your pomise chain

Hi there!
Since it involves usage and debugging (and you haven't used the issue template to indicate otherwise), your question is better suited for StackOverflow or Gitter. Please ask a question there with the 'protractor' tag or post in the Gitter Channel to get help.
From the getting help section of the README.

Please ask usage and debugging questions on StackOverflow (use the "protractor" tag) or in the Angular discussion group. (Please do not ask support questions here on Github.)

Thanks!


Error following windows cli tutorial

Followed the guide, Quickstart: Create a C# function in Azure from the command line for an install on windows and coming up with the following error after running func start:
Microsoft.Azure.WebJobs.Extensions.Http: Could not load file or assembly 'System.Net.Http.Formatting, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=31bf3856ad364e35'. The system cannot find the file specified.
Value cannot be null. (Parameter 'provider')

Commands run
func init dotnet8Function --worker-runtime dotnet-isolated --target-framework net8.0
cd .\dotnet8Function\
func new --name PostOfficeHealthCheck --template "HTTP trigger" --authlevel "anonymous"
func start

Tool Versions
dotnet sdk 8.0.204
Azure Functions Core Tools
Core Tools Version:       4.0.5700 Commit hash: N/A +71cc84964a60bfb07d95839b7c666bd239507bdd (64-bit)
Function Runtime Version: <IP_ADDRESS>72

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

ID: 8962c9c8-f550-db2f-6573-0dea51802a6e
Version Independent ID: 6340d0ac-0f03-3484-1a6b-d4bb5b64d645
Content: Create a C# function from the command line - Azure Functions
Content Source: articles/azure-functions/create-first-function-cli-csharp.md
Service: azure-functions
GitHub Login: @ggailey777
Microsoft Alias: glenga

@JakeDriscoll
Thanks for your feedback! We will investigate and update as appropriate.

My apologies.
My intent was to see if there was a missing step or anything that could/would be added to the docs to improve them so they wouldn't fall into the same pitfall I did.

@JakeDriscoll I'm running into the same issue. Were you able to resolve it?

@blayvant
I uninstalled "Azure Functions Core Tools". Then download it from https://go.microsoft.com/fwlink/?linkid=2174087 and reinstalled it.
Then it runs normal.
https://learn.microsoft.com/en-us/answers/questions/1665338/how-to-fix-could-not-load-file-or-assembly-microso
Is this helpful?

I was able to resolve the issue by clearing nuget cache and re-installing Azure Functions Core Tools thanks to https://github.com/Azure/azure-functions-core-tools/issues/3096


Scope issue with injected css for svg path

This is great, thanks for writing this.   There is one issue I have come across, the css being injected (specifically: svg path{fill:#fff}  is affecting ALL of the font awesome icons in the view.  How can I go about limiting the scope of that css?  Thanks.

BTW, simply commenting out that part of the injected css solved my issue,  however it is not very portable unless I fork this, or recreate the component based on your code within my project.

This issue is fixed in v0.0.2


Added unit tests for iot_pkcs11.c.

Description

Checklist:

[ ] I have tested my changes. No regression in existing tests.
[ ] My code is Linted.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

https://github.com/aws/amazon-freertos/pull/1926 is needed for all tests to pass.

/bot run checks


Please add documentation of how to use haskell_stack_ghc_options

I'd like to use this commit's functionality. I use stack_ghc linter with ale, and would like to pass compiler options to it that are more helpful.
Could you please respond here with an example, or possibly add documentation somewhere on how to use this?
Is it something like this?
let g:ale_linters.haskell = ['stack_ghc', 'hlint']
let g:haskell_stack_ghc_options = [ '-Wall', '-Wcompat', '-Wincomplete-record-updates', '-Wincomplete-uni-patterns', '-Wredundant-constraints' ]

g:haskell_stack_ghc_options should be a string, like let g:haskell_stack_ghc_options = '-Wall -Wcompat -Wincomplete-record-updates -Wincomplete-uni-patterns -Wredundant-constraints'


Fox Farming v7

The TMDC voted and we will deploy the contract shortly for another round of farming.
AFAIK this is a straightforward process where we have templating. Will update the ticket here as details emerge.

deployed and verified

#5071 all done


fix DeployBeanDescriptor.java of the method setView

the method setView in DeployBeanDescriptor has wrong of
this.dependentTables = this.dependentTables;

Thanks, funnily enough I literally just saw that 10 minutes ago as well.

That's so funny.Otherwise, I have a problem now.
I have one table named task, and other table named like items_taskId, the taskId is a dynamic value from table task.
How can i map the Entity Item to the dynamic table?
The simple thing want to do is change the final sql "from item" to "from item_xxx".
I just want to select, update, insert of the Item Entity,but not for Multi table operate.
How can I do?


Uncaught Error: Module did not self-register.

[Enter steps to reproduce:]

...
...

Atom: 1.45.0 x64
Electron: 4.2.7
OS: Unknown macOS version
Thrown From: Atom Core
Stack Trace
Uncaught Error: Module did not self-register.
At ELECTRON_ASAR.js:160

Error: Module did not self-register.
    at process.module.(anonymous function) [as dlopen] (ELECTRON_ASAR.js:160:31)
    at Object.Module._extensions..node (internal/modules/cjs/loader.js:722:18)
    at Object.module.(anonymous function) [as .node] (ELECTRON_ASAR.js:169:18)
    at Module.load (internal/modules/cjs/loader.js:602:32)
    at tryModuleLoad (internal/modules/cjs/loader.js:541:12)
    at Function.Module._load (internal/modules/cjs/loader.js:533:3)
    at Module.require (/app.asar/static/index.js:72:46)
    at require (internal/modules/cjs/helpers.js:20:18)
    at customRequire (/Applications/JuliaPro-1.4.1-1.app/Contents/MacOS/JuliaPro-Juno-1.4.1-1.app/Contents/Resources/JuliaPro-Juno-1.4.1-1/Contents/Resources/app/static/<embedded>:1:695134)
    at get_KeyboardLayoutManager (/Applications/JuliaPro-1.4.1-1.app/Contents/MacOS/JuliaPro-Juno-1.4.1-1.app/Contents/Resources/JuliaPro-Juno-1.4.1-1/Contents/Resources/app/static/<embedded>:14:2781650)
    at get_manager (/Applications/JuliaPro-1.4.1-1.app/Contents/MacOS/JuliaPro-Juno-1.4.1-1.app/Contents/Resources/JuliaPro-Juno-1.4.1-1/Contents/Resources/app/static/<embedded>:14:2781752)
    at Object.getCurrentKeyboardLayout (/Applications/JuliaPro-1.4.1-1.app/Contents/MacOS/JuliaPro-Juno-1.4.1-1.app/Contents/Resources/JuliaPro-Juno-1.4.1-1/Contents/Resources/app/static/<embedded>:14:2781878)
    at e.keystrokeForKeyboardEvent (/Applications/JuliaPro-1.4.1-1.app/Contents/MacOS/JuliaPro-Juno-1.4.1-1.app/Contents/Resources/JuliaPro-Juno-1.4.1-1/Contents/Resources/app/static/<embedded>:14:1083882)
    at KeymapManager.keystrokeForKeyboardEvent (/Applications/JuliaPro-1.4.1-1.app/Contents/MacOS/JuliaPro-Juno-1.4.1-1.app/Contents/Resources/JuliaPro-Juno-1.4.1-1/Contents/Resources/app/static/<embedded>:11:1251844)
    at KeymapManager.handleKeyboardEvent (/Applications/JuliaPro-1.4.1-1.app/Contents/MacOS/JuliaPro-Juno-1.4.1-1.app/Contents/Resources/JuliaPro-Juno-1.4.1-1/Contents/Resources/app/static/<embedded>:11:1249829)
    at WindowEventHandler.handleDocumentKeyEvent (/Applications/JuliaPro-1.4.1-1.app/Contents/MacOS/JuliaPro-Juno-1.4.1-1.app/Contents/Resources/JuliaPro-Juno-1.4.1-1/Contents/Resources/app/static/<embedded>:11:284496)

Commands

Non-Core Packages
default-language 0.3.1
ftp-remote-edit 0.18.0
ink 0.12.3
julia-client 0.12.4
language-julia 0.19.3
language-weave 0.7.2
latex-completions 0.3.6
project-manager 3.3.8
tool-bar 1.2.2

Thanks for taking the time to contribute!
We noticed that this is a duplicate of https://github.com/atom/atom/issues/20222. You may want to subscribe there for updates.
Because we treat our issues list as the Atom team's backlog, we close duplicates to focus our work and not have to touch the same chunk of code for the same reason multiple times. This is also why we may mark something as duplicate that isn't an exact duplicate but is closely related.
For information on how to use GitHub's search feature to find out if something is a duplicate before filing, see the How Can I Contribute? section of the Atom CONTRIBUTING guide.


[BUG]

Environment
I Have created a custom input text component using angular-formio,
and it is not detecting default value entered from settings to display in custom input text, Please help me regarding this.
Thanks in Advance
Praveen Madishetty

Even i am facing the same issue. Can't get default value in data tab. Also not getting default value data in the JSON output as well.

@praveenmadishetty @iknityanand
Thank you for reaching out! Form.io has support packages that will connect you with a developer to help you with your custom components. Please let me know if you are interested and I will ensure that you are connected to our Team with information regarding our Support Packages.

Thanks for your Reply @wag110894 , But it would be helpful if i get help without going to support package Team Presently, the reason for this, is nothing but i just want to give a try with Formio, and know to what extent it can be compatible for my upcoming projects.

Closing this thread as it is outdated. Please re-open if it is still relevant. Thank you for your contribution!


a few issues

Hi Lucas!
I found your repo from Adam King's project. First of all, great work and respect.
While my mac is training, there is a few issues I ran into.

When installing env with conda on my mac, matplotlib=2.2.3=py35h0e0179f_0 cannot be found.
I removed the last part py35h ... and it fixed the problem.
So, my question is: did you test this in ubuntu env, or mac?

I tried to train multi model as instructed on readme, python train_multi_model.py gave me error not finding SYMBOL_1 (from vars.py)
I looked into what might caused this, but I could not find where you pass in the SYMBOLs from vars.py
any idea how to fix this? or I did something wrong

While I was looking into train_multi_model.py, I noticed in experiment_spec, there is a line "local_dir": '/home/lucas/Documents/cryptocurrency_prediction/tensorboard'.
I could not run this train_multi_model.py file, but if I did, I imagine it would give me an error about folder not exist.

 But I could not find where is the output tensorflow folder, so I cannot really track its progress.
Again, any idea? or did I do something wrong...?

Again, great project! Thx!

Hey mate, let me try to clarify things for you

I only tested in ubuntu, the mac.yml is a contribution from a mac user.

In train_multi_model.py the following line is missing:

from configs.vars import *

Yeah you need to create a tensorboard folder and update "local_dir" to this recent create tensorboard folder (u do this if you want to save the checkpoints in the project root folder, if not, use the number 4 idea)

If you don't use the "local_dir" line your checkpoints will be saved at ~/ray_results/[EXPERIMENT_NAME] by default.

I know the project is a lil messy, my wish now is to organize it but I'm working in something else and I have no time at the moment, so fell free to contribute if you want and any question let me know.
Thanks :smile:

fixed https://github.com/Draichi/cryptocurrency_algotrading/commit/b3ef2ec220cc7d95a21ee548c176bcb2b2c89e2b
thanks for the catch


⚠ ClamAV has degraded performance

In 7cc9e67, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 219 ms

Resolved: ClamAV performance has improved in 65ad601.


multiplexingMode doesn't restore the outer mode's state

I created a simple mode for embedding arbitrary expressions inside of {{}} blocks inside JSON:
CodeMirror.defineMode("fancyjson", (config) => {
  return CodeMirror.multiplexingMode(
    CodeMirror.getMode(config, { name: "javascript", json: true }),
    {
      open: "{{",
      close: "}}",
      mode: CodeMirror.getMode(config, "text/plain"),
      parseDelimiters: false
    }
  );
});

const els = document.querySelectorAll(".code");
els.forEach((el) => {
  CodeMirror.fromTextArea(el, {
    lineNumbers: true,
    mode: "fancyjson"
  });
});

Codepen is here.
Screenshot:

Note how the string syntax highlight is incorrect in the second block. Is this expected behavior?

Please don't post the same thing both in the forum and the bug tracker. See my response here. This is unlikely to be fixed on the library side, but you might be able to build a completely custom wrapped node (without multiplex.js) that sidesteps the issue.


When user has screen rotation locked the viewport won't resize

Hi! I'm using your plugin, in Android works great but in IOS when the user has screen rotation locked using IOS general settings, the viewport doesn't fill the screen, it remains as it's in portrait when changing it to landscape. I have already tried all meta viewport settings and non3 of them works, the weired thing is that it was working, it was after using phonegap build (latest version) when I've got a warning telling me of deprecated pgp repositories and changing all my plugins to npm that it started to happen. Any advice will be very much appreciated.
Thanks a lot!

@mediamas, thanks for the report.
This repository is no longer being maintained (see README for more information).
Please log an issue in Apache Cordova JIRA for the "Plugin Screen Orientation" component.


Network limits for B-series VMs?

B-series VMs do not seem to show the network throughput limits, unlike other VMs - are these 'unlimited' network throughput, or is this variable too?

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

ID: 0cd2e24b-0fd3-d698-2840-3f8206933026
Version Independent ID: 832dc371-480a-c74a-e956-e1904473af31
Content: B-series burstable - Azure Virtual Machines - Azure Virtual Machines
Content Source: articles/virtual-machines/sizes-b-series-burstable.md
Service: virtual-machines
Sub-service: sizes
GitHub Login: @ayshakeen
Microsoft Alias: ayshak

@AdamSharif-MSFT
Thanks for the feedback! I have assigned the issue to content author to check and update the document as appropriate.

@ayshakeen
Can you please check and add your comments on this doc update request as applicable.

Due to the nature of the B series being a burstable series, this information can vary. We are working to make this more clear.
#please-close


False positive ISC001 in unmatched f-string

Never mind that this thing raises a SyntaxError, it also triggers ISC001.
f"{({None: ""}).get(key, key)}"

I think this is okay given that it's not syntactically valid. It's getting parsed as f"{({None: ", followed by "}).get(key, key)}" which would indeed by an implicit concatenation and is a reasonable tokenization. Happy to reconsider if you have a syntactically valid example that's causing trouble in the wild.


Question on eSCN

Hello, I have a question about eSCN.
I have read the section "A.3 ESCN CONVOLUTION" of your paper, and I wonder is there an implicit condition for the summation running the irreps order L_i and L_f?  More specifically, assuming tensor product "1ox(0e+1o+2e)", I want to consider "1o" as output irreps. So, how does equiformerv2 distinguish the odd irreps "1o" that comes from "1ox(0e+2e)" and even irreps "1e" that comes from "1ox1o"?

Thank you for your reply!
Do you mean to use even parity irreps for node feature, and "0e+1o+2e+..." for relative position projection?

The node features can have both even and odd irreps like (1x0e+1x0o+1x1e+1x1o...) while the relative positions are of 1x0e+1x1o+1x2e... if E(3) is considered.

I am sorry to reply so late, but I am still confusion about the implementation of so(2) convolution. The type-L irreps_out from so(3)'s tensor product is  a mixture of type-(L,e) and type-(L,o) irreps from the perspective of o(3),  right? Therefore equilformerV2 considering se(3) only do not need to distinguish parity?

Yes, I think what you said are correct.


Links in course detail not working and wrong label

Links in course detail not working and wrong label

In course detail in the navigation bar the links/ routes have the wrong label, and if you click on them they don't redirect you to the coresponding page

Package: Course

Expected Behavior

The links should have the correct Label, and redirect you to the coresponding page
Actual Behavior

The Links have the wrong label, and don't redirect you anywhere
Additional Information

As already stated in #31 and issued on #26.


feat: Add signed and unprotected JWT [SIW-283]

Short description
Adds support functions to:

Create a JWT (producer.ts)
Create an unprotected JWT (unprotected.ts)
Create a signed JWT (sign.ts)

It also adds functions for base64 encoding and decoding
How to test
Unit test!

I wonder why you choose the OOP paradigm 🤷

OOP was only used for creating the JWTs. This for two reasons:

To make jose compatible interfaces for node
Because this allows me to create an object quickly with the following syntax:


fix(django): Fix 404 Handler handler being labeled as "generic ASGI request"

It seems that we changed a lot since this PR was submitted originally. @BeryJu I know it is a lot to ask, but could you move your check for the 404 into the new _set_transaction_name_and_source? (you probably have to rebase your branch on master or merge master into your branch


add tracing

Signed-off-by: jyz0309<EMAIL_ADDRESS>
What this PR does:
Add tracing.
This is still in WIP status, DO NOT REVIEW!!!
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
This is still in WIP status, DO NOT REVIEW!!!
Does this PR introduce a user-facing change?:

Codecov Report

Merging #194 (01e1e58) into master (dd8ab0f) will decrease coverage by 0.00%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master     #194      +/-   ##
==========================================
- Coverage   33.66%   33.65%   -0.01%
==========================================
  Files          79       79
  Lines       12766    12769       +3
==========================================
  Hits         4298     4298
- Misses       7962     7965       +3
  Partials      506      506

Impacted Files
Coverage Δ

pkg/executor/redirect.go
9.50% <0.00%> (-0.15%)
:arrow_down:

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 dd8ab0f...01e1e58. Read the comment docs.

ref: #137 .


Remove unused methods in NetworkLobbyManager

Removed two unused methods

:tada: This PR is included in version 1.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Instant crash on starting another application

Brief description of your issue
Running a Godot-engine app with gdnative C++ code and embedded .pck file causes System Informer to insta-crash on each app start. This can be a security vulnerability, too.
Steps to reproduce (optional)
Run an .exe with a malformed icon data, maybe.
Expected behavior (optional)
No response
Actual behavior (optional)
Mini-dump content:
CONTEXT:  (.ecxr)
rax=0000000000007bbe rbx=0000000000009f4f rcx=0000000000000000
rdx=00000188e7521200 rsi=00000188e7521200 rdi=0000000000000000
rip=00007ff7b0c8d496 rsp=0000001138dff678 rbp=0000001138dff6e8
 r8=0000000000000000  r9=000000000000000e r10=0000000000000000
r11=00000188e755f000 r12=0000001138dff7a0 r13=00000188e7521200
r14=0000000000000000 r15=0000000000000000
iopl=0         nv up ei ng nz ac po cy
cs=0033  ss=002b  ds=002b  es=002b  fs=0053  gs=002b             efl=00010297
SystemInformer!PhExtractIcon+0x156:
00007ff7`b0c8d496 418b0b          mov     ecx,dword ptr [r11] ds:00000188`e755f000=????????
Resetting default scope

EXCEPTION_RECORD:  (.exr -1)
ExceptionAddress: 00007ff7b0c8d496 (SystemInformer!PhExtractIcon+0x0000000000000156)
   ExceptionCode: c0000005 (Access violation)
  ExceptionFlags: 00000000
NumberParameters: 2
   Parameter[0]:<PHONE_NUMBER>000000
   Parameter[1]: 00000188e755f000
Attempt to read from address 00000188e755f000

PROCESS_NAME:  SystemInformer.exe
READ_ADDRESS:  00000188e755f000
ERROR_CODE: (NTSTATUS) 0xc0000005
EXCEPTION_CODE_STR:  c0000005
EXCEPTION_PARAMETER1:<PHONE_NUMBER>000000
EXCEPTION_PARAMETER2:  00000188e755f000

STACK_TEXT:
SystemInformer!PhExtractIcon+0x156
SystemInformer!PhExtractIconEx+0x1f1
SystemInformer!PhImageListExtractIcon+0x125
SystemInformer!PhEnumProcessItems+0x44f
SystemInformer!PhGetStatisticsTimeString+0x21d9
SystemInformer!PhInitializeProviderThread+0x211
SystemInformer!PhAutoDereferenceObject+0x4ab
kernel32!BaseThreadInitThunk+0x14
ntdll!RtlUserThreadStart+0x21

STACK_COMMAND:  ~3s; .ecxr ; kb
SYMBOL_NAME:  SystemInformer+156
MODULE_NAME: SystemInformer
IMAGE_NAME:  SystemInformer.exe
FAILURE_BUCKET_ID:  INVALID_POINTER_READ_c0000005_SystemInformer.exe!Unknown
OS_VERSION:  10.0.19041.1
BUILDLAB_STR:  vb_release
OSPLATFORM_TYPE:  x64
OSNAME:  Windows 10
IMAGE_VERSION:  3.0.11049.6522
FAILURE_ID_HASH:  {b271a9f9-efc9-ea7b-e213-bd1d4817e73e}
Followup:     MachineOwner

Environment (optional)
Key  : WER.OS.Version
Value: 10.0.19041.1

Key  : WER.Process.Version
Value: 3.0.11049.6522

This can be a security vulnerability, too.
Run an .exe with a malformed icon data

This crash was caused by incorrect string concentration for system32 directories containing MUN resource redirection and was reported and fixed weeks ago. Malformed data cannot cause vulnerabilities because we're using commit sections for resource lookup and validate data unlike other software.

IMAGE_VERSION:  3.0.11049.6522

You're running an old version and need to update to r6550


Last version crashes when loaded in a Client UI

I try to use the latest JDBC driver with DbSchema.
This tool is loading the drivers dynamically, using java.net.URLClassLoader
When loading the latest single-driver-in-one-jar with dependencies, it gets the exception below.
The driver version 2.3.2 works fine.
How can we avoid this?

@wise-coders:
I try to use the latest JDBC driver with DbSchema: Can you confirm that you are using the JDBC version of github.com/googleapis/java-spanner-jdbc? The latest version we have is a [2.5.x] (https://github.com/googleapis/java-spanner-jdbc/blob/main/versions.txt).

I try to use version 2.5.9-SNAPSHOT-single-jar-with-dependencies, and it fails with 'Invalid signature file digest for Manifest main attributes'.
If I use 2.3.2 it works fine.
The fix would be a zip file with dependencies as separate jars.

@wise-coders: Thanks for bringing this to our notice. We are now advising customers to move away from fat jars(single jar with dependencies) as they lead to such unexpected errors. Having dependencies in separate jars is the way to go.


Function to access the Synset object using sense key

I might have missed it but is there a function to access the Synset object from the NLTK wordnet interface from the sense key?
If there isn't could we expose a function that can achieve that in nltk.corpus.wordnet? E.g. https://stackoverflow.com/questions/48170666/how-to-get-the-gloss-given-sense-key-using-nltk-wordnet/
Ideally, it would be good to have functionalities to access Synset objects using:

offset-pos, e.g. 1433493-a -> Synset('long.a.02')
sense_key, e.g. long%3:00:02:: ->  Synset('long.a.02')

Currently, we have the synset_from_pos_and_offset() for (1).
There's another function (_synset_from_pos_and_line ) that reads the following line to return Synset('long.a.02'):
01433493 00 a 01 long 1 016 = 05129201 n 0000 + 05133287 n 0101 ! 01436003 a 0101 & 01434007 a 0000 & 01434218 a 0000 & 01434530 a 0000 & 01434717 a 0000 & 01434841 a 0000 & 01434966 a 0000 & 01435060 a 0000 & 01435189 a 0000 & 01435290 a 0000 & 01435399 a 0000 & 01435507 a 0000 & 01435675 a 0000 & 01435891 a 0000 | primarily spatial sense; of relatively great or greater than average spatial extension or extension as specified; "a long road"; "a long distance"; "contained many long words"; "ten miles long"

but it's not the sense key.

I'd like to work on this!

@craaaa Sorry for the late reply, was away for a while.
Feel free to work on it and create a PR afterwards.
P/S: Don't worry about breaking anything, there'll be checks and reviews before we merge the code.

Implemented the function suggested in the stackoverflow, but it didn't seem to map to the correct senses -- for instance, synset_from_sense_key('afraid%3:00:00::') returned afraid.a.04 instead of afraid.a.01. This problem extends to other POS as well. (Sense keys were obtained from Wordnet's online interface)
Instead, using the method shown in the SemCor documentation appears to map correctly -- there is currently a lemma_from_key(key) function that seems to take in something similar to a sense key. However, lemma_from key(key) doesn't support adjective satellites (e.g. afraid%3:00:02:concerned:00). I can definitely implement a wrapper around lemma_from_key(key) to fix this and return a Synset.

@craaaa. I think there is still an issue with adjective satellites. I tried to use the synset_from_sense_key function and here is the error:

  File "/home/izorar/anaconda3/lib/python3.7/site-packages/nltk/corpus/reader/wordnet.py", line 1356, in synset
    raise WordNetError(message % lemma)
WordNetError: adjective satellite requested but only plain adjective found for lemma 'first'

Any idea on how to fix the error?


Deck for Presenting to Accessibility Partners

📇 User story
As a GC Design System team member, I want to present our accessibility considerations and process, so that our stakeholders understand and advance our product vision.
✅ Definition of Done / Outcomes
Deck is created.
📜 Acceptance criteria

[ ] Overview of GC Design System and our mandate
[ ] Overview of relevant tech choices specifically web components
[ ] Organized slides summarizing GC Design System Accessibility process
[ ] Insert more technical details re: "Process for creating web components" - Card #1246
[ ] Our key challenges and plans for the future
[ ] Identifying that this may change how accessibility teams audit applications and sites

📝 More info
This will likely be a deck but will also include some demo of the system.  Our goal here is to be transparent about our strategy and roadmap, the technology choices we've made, including how those impact accessibility standards and testing.
This deck will be created following the work in card #1246
🚫 Out of scope
Iterations to the accessibility process for GC Design System.

https://docs.google.com/presentation/d/1epIev_Jm-j_EdTka3uxoPZ6J63mKvf_JXo-EsuKCdK8/edit#slide=id.g2bcac7083ba_0_5892


Dead pixel appears with skype for Business web app

A dead pixel appears every time the web app is open. As soon as I exit and quit the app the dead pixel goes away.

I have seen this as well on Safari

Yes, I can see that too. It happens on the screen you initialise the program on. The pixel extends to the screen and it stays still. It's gone after the program is closed

It's still there.

It happens every time I use the app (even on external monitor) - MacOS Mojave, MacbookPro2017TB, RadeonPro 560 - no other app is causing such an issue, also, the Energy tab in Activity monitor shows, that skype does not use high performance gpu

This is surprising and scary. Yet if you can screenshot it, it is not a dead pixel :)
It seems to be the minimization point of the Web App Plug-in alert asking the authorization to access audio, video and screen. If you check the box "Always allow the plug-in for this domain", the "dead pixel" will not appear in subsequent calls with the same domain as the alert will not be shown.


The file is 0 bytes

eh we
Hello, when I was decrypting the game, it resulted in a 0-byte file, according to me I only had to have the game.win and the .bin, and nothing came of it

eh we
I saw what it was, the bin is empty, Im sorry


Index with the condition

Is your feature request related to a problem? Please describe.
So far, I don't see a way to add a condition when using indexes.
My project requires that we don't delete any data, just to mark them as deleted. This is simply done by updating deletedAt column and using filters. However, in order to have unique flags on some properties, we need a `UNIQUE INDEX.
We manage this by adding manually in the migration file something like this
this.addSql('CREATE UNIQUE INDEX "index_name" ON "table_name" (UPPER("column_name")) WHERE "deletedAt" IS NULL');

However, each time a new migration file is generated, those indexes are getting removed and we need to manually remove those lines from the file.
Describe the solution you'd like
Either a way to declare a condition, or to not auto delete added indexes.
Describe alternatives you've considered
So far, none, manually removing the lines after the migration is generated
Additional context
None

You can use index expressions added in v5, looks like they are not documented.
  @Index({ name: 'custom_index_expr', expression: 'alter table `book` add index `custom_index_expr`(`title`)' })
  @Property()
  title!: string;

Tnx


Can't change process

I'm trying to change the process for a project in Azure Devops Server 2020 and the CollectionSettings->Boards->Process screen does not look like this.  There is no way for me to select a process to list the projects under it.  I've tried clicking and it does nothing (the one shows here looks like a hyperlink).  The '...' menu shows 4 options: New Teams Project, Set as default process, Disable Process, Export, and sometimes Delete.
When I try to setup a board from within the project it says 'Configuration required' and has a button to 'Configure feature'.  This opens a page for Team Foundation Server 2017: https://learn.microsoft.com/en-us/previous-versions/azure/devops/reference/upgrade/add-features-manually?view=tfs-2017

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

ID: 365bff0e-8532-ae8a-f1c9-37ea7017e5b1
Version Independent ID: a324129d-0edb-00ce-1217-819be9fda4f6
Content: Change process from Basic to Agile - Azure Boards
Content Source: docs/organizations/settings/work/change-process-basic-to-agile.md
Service: azure-devops-boards
GitHub Login: @chcomley
Microsoft Alias: chcomley

@jasong-au -- thank you for your feedback.  Please consider these resources:

Azure DevOps Services
Azure DevOps on Stack Overflow

@chcomley -- please look into this issue.

if you are on Server 2020 and you are using the older XML model, you need to use witadmin commands to update the process on each project.
https://learn.microsoft.com/en-us/azure/devops/reference/witadmin/witadmin-customize-and-manage-objects-for-tracking-work?view=azure-devops
if you are using the inherited model, this should help you: https://learn.microsoft.com/en-us/azure/devops/organizations/settings/work/manage-process?view=azure-devops
If you are still stuck, lets chat

Hi @jasong-au, I'm closing this issue, but if you need to chat, please @mention danhellem and we can get you both connected to assist you further. Thank you!


Implement Google Cloud Pub/Sub Spring Cloud Stream binder

Is there currently any work on this Ticket? I have already done some implementation on this at: https://github.com/spring-cloud/spring-cloud-stream-binder-google-pubsub/pull/5 and was redirected to this project.

@Andy2003 thank you for the interest in the Spring Cloud Stream binder for Pub/Sub. As you can see it's on our roadmap for 1.0.0.M1. However, we haven't really started work on it yet. We will look at the PR you referenced. In any case, would you be interested in contributing the work you've done in that PR to this project?

@meltsufin Yes I can port my former work to use the artifacts of this project, this should not be a big deal. Before I start I just wanted to be sure that @joaoandremartins is not already working on this topic. Feel free to assign the Issue to me.

Haven't worked much on it besides some initial research.
Feel free to submit your PR and we can have a look.

Seems I can't assign you because you aren't a collaborator on the project.
Probably makes sense for me to keep the assignment as the owner of this feature overall, anyway.


"Webview is disposed" error notification on switching to a branch

When I switched to a branch with an active PR up (but I haven't interacted with GHPR extension), I got:

Found a reliable way to repro:

Checkout a PR
Open the PR overview
Change branches from command line
Change branches back to the PR branch from the command line.

Error comes from here:
https://github.com/microsoft/vscode-pull-request-github/blob/2384341c9a49da7ba550847e1399c3bb0c819c12/src/github/pullRequestOverview.ts#L237-L238

@alexr00 still experiencing this issue in v0.28.0.

The fix has not been released yet and will be in 0.29.0

The fix has not been released yet and will be in 0.29.0

Thanks!

Not able to verify due to https://github.com/microsoft/vscode-pull-request-github/issues/2860

Verified on v2021.7.42360

I know this issue is closed, but I'm seeing this a lot still.

Version: 1.75.0 (Universal)
Commit: e2816fe719a4026ffa1ee0189dc89bdfdbafb164

when I tried to paste some words to vscode, the error popped up, and many times. I have to restart vscode.

Same error when copy then paste:

system-info:
No LSB modules are available.
Distributor ID: Ubuntu
Description:    Ubuntu 16.04.4 LTS
Release:        16.04
Codename:       xenial
vscode-info:
1.84.0
d037ac076cee195194f93ce6fe2bdfe2969cc82d
x64

Same error when copy then paste in plantuml extension supported file

Sounds like this is a different extension.


foreign copyright notice in netcdf code file

I hate to bring up what can only be a dumb legal issue, but we have a file, examples/C/parallel_vara.c (a pnetcdf example). It contains:
/*********************************************************************
 *
 *  Copyright (C) 2014, Northwestern University and Argonne National Laboratory
 *  See COPYRIGHT notice in top-level directory.
 *
 *********************************************************************/

My (terrible) legal understanding is that when they contribute the code they assign the copyright to Unidata. So should we change this?
If this is not felt to be appropriate, should be remove this program? (I can easily rewrite the example from scratch).

I don't think it's a particular issue at this point, although if we can change/update the copyright or at least append the Unidata copyright, that would be ideal.  @wkliao do you have any thoughts on this?

Hey, that file was added in 2015 when I first developed a patch for supporting CDF-5 in NetCDF. I am OK with appending or prepending NetCDF's copyright statement. I have seen lots like this.

To be clear, though, contributing code to Unidata projects DOES NOT assign UCAR/Unidata the copyright. This is why we now have the CLA, which while it still does not assign copyright, it does explicitly require contributors to grant us a perpetual license to redistribute the contribution under any OSI approved permissive license.

OK, thanks all for the comments and info. My takeaway is that this and other foreign copyright notices are fine and should be left alone. Thanks for the clarification!


attention fp8 compute type

when we use fp8 data type , we found  ffn gemm/atten prj  support real fp8 comute(this is supported on H20、L20),   but  Q*transopse(Key) or softmax * value  in attention dosen't support fp8 compute,  need to first dequantize fp8 to fp16/bf16, why ?

@Tracin Could you please have a look? Thanks

@enozhu Because we do not implement FP8 FMHA before Hopper. So I think H20 can support attention with FP8 computation.

When enabling use_fp8_context_fmha, the context phase is supposed to perform the Q@K^T operation in FP8, but it seems that qkv is not being properly quantized to FP8. Instead, it uses a scale of 1.0 to forcefully cast FP16/BF16 to FP8 in the function applyBiasRopeUpdateKVCache with convert_to_fp8:
if (params.quantized_fp8_output)
{
    // use 1.0f scale currently for qkv input of FP8 FMHA.
    mmha::convert_to_fp8(quantized_q_ptr, q);
}

Am I missing some information, or is this done intentionally, and wouldn't this cause precision issues, or is the numerical range clamped during training?
@Tracin

When enabling use_fp8_context_fmha, the context phase is supposed to perform the Q@K^T operation in FP8, but it seems that qkv is not being properly quantized to FP8. Instead, it uses a scale of 1.0 to forcefully cast FP16/BF16 to FP8 in the function applyBiasRopeUpdateKVCache with convert_to_fp8:
if (params.quantized_fp8_output)
{
    // use 1.0f scale currently for qkv input of FP8 FMHA.
    mmha::convert_to_fp8(quantized_q_ptr, q);
}

Am I missing some information, or is this done intentionally, and wouldn't this cause precision issues, or is the numerical range clamped during training? @Tracin

Yeah, you are right. We set scale to 1.0 intentionally for fast conversion and this won't hurt model accuracy from our study.

When enabling use_fp8_context_fmha, the context phase is supposed to perform the Q@K^T operation in FP8, but it seems that qkv is not being properly quantized to FP8. Instead, it uses a scale of 1.0 to forcefully cast FP16/BF16 to FP8 in the function applyBiasRopeUpdateKVCache with convert_to_fp8:
if (params.quantized_fp8_output)
{
    // use 1.0f scale currently for qkv input of FP8 FMHA.
    mmha::convert_to_fp8(quantized_q_ptr, q);
}

Am I missing some information, or is this done intentionally, and wouldn't this cause precision issues, or is the numerical range clamped during training? @Tracin

Yeah, you are right. We set scale to 1.0 intentionally for fast conversion and this won't hurt model accuracy from our study.

Could you please share researches on the impact of setting the scale to 1.0 on model accuracy? @Tracin


Fix test_string_dictionary_merge (#5255)

Which issue does this PR close?

Closes #5255
Rationale for this change

A new version of ahash is causing this to fail
What changes are included in this PR?

Are there any user-facing changes?

I'm going to merge this to unblock CI as it is a relatively minor test-only change