Change rust types folder

Moving rust_types to ledger_models_rust.
This is step 1 of a refactor:

Renaming this folder
Make changes on imports in valuation-service

Step 2 will be to move the ledger-models-rust folder within this project.
Step 3 will be to rename java folder to 'ledger-models-java' (and update the pom.file which is equiv of Cargo.toml)

Closed 2 other PRs. I didn't reset my branches between code commits , this has 3 sets of changes

The main impact of this is in the valuation service the import in .rs goes from:
use rust_types::ledger_models::{valuation::valuation_client::ValuationClient, security::SecurityRequestProto, util::UuidProto};
to
use ledger_models::ledger_models::{valuation::valuation_client::ValuationClient, security::SecurityRequestProto, util::UuidProto};
ledger_models is duplicative, so perhaps we could change to ledger_models::models::{....};


merge to live

CATS issues - fixes

OPS Build status updates of commit 0f0041a:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/add-ins/add-mobile-support.md
:white_check_mark:Succeeded

docs/add-ins/extract-entity-strings-from-an-item.md
:white_check_mark:Succeeded

docs/add-ins/get-and-set-recurrence.md
:white_check_mark:Succeeded

docs/add-ins/metadata-for-an-outlook-add-in.md
:white_check_mark:Succeeded

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


Hosting session from Safari results in join error

Describe what happened:
When trying to host a share session from a Safari web instance (https://vscode.dev/liveshare/) no other users are able to join the session from a browser.
What was your system configuration?
Product and Version [VS/VSCode]: VS Code
OS Version[macOS/Windows]: macOS
Live Share Extension Version: v1.0.5883
Target Platform or Language [e.g. Node.js]: N/A
Steps to Reproduce / Scenario:

Open https://vscode.dev/liveshare in Safari (tested with Safari Version 16.6 (186<IP_ADDRESS>.2))
Start a share session
Try to join the session from any browser (tried both Safari and Edge)
See error (on join side)

Notes:

When I tried joining the same session from the VS Code app (not web), it worked
When starting a session from Edge, all scenarios work as expected
This is 100% reproducible for us

Please attach logs to this issue:
(I was followed the steps listed and did not see an "Export Logs" command)
Safari console:
[Log]   ERR – "Unable to write file 'vscode-log:/20230924T131935/window.log' (Error: IndexedDB database 'vscode-web-db' is closed.)…" (workbench.web.main.js, line 627)
"Unable to write file 'vscode-log:/20230924T131935/window.log' (Error: IndexedDB database 'vscode-web-db' is closed.): R@https://main.vscode-cdn.net/stable/abd2f3db4bdb28f9e95536dfa84d8479f1eb312d/out/vs/workbench/workbench.web.main.js:621:19352
@https://main.vscode-cdn.net/stable/abd2f3db4bdb28f9e95536dfa84d8479f1eb312d/out/vs/workbench/workbench.web.main.js:627:14229"
[Error] Unrecognized Content-Security-Policy directive 'require-trusted-types-for'.

[Error] Unrecognized Content-Security-Policy directive 'trusted-types'.

[Info] Successfully preconnected to https://main.vscode-cdn.net/ (x2)
[Log]  INFO – "Found additional builtin extension gallery resources in env" – ["https://github.vscode-unpkg.net/GitHub/codespaces/1.15.3/extension", "https://github.vscode-unpkg.net/GitHub/remotehub/0.60.0/extension", "https://ms-vscode.vscode-unpkg.net/ms-vscode/remote-server/1.4.3/extension", …] (16) (workbench.web.main.js, line 627)
["https://github.vscode-unpkg.net/GitHub/codespaces/1.15.3/extension", "https://github.vscode-unpkg.net/GitHub/remotehub/0.60.0/extension", "https://ms-vscode.vscode-unpkg.net/ms-vscode/remote-server/1.4.3/extension", "https://ms-vscode.vscode-unpkg.net/ms-vscode/azure-repos/0.36.0/extension", "https://ms-vscode.vscode-unpkg.net/ms-vscode/anycode/0.0.72/extension", "https://ms-vscode.vscode-unpkg.net/ms-vscode/anycode-cpp/0.0.5/extension", "https://ms-vscode.vscode-unpkg.net/ms-vscode/anycode-java/0.0.5/extension", "https://ms-vscode.vscode-unpkg.net/ms-vscode/anycode-typescript/0.0.5/extension", "https://ms-vscode.vscode-unpkg.net/ms-vscode/anycode-python/0.0.5/extension", "https://ms-vscode.vscode-unpkg.net/ms-vscode/anycode-go/0.0.5/extension", …]Array (16)
[Log]  INFO – "Updating additional builtin extensions cache" (workbench.web.main.js, line 627)
[Error] Failed to load resource: the server responded with a status of 404 (The specified blob does not exist.) (workbench.web.main.nls.js.map, line 0)
[Error] Unrecognized Content-Security-Policy directive 'require-trusted-types-for'.

[Error] Unrecognized Content-Security-Policy directive 'trusted-types'.

[Error] Failed to load resource: the server responded with a status of 404 () (ms.core.min.js.map, line 0)
[Error] Failed to load resource: the server responded with a status of 404 () (tas-client-umd.js.map, line 0)
[Error] Failed to load resource: the server responded with a status of 404 (The specified blob does not exist.) (layout.contribution.darwin.js.map, line 0)
[Error] Failed to load resource: the server responded with a status of 404 () (dynamicproto-js.min.js.map, line 0)
[Error] Failed to load resource: the server responded with a status of 404 () (ms.post.min.js.map, line 0)
[Error] Failed to load resource: the server responded with a status of 404 () (applicationinsights-core-js.min.js.map, line 0)
[Log]   ERR – "Extension 'ms-vscode.remote-server' appears in product.json but enables LESS API proposals than the extension wants.↵package.json (LOSES)…" (workbench.web.main.js, line 627)
"Extension 'ms-vscode.remote-server' appears in product.json but enables LESS API proposals than the extension wants.
package.json (LOSES): resolvers, tunnelFactory
product.json (WINS): resolvers, tunnels, contribViewsWelcome"
[Log]   ERR – "Extension 'ms-vscode.remote-server' appears in product.json but enables LESS API proposals than the extension wants.↵package.json (LOSES)…" (workbench.web.main.js, line 627)
"Extension 'ms-vscode.remote-server' appears in product.json but enables LESS API proposals than the extension wants.
package.json (LOSES): resolvers, tunnelFactory
product.json (WINS): resolvers, tunnels, contribViewsWelcome"
[Log] [Extension Host] Initializing tunnel extension (workbench.web.main.js, line 129)
[Error] Unrecognized Content-Security-Policy directive 'require-trusted-types-for'.

[Error] Unrecognized Content-Security-Policy directive 'trusted-types'.

[Log]  WARN – "Timed out waiting for all supported executions" (workbench.web.main.js, line 627)
[Error] Failed to load resource: the server responded with a status of 404 () (remoteSettings.json, line 0)

Screenshots

We’ve made improvements in this area since this issue was filed. We believe this may have improved your experience and are closing this issue. If this issue still persists for you, please comment on the issue and let us know. Thank you.

Thank you, @derekbekoe! This is now working for me.


New lift approach

Migrates Akshay high p-code lifting support to the old script.
Canonicalizes both the headless and gui codepaths to operate on lists of functions.
Saturates a set of functions to lift, where the initial input set is treated as those for which we want definitions (i.e. high p-code)
Starts doing p-code operation-specific lifting in the JSON representation. E.g. CBRANCH and CALL. This is important to eventually discover (and thus saturate) all referenced functions and global symbols.

@pgoodman can you clean up commit messages, please. I want to use change-logs for quarterly reports.


HDFS-16942. Addendum. Send error to datanode if FBR is rejected due t…

Description of PR
Remove the package-info.java and the pom exclusions added in HDFS-16942 as they are breaking the javadoc build.
How was this patch tested?
No changes to core logic.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 53s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  0s

codespell was not available.

+0 :ok:
detsecrets
0m  0s

detect-secrets was not available.

+0 :ok:
xmllint
0m  0s

xmllint was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

-1 :x:
test4tests
0m  0s

The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ trunk Compile Tests _

+0 :ok:
mvndep
15m 40s

Maven dependency ordering for branch

+1 :green_heart:
mvninstall
29m 52s

trunk passed

+1 :green_heart:
compile
27m 24s

trunk passed with JDK Ubuntu-11.0.18+10-post-Ubuntu-0ubuntu120.04.1

+1 :green_heart:
compile
23m 23s

trunk passed with JDK Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09

+1 :green_heart:
checkstyle
4m  4s

trunk passed

+1 :green_heart:
mvnsite
2m 15s

trunk passed

+1 :green_heart:
javadoc
1m 48s

trunk passed with JDK Ubuntu-11.0.18+10-post-Ubuntu-0ubuntu120.04.1

+1 :green_heart:
javadoc
2m  9s

trunk passed with JDK Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09

+0 :ok:
spotbugs
0m 30s

branch/hadoop-client-modules/hadoop-client-api no spotbugs output file (spotbugsXml.xml)

+1 :green_heart:
shadedclient
23m  4s

branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 25s

Maven dependency ordering for patch

+1 :green_heart:
mvninstall
4m 40s

the patch passed

+1 :green_heart:
compile
26m 58s

the patch passed with JDK Ubuntu-11.0.18+10-post-Ubuntu-0ubuntu120.04.1

+1 :green_heart:
javac
26m 58s

the patch passed

+1 :green_heart:
compile
23m 18s

the patch passed with JDK Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09

+1 :green_heart:
javac
23m 18s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

+1 :green_heart:
checkstyle
3m 55s

the patch passed

+1 :green_heart:
mvnsite
2m 10s

the patch passed

+1 :green_heart:
javadoc
1m 40s

the patch passed with JDK Ubuntu-11.0.18+10-post-Ubuntu-0ubuntu120.04.1

+1 :green_heart:
javadoc
2m 11s

the patch passed with JDK Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09

+0 :ok:
spotbugs
0m 31s

hadoop-client-modules/hadoop-client-api has no data from spotbugs

+1 :green_heart:
shadedclient
22m 56s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

+1 :green_heart:
unit
229m 53s

hadoop-hdfs in the patch passed.

+1 :green_heart:
unit
0m 46s

hadoop-client-api in the patch passed.

+1 :green_heart:
asflicense
1m 12s

The patch does not generate ASF License warnings.

465m 39s

Subsystem
Report/Notes

Docker
ClientAPI=1.42 ServerAPI=1.42 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5478/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/5478

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient codespell detsecrets xmllint spotbugs checkstyle

uname
Linux 81633298ddd6 4.15.0-197-generic #208-Ubuntu SMP Tue Nov 1 17:23:37 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / f31ffa4591da881b4580b89e0263b57491754289

Default Java
Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.18+10-post-Ubuntu-0ubuntu120.04.1 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5478/1/testReport/

Max. process+thread count
2115 (vs. ulimit of 5500)

modules
C: hadoop-hdfs-project/hadoop-hdfs hadoop-client-modules/hadoop-client-api U: .

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5478/1/console

versions
git=2.25.1 maven=3.6.3 spotbugs=4.2.2

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

This message was automatically generated.

Wow, checkstyle didn't fail this time (how fragile!).
Maybe does this mean they would only fail if we add new file under the module and not when we update or remove existing file?

Thanx folks, have merged this,


[Core] Implements prepareStyles as composition of functions in muiTheme

This commit implements the prepareStyles function as a composition of functions that can be configured in muiTheme. It switches the ./mixin/style-propable.js implementation to use the prepareStyles stored in context.
@oliviertassinari @alitaheri
This is an initial implementation of the approach of storing prepareStyles in context. I cleaned up the code a bit, however like I said, I don't think we should document this until we've confirmed we're happy and the API is final (I already have some ideas of how to improve it, but it shouldn't have any real immediate benefit). It's also very well possible this will all go away when we adopt a new styling framework.
Note: One minor difference I made is that the getMuiTheme function actually checks the computed theme for isRtl before composing the functions. If it's false, it doesn't even bother including the ensureDirections() (renamed to rtl()) method. Edit: It now does this for the autoprefixer() method based on the userAgent property
Right now, the list of transformers can be overridden by adding the following property to the muiTheme.  The following configuration would essentially disable the default behavior of always autoprefixing and calling rtl() when isRtl is equal to true. Removed customizing of transformers in this PR, other approaches have since been implemented.
Remaining tasks:

[x] Move implementation of "double call warning" out of rtl() into something that runs before all style transformations are executed. Done
[x] Change rtl() implementation to not care is isRtl No longer necessary

Future PR:

Convert components not using ./mixins/style-propable.js to get prepareStyles() from context, not from importing it from ./utils/style.js.
Deprecate and/or remove ./utils/style.js

Related to #2316, #2530, #2852.

@newoga Awesome work indeed :+1: :+1: Thank you :smile:

I believe we should use assign for every style merging too. :+1: Thanks @newoga

@oliviertassinari @alitaheri
I made some additional changes to this and rebased onto master so that it includes @oliviertassinari's auto-prefix changes. I decided to take passing in custom transformers out of the API for now. I think having that feature is now less urgent because the the two existing transformers (prefixing and RTL) can be disabled through muiTheme configuration (userAgent set to false and isRtl set to false disables both).
If we want to explore making this publicly accessible, I think we can explore options in a new PR.
The two key changes I made were the following:
Moved the check for running prepareStyle() twice out of the rtl() function
I decided to not to make the check a transformer that was part of the chain, because there would be no way to cancel the rest of the chain from running. I decided instead to wrap the chained function with the check so that it gets called first and exits if it sees if it was ran already.
In addition, I changed the check from using a object key to a Symbol. I think this is safer so that developers can't set the property to "trick" the check into running and it also gets the nasty did-flip keys out of the style objects in the DOM.
The end result is that this check is ran in development AND production, just that it only logs a warning in development. I think this is the better approach though.
Changed mergeStyles() implementation to Object.assign().
The first reason is because I think it's simpler, but also Object.assign() supports assigning Symbol keys and out previous merge implementation did not. This change was required for the first step above. I think we were all leaning towards moving towards either Object.assign() or lodash's assign anyway.

@newoga I see you have brainstormed the crap out of it :+1: :+1: Really great job. I really like your approaches :+1: :+1: :tada:

Also, Does the polyfill also assign Symbols?

That's an intersting question. I assumed so but assuming can be dangerous :stuck_out_tongue:. I'll have to investigate this more...

This is the polyfill. unfortunately it doesn't support Symbols. but I think this wouldn't matter because if Object.assign is not supported then so isn't Symbol.
var _extends = Object.assign || function (target) {
  for (var i = 1; i < arguments.length; i++) {
    var source = arguments[i];
    for (var key in source) {
      if (Object.prototype.hasOwnProperty.call(source, key)) {
        target[key] = source[key];
      }
    }
  }
  return target;
};

@oliviertassinari @alitaheri Alright guys, I think we're close :persevere:
This has the following changes:

Moved all the "transformer inclusion" checks into the transformer definitions themselves. They will be filtered out if they don't return a truthy value.
Since we decided that there are performance implications with the callOnce, I moved it out to a transformer like @oliviertassinari originally suggested. It returns early if it's in production environment.
Replaced the Symbol with just a string for the callOnce check.

@newoga I think we have it! I like the current approach :+1:. We will have to refactor a lot of code once it's merged.

@oliviertassinari I agree, I think we've made a lot of progress with this. Thanks for your feedback!
Looking forward to getting this merged and working towards getting style-propable out. That's definitely the most heavily used mixin at the moment.

All green here :+1: :green_heart:
Thanks a lot @newoga This was indeed a work of art :+1: :+1:

This one was definitely a group effort. Thanks a lot for the help guys :+1:

Any idea why this Popover example is broken?

Any idea why the RaisedButton examples are broken?

Wow. Really really good find. This is a really obscure issue but something to consider really carefully. The issue is here:
https://github.com/callemall/material-ui/blob/master/src/enhanced-button.jsx#L285
If that line was changes to:
appearance: linkButton ? null : 'button',

Then the issue goes away.
It has to do with difference in handling nulls and undefined with Object.assign vs merge. Merge ignores null and undefined and keeps target values and Object.assign overwrites.

 Ironically, I wonder if this is related to the autoPrefixer issue we were also having for the opposite reason where the animations stopped working with the all() method.

Setting appearance: linkButton ? '' : 'button', // edited doesn't solve the issue on my side. What do you think about my other finding?

Setting appearance: linkButton ? '' : 'button', // edited doesn't solve the issue on my side.

Bah you're right, a linting error messed my build and tricked me. Though I did confirm this is related to Object.assign. I thought this was working because I had switched our mergeStyles implementation to lodash's merge, and that seemed to fix it for me.

What do you think about my other finding regarding the hash order?

That's certainly interesting. We can't rule it out yet but I am curious why hash order would make a difference with the styling. I thought order shouldn't matter?

I thought order shouldn't matter?

The order does matter. This is broken:

This works:

@oliviertassinari Oh oh, I see what you mean now, the keys are different...I misunderstood. Since I thought the keys were the same I figured the previous style value would just get replaced.

That's really tricky and linked to how v8 is ordering a hash map.

Yeah, wow. I can't think of a way around this. I'm surprised I haven't seen anyone/any other projects talk about this...

https://bugs.chromium.org/p/v8/issues/detail?can=1&start=0&num=100&q=getOwnPropertyNames&colspec=ID Type Status Priority Owner Summary HW OS Area Stars&groupby=&sort=&id=4118

Well, I don't know exactly why the order change. I'm really curious to understand why. For now. I would say we can this simple fix:
- background: 'none',
+ background-color: 'transparent',

@oliviertassinari I just tried Firefox and it seems like the issue is actually with Chrome's implementation of getOwnPropertyNames. I originally read about this here: https://github.com/sindresorhus/object-assign/issues/22

For now. I would say we can this simple fix:

background: 'none',

background-color: 'transparent',

Yeah that's a good idea.

@oliviertassinari I was curious and downloaded Chrome Canary and checked out the commit before I made the style changed and confirmed that the ordering (and buttons) are working properly.
Looks like this was fixed in V8 version 4.9.148, commit 7d1263d. If I'm reading this correctly, version 4.9.148 of v8 is currently available in the Chrome development channel. Since there was just a new release of Chrome yesterday, I expect the fix for this should be out in 12 weeks if I'm understanding their release cycle

Looks like this was fixed in V8 version 4.9.148

That's good to know :+1:. In the mean time. I have found another issue. The FlatButton is too small.

I'm wondering if we shouldn't use this ponyfony https://github.com/sindresorhus/object-assign.

I agree with @oliviertassinari object-assign does offer the best implementation.

@oliviertassinari @alitaheri It looks like that ponyfill just returns Object.assign if it already exists so it may not fix the issue (because Object.assign does exist in Chrome, it's just not working properly ).  Only other idea is to fork/copy the implementation separately from the Object.assign || ... check

@newoga Oh, right there's that :grin: I would think that that's our best bet at the moment.

@newoga Oh, right there's that :grin: I would think that that's our best bet at the moment.

@alitaheri @oliviertassinari Would you two be against one of the following approaches:

Carefully inspect the rest of the components for times where key ordering is causing a styling issue and investigate approaches like the one above to fixing it (seems like it's only causing issues in isolated cases).
Create a babel plugin that does the same thing as babel-plugin-transform-object-assign but doesn't use existing native implementation for our project to get around this? (We would remove the plugin after Chromium releases their fixes)

I've given it some though and I've really come around and would prefer to just using Object.assign for all the style merging.

Wow, great job investigating this :+1: I think that's a better approach too. Object.assign is more expressive, less cluttered (no need to import) and more es-ish :grin: So I'm all in

Both solutions are valid. Option 2 is definitely quicker to do than option 1. I can't see real advantages of doing option 1 in the long run. I would vote for option 2 :+1:.

Thanks for the feedback guys. Time to learn babel plugins...:neckbeard:

Well, option 3 is to wait a bit before changing the merge function.

Well, option 3 is to wait a bit before changing the merge function.

Yup, I considered this too. Though after this gets merged, I wanted to start removing style-propable. If the existing mergeStyles functions could be replaced with Object.assign, we get to drop an import. I'd definitely vote for this if it meant only replacing it in one place such as utils/style's merge()but soon all the components would have Object.assign

If the existing mergeStyles functions could be replaced with Object.assign

I like this idea. However, using a custom version of babel-plugin-transform-object-assign would clearly increase the lib size.

I like this idea. However, using a custom version of babel-plugin-transform-object-assign would clearly increase the lib size.

What if it was written in a way that in which the function was imported rather than duplicated each time was used.  If we did that it shouldn't be any bigger than it is now (maybe even smaller). Would you be okay with that?
I haven't completely wrapped my mind around babel plugins but it seems like that it's possible with the little I've looked at it.

What if it was written in a way that in which the function was imported rather than duplicated each time was used.

That would do it :smile:.

@oliviertassinari @alitaheri
Take a look at this when you get a chance. Using this approach we're able to fix the issue with the raised-button since we now replace Object.assign with an implementation that doesn't using the native implementation.

@oliviertassinari @alitaheri
FYI just rebased onto master so this didn't get too far behind. I also decided to remove implementation of mergeStyles to Object.assign in this PR.  The feature presented in this PR (composing style transformers) works just fine with the existing merge implementation. I still think we should switch to Object.assign but I decided to present that in a new PR #3124 so we can finish that discussion there.

@newoga Yes, let's merge this :tada:

@oliviertassinari Sounds good! @alitaheri Do you want to give this a last look?

@alitaheri or @mbrookes feel free to merge if you are happy with it.

All good and awesome :+1: :+1: JUst fix those 2 things pointed out by @oliviertassinari And do the merge. Really nice work @newoga :+1: :+1:

Thanks @alitaheri, much appreciated :smile:
If all is green, this should be good to go.

:rocket:!

I know what a long one! Thanks guys! Big group effort :tada: :balloon:

Really nice work. this was a big step indeed :+1: :+1:


Updating to latest C# submodule - Moving PrivacyService to hang off User class

@jasonsandlin,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot


Create couchdb adapter

This pull request adds an adapter for Apache CouchDB to the Lucia project.
This PR includes the source code, tests, and documentation for this adapter.
Tests:

Thanks for contributing!


[SPARK-49333][SQL] Shutdown timeout thread while cleaning up SparkExecuteStatementOperation

What changes were proposed in this pull request?
Shutdown timeout thread while cleaning up SparkExecuteStatementOperation.
Why are the changes needed?
Avoid driver memory leak if query timeout is configured. For example, there are 4127 SparkExecuteStatementOperation instances in the Driver:
jmap  -histo 398 | grep SparkExecuteStatementOperation

 308:          4127        1122544  org.apache.spark.sql.hive.thriftserver.SparkExecuteStatementOperation
 563:          4127         363176  org.apache.spark.sql.hive.thriftserver.SparkExecuteStatementOperation$ErrRowCountType$
 876:          4127         132064  org.apache.spark.sql.hive.thriftserver.SparkExecuteStatementOperation$$anon$1
2101:           333           7992  org.apache.spark.sql.hive.thriftserver.SparkExecuteStatementOperation$$anon$5
3106:            32           1024  org.apache.spark.sql.hive.thriftserver.SparkExecuteStatementOperation$$anon$2
3303:            32            768  org.apache.spark.sql.hive.thriftserver.SparkExecuteStatementOperation$$Lambda$3755/0x00000008021fe800
3304:            32            768  org.apache.spark.sql.hive.thriftserver.SparkExecuteStatementOperation$$anon$2$$anon$3
3961:             9            360  org.apache.spark.sql.hive.thriftserver.SparkExecuteStatementOperation$$Lambda$5398/0x0000000802523900
3962:             9            360  org.apache.spark.sql.hive.thriftserver.SparkExecuteStatementOperation$$Lambda$5399/0x0000000802523bd8
20239:             1             16  org.apache.spark.sql.hive.thriftserver.SparkExecuteStatementOperation$
20240:             1             16  org.apache.spark.sql.hive.thriftserver.SparkExecuteStatementOperation$$$Lambda$5397/0x000000080251e180
20241:             1             16  org.apache.spark.sql.hive.thriftserver.SparkExecuteStatementOperation$$Lambda$11228/0x000000080306ba38
20242:             1             16  org.apache.spark.sql.hive.thriftserver.SparkExecuteStatementOperation$$Lambda$11230/0x00000008032962d8
20243:             1             16  org.apache.spark.sql.hive.thriftserver.SparkExecuteStatementOperation$$Lambda$11231/0x00000008032966b8
20244:             1             16  org.apache.spark.sql.hive.thriftserver.SparkExecuteStatementOperation$$Lambda$5363/0x0000000802509470
20245:             1             16  org.apache.spark.sql.hive.thriftserver.SparkExecuteStatementOperation$$Lambda$5367/0x000000080250a618
20246:             1             16  org.apache.spark.sql.hive.thriftserver.SparkExecuteStatementOperation$$Lambda$6475/0x00000008026fda40
20247:             1             16  org.apache.spark.sql.hive.thriftserver.SparkExecuteStatementOperation$$Lambda$7355/0x00000008028aa180
20248:             1             16  org.apache.spark.sql.hive.thriftserver.SparkExecuteStatementOperation$$Lambda$7356/0x00000008028aa560

Does this PR introduce any user-facing change?
No.
How was this patch tested?
Manual test.
Was this patch authored or co-authored using generative AI tooling?
No.

cc @cloud-fan

Much better than before.
Before this fix:

After this fix:

Merged to master and branch-3.5.

Late LGTM


Questions about gradio demo and inference on custom dataset

Thanks for your wonderful work!
However, I’ve noticed a minor issue in the offline demo. When switching to mask-based ROIs (bounding boxes), the input image seems to be cropped from the original, as can be seen in the following two images. This issue doesn't occur with point-based ROIs.
Input image:

Mask on Image:

Additionally, I have a question about inference on my custom dataset, which includes both images and bounding box coordinates. If I choose not to use SAM  and instead directly provide the rectangular box mask to the model, how should I go about this? Will the absence of SAM impact the model's performance?
Thanks for your time.

Thank you for your attention to our work!
For the first issue, we will try to reproduce the problem on our own device and get back to you as soon as possible with more information.
The absence of SAM will not affect the model’s performance. If you choose to use bounding box coordinates directly, you can first generate the corresponding mask based on these coordinates and provide it as input to SEAGULL.
We will add a module that uses rectangular box to indicate ROIs in the demo.
If you need to test a large amount of data, we will also release the testing scripts as soon as possible.

Thank you for your attention to our work!
For the first issue, we will try to reproduce the problem on our own device and get back to you as soon as possible with more information.
The absence of SAM will not affect the model’s performance. If you choose to use bounding box coordinates directly, you can first generate the corresponding mask based on these coordinates and provide it as input to SEAGULL. We will add a module that uses rectangular box to indicate ROIs in the demo.
If you need to test a large amount of data, we will also release the testing scripts as soon as possible.

Thanks! Looking forward to it!

Hi!
For the first issue, it appears to be a recurring bug with Gradio.  If you know how to fix it, please submit a Pull Request, which would be greatly beneficial to the community.
And the module that uses boxes points to indicate ROIs is added in the demo. We've also released a testing script. All details are available in the README.
Feel free to let me know if you need further assistance!

Hi!
For the first issue, it appears to be a recurring bug with Gradio. If you know how to fix it, please submit a Pull Request, which would be greatly beneficial to the community.
And the module that uses boxes points to indicate ROIs is added in the demo. We've also released a testing script. All details are available in the README.
Feel free to let me know if you need further assistance!

Thanks! I will test it lately.


Keke epe545

Change Description

Add the Ubuntu 20.04 binary package for EOSIO.CDT v1.8.x
After test the deb package works on a clean 20.04 cloud instance .
API Changes

[ ] API Changes

Documentation Additions

[ ] Documentation Additions

Can we change the name of this PR to something more descriptive.

Can we change the name of this PR to something more descriptive.

Sure we can , this name is generated by click "create pull request" button, I know it the name should be more meaningful.


configurable date format and handle decimal InvalidOperation

Adds a property date_format that makes the target key's date format configurable. It defaults to the original hard-coded date format, making this change backwards compatible.
Adds a line to adjust the decimal precision if it is too small compared to the tap's schema

@ome9ax are you open to a PR for this, or open to discuss?

Hi @ome9ax bumping this. Do you have time to review?

I would like to note that the current implementation allows for specifying the date format already. the additional parameter might be redundant.
e.g.
naming_convention: "{stream}/export_date={date:%Y-%m-%d}/{timestamp}-{stream}.json"

however, we should add some parsing logic to ensure that the naming convention provided is valid and update the documentation with some examples.

adjust-precision-for-schema merged in #75 and available in the v1.2.2 release @aroder .
Also closes #69 .
I believe this PR can now be closed unless reasons not to.

All the functionalities are now implemented.


Add colours, improve the UI of pushing and creating.

This adds colours to the logging / development experience when creating
and pushing a component.
This PR improves the user / developer experience by providing a more
unified / more feedback approach to deployment.

I'd like some feedback, but I believe the UI / UX of Odo could be vastly improved.
This PR has the following changes:

A "spinner" added for the building progress
The the output from the container has been MOVED to debug output. I believe having this here isn't necessarily needed unless something goes wrong.
The formatting has been slightly changed.

Improvements upon further discussion:

As a compromise, instead of completely hiding the build progress, we could instead show a few lines instead while it builds...

This is a work in progress, but here is what I currently have:

:white_check_mark: Build odo 33 completed and artifacts can be found here (for commit https://github.com/redhat-developer/odo/commit/18f30e6ddb by @cdrage)

I've also coded an alternative that will actually show what's currently going on in the build:

I've also coded an alternative that will actually show what's currently going on in the build:

Not sure if this is good ux. It hides the log messages, I found those useful.

@jorgemoralespou what do you think about this?
BTW, we have PR builds working, ;-) you can download build for this pr here https://ci.appveyor.com/project/odo-bot/odo/builds/19983930/artifacts

@kadel The issue I have is that it's quite an extensive build log output.. I was thinking that if it actually fails, the user can check it with verbosity.

@kadel I agree with you that build output is very useful. Sometimes our images (I guess) just spill out too many things, and I understand why @cdrage opted to hide it.
I would definitely show it by default as I think that most of the times I will want to see the output, although I have to admit that the output shown as it is is slick, and much nicer, specially when the output of the build is such a mess as in the nodejs image example below.
Maybe add a global config option to have the default behaviour defined by the user as a preference.
This:
$ ./odo push
Component: frontend
--> frontend: Pushing changes to component
--> frontend: Building component
OK  frontend: Changes successfully pushed
```

vs this:
```
$ odo push
Pushing changes to component: frontend
Please wait, building component....
+ set -eo pipefail
+ '[' -f /opt/app-root/src/.s2i/bin/assemble ']'
+ '[' -f /usr/local/s2i/assemble ']'
+ /usr/libexec/s2i/assemble
---> Installing application source
---> Building your Node application from source
Current git config
<EMAIL_ADDRESS>url.https://.insteadof=ssh://
<EMAIL_ADDRESS>core.repositoryformatversion=0
core.filemode=true
core.bare=false
core.logallrefupdates=true
core.ignorecase=true
core.precomposeunicode=true
remote.origin.url=https://github.com/openshift-evangelists/Wild-West-Frontend
remote.origin.fetch=+refs/heads/*:refs/remotes/origin/*
branch.master.remote=origin
branch.master.merge=refs/heads/master
---> Installing dependencies
---> Using 'npm install -s --only=production'
audited 119 packages in 2.894s
found 1 low severity vulnerability
  run `npm audit fix` to fix them, or `npm audit` for details
---> Pruning the development dependencies
npm info it worked if it ends with ok
npm info using<EMAIL_ADDRESS>npm info using<EMAIL_ADDRESS>npm timing stage:loadCurrentTree Completed in 698ms
npm timing stage:loadIdealTree:cloneCurrentTree Completed in 2ms
npm timing stage:loadIdealTree:loadShrinkwrap Completed in 286ms
npm timing stage:loadIdealTree:loadAllDepsIntoIdealTree Completed in 195ms
npm timing stage:loadIdealTree Completed in 577ms
npm timing stage:generateActionsToTake Completed in 13ms
npm timing audit compress Completed in 7ms
npm info audit Submitting payload of 5450bytes
npm timing stage:executeActions Completed in 514ms
npm timing stage:rollbackFailedOptional Completed in 1ms
npm timing stage:runTopLevelLifecycles Completed in 1900ms
npm timing audit submit Completed in 1364ms
npm http fetch POST 200 https://registry.npmjs.org/-/npm/v1/security/audits/quick 974ms
npm timing audit body Completed in 1ms
audited 119 packages in 2.762s
found 1 low severity vulnerability
  run `npm audit fix` to fix them, or `npm audit` for details
npm timing npm Completed in 3467ms
npm info---> Cleaning up npm cache
 ok
---> Fix permissions on app-root
chgrp: changing group of '/opt/app-root': Operation not permitted
chgrp: changing group of '/opt/app-root/etc': Operation not permitted
chgrp: changing group of '/opt/app-root/etc/passwd': Operation not permitted
chgrp: changing group of '/opt/app-root/etc/scl_enable': Operation not permitted
chgrp: changing group of '/opt/app-root/etc/install_node.sh': Operation not permitted
chgrp: changing group of '/opt/app-root/src': Operation not permitted
chgrp: changing group of '/opt/app-root/src/.env': Operation not permitted
chgrp: changing group of '/opt/app-root/src/assets': Operation not permitted
chgrp: changing group of '/opt/app-root/src/assets/gunshot.wav': Operation not permitted
chgrp: changing group of '/opt/app-root/src/assets/yeehaw.wav': Operation not permitted
chgrp: changing group of '/opt/app-root/src/assets/buildConfig.png': Operation not permitted
chgrp: changing group of '/opt/app-root/src/assets/gunsight.png': Operation not permitted
chgrp: changing group of '/opt/app-root/src/assets/storage.png': Operation not permitted
chgrp: changing group of '/opt/app-root/src/assets/route.png': Operation not permitted
chgrp: changing group of '/opt/app-root/src/assets/deploy.png': Operation not permitted
chgrp: changing group of '/opt/app-root/src/assets/game.js': Operation not permitted
chgrp: changing group of '/opt/app-root/src/assets/styles.css': Operation not permitted
chgrp: changing group of '/opt/app-root/src/assets/openshiftlogo.png': Operation not permitted
chgrp: changing group of '/opt/app-root/src/assets/explosion.wav': Operation not permitted
chgrp: changing group of '/opt/app-root/src/assets/service.png': Operation not permitted
chgrp: changing group of '/opt/app-root/src/assets/pod.png': Operation not permitted
chgrp: changing group of '/opt/app-root/src/assets/frame.png': Operation not permitted
chgrp: changing group of '/opt/app-root/src/assets/build.png': Operation not permitted
chgrp: changing group of '/opt/app-root/src/assets/gameplayfield.png': Operation not permitted
chgrp: changing group of '/opt/app-root/src/assets/sight.png': Operation not permitted
chgrp: changing group of '/opt/app-root/src/bin': Operation not permitted
chgrp: changing group of '/opt/app-root/src/bin/config.js': Operation not permitted
chgrp: changing group of '/opt/app-root/src/server.js': Operation not permitted
chgrp: changing group of '/opt/app-root/src/.pki': Operation not permitted
chgrp: changing group of '/opt/app-root/src/.pki/nssdb': Operation not permitted
chgrp: changing group of '/opt/app-root/src/README.md': Operation not permitted
chgrp: changing group of '/opt/app-root/src/.gitignore': Operation not permitted
chgrp: changing group of '/opt/app-root/src/.config': Operation not permitted
chgrp: changing group of '/opt/app-root/src/.config/configstore': Operation not permitted
chgrp: changing group of '/opt/app-root/src/.config/configstore/update-notifier-npm.json': Operation not permitted
...........
........... TRIMED AS OTHERWISE GITHUB WOULDN'T ALLOW MY COMMENT
...........
chgrp: changing group of '/opt/app-root/src/.npm-global/lib/node_modules/yarn/bin': Operation not permitted
chgrp: changing group of '/opt/app-root/src/.npm-global/lib/node_modules/yarn/bin/yarnpkg.cmd': Operation not permitted
chgrp: changing group of '/opt/app-root/src/.npm-global/lib/node_modules/yarn/bin/yarn.js': Operation not permitted
chgrp: changing group of '/opt/app-root/src/.npm-global/lib/node_modules/yarn/bin/yarn': Operation not permitted
chgrp: changing group of '/opt/app-root/src/.npm-global/lib/node_modules/yarn/bin/yarnpkg': Operation not permitted
chgrp: changing group of '/opt/app-root/src/.npm-global/lib/node_modules/yarn/bin/yarn.cmd': Operation not permitted
chgrp: changing group of '/opt/app-root/src/.npm-global/lib/node_modules/yarn/README.md': Operation not permitted
chgrp: changing group of '/opt/app-root/src/.npm-global/lib/node_modules/yarn/package.json': Operation not permitted
chgrp: changing group of '/opt/app-root/src/.npm-global/lib/node_modules/yarn/LICENSE': Operation not permitted
chgrp: changing group of '/opt/app-root/src/.npm-global/bin': Operation not permitted
chgrp: changing group of '/opt/app-root/src/.npm-global/bin/yarn': Operation not permitted
chgrp: changing group of '/opt/app-root/src/.npm-global/bin/yarnpkg': Operation not permitted
+ /var/lib/supervisord/bin/supervisord ctl stop run
run: stopped
+ /var/lib/supervisord/bin/supervisord ctl start run
run: started
changes successfully pushed to component: frontend
```

@jorgemoralespou
What about adding a parameter, that by default, it's hidden (unless there is an error). If a user would like to actually see the full progress, we could use:
odo push --progress

As an added parameter?
I find that this new implementation is a lot more modern and clean than the current one.

@cdrage we have discuss this (our team) and we agree that this output is nicer for user, and that probably providing two flags would help with experience:

--verbose (or similar) would show all the logs as now (maybe --follow)
--silent (or similar) should provide no intermediate output for automation (This could potentially be for every command, and just provide nice exit code)

@cdrage In addition to what @jorgemoralespou noted above, I would like to see a hint to the user of what they should do next when the command isn't successful. Otherwise, I am concerned that if there's a failure the user may not know what to do next and that could be frustrating.
For example, if I have a syntax error in my package.json file, then push will fail. It looks like this:
jkleiner-mac:Wild-West-Frontend jkleiner$ odo push
Component: frontend
--> frontend: Pushing changes to component
--> frontend: Building component
failed to push component: frontend

Maybe we could add a message that says something like "Run odo push --verbose to view log output." or some other message directing the user how to view logs to understand what went wrong.

I agree with both @jankleinert and @jorgemoralespou. Default output should be concise but there should be clear options to revert that behavior if needed. In particular, telling the user about the "verbose" mode in case of error would help a lot so that the user doesn't have to hunt down how to see the full output.

@kadel The issue I have is that it's quite an extensive build log output.. I was thinking that if it actually fails, the user can check it with verbosity.

But that would mean that he/she has to run push again. That is not a good workflow. When you are developing something you have build fails all the time.

@kadel We should hide it by default, but if it does error, we should actually show the failed output.

I've improved the UI. It now only shows the output if there is an error out, however, it still shows the output in debug.
Have a look at the new Asciinema / preview!

@kadel regarding your comment:

But that would mean that he/she has to run push again. That is not a good workflow. When you are developing something you have build fails all the time.

I think in case of failure, if the user didn't have verbose output odo should show a message like:
There was an error while pushing. To see the complete messages execute:
 odo log

This will help the user see the error without needing to re-push. But this might need us to be able to see build and run logs.

@geoand @jorgemoralespou @kadel
Does everyone agree on this new implementation? If so, I will go forward with implementing this throughout the rest of Odo.

+1 from me

@mik-dass Yeah, I know why now.. It seems that if we update "suffix" while it updates, it'll create the odd symbols. I've updated the PR!

Looks like I'm encounting this issue from the library: https://github.com/briandowns/spinner/pull/65 I'm going to put this on hold for now until we get this 100% working correctly :+1:

@cdrage why was this closed?


Journal error during upgrade to 9.2.2 RC

Clean install of 9.1.0 on Azure Websites with new SQL Azure DB.
Extract 9.2.2 RC upgrade zip over site and attempt in-place upgrade
All succeeds, with the exception of DNN Journal, which reports "Error", but nothing seems to be in Admin Log.

Post-upgrade, checked DNN Journal module, and still reports that version 9.1.0 is installed.

Can you check in the /Portals/_default/logs folder for an installer log file and check it for errors?

09/07/2018 13:03:07 [INFO] DotNetNuke.Services.Upgrade.Upgrade Start InstallPackage:D:\home\site\wwwroot\Install\Module\DNNCE_Journal_09.02.02_Install.zip
Is the only reference to “journal” in the log file

there is no text string match on the word “error” in the log file.

does this issue still persist in release version of DNN 9.2.2?

Since there was no reply to @sleupold comment since october I am assuming this got fixed with the recent improvements with sql install scripts and closing this issue for now.


SA1009 triggers with null forgiving operator

The following code is triggering SA1009:
provider => provider.GetService<PreferencesDbContextFactory>()!.CreateDbContext()

The only fix for this is to add a space between the closing parenthesis and the null forgiving operator to give
provider => provider.GetService<PreferencesDbContextFactory>() !.CreateDbContext()

which IMHO looks kind of weird. Seeing that it doesn't trigger with the null coalescing operator ?. I suspect this should be treated in the same way. With nullable reference types this is going to become more of an issue.

Does this still reproduce with the 1.2 beta releases? The 1.1 release doesn't support C# 8 syntax.

Appears that it does work as expected with 1.2 - thanks

Appears that it does work as expected with 1.2 - thanks

Duplicate of #3143


Introduce anonymous product analytics

Questions
Answers

Description?
This PR introduces anonymous analytics to know how many times the module is used to upgrade / restore and how.

Type?
new feature

BC breaks?
Yes, some files are being moved, i.e Tasks

Deprecations?
Nope

Fixed ticket?
Fixes https://github.com/PrestaShop/PrestaShop/discussions/36432

Sponsor company
@PrestaShopCorp

How to test?
Triggering updates and restores will be counted, as well as their ending (Successful or not).

It appeared during the tests that failing upgrade are not reported in case of a PHP timeout. There is room for improvement.

@Quetzacoalt91 Maybe it's the moment where register_shutdown_function() shines

Indeed, we already have it to send responses gracefully, so it may be completed with this feature.


Add back inbucket (supabase-mail) and disable autoconfirm

Any pointers in case I'm able to contribute?

Hey, sorry this one's lacking context - I saw you went ahead and made a PR 😅
For this one the end goal is to do custom SMTP through the OSS dashboard once it's integrated, and make inbucket opt-in since you may still need to use an email testing server.
So the current approach (autoconfirm, to answer your 2nd question) and inbucket are stopgaps so you can still test emails to some extent. But having inbucket now is fine I think, as long as it's opt-in. I'll add some comments to your PR tomorrow (past midnight atm)

Ah I see. I didn't make it opt in - not sure I'm able too currently since this is my first time even reading golang 😅
On the second question. I don't understand how autoconfirm can help me login though? My understand is that it simply flags confirmation in the auth.users table but I'm probably wrong.

It's basically the same as this:

Sorry but I don't understand. What is the same as those four switches. As I understand the email confirmation; you need to confirm your email in order to login. So with autoconfirm you enable users to login without confirming basically.
My issue is that I want to login with magic links so there is no way to login without actually receiving those magic links.

Ah, I see. I'm not familiar with the auth server, so I'm not sure if you can generate a magic link without sending an email. But for normal password login it works as you mentioned.

You can; https://supabase.com/docs/reference/javascript/auth-api-generatelink
However I don't want an unauthenticated user being able to do that. Then I should create some backend endpoint that only exists in development mode and allow anyone to generate those links at that endpoint.
With inbucket I'd be able to receive those emails instead which is much much nicer 😅

Actually now that I think about it. A workaround could be that if I'm working locally then I console log the generate link instead of sending an invite mail.
Still the inbucket solution is better imo though.

I just did some testing.
With the current setup with auto confirmation no links are generated. Not even using serverSupabase.auth.api.generateLink. It does work in my PR though


added new options for query string tracking

Scope & Purpose
Add new options for query string tracking

[ ] :hankey: Bugfix
[x] :pizza: New feature
[ ] :hammer: Refactoring
[x] :book: CHANGELOG entry made
[x] :muscle: The behavior in this PR was manually tested
[ ] :computer: The behavior change can be verified via automatic tests

http://<IP_ADDRESS>:8080/view/PR/job/arangodb-matrix-pr/12230/

http://<IP_ADDRESS>:8080/view/PR/job/arangodb-matrix-pr/12241/


[Question] How to bypass HTTPError 429: 'Too Many Requests'

Checklist

[X] I'm asking a question
[X] I've looked through the README and FAQ for similar questions
[X] I've searched the bugtracker for similar questions including closed ones

Question
I want to know how to bypass HTTPError 429: 'Too Many Requests' error message. This issue frequently appears. To solve it I restart my router whenever it occurs (for renewing the public IP).
Is there any workaround to bypass it without needing to restart my router or waiting google for ending the restriction?
Log
$ youtube-dlc --newline -i --force-ipv4 --age-limit 30 -R 99 --hls-prefer-native -o "/home/laptop/Downloads/youtube-dl/downloads/Documentary/Nat Geo Abu Dhabi/%(title)s.%(ext)s" -f mp4 --merge-output-format mp4 --write-sub --verbose --continue https://www.youtube.com/user/natgeoabudhabime/videos
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['--newline', '-i', '--force-ipv4', '--age-limit', '30', '-R', '99', '--hls-prefer-native', '-o', '/home/laptop/Downloads/youtube-dl/downloads/Documentary/Nat Geo Abu Dhabi/%(title)s.%(ext)s', '-f', 'mp4', '--merge-output-format', 'mp4', '--write-sub', '--verbose', '--continue', 'https://www.youtube.com/user/natgeoabudhabime/videos']
[debug] Loading archive file None
[debug] Encodings: locale UTF-8, fs utf-8, out utf-8, pref UTF-8
[debug] youtube-dlc version 2020.10.09
[debug] Python version 3.8.5 (CPython) - Linux-5.4.0-51-generic-x86_64-with-glibc2.29
[debug] exe versions: ffmpeg 4.2.4, ffprobe 4.2.4
[debug] Proxy map: {}
[youtube:user] natgeoabudhabime: Downloading channel page
[youtube:playlist] UULiTe0aOHShx7hXGyqZ9UIw: Downloading webpage
[download] Downloading playlist: Uploads from Nat Geo Abu Dhabi
[youtube:playlist] UULiTe0aOHShx7hXGyqZ9UIw: Downloading page #1
[youtube:playlist] UULiTe0aOHShx7hXGyqZ9UIw: Downloading page #2
[youtube:playlist] UULiTe0aOHShx7hXGyqZ9UIw: Downloading page #3
[youtube:playlist] UULiTe0aOHShx7hXGyqZ9UIw: Downloading page #4
[youtube:playlist] UULiTe0aOHShx7hXGyqZ9UIw: Downloading page #5
[youtube:playlist] UULiTe0aOHShx7hXGyqZ9UIw: Downloading page #6
[youtube:playlist] UULiTe0aOHShx7hXGyqZ9UIw: Downloading page #7
[youtube:playlist] UULiTe0aOHShx7hXGyqZ9UIw: Downloading page #8
[youtube:playlist] UULiTe0aOHShx7hXGyqZ9UIw: Downloading page #9
[youtube:playlist] UULiTe0aOHShx7hXGyqZ9UIw: Downloading page #10
[youtube:playlist] UULiTe0aOHShx7hXGyqZ9UIw: Downloading page #11
[youtube:playlist] playlist Uploads from Nat Geo Abu Dhabi: Downloading 1119 videos
[download] Downloading video 1 of 1119
[youtube] R9K9NC2MpDs: Downloading webpage
ERROR: Unable to download webpage: HTTP Error 429: Too Many Requests (caused by <HTTPError 429: 'Too Many Requests'>); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dlc with the --verbose flag and include its complete output.
  File "/home/laptop/.local/lib/python3.8/site-packages/youtube_dlc/extractor/common.py", line 632, in _request_webpage
    return self._downloader.urlopen(url_or_request)
  File "/home/laptop/.local/lib/python3.8/site-packages/youtube_dlc/YoutubeDL.py", line 2308, in urlopen
    return self._opener.open(req, timeout=self._socket_timeout)
  File "/usr/lib/python3.8/urllib/request.py", line 531, in open
    response = meth(req, response)
  File "/usr/lib/python3.8/urllib/request.py", line 640, in http_response
    response = self.parent.error(
  File "/usr/lib/python3.8/urllib/request.py", line 569, in error
    return self._call_chain(*args)
  File "/usr/lib/python3.8/urllib/request.py", line 502, in _call_chain
    result = func(*args)
  File "/usr/lib/python3.8/urllib/request.py", line 649, in http_error_default
    raise HTTPError(req.full_url, code, msg, hdrs, fp)

[download] Downloading video 2 of 1119
[youtube] d4lwAIglTrE: Downloading webpage
ERROR: Unable to download webpage: HTTP Error 429: Too Many Requests (caused by <HTTPError 429: 'Too Many Requests'>); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dlc with the --verbose flag and include its complete output.
  File "/home/laptop/.local/lib/python3.8/site-packages/youtube_dlc/extractor/common.py", line 632, in _request_webpage
    return self._downloader.urlopen(url_or_request)
  File "/home/laptop/.local/lib/python3.8/site-packages/youtube_dlc/YoutubeDL.py", line 2308, in urlopen
    return self._opener.open(req, timeout=self._socket_timeout)
  File "/usr/lib/python3.8/urllib/request.py", line 531, in open
    response = meth(req, response)
  File "/usr/lib/python3.8/urllib/request.py", line 640, in http_response
    response = self.parent.error(
  File "/usr/lib/python3.8/urllib/request.py", line 569, in error
    return self._call_chain(*args)
  File "/usr/lib/python3.8/urllib/request.py", line 502, in _call_chain
    result = func(*args)
  File "/usr/lib/python3.8/urllib/request.py", line 649, in http_error_default
    raise HTTPError(req.full_url, code, msg, hdrs, fp)
^C
ERROR: Interrupted by user
$

NOTE: I tried to use --cookies but it didn't solve it.

There is --sleep-interval however that will only sleep between downloads of files and not between (api) calls like Downloading page.
When I was working on the youtube history feed fix (which is not released btw)

I had to rate limit calls with sleep.
6 Seconds as in the example is a bit generous. It seems 1-3 seconds (where you should be completely safe with 3 seconds) is actually enough.
Furthermore it also seems this problem is rather related to youtube only as I haven't noticed any other service with similar problems.

try using a proxy or a VPN - https://github.com/ytdl-org/youtube-dl/issues/22839

@october262 that won't always work out. It's the anti bot design by YouTube. So the tools (youtube-dlc in this case) needs to adjust to that. As I already mentioned all that is needed is some timeout between calls. Current behaviour is easy to identify  as bots. But normal user behaviour is not able to process so many request at once so you have to mirror this behaviour.


MissingPluginException

Hello all,
I'm trying to install bundle of an app on raspberry/raspbian using flutter-pi .(giving some context).
When I execute the command to run the application in command line, I received this error:
[ERROR:flutter/lib/ui/ui_dart_state.cc(198)] Unhandled Exception: MissingPluginException(No implementation found for method ensureInitialized on channel window_manager) #0      MethodChannel._invokeMethod (package:flutter/src/services/platform_channel.dart:165:7) <asynchronous suspension> #1      WindowManager.ensureInitialized (package:window_manager/src/window_manager.dart:93:5) <asynchronous suspension> #2      main (package:totems/main.dart:25:5) <asynchronous suspension>
Do you think is something about compatibility of the window manager with linux/raspbian?
Thanks

Have you resolved ?

And which one do you use ?


Fix duplicate calculation of group member

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 99.972%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
3566

Relevant Lines:
3567

💛  - Coveralls


feat: include anvil-zksync in foundryup install script

Pending release of: https://github.com/matter-labs/era-test-node/pull/444.
CI expected to fail until era_test_node renamed to anvil-zksync
What :computer:

include anvil-zksync in foundryup install script

Why :hand:

allow users to make use of anvil-zksync

Evidence :camera:
Include screenshots, screen recordings, or console output here demonstrating that your changes work as intended

@dutterbutter is it time to hit the merge button? :)

@popzxc could you review please


[Feature Request] Refactor code to simplify the concepts

Motivation
We need to clean up code so that the API is easier to study and use. List of required changes:

[x] Remove UserChatMessage and AssistantChatMessage
[x] Remove AssistantSystemMessage and UserSystemMessage
[x] Rename role from ChatMessage into role_at_backend
[x] Remove role from ChatMessage altogether
[ ] Make BaseMessage, ChatMessage, SystemMessage a frozen=True dataclass
[x] Remove def __getattribute__ from BaseMessage (https://github.com/camel-ai/camel/pull/168)
[ ] Replace type of content in BaseMessage with TextPrompt
[ ] Rename model to model_type
[x] Fix mode_type typos (https://github.com/camel-ai/camel/pull/166, https://github.com/camel-ai/camel/pull/169)
[x] Extract RolePlaying into a separate societies package
[ ] Strict signature match for BaseAgent child classes

Solution
No response
Alternatives
No response
Additional context
No response

I think mostly the refectoring is done. The remaining open points we will address later on.

What should we remove role from ChatMessage? I'm planning to replace ChatAgent.stored_messages with memory module. It's inconvenient for memory API design if not all message-related information stores in BaseMessage.


Model Weights

Hello,
I am a master student in the field of Face expression recognition at Ecole de Technologie Superieure (ETS) of Montreal. I would be interesting in obtaining model weights for AUNets. Could you provide them ? Thanks a lot for sharing.
Best Regards,
Thomas TEIXEIRA
Master Student at Ecole de Technologie Superieure (Montreal)

Hello, thank you for your interest, you can find them here http://<IP_ADDRESS>/weights/AUNets/


Tailwind CSS is not updated

Hello,
I am experienced with the issue where Tailwind CSS is not applied after changed.
My current workaround is to import the css file directly in dev mode:
import fontStyleSheetUrl from './styles/fonts-stylesheet.css?url'
process.env.NODE_ENV === 'development' && import('./styles/tailwind.css')

I look forward to hearing from you.

Odd. This seems like a vite bug. Could you try to reproduce this without the epic stack to help determine what the actual problem could be?

I'm pretty sure it's a hydration issue. (It's always a hydration issue)
Use React Canary, and Tailwind+HMR should work correctly.

I tried to reproduce this without epic stack. Also, I tried it with and without React Canary and it still occurred. Here is the link to the reproduce code: https://stackblitz.com/edit/remix-run-remix-3nnng7?file=app%2Froot.tsx
But I think that this line in links is the reason because when I removed it, the code worked properly
export const links: LinksFunction = () => [
  ...
  // Preload CSS as a resource to avoid render blocking
  { rel: 'preload', href: tailwindStyleSheetUrl, as: 'style' },
  ...
];

We're probably safe to remove that since the stylesheet shows up in the HTML pretty immediately anyway. I don't think having a preload a couple lines above it in the HTML is going to make a difference.


[Issue] UE Links with Group RegEx using simple string input not updating when Group Target Regex string is updated

Steps to reproduce:
Using the test templates provided:

load the 3 test test templates as templates through the template manager (right click>node templates>manage>import
place template "test_ue_links_string_direct"
place template "GroupX". If not using AegisFlow Utility Nodes, place in a group called "GroupX"
Place a second "GroupX", and place in a group called "GroupY" If you are using AF utility nodes, then change the auto-named group to "GroupY" from "GroupX"
Place a second "test_ue_links_string_direct". This will produce clashes with GroupX's inputs.
Change the purple top node string to "GroupY". Clashes should be resolved, but are not. Wiggle some nodes; fix is not applied.
Refresh page in browser. All clashes are removed.
*Note: Test was performed w/ AegisFlow Util nodes disabled to ensure our auto-grouping and naming wasn't part of the problem)

Images and Video below:

after refresh

Video that shows the issue and what I think is happening:
https://www.youtube.com/watch?v=zxzpa56uILM
Zip of these 3 simple test templates (import using right click>node templates>manage>import
test_templates.zip
Confirming that the issue only happens when using Group Regex as an input; in widget mode the refresh is not required to clear the clash condition:

.

Confirmed that I can reproduce this error. Looking into it.

OK, pushed a fix for this one. Note that the fix only works if the upstream node is the provided Simple String.

resolved in 4.7.13


Scrake slow raging bug

If the scrake is both Burning and Crispified during sprinting/raged state, it will enter Slow Rage animation

Repro rate: ~60%

Alt: when a burning Scrake gets Crispified during the sprinting/raged state, it will Slow Rage until the burn wears off.

Repro rate: 100%
Steps:

Stun Scrake and push it under rage threhold
Ignite it immediately
Observe it entering slowrage as soon as Crispified is applied, and exiting it as soon as Burning is removed.

update: stunning a scrake during sprint/raging animation causes a slow rage as well.

Fixed by df4ee16dcaec0fd21e9206e6b8bab79cd0fc9e79.


removing feature flag

https://harness.slack.com/archives/C045V7DKHLG/p1678451529962019
Harness Developer Pull Request
Thanks for helping us make the Developer Hub better. The PR will be looked at
by the maintainers.
What Type of PR is This?

[ ] Issue
[ ] Feature
[x] Maintenance/Chore

If tied to an Issue, list the Issue(s) here:

Issue(s)

House Keeping
Some items to keep track of. Screen shots of changes are optional but would help the maintainers review quicker.

[ ] Tested Locally
[ ] Optional Screen Shoot.

Preview environment: https://hdh.pr.harness.io/pr-950


from_usize gives a generic, misleading panic message on out-of-bounds

Currently, the implementation of from_usize for derive(Enum) will panic if given an out-of-bounds integer. That seems fine.
However, the panic message is currently:
panicked at 'internal error: entered unreachable code'

To me, that suggests an internal logic error in the derived impl, which doesn't appear to be the case. This case is trivially reachable if the caller provides a bogus integer. unreachable is an unusual way of handling this; for instance, slice index out of bounds does not print unreachable but rather that a slice index was out of bounds.
As a user I think it'd be a better experience if the message were something like index out of range for Enum::from_usize.

Makes sense to me.
Historically, Enum trait methods were an undocumented implementation detail, but now that it is actually documented I probably should change the error message.


[main] Upgrade to latest dependencies

Cron -knative-prow-robot
/cc knative-sandbox/networking-wg-leads
/assign knative-sandbox/networking-wg-leads
Produced by: knative-sandbox/knobots/actions/update-deps
Details:
bumping knative.dev/networking ec52d71...55757e9:
  > 55757e9 Add HTTPS port variables (# 640)
  > ccab092 Adding code of conduct to reference knative/community's CoC (# 639)

/lgtm
/approve


Clarified how the build variables Build.SourcesDirectory and Build.Repository.LocalPath work (fixed partly wrong documentation)

Clarified how the build variables Build.SourcesDirectory and Build.Repository.LocalPath work. They are not synonymous and behave differently in some scenarios. Especially the behaviour of Build.Repository.LocalPath is currently documented wrong (or at least not to the full extend).
Since multi-checkouts are not possible with Azure DevOps Server 2019 we don't need to change the documentation for anything prior to Azure DevOps 2020 or the Azure DevOps Services.

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

Thanks @echalone!
#sign-off

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

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


luci-app-firewall: Missing icmp-type dropdown options for bad-header and unknown-header-type

I noticed that the default configuration of the firewall in openwrt includes the icmp-type bad-header and unknown-header-type for the 'Allow-ICMPv6-Input' and 'Allow-ICMPv6-Forward' (link to the file in the current master branch).
In the rules.js of luci-app-firewall both icmp-types are not a selectable option and when saving without any changes made, both icmp-types get removed.
My question is, if this is intended behaviour?
I tried search for both of those values and did not found anything. On the OpenWRT Wiki both types are absent too OpenWRT Wiki Link to ICMP Types.

The wiki page you link to only seems to cover IPv4.
https://www.iana.org/assignments/icmpv6-parameters/icmpv6-parameters.xml
running this on my 21-rc4 build:
ip6tables -p ipv6-icmp -h

dumps
Valid ICMPv6 Types:
destination-unreachable
   no-route
   communication-prohibited
   beyond-scope
   address-unreachable
   port-unreachable
   failed-policy
   reject-route
packet-too-big
time-exceeded (ttl-exceeded)
   ttl-zero-during-transit
   ttl-zero-during-reassembly
parameter-problem
   bad-header
   unknown-header-type
   unknown-option
echo-request (ping)
echo-reply (pong)
router-solicitation
router-advertisement
neighbour-solicitation (neighbor-solicitation)
neighbour-advertisement (neighbor-advertisement)
redirect

Not sure what problems arise when one tries to assign ipv6 only types to IPv4 traffic when building rules. Might just be silently ignored, might not. Could be the reason why they were excluded, but that doesn't seem like the right approach.

Not sure what problems arise when one tries to assign ipv6 only types to IPv4 traffic when building rules. Might just be silently ignored, might not. Could be the reason why they were excluded, but that doesn't seem like the right approach.

Yeah, seems this is an omission. There are other ipv6 only types in there, so I created a PR which should resolve this in the master branch. Needs attention from some admins to land in 21.x

Cherry-picked to openwrt-21.02 https://github.com/openwrt/luci/commit/3b3c2e5f9f82372df8ff01ac65668be47690dcd5


int argument for take doesn't work

>>> numpy.arange(10).take(1)
1
>>> cupy.arange(10).take(1)
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "cupy/core/core.pyx", line 552, in cupy.core.core.ndarray.take (cupy/core/core.cpp:10310)
    cpdef ndarray take(self, indices, axis=None, out=None):
  File "cupy/core/core.pyx", line 560, in cupy.core.core.ndarray.take (cupy/core/core.cpp:10193)
    return _take(self, indices, axis, out)
  File "cupy/core/core.pyx", line 1683, in cupy.core.core._take (cupy/core/core.cpp:52483)
    a = rollaxis(a, axis)
TypeError: 'NoneType' object cannot be interpreted as an index

related to #852


⚠ haccp has degraded performance

In 0026fb7, haccp (https://www.haccp.or.kr) experienced degraded performance:

HTTP code: 400
Response time: 5353 ms

Resolved: haccp performance has improved in 8c2325b.


[BUG] Config breaking from the simple example widget?

Checklist before submitting an issue

[X] I have searched through the existing closed and open issues for eww and made sure this is not a duplicate
[X] I have specifically verified that this bug is not a common user error
[X] I am providing as much relevant information as I am able to in this bug report (Minimal config to reproduce the issue for example, if applicable)

Description of the bug
Ok so here we go. I should say I am not new to coding, Ive done this kinda thing a thousand times, I have gone over this so many times but still when i add
(defwindow example
:monitor 0
:geometry (geometry :x "0%"
:y "20px"
:width "90%"
:height "30px"
:anchor "top center")
:stacking "fg"
:reserve (struts :distance "40px" :side "top")
:windowtype "dock"
:wm-ignore false
"example content")
it works no issue.. I know it works because i Can edit the "example content"
I add this and the code still works. However I dont see the addition of a widget anywhere. I add this code above the window block
(defwidget greeter [?text name]
(box :orientation "horizontal"
:halign "center"
text
(button :onclick "notify-send 'Hello' 'Hello, ${name}'"
"Greet")))
but when i add
(defwindow example
; ... values omitted
(greeter :text "Say hello!"
:name "Tim"))
or I should say replace the window part defwindow so it says the last block the bar breaks. and no updates happen.
string and see it change on the bar. but when i add
Reproducing the issue
when i add
(defwindow example
; ... values omitted
(greeter :text "Say hello!"
:name "Tim"))
:monitor 0
:geometry (geometry :x "0%"
:y "20px"
:width "90%"
:height "30px"
:anchor "top center")
:stacking "fg"
:reserve (struts :distance "40px" :side "top")
:windowtype "dock"
:wm-ignore false
"example content")
it works no issue.. I know it works because i Can edit the "example content"
I add this and the code still works. However I dont see the addition of a widget anywhere. I add this code above the window block
(defwidget greeter [?text name]
(box :orientation "horizontal"
:halign "center"
text
(button :onclick "notify-send 'Hello' 'Hello, ${name}'"
"Greet")))
but when i add
(defwindow example
; ... values omitted
(greeter :text "Say hello!"
:name "Tim"))
or I should say replace the window part defwindow so it says the last block the bar breaks. and no updates happen.
string and see it change on the bar. but when i add the new code to the origional block to get this (defwindow example
; ... values omitted
(greeter :text "Say hello!"
:name "Tim"))
:monitor 0
:geometry (geometry :x "0%"
:y "20px"
:width "90%"
:height "30px"
:anchor "top center")
:stacking "fg"
:reserve (struts :distance "40px" :side "top")
:windowtype "dock"
:wm-ignore false
"example content")
everything breaks and nothing works
Expected behaviour
i expect for the bar to get the new addition
Additional context
No response

try
(defwindow example
           :monitor 0
           :geometry (geometry :x "0%"
                               :y "20px"
                               :width "90%"
                               :height "30px"
                               :anchor "top center")
           :stacking "fg"
           :reserve (struts :distance "40px" :side "top")
           :windowtype "dock"
           :wm-ignore false
  (greeter :text "Say hello!"
           :name "Tim"))

(defwidget greeter [?text name]
  (box :orientation "horizontal"
       :halign "center"
    text
    (button :onclick "notify-send 'Hello' 'Hello, ${name}'"
      "Greet")))

as it shows in the eww configuration website
the greeter widget should probably come after the propreties of the example window.

oh alright, when i learn a new code style i tend to do these guides exact  because I cant tell what needs to be there and what doesn't. I also put the widget above because alll the example bars ive been looking at have the window block at the bottom or towards the bottom.  thanks for the advice.  any other words of wisdom to help with learning this coding style?  again i appreciate the help


Make Timing::sample const and change asc::ChaiScript constructors

Timing::sample can be made const without consequence as it doesn't modify.
Ascent's version of ChaiScript declares copy/move constructors/assignments as default.
When I compiled Ascent's ChaiScript under clang, I received multiple warnings that these functions are implicitly deleted:
/ascent/include/ascent/ChaiEngine.h:117:19: warning: explicitly defaulted move assignment operator is implicitly deleted [-Wdefaulted-function-deleted]
ChaiEngine& operator=(ChaiEngine&&) = default;
ascent/include/ascent/ChaiEngine.h:114:7: warning: explicitly defaulted copy constructor is implicitly deleted [-Wdefaulted-function-deleted]
ChaiEngine(const ChaiEngine&) = default;
and so on...

Thanks!


My program runs without errors, but does not respond.

Hello
I copied the code in the following path. In VS 2015 Windows application (With a few minor changes)
"https://github.com/TelegramBots/telegram.bot.examples/tree/master/Telegram.Bot.Examples.Echo"
The program runs without errors, but does not respond to messages.
what is the problem?
tanks

@mahdibg You didn't follow the issue template and gave very little info. No wonder no body commented.
Feel free to edit your post according to the template and reopen this.


Better error handling for PatientDatabase constructor

Is your feature request related to a problem? Please describe.
Error messages for loading a PatientDatabase object with an invalid argument are cryptic and unhelpful
Describe the solution you'd like
Better error handling and a more clear description of the problem e.g., "Path passed to PatientDatabase constructor was invalid: {invalid path}"

piton.datasets.PatientDatabase(None) now returns a better error message. The non existent directory error still needs to get improved.


Analytics for multiple formula should output single table

Provide a detailed description of the proposed feature
brew info --analytics formula1 formula2 should output single table instead of 2 separate ones
What is the motivation for the feature?
The whole point of using brew info --analytics looks like comparison of "popularity"
How will the feature be relevant to at least 90% of Homebrew users?
Make better decisions when installing.
What alternatives to the feature have been considered?
Filtering current output with grep or jq

Would review a pull request for this but passing on this as a feature request for maintainers or soliciting our community for now, sorry!


Migrate constants that are only used in Serving

The dual to https://github.com/knative/serving/pull/9636/
/assign @tcnghia

/hold

/lgtm
/unhold


Application.LogCallback is obsolete?

So using the google analytics guide, I've come across this console error. Where do I take it from here? I don't really know much about coding.

In file GoogleAnalyticsV4.cs line 122 there is
#if UNITY_5_0
it should be replaced by
#if UNITY_5
so all 5.x versions of Unity may work without warning.