Public Hostname is first public address or first reachable private

Public Hostname to be first public address or first reachable private address

I presume that the problem you're hitting is that there is a VM being provisioned that gets multiple (private) IPs, but that the corresponding entity advertises its sensor for "host.name" with the wrong IP.
I think we should look more at the call-chain for where we're calling getPublicHostname() from, and figure out if there is a better way of doing this. For example, if we're already checked reachability to decide on an IP then we could prefer that (rather than repeating the check).
Also, the semantics of subnet hostname and public hostname are unclear when dealing with private networks: it's unclear whether the IP should be the one reachable from Brooklyn. I guess if we're talking "public" then that's probably as good an indication as any.

The jenkins builds failure is probably unrelated - I've tried to fix it in https://github.com/apache/brooklyn-server/pull/357.

Code looks more sensible this way and will be very easy to isolate parts of it related with choosing the IP.
However testing those is best to be done live with the current code.
I think any further refactoring could lead to mistakes.
Thoughts?

@bostko can you say specifically what you're trying to fix here?

I am fixing
                machineLocation = registerWinRmMachineLocation(computeService, node, managementHostAndPort, sshHostAndPortOverride, setup);
                machineLocation = registerJcloudsSshMachineLocation(computeService, node, Optional.fromNullable(template), userCredentials, managementHostAndPort, sshHostAndPortOverride, vmHostname, setup);

to use the initial address.
I also trying to make choosing managementHostAndPort code simpler and easier to maintain.
@grkvlt @neykov could you also take a look at the docker part related to this PR?

This has caused JcloudsByonLocationResolverStubbedRebindTest.testRebind to be very slow (5 minutes). It is trying to find the first reachable IP (with a 5 minute timeout). After the 5 minutes, it does do everything else fast and then the test passes! This isn't ideal: we won't just stop early if the VM you are asking for is not reachable (e.g. is not running).
"main" prio=5 tid=0x00007fe6c1800000 nid=0x1703 waiting on condition [0x0000700000217000]
   java.lang.Thread.State: TIMED_WAITING (parking)
        at sun.misc.Unsafe.park(Native Method)
        - parking to wait for  <0x00000007f0047248> (a java.util.concurrent.CountDownLatch$Sync)
        at java.util.concurrent.locks.LockSupport.parkNanos(LockSupport.java:226)
        at java.util.concurrent.locks.AbstractQueuedSynchronizer.doAcquireSharedNanos(AbstractQueuedSynchronizer.java:1033)
        at java.util.concurrent.locks.AbstractQueuedSynchronizer.tryAcquireSharedNanos(AbstractQueuedSynchronizer.java:1326)
        at java.util.concurrent.CountDownLatch.await(CountDownLatch.java:282)
        at org.apache.brooklyn.util.net.ReachableSocketFinder.tryReachable(ReachableSocketFinder.java:141)
        at org.apache.brooklyn.util.net.ReachableSocketFinder.access$000(ReachableSocketFinder.java:50)
        at org.apache.brooklyn.util.net.ReachableSocketFinder$2.call(ReachableSocketFinder.java:93)
        at org.apache.brooklyn.util.net.ReachableSocketFinder$2.call(ReachableSocketFinder.java:91)
        at org.apache.brooklyn.util.repeat.Repeater.runKeepingError(Repeater.java:336)
        at org.apache.brooklyn.util.repeat.Repeater.run(Repeater.java:302)
        at org.apache.brooklyn.util.net.ReachableSocketFinder.findOpenSocketOnNode(ReachableSocketFinder.java:88)
        at org.apache.brooklyn.location.jclouds.JcloudsUtil.getFirstReachableAddress(JcloudsUtil.java:374)
        at org.apache.brooklyn.location.jclouds.JcloudsLocation.getFirstReachableAddress(JcloudsLocation.java:2998)
        at org.apache.brooklyn.location.jclouds.JcloudsLocation.resolveManagementHostAndPort(JcloudsLocation.java:2031)
        at org.apache.brooklyn.location.jclouds.JcloudsLocation.registerMachineLocation(JcloudsLocation.java:2378)
        at org.apache.brooklyn.location.jclouds.JcloudsLocation.registerMachine(JcloudsLocation.java:2371)
        at org.apache.brooklyn.location.jclouds.JcloudsByonLocationResolver$1.get(JcloudsByonLocationResolver.java:158)
        at org.apache.brooklyn.location.jclouds.JcloudsByonLocationResolver$1.get(JcloudsByonLocationResolver.java:1)
        at org.apache.brooklyn.location.byon.FixedListMachineProvisioningLocation.init(FixedListMachineProvisioningLocation.java:173)
        at org.apache.brooklyn.core.objs.proxy.InternalLocationFactory.createLocation(InternalLocationFactory.java:137)
        at org.apache.brooklyn.core.mgmt.internal.LocalLocationManager.createLocation(LocalLocationManager.java:94)
        at org.apache.brooklyn.core.location.BasicLocationRegistry.getLocationManaged(BasicLocationRegistry.java:336)
        at org.apache.brooklyn.location.jclouds.JcloudsByonLocationResolverStubbedRebindTest.getLocationManaged(JcloudsByonLocationResolverStubbedRebindTest.java:154)
        at org.apache.brooklyn.location.jclouds.JcloudsByonLocationResolverStubbedRebindTest.testRebind(JcloudsByonLocationResolverStubbedRebindTest.java:141)
        at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
        at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:57)
        at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
        at java.lang.reflect.Method.invoke(Method.java:606)
        at org.testng.internal.MethodInvocationHelper.invokeMethod(MethodInvocationHelper.java:84)
        at org.testng.internal.Invoker.invokeMethod(Invoker.java:714)
        at org.testng.internal.Invoker.invokeTestMethod(Invoker.java:901)
        at org.testng.internal.Invoker.invokeTestMethods(Invoker.java:1231)
        at org.testng.internal.TestMethodWorker.invokeTestMethods(TestMethodWorker.java:127)
        at org.testng.internal.TestMethodWorker.run(TestMethodWorker.java:111)
        at org.testng.TestRunner.privateRun(TestRunner.java:767)
        at org.testng.TestRunner.run(TestRunner.java:617)
        at org.testng.SuiteRunner.runTest(SuiteRunner.java:348)
        at org.testng.SuiteRunner.runSequentially(SuiteRunner.java:343)
        at org.testng.SuiteRunner.privateRun(SuiteRunner.java:305)
        at org.testng.SuiteRunner.run(SuiteRunner.java:254)
        at org.testng.SuiteRunnerWorker.runSuite(SuiteRunnerWorker.java:52)
        at org.testng.SuiteRunnerWorker.run(SuiteRunnerWorker.java:86)
        at org.testng.TestNG.runSuitesSequentially(TestNG.java:1224)
        at org.testng.TestNG.runSuitesLocally(TestNG.java:1149)
        at org.testng.TestNG.run(TestNG.java:1057)
        at org.testng.remote.AbstractRemoteTestNG.run(AbstractRemoteTestNG.java:126)
        at org.testng.remote.RemoteTestNG.initAndRun(RemoteTestNG.java:152)
        at org.testng.remote.RemoteTestNG.main(RemoteTestNG.java:57)

Merging this now - it's better to have the fix for the provisioning case (and live with the 5 mins delay for jcloudsByon when the VM is unreachable), than to delay this PR longer.


Autoselect query block for execution

Hi, thanks for creating such helpful tool.
One of the features I'd like to see is the ability to auto select query block to execute.
Example:
SELECT something FROM table
LEFT JOIN table2 ON ....

UPDATE table SET xx=1 WHERE id=1

Noticed above we have 2 queries but there's a few empty lines in between.
When the cursor is on SELECT and press Cmd+Enter, it should run the SELECT query only, not all the queries.
This will improve our productivity.
Thanks in advance.

Hey, thanks for the feedback! I also found myself wanting something like this so there's a big chance a feature like it is going to make it into the next release.

Should be available in the PR linked above.

This was implemented in 0.11.11 https://github.com/sosedoff/pgweb/blob/master/CHANGELOG.md#01111---2022-03-29


Bad html layout in mobile browser

Umm...self-explainatory?

Presentation for mobile browser has been improved a little in the latest version.


Add multi-evaluation support

If I read the protocol correctly, one signer could sign numerous messages simultaneously, like in BLS, using the same alpha and beta, and the same point not involving the hash Z, but supplying numerous π_2 and π_4 points.  All messages could then be verified simultaneously by using Z = Σ_i Z_i and π_2 = Σ_i π_{2,i} and π_4 = Σ_i π_{4,i}.  In this variant, adversaries could forge signatures on linear combinations of Z, and similarly malicious signers could play games, but this should yield nothing useful.
This looks useful because users could pay to query the threshold VRF alongside the randomness beacon.
Ideally one wants slightly more than this:  Users should query for secret evaluations.  We could implement secrecy by users providing a SNARK on BW6 that secretly evaluates the hash-to-curve on BLS12-377 and multiplies by a blinding scalar.  Again this should yield security for this scheme.
I'm unsure if this make sense as some brain wallet recovery method, but it should definitely makes sense for public rendezvous protocols, like SecureDrop or the Panda protocol in Pond.

Hey Jeff,
Yep, that all sounds about right, also discussed this briefly with @mmaller.
IMO the analogy to what you can do in BLS and what you can do here is apt.
I'm out for the next few weeks, and I would be interested in following up on this when I get back!

It'd yield some roll up for your randomness beacon too, which sounds more immediately useful.

In this vein, one could similarly do threshold decryption by replacing Z by a sender controlled scalar multiple of G_1, but then senders much include a proof-of-knowledge of Z, and you've a nice partially batched proof of correct decryption too.  It's slightly inferior to straight IBE-like designs like https://eprint.iacr.org/2020/638 or scalar threshold IBE itself, but still fairly nice.


Implement replacement of existing readers/writers

The behaviour of calling add_reader/add_writer on the same fd without removing the reader/writer before is currently quite erratic. It is seemingly undefined which reader/writer gets called, the old or the new, as the previous notifier is not disabled and disconnected properly.
Note: actually I would expect that garbage collection of the notifier is sufficient; this does not seem to be the case. It is certainly possible though that garbage collection does not happen in time, so in any case it is safer to disable and disconnect it beforehands.
Note: only disabling was not sufficient, for unknown reasons.
The new behaviour conforms to PEP 3156, Section "I/O-Callbacks":

add_reader(fd, callback, *args). Arrange for callback(*args) to be called whenever file descriptor fd is deemed ready for reading. Calling add_reader() again for the same file descriptor implies a call to remove_reader() for the same file descriptor.

Analogous to double remove of a reader/writer, this still issues a warning. I am working on another pull request to make remove these warnings and make remove_reader/remove_writer conform to PEP 3156, too (by returning True/False, as mandated by the PEP).

If there are any more code style issues with my patches, please let me know and fix them myself, so that I learn how to correctly indent tabbed python :)

If you have PySide installed globally and tox installed (either globally or in a virtualenv) you can do: tox -e flake8 to run the automatic style-checker with this project's custom settings. It would be nice to get Travis to run these checks.

@harvimt Thanks for the pointer. While I did not manage to get tox to work (PySide installation turned out non-trivial), running flake8 revealed some issues which are now fixed in both branches.

flake8 w/o the settings in tox will just not do variable naming convention checks and PEP 257 docstring checks.


Jenkins tutorial failing due to maven 3.8.1 and Tests not passing

Hi.
So I'm learning a bit of Jenkins and found 2 problems when using this Git project following the tutorial for Building a Java app and Maven with Jenkins.
I'm using Windows running Jenkins in Docker with Linux containers.
1st problem, it cannot Build the project because Jenkinsfile suggested by documentation or tutorial uses maven:3.8.1-adoptopenjdk-11 but enforcer complains it should be at least 3.8.6, so I had to manually change it in Jenkinsfile to maven:3.8.6-openjdk-18 for this  to build.
2nd problem, it cannot pass Tests. It always says "message" is not "Hello World!".
I tried everything I barely know, saving string to variables, using compareTo(), using equals(), checking the strings with println... no matter what I tried it always returns erro and I cannot finish the tutorial. I printed AssertionError e message and it shows this trace:
-------------------------------------------------------
 T E S T S
-------------------------------------------------------

Running com.mycompany.app.AppTest
Hello World!
org.opentest4j.AssertionFailedError: expected: <Hello World!
> but was: <>
Tests run: 2, Failures: 1, Errors: 0, Skipped: 0, Time elapsed: 0.017 sec <<< FAILURE!
com.mycompany.app.AppTest.testAppMain()  Time elapsed: 0.015 sec  <<< FAILURE!
org.opentest4j.AssertionFailedError: "message" is not "Hello World!"
	at org.junit.jupiter.api.AssertionUtils.fail(AssertionUtils.java:38)
	at org.junit.jupiter.api.Assertions.fail(Assertions.java:135)
	at com.mycompany.app.AppTest.testAppMain(AppTest.java:43)

Results :

Failed tests:   com.mycompany.app.AppTest.testAppMain(): "message" is not "Hello World!"

Tests run: 2, Failures: 1, Errors: 0, Skipped: 0

[INFO] ------------------------------------------------------------------------
[INFO] BUILD FAILURE
[INFO] ------------------------------------------------------------------------
[INFO] Total time:  1.572 s
[INFO] Finished at: 2022-12-19T11:56:12Z
[INFO] ------------------------------------------------------------------------
[ERROR] Failed to execute goal org.apache.maven.plugins:maven-surefire-plugin:2.12.4:test (default-test) on project my-app: There are test failures.
[ERROR]
[ERROR] Please refer to /var/jenkins_home/workspace/simple-java-maven-app/target/surefire-reports for the individual test results.

I'm out of ideas. I'm very limited at Java, Jenkins and so and I cannot understand why an official tutorial does not work by default, but no documentation is provided to workaround any errors. I'm very frustrated.
I saw this project was updated 2 days ago. Was it not tested? Was it tested following the Jenkins tutorial? Using Windows?

Tutorial followed:
https://www.jenkins.io/doc/tutorials/build-a-java-app-with-maven
Using Windows

Also fixed the test in the source code and significantly simplified the test so that it would run on Java 11.

Just to add to this I reviewed the code on GitHub and found this commit on pom.xml "[Mak this work on Java 17 onwards]" and inside the xml file it only supports java 11
so I downloaded Java 11 open jdk and tried it again and it works fine without any errors. It is not compatible with other versions as of now.  Should have noticed it earlier but thanks again.
@ricardogarceb @MarkEWaite

so I downloaded Java 11 open jdk and tried it again and it works fine without any errors. It is not compatible with other versions as of now. Should have noticed it earlier but thanks again.

I cloned https://github.com/jenkins-docs/simple-java-maven-app.git and built it with Java 17.0.5 and Apache Maven 3.8.6.  The pom.xml file defines that it needs Java 11 or newer.  The Jenkins project supports Java 11 and Java 17.  As far as I can tell, this sample Java application compiles with both Java 11 and Java 17.


Update Cache class and add new-fasion methods into script.py

Coverage remained the same at 0.0% when pulling 65c36274eeb402e105237533995182ef4aecd814 on FromZeus:master into 80928889a624e67723744590450b28ad27572ac3 on seecloud:master.

Coverage remained the same at 0.0% when pulling e3375e4faf1acb80c0b311d697b42259e5e4b615 on FromZeus:master into 80928889a624e67723744590450b28ad27572ac3 on seecloud:master.


ion-toggle buggy in ios mode on Windows platform

Ionic version:  (check one with "x")
[ ] 1.x
[ ] 2.x
[X] 3.0.1
I'm submitting a ...  (check one with "x")
[X] bug report
[ ] feature request
[ ] support request => Please do not submit support requests here, use one of these channels: https://forum.ionicframework.com/ or http://ionicworldwide.herokuapp.com/
Current behavior:
Actual conf-app lazy-load, set config mode to "ios". Run on Windows platform (ionic run windows --prod)
Open "Filter Sessions", click on ion-toggle buttons.

toggle control is broken and does not enter "off" state correctly. Rendering problem.
"Reset all Filters" doesn't affect all items. In fact they are logically reset, but visual state lacks.

Expected behavior:
Correct rendering/behavior of toggle-control in "ios" mode on Windows platform.
Steps to reproduce:
Actual conf-app lazy-load, set config mode to "ios". Run on Windows platform (ionic run windows --prod)
Ionic info: (run ionic info from a terminal/cmd prompt and paste output below):
 ordova CLI: 6.5.0
Ionic Framework Version: 3.0.1
Ionic CLI Version: 2.2.2
Ionic App Lib Version: 2.2.1
Ionic App Scripts Version: 1.2.4
ios-deploy version: Not installed
ios-sim version: Not installed
OS: Windows 10
Node Version: v6.9.1
Xcode version: Not installed

Maybe #10872 is related.

@manucorporat I hoped that your c792ab69ae0401a7426527c57823c4fc0dd9d94a would fix this, but it didn't. Misbehavior is still present in latest nightly as of today.


Basic performance test suite

Fixes #1119
This PR adds an initial performance test that runs iperf between two hosts. It generates a iperf log that is stored in test/performance/*.log. Performance tests can be run with the rest of the integration test by doing a PERFORMANCE=1 ./run_all.sh
The underlying objective of this PR is to determine how to deal with performance tests: to determine what we really need, what to do with the results, etc...
How it works:

Performance tests generate *.csv files that are archieved with the rest of the artifacts.
A Python script tries to get the *.csv files from the last n builds for each performance tests from CircleCI.
The Python script generates a peformance graph with the *.csvs
In contrast with other CI systems like Jenkins, CircleCI cannot provide a constant URL for accessing the latest build (and its artifacts), so we have to store the performance graph in an external service. In this case, perfomance graphs are plotted with the help of Plotly and stored here

TODO:

[x] Include the performance tests in the CircleCI builds
[x] Archive the artifacts (test/performance/*.log)
[x] Generate some graphs and link them from some place in our docs
[ ] Create a weaveworks user in Plotly for generating performance graphs. Remove @inercia's credentials (will do this once this is merged in master)
[ ] Skip branches other than master (will do this once this is merged in master)

@rade @tomwilkie any suggestion on what we could do with the performance tests results? should I generate a graph and upload it to somewhere?

You could stick the raw data it in the circle artifacts (see last few lines of circle.yml, this is what we do for the coverage data)
For extra points you could post it to an appengine app that plots results over time...

@tomwilkie Would you please take a look at this PR?

On hold until #1437 is fixed...

On hold until #1437 is fixed...

why? It's annoying to have that failure, but it shouldn't stop us from getting work done.

Because the only remaining thing is to get the CircleCI build fixed

Because the only remaining thing is to get the CircleCI build fixed

I see. That's not what the TODO says.

I'd really like to see this merged-- since I  use some pretty unusual topologies that include users, datacenters, mobile devices, and stuff we shall just call "other" having a way to test it all would be great-- in fact, I've got to say that this seems like an easy integration candidate for Weave.scope-- imagine the pretty pictures!

@tomwilkie does it make sense to rework this to use the new BI stuff instead of plotly?

Could do, but bi.weave.works isn't public, and this could be.

Closing due to inactivity


Add support for screen sharing

Now that browsers have standardized around getDisplayMedia(), it is time to add screen sharing.
It doesn't seem like open-easyrtc has support for the proper apis yet so they will need to be patched in. It may be non-trivial to support sending audio at the same time as the screen sharing because some browsers don't allow for capturing both with a single stream.
Think it makes sense to either show a computer button next to the webcam/mic buttons like this one
https://www.svgrepo.com/svg/1558/computer-monitor-plain . Then toggle the webcam when screen sharing mode is entered/exited.

https://github.com/open-easyrtc/open-easyrtc/issues/42#issuecomment-644727155


Fix log file with 0 byte offset from end

change for #9482
shouldn't use truthy/falsy values for this check - data.fileData = '' is still valid for a real file

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/4614/
Test FAILed.

@calvinjia PTAL, thanks

jenkins, test this please.

@tieujason330 In the commit message, please use Fixes #... as opposed to change for. This will automatically close the issue after the PR is merged.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/4620/
Test FAILed.

@calvinjia - good to know about the commit message, updated it. I agree with reusing the logic, didn't want to do it as part of the fix

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/4622/
Test FAILed.

Jenkins, test this please.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/4630/
Test PASSed.

alluxio-bot, merge this please.


Could you add a tag?

First of all thanks for the great work 👍
I'll use go-gitignore in one of my projects and would be nice if the project follows semver tagging style.

absolutely can do :) give me a few minutes - sorry I did not see this till today.

Done - ill add the ability for tag management via github/sabhiram/gover soon - for now this should be good.

Thank you @sabhiram 👍


Feature Request: Open with Media Player

It would be cool if there was a button that opens the Stream with a Media Player such as VLC or MPV.

Hey @0lhi,
thanks for creating this feature request! Could you elaborate, do you think this button should be placed in the popup, or inserted into the page of the owncast instance itself?

Inside the Pop-Up:

Would give quicker access to the Stream
Allows lurkers to watch without being called out in the Chat

An additional Button on the Webpage would be cool, but not what I'm requesting.

Inside the Pop-Up:

Would give quicker access to the Stream
Allows lurkers to watch without being called out in the Chat

An additional Button on the Webpage would be cool, but not what I'm requesting.


ENH: cupyx/scipy/signal: add missing LTI format conversions

cross-ref gh-7403

[x] tf2zpk
[x] tf2sos
[x] tf2ss
[x] ss2tf
[x] zpk2ss
[x] ss2zpk
[x] sos2tf
[x] sos2zpk
[x] BadCoefficients

Note that there are two utilities which delegate to NumPy on CPU, _polyutils.roots and .poly. Both rely on np.linalg.eigvals, which on GPU is only available from MAGMA.
this is on top of gh-7644

Rebased on main, this PR is ready for review.

/test full

/test full

Comments addressed.

/test full

Thanks, @ev-br!

Thank you for the reviews! (this PR and others)


Update supported scenarios for portable PDBs

This documentation seems to be out of date: https://github.com/dotnet/core/blob/master/Documentation/diagnostics/portable_pdb.md#supported-scenarios
It states that "Symbol server (ex: SymbolsSource.org)" do not support portable PDBs. This should be updated as nuget.org's symbol server (https://symbols.nuget.org/download/symbols) does support portable PDBs. For more information, please refer to: https://blog.nuget.org/20181116/Improved-debugging-experience-with-the-NuGet-org-symbol-server-and-snupkg.html
@karann-msft @tmat

@noahfalk

Thanks! Does this update look good?

LGTM


Wait for all operations to finish.

This pull request caters #30 and indirectly caters a issue which very rarely seen in some consuming tools like https://github.com/Azure/vscode-aks-tools/issues/44
Just like collectors we have now added wait logic for the zip and export to storage functionality.
Happy to detail more if anyone keen, but I think the linked issues will help in understanding the idea behind this fix. Gist is that the operation of upload still happening at the background and by adding wait we are guaranteeing that this tool will return 100% success upload before it finishes, earlier the tool runs at the background due to select {} which leads to this kind of behaviour, as detailed: https://github.com/Azure/vscode-aks-tools/issues/44  its similar behaviour in az-cli as well.

test image: docker.io/tatsat/wait-for-all-operations

Thanks heaps,
cc: @rzhang628 , @arnaud-tincelin , @peterbom , @sophsoph321 , @bcho 🙏☕❤

Changes look good for arc scenarios

Closing this PR in favour of #138 which fixes few things. Thanks, 🙏


change coverage to codecov

Codecov Report

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

@@           Coverage Diff           @@
##           master      #40   +/-   ##
=======================================
  Coverage   75.76%   75.76%
=======================================
  Files          85       85
  Lines       13178    13178
=======================================
  Hits         9984     9984
  Misses       2391     2391
  Partials      803      803

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 304afd0...2abaef5. Read the comment docs.


cleanup(auth): minor code simplification

Thanks, I have yet to acquire good taste in this language.

I am not sure I have good taste, in this language or any other.  It just seems to me that -- in general -- using standard library features to express something should be more readable. Not because the standard library(ies) are perfect, but because they are well-known.


store data isn't erase

On logout button i use box.erase() but i restart app store data in not clear i use only one initial container  and erase it but this is not working
Please tell me solution

Having the same issue

Having the same issue

Having the same issue

same issue

same issue,,, any update?

I have facing the same issues Some time get_storage not cleared all stored data.

Same here. Clearing the local storage works for that instance.
Once you restart the simulator, the value is back especially the token. Some other time, it returns the previously logged in user.


[Bug]: ==Italic-highlight== gets paler in Preview/PDF and turns the color e.g. red to brownish

Bug Description
In one of the newer updates, the theme now turns *==italic-highlighted==* words slightly paler in Preview mode and in PDFs which makes the red (that was once default and is now my default) sort of brownish.
This doesn't happen when I switch to the default theme, but the paling does also occur when I turn off that snippet.
Relevant Screenshot

Live Preview mode

Preview mode
To Reproduce

word highlighting
switch to preview mode
word that is highlighted and italic gets paler

Obsidian Version
0.14.15
Installer Version
0.14.6
Font Size
16
Which editor are you using?

[X] New Editor
[ ] Legacy Editor

In which mode(s) does the problem occur? (if applicable)

[X] Source (Edit)
[X] Reading (Preview)
[X] Live Preview

Which operating system?

[ ] macOS
[X] Linux
[ ] Windows
[ ] iOS
[ ] Android

Plugin-Specific Problems

[ ] I took note of the specific instructions regarding those plugins.

Checklist

[X] I updated to the latest version of the theme (Settings → Appearance → Themes - Manage → Update).
[X] I made sure that the issue is related to this theme by checking that the problem does not occur with the default theme.
[X] In case I am missing an UI element, I have checked that it cannot be re-enabled with the Style Settings plugin.
[X] If the bug occured after updating the theme, I read the last couple of entries in the changelog to make sure the bug is not an intentional change. (In that case, there will most likely be a style setting to turn it off again.)

oh sorry, I didn't forgot to consider italics that are used for multi-color highlights. Is fixed.

Awesome, thanks!


[Android] StatusBar jumps whole screen when hidden

Was coming across this issue: https://github.com/facebook/react-native/issues/24417 which was closed because the reporter didn't care to post his environment info. Just copy-pasting from that issue into here
React Native version: 0.60  (using a patched version with V8 as js engine)
System:
    OS: macOS Mojave 10.14.6
    CPU: (8) x64 Intel(R) Core(TM) i7-7700HQ CPU @ 2.80GHz
    Memory: 23.32 MB / 16.00 GB
    Shell: 5.3 - /bin/zsh
  Binaries:
    Node: 10.12.0 - /usr/local/bin/node
    Yarn: 1.10.1 - /usr/local/bin/yarn
    npm: 6.11.3 - /usr/local/bin/npm
    Watchman: 4.9.0 - /usr/local/bin/watchman
  SDKs:
    iOS SDK:
      Platforms: iOS 13.1, DriverKit 19.0, macOS 10.15, tvOS 13.0, watchOS 6.0
    Android SDK:
      API Levels: 28, 29
      Build Tools: 28.0.3, 29.0.2
      System Images: android-29 | Google APIs Intel x86 Atom, android-29 | Google Play Intel x86 Atom
  IDEs:
    Android Studio: 3.5 AI-191.8<IP_ADDRESS>00203
    Xcode: 11.1/11A1027 - /usr/bin/xcodebuild
  npmPackages:
    react: 16.8.6 => 16.8.6
    react-native: github:getdelta/react-native#0.60-patched+v8-prebuilt => 0.60.5-patch.5
  npmGlobalPackages:
    create-react-native-app: 1.0.0
    react-native-cli: 2.0.1
    react-native-git-upgrade: 0.2.7

Steps To Reproduce

Hide the statusbar on any screen

Describe what you expected to happen:
The statusbar should disappear, but the screen should not jump down.
This happens on the Google Pixel 3 XL, but is fine on the normal Google Pixel 3. It must be something with the notch.

This is still an issue

Still an issue on rn 0.62

Solved it with:
<item name="android:windowLayoutInDisplayCutoutMode">shortEdges</item>

in android/app/src/main/res/values/styles.xml
And
import android.view.View;

....

@Override
public void onCreate(Bundle savedInstanceState) {

...

    getWindow().getDecorView()
        .setSystemUiVisibility(View.SYSTEM_UI_FLAG_LAYOUT_STABLE | View.SYSTEM_UI_FLAG_LAYOUT_FULLSCREEN);

in android/app/src/main/java/com/gotobridge/MainActivity.java

This is still an issue

This is still an issue

Also still an issue on Galaxy Note 10 Ultra and Huawei P30 Pro in React Native 0.63.3.

Same issue


first pass at simulation testing for escrow

note escrow is not setup yet but this should help

merge

deprecated


V0.3 Bernard

#99 Converted CommandHistory to be Singleton design
#100 Heavily refactored Autocomplete feature to have a manager, a data structure interface, request and response structure
#101 Added collate jar file and a .bat file to generate collated codes
#102 Collated Codes
#120, #121 Added "load" and "saveas" commands to load or change the storage file respectively
#122 Added documentation for the 2 new commands (saveas, load)
#110 Created Sequence Diagrams (TODO)
#109 Created UI Component class diagram (TODO)
#124, #125, #130, #131 Update V0.0 comments from Akshay
Updated CSS to match style of GUI
Refactored some test case to reuse existing methods
Increase test coverage

Coverage increased (+0.2%) to 90.893% when pulling 3bd9d6b3c009ba34fa36f2670e4e87428abb0448 on V0.3-bernard into febd3eb267e91f771d75a77141a5c28a693bef7a on V0.3.

Coverage increased (+0.4%) to 91.112% when pulling 850b7cd68b896e748aa9620cca97b8bd1f417263 on V0.3-bernard into febd3eb267e91f771d75a77141a5c28a693bef7a on V0.3.

Coverage increased (+0.4%) to 91.112% when pulling cd14bd9d31003ceb382d4777f85565e8bd01e306 on V0.3-bernard into febd3eb267e91f771d75a77141a5c28a693bef7a on V0.3.

Coverage increased (+0.4%) to 91.112% when pulling cd14bd9d31003ceb382d4777f85565e8bd01e306 on V0.3-bernard into febd3eb267e91f771d75a77141a5c28a693bef7a on V0.3.

Coverage increased (+0.6%) to 91.265% when pulling 3d585475e56ddd9856e43306aa2e99a3a19814e0 on V0.3-bernard into febd3eb267e91f771d75a77141a5c28a693bef7a on V0.3.

Coverage increased (+0.5%) to 91.195% when pulling 53e939387223baf6ec58adb82fa0097118e1060b on V0.3-bernard into febd3eb267e91f771d75a77141a5c28a693bef7a on V0.3.

Coverage increased (+0.5%) to 91.195% when pulling 435da5117f3327615ba4679f4a3fe8fa4602bfeb on V0.3-bernard into febd3eb267e91f771d75a77141a5c28a693bef7a on V0.3.

Coverage increased (+0.5%) to 91.195% when pulling cebd684b0f3bc03581682f1ef5db4f48d5896775 on V0.3-bernard into febd3eb267e91f771d75a77141a5c28a693bef7a on V0.3.

Coverage increased (+0.5%) to 91.199% when pulling 52d64ca561cc2e43a6787df9564498e7f81d4f3b on V0.3-bernard into febd3eb267e91f771d75a77141a5c28a693bef7a on V0.3.

Coverage increased (+0.5%) to 91.199% when pulling 3888ca59cd1f5cc66b057346f44febc255fe5c7e on V0.3-bernard into febd3eb267e91f771d75a77141a5c28a693bef7a on V0.3.

Coverage increased (+0.5%) to 91.199% when pulling 3888ca59cd1f5cc66b057346f44febc255fe5c7e on V0.3-bernard into febd3eb267e91f771d75a77141a5c28a693bef7a on V0.3.

Coverage decreased (-3.1%) to 87.584% when pulling 0ed685d962d07d10cd12f3e7562c8aedbbdaa3db on V0.3-bernard into febd3eb267e91f771d75a77141a5c28a693bef7a on V0.3.

Coverage decreased (-3.1%) to 87.584% when pulling 718db55ffad5fcd44aa2489bd8ed9dfe7d8195ca on V0.3-bernard into febd3eb267e91f771d75a77141a5c28a693bef7a on V0.3.

Coverage decreased (-3.1%) to 87.584% when pulling 42de43cbda1437d0fb93c4113478d47ef72e873b on V0.3-bernard into febd3eb267e91f771d75a77141a5c28a693bef7a on V0.3.

Coverage decreased (-3.1%) to 87.549% when pulling a4d8ecc6829b488e4626133eeb3f25725d109277 on V0.3-bernard into febd3eb267e91f771d75a77141a5c28a693bef7a on V0.3.

Coverage decreased (-3.1%) to 87.549% when pulling 4cde3efff4ae2d9423e8f8d2e2060ecf5c1b31e8 on V0.3-bernard into febd3eb267e91f771d75a77141a5c28a693bef7a on V0.3.

Coverage decreased (-3.1%) to 87.549% when pulling 4cde3efff4ae2d9423e8f8d2e2060ecf5c1b31e8 on V0.3-bernard into febd3eb267e91f771d75a77141a5c28a693bef7a on V0.3.

Coverage decreased (-0.9%) to 89.826% when pulling d8baa8110863c28f6dc9c1b21d1b0cd2e3c01ffd on V0.3-bernard into febd3eb267e91f771d75a77141a5c28a693bef7a on V0.3.

Coverage decreased (-1.1%) to 89.542% when pulling 25a9105490a01425de5279901f7961af092957a5 on V0.3-bernard into febd3eb267e91f771d75a77141a5c28a693bef7a on V0.3.

Coverage decreased (-1.04%) to 89.642% when pulling 8e5b344703687e1a30754d250c418e4582258166 on V0.3-bernard into febd3eb267e91f771d75a77141a5c28a693bef7a on V0.3.

Coverage decreased (-1.04%) to 89.646% when pulling 56ea53a8c3f71e09a57c37ed3251d420c5f174b1 on V0.3-bernard into febd3eb267e91f771d75a77141a5c28a693bef7a on V0.3.

Coverage decreased (-0.9%) to 89.823% when pulling 7ee86b1626832a94bd0846d79a3987678ec65c8a on V0.3-bernard into febd3eb267e91f771d75a77141a5c28a693bef7a on V0.3.

Coverage decreased (-0.8%) to 89.848% when pulling 2c1c93c56efa343244da1f718ad308c7016a6497 on V0.3-bernard into febd3eb267e91f771d75a77141a5c28a693bef7a on V0.3.


New line is added when the composer expanded with an emoji in the message

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 1.4.59-1
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: @marcaaron
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1712079806486669
Action Performed:

Open app
Go to any chat
add an emoji and type message for 2 lines
Click on the expander and collapse it

Expected Result:
No extra line added in the composer
Actual Result:
A new line is added
Workaround:
unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence
https://github.com/Expensify/App/assets/38435837/b5dbf131-0869-4446-bc8c-6433039ee5d6
https://github.com/Expensify/App/assets/38435837/6a9860d8-7115-45a8-867e-d7cbdaf86acb

View all open jobs on GitHub

In the end I re-used a similar GH issue and changed the title and test steps. I will close it in favor of that one


Function removeToasty doesn't work

Function removeToasty from the constructor doesn't work

Fixed/Changed in v1.*


Issue using Enum type

Rust
let reason_code: i32 = balance_change.reason;
...
let row = tables
    .push_change_composite("balance_changes", keys, 0, table_change::Operation::Create)
    .change("address", ("", address.as_str()))
    .change("new_value", ("", new_value.as_str()))
    .change("old_value", ("", old_value.as_str()))
    .change("ordinal", ("", ordinal.as_str()))
    .change("reason", ("", reason.as_str()))
    .change("reason_code", ("", reason_code.to_string().as_str()));

SQL
CREATE TABLE IF NOT EXISTS balance_changes
(
    block_time          DateTime('UTC'),
    block_number        UInt64,
    block_hash          String,
    block_date          LowCardinality(String),
    address             String,
    new_value           UInt256,
    old_value           UInt256,
    ordinal             UInt64,
    reason              LowCardinality(String),
    reason_code         Enum(
                            'Unknown' = 0,
                            'RewardMineUncle' = 1,
                            'RewardMineBlock' = 2,
                            'DaoRefundContract' = 3,
                            'DaoAdjustBalance' = 4,
                            'Transfer' = 5,
                            'GenesisBalance' = 6,
                            'GasBuy' = 7,
                            'RewardTransactionFee' = 8,
                            'RewardFeeReset' = 14,
                            'GasRefund' = 9,
                            'TouchAccount' = 10,
                            'SuicideRefund' = 11,
                            'SuicideWithdraw' = 13,
                            'CallBalanceOverride' = 12,
                            -- Used on chain(s) where some Ether burning happens
                            'Burn' = 15,
                            'Withdrawal' = 16,
                            -- Rewards for Blob processing on BNB chain added in Tycho hard-fork, refers to BNB documentation to check the timestamp at which it was activated.
                            'RewardBlobFee' = 17,
                            -- Used on optimism chan
                            'IncreaseMint' = 18
                        )
)
    ENGINE = ReplacingMergeTree()
        PRIMARY KEY (block_date, block_time, block_number, ordinal)
        ORDER BY (block_date, block_time, block_number, ordinal);

JSON
{
  "table": "balance_changes",
  "operation": "OPERATION_CREATE",
  "fields": [
    {
      "name": "address",
      "newValue": "0x2a65aca4d5fc5b5c859090a6c34d164135398226"
    },
    {
      "name": "new_value",
      "newValue": "4471101564800316634637"
    },
    {
      "name": "old_value",
      "newValue": "4465945314800316634637"
    },
    {
      "name": "ordinal",
      "newValue": "66"
    },
    {
      "name": "reason",
      "newValue": "RewardMineBlock"
    },
    {
      "name": "reason_code",
      "newValue": "2"
    },
    {
      "name": "block_date",
      "newValue": "2016-05-11"
    },
    {
      "name": "block_time",
      "newValue": "1463003133"
    },
    {
      "name": "block_number",
      "newValue": "1500000"
    },
    {
      "name": "block_hash",
      "newValue": "0x83952d392f9b0059eea94b10d1a095eefb1943ea91595a16c6698757127d4e1c"
    }
  ],
  "compositePk": {
    "keys": {
      "block_date": "2016-05-11",
      "block_number": "1500000",
      "block_time": "1463003133",
      "ordinal": "66"
    }
  }
}

❌ Error - unknown element "2"

2024-07-29T12:36:18.720+0200 ERRO (sink-sql) handle BlockScopedData message at block #6949000 (dd6ec4479e9aa6fb880e5863131d619610e902e8dc48d7cea6fbc7afa78140f7): failed to flush at block #6949000 (dd6ec4479e9aa6fb880e5863131d619610e902e8dc48d7cea6fbc7afa78140f7): dialect flush: executing for entry ["0xea674fdde714fd979de3edf0f56aa9716b898ec8" "2018-12-25" "0xcb79f51e7915a31f97e60c0a622204234bbb8da7ca0df779b1385da876d77573" '�' '�' %!q(big.Int=684278163679825687603) %!q(big.Int=681278163679825687603) '௶' "RewardMineBlock" "2"]: clickhouse [AppendRow]: (reason_code Enum8('Unknown' = 0, 'RewardMineUncle' = 1, 'RewardMineBlock' = 2, 'DaoRefundContract' = 3, 'DaoAdjustBalance' = 4, 'Transfer' = 5, 'GenesisBalance' = 6, 'GasBuy' = 7, 'RewardTransactionFee' = 8, 'GasRefund' = 9, 'TouchAccount' = 10, 'SuicideRefund' = 11, 'CallBalanceOverride' = 12, 'SuicideWithdraw' = 13, 'RewardFeeReset' = 14, 'Burn' = 15, 'Withdrawal' = 16, 'RewardBlobFee' = 17, 'IncreaseMint' = 18)) unknown element "2"

Not an issue, using Clickhouse Enum, the String value should be provided instead of uint8
ex: "RewardMineBlock" instead of 2


Pod button pins defined backwards

fixes #245

Could you please update the link in DaisyExamples to the updated libdaisy?
Thank you.

@HartmutG Done.


Add a class that represents a song

First of all, what should we name it? I was thinking of something like Track, Song, StereoItem, MediaItem. Please let me know if you have other ideas.
What information should this class hold? Here are some thoughts:

Title
Artist
Album
URI
Length
Artwork URI

I think we should have two (or more) classes.

The first one would represents an AudioFile. We don't care more about it, it's just an audio file that needs to be read. We could get the following information:

URI
Length

The second one would be an AudioTrack, from which we should get the following information:

Title
Artist
Album
Artwork URI

How would we get the metadata from the audio file ? Would we need an external library to read the file and return the AudioTrack object or do we get the metadata from the OS ?


Generate curl code from a request and import it again causes stringified JSON body

I have checked the following:

[X] I've searched existing issues and found nothing related to my issue.

Describe the feature you want to add
Generating a curl code from a request and using the same curl to import somehow causes the JSON body of the request to be stringified. Would be great if it is not.
Mockups or Images of the feature
Generated cURL:

Importing cURL:

End result:

Hi, I wasn't able to reproduce this on the latest version (1.34.2). Can you try it out in the latest version and let me know how if the issue still persists?


Get storybook running

Codecov Report

Merging #2297 (75e6228) into main (480f83d) will decrease coverage by 0.01%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main    #2297      +/-   ##
==========================================
- Coverage   32.54%   32.53%   -0.02%
==========================================
  Files         293      293
  Lines        4704     4703       -1
  Branches      653      653
==========================================
- Hits         1531     1530       -1
  Misses       3173     3173

Impacted Files
Coverage Δ

packages/api-builder/src/xform/createScenegraph.ts
100.00% <100.00%> (ø)

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 480f83d...75e6228. Read the comment docs.


Click-to-copy for settings string

Users can click on the settings string to automatically copy it to their clipboard.

Should this instead be a copy icon to the right of the string?


Support way to differentiant API resume from timeout resume (supplied ValueFrom is ignoring default)

Discussed in https://github.com/argoproj/argo-workflows/discussions/7905
I am using argo workflows v3.3.0-rc4, and need a way to distinguish after a suspend step if its output parameter was supplied with a value per API and then resumed, or if the configured duration expired.
Currently this is not possible:

Originally posted by CodingOrca February 17, 2022
I try to implement an approve - release workflow. I need a timeout on the approve step, and need to be able to react in a sub-sequent step if the timeout has expired.
The documentation of the SupppliedValueFrom
shows an example without a duration:
https://github.com/argoproj/argo-workflows/blob/master/examples/suspend-template-outputs.yaml
Is is working, but it is not of practical use if I have no duration set on the  suspend Template. So I have modified it according to the documentation found here:
https://argoproj.github.io/argo-workflows/executor_swagger/#valuefrom
This is only the changed part of the example:
- name: approve
    suspend:
      duration: "60"
    outputs:
      parameters:
        - name: message
          valueFrom:
            default: "some default value"
            supplied: {}

If I run the workflow, and do not supply a value from outside, after 60 seconds the output printed by the "release" step is:
{{steps.approve.outputs.parameters.message}}
But I am expecting the "release" step to print:
some default value
If I feed a value to the parameter, as described in the example, e.g. "Approved", and then resume the workflow, the "release" step is printing the expected value:
Approved
So basically I need the example to be extended with a "duration" of the suspend template and a way to detect in a subsequent step if the suspend was resumed per API or due to the timout.

{} is not a string. Have you tried setting supplied to the default value?

@CodingOrca
outputs:
parameters:
- name: message
default: "default"

@CodingOrca Did you manage to get around this somehow? I have a very similar use case and this is currently blocking me, too.
@alexec Any updates or remarks on @CodingOrca last comment?

OK. I was wrong. You cannot use supplied.

I don't think this is a bug, it is an enhancement.
If anyone is interesting submitting a PR, I give pointers .

I don't think this is a bug, it is an enhancement.
If anyone is interesting submitting a PR, I give pointers .

I am sorry but I super respectfully disagree. I see this as a bug because if the default value is supported for the output parameters, then it should work for all kinds of output parameters, not just some of them. It would be an enhancement if we can live without it or if there's any workaround to the topic of distinguishing whether the workflow has resumed automatically or not. Your call though of course. I trust your judgement and I might be mistaken.

@alexec Any tips in the meantime to circumvent the issue of distinguishing whether a previous suspend step's output has been supplied? I am under a bit of a time pressure :D

Okay, I know I am not alexec...
but I use as work around another suspend, with delay, but with no output, from where I trigger the resume of the my own workflow...  so that my "approve" suspend gets resumed. If it is resumed like this, the configured default is taken:
. . .
  templates:
    - name: main
      dag:
        tasks:
          - name: waitForSeconds
            template: waitForSeconds
            arguments:
              parameters:
                - name: delay
                  value: 60

          - name: resumeWorkflow
            dependencies: [waitForSeconds]
            template: resumeWorkflow

        - name: approve
           template: approve

. . .

    - name: waitForSeconds
      inputs:
        parameters:
          - name: delay
      suspend:
        duration: "{{inputs.parameters.delay}}"

    - name: resumeWorkflow
      http:
        method: "PUT"
        url: "http://argo-server.argo-workflows:2746/api/v1/workflows/kondor-workflow/{{workflow.name}}/resume"

    - name: approve
        suspend: {}
        outputs:
          parameters:
            - name: message
              valueFrom:
                default: "some default value"
                supplied: {}

@CodingOrca Thanks for the hint. I tried your example and it did indeed take the defaults. I started wondering why is it that it works in that example (but not when I tried resuming through the API call from Postman, for example). I just managed to isolate the problem just now. For me, I noticed that the default value gets sets only when the following is true:

We resume via API call. Here it does not matter from another node or from an outside call via Postman, for instance (When the node gets auto-resumed after the duration time is up, the default still gets ignored)
Defaults are set for all output parameters of that node being resumed (it's possible to have more than one) and not just some of them
There is no nodeFieldSelector in the resume request!!!!

Adding the nodeFieldSelector to the resume request breaks it and the default gets ignored. This one really surprised me.
@alexec This information might be useful if someone ever attempts to fix this. Just FYI.

I think I can point to where the bug is:
https://github.com/argoproj/argo-workflows/blob/78f01f2b9f24a89db15a119885dfe8eb6420c70d/workflow/util/util.go#L369
ResumeWorkflow call a different func if there is a nodeFieldSelector. If there is not one, it takes into account defaults:
https://github.com/argoproj/argo-workflows/blob/78f01f2b9f24a89db15a119885dfe8eb6420c70d/workflow/util/util.go#L397
But if there is one, they are not taken into account:
https://github.com/argoproj/argo-workflows/blob/78f01f2b9f24a89db15a119885dfe8eb6420c70d/workflow/util/util.go#L529
I think the block of code doing defaults should be copied and pasted into the second location.
Would anyone like to submit a PR for this? It's by far the fastest way to fix bugs. The core team won't look at this anytime soon.

Hello, i am exactly same issue. Any advance to fix it?

@alexec I will try to submit a PR to fix this since I am right now blocked on this.


enable_testing() moved to project source root directory

Moves enable_testing() to the project source root directory as mentioned in the docs.
Starting from v3.15 the documentation explicitly mentions that it's "automatically invoked, when CTest module is included". Earlier version may do that too, but since the minimum version is v3.1 I kept the enable_testing() call, just be be safe.

Coverage remained the same at 99.815% when pulling 70106f7e99b0eac359970813100595108047bdc8 on offa:ctest into 6d0700ccc5a6875e3125e37888f2afbbe257a44c on cpputest:master.


Djinni is not dwelling on this plane of existence

Whenever i give a djinni machine operator a job, when its done, it just says the spirit bound on the book is not dwelling on this plane of existence
To Reproduce
Steps to reproduce the behavior:

Order Djinni to cook in furnace using storage accessor
when finished, i have to manually take the items and the djinni is gone

II was expecting for the djinni to be back and my items automatically transported to my storage
Screenshots

System (please complete the following information):

Occultism Version:  1.88.0
OS: Windows
Minecraft Version: 1.19.2
FTB Presents AbePack 1.1.1

@SeeMeEat can you reproduce this reliably if you spawn a new spirit and try again?
This message just means that the spirit cannot be found by the mod, that means he was either killed or removed by another mod.
Occultism does not have a builtin mechanism for the machine manager spirits to die (no essence decay), but the essence decay may be enabled by the modpack, or something else may have killed him (mobs, other mods functions, or even bugged into a block and suffocted [thanks MC pathfinding!])
Please try again and see if he survives, and maybe follow him to see if you can see what killed him

Closed as stale


Add a role to run compliance scans against the compute nodes and cont…

…rol plane (#2228)

Add compliance role

Add code to run scans on compute nodes

Add playbook to run compliance scans

Fix pre-commit errors

Add molecule test files

Add login credentials for registry.redhat.io

Fix various errors

Add check to ensure compliance-operator is ready

Also make sure that the scans has started and are complete. And set to ignore errors on compute compliance scans.

Fix the target directory for compute scan results

Modify the script to pass compute scans correctly

Avoid using shell when unzipping files

We make a couple modifications to the scan logic to avoid the use of shell and add a zuul job for the molecule tests.  Also added some changes to variables to make them more specific.

Fix issues from code review

Cherry-picked from: 48a1cfe45aa1933a9d8b48f2b74cb0c13325df5d

I'm +2 on this. @cjeanner do you want to merge this cherry pick?

/approve
/lgtm


Updates

This code is the update for the changes in the IBM CLoud new credentials system.

Thanks for the Pull Request. There are users with older instances of the SpeechToText and Assistant services that still use the username and password, so we won't remove these just yet. We can remove them in the future, after sufficient time has passed.


[CHORE] fix self clippy lints

looks good, we should however also add this commit to ignore revs like we do with our last import formatting. This will help preserve the git blame.
https://github.com/Eventual-Inc/Daft/pull/2807
https://moxio.com/blog/ignoring-bulk-change-commits-with-git-blame/

@samster25 tbh I am kinda confused

@andrewgazelka I believe github runs the CI on the post merge commit. I think you should be fine with rebasing and then merging real real fast

opening separate PR on this repo


fix: include .vue files as the project root files

✅ Closes: #452

The issue is that this plugin doesn't add .vue files to the root files of the project. So in order for it to work, you need .ts or .tsx entry file which imports .vue file. This PR fixes it by extending config parse host to include .vue files.

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Prevent overflow of donation percentage bar.

Not tested because I don't want to ruin my admin branch for now.

not sure if this'd work? will have to test in a bit

I hate having to wanting to change main branches, but not wanting to discard any changes from admin branch.. AAAA so a github edit was made..

oh.. i forgot,

feel free to change the admin submodule in prs, the prod server ignores the submodules anyway and force merges the admin folder to main so


Fixes for signs in 1.20, dependency/lintings updates

Fixes handling of signs in Minecraft 1.20+ (they changed the NBT for them fairly significantly)

This does break previous version of MC but the docker container is on 1.20.4 today

Updates ipython to 8.20

Adds a venv related hack - I couldn't figure out why tox fails with that venv warning/erro but pytest doesn't - disabling it in the test execution path works but is ugly :/

setting.json update / other whitespace changes seem to be due to vscode / linting changes/updates

This is great, thanks! Inspires me to get back into this project.
Do you want to take a look at why the CI is failing? I could take a look this weekend if not.

I have invited you as a collaborator so you can look at the CI if you wish.
I'm OK with only supporting the later versions of Minecraft - the goal of this project is to have fun with python and it would be pretty hard to support all versions of  MC.
BTW, it would be interesting to look into what else has changed re NBT. I found that signs were about the most useful thing for creating an interactive interface because they had the most rich NBT but perhaps there are better choices now.  Also, I think I recall that I have to constantly poll a range of blocks for the sign, it would be better if a search were available through the API but I never found such a thing (witness the clunky workaround for get_block https://github.com/mcipc-tools/mcwb/blob/96c16003d6365a4d017d82a62ce06b4677f95fe6/mcwb/api.py#L78-L96).

"Fixed" some of the CI by pinning the python version (seems broken due to disttools deprecation in 3.12 see https://docs.python.org/3.10/whatsnew/3.10.html#distutils-deprecated ).
Not entirely sure what's up with the permission denied error coming from the test_backup which works locally but not on actions.  I did try moving from /tmp to $HOME but that didn't make a difference.
FWIW speaking of updates/changes in MC: https://github.com/conqp/mcipc/commit/c231c63cf1611a7f324799cf3949485d27956b94

Looks Good. Merging.
re the v4 artifact action. This is fixed in the skeleton so is just the kind of thing that using skeleton is good for. I also think we are dropping the whole upload publishing of requirements.txt thing as too cumbersome. To be replaced by the use of a constraints file and dependabot.

just a headsup: @N6UDP don't do any more work on the CI - I'm pulling in our latest copier project which will rewrite all of the CI.


unlink empty directory

I've tried to learn TDD using PHPUnit with vfsStream. Using PHP's unlink in a directory normally raises up an warning ("Warning: unlink(/tmp/foo): Operation not permitted in ..."). Empty directories must be removed with rmdir. Doing the same with vfsStreamDirectory doesn't return an warning. I don't know if I'm missing something or if it's a real bug.
Test:
<?php
require 'vfsStream/vfsStream.php';
require '../Classes/Recursive/Delete.php';

class Recursive_Delete_Test extends PHPUnit_Framework_TestCase {
    public function testShouldRemoveAnEmptyDirectory()
    {
        vfsStream::setup();
        vfsStreamWrapper::getRoot()->addChild(vfsStream::newDirectory('dir'));
        $recursiveDelete = new Recursive_Delete(vfsStream::url('root/dir'));
        $recursiveDelete->delete();
        $this->assertFileNotExists(vfsStream::url('root/dir'));
    }
}

Production code:
<?php
class Recursive_Delete
 {
    private $_file;

    public function __construct($file)
    {
        $this->_file = $file;
    }

    public function delete()
    {
        unlink($this->_file);
    }
}

Thank you very much and sorry about my english.

After some investigation I can confirm it's bug. unlink() should not remove any directory at all but raise the aforementioned warning.


fix(priceRanges): Remove round from first range

This is to avoid having a no result "<= n" first range if the round was < to stats.min

How did you encounter the bug? Can you show it, not sure to get it all

@vvo The bug occurred when you have average > 100 (the precision=10)  and a min that is small
the round(min, precision) return 1 and then creating "<= 1" first range even if the min is 1.30 for example

got it, next time GIF the corresponding UI bug it will be faster (no big deal)


Add the ability to immediately mark submissions as spoilers

Feature Summary and Justification
This feature makes the spoiler param of the /api/submit method on the Reddit API available through PRAW's Subreddit.submit() method. This basically does the same thing for spoiler as https://github.com/praw-dev/praw/pull/1030 did for nsfw.
References

https://www.reddit.com/dev/api/#POST_api_submit

Thanks for the terrific PR! It's much appreciated.

Thanks for the terrific PR! It's much appreciated.


Is it possible to set the default color or startup color of the picker by using only a hex / html value?

like #F3F3F3 without specifying hsl/hsv/rgba ?

It is possible. I simply asked before giving it a try.


Nuget binaries

Does the Boogie nuget include both Framework and .NET core binaries? I'd like to switch Corral to use Boogie as a Nuget (rather than a submodule).

Hi @akashlal, the Boogie NuGet package is .NET Core only. Let's close this one and discuss more in #303.


Captcha captcha captcha (insert sad face here...)

I'm able to successfully run the bot.
However, for some reason, sometimes I get captchas and I'm unable to solve it.
There's nothing in my command prompt warning me about a captcha while it's happening (it's just stays on "[*] sharing poshmark listings for ** items in closet... please wait...").
If I try to solve it while it's happening, the captcha is stuck in a loop and just throwing another after another.
I end up watching the screen while the script is running...
Is there something I'm missing here ?

it worked for a day and then all of a sudden there is a captcha during the sharing part.

If you solve the captcha in a separate browser window post-share, you can just 'x' out or close the captcha on the bot's browser window and it will work successfully again. However, this must be done quite often.

@selahlion what do you mean solving the captcha in a separate browser? I'm running mine on Chrome and I can't x out the Captcha.


Feat: toggle sidebar on mobile

What it solves
Resolves https://github.com/gnosis/safe-pm/issues/64
How this PR fixes it
The sidebar on a small screen is shifted off screen. Only 5% of it is still visible, and if you tap on it, it will slide into view.
Tapping anywhere will again hide the sidebar.
Also adjusted modal windows to be full width.
How to test it
Open the app on a mobile phone
Screenshots

I think it works fine and we can implement it the way it is now
Still, is hard to notice that that is a sidebar you can display, so at some point at would add a small arrow at the top to show that we can click it to display the sidebar

Good idea @francovenica, I'll add a button. Perhaps at the bottom of the page would be best, so as not to cover anything.

I very much agree with @francovenica
My phone has a curved display and this makes it harder to tab on the hidden sidebar. Most apps have a menu indicator in the top bar.

Added a button:

Looks good to me.

@katspaugh Something similar would actually be pretty cool even on desktop. I.e. allowing to hide the sidebar in order to get more screen estate e.g. for Safe Apps.

@lukasschor true 👍


”VASLUI” is listed in the drop-down menu

Description:
"VASLUI" is listed in the drop-down menu alongside its corresponding county and commune, according to the law.
Precondition:
The website is up an running.

Step 1
Type "VASLUI" in the search bar.

Expected results
The location is listed in the drop-down menu as "Municipiul VASLUI, VASLUI".


546/shav-section-3-competition-information

Description:
This pull request enhances Section 3 of the competition page by implementing a structured layout that provides essential information about the competition. It includes two main columns: one detailing "How It Works" with a step-by-step guide, and the other outlining the reasons to join, accompanied by a description and benefits. Additionally, it features eligibility criteria and prize details in separate columns, ensuring clarity for participants. The section concludes with a dynamic timeline that highlights key events, using visually distinct cards for each entry, which improves user engagement and accessibility. Overall, these updates aim to create a more informative and user-friendly experience for potential competitors.
Ticket: isaaccomputerscience/isaac-cs-issues#544

Designs: https://www.figma.com/design/xLvYVdW2miyMeZk2QuJ0uU/Isaac---Designs---240412?node-id=859-161&node-type=frame&t=rkhnhHQNDsviKGZl-0
Media attachement(s):
Screen recording below will shows from responsiveness from mobile, tablet and to then desktop
https://github.com/user-attachments/assets/077cc6c3-66aa-456f-a74f-18199acb1937

Can we keep the structure of the folders consistent please. In this case, unless the buttons of competition information is being used in more than 1 section, they should be in their designated folder. Section 1, 2, 3 & 4 with their respective files.

In responsive design at 765px to 1000px screen size the timeline is not working as it should.
I have provided screenshots to help:

Also curious if we can get some testing wins here? Shouldn't take long to throw some unit tests together :)

happy with everything here. Just like with Parth's PR, it'd be nice to see your new timeline component tested. It shouldn't take too long :)


Using Marionette with backbone-forms

I am currently using Marionette and would like to use backbone-forms.  However, since forms inherits directly from Backbone, I am unsure how to proceed.  Can I just change backbone-forms to inherit from Backbone.Marionette.View (and also change the Backbone.View.prototype calls?  Or is backbone-forms incompatible without more major modifications?

Any update on it? I'm about to work on a similar feature. Do backbone-forms play well with Marionette? Any examples would be appreciated.

I'm currently using it.
I'm treating the Form as a Marionette.ItemView which triggers less events. So I sometime have to manually delay some actions (using setTimeout), because there's no equivalent to the "on:show" event.
I didn't need other workarounds, so I'd say that it works pretty well.
As an example, you can do something like this:
var user = new User();

var form = new Backbone.Form({
    model: user
});

YourMarionetteApp.yourRegion.show(form);

And it should work nicely.

Thanks, managed to put it in the Marionette.ItemView, you can find demo here http://codepen.io/natku/pen/emywaG?editors=101
var accounts = ['account1', 'account2', 'acocunt3', 'account4'];
var form = new Backbone.Form({
    schema: {
        name: { type: 'Text', validators: ['required'] },
        email: { type: 'Text', validators: ['required', 'email'] },
        notes: 'TextArea',
        accounts: { type: 'Select', options: accounts}
    }, submitButton: 'Submit'
})

var MarionetteFormView = Backbone.Marionette.ItemView.extend({
  events: {
    'submit': 'validate',
    'error': 'blur'
  },

  initialize: function(){
    this.form = this.getOption('form');
    this.listenTo(this.form, 'name:blur email:blur', this.blur);
 				  },
  render: function(){
    this.form.render();
    this.$el.html(this.form.render().el)
    return this;
  },
  validate: function(){
    var errs = form.validate();
    if (errs) event.preventDefault();
  },
  blur: function(){
    console.log('blur', arguments)
  }

});

formView = new MarionetteFormView({
  form: form,

})
Backbone.$('.body').append(formView.render().el);

Nice, thanks!

Hi,
Thanks a lot for your example natkuu, did you manage to use a model instead ?
I can't manage to make it work...
Thanks,
Nicolas

Finally it works very fine, don't bother asking an example if needed ;)

Hi, also had the same issue and was able solve it using _.extend. Not sure if this approach provides marrionette view events, but I was able to use forms with regions.
var FormView = Backbone.Form.extend({});
var FormView = _.extend(FormView, Marionette.View);

Usage:
var User = Backbone.Model.extend({
    schema: {
        // ...
    }
});

var user = new User();

var form = new FormView({model: user});
YourMarionetteApp.yourRegion.show(form);


Run bootstrapping job automatically every time the Docker image changes

I couldn't do it with Terraform. Alternatives include:

Using Cloud Build so the bootstrapping job runs before the deployment of the services. This would require migrating the provisioning of the Cloud Run services and job from Terraform to Cloud Build.
Integrate Eventarc, so we listen for relevant Cloud Run events (e.g., CreateJob, UpdateJob), and then execute the job ASAP -- hopefully before the new service revisions start serving traffic.

Superseded by #11


Fix sagemaker config

What does this PR do?
Puts enable_cpu_affinity into the SagemakerConfig and write_basic_config (Similar to debug)
We already have checks to verify if we should call cpu_affinity, so its safe to do (and will default to False if they didn't pass it in), so even if sagemaker doesn't support it it will still (almost always) be False
Fixes https://github.com/huggingface/accelerate/issues/2744
Before submitting

[ ] This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
[ ] Did you read the contributor guideline,
Pull Request section?
[ ] Was this discussed/approved via a Github issue or the forum? Please add a link
to it if that's the case.
[ ] Did you make sure to update the documentation with your changes? Here are the
documentation guidelines, and
here are tips on formatting docstrings.
[ ] Did you write any new necessary tests?

Who can review?
Anyone in the community is free to review the PR once the tests have passed. Feel free to tag
members/contributors who may be interested in your PR.
@BenjaminBossan

(Will be safe to merge after the user can verify :) )

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

@BenjaminBossan updated. Will also re-look at the test to make sure it doesn't need to be ran per-se, just check for config validity

thank you!


Spike: GPU feature discovery for NVIDIA cards

I'd like to leverage NVML to detect specific features about NVIDIA cards on a given node.
Potential features:

GPU family
Architecture
Maybe video memory?

cc @liyubobj @seelam

Hey @cmluciano, glad to accept more feature sources! Please ping if you have any questions.

@cmluciano Yes, we also need video memory and GPU topology

I've searched for a long time but seems no any nvml-lib in go released yet. Did you find any useful libs? thx :) @cmluciano

@cmluciano Compute Capability and Driver version I think is very important to us.

Compute capability could let customer know whether it could be used by some applications, like TensorFlow.
Driver version could help the scheduler know whether a container can run on a node.

I will definitely look into those @Hui-Zhi .
@tianshapjq You can try out nvidia-docker.

Agreed on Driver version, container mounting volumes need this. Currently I use:
{
    Name: "nvidia-driver",
    VolumeSource: apiv1.VolumeSource{
        HostPath: &apiv1.HostPathVolumeSource{
            Path: "/var/lib/nvidia-docker/volumes/nvidia_driver/384.47",
        },
    },
}

I was thinking if Kubernetes needs add nvidia-docker as a dependency to use NVML etc.

Yes I think the community is rallying behind hardware device plugins. I'm closing this in favor of https://github.com/kubernetes/community/pull/695. That proposal tracks the future of GPUs in k8s.


Evaluation on Aachen day-and-night benchmark

Thanks for your remarkable works! I follow your instruction to evaluate the local feature on Aachen day-and-night benchmark by using evaluation script mentioned. However, the results are much higher than ones provided in your paper (D2-Net, R2D2, SuperPoint, etc.) I have not fine-tuned the setup in the evaluation script and I cannot find the reason causing such gap. Could you give me some advice to reappear the correct results? Thanks :)
As a reference, my results are:
SuperPoint: 70.4/79.6/88.8
D2-Net: 74.5/85.7/99.0
R2D2: 74.5/84.7/98.0
which seems unreasonable ...

Aachen day-and-night dataset updated their ground truth poses, and all methods benefit from this ...

Thanks for your reply but the improvement seems excessively large... Have you resubmitted the evaluation results to the benchmark recently?


Is the project dead? Moved away? How can we get this to be maintained?

If you would like to contribute, please submit a pull request. Thanks! 🙏


Fix import mixin to resolve issue #36

I'm merging this without approval since it's a bug fix and the test says it's fine. Hope that's OK.

That's fine. Thanks for fixing it so quickly.


Improve UAC bypass rule based on sample in the wild

Update rule based on analysis of a sample from the wild. This sample has not been added to examples because it needs to be unpacked manually from memory.
The APIs have been broken into three categories:

APIs used to run an elevated program.
APIs for obtaining token information.
APIs for using the token.

I removed GetStartupInfoW, which is not essential to the UAC bypass technique and is typically used to run the elevated process in a hidden window.
Only the string wusa.exe is currently included. Opened issue #615 to discuss whether this should be expanded.

Thanks @mr-tz - I will try to find some additional samples and update the rule when time allows.


NEVPT2 Energies Depend on nroots from CASCI

Problem
I'm seeing some unexpected behavior while running NEVPT2 on a system we're studying.
I follow a procedure analogous to the one shown in the PySCF example below:
https://github.com/pyscf/pyscf/blob/adbdcbdd426356485d9a249d7461f668d19d6f9e/examples/mrpt/41-for_state_average.py#L31-L47
We've noticed that there is a slight dependence ~ 0.8 mHa of the final NEVPT2 energies on the number of roots we set for the CASCI (line 39 above) calculation. We've kept the number of states in the SA-CASSCF calculation the same so I wouldn't expect the NEVPT2 energies to change at all.
Possible explanation
Is this related to the Davidson solver in CASCI where you typically want to keep several more states than you're actually interested in? If that's the case should I always set nroots to be larger than the number of states I'm targeting in my SA-CASSCF calculation? See below
Reproducing Error: Naphthalene
Since our calculations on the full system we're studying are expensive I've tried to reproduce this behavior with a smaller system. The input script is shown below.
For naphthalene, I can't see variations as large as 8e-4 Ha, but I do see variations at about 6e-5 Ha. I also looked at benzene and saw variations in the NEVPT2 energies of about 1e-5 Ha.
If anyone has any knowledge or suggestions about this, let me know!
import sys
import numpy as np

from pyscf import gto, scf, mcscf, mrpt
from pyscf.tools import molden

nstates = 3
nroots = 10

if len(sys.argv) == 1:
    print(f"Setting nstates={nstates} and nroots={nroots}")
elif len(sys.argv) == 3:
    nstates = int(sys.argv[1])
    nroots = int(sys.argv[2])
    print(f"Setting nstates={nstates} and nroots={nroots}")

else: but when I look at [nevpt2.py](https://github.com/pyscf/pyscf/blob/master/pyscf/mrpt/nevpt2.py) I caanywhere that might n't find
    raise ValueError("Wrong number of command line arguments")

mol = gto.Mole()
mol.atom = """
 C                 -2.42729600   -0.70070000    0.00000000
 C                 -1.21364800   -1.40140000    0.00000000
 C                 -0.00000000   -0.70070000    0.00000000
 C                 -0.00000000    0.70070000    0.00000000
 C                 -1.21364800    1.40140000   -0.00000000
 C                 -2.42729600    0.70070000   -0.00000000
 H                  1.21364800   -2.47140000    0.00000000
 H                 -3.35394318   -1.23570000    0.00000000
 H                 -1.21364800   -2.47140000    0.00000000
 C                  1.21364800   -1.40140000   -0.00000000
 C                  1.21364800    1.40140000    0.00000000
 H                 -1.21364800    2.47140000    0.00000000
 H                 -3.35394318    1.23570000   -0.00000000
 C                  2.42729600    0.70070000    0.00000000
 C                  2.42729600   -0.70070000   -0.00000000
 H                  1.21364800    2.47140000   -0.00000000
 H                  3.35394318    1.23570000    0.00000000
 H                  3.35394318   -1.23570000   -0.00000000
"""
mol.basis = "ccpvdz"
mol.symmetry = True
mol.max_memory = 80000
mol.verbose = 4
mol.build()
mf = scf.RHF(mol).run()
# molden.from_scf(mf, "_molden/naphtalene_dz.molden")

ncas = 10
nelecas = 10

n_states = nstates
weights = np.ones(n_states) / n_states

mc = mcscf.CASSCF(mf, ncas, nelecas).state_average_(weights)
mo = mcscf.sort_mo(mc, mf.mo_coeff, [27, 31, 32, 33, 34, 35, 36, 37, 45, 48])
mc.kernel(mo)
mo = mc.mo_coeff

# CASCI to prep for NEVPT2
mc = mcscf.CASCI(mf, ncas, nelecas)
mc.fcisolver.nroots = nroots
mc.casci(mo)

e_corr = np.zeros((nstates))
e_tot = np.zeros((nstates))

for i in range(nstates):
    print(f"STATE {i}")
    e_corr[i] = mrpt.NEVPT(mc, root=i).kernel()
    e_tot[i] = mc.e_tot[i] + e_corr[i]
print("Nev", e_tot)

np.savetxt(f"_data/nroots={nroots}.txt", np.array([nroots] + e_tot.tolist()))

Update
I've run a few more tests and it seems that I'm seeing a difference between the final NEVPT2 energies even when I'm using the same number of roots. The discrepancy, in this case, is 0.2 mHa. I've tightened down the mc.conv_tol for the CASSCF calculation along with mc.fcisolver.conv_tol and checked that the CI vectors (which are fed into NEVPT2) are the same. This makes it seem like the difference is coming solely from the NEVPT2 calculations.

Hi: we've seen this before. One of the NEVPT2 component energies is
numerically unstable. Try running with a single thread.
On Wed, Jun 17, 2020 at 7:22 AM James E T Smith<EMAIL_ADDRESS>wrote:

Update
I've run a few more tests and it seems that I'm seeing a difference
between the final NEVPT2 energies even when I'm using the same number
of roots. The discrepancy, in this case, is 0.2 mHa. I've tightened down
the mc.conv_tol for the CASSCF calculation along with
mc.fcisolver.conv_tol and checked that the CI vectors (which are fed into
NEVPT2) are the same. This makes it seem like the difference is coming
solely from the NEVPT2 calculations.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/pyscf/pyscf/issues/638#issuecomment-645404824, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAN5NRAHNDNBQGS2WLRUJCTRXDGRHANCNFSM4N72EUAA
.

Hi @gkc1000 thanks for the heads up about that. I'm seeing the instability in the Srsi term, is that the same one you saw it in? I've switched to a single thread to see if things settle down, but the calculations are painfully slow. If that's the only term with the instability then I'll only set threads to 1 for that term.

I don't recall now, it was a while ago, but I think it was a problem only
in large basis sets in Cr2. We did not analyze it carefully as the
fluctuations were still fairly small (maybe on the 0.1 mH scale).
On Wed, Jun 17, 2020 at 7:54 PM James E T Smith<EMAIL_ADDRESS>wrote:

Hi @gkc1000 https://github.com/gkc1000 thanks for the heads up about
that. I'm seeing the instability in the Srsi term, is that the same one you
saw it in? I've switched to a single thread to see if things settle down,
but the calculations are painfully slow. If that's the only term with the
instability then I'll only set threads to 1 for that term.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/pyscf/pyscf/issues/638#issuecomment-645739019, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAN5NRFCG6S3ULQQVBOEL2TRXF6UPANCNFSM4N72EUAA
.

Ok thanks for the heads up!

For those interested, using a single thread fixes the issue. I've included two tables where I break down the effect of using single vs. mutlithreaded NEVPT2.
Listed in the tables are the differences in NEVPT2 components from running the same calculation twice.
Without using a single thread:

Si    (+1)'
Sij   (+2)
Sijr  (+1)
Sijrs (0)
Sir   (0)'
Sr    (-1)'
Srs   (-2)
Srsi  (-1)

1.16214e-07
-2.2906e-08
-3.35022e-07
2.10412e-08
1.0888e-07
1.51292e-07
-8.94214e-09
-0.0012994

Using a single thread

Si    (+1)'
Sij   (+2)
Sijr  (+1)
Sijrs (0)
Sir   (0)'
Sr    (-1)'
Srs   (-2)
Srsi  (-1)

4.73266e-08
-2.01128e-08
-1.06267e-05
-2.69257e-09
2.96425e-08
1.17653e-07
-2.18422e-08
3.82372e-05

The Srsi component is the troublesome one and although it's still not as consistent as the other components it's still a solid improvement.


Builld Darknet CPU Image from main branch

[x] Build Base Image
[x] Test Base Image
[x] Test with Full ODC Image

All is working here. Will close once it is merged into main branch.


Search grammar file in transformFs

Fixed #727.
Changes proposed in this pull request:

[x] func ReadGrammar reads grammar from transformFs instead of root fs.
[x] func LoadGrammar loads grammar from transformFs instead of root fs.
[x] combine the ChrootFs and ModFs

@anz-bank/sysl-developers

I've merged by changes which addresses this in a slightly different way (adding a root parameter to modfs), and then making the grammar relative to the transform root. I think this PR can be closed.


[DS-125] Date type repeatable in the submission

This has been tested in xmlui and should be verified in jspui.
http://jira.duraspace.org/browse/DS-125

Closing as outdated/obsolete (no activity in several years). Please resubmit if you are interested in helping move this forward. Thanks!


Should an deprecated-info added since docker-machine is only in maintenance mode anymore

Hi @josefglatz,
Despite the fact that Docker Machine itself is in the maintenance mode, it was not actually deprecated (https://github.com/docker/machine/issues/4537).
It could still be used. And, what's more important, libmachine library is not deprecated neither.
Our driver, which we traditionally call "docker machine driver" could also be used with other projects leveraging drivers based on libmachine. For example, with minikube:

https://minikube.sigs.k8s.io/docs/
https://minikube.sigs.k8s.io/docs/drivers/parallels/
It's a popular tool and it doesn't directly depend nor on docker-machine, neither on boot2docker.

So, I think we should not deprecate our driver now.
Though, it makes sense to update or main README.md and project description to point that it could be also used with minikube (like it's done in https://github.com/machine-drivers/docker-machine-driver-xhyve).


[CLOUDGA-18024] Handle the case when last rotated is null

Summary:
This PR handles the case when last rotated param for CMK config is null.

[arjain@dev-server-arjain ybm-cli]$ ./ybm cluster encryption list --cluster-name npavithra-azure-target
A newer version is available. Please upgrade to the latest version v0.1.17
Provider   Key Alias                              Last Rotated   Security Principals                      CMK Status
AZURE      99ad65ab-d6bf-4950-b5f2-1806b2fd682f   -              https://test-azure-gj.vault.azure.net/   ACTIVE

Add teest please.

@shinji62
Added UT, please TAL.

@shinji62 Added UT, please TAL.
Don 't change the test just add another one.
After that we should be good,

@shinji62 Added new unit test, please TAL


Install fonts on windows docker image for chromium-based browser and lock docker version

Description
Install fonts on windows docker image for chromium-based browser and lock docker version
Issues Resolved
https://github.com/opensearch-project/opensearch-build/issues/4370
https://github.com/opensearch-project/opensearch-build/issues/4416
https://github.com/opensearch-project/opensearch-build/issues/4126
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

https://build.ci.opensearch.org/job/docker-build/4778/console

https://build.ci.opensearch.org/job/docker-copy/929/console


fix yo-dawg problem with zng-alpha backward compat

This commit fixes the zng-alpha commit.  Importing an old version of zq
inside of zq because the import paths of the old version brought in the current
code instead of the old code.
The fix is to copy the needed zng package and zio pieces into zq/alpha.
This PR has two commits: the first is the big copy to zq/alpha and the second
are the changes to point to zq/alpha instead of zq/0220.

This is working for me: I ran into this working on #1390 . With these changes, I was able to take the alpha zng file from a current Brim release, and use the alpha zng reader to convert it to post-alpha zng. I compared the tzng for each - using the alpha zq code to generate its tzng, and the post-alpha zq code to generate its tzng - and they match.


HTTP Server API Endpoint not working

Hi,
I am using docker 0.4.2 tag and with this version the HTTP endpoint is not working. I am still using : curl --include
--request POST
--header "Content-Type: application/json"
--data-binary "{
"id": "2f7b7768188583faea3868e65c9474a475ce6065402334a7a5844ee64c9e20b0",
"signature": "30440220283066391658833bf098de547a339938da6307f5953cfe9df71689017abe224b0220424a83f689d97b3b4a943cb57c2c2427f71eb592c86fe1c50921b178ee84910b",
"transaction": {
"current_owner": "cpSayyvd4HH18GYAMwu6KvGKt7GN2cEdahpRA4ivQ1Lw",
"data": {
"hash": "efbde2c3aee204a69b7696d4b10ff31137fe78e3946306284f806e2dfc68b805",
"payload": null
},
"input": null,
"new_owner": "cpSayyvd4HH18GYAMwu6KvGKt7GN2cEdahpRA4ivQ1Lw",
"operation": "CREATE",
"timestamp": "1456847051.247979"
}
}"
'http://localhost:9984/api/v1/transactions/'
for testing purposes and the bigchain log shows an error
INFO:bigchaindb.commands.bigchain:BigchainDB Version 0.4.2
INFO:bigchaindb.config_utils:Configuration loaded from /data/.bigchaindb
INFO:bigchaindb.commands.bigchain:RethinkDB started with PID 8
INFO:bigchaindb.commands.bigchain:Starting BigchainDB main process
INFO:bigchaindb.processes:Initializing BigchainDB...
INFO:bigchaindb.processes:starting bigchain mapper
INFO:bigchaindb.processes:starting backlog mapper
INFO:bigchaindb.processes:starting block
INFO:bigchaindb.processes:starting voter
INFO:bigchaindb.processes:starting election
INFO:bigchaindb.block:bootstraping block module...
INFO:bigchaindb.voter:voter waiting for new blocks
INFO:bigchaindb.voter:new_block arrived to voter
INFO:bigchaindb.voter:new_block arrived to voter
INFO:bigchaindb.voter:new_block arrived to voter
[2016-06-24 18:16:35 +0000] [85] [INFO] Starting gunicorn 19.6.0
[2016-06-24 18:16:35 +0000] [85] [INFO] Listening at: http://<IP_ADDRESS>:9984 (85)
[2016-06-24 18:16:35 +0000] [85] [INFO] Using worker: threads
[2016-06-24 18:16:35 +0000] [101] [INFO] Booting worker with pid: 101
INFO:bigchaindb.voter:updating block 49549f62f5bc149868bc11005a8b64e4995520f525479f6359732381f134487e with number 1 and with vote {'vote': {'previous_block': '1124b45729d9153d831ef46ae321567a3dc012dd01292a26e17efcc1161a01ea', 'invalid_reason': None, 'is_block_valid': False, 'voting_for_block': '49549f62f5bc149868bc11005a8b64e4995520f525479f6359732381f134487e', 'timestamp': '1466792196'}, 'node_pubkey': 'BK2E3YfLPhvwzYumZ4msDhT5pxH8PEUu2mSHd6Z4Q51k', 'signature': '2aZGhr9hPjsuZaRadxj3aViqXk5dwgnjK3WRNxeM2Ye7zk77YQAB9bD3VJ5dt87KSnp31TUBVB57v3Ytp63RxvGT'}
[2016-06-24 18:16:35 +0000] [102] [INFO] Booting worker with pid: 102
INFO:bigchaindb.block:finished reading past transactions
INFO:bigchaindb.voter:updating block 0cf03627076e7cb3fa1e61eee815d4c408ec5cbfe9065535c109087a577ceb21 with number 2 and with vote {'vote': {'previous_block': '49549f62f5bc149868bc11005a8b64e4995520f525479f6359732381f134487e', 'invalid_reason': None, 'is_block_valid': False, 'voting_for_block': '0cf03627076e7cb3fa1e61eee815d4c408ec5cbfe9065535c109087a577ceb21', 'timestamp': '1466792196'}, 'node_pubkey': 'BK2E3YfLPhvwzYumZ4msDhT5pxH8PEUu2mSHd6Z4Q51k', 'signature': '4HKdL1WZFAiv2c1zjyNZbcRstYaZ6ud3hcdL38hw8cpMYxQtjZ3sZxcKzRvFMu7495E61uXUG739BrxKEvhNXs6V'}
INFO:bigchaindb.voter:updating block b766a0129d841bdb8ab186cda5429dc0fe0a381c2ef85448a81f051ad120e7bc with number 3 and with vote {'vote': {'previous_block': '0cf03627076e7cb3fa1e61eee815d4c408ec5cbfe9065535c109087a577ceb21', 'invalid_reason': None, 'is_block_valid': False, 'voting_for_block': 'b766a0129d841bdb8ab186cda5429dc0fe0a381c2ef85448a81f051ad120e7bc', 'timestamp': '1466792196'}, 'node_pubkey': 'BK2E3YfLPhvwzYumZ4msDhT5pxH8PEUu2mSHd6Z4Q51k', 'signature': 'KgXPWjcWhmtMy4zUzycjVy12pk9vyvgsavCUeZ77UeuWNtLLJK9RkMF2iXKqiZZLeW7b1nSwqzxAkvd3LHmEqfW'}
INFO:bigchaindb.block:finished bootstraping block module...
INFO:bigchaindb.block:starting block module...
INFO:bigchaindb.processes:

                                                                     *

Initialization complete. BigchainDB is ready and waiting for events.   *
You can send events through the API documented at:                     *

http://docs.bigchaindb.apiary.io/                                   *

                                                                     *

Listening to client connections on: <IP_ADDRESS>:9984                       *

                                                                     *

[2016-06-24 18:16:35 +0000] [128] [INFO] Booting worker with pid: 128
INFO:bigchaindb.block:exiting block module...
Process validate_transactions:
Traceback (most recent call last):
File "/usr/lib/python3.4/multiprocessing/process.py", line 254, in _bootstrap
self.run()
File "/usr/lib/python3.4/multiprocessing/process.py", line 93, in run
self._target(*self._args, **self._kwargs)
File "/usr/src/app/bigchaindb/block.py", line 74, in validate_transactions
is_valid_transaction = b.is_valid_transaction(tx)
File "/usr/src/app/bigchaindb/core.py", line 369, in is_valid_transaction
self.validate_transaction(transaction)
File "/usr/src/app/bigchaindb/core.py", line 354, in validate_transaction
return self.consensus.validate_transaction(self, transaction)
File "/usr/src/app/bigchaindb/consensus.py", line 137, in validate_transaction
if transaction['transaction']['fulfillments'][0]['input']:
KeyError: 'fulfillments'
Process validate_transactions:
Traceback (most recent call last):
File "/usr/lib/python3.4/multiprocessing/process.py", line 254, in _bootstrap
self.run()
File "/usr/lib/python3.4/multiprocessing/process.py", line 93, in run
self._target(*self._args, **self._kwargs)
File "/usr/src/app/bigchaindb/block.py", line 74, in validate_transactions
is_valid_transaction = b.is_valid_transaction(tx)
File "/usr/src/app/bigchaindb/core.py", line 369, in is_valid_transaction
self.validate_transaction(transaction)
File "/usr/src/app/bigchaindb/core.py", line 354, in validate_transaction
return self.consensus.validate_transaction(self, transaction)
File "/usr/src/app/bigchaindb/consensus.py", line 137, in validate_transaction
if transaction['transaction']['fulfillments'][0]['input']:
KeyError: 'fulfillments'
[2016-06-24 18:16:35 +0000] [140] [INFO] Booting worker with pid: 140
[2016-06-24 18:16:35 +0000] [141] [INFO] Booting worker with pid: 141
ERROR:bigchaindb.web.server:Exception on /api/v1/transactions/ [POST]
Traceback (most recent call last):
File "/usr/local/lib/python3.4/dist-packages/flask/app.py", line 1817, in wsgi_app
response = self.full_dispatch_request()
File "/usr/local/lib/python3.4/dist-packages/flask/app.py", line 1477, in full_dispatch_request
rv = self.handle_user_exception(e)
File "/usr/local/lib/python3.4/dist-packages/flask/app.py", line 1381, in handle_user_exception
reraise(exc_type, exc_value, tb)
File "/usr/local/lib/python3.4/dist-packages/flask/_compat.py", line 33, in reraise
raise value
File "/usr/local/lib/python3.4/dist-packages/flask/app.py", line 1475, in full_dispatch_request
rv = self.dispatch_request()
File "/usr/local/lib/python3.4/dist-packages/flask/app.py", line 1461, in dispatch_request
return self.view_functionsrule.endpoint
File "/usr/src/app/bigchaindb/web/views.py", line 89, in create_transaction
tx = util.transform_create(tx)
File "/usr/src/app/bigchaindb/util.py", line 611, in transform_create
new_tx = create_tx(b.me, transaction['fulfillments'][0]['current_owners'], None, 'CREATE', payload=payload)
KeyError: 'fulfillments'

The transaction structure changed significantly in v0.3.0 https://bigchaindb.readthedocs.io/en/latest/topic-guides/models.html#transaction-concepts.
We added support for multiple inputs and outputs and implemented the crypto-conditions from the interledger spec https://bigchaindb.readthedocs.io/en/latest/topic-guides/models.html#conditions-and-fulfillments.
You can check the temp_client to see how the new transactions look https://bigchaindb.readthedocs.io/en/latest/drivers-clients/python-driver-api-examples.html
Your problem is that you are trying to POST a transaction that has a schema that is no longer valid since v0.3.0

@r-marques is this still open?

This was a problem with a mismatch between different versions of bigchaindb.


Fix requests.get mock

Description
#1470 mocked out requests.get call, but #1471 uses another requests.get call that did not get mocked properly. We check which call this is and provide the appropriate mocked response.
Changelog
Itemize code/test/documentation changes and files added/removed.

test-datafetcher

Fixes
Should fix datafetcher test failures like in #1476.

This should be what's causing the test failures


Chrome Refresh 2023 back pls

Is your feature request related to a problem? Please describe.
I don't like the Th24 UI, it's just too old for me, I like the new and vibrant Chrome Refresh 2023 but I cannot find any option to enable it.
Describe the solution you'd like, including relevant patches or source
Add an option to enable Chrome Refresh 2023.

It still doesn't work

I want to change almost all of them.

I see.


[Feature] README Updates

Is there an existing feature request for this?

[X] I have searched the existing issues

Describe the Feature
This packages README is not currently inline with our existing standards. It would be ideal for this repo to see README updates to be in line with our current documentation guidelines.
Describe alternatives you've considered
No response
Are you interested in contributing this feature?

[X] Yes.
[ ] Yes, but I will need assistance and will schedule time during your office hours for guidance.
[ ] No.

Anything else?
During these updates, we should also add the identifier variables and ensure the integration_tests/dbt_project.yml references the identifiers as well.

Part of the v0.7.0 release, so this issue can be closed.


fix: broken documentation URL

What kind of change does this PR introduce?
docs update
What is the current behavior?
Broken link to documentation

@0xflotus good catch, thanks!


Question: Animal Taming

how would I do this ?
Taming th e Untamed
An untamed creature will always be feral, but over time (and
with your DM’s permission) can learn to trust its owner and its
owner’s allies.
To tame an untamed creature, you must take a short or long
rest in its company once each day, making a Wisdom (Animal
Handling) check at the end of that rest. The initial DC for this
check is 20, and the DC for each subsequent check is 20 – half
the number of successful checks you’ve made + the number of
consecutive days you’ve gone without making this check (maximum
10). Placing the creature under the animal friendship spell
does not decrease this DC, but instead grants you an advantage on
the check due to the charmed condition.
I see I can use a custome formula and use<EMAIL_ADDRESS>to get the number but I am not sure how to do the diminishing DC ...
Maybe something where instead of changing the DC the success count is incorporated in the the Roll bonus ...
Leave the DC@20 ... but then  1d20+@skills.ani.mod+(.5*SuccessCount) for the Custom Formula.  I am not seeing that in the data object.
The math would be simple enough IDK if the field takes it   20DC -(successfulRoles*.5)  the + days would be a consequence .. Ideally imposed by the DM ...
Would be cool if the role took place and then gave the DM the ability to say if it was a SUCCESS or NOT
how do we incorporate the successful roll back into the math of the Role..

With the way that this module is written there is no way to determine the number of successful rolls before all rolls have finished. This activity sadly can not be automated because of this limitation; however, I will look into the manual validation you have proposed.

Thank you for responding ...
I like what you did here and understand the effort in this ..
On Wed, Apr 28, 2021 at 12:02 PM Ethck @.***> wrote:

With the way that this module is written there is no way to determine the
number of successful rolls before all rolls have finished. This activity
sadly can not be automated because of this limitation; however, I will look
into the manual validation you have proposed.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Ethck/Ethck-s-Downtime-Tracking/issues/60#issuecomment-828576132,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AD6DTFA53LTAU6GFZTEYKV3TLAWTBANCNFSM43WBC35A
.

--
Regards,
Jason D. Siegrist
Google Voice # 757 95-2-NAVY (6289)
*sacram anchram solvere *

   -* as flying to the last refuge.


mobile, owner webapp: clicking on the won icon or menu button in topnav does not close the menu if menu is open

only reproducible if view is in mobile "style"

fixed


Add autocomplete for GitHub handles

I would like to see autocomplete for GitHub username @ handles.
Perhaps also add the other types of GitHub autocompletes.

Browser
Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/39.0.2171.95 Safari/537.36

:+1:
I've been missing that too.


02-load-data-with-power-query-in-power-bi-desktop.md

English:
click Standard, and then select Multiply.
Current Translation:
单击“标准”，然后选择“相乘”。
Suggested Translation:
单击“标准”，然后选择“乘”。

Hello contributor,
Thank you for your contribution! We are processing your suggestions and we will update you shortly.
Kind regards,
Microsoft Worldwide Learning Team

Fixed.


Fix that next-tier action also accepted packets.

Previously, we were setting the accept mark at the start of
the profile chain and then unsetting it at the end of the chain
if no rules matched.  That was correct while the only returning
rule type was "allow", but it meant that when we added
"next-tier" it also implicitly accepted the packet.
Fixes #1014

I think the change is good, although it did take me a few minutes to figure out how it actually worked.
Do we maintain a description of how the rules are supposed to look anywhere?  If not we probably should.

@matthewdupre I think that code is pretty well commented; might help to start in _build_to_or_from_chain(), which documents what it's expecting from the profile chains.

I think it's OK for now, and I'll merge once we've got FV approval: what I meant was there's nowhere you can read 1-2 pages and understand what the rules are.  The micro commenting is fine, but I had to think for a bit about things like how multiple tiers fit into iptables, and build the 'output' picture in my head.  It's more about information density than presence.


chore(javascript): broken tests

🧭 What and Why
🎟 JIRA Ticket: -
Changes included:
seems like I merged with a broken test https://github.com/algolia/api-clients-automation/actions/runs/8020801101/job/21911507593, this PR tries to fix it

is it random ?

it shouldn't be but I just saw the PR was merged with the broken test because of the same matrix issue as usual