USHIFT-2157: Update osbuild packages from copr after previous version cleanup

/lgtm

/cherry-pick release-4.15


Fix overflow when ctx.host_current_height().revision_height < ctx.host_chain_history_size()

Closes: #685
Description

For contributor use:

[ ] Updated the Unreleased section of CHANGELOG.md with the issue.
[ ] If applicable: Unit tests written, added test to CI.
[ ] Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
[ ] Updated relevant documentation (docs/) and code comments.
[ ] Re-reviewed Files changed in the Github PR explorer.

Thanks for the review @adizere. I've addressed your suggestions.
I'm thinking that the tests for connection open ack were relying on wrong behaviour since they started failing with this PR. I tried to fix them in 4da1d34c9d0505a360060c69957394b9bb55690a but there's one (now commented out) I didn't figure out how.

Codecov Report

Merging #686 (9b6d29a) into master (b1b37f5) will increase coverage by 31.2%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##           master    #686      +/-   ##
=========================================
+ Coverage    13.6%   44.8%   +31.2%
=========================================
  Files          69     150      +81
  Lines        3752    9899    +6147
  Branches     1374       0    -1374
=========================================
+ Hits          513    4444    +3931
- Misses       2618    5455    +2837
+ Partials      621       0     -621

Impacted Files
Coverage Δ

modules/src/address.rs
100.0% <ø> (ø)

...application/ics20_fungible_token_transfer/error.rs
0.0% <ø> (ø)

...ion/ics20_fungible_token_transfer/msgs/transfer.rs
0.0% <ø> (ø)

modules/src/events.rs
0.0% <ø> (ø)

modules/src/handler.rs
100.0% <ø> (ø)

modules/src/ics02_client/client_def.rs
48.3% <ø> (ø)

modules/src/ics02_client/client_type.rs
79.1% <ø> (+31.5%)
:arrow_up:

modules/src/ics02_client/context.rs
100.0% <ø> (ø)

modules/src/ics02_client/error.rs
100.0% <ø> (ø)

modules/src/ics02_client/events.rs
15.7% <ø> (+15.7%)
:arrow_up:

... and 255 more

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 ae36bba...f4e8c27. Read the comment docs.

I'm thinking that the tests for connection open ack were relying on wrong behaviour since they started failing with this PR. I tried to fix them in 4da1d34 but there's one (now commented out) I didn't figure out how.

The test that was commented out is expecting a MissingLocalConsensusState to be thrown here: https://github.com/informalsystems/ibc-rs/blob/vitor/overflow/modules/src/ics03_connection/handler/verify.rs#L167-L170
However, before this, we're already checking if the height is stale here: https://github.com/informalsystems/ibc-rs/blob/vitor/overflow/modules/src/ics03_connection/handler/verify.rs#L200-L203
If the height is not stale, then the consensus state must exist, in which case the error MissingLocalConsensusState should instead be a panic. If you agree with this analysis, I'll replace the error with a panic in a follow-up PR.

I'm thinking that the tests for connection open ack were relying on wrong behaviour since they started failing with this PR. I tried to fix them in 4da1d34 but there's one (now commented out) I didn't figure out how.

The test that was commented out is expecting a MissingLocalConsensusState to be thrown here: https://github.com/informalsystems/ibc-rs/blob/vitor/overflow/modules/src/ics03_connection/handler/verify.rs#L167-L170
However, before this, we're already checking if the height is stale here: https://github.com/informalsystems/ibc-rs/blob/vitor/overflow/modules/src/ics03_connection/handler/verify.rs#L200-L203
If the height is not stale, then the consensus state must exist, in which case the error MissingLocalConsensusState should instead be a panic. If you agree with this analysis, I'll replace the error with a panic in a follow-up PR.

I agree with your analysis. To summarize: in conn open try & ack handlers there are two verifications on the field msg.proofs.consensus_proof().height as follows:

first we verify for staleness with check_client_consensus_height, i.e., verify that this height is not older than the oldest height of the host (local) chain, nor newer than the current height of the host chain.
second we verify that the consensus state for this height exists in the local chain, as part of verify_consensus_proof.

I think the first verification is more of a fail-fast. While the second verification is a pre-check as part of the whole ensemble of proof verification. I agree there is some redundancy there. I would suggest to open an issue documenting this, but I wouldn't fix things with a panic (we have no panics in the modules yet, and not sure if it's a good idea to fix it like that).

I've updated #698. @adizere please confirm that it reflects the discussion here.
Regarding panics, there are already some unwraps, which can also panic. My view is that both are okay as long as they are prefixed with some explanation why there are "safe".


cant use in local network

hi
i see the login page from other pc but cant login
can you help me?

Maybe you should check server/server/settings.py, in line 96, CORS_ALLOWED_ORIGINS. The List contains the ip addresses that are allowed to do CORS. You should add the ip of your other pc in it, and the issue might be solved😀

hi i see the login page from other pc but cant login can you help me?

Maybe you should check server/server/settings.py, in line 96, CORS_ALLOWED_ORIGINS. The List contains the ip addresses that are allowed to do CORS. You should add the ip of your other pc in it, and the issue might be solved😀


Not really sync

I have the problem, I have a very big file, and if I search for a title, I see this on the left side, but not on the right. I have to scroll 2-3 screens to the the title.

is there a big code section or image?  It cannot sync when they are on the
right.  It doesn't know what to do until it gets past the code or image.
On Wed, May 20, 2015 at 3:35 AM, Alexander Frischbutter <
<EMAIL_ADDRESS>wrote:

I have the problem, I have a very big file, and if I search for a title, I
see this on the left side, but not on the right. I have to scroll 2-3
screens to the the title.
—
Reply to this email directly or view it on GitHub
https://github.com/mark-hahn/markdown-scroll-sync/issues/17.

this is an api documentation, so there are a lot of examples of code. no images.

Can you see that the problem happens around blocks of code?  Or is it
something else?
On Wed, May 20, 2015 at 11:54 AM, Alexander Frischbutter <
<EMAIL_ADDRESS>wrote:

this is an api documentation, so there are a lot of examples of code. no
images.
—
Reply to this email directly or view it on GitHub
https://github.com/mark-hahn/markdown-scroll-sync/issues/17#issuecomment-103994411
.

I think so, the document use a lot of
{
    "blabs": [
        {
            "id": 1,
            "name": "Host",
            "layer": "0",
            "blabs": "10793111",
            "top": "111",
            "nologin": false,
            "ipv4": "<IP_ADDRESS>/32",
            "blabs1": "unknown",
            "icon": "stub",
            "type": "Host",
            "left": "222",
            "mtu": "stub"
        },
	    [...]
}

I unfortunately I can send you the document

The markdown preview is rendered as a complete read-only text editor.  The dom is very complex.  It would be very hard to match scrolling to the inside of code.  I will mark this as an enhancement.

I've released a new version 2.0.1 that is a total rewrite.  I think it fixes all issues and works much better than before.  Please try it.


Matrix debug print tests

Current coverage is 100% (diff: 100%)

Merging #5 into master will increase coverage by 8.10%

@@             master    #5   diff @@
=====================================
  Files             6     6
  Lines           333   335     +2
  Methods           0     0
  Messages          0     0
  Branches          0     0
=====================================
+ Hits            306   335    +29
+ Misses           27     0    -27
  Partials          0     0

Powered by Codecov. Last update 23d3178...1aa09f6


[WIP] Add Settings Inspector plugin

Depends on #279

UI needs polishing, but the "inspect" part already works.


V4.0/fixtxvars

This PR contains four modifications:

set the correct actions order at the SecAction id 900990
remove paranoia-level/N tags from rules where nolog action presented
set the correct anomaly_score_plN variable for its PL
set the correct paranoia-level/N tag for its PL

After approve it, I can go away to fix the #2835.

Ops, I've already sent this PR... #2832


Text/Code mode

Should be able to work in non-code files via copy/paste editing

Added a modify_text tool that's visible when PSI change tools aren't supported


RAMP: Rename packages

We need to rename all packages in this bundle:

databoard will only contain frontend (website) tools, I am proposing ramp-website then.
ramp-database is already good.
ramp-engine is a not a good name. It contains the dispatcher and the workers implementations. Therefore I am thinking in ramp-launcher.
ramp-workflow should be renamed into ramp-sdk.
the current directory should be renamed from ramp-board to ramp. We have to check the name collision is actually an issue.

I would use ramp-trainer, since it's more specific. Even though it's train/(valid)test/score/blend, trainer gives me a better idea of what it does.

 The naming would require to be quite general then.

You mean not a big fan :). I would name it trainer for the exact reason you are not a big fan: it's a generic techno but we intend to use it for training ML models, so naming is the only way to express that. You don't want to give the scope artificially large cause the issues coming with people using it for something else will multiply exponentially.

We can reopen the issue if the naming is not good enough. But at least the imports are consistent.


Fix literal object initialization in CBC_INIT_LOCAL

Due to default arguments the lexical binding may exists before the literal object initialization.
This patch fixes #3396.
JerryScript-DCO-1.0-Signed-off-by: Robert Fancsik<EMAIL_ADDRESS>

function f(x = eval("var y = 5")) {
  console.log(y);
}
f();

Firefox says y is not defined. Nor inside and neither outside of the function. This needs a much deeper investigation.

Even more interesting:
function f(a = eval("var y = 5; console.log(y)"), b = y) {
}
f();

prints 5, and throws ReferenceError: y is not defined
So this looks like this is a strict eval even if we are not in strict mode.

@zherczeg Indeed, that's probably not the correct fix for the issue. Closing the PR for now.


authenticate without having public URL

Is there anyway to authenticate with auth2, without requiring a redirect URL? So this can be done server-side?

Yes, it's called a Service Account. Please read the following and see if that fits your needs: https://developers.google.com/accounts/docs/OAuth2ServiceAccount


No emoji available in chrome

When using the chrome extension the following warning appears:

I know the emoji are a limitation in chrome, but is it not possible to import a font with the emojis and used it?
Cheers! :beers:

Chrome extensions with emoji listings may hint the realisation (I use https://chrome.google.com/webstore/detail/emoji-cheatsheet-for-gith/nojknakoailnpbjlmfkpbbhoodlolfbh?utm_source=chrome-app-launcher-info-dialog)

Is this on Windows? Unfortunately, Chrome on Windows doesn't support native emoji and websites don't have a commonly supported format for emoji. I'm hopeful that emoji support for Windows will come soon!

Definitely not Windows-specific, the same ('Your browser does not support emoji') happens on Ubuntu.

I have the same concern...


[SPARK-19516][DOC] update public doc to use SparkSession instead of SparkContext

What changes were proposed in this pull request?
After Spark 2.0, SparkSession becomes the new entry point for Spark applications. We should update the public documents to reflect this.
How was this patch tested?
N/A

cc @sameeragarwal @hvanhovell

Test build #72590 has started for PR 16856 at commit 18d6daa.

Test build #72590 has finished for PR 16856 at commit 18d6daa.

This patch fails to build.
This patch merges cleanly.
This patch adds no public classes.

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/SparkPullRequestBuilder/72590/
Test FAILed.

several questions:

as SparkSession becomes the new entry point, now we require users to include spark-sql module in their sbt file or pom.xml, which seems weird if users only want to write a very simple spark application without SQL functionality. Shall we rename spark-sql to a more general name like spark-structure?
when it comes to java application with RDD, it's inconvenient to build SparkSession and then create JavaSparkContext, users can create JavaSparkContext directly.

cc @rxin @yhuai

Test build #72618 has started for PR 16856 at commit a5b91fd.

Test build #72618 has finished for PR 16856 at commit a5b91fd.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

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/SparkPullRequestBuilder/72618/
Test PASSed.

I think the issue is that the programming guide should probably switch over to the DataFrame one as the primary one, and then the RDD one as a RDD programming guide.
cc @matei for his input.

Test build #73306 has started for PR 16856 at commit 0ef8b2b.

Test build #73306 has finished for PR 16856 at commit 0ef8b2b.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

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/SparkPullRequestBuilder/73306/
Test PASSed.

Test build #73615 has started for PR 16856 at commit d989c68.

Test build #73615 has finished for PR 16856 at commit d989c68.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
* For other Hadoop InputFormats, you can use the SparkContext.hadoopRDDmethod, which takes an arbitraryJobConfand input format class, key class and value class. Set these the same way you would for a Hadoop job with your input source. You can also useSparkContext.newAPIHadoopRDD for InputFormats based on the \"new\" MapReduce API (org.apache.hadoop.mapreduce).
* For other Hadoop InputFormats, you can use the JavaSparkContext.hadoopRDDmethod, which takes an arbitraryJobConfand input format class, key class and value class. Set these the same way you would for a Hadoop job with your input source. You can also useJavaSparkContext.newAPIHadoopRDD for InputFormats based on the \"new\" MapReduce API (org.apache.hadoop.mapreduce).
A [Converter](api/scala/index.html#org.apache.spark.api.python.Converter) trait is provided
Note that while it is also possible to pass a reference to a method in a class instance (as opposed to
class MyClass
class MyClass
class GetLength implements Function<String, Integer>
class Sum implements Function2<Integer, Integer, Integer>
Note that while it is also possible to pass a reference to a method in a class instance (as opposed to
class MyClass(object):
class MyClass(object):
class VectorAccumulatorV2 extends AccumulatorV2[MyVector, MyVector]
class VectorAccumulatorV2 implements AccumulatorV2<MyVector, MyVector>
class VectorAccumulatorParam(AccumulatorParam):

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/SparkPullRequestBuilder/73615/
Test PASSed.

Test build #73864 has started for PR 16856 at commit e595fd0.

Test build #73864 has finished for PR 16856 at commit e595fd0.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
* For other Hadoop InputFormats, you can use the SparkContext.hadoopRDDmethod, which takes an arbitraryJobConfand input format class, key class and value class. Set these the same way you would for a Hadoop job with your input source. You can also useSparkContext.newAPIHadoopRDD for InputFormats based on the \"new\" MapReduce API (org.apache.hadoop.mapreduce).
* For other Hadoop InputFormats, you can use the JavaSparkContext.hadoopRDDmethod, which takes an arbitraryJobConfand input format class, key class and value class. Set these the same way you would for a Hadoop job with your input source. You can also useJavaSparkContext.newAPIHadoopRDD for InputFormats based on the \"new\" MapReduce API (org.apache.hadoop.mapreduce).
A [Converter](api/scala/index.html#org.apache.spark.api.python.Converter) trait is provided
Note that while it is also possible to pass a reference to a method in a class instance (as opposed to
class MyClass
class MyClass
class GetLength implements Function<String, Integer>
class Sum implements Function2<Integer, Integer, Integer>
Note that while it is also possible to pass a reference to a method in a class instance (as opposed to
class MyClass(object):
class MyClass(object):
class VectorAccumulatorV2 extends AccumulatorV2[MyVector, MyVector]
class VectorAccumulatorV2 implements AccumulatorV2<MyVector, MyVector>
class VectorAccumulatorParam(AccumulatorParam):

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/SparkPullRequestBuilder/73864/
Test PASSed.

Test build #74077 has started for PR 16856 at commit e284665.

Test build #74077 has finished for PR 16856 at commit e284665.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

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/SparkPullRequestBuilder/74077/
Test PASSed.

Test build #74120 has started for PR 16856 at commit c69d4c5.

Test build #74121 has started for PR 16856 at commit 34c75d4.

Test build #74120 has finished for PR 16856 at commit c69d4c5.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

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/SparkPullRequestBuilder/74120/
Test PASSed.

Test build #74121 has finished for PR 16856 at commit 34c75d4.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

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/SparkPullRequestBuilder/74121/
Test PASSed.

thanks for the review, merging to master!

@cloud-fan @sameeragarwal There are quite a few links in rest of Spark docs broken because of the name change from programming-guide -> rdd-programming-guide name change.

ah, sorry for that, let me do a grep and fix them.

Don't worry. I am fixing it. I am having a PR soon.
On Jun 29, 2017 8:08 PM, "Wenchen Fan"<EMAIL_ADDRESS>wrote:

ah, sorry for that, let me do a grep and fix them.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/apache/spark/pull/16856#issuecomment-312163114, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAoerFZuPRUEJg-B6lnSs0Rq-oP09Exuks5sJGatgaJpZM4L68zq
.

My PR to fix them https://github.com/apache/spark/pull/18485


TypeError: Cannot read property 'then' of undefined

https://github.com/phonegap/phonegap-cli/blob/4b0b681735679c6438c991227cb94d9c5b82021e/lib/phonegap/serve.js#L100
This returns nothing and
https://github.com/phonegap/phonegap-cli/blob/4b0b681735679c6438c991227cb94d9c5b82021e/lib/phonegap/serve.js#L87
expects a promise.

Did you ever get this working. I have this problem on AWS Cloud9 when developing
$ phonegap serve -b <IP_ADDRESS> -p 8080
   [error] project directory could not be found
There was an unhandled exception within phonegap-cli! If you would like to help the PhoneGap project, please file the following details over at https://github.com/phonegap/phonegap-cli/issues
TypeError: Cannot read property 'then' of undefined
    at AppCommand.run (/home/ec2-user/.nvm/versions/node/v6.14.4/lib/node_modules/phonegap/lib/phonegap/serve.js:87:43)
    at PhoneGap.<anonymous> (/home/ec2-user/.nvm/versions/node/v6.14.4/lib/node_modules/phonegap/lib/phonegap/util/command.js:28:25)
    at CLI.module.exports (/home/ec2-user/.nvm/versions/node/v6.14.4/lib/node_modules/phonegap/lib/cli/serve.js:36:14)
    at CLI.module.exports [as argv] (/home/ec2-user/.nvm/versions/node/v6.14.4/lib/node_modules/phonegap/lib/cli/argv.js:111:17)
    at runPhoneGapCommand (/home/ec2-user/.nvm/versions/node/v6.14.4/lib/node_modules/phonegap/bin/phonegap.js:46:9)
    at Object.<anonymous> (/home/ec2-user/.nvm/versions/node/v6.14.4/lib/node_modules/phonegap/bin/phonegap.js:38:5)
    at Module._compile (module.js:577:32)
    at Object.Module._extensions..js (module.js:586:10)
    at Module.load (module.js:494:32)
    at tryModuleLoad (module.js:453:12)

I was just pointing out that this was bad code. It's meant to just say "[error] project directory could not be found" and then exit with a failure status.
Instead it fails to return a promise, so it creates an unhandled exception afterwards which is just noise. Not a big issue, but I thought I'd point it out as it makes a confusing message.
Your problem is just that the it can't find a cordova (phonegap) project from the directory it's being run in. It starts in the current working directory and then continues up the file system. It ships out to the 'cordova' package to run the actual check of what it considers to be a cordova project. I'd guess that it looks for a 'config.xml' file, but I haven't looked that up or anything.

The same situation, I think the problem is in the installation defect related with that npm had no permission to write data into the specified folders.
Open the specified "lib" folder. Does it contains the specified folder "node_modules"?
In my case it is not.

"you should run your commands from your actual project folder. Common mistake is that people try out of the project folder. use "cd folder" and try. – Purus Nov 25 '13 at 18:55"
https://stackoverflow.com/questions/20197542/where-phonegap-defines-project-directory
So:
cd myApp
phonegap serve


Haptic feedback tweaks

📲 What
The tweak is to add haptic feedback when save/heart button on project card and project nav bar is tapped.
🤔 Why
It is ideal.
🛠 How
Created an action method for .touchDown.
✅ Acceptance criteria
Recommended Device: iPhone 7 (or newer)
On DiscoveryPostcardCell
Test Tap and release

[ ] Tap heart/save button and release
[ ] haptic feedback generates on touch down
[ ] further haptic feedback generates on release

Test Tap and pan away from touch area

[ ] Tap heart/save button and pan away from touch area
[ ] haptic feedback should generate on touch down
[ ] no further haptic feedback should generate on release

On ProjectNavBar
Test Tap and release

[ ] Tap heart/save button and release
[ ] haptic feedback generates on touch down
[ ] further haptic feedback generates on release

Test Tap and pan away from touch area

[ ] Tap heart/save button and pan away from touch area
[ ] haptic feedback should generate on touch down
[ ] no further haptic feedback should generate on release

@justinswart g2g?


Documentation issues

Architecture PowerMatcher
Too complex to start the story (in my option):
Separation - avoid crowding neighbors (short range repulsion)
Alignment - steer towards average heading of neighbors
Cohesion - steer towards average position of neighbors (long range attraction)
Tests is missing!
SimplePMCluster: OSGi components picture missing or broken
AdvancedPMCluster is missing!
CreationOfDeviceAgent is not complete!
PM security guidelines LINKS are missing

Most items are fixed, AdvancedPMCluster is missing will be reported in the powermatcher documentation branch.


Allow download of recorded media

recorded photos and videos shoot be made available for download

see https://github.com/vert-x3/vertx-web/issues/204 for the problems with this

Full disk access is disabled by default. This was a design decision to secure user servers, a badly configured server running with a privileged user (root for example) could just allow the full /etc directory to be dumped or even non application related data.
After careful analysis is up to the developer to allow it or not with:
StaticHandler.create()
  .setAllowRootFileSystemAccess(true)
  .setWebroot('/home/myuser')

@pmlopes - thx works like a charm ...


Dependencies Out Of Date - Issues On npm install

Please update dependencies:
"sauce-connect-launcher": "^0.12.0",
"saucelabs": "^0.1.1",
"selenium-standalone": "4.5.3",
"webdriverio": "^3.1.0"

Hi k1ngdr3w,
I have updated the dependencies, and also made a few changes to the structure of the configuration object. Everything in the README file has been updated to reflect this so you should be able to make the necessary changes.
All the npm tests seem to be passing and the example located in the examples folder also works.
Please let me know if you have any issues!

Sorry, I won't close it yet, so you have a chance to verify it...

https://github.com/alexbrombal/selenium-sauce/commit/931690dbbd8592dd0c729ada1fb808253bff1b90 looks good! Close it!


[chip] chip_sw_sleep_pin_wake

This PR implements the basics of PINMUX PIN Wakeup test. It does not fully randomize the sequence yet
TODOs

[x] Random delay between the low power enter and the PIN assertion (posedge)
[x] Random choice of the PADs (MIO vs DIO / GPIO vs other MIOs)

This is a little bit hard if we want to use virtual interface in vseq.

[ ] Precise monitoring in SV side. Currently SW checks if the chip has been wake up by pin wake or not
[ ] Random choice between the sleep and deep sleep
[ ] Multiple entry/exit without a chip reset (POR_N)

Related Issue: https://github.com/lowRISC/opentitan/issues/14160

"Random delay between the low power enter and the PIN assertion (posedge)" has been addressed in the second commit.

I think status-quo of this PR is OK to be merged. After Sri merged the PAD driver logic in DV side, I will revise the sequence.
OR we can merge this and address the sequence later. WDYT @tjaychen @sriyerg ?

I think status-quo of this PR is OK to be merged. After Sri merged the PAD driver logic in DV side, I will revise the sequence.
OR we can merge this and address the sequence later. WDYT @tjaychen @sriyerg ?

that sounds okay to me.

I think status-quo of this PR is OK to be merged. After Sri merged the PAD driver logic in DV side, I will revise the sequence.
OR we can merge this and address the sequence later. WDYT @tjaychen @sriyerg ?

that sounds okay to me.

wait actually..hmmm maybe it's worth waiting. Because Sri has to fix a bunch of tests already as part of this PR, this will add another one on.  If Sri thinks he can merge the tb pin thing very soon, maybe it's better to wait? so as to not add the burden over there.  I might do the same with the i2c test.

I think status-quo of this PR is OK to be merged. After Sri merged the PAD driver logic in DV side, I will revise the sequence.
OR we can merge this and address the sequence later. WDYT @tjaychen @sriyerg ?

that sounds okay to me.

wait actually..hmmm maybe it's worth waiting. Because Sri has to fix a bunch of tests already as part of this PR, this will add another one on. If Sri thinks he can merge the tb pin thing very soon, maybe it's better to wait? so as to not add the burden over there. I might do the same with the i2c test.

Yep. That's what I discussed with Sri. So better to wait until then.

OK. Thanks for all helpful comments!
Now, the test is passing (for 100 reseeds). So I am sure this can be merged (ofc after getting LGTM and approval).

Anyone can approve if OK?

As Sri is absent, can @weicaiyang , @cindychip check and approve?


git node land just hangs

I am unable to land PRs since a few months ago. I've tried issuing new credentials as well as different PRs. This happens for me on every PR.

Logs & Details

NCU_VERBOSITY=debug git node land 41170

[DEBUG] [SpawnSync] git config --get remote.upstream.url { cwd: '/Users/panva/repo/node' }

[DEBUG] [SpawnSync] git rev-parse --abbrev-ref HEAD { cwd: '/Users/panva/repo/node' }

[DEBUG] [SpawnSync] git rev-parse --abbrev-ref HEAD { cwd: '/Users/panva/repo/node' }

[DEBUG] [fetch] https://api.github.com/graphql

[DEBUG] [SpawnSync] git config --get remote.upstream.url { cwd: '/Users/panva/repo/node' }
⠋ Loading data for nodejs/node/pull/41170
[DEBUG] [fetch] https://raw.githubusercontent.com/nodejs/node/HEAD/README.md

[DEBUG] [fetch] https://api.github.com/graphql

[DEBUG] [fetch] https://api.github.com/graphql

[DEBUG] [fetch] https://api.github.com/graphql

[DEBUG] [fetch] https://api.github.com/graphql
✔  Done loading data for nodejs/node/pull/41170
----------------------------------- PR info ------------------------------------
Title      tls: permit null as a pfx value (#41170)
   ⚠  Could not retrieve the email or name of the PR author's from user's GitHub profile!
Branch     CallMeLaNN:fix/pfx-cant-load -> nodejs:master
Labels     tls, author ready, commit-queue-failed
Commits    2
 - tls: permit null as a pfx value
 - tls: permit null as an options value
Committers 1
 - CallMeLaNN<EMAIL_ADDRESS>------------------------------ Generated metadata ------------------------------
PR-URL: https://github.com/nodejs/node/pull/41170
Fixes: https://github.com/nodejs/node/issues/36292
Reviewed-By: Matteo Collina<EMAIL_ADDRESS>Reviewed-By: Filip Skokan<EMAIL_ADDRESS>--------------------------------------------------------------------------------
   ℹ  This PR was created on Tue, 14 Dec 2021 13:57:39 GMT
   ✔  Approvals: 2
   ✔  - Matteo Collina (@mcollina) (TSC): https://github.com/nodejs/node/pull/41170#pullrequestreview-832783699
   ✔  - Filip Skokan (@panva): https://github.com/nodejs/node/pull/41170#pullrequestreview-832785694
   ✔  Last GitHub Actions successful
   ℹ  Last Full PR CI on 2021-12-17T09:42:04Z: https://ci.nodejs.org/job/node-test-pull-request/41511/
⠋ Querying data for job/node-test-pull-request/41511/   ✔  Last Jenkins CI successful
--------------------------------------------------------------------------------
⠴ Querying API for job/node-test-pull-request/41511/

I thought there was already an issue for it.
This happens for everyone, I think, but it doesn't prevent you from landing PRs.
The issue is that you have to answer a question, but it is not displayed. Just press Y + Enter and it should work.

Ok, what is the question I'm answering? :)

OK.


Add support for alternative to MySQL

Having to install Maria/MySQL feels heavy. I'd like to enable the use of Sqlite as an alternative.


plugin card next fix

@donavanbecker did you merge this? and release alpha with this? because i dont see changes in alpha 53

@donavanbecker So something went wrong because i dont have changes.

Ok I will do PR again


Identify other codeowners

Maybe add @FranBonath to content/events/, and maybe @nf-core/outreach or core to content/about?

* @mashehu


unable to connect to Polarion Gen 2

Hello Team,
was trying to connect to Polarion gen 2 link using python 3.8.2. but keep getting
requests.exceptions.SSLError: HTTPSConnectionPool(host='polariong2.com', port=443): Max retries exceeded with url: /polarion/ws/services (Caused by SSLError(SSLCertVerificationError(1, '[SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: unable to get local issuer certificate (_ssl.c:1108)')))

You seem to use a self-signed SSL certificate. You can skip the certificate validation by setting the parameter skip_cert_verification in the Polarion object constructor.
polarion = polarion.Polarion(url, username, password, skip_cert_verification=True)

@Futsch1 I tired your solution but it says below.
Traceback (most recent call last):
client = polarion.Polarion(,skip_cert_verification=True)
TypeError: init() got an unexpected keyword argument 'skip_cert_verification'

Have a look at the discussion here: https://github.com/jesper-raemaekers/python-polarion/issues/44

I copied latest code in local and worked fine. Hope the pip installation issue will be resolved son So I can fetch latest using pip.

This is now available using pip from version 0.3.0.


Add a check option called "private" to the track window of Anilist

Describe your suggested feature

This is a third party app of Anilist and when i add a manga (or anime) the app makes avaiable this option letting us choose if the users want to keep it private or not.
Now, i will say it; this option is perfect for an hentai dedicated fork of tachiyimi since users might want to hide 'their dirt' keeping them hidden from the public, but still wanting them on their list.
Other details
the app of the screenshot is Anilib and it is on github if you need the code as a reference or something.
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open issue.
[X] I have written a short but informative title.
[X] If this is an issue with an extension, I should be opening an issue in the extensions repository.
[X] I have updated the app to version 1.8.3.
[X] I will fill out all of the requested information in this form.

I don't use tracking so I'm not too interested in this. Open a issue in the main Tachi repo since SY will get it by default if main Tachi gets it.


Feature request: Time to next step

overview

The currently the "Run time" is displayed when running a protocol.

It is known that this time does not actually correspond to the time from when the user pushed start protocol.  It would be nice to have a "time to next step" or a "time to protocol is complete" timer that counts down.
current behavior

Currently timer does not really display a "useful to the user" time.
steps to reproduce

Load any protocol, calibrate, and push start.
expected behavior

I would like to have a display of the time until the next step in the protocol happens, and a time until the protocol is completed.  In terms of precision of the time measurement, I would like to know if I can go get a coffee, or if I need to sit and watch the protocol go.

@mcous Thanks.  I's guessing the dynamic time needed to fix this error 2822 would be the time that needs to be displayed.


rpmbuild 4.17.0 is slow and stuck at Processing files for a long time

Describe the bug
Mariner 2.0 (https://github.com/Azure/azure-cli/pull/21245), UBI 8 (https://github.com/Azure/azure-cli/pull/21655) and CentOS Stream 8 (https://github.com/Azure/azure-cli/pull/20918) have rpmbuild 4.17.0 which is very slow while building the RPM. The CI job BuildRpmPackageMariner and BuildRpmPackageUbi8 take ~40min.
Turning on -vv (verbose) for rpmbuild shows the process got stuck at these steps for a very long time:
Processing files: azure-cli-dev-1.el8.x86_64
...
D:      execv(/usr/lib/rpm/pythondeps.sh) pid 4341
D:      waitpid(4341) rc 4341 status 0
D:      execv(/usr/lib/rpm/pythondistdeps.py) pid 4346
D:      waitpid(4346) rc 4346 status 0
D:      execv(/usr/lib/rpm/pythondeps.sh) pid 4347
D:      waitpid(4347) rc 4347 status 0
D:      execv(/usr/lib/rpm/pythondistdeps.py) pid 4352
D:      waitpid(4352) rc 4352 status 0
D:      execv(/usr/lib/rpm/pythondeps.sh) pid 4353
D:      waitpid(4353) rc 4353 status 0

htop shows the same:

To Reproduce
Run the follow commands on WSL or Linux machine:
git clone https://github.com/Azure/azure-cli
docker run -it --rm -v $(pwd)/azure-cli:/root/azure-cli --workdir /root/azure-cli registry.access.redhat.com/ubi8/ubi:8.4
yum update -y
yum install -y wget rpm-build gcc libffi-devel python3-devel openssl-devel make bash diffutils patch dos2unix python3-virtualenv perl
REPO_PATH=$(pwd) CLI_VERSION=dev rpmbuild -v -bb --clean scripts/release/rpm/azure-cli.spec

Additional context
Email: azure-cli FIPS compliance
See:

https://www.cyberciti.biz/faq/how-to-install-htop-on-rhel-8-using-yum/

Also tried to use iotop.
I added Extra Packages for Enterprise Linux (EPEL) and followed tutorials like

https://www.cyberciti.biz/hardware/linux-iotop-simple-top-like-io-monitor/
https://aws-labs.com/install-iotop-monitor-linux-disk-io-rhel-centos-fedora/
https://lintut.com/install-iotop-on-linux/

but got no luck:
# yum install https://dl.fedoraproject.org/pub/epel/epel-release-latest-8.noarch.rpm
# yum update
# yum install iotop
Updating Subscription Management repositories.
Unable to read consumer identity

This system is not registered with an entitlement server. You can use subscription-manager to register.

Last metadata expiration check: 0:05:10 ago on Mon Mar 28 04:18:46 2022.
No match for argument: iotop
Error: Unable to find a match: iotop

There is a package called iotop-c but it shows empty screen (https://github.com/Tomas-M/iotop/issues/34):
# yum install iotop-c
# iotop-c

Actually, I installed iotop on fedora container (WSL2-based) without problem, but it fails with :
> docker run -it --rm fedora
# yum install iotop
# sudo iotop
Netlink error: No such file or directory (2)

The Linux kernel interfaces that iotop relies on for process I/O statistics
were not found. Please enable CONFIG_TASKSTATS in your Linux kernel build
configuration, use iotop outside a container and or share the host's
network namespace with the container.

Please do not file bugs on iotop about this.

Please enable CONFIG_TASKSTATS in your Linux kernel build configuration

This is Microsoft built WSL, I am not sure I can do that.

use iotop outside a container

Guess I need to bump up a VM for testing?

share the host's network namespace with the container

This is not possible on Windows: https://docs.docker.com/network/host/: The host networking driver only works on Linux hosts, and is not supported on Docker Desktop for Mac, Docker Desktop for Windows, or Docker EE for Windows Server.


Fix redundant imports when selecting options from the dropdown widget

Pull Request Description
Fixes #6816.
The code for adding imports for dropdown widgets was unified with CB. The code was moved from the searcher controller to the graph controller.
Also, I changed the signature for a few lookup_* methods of the suggestion database, because I have always found it weird that they return Option instead of Result. They now work nicely with the surrounding code.
https://github.com/enso-org/enso/assets/6566674/49125f8d-096e-4cca-a922-4811ed717a4d
Important Notes

Checklist
Please ensure that the following checklist has been satisfied before submitting the PR:

[x] The documentation has been updated, if necessary.
[x] Screenshots/screencasts have been attached, if there are any visual changes. For interactive or animated visual changes, a screencast is preferred.
[x] All code follows the
Scala,
Java,
and
Rust
style guides. In case you are using a language not listed above, follow the Rust style guide.
All code has been tested:

[x] Unit tests have been written where possible.
[x] If GUI codebase was changed, the GUI was tested when built using ./run ide build.

@vitvakatu What's the status here?

QA Status: 💚
Looks great, barely anything now gets imported when from Standard.Base import all is present. And when I manually remove that import, the appropriate gets correctly imported on demand.


Spark lda only

removed MPI mentions from install script and install documentation

this is showing way too many differences
will redo on a new branch


Get web element by xpath

From<EMAIL_ADDRESS>on April 16, 2013 05:23:33
What steps will reproduce the problem? 1. Have got xpath from the web structure
2. Call method solo.getWebElement(By.xpath()) in the webView
3. Receive error What is the expected output? What do you see instead? junit.framework.AssertionFailedError: WebElement with xpath: '//*[@id="nameOfelementID"]/tbody/tr[11]/td[2]/input[1]' is not found!
at com.jayway.android.robotium.solo.Solo.getWebElement(Solo.java:1832)
at com.ocleen.tv.test3.OcleenTestSearchActivity.testChotels(OcleenTestSearchActivity.java:731)
at java.lang.reflect.Method.invokeNative(Native Method)
at android.test.InstrumentationTestCase.runMethod(InstrumentationTestCase.java:214)
at android.test.InstrumentationTestCase.runTest(InstrumentationTestCase.java:199)
at android.test.ActivityInstrumentationTestCase2.runTest(ActivityInstrumentationTestCase2.java:192)
at android.test.AndroidTestRunner.runTest(AndroidTestRunner.java:190)
at android.test.AndroidTestRunner.runTest(AndroidTestRunner.java:175)
at android.test.InstrumentationTestRunner.onStart(InstrumentationTestRunner.java:555)
at android.app.Instrumentation$InstrumentationThread.run(Instrumentation.java:1599) What version of the product are you using? On what operating system? Robotium 4.1
Original issue: http://code.google.com/p/robotium/issues/detail?id=437

From<EMAIL_ADDRESS>on April 16, 2013 06:11:15
Thanks for reporting this. Would it be possible for you to share an application that exhibits this issue?
Owner<EMAIL_ADDRESS>

From<EMAIL_ADDRESS>on April 17, 2013 11:11:48
Probably something wrong with the xpath. We have had reports before about issues with xpath and the problems have always been the xpath strings.
Status: Invalid

From<EMAIL_ADDRESS>on April 17, 2013 14:37:12
I have tried several elements as I wrote you in a letter this web page was written in ASP.NET


Create a data_maturity label for EIA data

We added the early release eia data in #1834. This PR adds a data_maturity column into the EIA data tables to communicate with users about where the data comes from in the hope of communicating how much trust they should place in the permanence of the data.

This looks like some hackish stuff happening. Can you explain more in the PR comment what all you had to do to make it work?


Need support for Amazon cognito

Would be great to add support for Amazon Cognito an OAuth2 service.

@jvence sure let me take a look tomorrow

@jvence I've added the Amazon Cognito  in the sample project and it does not require any update of the core library.
Hope it resolves this ticket.


more singleObjectConvert is required

Coverage remained the same at 100.0% when pulling ea4b517eb9260448e63d3ff3aa581fbd3b143561 on moreConverter into 17e699711928162f3a4d2bc1d33a06213115d786 on master.


MDN documentation command

Adds a new command, docs!mdn <query> which returns results from anywhere in https://developer.mozilla.org/, not limited to JavaScript.
This is NOT achieved by scraping, not a single bit of HTML is even touched, the data to search is obtained by fetching MDN's sitemap at https://developer.mozilla.org/sitemaps/en-us/sitemap.xml.gz, which is cached in memory and refreshed once every 12 hours on demand. (Because of this, consistency between the bot's results and MDN's own search box cannot be guaranteed and more often than not WILL be different)
When only a single result is found, instead of a list of results, in-depth data of the result's page is fetched from its /index.json (see: https://developer.mozilla.org/en-US/docs/MDN/Tools/Unsupported_GET_API) and displayed as a website embed, except with a small thumbnail image instead of MDN's default website embeds which give a massive useless image that floods the Discord channel. More data from /index.json may be used in the future such as the compatibility table and specification references but for now they are not used.
This PR contains a few other smaller miscellaneous changes such as updating some dependencies to fix reported vulnerabilities, adding a start script to the package.json and fixing some minor typos etc.

Thank you for the PR!


Transition Elements between magic move elements

Is your feature request related to a problem? Please describe.
I would like to have a changing title in my slide based on the code that is shown right now. Having a title that is in sync with magic move would be really nice to provide additional context and information to the code shown. Code surfer has a similar feature, demonstrated here: https://codesurfer.pomb.us/full/
Describe the solution you'd like
I think having a syntax like this makes sense (at least to me) and could utilize view transitions for the headlines :
# Service with a Subject
```angular-ts
@Component({
  standalone: true,
  template: `
     <div *ngFor="let item of source$ | async"></div>
  `,
})
export class FooComponent {
  source$ = inject(Service).someMethod();
}

```
# Service with a Signal

```angular-ts
@Component({
  standalone: true,
  template: `
     <div *ngFor="let item of source()"></div>
  `,
})
export class FooComponent {
  source = toSignal(inject(Service).someMethod());
}
```

Describe alternatives you've considered
Using the current click system could be confusing for users if something is considered a "new click" or in sync with the magic move clicks

I believe it would be beneficial to add a component called MagicMove that implenents smooth transitions between slots, taking into account all elements and their children (such as code blocks, paragraphs, equations, divs, lists, blockquotes, etc.), similar to the Auto-Animate feature in Reveal.js (Auto-Animate). For your use case, it could be implemented in this manner (using MDC):
::MagicMove
#subject
# Service with a subject

```angular-ts
@Component({
  standalone: true,
  template: `
     <div *ngFor="let item of source$ | async"></div>
  `,
})
export class FooComponent {
  source$ = inject(Service).someMethod();
}
```
#signal
# Service with a Signal

```angular-ts
@Component({
  standalone: true,
  template: `
     <div *ngFor="let item of source()"></div>
  `,
})
export class FooComponent {
  source = toSignal(inject(Service).someMethod());
}
```
::

Closes this and opened #1527, because #1527 is more universal. Thanks for your ideas!


XSS issue on gosearchresults.com

I found a reflected XSS issue on gosearchresults.com
Steps to reproduce:
Call the following URL in Mozilla Firefox: https://lavasoft.gosearchresults.com/?sbtn=&q=ola<script>alert('I+can+send+any+message+here!')<%2Fscript>&tt=VM__GS__S4LAVA__vmn__webcompa__1_0__go__ch_WCYID10438__180722__yrff&pid=5ac784309091147a162b4431&sr=0
An alert box with "I can send any message here!" appears. This means that an attacker has full control of the scripts, that are executed in the victims browser.
An attack vector would be sending an evil link via e-mail, messenger, etc. As the victim trusts the domain gosearchresults.com, it will click the link and could be redirected to a site hosting a browser exploit kit.
This abuses the trust of gosearchresults.com
When i enter any script in URL bar then after clicking enter it encrypt the URL in encoded form but still script gets exicuted. This is a big problem by which a hacker can steal the cookies of victim's browser and also can redirect that user to any other site or search engine.

gosearchresults.com has nothing to do with go-search.org.


[Feature] add rubberwood to axe

Add the rubberwood from MFR to the dark axe so it can chop down the whole rubber tree.

that was fast :-) :+1:

just a sec, was that the rubberwood that grows those rubberthingies you tap? If it is the other one it should work already.

It's "the other one", the one you use the tap on IC2 rubber wood.

ok, then I have to have a look what's wrong. Thx.

Minefactory reloaded has a rubber tree. That one. No idea about the IC2 one


ran 'brew install node'

I am unable to understand the message about setting flags, while installing node by home brew. I am a beginner. so i dont know what these setting compiler flags mean.
Here is the full log:
` brew install node
==> Installing dependencies for node: icu4c
==> Installing node dependency: icu4c
==> Downloading https://homebrew.bintray.com/bottles/icu4c-61.1.high_sierra.bott
######################################################################## 100.0%
==> Pouring icu4c-61.1.high_sierra.bottle.tar.gz
==> Caveats
This formula is keg-only, which means it was not symlinked into /usr/local,
because macOS provides libicucore.dylib (but nothing else).
If you need to have this software first in your PATH run:
echo 'export PATH="/usr/local/opt/icu4c/bin:$PATH"' >> ~/.bash_profile
echo 'export PATH="/usr/local/opt/icu4c/sbin:$PATH"' >> ~/.bash_profile
For compilers to find this software you may need to set:
LDFLAGS:  -L/usr/local/opt/icu4c/lib
CPPFLAGS: -I/usr/local/opt/icu4c/include
==> Summary
🍺  /usr/local/Cellar/icu4c/61.1: 249 files, 67.2MB
==> Installing node
==> Downloading https://homebrew.bintray.com/bottles/node-10.4.0.high_sierra.bot
######################################################################## 100.0%
==> Pouring node-10.4.0.high_sierra.bottle.tar.gz
==> Caveats
Bash completion has been installed to:
/usr/local/etc/bash_completion.d
==> Summary
🍺  /usr/local/Cellar/node/10.4.0: 6,793 files, 59.6MB `
The message which is in bold.
Thanks in advance.

This is an issue with a formula. It belongs in Homebrew/core.
When you open the issue there, please use fenced code blocks.

This is a question regarding a formula. It belongs in Homebrew/core. Nevertheless, there’s no real issue here, you can ignore that message as it won’t affect you.
In the future, please use fenced code blocks for enclosing code.


chef-server-ctl reconfigure fails on AWS hosts because node.fqdn is undefined

The nginx service is unable to start because its template depends on node.fqdn:  https://github.com/chef/chef-server/blob/master/omnibus/files/private-chef-cookbooks/private-chef/templates/default/nginx/nginx.conf.erb#L98
but this is nil because:
root@ip-33-33-33-254:~# chef-shell
loading configuration: none (standalone session)
Session type: standalone
Loading.done.

This is the chef-shell.
 Chef Version: 12.3.0
 http://www.chef.io/
 http://docs.chef.io/

run `help' for help, `exit' or ^D to quit.

Ohai2u!
chef > node.fqdn
NoMethodError: Undefined method or attribute `fqdn' on `node'
	from /opt/chef/embedded/apps/chef/lib/chef/node/attribute.rb:468:in `method_missing'
	from /opt/chef/embedded/apps/chef/lib/chef/node.rb:229:in `method_missing'
	from (irb):1
	from /opt/chef/embedded/apps/chef/lib/chef/shell.rb:76:in `block in start'
	from /opt/chef/embedded/apps/chef/lib/chef/shell.rb:75:in `catch'
	from /opt/chef/embedded/apps/chef/lib/chef/shell.rb:75:in `start'
	from /opt/chef/embedded/apps/chef/bin/chef-shell:37:in `<top (required)>'
	from /usr/bin/chef-shell:51:in `load'
	from /usr/bin/chef-shell:51:in `<main>'
chef > `hostname -f`
hostname: Name or service not known
 => ""

In my experience you must set an FQDN before you can reconfigure successfully.

@irvingpop Ihave seen that, you need to add an entry to the /etc/hosts. it is a problem with how OHAI is trying to get the fqdn from the box.

Wouldn't it be better to fix Ohai ?

hostname -f performs a reverse lookup (against /etc/hosts, DNS and NIS)
hostname -A performs a reverse lookup, but for all interfaces (possibly returning multiple names)
hostname  with no params returns the configured hostname (without performing /etc/hosts, DNS or NIS lookups)

In pseudo code:
def my_fqdn
  if shell("hostname -f").exitcode == 0
    return shell("hostname -f).output.chomp
  else
    return shell("hostname").output.chomp
  end
end

I kinda like the proposal from Mr @trickyearlobe ..

This issue is also being discussed in https://github.com/chef/chef-server/issues/21
For now we urgently need to get users unblocked who are trying this on aws


Create helloworld.pas

This Pull Request resolves Issue #606
Description:
In this Pull Request we have done the following additions/edits for the code:

PR closed due to inactivity


Updates Spatial Documentation

Updates the Spatial documentation.
Note that I removed the SQL fragments because the translation shown only works for Postgis, and it is not accurate as a description of how Spatial actually works.

Cool! I'm going to integrate it next week.

I integrated it on 5.0, 5.1, and master.


Add persistent cache for toolchain auto-detection mechanism

Context
The Daemon toolchain auto-detection mechanism is part of Phase 1 of toolchain support for Gradle Daemon, where the motivation behind it and other technical details can be found on the public spec document. This PR implements the persistent cache for the toolchain auto-detection mechanism by adding indexed cache on CachingJvmMetadataDetector storing the JvmInstallationMetadata resolved object based on toolchain installation path. This cache is a VersionStrategy.CachePerVersion allowing to have a separate cache instance for each Gradle version that are stored under Gradle User Home directory.
Testing
Unit tests:

CachingJvmMetadataDetectorTest.groovy
InvalidJvmInstallationCacheInvalidatorTest.groovy

Integration tests

CachingJvmMetadataDetectorIntegrationTest.groovy
InvalidJvmInstallationReportingIntegrationTest.groovy
DaemonToolchainIntegrationTest.groovy

Demo
Simple demo exposing how persistent cache for toolchain auto-detection mechanism works:
https://github.com/vmadalin/gradle/assets/18151158/de8ccb91-b02f-47aa-9624-3689040adf79
Contributor Checklist

[x] Review Contribution Guidelines
[x] Make sure that all commits are signed off to indicate that you agree to the terms of Developer Certificate of Origin.
[x] Make sure all contributed code can be distributed under the terms of the Apache License 2.0, e.g. the code was written by yourself or the original code is licensed under a license compatible to Apache License 2.0.
[x] Check "Allow edit from maintainers" option in pull request so that additional changes can be pushed by Gradle team
[x] Provide integration tests (under <subproject>/src/integTest) to verify changes from a user perspective
[x] Provide unit tests (under <subproject>/src/test) to verify logic
[x] Update User Guide, DSL Reference, and Javadoc for public-facing changes
[x] Ensure that tests pass sanity check: ./gradlew sanityCheck
[x] Ensure that tests pass locally: ./gradlew <changed-subproject>:quickTest

Closing PR since this was created directly on Gradle https://github.com/gradle/gradle/pull/28966 after addressing merge conflicts with partially merged Daemon toolchain implementation in 8.8


Cell to cell caching results

I’ve loaded up the latest anaconda 3.7 distro and I’m running notebook 6.0.0
I’m working with the Powershell kernel & Firefox 68 for a browser.
When I run one cell then go run the next it shows the output from the previous cell. When I run it a second time it then shows the correct output.
I’ve tried 3 different browsers, turned off caching, and even ran the no-f*cking-cache-browser but it still happens.

I just tried the console and it seems to be doing the same thing. Is it something like missing CR/LF? My Linux stuff doesn't have this issue and it uses just "LF" so maybe due to this being powershell on windows it is waiting for a "CR+LF" as windows uses both (Mac uses just CR, Unix just LF).

I may have found a work around. When I had these 3 lines in one code cell the third command would cause the problem. Once I put them all in their own cells it now works:
New-Item -ItemType Directory -Path "C:\Users\Administrator\Documents\WindowsPowershell"
Set-Location C:\Users\Administrator\Documents\WindowsPowershell
New-Item -ItemType file -force $profile
If I run them all on one line separated by semi-colons it also works. It also runs fine if I put it in a PS1 script. Only when the 3 lines are in one cell does the "caching" happen and you have to run following cells twice to get the correct result.
I'll close this one but leave this here in case someone else stumbles across this behaviour.

Oh, I think it's a bug in the kernel. It uses a crappy heuristic to identify the end of the output for a particular cell. Let me keep it open for investigation.

Okay thanks. In the mean time the work around is holding up. For very long hash tables where I want more than one line I go with a script.
Thanks for all the work on this. I wish Microsoft would do a Powershell kernel for Notebook.

Oh, I think it's a bug in the kernel. It uses a crappy heuristic to identify the end of the output for a particular cell. Let me keep it open for investigation.

I can definitely reproduce this issue.  I'd love to get it solved.

I can reproduce it too. The issue sometimes occurs even when using simple commands. For example, generally speaking if I start from a fresh kernel, if I execute Set-Location in one code cell, then execute Get-Location in the next, I get the expected behavior (i.e. no output from the Set-Location invocation, but the current directory gets output by Get-Location). However, sometimes executing the Get-Location cell show nothing, but then executing the Set-Location command again shows the result from the invocation of the previous Get-Location.
I'm running Python 3.7.3 (installed via Anaconda).

Fixed in #17


Debug assertion on too-wide, empty, right-aligned table cell

Description

When trying to render a particular table which is wider than the available area, ratatui panics on a debug assertion. This is similar to #474, but with a somewhat different backtrace.
The following seems to need to be true for the repro below to work:

Terminal should be narrower than the table's width constraint (e.g., 140 in the example)
There should be an empty, right-aligned line in the table
Also, using Line::raw in place of <Line as From<&str>>::from doesn't work; I suspect that's because Line::raw uses str::lines to normalize its content

thread 'main' panicked at 'Trying to access position outside the buffer: x=138, y=0, area=Rect { x: 0, y: 0, width: 138, height: 45 }', /home/wbain/.cargo/registry/src/index.crates.io-6f17d22bba15001f/ratatui-0.24.0/src/buffer.rs:240:9
stack backtrace:
   0: rust_begin_unwind
             at /rustc/eb26296b556cef10fb713a38f3d16b9886080f26/library/std/src/panicking.rs:593:5
   1: core::panicking::panic_fmt
             at /rustc/eb26296b556cef10fb713a38f3d16b9886080f26/library/core/src/panicking.rs:67:14
   2: ratatui::buffer::Buffer::index_of
             at /home/wbain/.cargo/registry/src/index.crates.io-6f17d22bba15001f/ratatui-0.24.0/src/buffer.rs:240:9
   3: ratatui::buffer::Buffer::set_stringn
             at /home/wbain/.cargo/registry/src/index.crates.io-6f17d22bba15001f/ratatui-0.24.0/src/buffer.rs:309:25
   4: ratatui::buffer::Buffer::set_line
             at /home/wbain/.cargo/registry/src/index.crates.io-6f17d22bba15001f/ratatui-0.24.0/src/buffer.rs:343:23
   5: ratatui::widgets::table::render_cell
             at /home/wbain/.cargo/registry/src/index.crates.io-6f17d22bba15001f/ratatui-0.24.0/src/widgets/table.rs:640:9
   6: <ratatui::widgets::table::Table as ratatui::widgets::StatefulWidget>::render
             at /home/wbain/.cargo/registry/src/index.crates.io-6f17d22bba15001f/ratatui-0.24.0/src/widgets/table.rs:558:17
   7: ratatui::terminal::Frame::render_stateful_widget
             at /home/wbain/.cargo/registry/src/index.crates.io-6f17d22bba15001f/ratatui-0.24.0/src/terminal.rs:632:9
   8: bad_table::main::{{closure}}
             at ./examples/bad_table.rs:20:13
   9: ratatui::terminal::Terminal<B>::draw
             at /home/wbain/.cargo/registry/src/index.crates.io-6f17d22bba15001f/ratatui-0.24.0/src/terminal.rs:334:9
  10: bad_table::main
             at ./examples/bad_table.rs:19:9
  11: core::ops::function::FnOnce::call_once
             at /rustc/eb26296b556cef10fb713a38f3d16b9886080f26/library/core/src/ops/function.rs:250:5
note: Some details are omitted, run with `RUST_BACKTRACE=full` for a verbose backtrace.

To Reproduce

use std::io;

use crossterm::{terminal, ExecutableCommand};
use ratatui::{
    backend::CrosstermBackend,
    layout::{Alignment, Constraint},
    text::Line,
    widgets::{Row, Table},
    Terminal,
};

fn main() -> io::Result<()> {
    io::stdout().execute(terminal::EnterAlternateScreen)?;
    terminal::enable_raw_mode()?;
    let mut terminal = Terminal::new(CrosstermBackend::new(io::stdout()))?;
    terminal.clear()?;

    loop {
        terminal.draw(|frame| {
            frame.render_stateful_widget(
                Table::new(vec![])
                    .header(Row::new([Line::from("").alignment(Alignment::Right)]))
                    .widths(&[Constraint::Min(140); 1]),
                frame.size(),
                &mut Default::default(),
            );
        })?;

        // Check for user input every 250 milliseconds
        if crossterm::event::poll(std::time::Duration::from_millis(250))? {
            // If a key event occurs, handle it
            if let crossterm::event::Event::Key(key) = crossterm::event::read()? {
                if key.kind == crossterm::event::KeyEventKind::Press {
                    match key.code {
                        crossterm::event::KeyCode::Char('q') => break,
                        _ => {}
                    }
                }
            }
        }
    }

    io::stdout().execute(terminal::LeaveAlternateScreen)?;
    terminal::disable_raw_mode()?;

    Ok(())
}

Expected behavior

Trying to render a table which doesn't fit should resolve in some form of truncation rather than a panic.

Environment

OS: Linux
Terminal Emulator: vscode (TERM=xterm-256color)
Font: N/A
Crate version: 0.24.0
Backend: crossterm

Additional context

In ratatui::buffer::Buffer::set_stringn the first call is to Buffer::index_of, which asserts. It looks like Buffer::set_stringn is supposed to handle truncating strings which are too long—if it weren't for the debug assert I believe that the call would hit the bound check in the loop and avoid actually writing anything. So it seems like a fix here would be to defer the index_of call until after a bounds check.

Thanks for the bug report - would you be interested in writing a PR to fix this?


make sure to create add creating a new branch before adding PR to autora

Need to add this documentation to https://autoresearch.github.io/autora/contribute/modules/#incorporating-your-module-into-the-autora-parent-package
It's not clear to create a new branch before working on the PR

Also elaborate on PR instructions


Adding role assignment for the devops managed identity

Which issue this PR addresses:
While updating OpenShift version in CosmosDB, the pipeline does not authorise to access CosmosDB. This PR will create role assignments when RP will be deployed so that when a OpenShift version is added/removed it has access to CosmosDB.
We will need to edit the rp-config as well.
Fixes - ARO-9920
Test plan for issue:
Will be tested in INT env

/azp run ci, e2e

/azp run ci, e2e

/azp run ci, e2e


Login-user is not defined in Setup_Load_Test.ps1

Getting below error after providing the number of reports to configure.
Login-PowerBI : The term 'Login-PowerBI' is not recognized as the name of a cmdlet, function, script file, or operable
program. Check the spelling of the name, or if a path was included, verify that the path is correct and try again.
At C:\Users\Jeby.Jacob\Downloads\PowerBI-Tools-For-Capacities-master\PowerBI-Tools-For-Capacities-master\RealisticLoadT
estTool\Setup_Load_Test.ps1:75 char:13

$user = Login-PowerBI

Try
Install-Module -Name MicrosoftPowerBIMgmt
and
Set-ExecutionPolicy -Scope CurrentUser -ExecutionPolicy Bypass -Force


Fix generate-yamls error by Ignoring grep error

grep returns 1 if there is no match, which is the case since there is only 1 file generated, which was negated by -v. Fixing it by ignoring grep error
/cc @k4leung4

/lgtm

@k4leung4 , an /approve is also needed

/approve


determining the size of segmentation

I was Going through your Code ..
How do we

Determine the Area Covered By Segmentation in the Complete image in Terms of Percentage
Determine the size  of segmentation which are detected...

Your Suggestion shall be highly appreciated.

Hello.. you can count the pixel intensities.
Perhaps with a certain threshold.


s390x: Support scalar min/max clif instructions

We don't have ISA instructions for that, so simply expand them to icmp + select.
Also enable fuzzing for those clif instructions now.

Thanks!  Can this get merged in (I don't have write access to the repo).


Add support for Bobabeam (L2 mainnet)

Website
https://docs.boba.network/
ChainId
Mainnet: 1294
RPC endpoints
Mainnet: Not yet available (Please create a signed transaction for future use)

Hello @rmeissner!
Bobabeam has been released, could you please sign the transaction? I have also sent some funds to 0xE1CB04A0fA36DdD16a06ea828007E35e1a3cBC37
Thank you!
ChainId
Mainnet: 1294
RPC endpoints
Mainnet: https://bobabeam.boba.network/
Current gas parameters
"gasPrice":<PHONE_NUMBER>,
"gasLimit": 1000000,

Hi @rmeissner , how're you?
Is something missing from our side to aprove this PR?
Thank you!

Hello @rmeissner , regarding this comment:
https://github.com/safe-global/safe-singleton-factory/issues/55#issuecomment-1234364090
we estimate the paramaters with the estimate script and these are the correct values:
{ chainId: 1294 } { estimate: '350508' } { gasPriceGwei: '0.000000001', gasPrice: '1' }


Documentation

I am very interested in your project, as it only meets my requirements.
Could you write a small documentation for me.
Thanks in advance.

Should 'eofTimeout' be a function of the baud rate?...

It should, but it won't work in node.js. For baud rates >= 19200 it should be 1.75 ms, for 9600 - 3.65 ms. Node isn't reliable with such small timers, so you should just test what is the smallest value that works for you.

Hey,
I try to make scada app in node.js ,i need modbus-tcp but i haven't find any example that explain modbus with node.js , i have installed modbus-tcp module but i don't find an example and modbus simulator ??
Please i need your help

Look into the example/ directory or check out any project of mine that is using this library.

I am interested in this project.
but the node version is too high.
is V0.10.X will be possible?

The one from the master branch won't work (unless you transpile it using something like Babel).
The old version will work: https://github.com/morkai/h5.modbus/tree/v0


[wip] color updates for explorer site

implements a muted color palette, while keeping the core functionality in tact

@jonahadkins on here: https://github.com/OvertureMaps/io-site/blame/main/site/src/Layers.js#L131 changing the minzoom for places from 0 to 13 means we can't see a "overview/heatmap" of Places anymore:

Is this an intentional change or can we switch it back? It does have the drawback that it works only for Places and not addresses/buildings so may create some unmet expectations.


Unbound: Update Miscellaneous.xml

Allow cert verification in transition.
I know we agreed to move to grid, but maybe for interim we just allow hash sign. We already dont to a validation for private domain field above (which is not an excuse) but at this point, maybe its better to allow this now while still have the option to migrate to grid view in later version

It’s better to migrate now. Already “@@@“ seems to be a valid entry and I have time to play with a grid pattern. 😊

You now have a PR to start as a base :)


Qt warning: model size is less than 0

There is annoying Qt warning  "Model size of -xxx is less than 0" right after opening main screen and when opening receive view. Need to investigate and fix. Probably it is connect to transactions table. This is present for a long long time already.

I've found this https://bugreports.qt.io/browse/QTBUG-72543
It should be fixed, but I reproduce it on 5.15

Since this is qt issue and not ours, and it cannot be fixed until fix is backported by qt devs let's consider this closed/won't fix.


bug: plugin is not configured during build on first install

Did you check docs and existing issues?

[X] I have read all the lazy.nvim docs
[X] I have updated the plugin to the latest version before submitting this issue
[X] I have searched the existing issues of lazy.nvim
[X] I have searched the existing issues of plugins related to this issue

Neovim version (nvim -v)
NVIM v0.11.0-dev+1107-g000129201 Build type: RelWithDebInfo LuaJIT 2.1.1720049189
Operating system/version
Fedora 41
Describe the bug
If a plugin has a build function that requires itself, when it is first installed it is not configured until after the build function has finished running - not when it is required.
Steps To Reproduce
On first nvim -u repro.lua, the plugin is installed and the following is printed:
build start
build end
config

Reopening nvim -u repro.lua and running ":Lazy build drop.nvim", the following is printed
build start
config
build end

Expected Behavior
Plugin is configured when required during build, even during first install.
Repro
local root = vim.fn.fnamemodify("./.repro", ":p")

-- set stdpaths to use .repro
for _, name in ipairs({ "config", "data", "state", "cache" }) do
	vim.env[("XDG_%s_HOME"):format(name:upper())] = root .. "/" .. name
end

-- bootstrap lazy
local lazypath = root .. "/plugins/lazy.nvim"
if not vim.loop.fs_stat(lazypath) then
	vim.fn.system({
		"git",
		"clone",
		"--filter=blob:none",
		"--single-branch",
		"https://github.com/folke/lazy.nvim.git",
		lazypath,
	})
end
vim.opt.runtimepath:prepend(lazypath)

-- install plugins
local plugins = {
	-- do not remove the colorscheme!
	"folke/tokyonight.nvim",
	-- add any other pugins here

	-- this could be any plugin, using drop.nvim as an example
	{
		"folke/drop.nvim",
		lazy = true,
		opts = {},
		build = function()
			vim.print("build start")
			require("drop") -- plugin should be configured here
			vim.print("build end")
		end,
		config = function(_, opts)
			vim.print("config")
			require("drop").setup(opts)
		end,
	},
}
require("lazy").setup(plugins, {
	root = root .. "/plugins",
})

-- add anything else here
vim.opt.termguicolors = true
-- do not remove the colorscheme!
vim.cmd([[colorscheme tokyonight]])

That is actually on purpose, since typically a plugin that needs a build step, needs that step completed before it can be setup.


I can't download the pre-trained FlowNet2 weights (i.e., FlowNet2_checkpoint.pth.tar) from here,can not open the link

how to solve it

https://drive.google.com/file/d/1hF8vS6YeHkx3j2pfCeQqqZGwA_PJq_Da/view?usp=sharing


Don't memcpy the contents of an empty fragment (1.0.2)

In DTLS if we have buffered a fragment for a zero length message (e.g.
ServerHelloDone) then, when we unbuffered the fragment, we were attempting
to memcpy the contents of the fragment which is zero length and a NULL
pointer. This is undefined behaviour. We should check first whether we
have a zero length fragment.
Fixes a travis issue.
[extended tests]
This is the 1.0.2 version of #6223.

Checklist

[ ] documentation is added or updated
[ ] tests are added or updated

Pushed. Thanks.


Carla 0.9.11 Pygame shows that serve has a very low FPS at around 3

Hi, I currently use the new version 0.9.11, here is the configuration of my laptop:

Ubuntu 18.04
Alienware m15 r4 (RTX 3080; intel core i9-10th gen)

It seems that my laptop has no problem with its configuration, but when I run the Carla simulator, it gets slow, the FPS of the server is around 3, and the client FPS is about 62. It runs slowly in the pygame when I start the manual_control.py. Does anyone know what's the reason causing this problem?

Are you using the packaged version, or are you in the Unreal Engine editor?

Possible solution: you need to open Unreal Engine Editor settings and find "Use less CPU in background" - uncheck the box and everything should be fine.

Possible Solution, find /PythonAPI/Utils/config.py.  run this script, e.g.: python3 config.py --fps=20.


Option for init / teardown blocks in tag JS?

I'm using Riot as a subsystem in a larger UI framework that I'm working on.  I'm providing a system by which our users can provide 'modules' that can contain arbitrary tools that would be rendered by the framework in a dialog box.  I register their modules on page load, set up some scaffolding that renders a link to their tool in our nav bar, then when the user clicks that link the framework builds a new dialog, injects their custom riot tag into the dialog then utilizes Riot to mount it.
This all works pretty well right now, except that their JS doesn't actually get executed until the tag is mounted.  So if that tool needed to perform some sort of initialization immediately at page load (like utilizing a framework provided socket service to start listening for server events), it won't happen.  As I see it right now, I have two options for shimming this functionality.

Modify the Riot compile process to somehow pull out code that matches some pattern and execute it immediately during the riot.tag() process.  Bleh.
Have Riot mount all custom tags for the tools immediately, even if they haven't explicitly been used yet.  This would involve rendering them in a hidden div then swapping their content into the dialog when they are requested.

Neither of these options is particularly appealing to me, and I'd love to have a sanctioned way to provide some sort of init code that gets executed immediately when the tag is registered with Riot (provided this idea falls in line with the spirit of Riot).
Thoughts?

+1 Trying to figure this out now. Playing with RiotControl and would like to make an event handler for a RiotControl Store to recognize when a new tag is registered.

No feedback here?  It's cool if the answer is 'No, this isn't something we want to support'.

Ooops. Somehow I have skipped this issue. No idea why. Sorry!
This definitely isn't against the spirit of Riot. Makes sense to know when a tag is registered.
Not sure yet how to implement this, but seems like a decent feature.

Sweet, thanks for the reply!

@clocklear I do not think this enhancement will land on the next riot releases. I am closing this bug for now.


Cannot enlighten self

Version: forge 0.10.1
Instructions: After getting Blind Diversion, cast from health, from full health, to less than half a heart.
Specifically, at 10 hearts, cast White Sun Nadir for 49.9 seconds at power 2.
Expected: Enlightenment.
Result: Reduced to (less than?) half a heart, and got weakness. But no enlightenment.

I really don't know what you could be doing here, because I managed with Black Sun's Nadir(1, 19.5) just fine...
What's your modlist?


SQLite

I wish I could contribute to this project, but unfortunately my python skills are beyond non-existent. :( Have you considered moving this to Go instead?
The bane of my existence these days is SQLite, because it requires cgo. For giggles I thought I would run this against it to see what I got. Here are the results:
There is no regex for 'ParenType'.
I will print out all the lines so a regex can be created:

 0x7fa205805b00 'int (void *, int, char **, char **)' sugar
 0x7fa20580b670 'void (void)' sugar
 0x7fa20580f660 'void (void *)' sugar
 0x7fa20584cdc0 'void (const Fts5ExtensionApi *, Fts5Context *, sqlite3_context *, int, sqlite3_value **)' sugar
 0x7fa205872be0 'void (int)' sugar
 0x7fa206058270 'int (int, const void *, UnpackedRecord *)' sugar
 0x7fa205815e30 'int (void *, int, const char *, const char *, const char *, const char *)' sugar
 0x7fa205098b40 'const sqlite3_io_methods *(const char *, unixFile *)' sugar
 0x7fa20690d4e0 'int (SortSubtask *, int *, const void *, int, const void *, int)' sugar
 0x7fa205c496c0 'int (sqlite3 *, char **, const sqlite3_api_routines *)' sugar
   `-ParenType 0x7fa206091310 'void (void *, int, const char *)' sugar
   `-ParenType 0x7fa20580fc10 'int (int)' sugar
   `-ParenType 0x7fa20580b670 'void (void)' sugar

Traceback (most recent call last):
  File "c2go.py", line 568, in <module>
    raise e
ValueError: No JSON object could be decoded

While that is probably not very helpful to you, I thought that giving you more examples might be of at least some help.
If you move this project to Go, or decide to set up a patreon.com page, please let me know.
I am willing to throw some money at converting SQLite to pure Go! :)
Goodluck

Hi Mark! Thanks for that - it funny you bring up SQLite3... It's one of my favourite projects, and it's source is the perfect candidate for a stable version of c2go (I have updated the README to reflect this).
Unfortunately c2go is far from being able to convert the SQLite3 codebase - but I don't think it's at all impossible with the right help.
Every issue is helpful to the project, if you could provide slices of the codebase that currently don't transpire they can be individually dealt with... (doesn't have to be part of the SQLite3 codebase, any really)

@markbates how about your Go skills? The project has been converted to pure Go for a few versions now.

SQLite project https://www.sqlite.org/index.html

Present amount of warnings:
○ → c2go transpile sqlite3.c

○ → cat sqlite3.go | grep "Warning" | wc -l
9236

○ → cat sqlite3.go | grep "Warning" | grep "function pointer" | wc -l
564

○ → cat sqlite3.go | grep "Warning" | grep "anon" | wc -l
1416

○ → cat sqlite3.go | grep "Warning" | grep "I couldn't find an appropriate Go type" | wc -l
1000

○ → cat sqlite3.go | grep "Warning" | grep "cannot determine type for LHS" | wc -l
3482

○ → cat sqlite3.go | grep "Warning" | grep "pointer of array is nil" | wc -l
556

○ → cat sqlite3.go | grep "Warning" | grep "probably an incorrect type translation" | wc -l
1308


Inadequate Validation of UObject in C++ UFunction Calls？

When calling C++ UFunction in C#, the IsValid method is not as effective as FWeakObjectPtr in determining whether a UObject has been destroyed. Below is a relevant code example:
void* UAActorExporter::GetRootComponent(AActor* Actor)
{
    if (!IsValid(Actor))
    {
        return nullptr;
    }

    return FCSManager::Get().FindManagedObject(Actor->GetRootComponent()).GetIntPtr();
}

What do you mean?

As shown in the above figure, call GUObject Array IndexToObject (Test ->InternalIndex) will return the FUObjectItem corresponding to the object; The IsVaild method only uses the Flags of FUObjectItem to determine whether an object is valid; But this FUObject Item may belong to another UObject, and the original UObject may already be enough to be destroyed.

If our IsValid fails, we have a GC issue on our hands and the problem is elsewhere.


Shplonk doesn't need to implement PCS so far

Would simplify the code and give more flexibility in refactoring the PCS abstraction

Fixed in 7dbbd232189dec248aa386f4925e43d465892730


Implement Initial Search Results Page and Associated Components

💡 Description

📆 01/2024 status: Deferring these landing pages to next build. This website delivery spans several builds, so no impact on delivery. No review from SWG needed.

New designs expected for product types.

Merged in https://github.com/NASA-PDS/portal-wp/pull/20


Allow to completely silence logs, configure from [config]

Feature Description
At the moment --quiet or --loglevel=error will still output log lines if an error is encountered.
If you're working on a CLI (or just don't want logs) and are using cargo-make to automate various tasks you may want to get "passthrough" output without any cargo-make specific logs
Describe The Solution You'd Like
A --silent (for example) flag that silences all output other than script / binary STDOUT / STDERR
(would also be useful to be able to configure this in the script [config] section)
Thanks for your consideration!

as for config section, its too late. cargo-make will spit out log output before even reading the makefile.toml, for example if it fails to find the makefile.toml it will say so and exit.

Are there default logs before that point that aren't errors?
It might make sense to make it suppress any log that isn't one of those errors, the goal being that everyone gets the same experience in a team rather than each person needing to define it for themselves.
The solution makes sense, I hope I'll be able to PR it sometime soon but can't make any promises, will update.
Thanks!

I've also noticed that --quiet can swallow STDOUT from scripts, e.g.
[tasks.prettier]
description = "Runs Prettier and writes the results"
script = '''
npx prettier --write .
'''

with
cargo make --quiet prettier

Will produce no output from Prettier itself on errors

@yoav-lavi a new --slient flag (or --loglevel off) has been added to dev branch 0.36.8
feel free to try it out.
i've not added (and not planning to) add logger setup to the toml file config section

@yoav-lavi  this is now officially released

Thank you @sagiegurari! will check it out and reply back here if there's any suggestions.


discovery: fix "too many open files" issue

What does this PR do?
This PR fixes the "Too many open files" issue that has been observed in CI runs. The issue could also be manually triggered by polling multiple times in a row the service discovery endpoints:
sudo lsof -c system-probe|wc -l & sudo curl -s --unix /opt/datadog-agent/run/sysprobe.sock http://unix/discovery/services

Running this multiple times in a row will show the number of file descriptors grow at each request to the endpoint. The maximum observed value from lsof was in 2000s. After the fix, it remains stable at around 22 file descriptors.
The issue was due to use of gopsutil's PidExists function, which calls the pidfd_open syscall, but does not release manually the FD returned by it, leaving to the GC to handle it.
This PR removes the call to PidExists, since, given the pidfd_open will be released by the GC at an unknown time, did not bring us any extra guarantee for the rest of the getService call. The PR also removes unneeded references to the gopsutil Process struct, as it was, in many places, used to get the pid of the process, which is already available from a variable on the stack. The Process struct was also created well in advance of its actual use, before checks that would make us leave the function early. The struct is now only created close to where it is used.
Motivation
Reduce test flakiness.
Describe how you validated your changes

The change fixes test flakiness + has been verified manually (see above).
Possible Drawbacks / Trade-offs
Additional Notes

/merge


snippet(c++): fix beginend

Missing brackets after "end".

[X] I searched the issue tracker and this hasn't been PRed before.
[X] My changes are not on the do-not-PR list for this project.
[X] My commits conform to the git conventions.

Good catch. Thanks for your help!


RegisterComponents() for a list of Module

I would like to use a Dictionary to store submodules in my model, but RegisterComponents() only take effect for Module and Tensor fields. Shall I use something like TorchSharp.torch.nn.Sequential to wrap the submodules in order to make them registered? And to my understanding there may be another solution -- adding detection of List and Dictionary in RegisterComponents(). (I'm not sure whether this will break any existing code logic.)

@NiklasGustafsson Thanks for your question. In the example we need different submodules for different NLP tasks. It is possible to define a field for each submodule, however if we want to add a brand new task, a new field should be added. Using a dictionary would be more elegant.
In another model class, we need a fixed number of layers (say 24). If we use direct fields, there will be 24 different layer fields, which also disables for-loop to operate these layers. In my opinion, it is natural to use a List to include them.

I'm not sure I understand why you wouldn't just have different models and share parameters, but I'll take your word for it.
I suggest making the Module.state_dict() function virtual, and then you can override it, so both save() and load() will use your implementation instead of the native code one. It does have to be a string->Tensor dictionary, though, not a list.
public Dictionary<string, Tensor> state_dict() { ... }

As far as the List scenario goes, that's what Sequential is for.

This is addressed in PR #355, which virtualizes state_dict() and load(BinaryReader). With that, a custom module may save and restore anything it desires to.


Flakey test failing to frequently

https://circleci.com/gh/graknlabs/client-python/1042
https://circleci.com/gh/graknlabs/client-python/1043

Okay, now they just fail consistently @flyingsilverfin :

https://circleci.com/gh/graknlabs/client-python/1048
https://circleci.com/gh/graknlabs/client-python/1050
https://circleci.com/gh/graknlabs/client-python/1052
https://circleci.com/gh/graknlabs/client-python/1053


ISRIC-WoSIS update and upload

I transferred the changes that you emailed us to the ISRIC-Wosis.rmd and uploaded it. See a921ea8 Can @ktoddbrown review this commit?

Looks good!


fix in README example with ensure version package

Installation of specific package version with ensure not version parameter. Version parameter is not defined.

Thanks!


deps: Bump deps of core to prepare for release

Which issue does this PR close?

Part of https://github.com/apache/opendal/issues/5203
Rationale for this change

What changes are included in this PR?

Are there any user-facing changes?

Thank you @PsiACE for the review!


Login redirects to /your_apps/api_key/regenerate

After login at https://cartodb.com/sessions/new I'm redirected to https://juanignaciosl.cartodb.com/your_apps/api_key/regenerate. I'd expect being redirected to my dashboard (or, maybe, last page used, but not api key regeneration).
Redirection is caused by a 302 response with header "Location:https://juanignaciosl.cartodb.com/your_apps/api_key/regenerate".
I followed smoke test guide yesterday, and I requested api key regeneration. I had some connection issues due to mobile coverage. It looks like I lost connection and that regeneration request is the remembered one.
As a workaround, changing url by hand works as expected.
https://cartodb.atlassian.net/browse/CDB-4371

This is not happening anymore, closing.


Adress already in use

Hello, the crash_switch.py script that its in the scripts folder does not work
It throws that The Dress is already in use even thought i disabled the plugin and run it as root, i have also installed the module required

Its been 2 days
please help

I've had this issue on a joycontrol fork too, also why are you running a file called crash_switch lmao


Sessions are not automatically extended when user is active [6]

Describe the Bug:
Our sso-express or sso-react packages may need an update, and we should review the configuration used.
Probability (1-5): 4 (any user being active for 28min)
Effect (1-5): 2

Assuming this PR is still awaiting to be merged? I went to test and now the app opens with the extend session message.

Tested in dev. Timeout message displayed when expected. Button to remain active worked as expected. Thanks @gurjmatharu!


Layout docs

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 63.315%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1317

Relevant Lines:
1814

💛  - Coveralls


How to use 'nsys' in mlc container

I'm using a pre-built MLC container on my ARM64 (aarch64) platform with an Orin GPU. I've successfully benchmarked the speed of LLM as per the provided documentation. However, I'm facing an issue where I can only trace CPU activities when attempting to analyze with nsys. Since GPU information is crucial for our analysis, I need to resolve this. How can I configure nsys within the container to include GPU tracing?

Hi @Louym, firstly I'm glad that you were able to get MLC running and benchmarked.  I haven't used nsys inside container before and am not sure if there is special setup or tools needed installed inside the container.  You might just want to start by profiling a simple CUDA app in a simple CUDA container if you haven't already.  Otherwise, you can find the wheels from here (which you can install outside container):  http://jetson.webredirect.org/jp5/cu114

Hi @Louym, firstly I'm glad that you were able to get MLC running and benchmarked. I haven't used nsys inside container before and am not sure if there is special setup or tools needed installed inside the container. You might just want to start by profiling a simple CUDA app in a simple CUDA container if you haven't already. Otherwise, you can find the wheels from here (which you can install outside container): http://jetson.webredirect.org/jp5/cu114

Thank you very much for your prompt response. I will try this website later, and I will get back to you if I have any issues.

Hmm I haven't done this outside of container, but it would seem that the MLC version of the wheel does not correspond to the TVM version of the wheel.  In the containers I lock the package versions to make sure the matching ones get installed, which you can see those versions here in its config.py:
https://github.com/dusty-nv/jetson-containers/blob/5ba535905ff275e02ff779112d351a754aece94f/packages/llm/mlc/config.py#L31
In general, I would try and replicate the install just how I have done it in the container.  It looks like you are using conda and I don't think that should impact it but not sure?

I tried again and find that I can use 'nsys profile -stats true' to see the summary of GPU kernels. But I also meet some issues when building tvm-unity(or relax) from source, although I have:
set(USE_CUDA  ON)
set(USE_FLASHINFER ON)
set(FLASHINFER_CUDA_ARCHITECTURES 87)
set(CMAKE_CUDA_ARCHITECTURES 87)

It would be great if you could help me solve this problem.

I manage to see the nsys timeline results by compiling models in containers and then benchmarking it in the environment built with the website http://jetson.webredirect.org/jp6/cu122 using jp 6 and cuda 12.2 out of containers.


Wrong info on Nested OU scope

The image in the doc shows that Nested OUs will not be included with Sync on scoping. However, our testing shows that Nested OUs do in fact sync their objects.
This is the second image in:
Scope provisioning to specific users and groups

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

ID: 1efe4486-2449-4ec7-671b-63d398eec65a
Version Independent ID: 11d3a480-8cde-b13b-d2c2-4ffe8c782d83
Content: Azure AD Connect cloud sync new agent configuration
Content Source: articles/active-directory/cloud-sync/how-to-configure.md
Service: active-directory
Sub-service: hybrid
GitHub Login: @billmath
Microsoft Alias: billmath

@sjflanagan
Thanks for calling this out.  I have an internal work item open on this and it will be getting updated.  At this point we can close this issue and I will post to this once I have more.  Thanks!
#please-close


Discussion: Adding option to run as different user

Problem
By default on Raspbian, running an audio application alongside Raspotify causes the applications to compete for device access.
Elaboration
Since it's possible that a user will be running more than one audio application alongside raspotify, a user is reasonably likely to run Pulseaudio since it ships by default with Raspbian. Doing so is far less complex than having to configure an ALSA dmix.
Pulseaudio is run by default per-user (under pi by default), whereas raspotify runs as the raspotify user. Raspotify sends its audio to the default ALSA device, but this can't be redirected to Pulse by ALSA as it doesn't have access to Pulse. This will either cause raspotify to compete for audio devices, or crash (based on my testing).
Solutions
My suggestion would be to allow the user to run raspotify as the default user, rather than its own. Doing so now requires some non-user-friendly config changes to the systemd service.
My understanding of ALSA/Pulse is rudimentary, so graciously looking for feedback on this idea.
@dtcooper
@JasonLG1979

Since it's possible that a user will be running more than one audio application alongside raspotify, a user is reasonably likely to run Pulseaudio since it ships by default with Raspbian. Doing so is far less complex than having to configure an ALSA dmix.

AFAIK PulseAudio does not come installed by default in the desktop version of Raspberry Pi OS and I know for a fact that it's not installed by default on the Lite version. It's possible though something else you installed pulled PulseAudio in as a dependency.
The default ALSA input is dmix'd. You don't have to roll your own dmixer if you don't want to. The most basic config just involves making sure you're telling ALSA which card you want to use as default.

Pulseaudio is run by default per-user (under pi by default), whereas raspotify runs as the raspotify user. Raspotify sends its audio to the default ALSA device, but this can't be redirected to Pulse by ALSA as it doesn't have access to Pulse. This will either cause raspotify to compete for audio devices, or crash (based on my testing).

PulseAudio is not designed to run system wide (it bad enough that it runs as the privileged pi user), that's intentional, it's the same reason raspotify runs as the unprivileged raspotify system user. For security reasons.
Don't get me wrong PulseAudio is great for doing what it's supposed to do most of the time but this is not what it was designed to do.
If you're having issues configuring ALSA properly follow the steps in this /etc/asound.conf to walk you through creating a basic working system wide config.
# /etc/asound.conf

###############################################################################

# Find your sound card:
# aplay -l

# Find your card's supported format(s) and sampling rate(s) While no other audio is being played:
# aplay -Dhw:<card #>,<device #> --dump-hw-params /usr/share/sounds/alsa/Front_Right.wav
# For example:
# aplay -Dhw:0,0 --dump-hw-params /usr/share/sounds/alsa/Front_Right.wav

# The output should look something like this if the card and device combination is correct:

# ACCESS:  MMAP_INTERLEAVED RW_INTERLEAVED
# FORMAT:  S16_LE S24_LE S32_LE
# SUBFORMAT:  STD
# SAMPLE_BITS: [16 32]
# FRAME_BITS: [32 64]
# CHANNELS: 2
# RATE: [8000 192000]
# ...

# The above card supports formats S16_LE, S24_LE, and S32_LE, at up to a sampling rate of 192000.

# Uncomment and replace with the sampling rate of your choice.
# defaults.pcm.dmix.rate 48000

# Uncomment and replace with the format of your choice.
# defaults.pcm.dmix.format S16_LE

# Change to the card number or name that you want to be the default control card.
# Default: 0
defaults.ctl.card 0

# Change to the card number or name that you want to be the default playback card.
# It should usually be the same as defaults.ctl.card.
# Default: 0
defaults.pcm.card 0

# Change to the device number that you want to be the default device on the default card.
# 0 or 1 is usually the correct device number.
# Default: 0
defaults.pcm.device 0

# Change to the subdevice number that you want to be the default subdevice on the default device.
# Should rarely need to be changed.
# Default: -1
defaults.pcm.subdevice -1

# To install high quality samplerate converters on Debian based systems:
# sudo apt install -y --no-install-recommends libasound2-plugins

# To list available rate_converter's:
# echo "$(ls /usr/lib/*/alsa-lib | grep "libasound_module_rate_")" | sed -e "s/^libasound_module_rate_//" -e "s/.so$//"

# Uncomment and replace speexrate_medium with the rate_converter of your choice.
# defaults.pcm.rate_converter speexrate_medium

# Once everything is configure close and save this file and test with:
# speaker-test -c2 -l1

# While audio is playing you can verify your settings with:
# cat /proc/asound/card*/pcm*p/sub*/hw_params

I more complex /etc/asound.conf can be found here.

Raspotify is primarily intended to be used in a headless enviroment (Raspberry Pi OS lite) For desktop versions of Raspberry Pi OS spotifyd offers similar functionality, much better PulseAudio compatibility and is a better choice.


Add SkipDotNet to Redo-LTService

For you, sir.  We needed this so I figured I'd give back.

Cancel this.  Going to resend it.


Connection demo app: Account summary block

Show user connected status. Shows user short address. Shows current balance. If balance is low, style the balance in red. Currency internationalization? Maybe click on summary block, show dropdown with option to Disconnect. Can also be space for dApp specific-functionality/navigation. (no blockie here)

[ ] Display public address (truncated)
[ ] Display wallet balance
[ ] Low balance alternate presentation
[ ] Currency conversion (check MM for this?)
[ ] Dropdown on click
[ ] Disconnect option
[ ] Tests

Resolved by: https://github.com/ConsenSys/rimble-app-demo/issues/22