Add functionality to handle license choices

It should be possible to select a license if a package is dual licensed.
This should be possible to do ORT instance globally (~/.ort/config/licenseChoices.yml) and project specific (.ort.yml).
Project specific configurations should override global configrations.
The logic how to select and replace SPDX OR clauses are already defined in these tests:
https://github.com/oss-review-toolkit/ort/blob/5b612747fc19b2ce9220b2340b7d7edaac3bf655/spdx-utils/src/test/kotlin/SpdxExpressionTest.kt#L288-L368

Just a remark, such a file should be called license-choices.yml (no camel-case but a dash), and we should probably also give it a though whether we could add license choices to the existing license-classifications.yml instead of adding yet another configuration file.

Copied comment from Slack which contains info relevant for this topic:
I would separate the discussions about (1) for which scope license choices should be configurable and (2) how it should be implemented.
For (1) I believe we need (a) global license choices (e.g. in license-choices.yml), something like "Apache-2.0 OR MIT" always chose "Apache-2.0". The question is, should this also be applied to supexpressions automatically? (b) Per package license choices, because for some packages we might want to make a different choice. This should override global choices. This could be configured in curations.yml. (c) Per project license choices, because two different projects might want to make a different choice for the same library. This should be configurable in ort.yml and override (a) and (b). And maybe in ort.yml we also need to support global and per package choices to support all possible use cases.
If we really want to implement all these options depends on how important we think the different use-cases are.
(2) The logic that applies the license choice should be implemented in LicenseInfoResolver. The problem here is that the ResolvedLicenseInfo for a package does currently not provide an SPDX expression but simply a list of individual licenses as ResolvedLicense objects.
(a) The first step would be to find out if there is a license choice to be able to provide a resolved license expression that uses the correct AND and OR operators in ResolvedLicense. Because if the model does not provide the information if there is a license choice, it does not make sense to implement a way to configure license choices. For this we can check if the declared license contains a license choice, and we could implement support for license choices detected by ScanCode. At least I was told that ScanCode would provide such information in the latest versions, but I have never verified that myself. Otherwise we have to assume that all detected licenses are concatenated by AND.
(b) If we have solved (a) the implementation in LicenseInfoResolver should follow the principle to be completely transparent, that means the API must allow to check what the original SPDX expression was, which choices were applied, and what the resulting SPDX expression is.

Copied comment from Slack which contains info relevant for this topic:
I would separate the discussions about (1) for which scope license choices should be configurable and (2) how it should be implemented.
For (1) I believe we need (a) global license choices (e.g. in license-choices.yml), something like "Apache-2.0 OR MIT" always chose "Apache-2.0". The question is, should this also be applied to supexpressions automatically? (b) Per package license choices, because for some packages we might want to make a different choice. This should override global choices. This could be configured in curations.yml. (c) Per project license choices, because two different projects might want to make a different choice for the same library. This should be configurable in ort.yml and override (a) and (b). And maybe in ort.yml we also need to support global and per package choices to support all possible use cases.
If we really want to implement all these options depends on how important we think the different use-cases are.
(2) The logic that applies the license choice should be implemented in LicenseInfoResolver. The problem here is that the ResolvedLicenseInfo for a package does currently not provide an SPDX expression but simply a list of individual licenses as ResolvedLicense objects.
(a) The first step would be to find out if there is a license choice to be able to provide a resolved license expression that uses the correct AND and OR operators in ResolvedLicense. Because if the model does not provide the information if there is a license choice, it does not make sense to implement a way to configure license choices. For this we can check if the declared license contains a license choice, and we could implement support for license choices detected by ScanCode. At least I was told that ScanCode would provide such information in the latest versions, but I have never verified that myself. Otherwise we have to assume that all detected licenses are concatenated by AND.
(b) If we have solved (a) the implementation in LicenseInfoResolver should follow the principle to be completely transparent, that means the API must allow to check what the original SPDX expression was, which choices were applied, and what the resulting SPDX expression is.

The question is, should this also be applied to supexpressions automatically?

@mnonnenmacher It was my understanding that it should be applied to choices automatically. This was also covered in this test:
https://github.com/oss-review-toolkit/ort/blob/master/spdx-utils/src/test/kotlin/SpdxExpressionTest.kt#L388-L395

The question is, should this also be applied to supexpressions automatically?

@mnonnenmacher It was my understanding that it should be applied to choices automatically. This was also covered in this test:
https://github.com/oss-review-toolkit/ort/blob/master/spdx-utils/src/test/kotlin/SpdxExpressionTest.kt#L388-L395

At least I was told that ScanCode would provide such information in the latest versions, but I have never verified that myself.

ScanCode 3.2.3 result for a file from h2database which is licensed under MPL-2.0 OR EPL-2.0:
          "matched_rule": {
            "identifier": "mpl-2.0_or_epl-1.0_and_proprietary-license_2.RULE",
            "license_expression": "(mpl-2.0 OR epl-1.0) AND proprietary-license",
            "licenses": [
              "mpl-2.0",
              "epl-1.0",
              "proprietary-license"
            ],
            "is_license_text": false,
            "is_license_notice": true,
            "is_license_reference": false,
            "is_license_tag": false,
            "matcher": "3-seq",
            "rule_length": 54,
            "matched_length": 11,
            "match_coverage": 20.37,
            "rule_relevance": 100
          }

At least I was told that ScanCode would provide such information in the latest versions, but I have never verified that myself.

ScanCode 3.2.3 result for a file from h2database which is licensed under MPL-2.0 OR EPL-2.0:
          "matched_rule": {
            "identifier": "mpl-2.0_or_epl-1.0_and_proprietary-license_2.RULE",
            "license_expression": "(mpl-2.0 OR epl-1.0) AND proprietary-license",
            "licenses": [
              "mpl-2.0",
              "epl-1.0",
              "proprietary-license"
            ],
            "is_license_text": false,
            "is_license_notice": true,
            "is_license_reference": false,
            "is_license_tag": false,
            "matcher": "3-seq",
            "rule_length": 54,
            "matched_length": 11,
            "match_coverage": 20.37,
            "rule_relevance": 100
          }

For the already discussed scope: Project specific for a specific package.
We have multiple possibilities to design the configuration model for the .ort.yml:
(1) This design allows the user to set the absolute desired result license.
- id: package-id
  given: (A OR B) AND C
  choice: A AND C

(2) This design allows the user to set a choice with an optional sub-expression. The desired result license would be computed. Several choices can be given.
- id: `package-id`
  given: (A OR B) AND C
  apply-choice:
  - choice: A
    sub-expression?: A OR B

(3) This design allows the user to EITHER set the desired result license or to set optional sub-expressions with corresponding choices or both.
- id: package-id
  given: (A OR B) AND C
  choice?: A AND C
  sub-expressions?:
  - expression: A OR B
    choice: A

Design (1) would not be possible when applying the license choice without a specific package and would therefore result in a different configuration model.
Design (2) and (3) allow the user to simply choose their desired sub-expressions and their choices without having to calculate the absolute resulting license themselves.
@fviernau, @mnonnenmacher, @sschuberth
What is your preferred solution?

We looked at our use case again. When different given are provided (e.g. detected has two license findings for the same package with different OR expressions) we need to be able to handle that. This is not covered by any of the designs (1) to (3). A possible solutions would be:
(4) The design allows for one package identifier to get several choices that can have sub-expressions. These choices can be applied to declared, detected and concluded, which can have different given license expressions. This design also allows to choose an "absolute" license for a given license expression, as can be seen in the second given choice in the example.
- id: package-id
  choices:
  - given: (A OR B) AND C
    choice: A
    sub-expressions?: A OR B
  - given: (A OR B) AND C
    choice: A AND C
  - given: (D OR F)
    choice: D

What do you think?
@fviernau @sschuberth @mnonnenmacher

Opened a PR for the suggestion by @neubs-bsi


Codepush release-react doesn't find Android app id.

Codepush react-react doesn't find the android app id.
Running:

appcenter codepush release-react -a adrianfernandez/SpotaHome-Android

or with the full command as well:

appcenter codepush release-react -a "adrianfernandez/SpotaHome-Android" -d Production --target-binary-version "4 - 4.6.3" --mandatory

returns the following error:

Error: Command 'codepush release-react -a adrianfernandez/SpotaHome-Android' failed with exception "' adrianfernandez/SpotaHome-Android' is not a valid application id"

The application id seems correct, it is shown in the output of appcenter apps list:

iOS seems fine, I was able to do use codepush using:

appcenter codepush release-react -a adrianfernandez/SpotaHome-iOS -d Production --target-binary-version "4 - 4.6.3" --plist-file='ios/native/Info.plist' --mandatory

I've successfully used codepush in the past with a very similar command:

appcenter codepush release-react -a adrianfernandez/SpotaHome-Android -d Production --target-binary-version '3.6.0 - 3.7.1' --mandatory

Using<EMAIL_ADDRESS>React Native: 0.59.10
react-native-code-push: "^5.6.0"

I also tried to downgrade "appcenter-cli" all the way down to 1.2.2 and the problem still persists.

Alright, so I tried to run locally appcenter-cli and it actually worked.
I did:
-cloned repo (github.com/microsoft/appcenter-cli)
-npm install
-npm run build
and then from my react-native project run:

node ../appcenter-cli/dist/index.js codepush release-react -a adrianfernandez/SpotaHome-Android -d Production --target-binary-version "4 - 4.6.3" --mandatory --debug

Looking at the appcenter dashboard, the codepush release is there correctly now:

Not sure what is the commit that makes it work from master and not from the latest 2.2.1 release.

Thank you for the detailed report. I'm glad you found a workaround but will keep the bug open to investigate it further.

Follow up on the issue (better late than never ;)):
TLDR
We can close the bug, I cannot reproduce it anymore.
Debugging the issue

This wasn't working for me:

appcenter codepush release-react -a adrianfernandez/SpotaHome-Android

Error: Command 'codepush release-react -a adrianfernandez/SpotaHome-Android' failed with exception "' adrianfernandez/SpotaHome-Android' is not a valid application id"

It looks like there is an unwanted space at the beginning of the appName.

This worked

appcenter codepush release-react -a=adrianfernandez/SpotaHome-Android

The difference is in the command params -a <ownerName>/<appName> vs -a=<ownerName>/<appName>.
Appcenter cli code
In this line:
https://github.com/microsoft/appcenter-cli/blob/master/src/util/commandline/app-command.ts#L24
this.appOption has already an incorrect value (with unwanted spacing), so the problem comes probably from the cli argument parsing that happens here:
https://github.com/microsoft/appcenter-cli/blob/master/src/util/commandline/command.ts#L41
Conclusion
After finding the location of the issue, I was trying to debug a bit more and now the original command (with -a <ownerName>/<appName>) works correctly, I cannot reproduce anymore my original issue.
So I guess we can close this bug (finally).

How can you find ENVIRONMENT ?  I have tried :
px appcenter codepush deployment list --app $APPCENTER_PROJECT_ID
Error: Failed to get list of deployments for the app

Hi @kopax-polyconseil , you need to create a new deployment first. You can do it on the AppCenter Portal by navigating "your app -> Distribute -> CodePush" and make sure you have "Production" deployment there. Otherwise just create one.


API section: Accessibility: [MAS1.4.3]:Text elements must have sufficient color contrast against the background.

Build details:
OS Version: 2004(20221.1000)
Edge Dev Version: 87.0.654.0 (Official build) dev (64-bit)
URL: https://fluidframework.com/apis/
Pre-Requisite:
Add Accessibility Insights extension for web to Edge Anaheim (url:https://accessibilityinsights.io/)
Repro Steps:
1. Open the given URL with valid credentials.
2.Now, Run fastpass using Accessibility Insights for Web and observe the failure instances of color-contrast
Actual Result
Text elements are not having sufficient color contrast ratio.
Expected Result:
Text elements must have sufficient color contrast against the background.
Note: This issues is observed for all the "List of.." texts above the all the tables in the page.
User Impact:
Some people with low vision experience low contrast, meaning that there aren't very many bright or dark areas. Everything tends to appear about the same brightness, which makes it hard to distinguish outlines, borders, edges, and details. Text that is too close in luminance (brightness) to the background can be hard to read.
MAS Reference:
MAS 1.4.3 - Contrast (Minimum)

hah...off by 3/100th


Collaboration Service #2 select random interview question for the collaboration

after matching 2 users, need to select a random question from our question database that match the difficulty and topic(s). Bring that question information to the collaboration page

btw one issue is that we don't have one question for every topic so what happens if there are no suitable questions?
we may need to update the question service and the topics list to ensure there at least exists one question of the topic accordingly

btw one issue is that we don't have one question for every topic so what happens if there are no suitable questions?
we may need to update the question service and the topics list to ensure there at least exists one question of the topic accordingly

simplest solution is what u suggested ba, put in more questions for each topic. Can reduce the number of topics in the hard-coded list too


Disable search filter

Hi,
is it possible to disable the search filter?
Actually I haven't found anything which might help in the README.

Actually, I need this feature too :D I will add it possibly tomorrow

Oh you have a PR for that, let me check and test it

Uh that's nice 😎
Check it, but if it does not fit feel free to move it to the basket 🤪

Ah thank you for kind words :) I reviewed it, can you check it? I love to merge your PR

Done and thx for the merge


We should have a table layout widget

For the particular use case I had, I wanted a fixed number of columns all the same size.  I used a bunch of Rows and Columns with Flexible around each item, but we should have a table layout widget to help.  We might want to just add some more knobs to Grid to make it work.

Ok, I'm going to start out by generalizing our existing RenderGrid to handle more regular grid use cases.


version number should reflect tagged version

I'm checking out the latest tag (v2.0.1) and building it via:
  % echo $GOPATH
  /go
  % git clone https://github.com/docker/distribution.git
  % cd distribution
  % git checkout v2.0.1
  % go get -v github.com/docker/distribution/cmd/registry
  % godep restore
  % DISTRIBUTION_DIR=/go/src/github.com/docker/distribution
  % GOPATH=$DISTRIBUTION_DIR/Godeps/_workspace:$GOPATH
  % make PREFIX=/go clean binaries
```

But then the version information still reports v2.0.0:

% /go/bin/registry -version
/go/bin/registry github.com/docker/distribution v2.0.0+unknown

I'd expect that to be v2.0.1 (or if I'm holding godep wrong - since 'godep get' inside the git repository doesn't seem to work - then at least v2.0.1+unknown or similar). Am I'm holding it wrong?

You probably want to synch the version info from the git info.
Have a look into the makefile here: https://github.com/docker/distribution/blob/master/Makefile#L16

Please read the document on building. The version number is only set If the Makefile is used, since we do this through the linker. I suspect that the version you are running was not actually built by the Makefile. With the flow you're using, calling go get is not necessary but may have built the binary.
What is the output of each command?

@dmp42: I noticed  ./version/version.sh and tried to play with it, but manually running it as part of the way I built the binary doesn't seem to work, I assume version/version.go should be kept up2date if it's part of the git repository with the according version tag information?
@stevvooe I did read the building document of course, but I assumed that "Build the development environment" would be enough, especially since the version number is event explicitly mentioned there. :)
I also tried to follow the "Repeatable Builds" steps but this doesn't work for me, it looks like it's not using the pinned versions from Godeps/Godeps.json, because I'm running into build failures which I assume shouldn't happen with v2.0.1? Output available here: https://gist.github.com/mika/1c473742bcfb566359ab
Output of each command from my build process: https://gist.github.com/mika/b8422537dc363213ba98

@mika From the output, it looks like the GOPATH has a newer version the distribution packages. The build failures indicate that you're trying to build the v2.0.1 code base from the master package. It is possible that a prior build is exposing the new interfaces when you need to build against the old one. Make sure there isn't a fresh checkout somewhere in the GOPATH and delete the pkg directory if that doesn't help.
Looking closer at the output, it looks like the problem may be that you haven't properly checked out the code base in the Go workspace:
_/home/mika/docker-distribution
_/home/mika/docker-distribution/cmd/dist

Generally, this is covered when reading the build instructions and issuing the following command:
go get github.com/docker/distribution/cmd/registry

This will checkout the distribution project into the GOPATH and you can develop there. This will likely fail when building but that doesn't matter. Once that is complete, you can change into that directory ($GOPATH/src/github.com/docker/distribution), checkout the version you want to build and work from there.
Please make sure you are following how to write Go code when setting up the development environment. I'll add a section to the building instructions to make this is apparent.
Regarding ./version/version.sh, please read the comments carefully. In particular, this one:
// Version indicates which version of the binary is running. This is set to
// the latest release tag by hand, always suffixed by "+unknown". During
// build, it will be replaced by the actual version. The value here will be
// used if the registry is run after a go get based install.

The value generated there is simply so that builds that don't have the linker set the version get a reasonable version number. It can never be checked in correctly, since the very act of making a commit would change the version number. The real version number is detected in this line of the Makefile:
GO_LDFLAGS=-ldflags "-X `go list ./version`.Version $(VERSION)"

@mika I've added clarification to the build instructions #591.

@stevvooe thanks for your assistance, this clarifies it for me! I got confused by the build failure of  go get github.com/docker/distribution/cmd/registry (see https://gist.github.com/mika/cf5ebd90fadad9aa90c0 which is https://github.com/docker/distribution/issues/573) and didn't think about just switching to $GOPATH/src/github.com/docker/distribution to continue with according builds. :)
So this is what worked for me:
# GOPATH=/go go get golang.org/x/tools/cmd/vet # this was additionally needed

% export GOPATH=/go
% export PATH=$PATH:/go/bin
% go get github.com/docker/distribution/cmd/registry # ignoring mentioned build failure
% go get github.com/tools/godep github.com/golang/lint/golint
% cd $GOPATH/src/github.com/docker/distribution
% git checkout v2.0.1
% GOPATH=`godep path`:$GOPATH make
% /go/src/github.com/docker/distribution/bin/registry -version
/go/src/github.com/docker/distribution/bin/registry github.com/docker/distribution v2.0.1.m

As follow up to "Regarding ./version/version.sh, please read the comments carefully. [...] It can never be checked in correctly, since the very act of making a commit would change the version number. ": I don't really see the problem since version/version.go is a file being under gits version control. Before tagging a new release just raise the version number in version/version.go to the version you want to tag, so it would be setting "v2.0.1+unknown" in version/version.go in one commit, then tag this commit as "v2.0.1" and it's clear then? This is the workflow I'm used to if the version number is supposed to be hard-coded somewhere.

That build failure is obnoxious. It is due to the azure drivers. We may want to disable them by default, since go get should always work.

As follow up to "Regarding ./version/version.sh, please read the comments carefully. [...] It can never be checked in correctly, since the very act of making a commit would change the version number. ": I don't really see the problem since version/version.go is a file being under gits version control. Before tagging a new release just raise the version number in version/version.go to the version you want to tag, so it would be setting "v2.0.1+unknown" in version/version.go in one commit, then tag this commit as "v2.0.1" and it's clear then? This is the workflow I'm used to if the version number is supposed to be hard-coded somewhere.

We do not hardcode the version number and the one present there is just to provide a reference. When we see it, we know that it was built after that commit and definitely not with the Makefile. The version number is set by the tag name, which is PGP signed. For my current environment, the version is v2.0.0-168-gb9eeb32. If I make a commit, that will change, so it cannot be committed accurately.
Really, this doesn't matter at all. We're going to take a slightly different approach for the next release.

Stephen Day [Wed Jun 03, 2015 at 06:06:47PM -0700]:

That build failure is obnoxious. It is due to the azure drivers.
We may want to disable them by default, since go get should
always work.

Sounds good, thanks.

We do not hardcode the version number and the one present there is
just to provide a reference.

Sure, I just recommended that it might be good to raise the version
number when tagging a new release so the reference is more accurate.

When we see it, we know that it was built after that commit and
definitely not with the Makefile.

Agreed and understood.

The version number is set by the tag name, which is PGP signed.

This doesn't seem to be the case though (or we're still talking
about different things? :)). But both master as well as the v2.0.1
tag contain 'v2.0.0+unknown', while that could (and IMHO should) be
'v2.0.1+unknown' nowadays (and that's all I care about and am asking
for, but of course feel free to disagree and ignore it).

For my current environment, the
version is v2.0.0-168-gb9eeb32. If I make a commit, that will
change, so it cannot be committed accurately.

Sure.

Really, this doesn't matter at all. We're going to take a slightly
different approach for the next release.

Ok :)
regards,
-mika-


Visualizer Doesn't move

I installed Monstercat Visualizer, and I have the same problem as most other people, the visualizer doesn't move. The spotify plugin is fine and I know I have to uninstall Realtek to get the visualizer working but if I uninstall Realtek then I loose sound through my headphones.

Rainmeter Version: 4.2.0
OS Version: Windows 10 Home 64 Bit
Spotify Version: <IP_ADDRESS>
#93

You should have no issues after a reboot

Which reboot?

Not sure, feel free to try and report back.

So i uninstalled realtek and rebooted it but still can't get sound from headphones.

Go into the windows device manager and find the device and tell windows to install the drivers for it

There are none

@og-rahoon Any progress on the issue?

If you open your menu and check bar count what do you see? Because I have a similar issue where if you want to change the bar count it goes to 0 and it becomes unable to move to a higher count. What makes it invisible.

Rainmeter 4.2.0
OS: Windows 10 Prof  64 bit

@Ruben533 Sounds like the Mouse plugin wasn't properly installed, try to run the rmskin and install all necessary plugins.

Thank you for your feedback. I will try 👍

I'm sorry I've mistaken myself I've checked again and it was bar width that jumps to 0 and you cant move the bar count works just fine. :/  Sorry

@Ruben533 If your problem is still there and it is not the same, i would recommend to create your own issue so we don't clutter this one. :)


Any way to skip the countdown? (Lua)

What is your question?
the title says it all

setProperty('skipCountdown', true)

function onCreatePost()
    setProperty('skipCountdown', true);
end

didn't work

onCreate()

onCreate()

onCreate()

Thanks, it works now


Compilation Error.

Issue Description
Please describe our issue, along with:

Compilation Error

Version Information
Please indicate relevant versions, including, if relevant:

Deeplearning4j version:
CURRENT SNAPSHOT

Platform information (OS, etc):
Windows 10 Pro 20H2 (64Bit)
AMD Ryzen 5 5600X 6-Core Processor
GeForce RTX 3070

CUDA version, if used:
v11.0

NVIDIA driver version, if in use:
v457.51 (DCH)

Additional Information
gcc.exe (Rev1, Built by MSYS2 project) 11.2.0
cmake version 3.21.4
Visual Studio 2019 (Community)

Full log or exception stack trace (ideally in a Gist: gist.github.com):

cd /c/Users/Administrator/Downloads/deeplearning4j
export JAVA_HOME=/c/Program\ Files/Java/jdk1.8.0_281
./change-cuda-versions.sh 11.0
cd libnd4j
export LIBND4J_HOME=pwd
./buildnativeoperations.sh -a native -c cuda -v 11.0 -h cudnn -p msi -j 12 clean true

.
.
.
[297/841] C:\PROGRA~1\NVIDIA~2\CUDA\v11.0\bin\nvcc.exe -forward-unknown-to-host-compiler --options-file blas\CMakeFiles\samediff_obj.dir_\include\ops\declarable\helpers\cuda\dilation2d.cu.obj.rsp -MD -MT blas\CMakeFiles\samediff_obj.dir_\include\ops\declarable\helpers\cuda\dilation2d.cu.obj -MF blas\CMakeFiles\samediff_obj.dir_\include\ops\declarable\helpers\cuda\dilation2d.cu.obj.d -x cu -c C:\Users\Administrator\Downloads\deeplearning4j\libnd4j\include\ops\declarable\helpers\cuda\dilation2d.cu -o blas\CMakeFiles\samediff_obj.dir_\include\ops\declarable\helpers\cuda\dilation2d.cu.obj -Xcompiler=-Fdblas\CMakeFiles\samediff_obj.dir,-FS
dilation2d.cu
ALL OPS ARE DEFINED
C:/Users/Administrator/Downloads/deeplearning4j/libnd4j/include\types/types.h(242) : WARNING: MSVC old preprocessor
[298/841] C:\PROGRA~1\NVIDIA~2\CUDA\v11.0\bin\nvcc.exe -forward-unknown-to-host-compiler --options-file blas\CMakeFiles\samediff_obj.dir_\include\ops\declarable\helpers\cuda\gradient.cu.obj.rsp -MD -MT blas\CMakeFiles\samediff_obj.dir_\include\ops\declarable\helpers\cuda\gradient.cu.obj -MF blas\CMakeFiles\samediff_obj.dir_\include\ops\declarable\helpers\cuda\gradient.cu.obj.d -x cu -c C:\Users\Administrator\Downloads\deeplearning4j\libnd4j\include\ops\declarable\helpers\cuda\gradient.cu -o blas\CMakeFiles\samediff_obj.dir_\include\ops\declarable\helpers\cuda\gradient.cu.obj -Xcompiler=-Fdblas\CMakeFiles\samediff_obj.dir,-FS
FAILED: blas/CMakeFiles/samediff_obj.dir/__/include/ops/declarable/helpers/cuda/gradient.cu.obj
C:\PROGRA~1\NVIDIA~2\CUDA\v11.0\bin\nvcc.exe -forward-unknown-to-host-compiler --options-file blas\CMakeFiles\samediff_obj.dir_\include\ops\declarable\helpers\cuda\gradient.cu.obj.rsp -MD -MT blas\CMakeFiles\samediff_obj.dir_\include\ops\declarable\helpers\cuda\gradient.cu.obj -MF blas\CMakeFiles\samediff_obj.dir_\include\ops\declarable\helpers\cuda\gradient.cu.obj.d -x cu -c C:\Users\Administrator\Downloads\deeplearning4j\libnd4j\include\ops\declarable\helpers\cuda\gradient.cu -o blas\CMakeFiles\samediff_obj.dir_\include\ops\declarable\helpers\cuda\gradient.cu.obj -Xcompiler=-Fdblas\CMakeFiles\samediff_obj.dir,-FS
C:/Users/Administrator/Downloads/deeplearning4j/libnd4j/include/ops/declarable/helpers/cuda/gradient.cu(32): error: On Windows, the enclosing parent function ("applyGradientDescent_") for an extended device lambda cannot have internal or no linkage
detected during instantiation of "void sd::ops::helpers::applyGradientDescent_(sd::LaunchContext *, sd::NDArray *, sd::NDArray *, double, sd::NDArray *) [with T=bfloat16]"
(40): here
C:/Users/Administrator/Downloads/deeplearning4j/libnd4j/include/ops/declarable/helpers/cuda/gradient.cu(32): error: On Windows, the enclosing parent function ("applyGradientDescent_") for an extended device lambda cannot have internal or no linkage
detected during instantiation of "void sd::ops::helpers::applyGradientDescent_(sd::LaunchContext *, sd::NDArray *, sd::NDArray *, double, sd::NDArray *) [with T=float]"
(40): here
C:/Users/Administrator/Downloads/deeplearning4j/libnd4j/include/ops/declarable/helpers/cuda/gradient.cu(32): error: On Windows, the enclosing parent function ("applyGradientDescent_") for an extended device lambda cannot have internal or no linkage
detected during instantiation of "void sd::ops::helpers::applyGradientDescent_(sd::LaunchContext *, sd::NDArray *, sd::NDArray *, double, sd::NDArray *) [with T=double]"
(40): here
C:/Users/Administrator/Downloads/deeplearning4j/libnd4j/include/ops/declarable/helpers/cuda/gradient.cu(32): error: On Windows, the enclosing parent function ("applyGradientDescent_") for an extended device lambda cannot have internal or no linkage
detected during instantiation of "void sd::ops::helpers::applyGradientDescent_(sd::LaunchContext *, sd::NDArray *, sd::NDArray *, double, sd::NDArray *) [with T=float16]"
(40): here
4 errors detected in the compilation of "C:/Users/Administrator/Downloads/deeplearning4j/libnd4j/include/ops/declarable/helpers/cuda/gradient.cu".
gradient.cu
.
.
.

@baedaron could you please let us know your git hash of your commit for the compilation error? Thanks! cc @quickwritereader

Do you mean this?

thanks for the help. we will need you to confirm that everything works as intended when we apply new changes.
if you are in hurry, you could check my repo to see if it solves your problem. thanks
 git clone -b qwr_refactor https://github.com/quickwritereader/deeplearning4j.git

ok, I will wait. thanks.

@baedaron could you check if the latest master fixes the compilation error for you
thanks

:(
[  2%] Building CXX object blas/CMakeFiles/samediff_obj.dir//include/loops/cpu/scalar_int.cpp.obj
cd /C/Users/Administrator/Downloads/deeplearning4j/libnd4j/blasbuild/cpu/blas && /C/MSys64/mingw64/bin/g++.exe -DCPU_FEATURES=true -DDNNL_ENABLE_CONCURRENT_EXEC=true -DSD_ALL_OPS=1 -D__CPUBLAS=true -D__EXTERNAL_BLAS__=true @CMakeFiles/samediff_obj.dir/includes_CXX.rsp -DSD_WINDOWS_BUILD=true -Wa,-mbig-obj -mmmx -msse -msse2 -msse3 -msse4.1 -msse4.2 -mavx -mavx2 -mfma -mf16c -mprefetchwt1 -DSD_F16C=true -DF_AVX2=true -mno-avx256-split-unaligned-load -mno-avx256-split-unaligned-store -fmax-errors=49999  -fopenmp -O3 -fPIC -D_RELEASE=true -std=gnu++11 -MD -MT blas/CMakeFiles/samediff_obj.dir//include/loops/cpu/scalar_int.cpp.obj -MF CMakeFiles/samediff_obj.dir//include/loops/cpu/scalar_int.cpp.obj.d -o CMakeFiles/samediff_obj.dir/__/include/loops/cpu/scalar_int.cpp.obj -c /C/Users/Administrator/Downloads/deeplearning4j/libnd4j/include/loops/cpu/scalar_int.cpp
[  2%] Built target utils
In file included from C:/Users/Administrator/Downloads/deeplearning4j/libnd4j/include/exceptions/allocation_exception.h:25,
from C:/Users/Administrator/Downloads/deeplearning4j/libnd4j/include/system/op_boilerplate.h:72,
from C:/Users/Administrator/Downloads/deeplearning4j/libnd4j/include/cnpy/cnpy.h:37,
from C:/Users/Administrator/Downloads/deeplearning4j/libnd4j/include/legacy/impl/cnpy.cpp:29:
C:/Users/Administrator/Downloads/deeplearning4j/libnd4j/include/system/common.h:154:22: error: 'uint64_t' does not name a type
154 | using UnsignedLong = uint64_t;
|                      ^~~~~~~~
C:/Users/Administrator/Downloads/deeplearning4j/libnd4j/include/system/common.h:71:1: note: 'uint64_t' is defined in header ''; did you forget to '#include '?
70 | #include <unordered_map>
+++ |+#include
71 | #define SD_MAP_IMPL std::unordered_map
In file included from C:/Users/Administrator/Downloads/deeplearning4j/libnd4j/include/array/PointerDeallocator.h:28,
from C:/Users/Administrator/Downloads/deeplearning4j/libnd4j/include/array/PointerWrapper.h:28,
from C:/Users/Administrator/Downloads/deeplearning4j/libnd4j/include/array/ConstantDataBuffer.h:26,
from C:/Users/Administrator/Downloads/deeplearning4j/libnd4j/include/helpers/ConstantTadHelper.h:26,
from C:/Users/Administrator/Downloads/deeplearning4j/libnd4j/include/loops/cpu/reduce/reduce_same.cpp:23:
C:/Users/Administrator/Downloads/deeplearning4j/libnd4j/include/system/common.h:154:22: error: 'uint64_t' does not name a type
154 | using UnsignedLong = uint64_t;
|                      ^~~~~~~~
C:/Users/Administrator/Downloads/deeplearning4j/libnd4j/include/system/common.h:71:1: note: 'uint64_t' is defined in header ''; did you forget to '#include '?
70 | #include <unordered_map>
+++ |+#include
71 | #define SD_MAP_IMPL std::unordered_map
In file included from C:/Users/Administrator/Downloads/deeplearning4j/libnd4j/include/array/ArrayOptions.h:26,
from C:/Users/Administrator/Downloads/deeplearning4j/libnd4j/include/array/NDArray.h:22,
from C:/Users/Administrator/Downloads/deeplearning4j/libnd4j/include/helpers/StringUtils.h:29,
from C:/Users/Administrator/Downloads/deeplearning4j/libnd4j/include/helpers/DebugHelper.h:26,
from C:/Users/Administrator/Downloads/deeplearning4j/libnd4j/include/loops/pairwise_bool.h:29,
from C:/Users/Administrator/Downloads/deeplearning4j/libnd4j/include/loops/cpu/pairwise_bool.cpp:22:
C:/Users/Administrator/Downloads/deeplearning4j/libnd4j/include/system/common.h:154:22: error: 'uint64_t' does not name a type
154 | using UnsignedLong = uint64_t;
|                      ^~~~~~~~

I see . Could you include what it wants and try again

I believe we solved it. at least for cpu builds. probably it will work for cuda builds as well.
when you be free we'd like to get the feedback

Today, I just pulled again and tried to recompile.

was there any problem?

And here is git log.
git log
commit fdff567d5c771e9e5c17fcb0bb35b8f92c94006f (HEAD -> master, origin/master, origin/HEAD)
Merge: cca5ba09b7 2e778c6d66
Author: Adam Gibson<EMAIL_ADDRESS>Date:   Wed Nov 10 18:45:12 2021 +0900
Merge pull request #9518 from eclipse/ag_tests_3

More test fixes as follow up to previous test fix PR

commit 2e778c6d663e2f970ab8f8b6eb2119eaf66ae22d
Merge: 55c570064b 7b418c932f
Author: Adam Gibson<EMAIL_ADDRESS>Date:   Wed Nov 10 18:42:50 2021 +0900
Merge branch 'ag_tests_3' of https://github.com/eclipse/deeplearning4j into ag_tests_3

commit 55c570064b05776073991b11659c7596e4daa6d5
Author: Adam Gibson<EMAIL_ADDRESS>Date:   Wed Nov 10 18:42:41 2021 +0900
Update .gitignore

.
.
.

Yes, cpu build was ok.

/declarable/helpers/cuda/gradient.cu(32): error: On Windows, the enclosing parent function ("applyGradientDescent_") for an extended device lambda cannot have internal or no linkage detected during instantiation of "void sd::ops::helpers::applyGradientDescent_(sd::LaunchContext *, sd::NDArray *, sd::NDArray *, double, sd::NDArray *) [with T=bfloat16]" (40): here

ok here it was complaining because on Windows it dislikes having static

C:/Users/Administrator/Downloads/deeplearning4j/libnd4j/include\ops/ops.h(2303): error: expression must be a modifiable lvalue

here it complains because mvsc is not set to use their new preprocessor so it simply cant handle - before SD_MIN_CUTOFF

Yes, I retried and confirmed it has been much improved.
But... now following problem...
both cpu and cuda buildnativeoperations.sh were ok.
mvn clean install -B -V -U -T 12 -e -pl 'org.nd4j:nd4j-cuda-11.0-preset,org.nd4j:nd4j-cuda-11.0'
-Dlibnd4j.platform=windows-x86_64 -Djavacpp.platform=windows-x86_64 -Dlibnd4j.chip=cuda -Dlibnd4j.cuda=11.0 -Dlibnd4j.compute=8.6 -Dlibnd4j.helper=cudnn -Djavacpp.helper=cudnn
-DskipTests -Dmaven.javadoc.skip=true -Dmaven.test.skip=true
Microsoft (R) Incremental Linker Version 14.29.30136.0
Copyright (C) Microsoft Corporation.  All rights reserved.
.
.
.
jnind4jcuda.obj : error LNK2001: unresolved external symbol "public: static void __cdecl sd::sparse::SparseUtils<unsigned int>::sortCooIndicesGeneric(__int64 *,void *,__int64,int)" (?sortCooIndicesGeneric@?$SparseUtils@I@sparse@sd@@SAXPEA_JPEAX_JH@Z)
jnind4jcuda.obj : error LNK2001: unresolved external symbol "public: static void __cdecl sd::sparse::SparseUtils<signed char>::sortCooIndicesGeneric(__int64 *,void *,__int64,int)" (?sortCooIndicesGeneric@?$SparseUtils@C@sparse@sd@@SAXPEA_JPEAX_JH@Z)
jnind4jcuda.obj : error LNK2001: unresolved external symbol "public: static void __cdecl sd::sparse::SparseUtils<__int64>::sortCooIndicesGeneric(__int64 *,void *,__int64,int)" (?sortCooIndicesGeneric@?$SparseUtils@_J@sparse@sd@@SAXPEA_JPEAX_JH@Z)
jnind4jcuda.obj : error LNK2001: unresolved external symbol "public: static void __cdecl sd::sparse::SparseUtils<float>::sortCooIndicesGeneric(__int64 *,void *,__int64,int)" (?sortCooIndicesGeneric@?$SparseUtils@M@sparse@sd@@SAXPEA_JPEAX_JH@Z)
jnind4jcuda.obj : error LNK2001: unresolved external symbol "public: static void __cdecl sd::sparse::SparseUtils<unsigned char>::sortCooIndicesGeneric(__int64 *,void *,__int64,int)" (?sortCooIndicesGeneric@?$SparseUtils@E@sparse@sd@@SAXPEA_JPEAX_JH@Z)
jnind4jcuda.obj : error LNK2001: unresolved external symbol "public: static void __cdecl sd::sparse::SparseUtils<struct bfloat16>::sortCooIndicesGeneric(__int64 *,void *,__int64,int)" (?sortCooIndicesGeneric@?$SparseUtils@Ubfloat16@@@sparse@sd@@SAXPEA_JPEAX_JH@Z)
jnind4jcuda.obj : error LNK2001: unresolved external symbol "public: static void __cdecl sd::sparse::SparseUtils<double>::sortCooIndicesGeneric(__int64 *,void *,__int64,int)" (?sortCooIndicesGeneric@?$SparseUtils@N@sparse@sd@@SAXPEA_JPEAX_JH@Z)
jnind4jcuda.obj : error LNK2001: unresolved external symbol "public: static void __cdecl sd::sparse::SparseUtils<int>::sortCooIndicesGeneric(__int64 *,void *,__int64,int)" (?sortCooIndicesGeneric@?$SparseUtils@H@sparse@sd@@SAXPEA_JPEAX_JH@Z)
jnind4jcuda.obj : error LNK2001: unresolved external symbol "public: static void __cdecl sd::sparse::SparseUtils<short>::sortCooIndicesGeneric(__int64 *,void *,__int64,int)" (?sortCooIndicesGeneric@?$SparseUtils@F@sparse@sd@@SAXPEA_JPEAX_JH@Z)
jnind4jcuda.obj : error LNK2001: unresolved external symbol "public: static void __cdecl sd::sparse::SparseUtils<bool>::sortCooIndicesGeneric(__int64 *,void *,__int64,int)" (?sortCooIndicesGeneric@?$SparseUtils@_N@sparse@sd@@SAXPEA_JPEAX_JH@Z)
jnind4jcuda.obj : error LNK2001: unresolved external symbol "public: static void __cdecl sd::sparse::SparseUtils<unsigned __int64>::sortCooIndicesGeneric(__int64 *,void *,__int64,int)" (?sortCooIndicesGeneric@?$SparseUtils@_K@sparse@sd@@SAXPEA_JPEAX_JH@Z)
jnind4jcuda.obj : error LNK2001: unresolved external symbol "public: static void __cdecl sd::sparse::SparseUtils<struct float16>::sortCooIndicesGeneric(__int64 *,void *,__int64,int)" (?sortCooIndicesGeneric@?$SparseUtils@Ufloat16@@@sparse@sd@@SAXPEA_JPEAX_JH@Z)
jnind4jcuda.obj : error LNK2001: unresolved external symbol "public: static void __cdecl sd::sparse::SparseUtils<unsigned short>::sortCooIndicesGeneric(__int64 *,void *,__int64,int)" (?sortCooIndicesGeneric@?$SparseUtils@G@sparse@sd@@SAXPEA_JPEAX_JH@Z)
jnind4jcuda.dll : fatal error LNK1120: 13 unresolved externals.
[INFO] ------------------------------------------------------------------------
[INFO] Reactor Summary for nd4j-cuda-preset 1.0.0-SNAPSHOT:
[INFO]
[INFO] nd4j-cuda-preset ................................... SUCCESS [ 36.643 s]
[INFO] nd4j-cuda .......................................... FAILURE [ 28.269 s]
[INFO] ------------------------------------------------------------------------
.
.
.

I see. I only compiled libnd4j
Could you try again by changing line  libnd4j/include/ops/specials_sparse.h:38
class SparseUtils
With
class SD_LIB_EXPORT SparseUtils

Ok~~~ It worked.
But another one.

alright I was told that its better to show you our workflow files and build logs . there you can proper command lines. Also note that  there are places where you should put hyphen  here before avx,  -Djavacpp.platform.extension=-avx2
mvn   -Dhttp.keepAlive=false -Dmaven.wagon.http.pool=false -Dmaven.wagon.http.retryHandler.count=3   -Possrh -Dlibnd4j.buildthreads=1 -Djavacpp.platform=windows-x86_64   -Dlibnd4j.platform=windows-x86_64 deploy -DskipTests -Dlibnd4j.classifier=windows-x86_64-avx2 -Dlibnd4j.extension=avx2 -Djavacpp.platform.extension=-avx2 -Djavacpp.platform=windows-x86_64   -Dlibnd4j.platform=windows-x86_64    deploy -DskipTests
See this link to get more info workflow

@baedaron were you able to build?

No! :(
I cannot understand why CPU build could not find the selected project in the reactor: org.nd4j:nd4j-native-preset.

did you add -Pcpu while building using mvn?

No.
Ok, I will try if some time available.

It worked.
thanks a lot. :)


enhancement: run tests & auto-upgrade

see updtr.
I like this package, and I like that package too.  I wonder if you guys could come together on something?  Failing that, roll updtr's functionality into npm-check?
Just an idea.

I love the idea about test&upgrade. From my understanding, it could be great to select some packages in npm-check, press Alt+Enter (or other keys) to do test&upgrade.


Refactor code

@jcbeyler @ryrose  @caoman Some code refactoring. Decoupling parts of the code.

That is a big change, any way it could get cut up a bit; otherwise it will take me some time to get through it all ;-)

Sorry I can try breaking it down into smaller pieces. I haven't added or removed any code. I just moved things around.

It might be fine, it just seemed like a lot of code. Let me look again
later :)
On Mon, Apr 9, 2018 at 7:56 PM Leandro Watanabe<EMAIL_ADDRESS>wrote:

Sorry I can try breaking it down into smaller pieces. I haven't added or
removed any code. I just moved things around.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/leandrohw/et2-simulator/pull/1#issuecomment-379957824,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AMs8xq0mturrE1_znHszcIXf4AEgBBwzks5tnB9EgaJpZM4TNd_-
.

Sounds good :-) I don't want to waste too much of your time. I can break it down a little bit if it's too inconvenient. The main thing was separating the node and method classes from tree and separating parsing - simulation work.

This is the first commit and I wanted to stay true to the original code. The functionality of the code shouldn't change. It was mainly clean up. For now, I will assume the core functionality hasn't changed and I will add tests as I add new features.

LGTM except for the bug then. All other comments are notes on how to refactor the code if we wanted to


Amethyst Knife just has better stats than the Amethyst Sword.

Amethyst sword has 6 damage and 1.6 attack speed
Amethyst knife has 6 damage and 3 attack speed
seems like an oversight of some sort

Fixed in dev!


Added files

Added files from working copy.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/enterprise/admin-request.md
:white_check_mark:Succeeded

docs/enterprise/configure-tenant.md
:white_check_mark:Succeeded

docs/enterprise/measure.md
:white_check_mark:Succeeded

docs/enterprise/media/images/enterprise-it_portal-approve.png
:white_check_mark:Succeeded

docs/enterprise/media/images/enterprise-it_portal-certification_records.png
:white_check_mark:Succeeded

docs/enterprise/media/images/enterprise-it_portal-reject.png
:white_check_mark:Succeeded

docs/enterprise/media/images/enterprise-it_portal-skill.png
:white_check_mark:Succeeded

docs/enterprise/media/images/enterprise-tenant_publishing-not_configured.png
:white_check_mark:Succeeded

docs/enterprise/monitor.md
:white_check_mark:Succeeded

docs/enterprise/request-deployment.md
:white_check_mark:Succeeded

docs/enterprise/toc.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


android-async-http 1.4.9 isConnected failed: ECONNREFUSED (Connection refused)  in wifi

android-async-http 1.4.9 isConnected failed: ECONNREFUSED (Connection refused)  in wifi,but 2g/3g/4g is good,but android-async-http 1.3.6 is good in wifi.Why?

java.net.ConnectException: failed to connect to localhost/<IP_ADDRESS> (port 8080) after 30000ms: isConnected failed: ECONNREFUSED (Connection refused)

maybe here is the problem #1059

android-async-http 1.4.9 :
java.net.ConnectException: failed to connect to localhost/<IP_ADDRESS> (port 8080) after 3000ms: isConnected failed: ECONNREFUSED (Connection refused)

我也出现相同的问题了
java.net.ConnectException: failed to connect to ip6-localhost/::1 (port 8080) after 30000ms: isConnected failed: ECONNREFUSED (Connection refused)

android-async-http 1.4.9 :
java.net.ConnectException: failed to connect to localhost/<IP_ADDRESS> (port 8080) after 3000ms: isConnected failed: ECONNREFUSED (Connection refused)


Update import-collections.mdx

corrected spelling mistake for colleagues

Closing this since it has already been updated in the documentation.


Check async query status fails for by_team and by_api

Checking query status via /v1/check_query_status/:id?by=team or /v1/check_query_status/:id?by=api fails validation due to the query parameter by not being expected in /docs/smartapi.yaml.

is this supposed to work? I wasn't aware that this was possible or what this would do...

This is used for checking the status of async queries made to /v1/smartapi/:smartapi_id/asyncquery (async equivalent of /v1/smartapi/:smartapi_id/query), and /v1/team/:team_name/asyncquery (equiv. /v1/team/team_name/query).
Each endpoint has a separate queue -- the by query parameter supports selecting the appropriate queue to get the job status from.

Is there an example of exactly what the GET url is and what the response will look like?
It sounds like I could add the needed info into the smartapi spec if I know this...

An example GET URL would be https://api.bte.ncats.io/v1/check_query_status/0VjwhsxPib?by=team (example ID, wouldn't actually pull anything up), which I would expect to return the job status of an asynchronous query (exactly the same as a normal check_query_status), like this: https://api.bte.ncats.io/v1/check_query_status/f5jhKt9dtH

so it would definitely say by=team rather than say a specific team like by=Service Provider?

by is only ever be equal to the string team or the string api, and never anything else.

I have confirmed locally:

starting BTE w/ redis (REDIS_HOST=localhost REDIS_PORT=6379 RESULT_CACHING=false npm run debug --workspace='@biothings-explorer/bte-trapi')
sending a query to http://localhost:3000/v1/smartapi/671b45c0301c8624abbd26ae78449ca2/asyncquery
the query completes (I can see the log), but trying to check the job leads to this response:

{
    "error": "Your input query graph is invalid",
    "more_info": [
        "query should NOT have additional properties 'by'"
    ]
}

I'm going to look into changing the smartapi specs now...

so this branch's commit should address this issue: https://github.com/biothings/BioThings_Explorer_TRAPI/commit/0a9a5cc93e992de232d6ee6fe5867470658ce06a
However, I have questions on smartapi annotation of the response to this endpoint, which I detail here: https://suwulab.slack.com/archives/CC218TEKC/p1638934317332400

Confirmed on my local that this branch does fix this issue. Depending on the Slack discussion, we can wait for any other changes to be made and then start testing a PR on the dev instance.

I've pushed a new commit to this repo. If it continues to work well....do we want this as an after-demo change or before?

@colleenXu I imagine this is going to be a post-demo fix as it shouldn't affect anything in the demo (@andrewsu can confirm?).
I'll run this on dev tomorrow so we can have it ready to go.

definitely should not be deployed before the demo...
Also, it's not entirely clear to me what ?by=team and ?by=api are meant to accomplish?  Can someone point to the relevant discussion/rationale, or add that here to this issue if it isn't already organized elsewhere?

@andrewsu As far as I understand this was as originally implemented for async. I don't know if there was any discussion, however the rationale seems to be the following:

There are separate queues for each async endpoint (I'm not sure why..this could be a separate discussion)
In order to check the status of a given job, the correct queue must be checked, and there's no way to tell this from the jobId (JobIds could be changed to support this, however they would be "less anonymous" in that case, carrying specific information...though not much of impact...)
By giving the user a query parameter in their URL to check the query status, it's easy to tell which queue to check

I think that's really all there is to it...

@colleenXu @andrewsu requirement for this query parameter is removed by updates made in the above PR

@tokebe should this issue be closed then?


Inconsistent "Migrations" folder path when Re-Scaffold

Re-Scaffold migrations created "Migrations" folder in project root instead of the original path /Data/Migrations.  It still works but a consistent folder path is better.
Steps to reproduce the issue

Create an "ASP.NET Core Web Application (.NET Core) with "Individual User Accounts" authentication
Delete the "/Data/Migrations" folder
Run "Add-Migration MyDB" in Package Manager Console
The "Migrations" folder is created in the project root instead of "/Data"

Migrations will be placed in whatever folder the Model Snapshot file is located in, but the default has always been /Migrations.  Since they included a snapshot file in Data/Migrations in the new templates, it scaffolds additional migrations there.
They should probably change the default to match the templates.
In the meantime, a workaround does exist... one of the parameters for the migration add command includes specifying the path.
From the ef docs:
Usage: dotnet ef migrations add [arguments] [options]

Arguments:
  [name]  The name of the migration

Options:
  -o|--output-dir <path>          The directory (and sub-namespace) to use. If omitted, "Migrations" is used. Relative paths are relative the directory in which the command is executed.
  -c|--context <context>          The DbContext to use. If omitted, the default DbContext is used
  -e|--environment <environment>  The environment to use. If omitted, "Development" is used.
  --json                          Use json output. JSON is wrapped by '//BEGIN' and '//END'
  -h|--help                       Show help information
  -v|--verbose                    Enable verbose output

To run your intended migration with your output going to the Data/Migrations folder, you'd do the following:
dotnet ef migrations add InitialMigration -o Data/Migrations

@rdefreitas, thanks for the explanation and workaround!

@stevesiu glad it worked out for you :)

This issue is being closed because it has not been updated in 3 months.
We apologize if this causes any inconvenience. We ask that if you are still encountering this issue, please log a new issue with updated information and we will investigate.


Tabs: Remove Workload component styling customization

Description
Remove current component styling for Workload and create further prop for nesting case.
Context
Remove current styling from Workload in favor of a prop
.deployment-tabs {
  > .tabs.horizontal {
    border-bottom: 1px solid var(--border);
    margin-bottom: 10px;
  }
}

Current look

Style highlight

Look for the component without customization

Purely presentational - manually test


Use parsing from Intervals.jl

Makes use of the change in invenia/Intervals.jl#128. Originally I made this change in #195 but when I forced pushed the update the PR refused to update the commit.

Something seems wrong with GitHub: https://www.githubstatus.com/

Both nightly and 1.5 are failing as they aren't seeing Intervals 1.4. Maybe the new package server takes longer to sync up?

Restarting CI and hopefully Julia 1.5+ will have Intervals 1.4 available


OpenUnison - AzureAD - K8S - include_auth_chain: "azuread-load-groups" - Could not find user

Hello,
I Installed ths solution, but I had a problem.
When include_auth_chain: "azuread-load-groups" is enabled, OpenUnison logs an error indicating "Could not find user," and the user is not redirected to the Kubernetes dashboard as expected. When this option is disabled, individual user permissions work, but using group-based permissions still does not.
Pre reqs is OK:
https://www.tremolo.io/post/kubernetes-and-azuread-sso
Kubernetes Version - 1.29
I installed using helm :
openunison-operator | CHART: openunison-operator-3-0.9 | APP VERSION: 1.0.39
orchestra | CHART: orchestra-2.10.61 | APP VERSION: 1.0.41
orchestra-login-portal | CHART: orchestra-login-portal-2.3.57 | APP VERSION: 1.0.41
orchestra-login-azuread | CHART: orchestra-login-azuread-1.0.3 | APP VERSION: 1.0.30
Image versions OpenUnison:
operator:
openunison-kubernetes-operator:1.0.6
orchestra :
openunison-k8s:1.0.41
activemq-docker:5.18.4
kube-oidc-proxy:1.0.7
python:3:1.0.0
orchestra-login-portal:
openunison-k8s-react:1.0.0

Error_and_values.zip
This is the error and values.yaml with parameters used

Logs_erro.txt

Hello @mlbiam, attached the logs.. thank you for your help.

Hello @mlbiam,
Are these logs ok to find the reason of error ?
Tks


🚀 Feature: Rename "TypeParts" functions to "Constituents"?

Bug Report Checklist

[X] I have tried restarting my IDE and the issue persists.
[X] I have pulled the latest main branch of the repository.
[X] I have searched for related issues and found none that matched my issue.

Overview
There are three functions exported or proposed for ts-api-utils around returning the .types property of a type:

intersectionTypeParts
unionTypeParts
#374

The terminology we use in typescript-eslint for "type parts" is constituent (https://github.com/typescript-eslint/typescript-eslint/issues/5841). My proposal is:

Make the same 2-3 functions available under names get(Intersection|Union)?Constituents, marking the old functions as @deprecated
Update them in any libraries we maintain - especially if we're forced to by eslint-plugin-deprecation
Remove the old names in the next major version

Additional Info
No response

Marking as accepting PRs. The old function names will have to remain as @deprecated aliases until the next major version.


[docs] Inline code blocks are not rendered properly in the sidebar

Describe the problem
Inline code blocks are rendered as Some heading in the docs in the sidebar nav.

Describe the proposed solution
Remove backticks from section headings.
Alternatives considered
Using {@html ...} to render section headings in the sidebar causes problems (code block contents are also rendered as HTML, even if escaped), so making it work would require other rewrites.
Importance
nice to have
Additional Information
I also noticed that backticks are not rendered as code blocks in the new search utility.

Fixed in #4011


Added test cases for new constructors in Person and ExistingUser

Codecov Report

Merging #224 (c882e6f) into master (f7daf3a) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #224   +/-   ##
=========================================
  Coverage     79.12%   79.12%
  Complexity      977      977
=========================================
  Files           132      132
  Lines          2989     2989
  Branches        365      365
=========================================
  Hits           2365     2365
  Misses          548      548
  Partials         76       76

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Authorization flow

Don't init CacheService until the addon is enabled in the current doc. Add an init method.

Closed via https://github.com/bennettscience/docutube/commit/63707d09670985a1362067abbed6901b20d9123e


GDPR: Disable grafana agent analytics by default and make them OPT-IN only.

Background
The grafana agent sends analytics to stats.grafana.org by default. This is neither documented in the installation site of the grafana cloud nor is this a good practise according to the gdpr.
Moreover, this behaviour cannot be disabled in the config file and requires a start parameter to be set.
This, on 90%+ of all Linux systems, requires a systemd unit overwrite, something that should be the last resort.
Overall, this is a bad practise that should be changed asap.
Proposal
Change the default of grafana agent to not send any kind of analytics.
Make the setting OPT-IN via config file.

Closing this since it looks like it is resolved.


[WBCAMS-220] gracefully handle BadHttpRequestException rev 2

Hold until changes associated with the removal of Jobs without salary is complete.   This changes shown in this PR request have been previously reviewed and approved but were reverted from master to minimize the changes associated with a production release.  This PR has been created so the changes can be merged into master when appropriate.

This PR was previously approved.  Waiting for client go-ahead before merging.

Is this still needed or can we close this PR?


Add 'prune' command to parquet-tools

During data retention, there is a need to remove unused or personal columns. Adding a 'prune' command in Parquet-tool to remove columns and retain all other columns unchanged. The way it works should be like 'merge' command in Parquet-tools, for example moving the column chunks as a whole.
Reporter: Xinli Shang / @shangxinli
Assignee: Xinli Shang / @shangxinli
PRs and other links:

GitHub Pull Request #755

Note: This issue was originally created as PARQUET-1791. Please see the migration documentation for further details.

Gidon Gershinsky / @ggershinsky:
A good idea. I'd suggest adding this to parquet-cli too.


Postgres permissions for a database user

I'm not an expert in Postgres configuration, so maybe there's something obvious that I'm missing, but I can't give a new database user permissions on tables in a database. The default recipe gives the user CONNECT privileges to a database, but tables can't be accessed.
Here's what I started with:
postgresql_connection_info = {
  :host     => 'localhost',
  :port     => 5432,
  :username => 'postgres',
  :password => 'password'
}
postgresql_database 'foo' do
  connection postgresql_connection_info
  action     :create
end
postgresql_database_user 'foo_user' do
  password      'foo_user_password'
  connection    postgresql_connection_info
  action        :create
end
postgresql_database_user "grant permissions to foo_user" do
  connection    postgresql_connection_info
  username      'foo_user'
  database_name 'foo'
  action        :grant
  privileges    [:all]
end

The last step, "grant permissions to foo_user", does not actually appear to do anything.
I've also tried
postgresql_database "grant permissions to foo_user" do
  connection    postgresql_connection_info
  database_name'foo'
  action        :query
  sql           "GRANT ALL PRIVILEGES ON ALL TABLES IN SCHEMA public TO foo_user"
end

which doesn't do anything. If I run the command directly in psql, it does work.

Same problem here!

I have a coworker trying to create a user that can insert rows into tables, but can't get the permissions granted through the recipe.

@gdavison In all cases I've seen, the username goes after "postgres_database_user", so are you sure that where you try to do grants with
postgresql_database_user "grant permissions to foo_user" do
it shouldn't be
postgresql_database_user "foo_user" do
instead?  (Yes, I realize you filled out the username attribute, and I don't know if that takes precedent.)

@rschwiebert I think the problem here is that I already have a resource postgresql_database_user 'foo_user' which creates the user. Each resource needs to have a distinct name, and the username attribute does take precedence.
Perhaps there should be different resources for creating a user and assigning permissions.

I believe our improved grant functionality in 6.1.0 fixes part of this, but also you need to have unique names to avoid merging the resources together. I'm going to close this out now. Feel free to open it back up with additional information if you have insight into the issue with 6.1.0+


Google authentication backend not working

2015-09-25 21:35:34,092 ERROR    Internal Server Error: /register/
Traceback (most recent call last):
  File "/usr/lib/python2.7/dist-packages/django/core/handlers/base.py", line 132, in get_response
response = wrapped_callback(request, *callback_args, **callback_kwargs)
  File "/home/zulip/deployments/2015-09-26-01-18-40/zerver/views/__init__.py", line 855, in accounts_home
context_instance=RequestContext(request))
  File "/usr/lib/python2.7/dist-packages/django/shortcuts.py", line 45, in render_to_response
using=using)
  File "/usr/lib/python2.7/dist-packages/django/template/loader.py", line 116, in render_to_string
template_name, context, context_instance, dirs, dictionary)
  File "/usr/lib/python2.7/dist-packages/django/template/engine.py", line 221, in render_to_string
return t.render(context_instance)
  File "/usr/lib/python2.7/dist-packages/django/template/base.py", line 208, in render
with context.bind_template(self):
  File "/usr/lib/python2.7/contextlib.py", line 17, in __enter__
return self.gen.next()
  File "/usr/lib/python2.7/dist-packages/django/template/context.py", line 241, in bind_template
updates.update(processor(self.request))
  File "/home/zulip/deployments/2015-09-26-01-18-40/zerver/context_processors.py", line 24, in add_settings
'password_auth_enabled':     password_auth_enabled(realm),
  File "/home/zulip/deployments/2015-09-26-01-18-40/zproject/backends.py", line 22, in password_auth_enabled
for backend in django.contrib.auth.get_backends():
  File "/usr/lib/python2.7/dist-packages/django/contrib/auth/__init__.py", line 38, in get_backends
return _get_backends(return_tuples=False)
  File "/usr/lib/python2.7/dist-packages/django/contrib/auth/__init__.py", line 27, in _get_backends
backend = load_backend(backend_path)
  File "/usr/lib/python2.7/dist-packages/django/contrib/auth/__init__.py", line 21, in load_backend
return import_string(path)()
  File "/usr/lib/python2.7/dist-packages/django/utils/module_loading.py", line 33, in import_string
six.reraise(ImportError, ImportError(msg), sys.exc_info()[2])
  File "/usr/lib/python2.7/dist-packages/django/utils/module_loading.py", line 29, in import_string
return getattr(module, class_name)
ImportError: Module "zproject.backends" does not define a "GoogleBackend" attribute/class

if I set the backend by uncommenting the GoogleBackend in settings.py

Does GoogleMobileOauth2Backend in the same file work for you?
I'd expect it to get further and then complain until you give it google oauth tokens in the settings.py files; you can see the public keys in zproject/local_settings.py, search for GOOGLE (and you'll need to add the private keys in /etc/zulip/zulip-secrets.conf).
Let me know if this works and I can fix the docs...

Yup, that backend works. Am at AttributeError: 'Settings' object has no
attribute 'GOOGLE_OAUTH2_CLIENT_ID'
now.
However, if I go to 'register' it still asks me for email too
On Fri, Sep 25, 2015 at 6:45 PM, Tim Abbott<EMAIL_ADDRESS>wrote:

Does GoogleMobileOauth2Backend in the same file work for you?
I'd expect it to get further and then complain until you give it google
oauth tokens in the settings.py files; you can see the public keys in
zproject/local_settings.py, search for GOOGLE (and you'll need to add the
private keys in /etc/zulip/zulip-secrets.conf).
Let me know if this works and I can fix the docs...
—
Reply to this email directly or view it on GitHub
https://github.com/zulip/zulip/issues/10#issuecomment-143390193.

--
Yuvi Panda T
http://yuvi.in/blog

What should be the redirect URL for getting credentials?

Whether there's a password prompt on the login page should be determined by zproject/backend.py's password_auth_enabled() so the issue is likely there; if the realm you're using is called "zulip.com" then I bet the settings.PRODUCTION there should be a settings.ZULIP_COM instead.
In the same file there's some links to docs on Google's oauth stuff in the doc strings for GoogleMobileOauth2Backend:
https://developers.google.com/+/mobile/android/sign-in#server-side_access_for_your_app
https://developers.google.com/accounts/docs/CrossClientAuth#offlineAccess

Apparently https://zulip.wmflabs.org/accounts/login/google/done/ is the redirect URL.

Now I'm at:
2015-09-25 21:59:29,446 ERROR    Internal Server Error: /accounts/login/google/done/
Traceback (most recent call last):
  File "/usr/lib/python2.7/dist-packages/django/core/handlers/base.py", line 132, in get_response
response = wrapped_callback(request, *callback_args, **callback_kwargs)
  File "/home/zulip/deployments/2015-09-26-01-18-40/zerver/views/__init__.py", line 695, in finish_google_oauth2
raise Exception('Could not convert google pauth2 code to access_token\r%r' % resp.text)
Exception: Could not convert google pauth2 code to access_token^Mu'{\n "error": "invalid_request",\n "error_description": "client_secret is missing."\n}\n'

In zproject/settings.py it has:
GOOGLE_OAUTH2_CLIENT_SECRET = get_secret('google_oauth2_client_secret')
So you need to put your oauth secret in /etc/zulip/secrets.conf with that
as the secret name.
    -Tim Abbott

On Fri, Sep 25, 2015 at 7:01 PM, Yuvi Panda<EMAIL_ADDRESS>wrote:

Now I'm at:
2015-09-25 21:59:29,446 ERROR    Internal Server Error: /accounts/login/google/done/
Traceback (most recent call last):
File "/usr/lib/python2.7/dist-packages/django/core/handlers/base.py", line 132, in get_response
response = wrapped_callback(request, *callback_args, **callback_kwargs)
File "/home/zulip/deployments/2015-09-26-01-18-40/zerver/views/init.py", line 695, in finish_google_oauth2
raise Exception('Could not convert google pauth2 code to access_token\r%r' % resp.text)
Exception: Could not convert google pauth2 code to access_token^Mu'{\n "error": "invalid_request",\n "error_description": "client_secret is missing."\n}\n'
—
Reply to this email directly or view it on GitHub
https://github.com/zulip/zulip/issues/10#issuecomment-143391019.

Now at:
Traceback (most recent call last):
File "/usr/lib/python2.7/dist-packages/django/core/handlers/base.py", line 132, in get_response
response = wrapped_callback(request, *callback_args, **callback_kwargs)
File "/home/zulip/deployments/2015-09-26-01-18-40/zerver/views/init.py", line 696, in finish_google_oauth2
access_token = resp.json['access_token']
TypeError: 'instancemethod' object has no attribute 'getitem'

Found at least a couple places where resp.json is used instead of resp.json(). Live hacking those fixed the issue.

OK awesome!  it'd be great if you could post the changes you needed, even
if in not-merge-ready form, so we can fix this for the next group of people
who try to set this up :)
    -Tim Abbott

On Fri, Sep 25, 2015 at 7:30 PM, Yuvi Panda<EMAIL_ADDRESS>wrote:

Found at least a couple places where resp.json is used instead of
resp.json(). Live hacking those fixed the issue.
—
Reply to this email directly or view it on GitHub
https://github.com/zulip/zulip/issues/10#issuecomment-143392266.

Sure! #14 is the PR :)

OK I think I fixed this and also I attempted to document how to set this up in https://github.com/zulip/zulip/commit/5dd330e7691d381f263f0723ffe9a58050d12ed7
I'd appreciate it if you could check that over for any mistakes, and maybe expand the steps for setting it up with Google if you think it's appropriate.

Closing this issue since I think this is now resolved in the current documentation.


React 16 support

<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>|| 15 but none was installed
An update to package.json deps should do it.

Updated in 0.8.8


Enhance Trickle algorithm documentation

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@f897215). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master   #13581   +/-   ##
=========================================
  Coverage          ?   52.45%
=========================================
  Files             ?      322
  Lines             ?    46592
  Branches          ?    10768
=========================================
  Hits              ?    24441
  Misses            ?    17246
  Partials          ?     4905

Impacted Files
Coverage Δ

include/net/trickle.h
66.66% <ø> (ø)

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 f897215...07dec63. Read the comment docs.


Incompatible with Node ESM

This library is currently incompatible with Node ESM (which includes any Typescript users on node16 or nodenext module mode) due to the default export not also setting module.exports in the generated CJS output.
More information about this issue is available here: https://github.com/arethetypeswrong/arethetypeswrong.github.io/blob/main/docs/problems/CJSOnlyExportsDefault.md.
The solution is to make the following changes to the untranspiled source:
-export default class VoximplantApiClient {
+class VoximplantApiClient {
...
}
+
+export = VoximplantApiClient;

A detailed explanation of the suggested solution is also available in the official Typescript docs: https://www.typescriptlang.org/docs/handbook/modules/appendices/esm-cjs-interop.html#library-code-needs-special-considerations

Users will detect a misconfigured module smell if they have to access the .default property of a default import, and if they’re trying to write code that will run both in Node.js and a bundler, they might be stuck. Some third-party TypeScript transpilers expose options that change the way default exports are emitted to mitigate this difference, but they don’t produce their own declaration (.d.ts) files, so that creates a mismatch between the runtime behavior and the type checking, further confusing and frustrating users. Instead of using default exports, libraries that need to ship as CommonJS should use export = for modules that have a single main export, or named exports for modules that have multiple exports:

Hi there! Thanks a lot for the issue (again 😄) and for participating in the project!
As I said, we have quite a few other ideas for improving this package that we would like to include in the next major version. This issue will be solved there too.


[BUG]memory leak

Please prefix your issue with one of the following: [BUG] [PROPOSAL] [QUESTION].
Package version, Laravel version
Laravel 5.3 and package 2.1
Expected behaviour
there should be  no error after adding the service provider
Actual behaviour
after adding the service provider into the config/app.php file, throwing-
Allowed memory size of 134217728 bytes exhausted (tried to allocate 262144 bytes)
Exception stack trace
none
Screenshot of Excel file
none
Steps to reproduce the behaviour
none. asking for a solution

It was not a bug! sorry. i made a silly mistake.


Integrate functional maps feature

Dear @kirbiyik ,
I know we have waited for a long time for maps. Sometimes we cried and sometimes we laughed together. But the time we destroy the castle of maps feature comes.
I am waiting for your review.
Good night.

Your pain shall be ceased...


bug: ui.dialog closable option is not working properly

Wave SDK Version, OS
0.19.0
Actual behavior
X button is visible and active when
closable=False
blocking=True

Expected behavior
X button should not be visible and the user shouldn't be able to close the dialog when closable option is set to False

Thanks for your report, @dulajra! It should be fixed in v0.20.0, therefore I am closing this issue. If the problem persists there, please reopen this issue.


New Story: Allow adjustment of 'Creativity'

Problem
New Story: Allow adjustment of 'Creativity'
See #10
Solution
add db migration to add creativity attribute to story model
add active record validations for creativity in story model
update form in views/stories/new.html.erb to request creativity from user
update strong parameters in stories controller to permit creativity attribute
Details
NA
Hope everything is in order, let me know if any changes are required (:

While I was at it, I also set a default value for the mythology and the creativity in the form (I think it looks better than the blank slot, considering that we already have Unspecified for the mythology and that a value for both inputs is mandatory).

Looks great!  I'll merge it and add an separate issue to change the 'temperature' sent to the LLM when it generates the story.
Also, on my screen there's a  tag on the Title and Creativity labels that shows up on the screen.  What's that? Do you get that too?

I think adding one of these cleans up the label:
https://github.com/abevoelker/simple_form_tailwind_css
OR
https://github.com/tarellel/simple_form-tailwind
Do you want to try that?


Two issues on multi selection mode

When I clicked the image (collectionview didSelect) in gallery page, which was opened after selection of images, it crashed. Added breakpoint and saw that the crash occurred at line 72 in YPPhotoFiltersVC.swift file. I didn't want to filter, which I set in configuration, but it still opened.

The shared configuration that I used;
func setup() {
        var config = YPImagePickerConfiguration()
        func setupLibrary() {
            config.library.mediaType = .photo
            config.library.onlySquare = false
            config.library.maxNumberOfItems = 5
            config.library.skipSelectionsGallery = false
        }
        func setupCamera() {
            config.targetImageSize = .original
            config.usesFrontCamera = true
            config.showsFilters = false
            config.filters = []
            config.shouldSaveNewPicturesToAlbum = true
            config.showsCrop = .none
            var screens:[YPPickerScreen] = [.library]
            if UIImagePickerController.isSourceTypeAvailable(.camera) {
                screens.append(.photo)
            }
            config.screens = screens
        }
        setupLibrary()
        setupCamera()
        config.startOnScreen = .library
        config.hidesStatusBar = false
        YPImagePickerConfiguration.shared = config
}

I tried to bypass the crash by changing the setting with config.library.skipSelectionsGallery = true. But this time, when I selected multi images and pressed next button, then the didFinishPicking function of the image picker was called twice.

@barisozr Thanks for the detailed report! This surely looks like a bug indeed.
the didFinishPicking calls twice just got fixed so It'll be in the next version (ppbly later today).
I am going to look into it :)

@barisozr https://github.com/Yummypets/YPImagePicker/commit/828aa47c0aca9fb9ffe6ad7f2069cbee0f1551a0 will fix 1 :)
2 should be fixed already in 3.2.0 -->  https://github.com/Yummypets/YPImagePicker/releases/tag/3.2.0

@s4cha seems ok, I'll test and let you know :)

@barisozr The second, "config.filters = []" crash is committed already so it'll be fixed in the next version :) (probably later today)


Function get_program_accounts broken?

The following example from docs seems to be broken:
>>> from solana.rpc.types import MemcmpOpts
>>> solana_client = Client("http://localhost:8899")
>>> memcmp_opts = [
...     MemcmpOpts(offset=4, bytes="3Mc6vR"),
... ]
>>> pubkey = PublicKey("4Nd1mBQtrMJVYVfKf2PJy9NZUZdTAsp7D4xWLs4gDB4T")
>>> filters = [17, memcmp_opts]
>>> solana_client.get_program_accounts(pubkey, filters=filters).value[0].account.lamports

Throws:
    solana_client.get_program_accounts(pubkey, filters=filters)
  File "/usr/local/lib/python3.9/site-packages/solana/rpc/api.py", line 632, in get_program_accounts
    body = self._get_program_accounts_body(
  File "/usr/local/lib/python3.9/site-packages/solana/rpc/core.py", line 308, in _get_program_accounts_body
    None if filters is None else [x if isinstance(x, int) else Memcmp(*x) for x in filters]
  File "/usr/local/lib/python3.9/site-packages/solana/rpc/core.py", line 308, in <listcomp>
    None if filters is None else [x if isinstance(x, int) else Memcmp(*x) for x in filters]
TypeError: Memcmp.__new__() missing 1 required positional argument: 'bytes_'

Tested on:

Python v3.9.9
solana v0.27.2

Seems like it comes from an inconsistency between this library and solders, but I don't know if the right solution should be either renaming bytes_to bytes in the  Solders filter or update your type.py file the other way around to match solders.

@kevinheavey  sorry to call you in Dubliner :) but I guess you would know what needs to be done here.

@AnderUstarroz I think your second issue is just because you have a solders Pubkey there, not a solana.public_key.PublicKey

But shouldn't solana_client.get_program_accounts(PROGRAM_ID) return instances of pubkeys with their corresponding  .to_solders()method without having to cast it using PublicKey(account.pubkey)?
Thanks @kevinheavey

I think we'll just get rid of solana.publickey.PublicKey soon as it doesn't provide any benefit over solders.pubkey.Pubkey


[ZEPPELIN-3814] Add apply button to table settings

What is this PR for?
Now changes in table settings applies only after page refreshing which isn't convenient

Settings menu before PR:

Menu with new button:

What type of PR is it?
Improvement
What is the Jira issue?

issue - https://issues.apache.org/jira/browse/ZEPPELIN-3814

How should this be tested?

CI pass
Manually tested

Questions:

Does the licenses files need update? No
Is there breaking changes for older versions? No
Does this needs documentation? No

I decided to put floppy icon as in advanced-transformation-setting.
@jongyoul please help reviewing this.


[feature] $timestamp token variable

Is your feature request related to a problem? Please describe.
I have the need to define a task in .moon/tasks.yml which dynamically include the current system timestamp in a command argument. e.g.
  deploy:
    command: 'cdk'
    args:
      - 'deploy'
      - '--all'
      - '--require-approval=never'
      - '--app "deploy --target=$projectRoot"'
      - '--output $workspaceRoot/.cdk/$project/$(date +%s)'
    deps:
      - ~:build

The $(date +%s) portion is problematic as we're not in a sh or bash shell and there doesn't appear to be a way to get that kind of data.
Describe the solution you'd like
The quick and dirty fix for our needs would be to create a new token variable $timestamp that was set to the system timestamp/epoch. It doesn't have to be high resolution, but I certainly wouldn't complain if it was.
Describe alternatives you've considered
I'm not sure at the moment how to get around this. A nice alternative might be something like a @shell function that would allow executing shell commands and returning the stdout output. Something like @shell($(date +%s)) would get the job done.
Additional context

I dig it 👍
Can add for the next release.

I'm not sure at the moment how to get around this. A nice alternative might be something like a @shell function that would allow executing shell commands and returning the stdout output. Something like @shell($(date +%s)) would get the job done.

A workaround would be to put this command in a shell script and have the moon task execute the script instead of the CDK deploy command (if you need this before the next release).

Will land in the next version: https://github.com/moonrepo/moon/pull/713

Landed in v1


Fixed page responsiveness and scroll issue in WebView (#10)

I have fixed the scrolling and page responsiveness issue by obtaining the content height of our WebView page and adjusting the modifier size accordingly.

I think you missed the change you made to the initialUrl value.

I had just add the test URL in initialUrl for testing to check every thing is working fine or not. We can change this URL to the previous one.

I had just add the test URL in initialUrl for testing to check every thing is working fine or not. We can change this URL to the previous one.

I think that would be best


SetImage and SetCommand should require a instance.Stop()

This is not needed anymore. Ref: https://github.com/celestiaorg/knuu/pull/515#issuecomment-2270501979


Next3

Hi, again. This is my continued v2 patch series. I would like to merge following changes into upstream:

Process reports in one shot as last stage of do_analyse_sources and *do_analyse_sources_all tasks, as well as default task (e.g. bitbake core-image-minimal) (commit 18aba3a).
Enable HTML reports in ISA_cve_plugin.py with additional cve-check-tool call. Now we have faux data file prepared by the do_analysesource internal task and we can call cve-check-tool multiple times to get required report types. Now we support CSV and HTML reports (commit 17fdb8b).
Exclude more package classes from the do_analysesource. We already exclude native and cross, add nativesdk, crosssdk, cross-canadian, packagegroup and image classes (commit dfa2597).

I've made tests against YOCTO jethro branch. Following tests are passed.
bitbake -c analyse_sources openssl
bitbake -c analyse_sources_all core-image-minimal
bitbake core-image-minimal (on already build image, nostamp isn't used in new patches).
Please test.

Unfortunately there is still issues with
bitbake core-image-minimal
when building image from scratch (it just won'ts build).
I'll continue to investigate it.

Yes, I made tests now and ended up with the same result. do_rootfs is not called and image is not build even if you wipe the whole tmp dir

Could you retry tests with following patch (commit 2018667)?
I see no problem with image/target build for now on both YOCTO fido and jethro branches. Also I retried all tests from original pull request description.
Hint: to use meta-security-isafw on fido branch at least two following commits are need to be cherry picked from master branch in openembedded-core:

Commit e11de17 json-glib: enable native builds
Commit 1405253 json-glib: inherit gnomebase instead of gnome

Seems working for me fine now. Hope I didn't miss anything. I am merging this now. Thank you!

Actually there is one issue that showed up now and I am debugging this. If you build an image like:
bitbake my_image:do_build
then process_reports task is not called at all. Probably smth related to default tasks...

Interesting...
Probably adding something like
do_build[postfuncs]` = "do_process_reports"

in the similar way as done for do_analyse_source and do_analyse_sources_all would solve this problem for
bitbake my_image:do_build

do_build task execution. But for other builds of my_image:do_, where  is something other than do_build this of course will not work.
I will look at this problem closely. Thanks for report.

I brought the issue to Yocto architect yesterday, even he was actually surprised with this behavior, so I am still waiting on his investigation on this.

Actually do_build[postfuncs] = "do_process_reports" doesn't help and some other ways that I tried. It seems smth is really not right when you do bitbake my_image:do_build...

Ok, now I have been explained what is going on. So, when you specify smth like:
bitbake my_image:do_xyz
it would execute do_xyz task and all its dependencies, but it won't execute anything after that task. So, if we have our process_report that is specified to run after do_build, it won't be run since we asked bitbake to execute do_build and no further. When you run it normally bitbake my_image, it would execute all tasks.
So, the suggestion I got for running our task is really to use event handler (bb.event.BuildCompleted). I am going to try it now and see if it works for all the cases. I will post a pull request for testing if it looks ok on my side.


FunctionCommand has no id

In function.proto the message FunctionReply defines response to be one of Failure and other messages. Failure is defined in entity.proto and carries a command_id field which "Must match the input command". Unfortunately FunctionCommand – which is used for all StatelessFunction methods – carries no command id.
Why do we use FunctionCommand instead of the "normal" Command (from entity.proto) which is used elsewhere (e.g. for EventSourced)? That "normal" Command carries an id.
Alternatively we could add an id to FunctionCommand. But we should also clarify and document why StatelessFunction uses a specialized command.

Browsing through the protocol I get the impression that function.proto is very different from event_sourced.proto and crdt.proto. Why is it not using ClientAction and SideEffect from entity.proto? Is it a little outdated and needs some love?

I think the reason that function.proto is different is because it doesn't deal with Entities at all.

Well, it's not an entity in the sense that it is served with data by the platform nor is able to write data to the platform. But nevertheless it is treated as an Entity in the Cloudstate protocol and by the proxy.
Apart from this rather philosophical question, the protocol is not correct wrt StatelessFunctions as I have described above: it is impossible to return a FunctionReply with a response of type Failure. We need to fix that either way.

@hseeberger Having a command_id does indeed make sense in the general case, yes, because of correlation.

Instead of special casing StatelessFunction we should consider using Command instead of FunctionCommand and inside of FunctionReply.response we should use ClientAction.

On a similar topic in the Spec Draft PR about concurrent command handling, James mentioned https://github.com/cloudstateio/cloudstate/pull/119#discussion_r404449523 that

the proxy only issues one command at a time

This was lead to the question how the proxy can correlate a Failure without a command_id set. Since "The protocol supports commands being handled concurrently" it would be beneficial to have this correlation explicitly.

I agree with @marcellanz that if simultaneous manipulations are possible then the correlations must be explicit. In fact, mandatorily they should be explicit. @hseeberger maybe that's why it is not possible to return a failure currently

@pvlugter do you have an opinion here?

Hi all What's wrong with having an id for correlation?

I've had a look through, including the history for this.
So #48 added support for multiple entities — entity.Reply was simplified down to just the payload, EventSourcedReply and CrdtReply track the command id, event-sourced and CRDT share the common entity.Failure while the FunctionReply doesn't use failures.
In #154, FunctionReply responses could be failures, and I expect that entity.Failure should've had the command id removed, with EventSourcedFailure and CrdtFailure messages added — so that everything mirrored the reply messages. There's no command id tracking or checking for functions.
So it looks like there shouldn't be a command id on entity.Failure, and there should instead be failure messages specific to event-sourced and CRDTs.
I think supporting correlation ids for functions would be separate from cleaning up the command ids.
And agree the way that things are structured and messages shared could be more understandable, especially with functions being and not being entities.


Great package and want to explore. Is that possible to work in VS code instead of console?

I use Anaconda and I like to work in VS code. Is that possible?
Thank you and love to explore this package.

I've been running it in VS-code, then you control it inside the terminal folder in VS-code

As @HenryAllen04 suggested, you can start the program in the terminal of VS code. The user experience should be same as other terminals. Feel free to reopen the issue if you could not figure it out.


Race condition on VirtualSpout close cleaning up consumer state

Summary
On VirtualSpout.close() when a sideline has been completed, it attempts to cleanup consumer state.  If you have multiple Spout instances (and therefor multiple VirtualSpout instances for the sideline) you can run into a race condition cleaning up consumer state.  When it attempts to delete the parent state node in zookeeper, it blindly attempts to wipe the parent node, but does not handle if the parent node has already been wiped by another instance.
Stack Trace
2017-12-13 16:44:41.875 topo-id] c.s.s.s.d.c.SpoutRunner [ERROR] SpoutRunner for topo-id:sideline:14064E9096F16FF755BD21C640976428 threw an exception org.apache.zookeeper.KeeperException$NoNodeException: KeeperErrorCode = NoNode for /sideline-spout/topo-id/consumers/topo-id:sideline:14064E9096F16FF755BD21C640976428
java.lang.RuntimeException: org.apache.zookeeper.KeeperException$NoNodeException: KeeperErrorCode = NoNode for /sideline-spout/topo-id/consumers/topo-id:sideline:14064E9096F16FF755BD21C640976428
	at com.salesforce.storm.spout.dynamic.persistence.zookeeper.CuratorHelper.deleteNodeIfNoChildren(CuratorHelper.java:184) ~[stormjar.jar:?]
	at com.salesforce.storm.spout.dynamic.persistence.ZookeeperPersistenceAdapter.clearConsumerState(ZookeeperPersistenceAdapter.java:168) ~[stormjar.jar:?]
	at com.salesforce.storm.spout.dynamic.kafka.Consumer.removeConsumerState(Consumer.java:620) ~[stormjar.jar:?]
	at com.salesforce.storm.spout.dynamic.VirtualSpout.close(VirtualSpout.java:260) ~[stormjar.jar:?]
	at com.salesforce.storm.spout.dynamic.coordinator.SpoutRunner.run(SpoutRunner.java:181) [stormjar.jar:?]
	at java.util.concurrent.CompletableFuture$AsyncRun.run(CompletableFuture.java:1626) [?:1.8.0_102]
	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1142) [?:1.8.0_102]
	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:617) [?:1.8.0_102]
	at java.lang.Thread.run(Thread.java:745) [?:1.8.0_102]

This will need to be fixed in 0.9 and 0.10.

Fixed in master (0.10.x) and backported to 0.9.x as version 0.9.2


[rv_timer] Separate reset for watchdog

This is as much a question as it is a request.
I'm currently claiming that a normal timer and the watchdog timer sit on different reset domains. The idea is mostly we don't want someone to be able to reset the processing core and the watchdog together, which effectively silences the watchdog.  See diagram
https://docs.google.com/drawings/d/1a3nkFVJhCFQUR9uQMm7Hfr2cSLGmaJ5HOuLX4ZyNkCY/edit
If we agree that this the behavior that we want, then we'd either need rv_timer to support different resets (i think it was intended for watchdog?) or it might be easier if we just instantiate a separate rv timer for watchdog.
This is not something that needs to be urgently discussed yet, but I just wanted to document it and see how people feel.
The link below also contains the thinking that went into the timer if it were 64b.
https://docs.google.com/drawings/d/1xyxdF5ZFSy5hz4reumzcU3PFcHfhgBmAoGqf5lgYnL0/edit?usp=sharing

sory for the late reply on this.. but yes i was originally thinking two timers. This area however, we might want to discuss a bit more with the software teams. There are specific behaviors they're looking for around low power, and it's not quite the same as standard rv timer.

@mateenfaisal @tomroberts-lowrisc
this is a quick write-up we did on how we envision the timer to work.
Most of the bits are worked out, with the exception of how we should handle multiple NMIs in the system.  In order to ensure that the ROM isn't immediately interrupted upon wakeup, I am recommending another rv_plic-like module that handles the initial enabling of NMIs (this of course needs to be one-way, as in, once enabled we cannot revert it).
Please have a look.
https://docs.google.com/document/d/1EBA7Yg-k1eW61SQRSfiITTaiIwao6drwQOIPsf-mGdc/edit?usp=sharing

Since we've implemented and instantiated a watchdog timer by now, we should be able to close this issue soon.
The only thing that remains to be done is to convert the regular IRQ coming from the watchdog to an NMI.
See also comment here: https://github.com/lowRISC/opentitan/pull/7257#issuecomment-879389067

i'm still not sure our latest discussion actually wanted the watchdog to be an NMI.
I have some very vague recollections (i could be wrong) that it's okay to treat it as an interrupt.  If the interrupt is masked off (for whatever reason), the real watchdog will kick in anyhow.

i'll take an action to check with software again and actually document this somewhere.

i'm still not sure our latest discussion actually wanted the watchdog to be an NMI.
I have some very vague recollections (i could be wrong) that it's okay to treat it as an interrupt. If the interrupt is masked off (for whatever reason), the real watchdog will kick in anyhow.

That's also what I remember from these discussions!

Hey @tjaychen, has this been documented somewhere so that we can close it?

yep, this can be closed.


Add Difference (Increase in cases from the last hour / day )

Describe the solution you'd like
Add the increase in cases / deaths , etc below the statistics for example: Cases: 100,000 (+1,000)
Additional context
To be added in country and all

I think the line number in to be added here needs to be changed as it refers to an embed

I think the line number in to be added here needs to be changed as it refers to an embed

Done


Estimated ThorSaver Miner Fee's are wrong

Overview
Currently, the estimated miner fee for ThorSaver Deposits are very wrong. Thus also giving a bogus time to break even.

References and additional details
Recreation Steps:

Go to the DeFi page
Go through the deposit steps for a ThorSavers vault until you get to Sign & Broadcast
Notice the wrong estimated fee and break even time.

Acceptance Criteria
Estimated Fee and Break Even Time for ThorSaver Vaults should be correct
Need By Date
No response
Screenshots/Mockups
No response
Estimated effort
No response

Still broken @0xdef1cafe

@tshifty as previously discussed with @shapeshift/operations and @reallybeard , this isn't wrong fees per se but wrong terminology.
What we call "Estimated Fee" here isn't a "Miner Fee" as referred to in this ticket, but a network fee, i.e the total downside for depositing into said savers vault. This has no correlation to miner fees at all. Getting in touch with @reallybeard so we land on a more descriptive UX for this 🙏🏽
Time to break even is definitely fixed for some time already - just gave it a sanity check https://github.com/shapeshift/web/pull/4194


plantweb --format auto file.txt fails to format AsciiMath / Latex

I copied this example from Math in PlantUML
@startuml
Bob -> Alice : Can you solve: <math>ax^2+bx+c=0</math>
Alice --> Bob: <math>x = (-b+-sqrt(b^2-4ac))/(2a)</math>
@enduml

to p.uml
In a JupyterLab notebook, I have this code cell
%%bash
plantweb --format auto p.uml

The generated p.svg does not shows beautiful math, it still shows the AsciiMath code:

But when I changed the code cell to
%%bash
plantweb --format png p.uml

the generated p.png looks beautifully formatted.

The --format auto and --format svg do not correctly format the AsciiMath and jLatexMath code.

I'm not sure whether the bug is in iplantuml or plantweb.  I did an experiment based on Python API,  I use this code cell
from plantweb.render import render

CONTENT = """
Bob -> Alice : Can you solve: <math>ax^2+bx+c=0</math>
Alice --> Bob: <math>x = (-b+-sqrt(b^2-4ac))/(2a)</math>
"""

output = render(
    CONTENT,
    engine='plantuml',
    format='svg',
    cacheopts={
        'use_cache': False
    }
)

with open("alice+bob.svg", "wb") as f:
    f.write(output[0])

The AsciiMath code was not correctly formatted.  But when I changed the format to png, it turned out correct.
This shows that the bug is in plantweb.


"Open in Finder"

Would be nice to have an "Open in Finder" option when reading a paper that just open the folder where the data is stored

For cross-platform perhaps "Open data folder"?

And we're already using https://www.npmjs.com/package/open for URLs, so this should be trivially easy to implement.

Since you're using Electron, I think it would be better to use shell.showItemInFolder().

thanks @sindresorhus, any particular reason?

@blahah It's better as it uses native APIs instead of being a wrapper over some command-line tools. I'm by the way the maintainer of opn, a maintained and better alternative to the open module you mention.

Thanks @sindresorhus - looks like we should switch from open to opn, and we'll use shell.showItemInFolder() as suggested.


Added utils-generic template

I want a simple template for shells, experiments and all sorts of mixed stuff, which is hard to put into specific templates.

I agree, this should be added to the repository.

Come ooon, is the nixos/templates dead again? @domenkozar ?

@cab404 could you rebase this with master?

@domenkozar can you please merge this?

Fixed conflict and merging as it is a common pattern and easy way to get started.