eternal loading

ubuntu 20.04 eternal download of the ip address in the settings changed to your own.

Are you accessing it from some other computer rather than where it is hosted?

Yes. But it's the same on the same computer.

Yes. But it's the same on the same computer.

What are the logs (when connecting from the same computer) in the backend and browser's console?

Why are you not accessing it on localhost on the same computer? We currently do not provide support for its deployment or exposing it to network rather its installation locally. If you try to access it locally it should work fine


How to handle unicode strings & lowercase letters not able to detect in ascii field

I force the Emoji to turn into image but now what If user paste a 😀 which is a unicode?

Also I tried to put ':p' in the ascii but its not able to detect it only capitalize "P"

@johanbrook

This should perhaps be two issues?

Anyway, I think we should go with capitalized ASCII emojis. Otherwise, :d would be a :D, which looks funny.
It's been out of scope for this package to parse Unicode emojis. Happy to look at a PR for that though.

Thanks for the reply man! @johanbrook Yea probably right about sticking to capitalized I'm just used to entering :p. But the issue tho some ascii's not working. Like you defined this ascii :') but its not able to find the emoji and also trying other letters doesn't work also like capital L for example.
Ok not a big deal in the second one. Hope somebody PR haha.

@johanbrook btw loving the package. So easy to use and customize

Thank you!
Are you sure :') isn't parsed? The tests for it go through: https://github.com/lookback/meteor-emoji/blob/master/spec/emojis-spec.coffee#L63-L71

Yeah. :* not working also. Btw I forced all to use images.

OK I figured why :') is not being parsed cause I escaped it with an HTML escape script. But :* still doesn't work after removing all my parsers. Just the plain message.


Build Weaviate's Transformer-2-Vectorization Containers Locally

What's being changed:
Scripts and code for building Weaviate's T2V containers locally.

@jtbenson116 thanks for making that change - looks good.  Remember to approve the PR overall if you think it's ok for us.  Note that we can improve the troubleshooting documentation later ( unless you feel we can fix it with some minor mods )...


[App]: Repots crash when changing the run set in a panel grid

Current Behavior
I have a panel grid with some runs:

and then I click the tickmark, and get a crash report:

Just closing it makes the reports environment only partially work. E.g., I can click on the tick mark again and see the runs appear in the parallel coordinates panel, but I can't in the images panel:

Expected Behavior
The runs disappearing and reappearing without a crash.
Steps To Reproduce

Go to the reports section
Create a panel grid
click on the tickmark on the run set title.

Screenshots
Added above ^
Also in the web console it shows this when the error happens, if it helps:

Environment
OS: Both on Linux and Windows
Browsers: both Chrome and Edge
Additional Context
This is a bug I've dealt with for at least a year and a half :(

https://github.com/wandb/wandb/assets/122482606/b39a456b-2af5-41bb-868e-d6b7cc2d4b2f
I was able to create a report and tick the box with no issue on my end. Could you send one of your reports that is crashing? I am unsure why this has been consistently crashing on your end.

https://api.wandb.ai/links/hilamanor/edbwt4oh
Maybe it's related to having an audio panel?

Ah, I can see the behavior now. It does look like it is the Audio Panel that is crashing the page. I will write a bug report on this and investigate further as to why this is happening.


Implement French translation

How do you make a program more accessable? Exactly, by translating it into as many languages as you can. As I'm currently learning French, I thought this can be done in the foreseeable future. In the next update, I will make it implementable, when the language is going to be implemented is still unclear.

It's now implementable, but not implemented.


checker.ts: explicit boolean type annotation overrides type guard, silencing an error

Bug Report
I wouldn't call this a bug, but rather a piece of code inconsistent with "best practices" and prone to cause more confusion in future.
This is a spin-off from issue https://github.com/microsoft/TypeScript/issues/45572. I have investigated the line mentioned in @andrewbranch 's comment, and answered my own question in the subsequent reply.
If you indeed recognize it as a bug, please read the "Solution" section and assign the bug to me. Thanks for working on and maintaining such a wonderful project!
🔎 Search Terms
Control flow analysis of aliased conditional expressions
Type guard result saved to variable
🕗 Version & Regression Information
This was introduced in v4.4.0-beta, by pull request https://github.com/microsoft/TypeScript/pull/44730 "Control flow analysis of aliased conditional expressions", commit https://github.com/microsoft/TypeScript/pull/44730/commits/08b6e2e0fe8d1cb1b5b3d563244a22e82e8e91df.
"inlining CFA" through an explicit type annotation:
@@ -38340,9 +38340,9 @@
function checkModuleDeclaration(node: ModuleDeclaration) {
    ...
-   const isAmbientExternalModule = isAmbientModule(node);
+   const isAmbientExternalModule: boolean = isAmbientModule(node);

⏯ Playground Links & 💻 Code
// utilities.ts

export function isAmbientModule(node: Node): node is AmbientModuleDeclaration {
    return isModuleDeclaration(node) && (node.name.kind === SyntaxKind.StringLiteral || isGlobalScopeAugmentation(node));
}

export function isExternalModuleAugmentation(node: Node): node is AmbientModuleDeclaration {
    return isAmbientModule(node) && isModuleAugmentationExternal(node);
}          ^^^^^^^^^^^^^^^^^^^^^

Playground link illustrating the "Before" code example
Before https://github.com/microsoft/TypeScript/pull/44730 (v4.3.5):
function checkModuleDeclaration(node: ModuleDeclaration) {
    // ...

    // Type guard result gets assigned to a variable.
    const isAmbientExternalModule = isAmbientModule(node);

    // ...

    // Control flow analysis would've narrowed `node` to AmbientModuleDeclaration
    if (isAmbientExternalModule) {
        // ... so, here, `node` doesn't get narrowed to AmbientModuleDeclaration — it stays just a ModuleDeclaration,
        // as annotated in the function signature

        if (isExternalModuleAugmentation(node)) {

            // Now, because the type guard is inside the conditional statement, `node` will get narrowed to AmbientModuleDeclaration.

        }
        // However, outside the if statement, `node` is still ModuleDeclaration. No errors.
        // If CFA recognized that isAmbientExternalModule narrows `node` to AmbientModuleDeclaration,
        // the redundancy in isExternalModuleAugmentation() type guard (which contains isAmbientModule() in itself)
        // would cause `node` to be narrowed down to `never`.
        else if ( /* `node` is `ModuleDeclaration` here... */ ) {

        }
        else {
            // ...and here.
        }
    }

    // ...
}

Playground link illustrating the "After, current" code example
After (with explicit type annotation, e.g. current version, WITHOUT error) https://github.com/microsoft/TypeScript/pull/44730 (v4.4-beta):
function checkModuleDeclaration(node: ModuleDeclaration) {
    // ...

    // Type guard result gets assigned to a variable. CFA would've narrow down `node`
    // in conditional statements below, IF NOT for the `boolean` explicit annotation. Because of it, CFA no longer treats isAmbientExternalModule as a type guard result. No type narrowing will happen below, so further this example
    // is identical to the "Before" example above.
    const isAmbientExternalModule: boolean = isAmbientModule(node);

    // ...
}

Playground link illustrating the "After, current" code example
After https://github.com/microsoft/TypeScript/pull/44730 (edited out the type annotation, GIVES ERROR) (v4.4-beta):
function checkModuleDeclaration(node: ModuleDeclaration) {
    // ...

    // Type guard result gets assigned to a variable. Now, there's no explicit type annotation, 'boolean' doesn't override
    // the type guard's result (predicament?)
    const isAmbientExternalModule = isAmbientModule(node);

    // ...

    // Control flow analysis NARROWS `node` to AmbientModuleDeclaration
    if (isAmbientExternalModule) {
        // And here, because both isAmbientExternalModule and isExternalModuleAugmentation assert that `node` is AmbientModuleDeclaration ...
        if (isExternalModuleAugmentation(node)) {
            // `node` is `AmbientModuleDeclaration` here
        }
        // ... CFA will exhaust all possible types for `node` after the first if() statement
        // (what else can AmbientModuleDeclaration be if not an AmbientModuleDeclaration?)
        // so `node` will be `never`.
        else if ( /* `node` is `never` here... */ ) {

        }
        else {
            // ...and here.
        }
    }

    // ...
}

✅ Solution
Playground link the proposed solution based on prev. examples
src/compiler/checker.ts

function checkModuleDeclaration(node: ModuleDeclaration) {
    ...
    // Removing explicit boolean type annotation will allow CFA to narrow `node` in subsequent conditional statements.
-   const isAmbientExternalModule: boolean = isAmbientModule(node);
+   const isAmbientExternalModule = isAmbientModule(node);

    ...

    if (isAmbientExternalModule) {
         // Removing the second isAmbientModule() check (as a part of isExternalModuleAugmentation()) will prevent
         // CFA from narrowing `node` to `never`.
-       if (isExternalModuleAugmentation(node)) {
+       if (isModuleAugmentationExternal(node)) {
        ...

By design, per #44730:

Narrowing through indirect references occurs only when the conditional expression or discriminant property access is declared in a const variable declaration with no type annotation, and the reference being narrowed is a const variable, a readonly property, or a parameter for which there are no assignments in the function body.

Emphasis mine.

@fatcerberus thank you for the reference! I missed that somehow.


Updating setup-python action version to add support for python 3.10. …

…making all quotes single to be consistent. Adding python 3.10 to tox.ini
Signed-off-by: Alexander Xydes<EMAIL_ADDRESS>

Codecov Report

Merging #19 (02e1f09) into main (a939ea4) will increase coverage by 7.61%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##             main       #19      +/-   ##
===========================================
+ Coverage   92.38%   100.00%   +7.61%
===========================================
  Files           4         4
  Lines         289       289
===========================================
+ Hits          267       289      +22
+ Misses         22         0      -22

Impacted Files
Coverage Δ

...tooling/plugins/tool/dockerfilelint_tool_plugin.py
100.00% <0.00%> (+4.34%)
:arrow_up:

...ooling/plugins/tool/dockerfile_lint_tool_plugin.py
100.00% <0.00%> (+6.02%)
:arrow_up:

...atick_tooling/plugins/tool/hadolint_tool_plugin.py
100.00% <0.00%> (+12.38%)
:arrow_up:

Continue to review full report at Codecov.

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

Want to update .github/workflows/publish.yaml in this PR?


Refine dependencies on import-map for npm.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 92.04%. Comparing base (bea1c7d) to head (cd240db).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #364   +/-   ##
=======================================
  Coverage   92.04%   92.04%
=======================================
  Files          53       53
  Lines        3807     3807
  Branches      352      352
=======================================
  Hits         3504     3504
  Misses        302      302
  Partials        1        1

Flag
Coverage Δ

unittests
92.04% <ø> (ø)

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

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


When installing from source

When building from source using 'python setup.py install --user', I get this error:
Installed /home/golharr/.local/lib/python2.7/site-packages/pinball-0.2.9-py2.7.egg
Processing dependencies for pinball==0.2.9
Traceback (most recent call last):
File "setup.py", line 81, in
test_suite='tests',
File "/apps/sys/python/2.7.8/lib/python2.7/distutils/core.py", line 151, in setup
dist.run_commands()
File "/apps/sys/python/2.7.8/lib/python2.7/distutils/dist.py", line 953, in run_commands
self.run_command(cmd)
File "/apps/sys/python/2.7.8/lib/python2.7/distutils/dist.py", line 972, in run_command
cmd_obj.run()
File "build/bdist.linux-x86_64/egg/setuptools/command/install.py", line 67, in run
File "build/bdist.linux-x86_64/egg/setuptools/command/install.py", line 117, in do_egg_install
File "build/bdist.linux-x86_64/egg/setuptools/command/easy_install.py", line 370, in run
File "build/bdist.linux-x86_64/egg/setuptools/command/easy_install.py", line 594, in easy_install
File "build/bdist.linux-x86_64/egg/setuptools/command/easy_install.py", line 645, in install_item
File "build/bdist.linux-x86_64/egg/setuptools/command/easy_install.py", line 704, in process_distribution
TypeError: not enough arguments for format string

it seems not related to pinball but just easy_install issue: https://github.com/pypa/setuptools/issues/281
I also can't reproduce this issue. can you upgrade your setuptools and retry?


fix(tags): exclude non-specified

Previously, omegabrr was adding all monitored shows/movies to the list, regardless of whether they had the specified tags in tagsInclude. With this PR, omegabrr will now only add shows/movies with the tag(s) specified in tagsInclude in config.yaml.
If tagsInclude is not used, all monitored shows/movies (except those part of tagsExclude) will be added to the list.

This logic should allow both to work
Initially filter include all series for only tagsInclude
Then remove any series that exist in tagsExclude

Are you referring to a scenario where a show has tags in both tagsInclude and tagsExclude? Those are not added to autobrr.

Are you referring to a scenario where a show has tags in both tagsInclude and tagsExclude? Those are not added to autobrr.
EDIT: Answered and confirmed on Discord

What was the conclusion here? Needed or not?

This PR isn't needed. The program is already functioning as intended.
If tagsInclude is used, then only titles with those tags are added. If the titles also have a tag specified in tagsExclude, it will be excluded.

Not needed.


Adjust disk utilisation devices variable

The current variable "devices" overlaps with ceph, in that we need to
specify a list of "devices" for ceph to use, and this var is then taken
to be used by the disk utilisation monitoring.
This causes incorrect (and failing) disk utilisation MaaS
checks/alarms to be setup.
Fixes-Issue: #676
(cherry picked from commit d499449d84cf73cbaf7975aeaa921a50af1dbdfe)

lgtm :+1:


cannot run e2e tests locally

Describe the Bug
No response
Steps to Reproduce the Problem

modify e2e/Dockerfile to test localhost
docker build -t permaplant-e2e:build ./e2e
docker run -it permaplant-e2e:build
make test-e2e
python3  -m pytest -n auto e2e/ --retries 2 --video retain-on-failure --html=report.html --self-contained-html --cucumberjson=cucumber.json

Expected Result
successfully executed e2e tests
Actual Result
root@b25b26e9d5d9:/home/batman/permaplante2e# make test-e2e
make: *** No rule to make target 'test-e2e'.  Stop.
root@b25b26e9d5d9:/home/batman/permaplante2e# python3  -m pytest -n auto e2e/ --retries 2 --video retain-on-failure --html=report.html --self-contained-html --cucumberjson=cucumber.json
ImportError while loading conftest '/home/batman/permaplante2e/conftest.py'.
conftest.py:4: in <module>
    from e2e.pages.home import HomePage
E   ModuleNotFoundError: No module named 'e2e'

PermaplanT version
master
Operating System
Linux
Browser
chromium
Versions of other relevant software/extensions
No response
Additional info/logs
No response

I just noticed that make run-storybook installs e2e deps?

Yea, we should split installation now. At the beginning everything was added to install, but I agree, this doesnt make any sense.

Thank you btw for reporting this. Whilest developing the e2e tests, I was only using that Dockerfile inside the pipeline and it seems like Jenkins does the paths a bit different when building images. Jenkins copies the whole root folder, thats why we can use make test-e2e inside the pipeline. When building the image locally though, you only have /e2e/ and you are already inside that folder.
In #784 the following syntax now to run that docker image.
in /workspaces/PermaplanT/e2e $ docker build -t permaplant-e2e .
in /workspaces/PermaplanT/e2e $ docker run --network="host" permaplant-e2e ./e2e.sh

--network="host" only needed if your app is running on localhost.
I hope you can run e2e tests now locally 🚀.

I just noticed that make run-storybook installs e2e deps?

Thank you. Should also be fixed in #784


Feat/fuzz

Description
add fuzz
Related Issue
part of https://github.com/open-spaced-repetition/rs-fsrs/issues/22
Dependencies change
add rand as [dev-dependencies] for test

@L-M-Sherlock Would you mind reviewing my recent code changes?

I am ok with the code, waiting for @L-M-Sherlock 's opinion.

the fuzz seed needs refactoring in the future.
ok


Unexpected Effects of Special Char Restrictions

I think they should only affect Name and Affiliation, as they're the fields that end up in file names. See also #193.

Reverted your change for now.

The issue seems to be that spaces were not allowed in the pattern.
But like spaces there may be other characters that should perhaps be allowed.
I'll try to invert the pattern and instead deny only problematic characters.

Pushed [^\/]{1,100} now in c5df2b43cde480cabc1b3f81ff6821a7f3162174
Tested locally and it passed. I believe before it was also forcefully expecting 100 characters.

Thanks for the fix, Renato! Inverting the pattern seems like a very good solution. 👍
St*ff happens; no worries. We didn't get any other emails, so people may have just tried again a bit later, when I had already reverted it. 😌

Thanks for the quick action on your part :rocket:


Fix flamechart bleeding

Fixes #150

Coverage decreased (-0.007%) to 42.116% when pulling c73041a93b01172c0f56d168b0ab7053bad24861 on jlfwong/fix-flamechart-bleed into da2de64d97512b4a97dbd9f54914d00ca6bdd06a on master.


HBASE-24152 Add ServerSsdLocalityCostFunction to StochasticLoadBalancer

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 48s
Docker mode activated.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
5m 53s
the patch passed

+1 :green_heart:
compile
1m  5s
the patch passed

+1 :green_heart:
javac
1m  5s
the patch passed

+1 :green_heart:
shadedjars
5m 10s
patch has no errors when building our shaded downstream artifacts.

-0 :warning:
javadoc
0m 40s
hbase-server in the patch failed.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
hbaseanti
0m  0s
Patch does not have any anti-patterns.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

_ master Compile Tests _

+1 :green_heart:
mvninstall
4m 14s
master passed

+1 :green_heart:
checkstyle
1m 22s
master passed

+1 :green_heart:
spotbugs
2m 15s
master passed

_ Patch Compile Tests _

-1 :x:
mvninstall
2m  1s
root in the patch failed.

-0 :warning:
checkstyle
1m 21s
hbase-server: The patch generated 4 new + 185 unchanged - 0 fixed = 189 total (was 185)

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

-1 :x:
hadoopcheck
2m 22s
The patch causes 17 errors with Hadoop v3.1.2.

-1 :x:
hadoopcheck
4m 47s
The patch causes 17 errors with Hadoop v3.2.1.

-1 :x:
spotbugs
0m 35s
hbase-server in the patch failed.

_ Patch Compile Tests _

-1 :x:
mvninstall
2m 13s
root in the patch failed.

-1 :x:
compile
0m 45s
hbase-server in the patch failed.

-0 :warning:
javac
0m 45s
hbase-server in the patch failed.

-1 :x:
shadedjars
4m 11s
patch has 17 errors when building our shaded downstream artifacts.

-0 :warning:
javadoc
0m 43s
hbase-server in the patch failed.

_ Other Tests _

-1 :x:
unit
0m 47s
hbase-server in the patch failed.

22m 29s

Subsystem
Report/Notes

Docker
Client=19.03.8 Server=19.03.8 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1482/2/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/1482

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux 79443bfd2321 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / ec0d9d767e

Default Java
2020-01-14

javadoc
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1482/2/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-server.txt

mvninstall
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1482/2/artifact/yetus-jdk11-hadoop3-check/output/patch-mvninstall-root.txt

compile
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1482/2/artifact/yetus-jdk11-hadoop3-check/output/patch-compile-hbase-server.txt

javac
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1482/2/artifact/yetus-jdk11-hadoop3-check/output/patch-compile-hbase-server.txt

shadedjars
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1482/2/artifact/yetus-jdk11-hadoop3-check/output/patch-shadedjars.txt

javadoc
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1482/2/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-server.txt

unit
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1482/2/artifact/yetus-jdk11-hadoop3-check/output/patch-unit-hbase-server.txt

Test Results
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1482/2/testReport/

Max. process+thread count
102 (vs. ulimit of 12500)

modules
C: hbase-server U: hbase-server

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1482/2/console

versions
git=2.17.1 maven=(cecedd343002696d0abb50b32b541b8a6ba2883f)

Powered by
Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
1m  6s
Docker mode activated.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
4m 40s
the patch passed

+1 :green_heart:
compile
1m 12s
the patch passed

+1 :green_heart:
javac
1m 12s
the patch passed

+1 :green_heart:
shadedjars
6m 16s
patch has no errors when building our shaded downstream artifacts.

-0 :warning:
javadoc
0m 51s
hbase-server in the patch failed.

 Rebase required? Wrong Branch? See https://yetus.apache.org/documentation/in-progress/precommit-patchnames for help.

Subsystem
Report/Notes

GITHUB PR
https://github.com/apache/hbase/pull/1482

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1482/4/console

versions
git=2.17.1

Powered by
Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m  0s
Docker mode activated.

-1 :x:
patch
0m  3s
https://github.com/apache/hbase/pull/1482 does not apply to master. Rebase required? Wrong Branch? See https://yetus.apache.org/documentation/in-progress/precommit-patchnames for help.

Subsystem
Report/Notes

GITHUB PR
https://github.com/apache/hbase/pull/1482

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1482/4/console

versions
git=2.17.1

Powered by
Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m  0s
Docker mode activated.

-1 :x:
patch
0m  3s
https://github.com/apache/hbase/pull/1482 does not apply to master. Rebase required? Wrong Branch? See https://yetus.apache.org/documentation/in-progress/precommit-patchnames for help.

Subsystem
Report/Notes

GITHUB PR
https://github.com/apache/hbase/pull/1482

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1482/4/console

versions
git=2.17.1

Powered by
Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.

rebase

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m  0s
Docker mode activated.

-1 :x:
patch
0m  3s
https://github.com/apache/hbase/pull/1482 does not apply to master. Rebase required? Wrong Branch? See https://yetus.apache.org/documentation/in-progress/precommit-patchnames for help.

Subsystem
Report/Notes

GITHUB PR
https://github.com/apache/hbase/pull/1482

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1482/5/console

versions
git=2.17.1

Powered by
Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m  0s
Docker mode activated.

-1 :x:
patch
0m  3s
https://github.com/apache/hbase/pull/1482 does not apply to master. Rebase required? Wrong Branch? See https://yetus.apache.org/documentation/in-progress/precommit-patchnames for help.

Subsystem
Report/Notes

GITHUB PR
https://github.com/apache/hbase/pull/1482

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1482/5/console

versions
git=2.17.1

Powered by
Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m  0s
Docker mode activated.

-1 :x:
patch
0m  3s
https://github.com/apache/hbase/pull/1482 does not apply to master. Rebase required? Wrong Branch? See https://yetus.apache.org/documentation/in-progress/precommit-patchnames for help.

Subsystem
Report/Notes

GITHUB PR
https://github.com/apache/hbase/pull/1482

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1482/5/console

versions
git=2.17.1

Powered by
Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.

In case of cluster not over HDFS , all these can be skipped using hbase.master.balancer.uselocality = false correct?  Then none of these code path will get executed by the LB system. Correct?
On a side note (not on this patch) , seems we are using the HDFSBlockDistribution at many other places like CompactionTool, DateTieredCompactionPolicy etc. On non HDFS based storage, all these are unwanted ops but we are not skipping.  Might be good to do those also? Ping @saintstack

Can't find the reason of the error,  compile and run success at local.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m  0s
Docker mode activated.

-1 :x:
patch
0m  4s
https://github.com/apache/hbase/pull/1482 does not apply to master. Rebase required? Wrong Branch? See https://yetus.apache.org/documentation/in-progress/precommit-patchnames for help.

Subsystem
Report/Notes

GITHUB PR
https://github.com/apache/hbase/pull/1482

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1482/1/console

versions
git=2.17.1

Powered by
Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m  0s
Docker mode activated.

-1 :x:
patch
0m  3s
https://github.com/apache/hbase/pull/1482 does not apply to master. Rebase required? Wrong Branch? See https://yetus.apache.org/documentation/in-progress/precommit-patchnames for help.

Subsystem
Report/Notes

GITHUB PR
https://github.com/apache/hbase/pull/1482

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1482/1/console

versions
git=2.17.1

Powered by
Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m  0s
Docker mode activated.

-1 :x:
patch
0m  4s
https://github.com/apache/hbase/pull/1482 does not apply to master. Rebase required? Wrong Branch? See https://yetus.apache.org/documentation/in-progress/precommit-patchnames for help.

Subsystem
Report/Notes

GITHUB PR
https://github.com/apache/hbase/pull/1482

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1482/1/console

versions
git=2.17.1

Powered by
Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m  0s
Docker mode activated.

-1 :x:
patch
0m  3s
https://github.com/apache/hbase/pull/1482 does not apply to master. Rebase required? Wrong Branch? See https://yetus.apache.org/documentation/in-progress/precommit-patchnames for help.

Subsystem
Report/Notes

GITHUB PR
https://github.com/apache/hbase/pull/1482

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1482/2/console

versions
git=2.17.1

Powered by
Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m  0s
Docker mode activated.

-1 :x:
patch
0m  3s
https://github.com/apache/hbase/pull/1482 does not apply to master. Rebase required? Wrong Branch? See https://yetus.apache.org/documentation/in-progress/precommit-patchnames for help.

Subsystem
Report/Notes

GITHUB PR
https://github.com/apache/hbase/pull/1482

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1482/2/console

versions
git=2.17.1

Powered by
Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m  0s
Docker mode activated.

-1 :x:
patch
0m  3s
https://github.com/apache/hbase/pull/1482 does not apply to master. Rebase required? Wrong Branch? See https://yetus.apache.org/documentation/in-progress/precommit-patchnames for help.

Subsystem
Report/Notes

GITHUB PR
https://github.com/apache/hbase/pull/1482

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1482/2/console

versions
git=2.17.1

Powered by
Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
1m 35s
Docker mode activated.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
4m 53s
the patch passed

+1 :green_heart:
compile
1m 15s
the patch passed

+1 :green_heart:
javac
1m 15s
the patch passed

+1 :green_heart:
shadedjars
6m 57s
patch has no errors when building our shaded downstream artifacts.

-0 :warning:
javadoc
0m 45s
hbase-server in the patch failed.

_ Other Tests _

+1 :green_heart:
unit
139m  4s
hbase-server in the patch passed.

169m 39s

Subsystem
Report/Notes

Docker
Client=19.03.8 Server=19.03.8 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1482/3/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/1482

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux f6f6584c14cd 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / c1cb22f0b3

Default Java
2020-01-14

javadoc
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1482/3/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-server.txt

javadoc
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1482/3/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-server.txt

Test Results
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1482/3/testReport/

Max. process+thread count
4257 (vs. ulimit of 12500)

modules
C: hbase-server U: hbase-server

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1482/3/console

versions
git=2.17.1 maven=(cecedd343002696d0abb50b32b541b8a6ba2883f)

Powered by
Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 27s
Docker mode activated.

-0 :warning:
yetus
0m  3s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ master Compile Tests _

+1 :green_heart:
mvninstall
4m 23s
master passed

+1 :green_heart:
compile
1m  1s
master passed

+1 :green_heart:
shadedjars
6m 14s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 44s
master passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
4m 19s
the patch passed

+1 :green_heart:
compile
1m  8s
the patch passed

+1 :green_heart:
javac
1m  8s
the patch passed

+1 :green_heart:
shadedjars
6m 15s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 42s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
207m 59s
hbase-server in the patch passed.

235m 11s

Subsystem
Report/Notes

Docker
Client=19.03.8 Server=19.03.8 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1482/3/artifact/yetus-jdk8-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/1482

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux 1025f4c6f190 4.15.0-74-generic #84-Ubuntu SMP Thu Dec 19 08:06:28 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / c1cb22f0b3

Default Java
1.8.0_232

Test Results
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1482/3/testReport/

Max. process+thread count
2504 (vs. ulimit of 12500)

modules
C: hbase-server U: hbase-server

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1482/3/console

versions
git=2.17.1 maven=(cecedd343002696d0abb50b32b541b8a6ba2883f)

Powered by
Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.

@bsglz  Any comment on the @anoopsjohn remark above?
File a follow-on @anoopsjohn  ?


Possible High CPU Usage Caused by Code Cracker

I opened the solution of Code Cracker, and tried to edit it, for instance, by removing the ? operator in line 78 of DisposableFieldNotDisposedCodeFixProvider.cs.
The CPU usage went high (up to 32% on my Intel i5 Processor in 5 seconds later) after removing that character in the editor and fell back to normal after some seconds.
When I edited another part of the code or undid the previous change, the CPU usage went high again.
If I disabled Code Cracker from the Extension Manager, the high CPU usage issue was not so obvious (never went higher than 10%).
Sometime I have to work on my laptop with the battery, high CPU usage could reduce the duration of the power supply. Could you please take some time on this performance issue?

I just tried that, I edited this line:
https://github.com/code-cracker/code-cracker/blob/b4e2060894506f7ec959ced727f05891c2ecd70a/src/CSharp/CodeCracker/Usage/DisposableFieldNotDisposedCodeFixProvider.cs#L53
I removed the ? as you indicated, my CPU spiked to 50% and went back to normal. I removed CodeCracker from the analyzers for the CodeCracker.csproj project, and tried again, and noticed the exact same spike.
I am sure that analyzers impact performance. The way roslyn does it is that error and warnings analyzers run on all the project files, and info and hidden only when you open a document (I am not sure about the details). Code fixes run before you ask for them, when you have your cursor on top of a node that has a diagnostic.
I have asked the Roslyn team for advice on how to measure performance for analyzers, and they have no guidance whatsoever at this moment. What is a fast analyzer? What is a slow analyzer? What are the numbers?
I will keep this issue open, but I am not sure yet on how to proceed. If anyone has any idea we would like to hear you.

Also, there is an open issue here to discuss that: https://github.com/dotnet/roslyn-analyzers-contrib/issues/15

@giggio One quick check would be to build your project with /reportanalyzer command line switch (you can also specify it in the msbuild project file with a property ReportAnalyzer). The build output should contain relative analyzer execution times (these are not absolute times as build runs analyzers in parallel). If you see any particular analyzer taking up a large portion of this time, then the issue might be with that analyzer. Otherwise, it may be in the IDE background diagnostic analysis service or code fixers. If it is the later, then I can try to repro and profile this locally to find the offender.

I used the "Extensions Monitor" (an extension in the VS gallery) to investigate the performance issue.
The monitor showed that the high-CPU consumption was occurring in the code analysis module of VS.
After disabling Code-Cracker, the high-CPU consumption caused by the code analysis module was gone in the mean time. Thus I suspect that it had something to do with Code-Cracker.
At most time while we are editing the code, we only change the implementation part INSIDE methods or properties, without changing their signature.
If Code-Cracker can limit the code analysis within the changed member, without parsing the whole project from scratch, the performance issue shall be fixed.

@mavasani Ok, I'll try that and let you know.


W2D1 curriculum specialist & day lead updates

This PR replaces last year's W2D1 tutorials with this years updated set of 4 tutorials.
Checked with Python 3.9 locally

@iamzoltan I have updated the PR to fix all the issues you mentioned. I won't be able to address any other significant changes until next week.


Make availability set deletion optional

Is your feature request related to a problem? Please describe.
We have a configuration where we get specific availability sets of a specific resource group pinned to a specific cluster. This allows us to make use of dedicated resources. We create all the AVSets up front and configure all instance group to deploy into those AVsets. If we now recreate certain deployments and BOSH finds a AVSet without any VMs left, it will delete it and our pinning will be gone. BOSH will recreate the AVSet once it creates new resources in them, but those won't be pinned now.
Describe the solution you'd like
I think an option like delete_empty_availability_set which defaults to true in the global CPI settings might be reasonable.
Describe alternatives you've considered
We are looking into locking the AVSets we care about. We have not tested yet and don't know yet how the CPI behaves when it want's to delete a locked AVSet.
We also considered deploying "a dummy VM" into the AVSets in question. However this is a pretty heavy workaround.

As far as I can see, this is the (only) place where the deletion is actually happening:
https://github.com/cloudfoundry/bosh-azure-cpi-release/blob/8375dce86c6f3d3632061fa6657cba7b5f540a9c/src/bosh_azure_cpi/lib/cloud/azure/vms/vm_manager_availability_set.rb#L65-L70

Ideally the CPI should only cleanup or delete availability sets in which bosh creates. This could be handled via a tag on the AV set or similar.

What I proposed previously would be ideal for my situation, it's not backwards compatible therefore it's not tenable. One of @hoegaarden 's suggestions would be better.


Add support for the read half of the PR Review API

Just putting this up to solicit some initial feedback before I get too deep in the rest of this API.
Fixes GH-133

@mscharley Indeed, it's looking nice 👏

This is ready for final review now.

I've limited the scope of this PR to the read-only portion of the review API as this is all I need right now, I'm on a deadline, and the write half needs some special logic around draft reviews.

Please, also see this one https://github.com/47deg/github4s/pull/139#pullrequestreview-39483448 ;)

@juanpedromoreno That's already fixed, I noticed that when I did the merge.

Before merging, let's wait until @BenFradet 's 👍 , many thanks @mscharley !

Still not sure how this happened, looks like it was actually a regression. Is someone able to run the tests where they're known to be working before merging this? I'm still getting a few failures locally but they look to me like they're related to my key rather than the test... but I could easily be wrong too.
[info] Activity >> Set a thread subscription
[info] - should return the provided gist
[info] - should return a list of references
[info] - should return at least one reference
[info] - should return an error when an invalid repository name is passed
[info] - should return one commit
[info] - should return an error when an invalid repository name is passed
[info] - should return a list of issues
[info] - should return at least one issue for a valid query
[info] - should return an empty result for a non existent query string
[info] - should edit the specified issue *** FAILED ***
[info]   false was not equal to true (BaseIntegrationSpec.scala:47)

The tests are passing locally for me.

:+1: :shipit:

Merging when green :+1:

The tests don't seem to work on Travis because I'm not a contributor so Travis valiantly refuses to give me your keys for running tests in case i'm evil and send them on somewhere else. There's a bunch of 401 (auth required) errors in the integration tests as a result.

Yeah, I'm not sure how could we make it working for project forks. We usually test the PRs locally.


Transform a change by using an undeleted visible cell or by ignoring exact replacements

Coming from TomasMikula/RichTextFX#390 and #39, the code did not account for the possibility of a deletion that occurs before a StartOffStart's itemIndex and continues into the viewport. Thus, the code before this PR would scroll to the first cell even if this was not desired. This PR fixes the issue.
However, in the situation where the removedSize == addedSize, one does not know whether the offset value should be adjusted to 0.0 or left as is so that the viewport appears not to scroll at all.
Using RichTextFX as an example, if one scrolled the viewport slightly so that the first visible line was only halfway visible before removing some regular text and replacing it with a tall image, should the resulting offset value be 0.0 or left as is? What might prevent undesirable scrolling in one situation may force the same in another.

In this new approach, the 2nd branch of the IF-block does 2 things before defaulting to the behavior before this PR:

use the first or last visible cell as the new anchor cell if they weren't deleted
don't change anything if an exact replacement occurs (removed == added)

Testing this out with synth3's example still scrolled a bit. So, I think this problem should be studied a bit more before merging.

The problem I encountered above occurred when I would scroll to the bottom of the area before clicking the reset button. I expected the last cell to be rendered at the bottom of the viewport. However, the viewport would render the second to last cell on the bottom and fill up the rest of the viewport from there.
After searching for quite a while, I narrowed down what was causing the problem. Turns out this current PR works just fine when the VirtualFlow is not wrapped in a VirtualizedScrollPane. It seems that, somewhere in the layout process, the length of the viewport is changed due to the scroll bars disappearing temporarily. Thus, distanceFromSky returns a positive value since the viewport's length is greater than the last cell's maxY value when they should be equal. Thinking that there is still space to fill, the viewport will render additional content from the "ground" up, causing the actual anchor cell to be pushed down in the viewport, causing the last cell to be pushed out of the viewport.

I decided not to use the first/last visible cell as the new anchor cell should the current one be deleted. When transforming the targetPosition across multiple modifications, the "first" and "last" visible cells aren't always the first/last ones through multiple transformations if they get deleted. Addressing this to account for that possibility is outside of the scope of this PR, which only seeks to fix #39. So, I'll not address it now.


Adds Table of Content to... well, Table of Content

I don't know why, but it's kind of funny to me to see Table of Content listed in Table of Content, so I thought I should submit that as a pull request.
Note to other collaborators
Please let @sindresorhus decide whether or not to accept this. I'm submitting this as a pull request because I don't want to make him mad.

I'm happy to rename it to Table of Contents, but having it in itself is pretty moot, so no.

I'm submitting this as a pull request because I don't want to make him mad.

Sorry to dissapoint, but I don't think anyone on the internet could make me mad :p


UR Banner - add heading for navigation and identification

Library or service name
https://github.com/hmrc/play-frontend-hmrc
Related component
UR Banner
Context
The UR banner has been flagged by DAC in a couple of audits as having what looks like a heading but is not a heading, prompting confusion at it not showing up in the heading list / be skipped over when navigating by heading.
https://github.com/hmrc/play-frontend-hmrc/blob/main/src/main/twirl/uk/gov/hmrc/hmrcfrontend/views/components/HmrcUserResearchBanner.scala.html#L26
Seems a fair issue when similar temporary banners have h2 to aid navigation.
The fix would be to add a heading (eg a h2 to stay with the cookie banner / error summary / notification banner pattern of things above the h1 getting an h2).

Hi @adamliptrot-oc, thank you for raising this Github issue. PlatUI are now reviewing your issue. The next step will be for us to assign one of the following labels: accepted, cantfix, invalid, wontfix. We’ll update on this issue when a label is assigned.

Raised Jira ticket PLATUI-1721 to fix.


Update: WestWind.MarkdownMonster v<IP_ADDRESS>

[X] Have you signed the Contributor License Agreement?
[X] Have you checked that there aren't other open pull requests for the same manifest update/change?

[X] Does your manifest conform to the 1.0 schema?

Microsoft Reviewers: Open in CodeFlow

Another one of these..

@palenshus can you take a look at this one as it passes Sandbox?

/azp run


#672 Pass -Proxy parameter so we can find packages

Fix for dnvm upgrade -Proxy <server> not working behind a proxy server, by passing parameter on to Find-Latest and Find-Package methods.
I have docu-signed the Contributor License Agreement.
#672

Hi @stajs, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
We will now validate the agreement and then real humans will evaluate your PR.
TTYL, DNFBOT;

Thanks @stajs but changes to DNVM need to happen in the DNVM repo. This is the output of the DNVM build. Could you move this over there?

Will do.


Update backup-rbac-rs-vault.md

Under the heading minimum role requirements for the Azure File share backup, for the operation to enable backup of Azure file shares, the document lists "Storage Account" in the role required column and "Contributor Storage account resource" in the resources column.
The role required should be "Storage Account Contributor" and the resource should be "Storage account resource". This change moves the third pipe on that line to fix it.

@llewellyn-marriott : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@ktoliver and Team, requst you to please merge. Thanks.

#sign-off


Add Touch support

Is there a way to add touch support to the demo?
I tried hooking up to the ontouchstart or other ontouch... events but Container.Payload is always null. Any idea?

@DashNY You need to add the polyfill provided by mobile-drag-drop as mentioned here: https://github.com/Postlagerkarte/blazor-dragdrop#mobile-devices


[WIP] Feature/01/teku catalyst spinoff

This is docker approach, its still in progress. I'll leave more information here later on

This looks more like an epic.
I have decided to push changes into epic branch instead of main


How to retrieve relationship with its metadata

** Relationship and metadata **
Le'ts say we have a simple query:
Client.Cypher
.Match("(n:ASSET)-[r]->[t:TARGET]")
.Return((n, r, t) => new {
  relationship = r.As<RelationshipEntity>()
}).ResultsAsync

Given entity class:
public class RelationshipEntity {
    [JsonProperty(PropertyName = "myValue")]
     public string MyValue { get; set; }
}

If I execute this, it works as expected, however I want to get the metadata, such as ID and Start/End node ID's as I need to rebuild the graph in the UI.
The only example I found is changing code to something like this:
Client.Cypher
.Match("(n:ASSET)-[r]->[t:TARGET]")
.Return((n, r, t) => new {
  relationship = r.As<RelationshipInstance<RelationshipEntity>>()
}).ResultsAsync

But executing this gives an exception saying -> Value cannot be null. (Parameter 'uriString')
Any thoughts?

Perhaps a working example as I am debugging Neo4jCLient lib now, this is running against local docker container with movies sample DB:
`using System;
using System.Collections.Generic;
using Neo4jClient;
using Newtonsoft.Json;
using Newtonsoft.Json.Serialization;
var uri = new Uri("bolt://localhost:7687");
var client = new BoltGraphClient(uri, "neo4j", "slapt123")
{
JsonContractResolver = new CamelCasePropertyNamesContractResolver()
};
await client.ConnectAsync();
var result = await client.Cypher.Match("(a:Person { name: 'Tom Hanks' })-[r]->(m)").Return((a, r, m) =>
new {
relationship = r.As<RelationshipInstance>()
}).ResultsAsync;
Console.WriteLine(JsonConvert.SerializeObject(result, Formatting.Indented));
public sealed class Relationship
{
public List Roles { get; set; }
}`
Full exception message:
Unhandled exception. System.ArgumentException: Neo4j returned a valid response, however Neo4jClient was unable to deserialize into the object structure you supplied.
First, try and review the exception below to work out what broke.
If it's not obvious, you can ask for help at http://stackoverflow.com/questions/tagged/neo4jclient
Include the full text of this exception, including this message, the stack trace, and all of the inner exception details.
Include the full type definition of <>f__AnonymousType01[[Neo4jClient.RelationshipInstance1[[Relationship, Neo4jSample.ConsoleApp, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null]], Neo4jClient, Version=<IP_ADDRESS>, Culture=ne
utral, PublicKeyToken=null]].
Include this raw JSON, with any sensitive values replaced with non-sensitive equivalents:
{"columns":["relationship"],"data":[[{"data":{"roles":["Jim Lovell"]}}]]} (Parameter 'content')
---> System.Reflection.TargetInvocationException: Exception has been thrown by the target of an invocation.
---> System.ArgumentNullException: Value cannot be null. (Parameter 'uriString')
at System.Uri..ctor(String uriString)
at Neo4jClient.ApiModels.RelationshipApiResponse1.GetLastPathSegment(String uri) in D:\Work\github\Neo4jClient\Neo4jClient\ApiModels\RelationshipApiResponse.cs:line 48    at Neo4jClient.ApiModels.RelationshipApiResponse1.ToRelationshipInstance(IGraphClient client) in D:\Work\github\Neo4jClient\Neo4jClient\ApiModels\RelationshipApiResponse.cs:line 28
--- End of inner exception stack trace ---
at System.RuntimeMethodHandle.InvokeMethod(Object target, Span1& arguments, Signature sig, Boolean constructor, Boolean wrapExceptions)    at System.Reflection.RuntimeMethodInfo.Invoke(Object obj, BindingFlags invokeAttr, Binder binder, Object[] parameters, CultureInfo culture)    at System.Reflection.MethodBase.Invoke(Object obj, Object[] parameters)    at Neo4jClient.Serialization.CypherJsonDeserializer1.b__12_6(Object n) in D:\Work\github\Neo4jClient\Neo4jClient\Serialization\CypherJsonDeserializer.cs:line 214
at Neo4jClient.Serialization.CommonDeserializerMethods.MutateObject(DeserializationContext context, JToken value, IEnumerable1 typeMappings, Int32 nestingLevel, TypeMapping mapping, Type propertyType) in D:\Work\github\Neo4jCl ient\Neo4jClient\Serialization\CommonDeserializerMethods.cs:line 343    at Neo4jClient.Serialization.CommonDeserializerMethods.CoerceValue(DeserializationContext context, PropertyInfo propertyInfo, JToken value, IEnumerable1 typeMappings, Int32 nestingLevel) in D:\Work\github\Neo4jClient\Neo4jClie
nt\Serialization\CommonDeserializerMethods.cs:line 210
at Neo4jClient.Serialization.CypherJsonDeserializer1.<>c__DisplayClass22_0.<ReadProjectionRowUsingCtor>b__0(JToken cell, Int32 cellIndex) in D:\Work\github\Neo4jClient\Neo4jClient\Serialization\CypherJsonDeserializer.cs:line 5 33    at System.Linq.Enumerable.SelectIterator[TSource,TResult](IEnumerable1 source, Func3 selector)+MoveNext()    at System.Collections.Generic.LargeArrayBuilder1.AddRange(IEnumerable1 items)    at System.Collections.Generic.EnumerableHelpers.ToArray[T](IEnumerable1 source)
at System.Linq.Enumerable.ToArray[TSource](IEnumerable1 source)    at Neo4jClient.Serialization.CypherJsonDeserializer1.ReadProjectionRowUsingCtor(DeserializationContext context, JToken row, IDictionary2 propertiesDictionary, IList1 columnNames, IEnumerable1 jsonTypeMappings, ConstructorIn fo ctor) in D:\Work\github\Neo4jClient\Neo4jClient\Serialization\CypherJsonDeserializer.cs:line 525    at Neo4jClient.Serialization.CypherJsonDeserializer1.<>c__DisplayClass21_1.b__5(JToken token) in D:\Work\github\Neo4jClient\Neo4jClient\Serialization\CypherJsonDeserializer.cs:line 488
at System.Linq.Enumerable.SelectEnumerableIterator2.ToArray()    at System.Linq.Enumerable.ToArray[TSource](IEnumerable1 source)
at Neo4jClient.Serialization.CypherJsonDeserializer1.Deserialize(String content, Boolean isHttp) in D:\Work\github\Neo4jClient\Neo4jClient\Serialization\CypherJsonDeserializer.cs:line 76    --- End of inner exception stack trace ---    at Neo4jClient.Serialization.CypherJsonDeserializer1.Deserialize(String content, Boolean isHttp) in D:\Work\github\Neo4jClient\Neo4jClient\Serialization\CypherJsonDeserializer.cs:line 110
at Neo4jClient.BoltGraphClient.ParseResults[TResult](IEnumerable`1 result, CypherQuery query) in D:\Work\github\Neo4jClient\Neo4jClient\BoltGraphClient.cs:line 395
at Neo4jClient.BoltGraphClient.Neo4jClient.IRawGraphClient.ExecuteGetCypherResultsAsync[TResult](CypherQuery query) in D:\Work\github\Neo4jClient\Neo4jClient\BoltGraphClient.cs:line 365
at Program.$(String[] args) in D:\Work\trash\Neo4jSample\Neo4jSample.ConsoleApp\Program.cs:line 17
at Program.(String[] args)

Specifically what I am after is response like Neo4J browser gives
Node
{   "identity": 71,   "labels": [     "Person"   ],   "properties": { "born": 1956, "name": "Tom Hanks"   } }
Relationship
{   "identity": 202,   "start": 71,   "end": 144,   "type": "ACTED_IN",   "properties": { "roles": [       "Jim Lovell"     ]   } }

Awesome, thats exactly what I am after. How would I use Node or RelationshipInstance or these are for internal use only, as I can't find any examples in the docs?


Clicking on Books Does not Display Chapters, Only Whitespace

For Bug Reports

BookStack Version (Found in settings, Please don't put 'latest'): v0.18.5
PHP Version: 5.6.xxxx
MySQL Version: 5.6

Expected Behavior
Page loads content
Current Behavior
Page does not load content, I'm able to login to the application, am provided the recent docs, able to navigate books, but any further and I'm greeted with a blank page where content should be. I can still access the rest of the web app without issue, only am unable to view chapters.
Here is a screencap of a Book that has several chapters and pages after clicked on:

Steps to Reproduce
Logged in, click books, select book, no pages.
Additionally... I enabled some security functions on the vhost, rolled them back, didnt make a difference.

Hey @lithium-ap,
Sorry to hear that you're having problems. Could you please follow the steps here - https://www.bookstackapp.com/docs/admin/debugging/ and give me the laravel.log file?

Oh I see now, you should be using PHP 7.x. That probably is the issue.
Requirements available here - https://www.bookstackapp.com/docs/admin/installation/

@lithium-ap Is this on a new instance or after an update?
Previous to BookStack v0.19 the min required version of php was 5.6.4. Double check your exact php version to double check if this could be the issue.
For v0.19+ php 7.0.0 is the minimum so might be worth updating both PHP and BookStack if possible.

Hi Dan,
I am using PHP Version 5.6.31-4 on this server, but did have it working prior to testing a php7.1 update. Anyhow, I moved it to a new web server with php7.1 and it's working fine.


On BufUnload, unload clients started by lspconfig

See https://github.com/neovim/nvim-lspconfig/issues/2816

The timer stuff looks like trouble. And the counterpoint is that some servers might take time to start, then users have to wait for them again.


Fix exported and renamed bank tags containing more items than necessary (cosmetic issue).

Currently it adds all items ids that are in other bank tags that could be included in the target bank tag if those items were in the bank. e.g., if you have a specific clue scroll in one bank tag and variation tagged clues in another, when you export the second tag, it will contain not only the variation tag clue scroll id, but also the specific clue scroll id in the first bank tag.
Tested export/import, renaming bank tags, and ge search, which all use the changed method.
I don't know if this is the best way to do it - it might be possible to just read the tag list directly from the config rather than going through getTags to get it. I didn't try that.

This is not a cosmetic issue, as it can cause a variation id to show up in exported tags that aren't in the tag. This will cause the tag to import incorrectly. e.g. a tag with an uncharged glory will contain both uncharged glory and all glory variants when exported.


Bug 2262052: object: fix the default multisite zonegroup creation

there was a bug in the code which doesn't
sets realm if we try to get the zonegoup
so because of that 1.12 this backport pr
for original change was not working
Signed-off-by: parth-gr<EMAIL_ADDRESS>(cherry picked from commit a9c77280c6cde03cf129afc988711ad7135fe98c)

Description of your changes:
Which issue is resolved by this Pull Request:
Resolves #
Checklist:

[ ] Commit Message Formatting: Commit titles and messages follow guidelines in the developer guide).
[ ] Skip Tests for Docs: If this is only a documentation change, add the label skip-ci on the PR.
[ ] Reviewed the developer guide on Submitting a Pull Request
[ ] Pending release notes updated with breaking and/or notable changes for the next minor release.
[ ] Documentation has been updated, if necessary.
[ ] Unit tests have been added, if necessary.
[ ] Integration tests have been added, if necessary.

/bugzilla refresh

/lgtm
/approve

/lgtm

helm upgrade suite(1.23) looks failing in master always https://github.com/rook/rook/actions/workflows/integration-test-upgrade-suite.yaml
So this needs to be fixed in the master, not per the above PR changes.


Look into a json diff solution

To attach to PRs for updating golden.json so the reviewer can see what the changes are.

https://stackoverflow.com/questions/31930041/using-jq-or-alternative-command-line-tools-to-compare-json-files
https://github.com/josephburnett/jd
https://github.com/jlevy/pdiffjson

@rebeccalipton A diff tool visualizes the changes made to a file. Typically the original file is shown on the left, and the new version on the right. Here's an example of a diff generated by GitHub. https://github.com/nasa-petal/PeTaL-labeller/pull/85/files Unfortunately GitHub's built-in diff tool doesn't work for large files. We need to generate diffs for this file - https://github.com/nasa-petal/data-collection-and-prep/blob/main/golden.json. The diff report should ideally be exportable to pdf, or html so it can be attached to GitHub pull requests.

I found three options to create a diff file for JSON comparisons. These all render output that can be attached to a commit, and therefore GitHub pull requests as well.
https://pypi.org/project/json-diff/
"Compares two JSON files and generates a new JSON file with the result."
https://github.com/trailofbits/graphtage
Compares two file types (including JSON) and generates a new file with the output. Can specify output type, such as html.
https://github.com/j-tester/json-diff-cli
Finds the diff between two JSON URLs. Prints the diff to console or outputs as csv file.

Thanks! I will check them out.

I like the generated JSON file as a result approach. This should work for all cases; however, sometimes a JSON can be represented as a CSV file, I think golden json can be a csv so that's a better option.

Looks like graphtage is the most promising out of the three.

After evaluating multiple libraries and packages to generate a diff file for the golden.json, three options to complete this task are detailed in an earlier comment. Upon further inspection by the PeTaL team, the graphtage library seems like a top contender since it can render the diff as an HTML file, which can then be committed with Git. The graphtage library has the most similarity to the traditional diff files that GitHub renders.


Several bugs on collection's item template

Describe the bug
There are several issue while handling a collection's item template :

after generating a template item for a collection the system redirect you on the template edit page. If you go back directly to the collection edit page the new template item is not yet displayed, but you have to refresh the page to have it. I suppose it's only a cache issue.
when generating a template item for a collection the template item edit page show the metadata belonging to the related collection. This can be seen quickly by opening the browser developer tool and looking for the creation request :

trying to remove displayed metadata doesn't work because they don't really belong to the item template object, but are collection's metadata
trying to add a new metadata returns a 404 error.
when deleting the item template the interface show both a success and an error notification, but it seems it has been deleted successfully. indeed if you refresh the page item template no longer exists

@artlowel : Could you (or someone on your team) provide an estimate for these fixes?  It looks like the initial implementation was in #749

All issues, except the dual notification error are due to the fact that the item template editor shares code with the item metadata editor, and it will know which item to work on by using the dso resolved in the route. For item templates, that dso is the collection. So the ItemMetadataComponent will need to be refactored to get its item from an input.
The dual notification issue is separate and is likely due to the notification not waiting for the remotedata to be resolved. So it shows a failed notification while it's loading and a succeeded notification as it completes
Fixing these issues will take an estimated 6h


Turn off banner in STDERR

I don't really find it any useful or even common for allocators to report they're being put in use, what's the logical reasoning behind it?

No one else is doing that.
For those who need details there's pmap that handles most of cases.

Fair enough. Certainly not a problem to disable it.

Disabled with https://github.com/emeryberger/Hoard/commit/de06b8b79b06a7e52caf1b7187419af9c57ea43e


No db when running locally

I am setting up a local development environment according to the README.
Everything installs and starts successfully.
But, when I run docker-compose up -d nginx ckan logs the following error: psql: could not translate host name "db" to address: Name or service not known
provisioning api throws:
provisioning-api_1     | KeyError: 'PRIVATE_SSH_KEY'
I guess I need a an SSH key to provision the database.

Derrf... this is documented. Working my way through the installation.


Add tests for recursive types in defaulter-gen

Fix for defaulter-gen to support recursive types is here: https://github.com/kubernetes/gengo/pull/61.
This commit adds tests for it. The tests should pass once https://github.com/kubernetes/gengo/pull/61 is merged.
/cc @sttts

Updated and addressed comments.

lgtm


Request to Review Draft Proposal for GSoC 2023

I sent a request to the BSureshKrishna GitHub account regarding the Project Proposal. Would you please look into it?

I have not received it.. or do not know how to find it.. in any case, Neurostars does have a private messaging feature if you want to reach me.

Ohh ok, Then here's my google doc link for you to access my draft proposal. Kindly take a look into it and suggest any feedback/suggestions in the google doc link since I am unable to message you privately on Neurostars Forum.
Draft Proposal

Also here's a screenshot regarding the collaborator request I sent to your GitHub account

Hope I sent it to the correct GitHub account for you to reach it.


spike: research how vtrust and vpermit works

We currently see that miners on our network get vtrust which implicitly makes them validators.
Research how vtrust gets set for both a validator and miner, an miner should not get vtrust otherwise it is a validator or miner/validator

Ask in Discord

Tracking conversation here

Based on the conversation in Discord, the Testnet may be configured in such a way that there is no minimum TAO required to acquire a vpermit.  Attempted to compare hyperparameters between devnet and testnet, but the call:
btcli subnets hyperparameters --subtensor.$(SUBTENSOR_ENVIRONMENT) --netuid $(NETUID)
is failing on devnet.  creating a fresh miner now on devent to see the vpermit behavior

After doing some work with miner-2, we see that it now has a vpermit!  Interesting because no staked TAO appears on miner-2, but perhaps it's a super small amount.  Will pivot into more research

Research
VPermit

Dynamic Competition: The minimum stake required to obtain a VPermit is dynamically determined based on the stakes of the top "x" validators in a given subnet. This means that the minimum stake is the stake of the "x"th highest staked neuron within that subnet.  This is why once a miner does work and has staked TAO as a result, they are marked w/ a vpermit.
Observation: We can set a minimum amount of staked TAO required to be considered a validator by a miner to ensure that initial miners aren't considered validators.

VTrust

Description measure of a neuron's reputation within the network. It is calculated based on various factors including the neuron's historical activity, the accuracy of its validations, and its overall contribution to the network's health.

Validator / Miner Identification

VPermit Status: If a neuron has a VPermit, it is authorized to act as a validator.
Stake Amount: Generally, validators have higher stakes compared to miners, reflecting their greater responsibilities and the trust placed in them by the network.
Activity Logs: By examining the types of tasks a neuron performs (validating transactions vs. performing computational tasks for the network), you can distinguish between validators and miners.

Adding a stake check on the synapse can look like the following in miner/forward.py:
# Get the current stakes from the metagraph
        current_stakes = self.metagraph.S

        # Define the minimum stake required
        min_stake = 1000  # Adjust this value as needed

        # Check if the calling neuron meets the minimum stake requirement
        caller_uid = self.metagraph.hotkeys.index(synapse.dendrite.hotkey)
        if current_stakes[caller_uid] < min_stake:
            bt.logging.warning(f"Rejecting synapse from neuron with insufficient stake: {current_stakes[caller_uid]}")
            return synapse  # You might want to modify the response to indicate rejection

It looks like applying that min_stake may be the way to go, have you done any research about what kind of minimums are other subnets using?
Based on that we should be able to create a task for implementation

Really great work here @grantdfoster - lets do an implementation ticket and move this forward.
I am adding min_stake checks to our production release checklist here: https://github.com/orgs/masa-finance/projects/12/views/1?pane=issue&itemId=66418693
cc @Luka-Loncar


Hardware not supported - raspberry pi zero 2W

I confirm that this is an issue rather than a question.
Bug report, debug log and your config file (FULL LOGS ARE MANDATORY)
Full log: https://pastebin.com/hZAC77Rz
As additional information hyperion had the same issue and it seems to be fixed in the latest nightly (not tested by myself, HDR is a must for me): https://issueexplorer.com/issue/hyperion-project/hyperion.ng/1367
Steps to reproduce
Fresh install of HyperHDR 17.0 using WS2812b led strip and ws281x Controller type.
What is expected?
LED strip light up while "RGB order wizard" is running
What is actually happening?
Getting error: [LEDDEVICE] Device disabled, device 'ws281x' signals error: 'Failed to open. Error message: Hardware revision is not supported'
System
HyperHDR Server:

Build:           (HEAD detached at dec81c0) (Awawa-2a2ed8d/dec81c0-1631541363)
Build time:      Sep 15 2021 16:21:00
Git Remote:      https://github.com/awawa-dev/HyperHDR
Version:         <IP_ADDRESS>
UI Lang:         auto (BrowserLang: es-ES)
UI Access:       expert
Avail Capt:      Linux (V4L2)
Database:        read/write

HyperHDR Server OS:

Distribution:   Debian GNU/Linux 11 (bullseye)
Architecture:   arm64
CPU Type:       Raspberry Pi Zero 2 Rev 1.0
CPU Revision:   902120
CPU Hardware:   BCM2835
Kernel:         linux (5.10.63-v8+ (WS: 64))
Qt Version:     5.11.3
Browser:        Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/96.0.4664.93 Safari/537.36

Hi
First of all: as it's stated in the FAQ direct usage of LED strip from Raspberry Pi is not officially supported anymore so I don't consider it as HyperHDR issue but limitation of rpiws1x library model... need of rpiws821x developers to chase new models Rpi is just one of many reasons (other are more serious). I just leave it for backward compatibility, but will try to update rpiws821x library before next release. So please be patient.

Hi,
in case it helps anyone else, I built the current version in the master branch (version <IP_ADDRESS>beta1/c8049f1 hash) and it is working like a charm.
Thank you!


var_export in DB.php outputting string

When a DSN string is supplied and it contains parameters such as
"db_debug=TRUE", var_export will currently output the parsable string
representation of the variable. By setting the second parameter to TRUE,
the output is killed.

Another tip for your future contributions - you don't have to resubmit, just commit & push to the same branch.


Figure 8 & Figure 7 plot data

Hi @hassony2 ,
It would be great if you share plot data, for the fair comparison,
Thanks,

Hi @Ivamcoder,
I just pushed code which can regenerate the figures at https://github.com/hassony2/handobjectconsist/tree/master/figures.
Hope this helps !
Best,
Yana

Hi @hassony2 ,
It would be great if you share plot data, for the fair comparison,
Thanks,
Hi, I wonder how I could caluate the hand pose error mentioned in  Figure 5 and Figure 7. I did get the groundtruth joints 2D location in RGB image coordinate and joints 2D location reprojected by 3D joint location predicted. I couldn't find any such evaluation code in this repo.
Could u be kind to give any advice?
Thanks a lot!

Hi @hassony2 ,
It would be great if you share plot data, for the fair comparison,
Thanks,

Hi, I wonder how I could caluate the hand pose error mentioned in  Figure 5 and Figure 7. I did get the groundtruth joints 2D location in RGB image coordinate and joints 2D location reprojected by 3D joint location predicted. I couldn't find any such evaluation code in this repo.
Could u be kind to give any advice?
Thanks a lot!


TP-1020: ME58 should cover conditions without certificates as well

Why
"Our current implementation doesn't run the rule if there is no certificate, but actually, this rule should still apply. I.e. there should only be one measure condition with "no required certificate" per measure condition code."
What

Update ME58 to filter by kwargs set conditionally on whether a measure_condition has a required_certificate
Add case to business rule test

Codecov Report

Merging #449 (065ac4f) into master (9216bfd) will increase coverage by 0.01%.
The diff coverage is 100.00%.

:exclamation: Current head 065ac4f differs from pull request most recent head 7732b31. Consider uploading reports for the commit 7732b31 to get more accurate results

@@            Coverage Diff             @@
##           master     #449      +/-   ##
==========================================
+ Coverage   91.91%   91.93%   +0.01%
==========================================
  Files         284      284
  Lines       17322    17328       +6
  Branches     1421     1421
==========================================
+ Hits        15922    15930       +8
+ Misses       1129     1128       -1
+ Partials      271      270       -1

Impacted Files
Coverage Δ

measures/business_rules.py
89.25% <100.00%> (+0.61%)
:arrow_up:

measures/tests/test_business_rules.py
96.76% <100.00%> (+0.02%)
:arrow_up:

Continue to review full report at Codecov.

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


Plutip: test server interaction

We need to put more effort into testing, unlike in the past.
We want to have another (3rd) test suite (different from Integration) that tests interaction between plutip-server and CTL (both plutip-server endpoints).
Test request/response flow (most importantly, decoding of the datatypes).
We need two scenarios:

start server, stop server
start server, start server (decode the "server's already running" error)

Resulting PR should be based on one (or both) of these branches:
https://github.com/mlabs-haskell/plutip/pull/79 on plutip side
https://github.com/Plutonomicon/cardano-transaction-lib/pull/470 on CTL side

Done in:
51a3519 (#470)
mlabs-haskell/plutip@ab17dc2 (#79)


Add breakpoints using dbstop

I have been experimenting with a workflow which involves editing MATLAB files using vs code, then running them by calling the script through an open MATLAB command window (separate from vs code).
Suppose you have a MATLAB script. You can add a breakpoint by using dbstop, like so

By running the script through the MATLAB command window, I get

Which creates a debug point here

I feel like there could be functionality to add breakpoints this way (for now). Perhaps adding a breakpoint in vs code could automatically add the line dbstop at [line number add 1] in [current file]
I know there is work to integrate MATLAB debugging in vs code, but this could be a simple workaround for now.
Edit: Apparently, the keyboard command may work better in this case. I have never tried this, but it could be a preferable alternative to the clunky dbstop line. Still, something to bind the add/remove debug point shortcut to this command would be great.

I have marked this closed with the addition of debugging support in v1.3.0 of the extension. Please feel free to open a new issue if there is something which is not addressed.


Icon design theme color for visual studio 2022

A big job has been done in Visual Studio 2022 about the icons design. Please update it is icon color design 😁

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

ID: this page does not support. 😢
Version Independent ID:  this page does not support.😢
Content: Images and Icons for Visual Studio
Content Source: docs/extensibility/ux-guidelines/images-and-icons-for-visual-studio.md
Product: visual-studio-windows
Technology: vs-ide-sdk
GitHub Login: @leslierichardson95
Microsoft Alias: lerich

@qwqoq -- thank you for your suggestion.  Please add it here:

Suggest a feature, track product issues, and find answers in the Visual Studio Developer Community.

I am closing this issue now.  You are welcome to @ mention me for any followup.  We hope to hear from you again.


Travis ci

Add a travis config which build four cases:

rust nightly 2019-05-22
latest nightly

each combine with

git borsh/near-vm-logic
release borsh/near-vm-logic

Add a helper script actually run the test. This script also works locally.

@nearmax our way to compare compiler error message is very depending on specific version of rust compiler. I the following successful/failed build: nightly 2019-09-03 success, 2019-05-22 (nearcore) and latest nightly failed.
Unfortunately travis doesn't allow more than two nightly rust so I put 09-03 and latest nightly to make it pass.
https://travis-ci.com/nearprotocol/near-bindgen/builds/130112329
https://travis-ci.com/nearprotocol/near-bindgen/builds/130113538

Also latest nightly build fails for a different reason, those smart contract supposed to pass also failed.

Also latest nightly build fails for a different reason, those smart contract supposed to pass also failed.

I don't see from this link (2019-05-22 rust: https://travis-ci.com/nearprotocol/near-bindgen/builds/130112329 ) failing contracts. Also, the tests for each contract should be run with:
cargo test --features env_test

This is until we have
#29 fixed

rust 20190522 is still problematic (because of different compilation message. Try this locally:
echo "nightly-2019-05-22" > rust-toolchain
cargo test --features env_test

@nearmax fixed


Strategic merge with envelope charts does not work

Operating system
linux / mac
Helmfile Version
0.151.0
Helm Version
not that relevant
Bug description
Trying to apply kustomizations (strategicMergePatch) to my generated helm chart fails. I am trying to manipulate the gitlab chart, that itself is an envelope chart for several other charts.
Example helmfile.yaml
...

Steps to reproduce
try to apply a strategicMerge to nested charts
Working Helmfile Version
none
Relevant discussion
No response

@markussiebert sorry. I can't get where is your key issue.

Sorry - it's hard to describe I think - I updated the issue - do you understand my problem? Or do you need further information?


Issues when removing imported from Java Bote packets

Description
Apparently there is a difference in the name assignment when saving DHT data to files.
When reading the DHT key from the file name and converting it from base64 the names does not match.

Well, after some tests looks fixed now.


[BUG] Circular import of 'types'

Describe the bug
The types.py file e.g. in ppo interferes with the types module from the standard Python library in certain situations.
Running from the debugger result is the following stacktace:
Traceback (most recent call last):
  File "/usr/lib/python3.10/site.py", line 636, in <module>
    main()
  File "/usr/lib/python3.10/site.py", line 622, in main
    known_paths = addusersitepackages(known_paths)
  File "/usr/lib/python3.10/site.py", line 349, in addusersitepackages
    addsitedir(user_site, known_paths)
  File "/usr/lib/python3.10/site.py", line 232, in addsitedir
    addpackage(sitedir, name, known_paths)
  File "/usr/lib/python3.10/site.py", line 202, in addpackage
    import traceback
  File "/usr/lib/python3.10/traceback.py", line 5, in <module>
    import linecache
  File "/usr/lib/python3.10/linecache.py", line 8, in <module>
    import functools
  File "/usr/lib/python3.10/functools.py", line 22, in <module>
    from types import GenericAlias
  File "/home/mikey/PycharmProjects/Stoix/stoix/systems/ppo/types.py", line 1, in <module>
    from typing import Dict
  File "/usr/lib/python3.10/typing.py", line 25, in <module>
    import contextlib
  File "/usr/lib/python3.10/contextlib.py", line 6, in <module>
    from functools import wraps
ImportError: cannot import name 'wraps' from partially initialized module 'functools' (most likely due to a circular import) (/usr/lib/python3.10/functools.py)

We can see that functools is confusing the default python library types with the stoix types, by trying to import GenericAlias from the stoix types file.
To Reproduce
Steps to reproduce the behavior:

Run stoix/systems/ppo/ff_ppo.py using the PyCharm debugger

Context (Environment)
Same as https://github.com/EdanToledo/Stoix/issues/34
Possible Solution
I think the only solution is to rename the types.py files to something else - I've come across a similar situation before (see https://stackoverflow.com/questions/69007748/fatal-python-error-init-import-size-failed-to-import-the-site-module-in-anacon/72678670#72678670)

Thanks so much for bringing this up! I've seen this before as well but I wonder why this isn't happening to me? regardless i think its probably worthwhile changing the file name anyway to avoid this issue popping up here and there. I'll make a quick change and push it.

I've just merged a fix that should solve this, let me know if it works for you.

Fixed, thanks again!


Added support link and hotfix package link

as per the user report  #5732  i added the  download link for Microsoft Desktop Optimization Pack March 2017 Servicing Release

I may have gone somewhat overboard with my suggestions, but I presume @JohanFreelancer9 will be able to suggest less radical changes.

Very well. I will wait for a copy review by @JohanFreelancer9 & @nenonix , just in case they see that this added content needs improvement.

@FMCUSystemAdmins : How do you feel about the added content? Any other suggestions or recommendations?

Thank you, those changes look appropriate and well considered.
@RAJU2529 : Please commit the suggested changes.

@illfated . Committed changes done.

Looks good! Thank you for improving this. It helps the flow of troubleshooting 😊.
From: Trond B. Krokli<EMAIL_ADDRESS>Sent: Saturday, December 21, 2019 7:46 PM
To: MicrosoftDocs/windows-itpro-docs<EMAIL_ADDRESS>Cc: System Administrators<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/windows-itpro-docs] Added support link and hotfix package link (#5735)
@FMCUSystemAdminshttps://github.com/FMCUSystemAdmins : How do you feel about the added content? Any other suggestions or recommendations?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/windows-itpro-docs/pull/5735?email_source=notifications&email_token=AMA556MPVE44ZRNFTNEG4YDQZ3BFPA5CNFSM4J6H6FZKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEHPHADA#issuecomment-568225804, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AMA556N6UVJLDQHHSAR6B5LQZ3BFPANCNFSM4J6H6FZA.


Wrong language layout of physical keyboard in Game Mode

Describe the bug
I prefer using azerty instead of qwerty.
In desktop mode I have the correct layout for my physical keyboard, in game mode I have the correct layout for the virtual keyboard.
But I don't have the correct layout in game mode for my physical keyboard.
What did you expect to happen?
I expected in game mode that my physical keyboard to have the azerty layout like the virtual keyboard and like the physical keyboard in desktop mode.
Output of rpm-ostree status
No response
Hardware
I'm using a home theater pc
Extra information or context
No response

You can Change it in settings-keyboard

lm209

It changes only the virtual Keyboard

lm209

It only modifies the virtual keyboard

Sry i readed it wrong. You can try to Add your Keyboard Layout in the Desktop Mode under settings-keyboard. There is a Tab for Layouts. Make Sure that your Layout is selected.

this is an upstream gamescope issue, gamescope defaults to US keyboard and no way to change it.
you might have luck if you set the environment variable XKB_DEFAULT_LAYOUT=no replacing no with the correct layout for you.
you can add this to /etc/environment but i would recommend testing this in desktop mode first before applying it to the whole system.
This works on desktop mode for things running in nested gamescope, but it has its own quirks (like altgr+2 to write @ on the norwegian layout will still not work, but the basic keyboard layout works as long as i do not need to use altgr which in normal typing you will not need it, at least for the norwegian layout)
not sure if the same thing will work for gamescope-session though

this is an upstream gamescope issue, gamescope defaults to US keyboard and no way to change it. you might have luck if you set the environment variable XKB_DEFAULT_LAYOUT=no replacing no with the correct layout for you. you can add this to /etc/environment but i would recommend testing this in desktop mode first before applying it to the whole system.
This works on desktop mode for things running in nested gamescope, but it has its own quirks (like altgr+2 to write @ on the norwegian layout will still not work, but the basic keyboard layout works as long as i do not need to use altgr which in normal typing you will not need it, at least for the norwegian layout) not sure if the same thing will work for gamescope-session though

THANK YOU A LOT!
It worked!
And as a bonus, altgr works for the french layout

And as a bonus, altgr works for the french layout
lucky, im jealous 😂


Updated version specific package instruction First Web API

Fixes #29303
Updating versions 3-6 to their version specific packages available at the time for VS Code instruction.
Updating version 7 to use the default latest for packages for VS Code.
Testing both Windows and MacOS for each.

Internal previews

📄 File
🔗 Preview link

aspnetcore/tutorials/first-web-api.md
Tutorial: Create a web API with ASP.NET Core

Close draft.  Will reopen when I can get back to this.


simple-dnscrypt: add version 0.7.1

Created simple-dnscrypt.json

[x] I have read the Contributing Guide.

/verify

/verify

/verify

/verify

/verify

/verify

/verify

/verify

I think we can just name the package as simple-dnscrypt. Same as the window title.

I think we can just name the package as simple-dnscrypt. Same as the window title.

That works for me. I think that name will be easier to remember.

/verify

Thanks @rasa

I went to delete my version, and found
https://github.com/rasa/scoops/blob/master/scripts/simple-dnscrypt/simple-dnscrypt.ps1
which this version lacks. My apologies for not pointing this out sooner,

@rasa I believe you can easily disable the built-in auto update through the settings.

I know, but I think generally scoop disables auto-update logic. If we are going to leave it in, we may want to alert the user that it's on, and how to disable it.

Should I add a notes section to the manifest that tells the user how to disable built-in auto update?
Or could you please add the simple-dnscrypt script to the script folder of this bucket?


Bug in =schedule command interaction

There's a bug when pressing the buttons in =schedule command, i don't really know why this is happening, but when i tried to run it in my laptop rather than the Heroku server, the buttons works perfectly fine.

Fixed in the latest commit


[INFRA] Update gatsby-plugin-styled-components to 5.24.0

Before to update Gastby to v5, we update all its plugins on their last v4 version.
Depends on #737

Done with #740


Inherit .gitignore

Prettier inherits .gitignore where as parrallel-prettier doesn't
I'm happy to make a PR but how do you want to achieve it? allow multiple ignore paths? include it if it exists?

@lukeapage Prettier does not inherit .gitignore, see https://prettier.io/docs/en/ignore.html
Is this the cause for your issues? https://github.com/microsoft/parallel-prettier/issues/25

The docs you pointed at say this:

Prettier will also follow rules specified in the ".gitignore" file if it exists in the same directory from which it is run.

@lukeapage to be fair, that page was updated one month ago: https://github.com/prettier/prettier/commit/9ebcf9b0ffdf568d884f748510fd6b730dd28010
When this issue was written Prettier did not follow .gitignore.
I think https://github.com/microsoft/parallel-prettier/issues/23 is needed first then, and maybe it will/would be done as part of this, but I also believe it is good to keep this issue open then.


User uniqueness not required

Currently, the user model does not validate for uniqueness by email address.  We should only have one user account per email address.

closed as dupicate to issue #80


feat(rows): wrapper for pre applying directives

Introduced DatatableRowDefComponent which can be used by consumers to pre apply certain directives on each row.
Usage:
<ngx-datatable>
  <ng-template rowDef>
    <datatable-row-def cdkDrag />
  </ng-template>
</ngx-datatable>

What kind of change does this PR introduce? (check one with "x")

[ ] Bugfix
[x] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Other... Please describe:

What is the current behavior? (You can also link to an open issue here)
What is the new behavior?
Does this PR introduce a breaking change? (check one with "x")

[ ] Yes
[x] No

If this PR contains a breaking change, please describe the impact and migration path for existing applications: ...
Other information:

@spike-rabbit This is a proposal for allowing consumers to pre apply directives (e.g cdk drag drop)  at row levels. Please have a look and provide your feedback.

Many thanks. May you provide an example for it. Maybe add the CDK to the demo and support drag and drop.

Please provide an example.

@timowolf I have added an example to show the CDK drag drop usage.


Add cio.Load for loading io set

This adds a Load Opt for cio to load a tasks io/fifos without
attaching or starting the copy routines.
It adds the load method in ctr by default so that fifos or other IO
are removed from disk on delete methods inbetween command runs.  It is
not the default for all task loads for backwards compat. and a user may
want to keep io around to reuse or if log files are used.
Fixes #2421
Signed-off-by: Michael Crosby<EMAIL_ADDRESS>

Codecov Report

Merging #2422 into master will decrease coverage by 0.02%.
The diff coverage is 0%.

@@            Coverage Diff            @@
##           master   #2422      +/-   ##
=========================================
- Coverage   45.03%     45%   -0.03%
=========================================
  Files          92      92
  Lines        9407    9412       +5
=========================================
  Hits         4236    4236
- Misses       4488    4493       +5
  Partials      683     683

Flag
Coverage Δ

#linux
49.23% <0%> (-0.04%)
:arrow_down:

#windows
41.3% <ø> (ø)
:arrow_up:

Impacted Files
Coverage Δ

cio/io.go
33.33% <0%> (-2.11%)
:arrow_down:

Continue to review full report at Codecov.

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

LGTM


Server Jobs: are there any in the marketplace?

I am looking for a server job that lets me trigger and wait for pipelines to finish.
It seems to be a waste of ressources to block an agent just to trigger a build using the "Trigger Build Task" job.
I can not find a server job that does the task because I can't tell if a job in the marketplace requires an agent or not. (there are no filters)
I turned to the documentation in order to find out, if there are server jobs in the marketplace at all.

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

ID: 67504b34-d64b-02a4-2e10-ab99f3b8cfe4
Version Independent ID: 2cf63b2e-184b-7726-3b8a-d8baffd6fcce
Content: Jobs in Azure Pipelines, Azure DevOps Server, and TFS - Azure Pipelines
Content Source: docs/pipelines/process/phases.md
Product: devops
Technology: devops-cicd
GitHub Login: @vijayma
Microsoft Alias: vijayma

Thanks for your question. It looks like your question isn't an issue with the documentation, but an issue or question specific to your implementation or a bug in the product itself.
For help with a specific task, here are a couple of options where you might consider asking your question:

Azure DevOps on Stack Overflow
Azure DevOps Support Bot

If you've encountered an issue with the product itself, to make sure it gets in front of the right people, please submit your bug here.
If you have an idea for improving the product, please submit your idea here.