Enable non-constructor use of shape inference

'validate_nodes_and_infer_types' now runs shape inference on all underlying nodes instead of requiring the IN_TRANSITION in node.hpp to be set.
Note that this is currently called from the pass manager after each pass.

This could break existing backends since shape inference will now be re-run after every pass in the pass manager. Backends should be OK as long as shape-inference for backend specific ops is idempotent.

I'm not sure how this'll interact with the elementwise broadcasting PlaidML does, but I think we can work around that if needed.

https://github.com/NervanaSystems/ngraph/blob/master/src/ngraph/runtime/plaidml/plaidml_compiler.cpp#L115-L119
With this PR, we will run shape validation/inference after every pass.


Second lesson

I completed Chapter 2 Js Basics

@koyuncuoglum95 great to see you working through the lessons! The best way to track your work is to create a fork of this repo to your account and merge your completed lessons to your own repo.
Closing this PR, as no changes to the lessons. As you have questions, feedback or an issue with any lesson please feel free to open a discussion or open a new issue. Thank you for using this curriculum!


In addition to Task<IEnumerable<>> output, support IAsyncEnumerable outputs

This would be a sensible addition, if possible

Seems like a reasonable request.
/cc @mhoeger

Closing this issue as we are tracking the progress of this enhancement in #773


fix(PeriphDrivers): Add Clock Setter API for all parts using UART RevB Drivers

Description
This PR is an extended addon to https://github.com/analogdevicesinc/msdk/pull/1108 for the UART RevB drivers. Since the parameters for the MXC_UART_RevB_SetFrequency were changed, the parts without these new functions did not properly set the baud rate.
Only tested on ME17 so far.
Checklist Before Requesting Review

[ ] PR Title follows correct guidelines.
[ ] Description of changes and all other relevant information.
[ ] (Optional) Link any related GitHub issues using a keyword
[ ] (Optional) Provide info on any relevant functional testing/validation.  For API changes or significant features, this is not optional.

/clang-format-run


add rake compiler to devel deps, and add Gemfile

The Rakefile loads rake/extensiontask which is present in the rake-compiler gem. The first commit in this PR adds a development dependency on this gem so that bundler will pull it in for the build.
This second commit in this PR adds a Gemfile to the tree. Prior to this change, Travis CI would not use bundler to install any dependencies in the gemspec.

Hi @vmg , would you mind merging this? It would be great to have these build fixes in.

Hi @vmg , just a friendly ping to please merge this PR?

Jeez so sorry, somehow this flew under my radar. Looks good!


MemoryCacheExtensions.RegisterChild() MAY be issue with the code

Hi MichaMo,
I am trying to use the  CacheManager.Microsoft.Extensions.Caching.Memory.
I found that the code for MemoryCacheExtensions.RegisterChild() is inappropriate or I could not understand.
internal static void RegisterChild(this MemoryCache cache, object parentKey, object childKey)
{
if (cache.TryGetValue(parentKey, out var temp))
{
var set = (HashSet)temp;
set.Add(childKey);
}
}
Variable "set" is not used with "cache" seems RegisterChild() method will be just called but will not do anything.
Can you explain what it does and how it maintains the region in - memory?
Thanks,
Keyur

Hi MichaMo,
I feel like the method would be similar to below, correct me if I am wrong.
internal static void RegisterChild(this MemoryCache cache, object parentKey, object childKey)
{
var set = new HashSet();
if (cache.TryGetValue(parentKey, out var temp))
{
set = (HashSet)temp;
set.Add(childKey);
}
else
{
set.Add(childKey);
}
cache.Set(parentKey, set);
}
Thanks,
Keyur

Ignore this I got it . its reference type so no need.
my bad !


Option to filter VCS ignored files

Similar to rg and fd, add a option to ignore paths in .gitignore, .ignore, .svnignore etc. It's really useful with the --tree option (maybe it should be active by default?)
We can use https://crates.io/crates/ignore as the mentionated libaries. I might make a pull-request about it this weekend, should be easy to plug the ignore lib in

Hi @augustobmoura ,
It seems a cool feature indeed. You PR will be welcomed!

I don't having it active by default is a good idea as it might conflict with people who wanna use lsd as an alternative to ls.  But having this feature under a flag would be awesome.

Turns out that the implementation is not as easy as I imagined (it's like a habit to me). The ignore crate works with a custom path walker and customs structs for representing files and directories, so probably we will need to rewrite a lot of code. I don't have much time to work on it, and probably a regular maintainer should be responsible for a refactor of this scale

Looking forward into this feature, as sometimes I want to list the tree but want to ignore the node_modules folder.

Is it feasible now that @sumitsahrawat dropped #275? It'd also be really nice to pass --ignore-file to my alias like fd too. I think ripgrep's ignore crate provides a lot of that out of the box.

Here is mine:
alias lt='lsd --tree $(IGNORE=$(git rev-parse --show-toplevel)/.gitignore; if [ -f "$IGNORE" ]; then xargs printf " --ignore-glob=%s" < "$IGNORE"; fi)'


NEW INDEX: SLAVI (Specific Leaf Area Vegetation Index)

Please, complete the following information:
short-name-of-index=SpectralIndex(
    short_name='SLAVI',
    long_name='Specific Leaf Area Vegetation Index',
    formula='N/(R + S1)',
    reference='https://www.asprs.org/wp-content/uploads/pers/2000journal/february/2000_feb_183-191.pdf',
    application_domain='vegetation',
    date_of_addition='2023-04-18',
    contributor="https://github.com/geoSanjeeb"
)

Thank you for your contribution!
The new index will be available in the next version of ASI!


Test & Evaluate Etherium Performance on RasPi or other IoT. Check if it can be included in project

Compile or use the existing Geth build for ARM architecture from http://ethereum.gitbooks.io/frontier-guide/content/installing_arm.html
Just use Geth for JSON RPC and synced local Ethereum Blockchain.

Tests in VM shows Geth will support RasPi 2, but still more testing is required if it can run along side NXT over RasPi 2.
More performance testing is needed.
Test on Odroid is needed.

Tested Precompiled Geth from https://github.com/ethereum/go-ethereum/wiki/Installation-Instructions-for-ARM
In few hours of performance testing

Geth runs fine,
uses less system resources (RAM/CPU),
Sync with the full blockchain in few hours,
answers to JSON-RPC queries on port 8545

CPU usage could be an issue on older models of RasPi. Recommended Hardware to run Geth along side NXT would be then RasPi 2 or above.
Things to do next:

Make further tests with building Geth on RasPi, so ethereum can be set to autoupdate as new builds comes out.

Separate tasks will be created for easy install and easy manage through Control Centre.


Google-Fit connector object not defined

I have followed the instruction here: https://github.com/googledatastudio/community-connectors/blob/master/deploy.md
When I connect\run the App Script I get an error. This is what the StackDriver logs are saying.
Jan 18, 2020, 10:40:22 AM	Error	ReferenceError: "connector" is not defined.
at getAuthType(main.js:27)
I'm OKish with JavaScript and it looks like the connector object is not defined..... :-p
Running through the code here https://github.com/googledatastudio/community-connectors/tree/master/google-fit/src I don't actually see it being declared anywhere. Should be be creating the object from the name of the file or something else occurring?
Thanks in advance,
Matt

I got the same error, but looking through the diffs I realized that the connector's functions have been factored out to the global scope. This means that we do not need main.js' functions anymore.
For example, check out the getSchema definition here: https://github.com/googledatastudio/community-connectors/commit/460965a73bed59518da078a917892516affa40d5#diff-c78a44a34e314eeb97ef993b080f3935L379.
Actually, it seems we do not need main.js anymore.
Nonetheless, after succesfully connecting, I got stuck on issue  #266 .


Error: Twitter icon when clicked points to another profile

Describe the Error
On clicking twitter icon, it points to another account
Expected behaviour
When twitter icon is pressed, it should point to https://x.com/sanjay_kv_/
Additional context

Kindly assign me this issue


ocs starting csv option removed

:+1: No more specific release requests, just rely on channel latest.


feat: Added support for connection_attempts, connection_timeout, and origin_shield

This adds support for connection_attempts, connection_timeout, and origin_shield.
https://github.com/hashicorp/terraform-provider-aws/releases/tag/v3.43.0

Thanks, @dudicoco !
v2.7.0 has been just released.


Code implementation with out Nerf Studio

Hello @LingzheZhao!
Do you have code of bad-gaussians which is implemented in vanilla gaussian splatting framework without nerf studio? If yes, can you please share. It will be really helpful for my research.
I am trying to understand the code and do some modifications to it as per my requirement. As it is in Nerf Studio framework, I am facing difficulty.
Thank you
Gopi

Gentle reminder of the same!

Hi Gopi! I uploaded the code without nerfstudio on the gsplat-only branch. I will also update the documentation in the next few days. Hope this helps your project and the community!

Thank you so much Lingzhe, very much appreciated!


update to latest rust

hi,Guillaume
i regenerate two rs files with newest rust-protobuf and make some some updatem but i still get following compile errors, could you please take a look at it ? thanks
src/blocks.rs:19:36: 19:55 error: associated type bindings are not allowed here [E0229]
src/blocks.rs:19 impl<'a> Fn(&'a PrimitiveGroup) -> groups::OsmObjs<'a> for FnOsmObjs<'a> {
^~~~~~~~~~~~~~~~~~~
src/blocks.rs:34:36: 34:53 error: associated type bindings are not allowed here [E0229]
src/blocks.rs:34 impl<'a> Fn(&'a PrimitiveGroup) -> groups::Nodes<'a> for FnNodes<'a> {
^~~~~~~~~~~~~~~~~
src/blocks.rs:49:36: 49:52 error: associated type bindings are not allowed here [E0229]
src/blocks.rs:49 impl<'a> Fn(&'a PrimitiveGroup) -> groups::Ways<'a> for FnWays<'a> {
^~~~~~~~~~~~~~~~
src/blocks.rs:64:36: 64:57 error: associated type bindings are not allowed here [E0229]
src/blocks.rs:64 impl<'a> Fn(&'a PrimitiveGroup) -> groups::Relations<'a> for FnRelations<'a> {

Thanks a lot!
Your problem was that the Fn trait changed tu use associated type. Some more fixes and it should now work!


fixed printing of log statements even if @@logging==false

--dpree

+1 it's quite offputting :) Wow, this pull request is nearly 10 months old

A more comprehensive solution in https://github.com/appoxy/mini_fb/pull/37.
Close this, @treeder


Add a ton of spec links

CC @nox

Thanks a lot for that! Also I like your style where you use /** and avoid comment noise, I wonder if screen readers prefer that…

bors r= @nox
Cool. Yeah, I did have screen readers in mind when I wrote it that way.


feat: add onClose to Modal options

Could you please consider to add onClose to the Modal options to allow to perform some actions ?
Currently the modal is automatically closed on blur but there is no way to catch the event.

Thank you for the suggestion, it is indeed a good idea. We will work on this as soon as possible and let you know when it's ready 🙏


[5.3] Model::toJson() silently fails on error

I had not UTF-8 chars into a legacy database and this method was silently failing.
I think it's a good impovement to throw an InvalidArgumentException

Maybe this should go to 5.4 rather than 5.3?

ok for 5.4


Question: Issue template format for workshops

@tumido @vpavlin
What about having a GitHub template issue for a problem in any of the tasks the user has to perform in a workshop, they can cc us and we can review the problem there (virtual event with Github) so we can receive notification and we don’t lose track of anyone.
something like
---
name: Workshop issue
about: Task issue during workshop
labels: ws-ml-prague-issue
title: Workshop issue ML Prague
assignees: @vpavlin @pacospace
---

**Task**
A clear and concise description of the task you have to perform.

**Describe the issue**
A clear and concise description of what problem are you encountering.

**Screenshots**
If applicable, add screenshots to help explain your problem.

**Additional context**
Add any other context about the problem here.

We can explain issues live during the workshop.

@tumido @vpavlin cc

Related-to: https://github.com/operate-first/support/issues/77

I like that idea! :+1:

this also enables us to have more people working and helping on the workshop in the background.... +1

That's great! +1


Fix panics in ecs_guide example

Objective
Fixes #7565
ecs_guide example is currently panicking. This seems to just be a problem with the example only caused by the base sets PR.
Solution
First, changed a few in_set to in_base_set to fix a few of
thread 'main' panicked at 'Systems cannot be added to 'base' system sets using 'in_set'. Use 'in_base_set' instead.', examples/ecs/ecs_guide.rs:301:45

And then added an in_base_set to fix the resulting (confusing) cycle error
2023-02-06T13:54:29.213843Z ERROR bevy_ecs::schedule_v3::schedule: schedule contains at least 1 cycle(s) -- cycle(s) found within:
 ---- 0: ["ecs_guide::game_over_system", "ecs_guide::score_check_system"]

I also changed this add_system call so the comment above and below make sense:
         // add_system(system) adds systems to the Update system set by default
         // However we can manually specify the set if we want to. The following is equivalent to
         // add_system(score_system)
-        .add_system(score_system)
+        .add_system(score_system.in_base_set(CoreSet::Update))
         // There are other `CoreSets`, such as `Last` which runs at the very end of each run.

Notes

Does MySet even need to be a base set? Seems like yes.
Is that cycle error when there is no explicit in_base_set actually expected?

This comment also no longer makes sense:
        // We can check our systems for execution order ambiguities by examining the output produced
        // in the console by using the `LogPlugin` and adding the following Resource to our App :)
        // Be aware that not everything reported by this checker is a potential problem, you'll have
        // to make that judgement yourself.

This could be fixed by

adding ambiguity reporting
removing the comment and LogPlugin entirely
making some sort of callout to the nondeterministic_system_order example?

This comment also no longer makes sense:
        // We can check our systems for execution order ambiguities by examining the output produced
        // in the console by using the `LogPlugin` and adding the following Resource to our App :)
        // Be aware that not everything reported by this checker is a potential problem, you'll have
        // to make that judgement yourself.

This could be fixed by

adding ambiguity reporting back in.
removing the comment and LogPlugin entirely
making some sort of callout to the nondeterministic_system_order example?

Remove the comment and LogPlugin IMO. There's a dedicated example now.

bors r+


Add quotes for inputs and output paths as they can contain spaces

This PR aims to ensure spaces in input or output paths work.
Otherwise an error occurs as you can see in this build result:

Hopefully @fedeoliv can have a look on that :)

Good catch, thanks @jbpaux! 😊


Splunk and ES output issue

Dear team.
Is there support for below bug, it is blocking our solution, hope it can be fixed offically.
https://github.com/banzaicloud/logging-operator/issues/907
Thanks

@g-traveller I've created a pr for the splunk team, I hope this will solve the problem. You can follow the status on the origin issue.


Infer response schema

Hi!
I've just started using this library and I'm a bit confused about the response type. Is it correct that I need to add a response type schema to each hook manually?
Would it generally be possible to infer the response schema from the type information in the elysia object like it's done in eden treaty or am I missing some conceptual differences here?
This would be super useful.

If you're meaning the typebox schema, yes it's necessary as Typescript's type information is not available at the runtime.

Thank you for answering my question and sorry for the late reply. I see that this is making inference of response types impossible currently.
Nevertheless, having correct documentation of the response for apis that don't explicitly describe the response would be super nice. An app of a customer for instance uses drizzle and eden treaty and therefore profits from end to end type safety. But the swagger docs don't reflect that and it's also not possible to generate api clients for other languages over the openapi spec (would be great for the mobile app).
I think that there should be some kind of CLI included with this package that can be used to generate a json file that contains the missing type information for the response schema so that this can then be used at runtime. I've already played around with the TS compiler APIs and I think that an approach would be totally feasible. Additionally this can be a purely optional thing in the library.
Please let me know what you think about it and if something like this could be merged into the library if I came up with the code.


feat: use prometheus kurtosis package

Moves setup of prometheus server into a kurtosis package to simplify launch_prometheus logic. Now launch_prometheus logic only configures metrics jobs as opposed to configuring metrics jobs and configuring + starting a prometheus server.

Looks like it doesn't work for me locally:
ethereum-package git:(tedi/prometheus) kt run . --args-file .github/tests/mix-with-tools.yaml
INFO[2023-12-11T11:05:19+01:00] Creating a new enclave for Starlark to run inside...
INFO[2023-12-11T11:05:20+01:00] Enclave 'looming-chasm' created successfully
INFO[2023-12-11T11:05:20+01:00] Executing Starlark package at '/Users/bbusa/Documents/Ethereum.nosync/testnets/kurtosis/ethereum-package' as the passed argument '.' looks like a directory
INFO[2023-12-11T11:05:20+01:00] Compressing package 'github.com/kurtosis-tech/ethereum-package' at '.' for upload
INFO[2023-12-11T11:05:21+01:00] Uploading and executing package 'github.com/kurtosis-tech/ethereum-package'
There was an error interpreting Starlark code
Evaluation error: Multiple errors caught interpreting the Starlark script. Listing each of them below.
        at [github.com/kurtosis-tech/prometheus-package/main.star:44:41]: undefined: service_metrics_configs
        at [github.com/kurtosis-tech/ethereum-package/src/prometheus/prometheus_launcher.star:2:27]: <toplevel>
        at [github.com/kurtosis-tech/ethereum-package/src/beacon_metrics_gazer/beacon_metrics_gazer_launcher.star:2:27]: <toplevel>
        at [github.com/kurtosis-tech/ethereum-package/main.star:20:37]: <toplevel>

Error encountered running Starlark code.

Ah my bad. Try again, I had to fix a typo in the prometheus-package

@tedim52 could you resolve the conflict, then lets get this merged.


Edited Bolos Are Not Returned In API

What version of Node.js are you using?
10.19.0
What version of npm are you using?
No response
What operating system are you using?
Ubuntu
What version of SnailyCAD are you currently using?
1.47.1
Describe the Bug
Editing the plate of a bolo after it has been created does not change the results of bolo API requests.
Expected Behavior
Bolo API requests (/bolos?query={new_bolo}) using the new bolo plate number should return the bolo status.
To Reproduce
Create a bolo with the wrong plate number.
Edit the bolo to contain the correct plate number.
Make an API request (/bolos?query={new_bolo}) with the correct plate number.

I'm unable to reproduce this. Can you provide a video reproduction of this?

Turns out I can't edit the bolo at all, as in it doesn't change in the UI either (I found this when I had a mix up between a zero and an o, so was difficult to tell in the UI.) (Due to file size limitations, had to put the video in Google Drive. https://drive.google.com/file/d/15swKHWhV9DOhEBDdhX9wHgzXLHlXG22h/view?usp=sharing)
As for the nodejs version, I am not 100% certain that is the version that Snaily is using, as I am not very familiar with connecting to docker containers to check their internal software versions. If the docker build pulled in a later nodejs version, then that should be the one that it is using. I just pulled the version native to my OS.


Add support for <oqs/rand.h>

We should also provide wrapping support for OQS_randombytes etc. from <oqs/rand.h>. I'm currently implementing this for the C++/Go wrappers.

https://github.com/open-quantum-safe/liboqs-python/issues/18#issuecomment-545622005

Optional feature, closing until someone wants to reopen and do this.


Refactor product-list component, mapping to SearchResultsListComponent

Using the generic page layout, we can now render CMS components using the backend configuration. The backend provides a SearchResultsListSlot slot with a "SearchResultsListComponent" component.
Please not that this refactor should be done in parallel with #1075 and #1076. The underlying component should be the same. We could potentially map the same component to multiple CMS types.
FR2 tickets for cmscomponrefactor going into epic/cmscomprefactor-fr2

@Matejk00 facet are not visible

@kacperknapik bug created #1610


USHIFT-1202: Use hostnames over IPs in apiserver when connecting to kubelet

Which issue(s) this PR addresses:

Closes #

This was tested to work in my local multi-node env.  LGTM

The implementation and new comments look OK, but it wasn't clear from the slack thread whether we still need this. Zenghui and Jeremy should take a look.
/assign @zshi-redhat @jerpeter1

I think we still want the first commit, which is a bug fix according to @pacevedom, not specific to multi-node.
Whether second commit is required depending on how kubelet runs on the extra worker node, will have a follow-up discussion.

/lgtm

I had to make a minor adjustment to my script, but I tested this and it works as intended.
/lgtm

/test e2e-openshift-conformance-reduced

/lgtm

/retest

/retest

/test e2e-openshift-conformance-reduced

/hold until https://github.com/openshift/release/pull/39662 is merged.

/test e2e-openshift-conformance-reduced

/lgtm

/override ci/prow/e2e-openshift-conformance-reduced-arm
/override ci/prow/microshift-e2e-arm
arm jobs are flaky due to a kernel bug.
/override ci/prow/e2e-openshift-conformance-reduced
Overriding conformance reduced as the failure relates to 1.27 rebase. Should this PR fail then any test trying to retrieve anything from the kubelet would fail, which is not the case.

/unhold

/override ci/prow/e2e-openshift-conformance-reduced


In data mapper, cannot select outer function call to add check expression

Description:
I tried to use lang.int:fromString() to convert a string to int. Now I want add check expression for the entire expression but cannot select it in statement editor.

Steps to reproduce:
Affected Versions:
OS, DB, other environment details and versions:
Related Issues (optional):

Suggested Labels (optional):

Suggested Assignees (optional):

Statement editor now provides code actions to add check expressions
https://user-images.githubusercontent.com/36252572/216002066-185c0307-61c6-47ea-a2b1-c6459d122ca5.MOV
Fixed with https://github.com/wso2-enterprise/ballerina-plugin-vscode/pull/687


Update manage-topics.md

Made a small update to the topic scores section

Changes committed - thanks!

@ruthholls
PR has been copyedited and is ready for final review, could you please check and merge? Thanks!


[Exam] Exams only clickable upon browser refresh

Describe the bug
As a student/instructor/tutor, go to the "exams" tab and try to select any visible exam.
Expected: we should be able to select an exam
Actual: we cannot select any exam. Clicking any 'exam' button does not work. After clicking the 'refresh' button, the exam is still not selectable. After refreshing the browser, then we can select an exam.
To Reproduce
Tested on test server 1 in Chrome browser

(As a student/instructor/tutor) Go to Course Overview -> Select a Course -> Select the 'Exams' tab
Try to select any visible exam
We are not able to select an exam
Try the refresh button and selecting an exam -> also does not work
Refresh the browser and try selecting an exam -> does work

Expected behavior
Expected: we should be able to select an exam without having to refresh the browser
Screenshots

Environment
Browser: Chrome
Additional context
Workaround: Refresh the browser
Output in Web Console
Stack trace from web console:
instrument.js:110 ERROR Error: Uncaught (in promise): DataCloneError: Failed to execute 'pushState' on 'History': function(t){var n=t%10;return t+(1===G(t%100/10)?"th":1===n?"st":2===n?"nd":3===n?"rd":"th")} could not be cloned.
Error: Failed to execute 'pushState' on 'History': function(t){var n=t%10;return t+(1===G(t%100/10)?"th":1===n?"st":2===n?"nd":3===n?"rd":"th")} could not be cloned.
    at History.pushState (instrument.js:286)
    at m.pushState (common.js:159)
    at P.pushState (common.js:481)
    at x.go (common.js:634)

51 websocket subscriptions:  MapIterator {"/topic/management/feature-toggles", "/topic/course/1/INSTRUCTOR", "/topic/course/21/INSTRUCTOR", "/topic/course/28/INSTRUCTOR", "/topic/course/33/INSTRUCTOR", …}

There was an issue with the state being set with a probably way too large object. I removed this part and added a TODO that we should implement this differently


FTP Explicit PROT P communication for zero byte file not working

I am trying to perform a FTP transfer for a file using FTP Explicit PROT P communication. It is working fine with non-zero byte file but with zero byte (blank) file, it is giving me below error -
javax.net.ssl.SSLHandshakeException: Remote host terminated the handshake
at java.base/sun.security.ssl.SSLSocketImpl.handleEOF(Unknown Source)
at java.base/sun.security.ssl.SSLSocketImpl.decode(Unknown Source)
at java.base/sun.security.ssl.SSLSocketImpl.readHandshakeRecord(Unknown Source)
at java.base/sun.security.ssl.SSLSocketImpl.startHandshake(Unknown Source)
at java.base/sun.security.ssl.SSLSocketImpl.startHandshake(Unknown Source)
at securelink.ftp.server.FTPDConnection.buildDataConnection(FTPDConnection.java:374)
at securelink.ftp.server.FTPDConnection.storInternal(FTPDConnection.java:1313)
at securelink.ftp.server.FTPDConnection.STOR(FTPDConnection.java:1265)
at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(Unknown Source)
at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(Unknown Source)
at java.base/java.lang.reflect.Method.invoke(Unknown Source)
at securelink.ftp.server.FTPDConnection.run(FTPDConnection.java:675)
Caused by: java.io.EOFException: SSL peer shut down incorrectly
at java.base/sun.security.ssl.SSLSocketInputRecord.read(Unknown Source)
at java.base/sun.security.ssl.SSLSocketInputRecord.readHeader(Unknown Source)
at java.base/sun.security.ssl.SSLSocketInputRecord.decode(Unknown Source)
at java.base/sun.security.ssl.SSLTransport.decode(Unknown Source)
... 12 more
This error is occurring when the FTP client is sending STOR command to server and the server starts the the data binding. After I checked the code as per the trace starting from startHandshake() method, I reached to the decode() method in class SSLSocketInputRecord.java, where the below code is written -
int really = read(is, temporary, 0, headerSize);
if (really < 0) {
throw new EOFException("SSL peer shut down incorrectly");
}
Since the file size is zero, it is throwing the exception.
So is this the actual behavior or this is a bug that will be resolved or has been resolved ?
If it has been resolved, please specify the version.

Which Java are you using? (java --version)

I am using Java 11. I tried with java 8 also, but the issue is same in that one.

are you using SFTP or FTPS?

Using FTPS.

I'd recommend trying Apache Commons Net (which does handle 0 byte files OK) https://commons.apache.org/proper/commons-net/

That might require a lot of changes in the code itself. Is there any other work around or any possibility that it might be handled in the future java release ?

That might require a lot of changes in the code itself. Is there any other work around or any possibility that it might be handled in the future java release ?

I'm not sure if it a core Java issue to be fixed.  This would require a lot of discussion upstream at the OpenJDK project (openjdk.java.net)


debug mode

a configuration value that would log out network requests and their responses. perhaps with a log level

you can set the environment var NODE_DEBUG=request in your shell if you want request logging.

good tip - thanks @slang800

This should probably be added to the readme

yeah, can you include that as part of your port


bug(MatProgressBar): Class, which only works with insecure CSP header, gets applied where not needed

Is this a regression?

[ ] Yes, this behavior used to work in the previous version

The previous version in which this bug was not present was
No response
Description
I was unsure if this was a bug, feature request or a security vulnerability, but I decided to put it as a bug. Let me know if I should put it somewhere else.
It appears the class mdc-linear-progress__buffer-dots gets applied to progress bars not in buffer mode, and it appears to have no effect as well. This can be seen in the documentation for the component, where it is applied to both the determinate and indeterminate examples. This is a problem for the project I am working in since that specific class uses data as the source for the mask-image CSS property, which we don't allow because of a strict CSP header policy.
Reproduction
StackBlitz link: Not needed since it can be seen in the documentation, to which I added an url.
Steps to reproduce:

Go to: https://material.angular.io/components/progress-bar/overview
Check the classes applied to the determinate and indeterminate examples

Expected Behavior
I expect that the buffer specific class isn't applied to not-related progress bars. I also expect that the mask-image property is not set using data
Actual Behavior
The class is applied to multiple progress bars where it does not seem to have an effect and the value set for the mask-image property uses data
Environment

Angular:17.0.9
CDK/Material: 17.0.5
Browser(s): Chrome
Operating System (e.g. Windows, macOS, Ubuntu): macOS

I'm not super well versed in CSP. Would removing it from the DOM when not in use be enough or do we also have to not include the CSS?

It is only when the class is applied that the violation occurs, so it isn't an issue if it is included in the CSS. Unless of course we want to use the progress bar in buffer mode at a later time.

In that case I'll continue with #28938. It's not optimal that using buffer would cause a new CSP violation to show up, but currently we don't have much control over how that data: URL gets generated. We may be able to rework this in the future.


Feature/fit nebula module

Require https://github.com/vesoft-inc/nebula/pull/2610

Replace by https://github.com/vesoft-inc/nebula-cpp/pull/54


Integrate ruffle and add adobe flash (.swf) support

Add ruffle as npm dependency for easy upgradability
Bundle ruffle as an independent output javascript file with webpack
Add support for uploading SWF-files, validate mime-type properly
Add thumbnail for SWF-files
Add support for expand/shrink of SWF-files
Expand should create ruffle player and size the content same way as is done for other file types

This is done.


Running Authenticode-signed script under AllSigned fails when script has LF line endings

Related to #1195 / Found with https://github.com/chocolatey/choco/issues/1203
You are likely to see this more now that PowerShell is cross platform. We just had an issue filed about attempting to run scripts with execution policy "AllSigned". Upon researching this, we were able to determine that any scripts that were UTF-8 / CRLF, those would load just fine. However any scripts that were UTF8 / LF, it would error saying 'filename "may have been tampered because the hash of the file does not match the hash stored in the digital signature."'
I have not had an attempted to check this with the new versions of PowerShell just yet but I did check it against PowerShell v3 and v4 (so far). After talking to @lzybkr and @vors (https://gitter.im/PowerShell/PowerShell?at=58cc405c6d7eb18404dbca07), they said to followup with an issue as they have not had a report of this anywhere.
Steps to reproduce

Create a script with LF line endings. Save it as FileSignedWithUnixLineEndings.ps1
Authenticode sign the script.
Open the script and note that it now has LF in the script code and CRLF in the authenticode signature (Mixed mode line endings).
Check to be sure the signature is valid. Get-AuthenticodeSignature .\FileSignedWithUnixLineEndings.ps1
Change execution policy to all signed. Set-ExecutionPolicy AllSigned
Attempt to run the script. .\FileSignedWithUnixLineEndings.ps1

Expected behavior
It should run the script as it has not been modified since it was signed.
Actual behavior
It thinks the file has been modified:
.\FileSignedWithUnixLineEndings.ps1 : File FileSignedWithUnixLineEndings.ps1 cannot be
loaded. The contents of file FileSignedWithUnixLineEndings.ps1 may have been tampered
because the hash of the file does not match the hash stored in the digital signature. The
script will not execute on the system. Please see "get-help about_signing" for more
details..
At line:1 char:1
+ .\FileSignedWithUnixLineEndings.ps1
+ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : SecurityError: (:) [], PSSecurityException
    + FullyQualifiedErrorId : UnauthorizedAccess

To be clear, this was all when running on Windows.

@SteveL-MSFT @joeyaiello It seems we should definitely fix this in Beta.

It gets better. :/
If the file is UTF8 with no BOM but contains a unicode character, it also fails.

Followed up with https://github.com/PowerShell/PowerShell/issues/3466

If #3466 overlaps the Issue please close the Issue.

Not able to repro this with alpha.18 on Win10 Creators Update.  Here I have a simple script.  I show it only has a LF.  I sign it.  Changed executionpolicy.  And it successfully ran.  I show that the script portion has LF and the signature has CRLF (unfortunately, we can't control this aspect as we're calling a Win32 api to sign the file and there's no options for line endings).
PS C:\Users\slee\test> .\script_lf.ps1
hello
PS C:\Users\slee\test> Format-Hex .\script_lf.ps1

           Path: C:\Users\slee\test\script_lf.ps1

           00 01 02 03 04 05 06 07 08 09 0A 0B 0C 0D 0E 0F

00000000   77 72 69 74 65 2D 68 6F 73 74 20 22 68 65 6C 6C  write-host "hell
00000010   6F 22 0A                                         o".

 SIG # Beg
00000020   69 6E 20 73 69 67 6E 61 74 75 72 65 20 62 6C 6F  in signature blo
00000030   63 6B 0D 0A 23 20 4D 49 49 45 4D 77 59 4A 4B 6F  ck..# MIIEMwYJKo
00000040   5A 49 68 76 63 4E 41 51 63 43 6F 49 49 45 4A 44  ZIhvcNAQcCoIIEJD
00000050   43 43 42 43 41 43 41 51 45 78 43 7A 41 4A 42 67  CCBCACAQExCzAJBg
00000060   55 72 44 67 4D 43 47 67 55 41 4D 47 6B 47 43 69  UrDgMCGgUAMGkGCi
00000070   73 47 41 51 51 42 0D 0A 23 20 67 6A 63 43 41 51  sGAQQB..# gjcCAQ

Sweet!

@ferventcoder can you confirm it works for you with alpha.18 and we can close this?

.Net Core 2.0 works wonders?

@iSazonov no, alpha.18 doesn't have .NET Core 2.0 yet. It'll be there in the next release.


Update LICENSE_HEADER.md

Link updated.
Just changed de Adobe copyright url.
Description
Replaced deprecated url with the right Adobe copyright link.
Related Issue

Issue #10
Motivation and Context
If the user wants to go deeper into Adobe copyright then the link should point to the right page.
How Has This Been Tested?
Replaced the link and tested it.
Screenshots (if appropriate):
Types of changes

[x ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x ] I have signed the Adobe Open Source CLA.
[x ] My code follows the code style of this project.
[x ] My change requires a change to the documentation.
[x ] I have updated the documentation accordingly.
[ x] I have read the CONTRIBUTING document.
[x ] I have added tests to cover my changes.
[x] All new and existing tests passed.

There is an existing PR open to deal with this: #13

Bummer.


[dv/edn] Signoff for V2S

This PR signoff EDN for V2S.
Signed-off-by: Cindy Chen<EMAIL_ADDRESS>

V2 TODO: #11489
V2S TODO: #16268
Regression results:

https://github.com/lowRISC/opentitan/pull/16372 this PR further improves functional coverage for CSRNG agent.


chore(deps): update storybook monorepo to v6.1.20

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


rtabmap: 0.21.4 -> <IP_ADDRESS>

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/project/rtabmap/versions.
meta.description for rtabmap is: Real-Time Appearance-Based 3D Mapping
meta.homepage for rtabmap is: https://introlab.github.io/rtabmap/
Updates performed

Version update

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found <IP_ADDRESS> in filename of file in /nix/store/40cdshyvcpv7yym9ha7bd5kal07sr5z3-rtabmap-<IP_ADDRESS>

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
rtabmap

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/40cdshyvcpv7yym9ha7bd5kal07sr5z3-rtabmap-<IP_ADDRESS> \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A rtabmap https://github.com/r-ryantm/nixpkgs/archive/01292149b8c97b8480236fb7e179cb114b498d15.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/01292149b8c97b8480236fb7e179cb114b498d15#rtabmap

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/40cdshyvcpv7yym9ha7bd5kal07sr5z3-rtabmap-<IP_ADDRESS>
ls -la /nix/store/40cdshyvcpv7yym9ha7bd5kal07sr5z3-rtabmap-<IP_ADDRESS>/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review --extra-nixpkgs-config '{ allowInsecurePredicate = x: true; }' run on x86_64-linux 1

1 package built:

rtabmap

Maintainer pings
cc @ckiee for testing.

Version doesn't exist


Fix the "next" parameter for the new API

I'm not seeing this, looks resolved already?
curl 'https://www.metaculus.com/api2/questions/?limit=1' -H 'Authorization: Bearer [redacted]' | jq '.next'
"http://www.metaculus.com/api2/questions/?limit=1&offset=1"

curl "https://www.metaculus.com/api/posts/?limit=1" -H 'Authorization: Bearer [redacted]' | jq '.next'
"http://www.metaculus.com/api/posts/?limit=1&offset=1"

@draaglom Thanks for checking! Seems like it was auto-resolved by my Nginx integration — https://github.com/Metaculus/metaculus/pull/1213


Error in spot.accountInfo

Hi Turlough. I was delighted to chance upon your package. I greatly appreciate your invested time and effort, and encourage you to keep up the good work! I am going through your given steps to get started, and I'm stuck on step 3:
[~,~,response] = spot.accountInfo
I'm getting the following error in MATLAB R2021a Update 5:
Unrecognized function or variable 'isValidrecv'.
Error in spot.accountInfo (line 16)
OPT.recvWindow (1,1) {isValidrecv(OPT.recvWindow)} = 5000
Thanks!

Thank you. Was the subfunctions folder on the MATLAB path when you tried to do this?

It wasn't, but I now added it and the other subfolders. Getting a different error now:
_Error using isValidrecv (line 3)
Invalid default value for argument 'recvWindow'. Too many output arguments.
Error in spot.accountInfo (line 16)
OPT.recvWindow (1,1) {isValidrecv(OPT.recvWindow)} = 5000_

Can you confirm that the isValidrecv function is identical to the one in my repository?

Sorry for my belated reply. You were right. I removed the "end" from the end of isValidrecv. Working fine now. I never put that in functions, not sure what is its use. Perhaps because no output is defined.


fix a TapAndDragGestureRecognizer bug

Fixes https://github.com/flutter/flutter/issues/122141

@xu-baolin I think we can close this now because #123055 was merged.


Register outroninja.is-cool.dev

Requirements

[ ] You have completed your website.
[x] The website is reachable.
[x] The CNAME record doesn't contain https:// or /.
[x] There is sufficient information at the owner field.

Description

Link to Website

https://outroninja.github.io

idk if i did this right, if not please let me know


Update to whisper.cpp v1.3.0.

This PR includes major breaking changes! Check out the CHANGELOG.md file for more information on the changes, and provide feedback after merge here as well.

Closes #28 and #29

Hmm this PR seems to have triggered some issues on my system (Arch Linux, kernel 6.2.11-arch1-1)
  --- stderr
  CMake Error: File /home/zero/data/CLionProjects/whisper-rs/target/package/whisper-rs-sys-0.4.0/whisper.cpp/bindings/javascript/package-tmpl.json does not exist.
  CMake Error at CMakeLists.txt:23 (configure_file):
    configure_file Problem configuring file

  CMake Error at CMakeLists.txt:298 (add_subdirectory):
    add_subdirectory given source "bindings" which is not an existing
    directory.

  thread 'main' panicked at 'Failed to generate build script', build.rs:77:9
  note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

Ah I believe this is the exclude directive in the Cargo.toml excluding the bindings. I'll see how little I can get away with, as I don't want to bloat the size.

I have problems using this version as it looks like create_state() isn't publicly visible (maybe #41?)

Oops, I thought I published a new version at some point. I'll do so in an hour or so. Thanks for pointing it out.

Forgot to mention this, but I published v0.7.0 last night.


Zoom doesn't work properly in Welcome page

Zooming in/out using app menu or Cmd+"+" and Cmd+"-" in Welcome page would only affect banner images but not the app itself.
Proper behavior would do following:

Zooming in Welcome page as well as any other pages should affect whole app zoom level
After page reload app zooming level should be in sync with the banners.

Closed since Welcome Banners now rendered locally, without iframes or webviews.


Add require_passed_checks input to control if should skip checks step

This new input will allow users to control wether failed checks should be taken into account when searching for the next updatable PR or not.
In order to maintain backwards-compatibility I have set the input as optional with a default value of true.

Pull Request Test Coverage Report for Build 708733214

6 of 6 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build 622496972:
0.0%

Covered Lines:
82

Relevant Lines:
82

💛  - Coveralls

Published v0.5.0 with this change https://github.com/adRise/update-pr-branch/releases/tag/v0.5.0


[openexr] Update to 3.2.3

Remove build patch as similar was applied upstream.
BUILD_DOCS was renamed to BUILD_WEBSITE.
With version 3.2, libdeflate is now used rather than zlib.

[x] Changes comply with the maintainer guide.
[x] SHA512s are updated for each updated download.
[x] The "supports" clause reflects platforms that may be fixed by this new version.
[x] Any fixed CI baseline entries are removed from that file.
[x] Any patches that are no longer applied are deleted from the port's directory.
[x] The version database is fixed by rerunning ./vcpkg x-add-version --all and committing the result.
[x] Only one version is added to each modified port's versions file.

Also trying to remove the !uwp for supports since it seems to be building on at least some of the UWP triplets.

Nevermind, can't remove !uwp.


fix: add print flag to subcommand and set default behavior

Hi @Icelk , thank you for creating such a helpful tool!
I encountered an error when running the example, so I fixed it. I would appreciate it if you could take a look.
Summary of Fixes
First, I encountered following error at clap_autocomplete/src/lib.rs:118:24:
Mismatch between definition and access of print. Unknown argument or group id.  Make sure you are using the argument id and not the short or long flags

So, I modified the print flag to be added to the complete subcommand.
Second, the following error occurred at clap_builder-4.5.20/src/parser/matches/arg_matches.rs:181:17:
arg print's ArgAction should be one of SetTrue, SetFalse which should provide a default

To fix this, I added default behavior by using .action(clap::ArgAction::SetTrue) for the print arg (this means when the flag is set, it will be treated as true).
Clap Version:
4.5.20

Hi!
Thank you for the kind words :) They are appreciated.
Looks good to me. Will publish a new version.

New version published:
https://crates.io/crates/clap_autocomplete/0.4.2


Add BSC support

This removes maxTimelockDurationSeconds and timelockTotalSupply. Not sure if that's what we want to do though. 3.4.3 pools no longer support those fields. We currently don't use timelockTotalSupply and only use maxTimelockDurationSeconds for the withdraw wait time gauge, which I don't think actually gets displayed anymore?

"...and only use maxTimelockDurationSeconds for the withdraw wait time gauge, which I don't think actually gets displayed anymore?"

I believe we still use that in the flagship. Could we leave maxTimelockDurationSeconds in the API but return null for newer pools?


Periodically clean up template releases and tags

Template packages clutter the release pages. This is normally not an issue when releasing to a package manager like nuget. The C++ SDK needs cleaner release pages because the GitHub releases are an important distribution mechanism for these releases.

Yes, please! :)


Badges state

Is it binary is it OK vs not OK?

can you clarify. it depends on the badges route used

I had tried [![cran checks](https://cranchecks.info/badges/summary/reshape)](https://cran.r-project.org/web/checks/check_results_reshape.html). https://github.com/ropensci/cchecksapi#badges

im confused.

Sorry what I mean is I wonder what color code there is.
Just

green "OK"
red "not OK" for any NOTE/WARNING/ERROR?
I had tried it for av

correct,
green = OK
read = NOT OK

Which might mean some developers won't want to adopt them?

perhaps we could allow passing a color as a parameter like Gabor does for his badges.
I think it makes sense to keep the defaults as is, e.g., travis has the same colors, or no?

Ideally we would be able to whitelist some NOTEs. But that's quite hard to implement.

Right, perhaps categorize NOTEs into types and then allow parameter passed in to say which types of NOTEs to ignore?

I think we should just start easy, and ignore some, e.g. the one about the package size, for a start. If  CRAN is OK with the NOTE, then the package author will be OK as well.

I mean when I say "we", I mean, if I were you.

moving conversation to new issue


Support semantic error messages by supporting deserializing spans

See toml::Spanned for inspiration

(I rolled by this issue because my project uses toml-edit for it's higher-quality serialization, but its low-quality spans for deserialization means I've been experimenting with preferring toml-rs for the other side of the equation, even though toml-edit/easy is explicitly designed to preserve more info, alas!)
I've been using this API in toml-rs in anger a lot this week, so I figured I'd chime in with some details on how it works, and some issues it has:
Spanned defines four magic strings:
pub(crate) const NAME: &str = "$__toml_private_Spanned";
pub(crate) const START: &str = "$__toml_private_start";
pub(crate) const END: &str = "$__toml_private_end";
pub(crate) const VALUE: &str = "$__toml_private_value";

And then uses a custom visitor and does the following:
        let visitor = SpannedVisitor(::std::marker::PhantomData);

        static FIELDS: [&str; 3] = [START, END, VALUE];
        deserializer.deserialize_struct(NAME, &FIELDS, visitor)

The deserializer then, at various points in the code, checks for that exact state and wraps itself in a SpannedDeserializer that emits the 3 magic fields, the first 2 being the span info:
        if name == spanned::NAME && fields == [spanned::START, spanned::END, spanned::VALUE] {
            let start = 0;
            let end = self.input.len();

            let res = visitor.visit_map(SpannedDeserializer {
                phantom_data: PhantomData,
                start: Some(start),
                value: Some(self),
                end: Some(end),
            });
            return res;
        }

That the SpannedVisitor expects:
            fn visit_map<V>(self, mut visitor: V) -> Result<Spanned<T>, V::Error>
            where
                V: de::MapAccess<'de>,
            {
                if visitor.next_key()? != Some(START) {
                    return Err(de::Error::custom("spanned start key not found"));
                }

                let start: usize = visitor.next_value()?;
                ...

I've been trying to use this in anger a fair bit this week and it seems to largely work for "normal" derived types but is more of a mess with custom deserialization. Because it relies on random places in the code detecting the magic fields, if you ever invoke it in a way it wasn't expecting (like variants on the "string or struct" pattern in serde's docs) then you get nasty "expected spanned" from it not creating the SpannedDeserializer and emitting the magic fields.
I'm not sure if those problems are "I'm bad at serde" or "this design is buggy" or "serde makes this impossible to do robustly".
The actual Spanned type also has a pretty suboptimal API. It's very opaque, and you have to explicitly call a method to get the wrapped value, so retroactively adding spans to your existing types is extremely disruptive to code. Thankfully because it communicates with magic fields, you're able to "fork" just Spanned. I did this to make it more like Box<T>, in that it does its best to forward everything from T via Deref and trait impls. All that actual span methods I made into associated functions so you have to do things like Spanned::start(&spanned_val), which is fine since you don't actually care about the span until you want to emit an error.
I am... incredibly bad at serde stuff so I'm not sure if I could help with bringing this stuff up, but here's a brain dump.

Thanks for the brain dump!  These details will be important when implementing this!

Some extra notes from landing this:

serde(from = "toml::Spanned<T>") seems to work well as an alternative to get a better API without importing the private details
It seemed that my woes were rooted more in #[serde(flatten)] than anything else. #[serde(untagged)] did also seem messed up. basically anything that requires the input to be "extra" buffered makes Spanned break. I worked around this by replacing these kinds of things with a #[serde(try_from = "OuterTypeWithEveryPossibleField")] that manually implements flatten/untagged logic after deserializing all the Spanned more naturally. Not sure if the problem is in serde or Spanned, and whether it's fundamental to the design.

I'm pretty sure the answer is "serde makes this impossible to do robustly". I looked into span support for serde things a bit ago in https://github.com/serde-rs/serde/issues/1811#issuecomment-1282900892, and as far as I can tell there's no way to propagate span info through serde without resorting to wild hacks like what toml::Spanned was doing previously.
It's possible to extend serde's API to better support spans. Unfortunately, with the designs I've found so far, you generally have two choices:

Support propagating spans in a way that allows the deserializer to choose an appropriate span representation, which requires a breaking change to serde's API
Bake a particular representation choice into serde's API, which will not be appropriate for all formats

I wonder if it'd make sense to check that the magic strings are the same instead of checking if they are equal, that means comparing the address rather than contents. Unless I'm missing something this should make it more robust (if anyone for whatever reason uses these strings in the input it shouldn't confuse the parser). If there was some way to prevent the linker from merging same symbols they could all be empty strings which would be even nicer but I didn't find how to do that.

Note that I'm keeping the scope of this issue fairly limited to "do what toml did" so we can make progress towards merging the crates.  I'd welcome people to open a new issue with their feedback for further improvement.


Cross-Origin Request Blocked

Cross-Origin Request Blocked: The Same Origin Policy disallows reading the remote resource at http://<IP_ADDRESS>:5000/top_markets. (Reason: CORS header ‘Access-Control-Allow-Origin’ missing).
Cross-Origin Request Blocked: The Same Origin Policy disallows reading the remote resource at http://<IP_ADDRESS>:5000/top_proxies. (Reason: CORS header ‘Access-Control-Allow-Origin’ missing).
Cross-Origin Request Blocked: The Same Origin Policy disallows reading the remote resource at http://<IP_ADDRESS>:5000/header. (Reason: CORS header ‘Access-Control-Allow-Origin’ missing).
Cross-Origin Request Blocked: The Same Origin Policy disallows reading the remote resource at http://<IP_ADDRESS>:5000/top_smartcoins. (Reason: CORS header ‘Access-Control-Allow-Origin’ missing).
Cross-Origin Request Blocked: The Same Origin Policy disallows reading the remote resource at http://<IP_ADDRESS>:5000/top_operations. (Reason: CORS header ‘Access-Control-Allow-Origin’ missing).
Cross-Origin Request Blocked: The Same Origin Policy disallows reading the remote resource at http://<IP_ADDRESS>:5000/top_holders. (Reason: CORS header ‘Access-Control-Allow-Origin’ missing).
Cross-Origin Request Blocked: The Same Origin Policy disallows reading the remote resource at http://<IP_ADDRESS>:5000/top_uias. (Reason: CORS header ‘Access-Control-Allow-Origin’ missing).

None of the external data is currently loading on open-explorer.io & the console is being spammed with the above warnings.

service was down, just restarted and working ok now.


jsdom depends on contextify somewhere, which is an abandoned package.... =/

This is an installation rather than a code issue - jsdom has a dependency on contextify somewhere, which is an obsolete package (its readme literally starts with "YOU DON'T NEED THIS MODULE ON NODE >= 0.12" =), but is apparently necessary enough that it can break jsdom's installation: it needs compiling against whatever version of Node >0.12 is being used, which means it tries to use node-gyp, and node-gyp only works with ye olde schoole Python 2.7, so any machine that is set up for modern JS and Python work with Node 8 and Python 3 won't be able to compile contextify, breaking the jsdom install =(

➜  ~ mkdir testing
➜  ~ cd testing
➜  testing npm i -q jsdom
➜  testing npm ls contextify
/Users/sebmaster/testing
└── (empty)

Doesn't seem like that's the case. We'll need a repro to reopen.

Wow, this makes no sense. I wiped node_modules, reinstalled jsdom, got the same crash - deleted temp, tried again, no crash... npm cache souring things maybe O_o
Thanks for responding.

I just had this problem and I think I have the solution.
I had tried to install js-dom instead of jsdom.


k9s fails to download

------
 > [dev_containers_target_stage 7/8] RUN --mount=type=bind,from=dev_containers_feature_content_source,source=k9s_2,target=/tmp/build-features-src/k9s_2     cp -ar /tmp/build-features-src/k9s_2 /tmp/dev-container-features  && chmod -R 0755 /tmp/dev-container-features/k9s_2  && cd /tmp/dev-container-features/k9s_2  && chmod +x ./devcontainer-features-install.sh  && ./devcontainer-features-install.sh  && rm -rf /tmp/dev-container-features/k9s_2:
0.281 ===========================================================================
0.281 Feature       : k9s
0.281 Description   : A feature to install k9s
0.281 Id            : ghcr.io/natescherer/devcontainers-custom-features/k9s
0.281 Version       : 1.0.0
0.281 Documentation :
0.281 Options       :
0.281     VERSION="latest"
0.281 ===========================================================================
0.619 Installing https://github.com/derailed/k9s/releases/download/v0.31.7/k9s_Linux_amd64.tar.gz https://github.com/derailed/k9s/releases/download/v0.31.7/k9s_Linux_amd64.tar.gz.sbom to /opt/k9s...
0.624 curl: (3) URL using bad/illegal format or missing URL
0.625 ERROR: Feature "k9s" (ghcr.io/natescherer/devcontainers-custom-features/k9s) failed to install!
------
Dockerfile.extended:42
--------------------
  41 |
  42 | >>> RUN --mount=type=bind,from=dev_containers_feature_content_source,source=k9s_2,target=/tmp/build-features-src/k9s_2 \
  43 | >>>     cp -ar /tmp/build-features-src/k9s_2 /tmp/dev-container-features \
  44 | >>>  && chmod -R 0755 /tmp/dev-container-features/k9s_2 \
  45 | >>>  && cd /tmp/dev-container-features/k9s_2 \
  46 | >>>  && chmod +x ./devcontainer-features-install.sh \
  47 | >>>  && ./devcontainer-features-install.sh \
  48 | >>>  && rm -rf /tmp/dev-container-features/k9s_2
  49 |
--------------------
ERROR: failed to solve: process "/bin/sh -c cp -ar /tmp/build-features-src/k9s_2 /tmp/dev-container-features  && chmod -R 0755 /tmp/dev-container-features/k9s_2  && cd /tmp/dev-container-features/k9s_2  && chmod +x ./devcontainer-features-install.sh  && ./devcontainer-features-install.sh  && rm -rf /tmp/dev-container-features/k9s_2" did not complete successfully: exit code: 3

Sorry, no time for me to investigate more.
Edit: Image used:
	"image": "mcr.microsoft.com/devcontainers/typescript-node:1-20-bookworm",

Thanks a lot ❤

Hey, does this has been resolved?


fix(build):add openresty/lualib into rpath

Summary
Fix an issue that resty.signal fails to find librestysignal.so and fails to parse certain LUA_CPATH.
Checklist

[x] The Pull Request has tests
[na] A changelog file has been added to CHANGELOG/unreleased/kong or adding skip-changelog label on PR if unnecessary. README.md
[na] There is a user-facing docs PR against https://github.com/Kong/docs.konghq.com - PUT DOCS PR HERE

Full changelog
fix(build):add openresty/lualib into rpath
Issue reference

From https://github.com/Kong/kong/pull/11382

good packaging run https://github.com/Kong/kong/actions/runs/6296243549/job/17091136057?pr=11629


Batch load query-index

In the current implementation, we load the full query-index.json. The EN site contains 13k articles, the file is huge (time to download and time to parse). We should load batch of 1000 articles.
@davidnuescheler said he will take care of the implementation.

Fixed with #71


Make the TextSplitter in WebBrowser Tool configurable

I can do it.

PR for this at #857 @nfcampos. It is my frist time to contribute a project.


[cherry-pick] prohibit agg or groupby pushdown on double read (#1004)

cherry-pick https://github.com/pingcap/tispark/pull/1004

/run-all-tests


add test for getting started guide from jvm commons

Deferring to language maintainer, so removing languages-review label for now :-)

Test looks good to me. Let's get feedback from @Malax to see if there are any other changes requested.


fix: panic when setting config value

Rough draft around the discussion #3820.
This handles the basic use-case and will behave like the following.
# set simple value or nested value
mise config set tools.node 20 --> node = "20"
mise config set tools.node.version 20 --> node = { version = "20" }

# transform version to inline table if its already set
mise config set tools.node 20 --> node = "20"
mise config set tools.node.postinstall 'corepack enable' --> node = { version = "20", postinstall = "corepack enable" }

mind adding an e2e test? it doesn't look like we have any for these commands, but I think it should be pretty simple. I would copy https://github.com/jdx/mise/blob/main/e2e/cli/test_settings_add

will add a test and look into the additional type inference issue but this might have to wait until tomorrow

sounds good, I think what you have in the description is good enough. A pattern I've been doing is this which might be useful here:
https://github.com/jdx/mise/blob/b66f7cac26186b1f935c11e296647baebf1a20f1/e2e/lockfile/test_lockfile_use#L14-L16
it's basically a snapshot test


Created SECURITY.md

Well detailed description of the change :

 I worked on the SECURITY.md

Context of the change :

 - There is no SECURITY.md file in this repo

Type of change :

[ ] Bug fix
[x] New feature
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

[x] I have read the CONTRIBUTING document.
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas

resolved


SQL Format Breaks Table Names in Query [MSSQL]

In some cases, table column names with spaces and dashes in them get broken by the query reformatter.
before:
SELECT
	[table name with spaces],
	[table name with spaces -and dashes],
	[another dash- example],
	[final-dash-example],
FROM
	db.tablename;

after:
SELECT
	[table name with spaces],
	[table name with spaces -
	and dashes],
	[another dash - example],
	[final - dash - example],
FROM
	db.tablename;

Of course, I don't have a table named final - dash - example, the table is named final-dash-example, so the query breaks.

Thanks for report.

Dang, that was fast.
Thanks for the fix.


"Windows 10 Anniversary Update" (Version 1607) commit 96eb96d deleted the audio sysvad .sln file

https://github.com/Microsoft/Windows-driver-samples/tree/master/audio/sysvad
No .sln file there, Step 1 of README.md instructions unclear.
Is the .sln missing?

Fix in commit c348ac7


The Fisherman's Gambit patterns use different indexing schemes

Fisherman's Gambit is 1-indexed. Fisherman's Gambit II is 0-indexed.

Tempted to call that intended behavior; the reason I is "1-indexed" is because it's counting the number that's popped off as 0...

Fair, I guess, but in my opinion it's so unintuitive for two patterns with the same name and the same sort of behaviour to use two different indexing systems. This whole thing with Fisherman's being 0 or 1 indexed changing by the update has tripped me up more times than I can count.

I've set this as tentatively planned for 0.11.1, because this is the sort of thing that feels like it would be better to change when we're already breaking hexes in a bunch of other ways, than in a patch release.

duplicate of #447 apparently


WIP: fix: logical size limit is broken during PS restart

fixes https://github.com/neondatabase/neon/issues/5963

Abandoned in favor of https://github.com/neondatabase/neon/pull/6010


Variable not rendered in message_after_copy

Describe the problem
I am creating a copier template with a _message_after_copy that contains an algorithm_name variable, but it is not rendered.
This part of my copier.yaml:
_message_after_copy: |
  Your algorithm template "{{ algorith_name }}" has been created successfully!

While providing a value for algorithm_name, it is not printed at the end of copier copy:
Your algorithm template "" has been created successfully!

Other details:

algorithm_name is present as expected in .copier-answers.yml
I followed the instructions in this part of the docs

Template
algorithm_name:
    type: str
    help: What is your algorithm name?

_message_after_copy: |
  Your algorithm template "{{ algorith_name }}" has been created successfully!

To Reproduce

conda create -n test-copier python=3.10
conda activate test-copier
pip install copier
inside the directory with the copier template, run copier copy . ../test
Fill out a value for the algorithm name
Observe issue that algorithm name is not printed

Logs
No response
Expected behavior
Expect some_input_name at the end as in:
Your algorithm template "some_input_name" has been created successfully!

Screenshots/screencasts/logs
No response
Operating system
Windows
Operating system distribution and version
Windows 10 Enterprise
Copier version
8.3.0
Python version
3.10.13
Installation method
pip+pypi
Additional context
No response

You have a typo at least in the issue description:
 _message_after_copy: |
-  Your algorithm template "{{ algorith_name }}" has been created successfully!
+  Your algorithm template "{{ algorithm_name }}" has been created successfully!

Does this solve your problem?

Oops... sorry!

No problem. 😄


Added files via upload

Changed the CSS Names. Still need your help with the DOCTYPE and the a ref underline

@nbarel
Figured out your doctype problem - you're missing units for the sizes in CSS (e.g. height: 40).
Since you're using a wrong doctype you effectively are in quirks mode, which honors this non-standard behaviour.
The underline - if you need to reset a link which is part of your component then use a class name for it which states clearly that it's part of the component.


[HTTP Status Codes:] Handle failure status codes in Long Running Operations

See: https://github.com/Azure/azure-sdk-for-net/blob/ce046949f17fc4ceeab5c87beb4750047b983565/sdk/formrecognizer/Azure.AI.FormRecognizer/src/ExtractFormOperation.cs#L73
Also:

[ ] ExtractLayoutOperation
[ ] ExtractReceiptOperation

@kinelski this is the one you've been looking into, yes? I've reassigned based on that assumption.

CodeGen handles this for us.


Firebase crashlytics failed to auto upload dSYM files through XCode run script

[REQUIRED] Step 1: Describe your environment

Xcode version: 12.3
Firebase SDK version: 8.0.0
Installation method: pod
Firebase Component: Crashlytics

[REQUIRED] Step 2: Describe the problem
I am trying to upload dSYM files thorough XCode runscript, this script will execute while archiving the build.
Here is my run scripts for Firebase
Run Script 1:
"${PODS_ROOT}/FirebaseCrashlytics/run"
Input Files:
${DWARF_DSYM_FOLDER_PATH}/${DWARF_DSYM_FILE_NAME}/Contents/Resources/DWARF/${TARGET_NAME}
$(SRCROOT)/$(BUILT_PRODUCTS_DIR)/$(INFOPLIST_PATH)
Run script 2 to upload dsyms:
if [ ! -d ${DWARF_DSYM_FOLDER_PATH}/${DWARF_DSYM_FILE_NAME} ]; then
    echo "Couldn't find dsym file"
    exit 1
fi

stat -x ${DWARF_DSYM_FOLDER_PATH}/${DWARF_DSYM_FILE_NAME}

"${PODS_ROOT}/FirebaseCrashlytics/upload-symbols" -gsp "${PROJECT_DIR}/Resources/GoogleService-Info.plist" -p ios "${DWARF_DSYM_FOLDER_PATH}/${DWARF_DSYM_FILE_NAME}"

Finally run script executed and here is my output from swift console log

But dSYM files are not reflecting in Firebase crashlytics dashboard. Dashboard showing error message like Upload missing dSYMs to see crashes
Properly setting Debug Information Format to DWARF with dSYM File for all your build types.

Hi @HazarathReddy ,
A couple questions:

Do you have bitcode enabled for your app? (If so, are the dSYMs you're uploading with the second run script the files downloaded from Apple?)
Are you seeing missing frames for crashes in your dashboard?

Hi @elenadoty
#1 => Yes, bitcode is enabled. dSYMs are fetching from local archive not from Apple
#2 -> No, I can't see crashes in dashboard until manually upload missing dSYMs

Hi @HazarathReddy ,
Thanks for the clarification.
Unfortunately I believe what you're experiencing is expected behavior for apps with bitcode enabled. Because of the processing that Apple performs on the uploaded app, the local dSYMs no longer match the symbols for the app. Instead, Apple provides new dSYMs to download that you can then manually uploaded to Crashlytics using the run script/CLI tool. The new dSYMs can be found either in Xcode Organizer (click on the archive version you want symbols for then the download dSYM button in the right pane) or in App Store Connect (navigate to the correct build and there should be a download dSYM button in the Build Details tab).


Add support for Unix domain sockets (UDS)

See PR #17 for possible implementation.

Progress update: Feature branch is ready, I'll file a PR for this once the v0.5.0 release is out :shipit:

🎉

Deferred to future release as per #75

Checking back here. It seems UDS Connector has meanwhile been merged, but UDS Server still seems missing. It might also make sense to enhance Server to create the underlying UDS/TCP/Secure server according to URI instead of proxying a TcpServer?

@andig Correct :+1: We're currently focusing our efforts on the HTTP component and on stabilizing things, but in the meantime I suppose we would be super happy to accept new feature PRs :+1:

This feature was originally planned for the v1.1.0 release milestone, but given the awesome PR #25 by @andig, it's safe to get this in with the next v0.8.6 release already :shipit: :tada:


Toolbar icon colors when selected, unselected, etc are not respected

Is there an existing issue for this?

[X] I have searched the existing issues

Flutter Quill version
9.x
Other Flutter Quill packages versions
9.x
Steps to reproduce
The colors for the selected, unselected toolbar icons are not respected when placed into the button options, the amount of padding is also much to much.
Major regression
Expected results
Non regressive code
Actual results
Regressive code
Code sample
Code sample
[Paste your code here]

Screenshots or Video

Screenshots / Video demonstration
[Upload media here]

Logs
Logs
[Paste your logs here]

Is there an existing issue for this?

[x]  I have searched the existing issues

Flutter Quill version
9.x
Other Flutter Quill packages versions
9.x
Steps to reproduce
The colors for the selected, unselected toolbar icons are not respected when placed into the button options, the amount of padding is also much to much.
Major regression
Expected results
Non regressive code
Actual results
Regressive code
Code sample
Code sample
Screenshots or Video
Screenshots / Video demonstration
Logs
Logs

Correct and that's because the version is not ready yet as we said but we were forced to publish

the amount of padding is also much to much.

You are not sharing any info, please share screenshots or steps to encounter the issues

The colors for the selected, unselected toolbar icons are not respected when placed into the button options, the amount of padding is also much to much.

True but remember it will follow the one in the theme data of the app widget so it will be more constant, but sure we will add an option to change it as well


[JS] Fix few bugs present in the pdf for issue #14862

since resetForm function reset a field value a calculateNow is consequently triggered.
But the calculate callback can itself call resetForm, hence an infinite recursive loop.
So basically, prevent calculeNow to be triggered by itself.
in Firefox, the letters entered in some fields were duplicated: "AaBb" instead of "AB".
It was mainly because beforeInput was triggering a Keystroke which was itself triggering
an input value update and then the input event was triggered.
So in order to avoid that, beforeInput calls preventDefault and then it's up to the JS to
handle the event.
fields have a property valueAsString which returns the value as a string. In the
implementation it was wrongly used to store the formatted value of a field (2€ when the user
entered 2). So this patch implements correctly valueAsString.
non-rendered fields can be updated in using JS but when they're, they must take some properties
in the annotationStorage. It was implemented for field values, but it wasn't for
display, colors, ...
it fixes #14862 and #14705.

/botio unittest

/botio unittest

/botio integrationtest

Also, it seems that the Interaction in 160F-2019.pdf must reset all test is timing out consistently now.

Also, it seems that the Interaction in 160F-2019.pdf must reset all test is timing out consistently now.

It's very likely because of the error you caught (the typeof thing).

/botio integrationtest

/botio unittest

/botio integrationtest

@Snuffleupagus, are you still ok with this change ?


Proposition of a new wiki page: how to do manual testing

Hello,
I propose the following new page in the wiki: https://github.com/guilhem-martin/go-guerrilla-wiki/blob/docs/manual_tests/Manual-testing.md
Let me know if you think this brings value. If it does, let me know if you see anything to improve so that I review it.
Thanks.

Hi, that looks quite good, could you create a PR for this?

@phires I'm not sure we can create a PR for wiki in GitHub?
If not, please feel free to copy paste the content in the main repo wiki.

Hi, if added the page to the wiki, thank you a lot :)


Failed to initialize CUDA backend on multi-process distributed environments

Description
The issue is happening when trying to initialize a multi-process multi-GPU environment with Slurm (but I think the problem is external to that).
Take the following simple script
import jax
import logging

logging.getLogger().setLevel(logging.DEBUG)

jax.distributed.initialize()

if jax.process_index() == 0:
  print(jax.devices())
  print(jax.device_count())    # total number of accelerator devices in the cluster
  print(jax.local_device_count())    # number of accelerator devices attached to this host

and executed with  srun --gres=gpu:2 --ntasks=2 --nodes=1  python main.py and it return
INFO:absl:JAX distributed initialized with visible devices: 0
INFO:absl:JAX distributed initialized with visible devices: 1
INFO:absl:Starting JAX distributed service on ainode17:4192
INFO:absl:Connecting to JAX distributed service on ainode17:4192
INFO:absl:Connecting to JAX distributed service on ainode17:4192
DEBUG:absl:Initializing backend 'interpreter'
DEBUG:absl:Initializing backend 'interpreter'
DEBUG:absl:Backend 'interpreter' initialized
DEBUG:absl:Initializing backend 'cpu'
DEBUG:absl:Backend 'cpu' initialized
DEBUG:absl:Initializing backend 'tpu_driver'
INFO:absl:Unable to initialize backend 'tpu_driver': NOT_FOUND: Unable to find driver in registry given worker:
DEBUG:absl:Initializing backend 'cuda'
DEBUG:absl:Backend 'interpreter' initialized
DEBUG:absl:Initializing backend 'cpu'
DEBUG:absl:Backend 'cpu' initialized
DEBUG:absl:Initializing backend 'tpu_driver'
INFO:absl:Unable to initialize backend 'tpu_driver': NOT_FOUND: Unable to find driver in registry given worker:
DEBUG:absl:Initializing backend 'cuda'
2022-09-27 19:23:48.425044: E external/org_tensorflow/tensorflow/compiler/xla/status_macros.cc:57] INTERNAL: RET_CHECK failure (external/org_tensorflow/tensorflow/compiler/xla/pjrt/gpu_device.cc:345) local_device->device_ordinal() == local_topology.devices_size()
*** Begin stack trace ***

	PyCFunction_Call
	_PyObject_MakeTpCall
	_PyEval_EvalFrameDefault
	_PyEval_EvalCodeWithName
	_PyFunction_Vectorcall
	PyObject_Call
	_PyEval_EvalFrameDefault
	_PyEval_EvalCodeWithName
	_PyFunction_Vectorcall
	_PyObject_FastCallDict

	_PyObject_MakeTpCall
	_PyEval_EvalFrameDefault
	_PyFunction_Vectorcall
	_PyEval_EvalFrameDefault
	_PyFunction_Vectorcall
	_PyEval_EvalFrameDefault
	_PyEval_EvalCodeWithName
	_PyFunction_Vectorcall
	_PyEval_EvalFrameDefault
	_PyEval_EvalCodeWithName
	_PyFunction_Vectorcall
	PyObject_Call

	_PyObject_MakeTpCall
	_PyEval_EvalFrameDefault

	_PyEval_EvalFrameDefault
	_PyEval_EvalCodeWithName
	PyEval_EvalCode

	PyRun_SimpleFileExFlags
	Py_RunMain
	Py_BytesMain
	__libc_start_main
	_start
*** End stack trace ***

INFO:absl:Unable to initialize backend 'cuda': INTERNAL: RET_CHECK failure (external/org_tensorflow/tensorflow/compiler/xla/pjrt/gpu_device.cc:345) local_device->device_ordinal() == local_topology.devices_size()
DEBUG:absl:Initializing backend 'rocm'
INFO:absl:Unable to initialize backend 'rocm': NOT_FOUND: Could not find registered platform with name: "rocm". Available platform names are: CUDA Interpreter Host
DEBUG:absl:Initializing backend 'tpu'
INFO:absl:Unable to initialize backend 'tpu': module 'jaxlib.xla_extension' has no attribute 'get_tpu_client'
WARNING:absl:No GPU/TPU found, falling back to CPU. (Set TF_CPP_MIN_LOG_LEVEL=0 and rerun for more info.)

Recently (in 0.3.18) there has been an update to the interface for clusters (Slurm and TPUpods), but it doesn't look like it's due to that (i.e. manually setting coordinator_address, num_processes and process_id in distributed.initialize(...) has the same effect).
Am I doing something wrong?
What jax/jaxlib version are you using?
jax==0.3.18, jaxlib==0.3.15+cuda11.cudnn82
Which accelerator(s) are you using?
GPUs
Additional system info
No response
NVIDIA GPU info
No response

The current jaxlib release (0.3.15) is missing support for using subsets of the CUDA devices, which the SLURM support needs. We need either to make a new jaxlib release (which we are working on already) or you can build jaxlib from source for the moment. We're working on it!

Thanks for the info. Is there a timeline for the next release of jaxlib?

Should be fixed IIUC.

This should be fixed with jax and jaxlib 0.3.20, which we just released. Please try it out!

Thanks for the update. I was compiling the new version, but I guess tomorrow I’ll try now with the precompiled.


(In Code Review) Add support for pagination in the get opportunity activities endpoint

https://jira.desk.technology/browse/APIJ-48

:+1:


READY FOR REVIEW - Fix #185. Return argument unmolested from ensure_new_type if conversion to new type does not exist.

Fix #185. Return argument unmolested from ensure_new_type if conversion to new type does not exist.

Great, thanks for your help, Matt!


Npm package old version

@ohalay
Oops. Seems you're right. I'll do it when I'm not so busy.

I just uploaded a new version to npm.

@VladimirRybalko Great job dude


npm package

commit with version 2.0.0 (published on npm) not in repository

Pushed 5b2d03d,  had to rebase so published as 2.0.1.


OCPBUGS-5502: Rebase v1.34.5

Issue #, if available:
https://issues.redhat.com/browse/OCPBUGS-5502
Description of changes:
Rebase to the latest upstream version that fixes multiple problems, among others the race described in the OCPBUGS-5502.
Diff to upstream: https://github.com/aws/efs-utils/compare/v1.34.5...tsmetana:rebase-4.12-v1.34.5

/jira refresh

/backport-risk-assessed
/cherry-pick-approved

/label backport-risk-assessed
/label cherry-pick-approved

/label backport-risk-assessed

/label cherry-pick-approved

CI passed twice in 4.13 with the rebased efs-utils
/lgtm
/approve

/cherry-pick release-4.11
Let's how it looks like in 4.11.


Add strong/weak pointer counts to Shared and WeakShared

Closes #2330

CI failing on 1.37.0 and 1.39.0 due use of unstable library feature 'weak_counts'. Any advice on how I should proceed?

CI failing on 1.37.0 and 1.39.0 due use of unstable library feature 'weak_counts'. Any advice on how I should proceed?

I would prefer to add these methods only to Shared for now. (but, if someone really needs WeakShared::*count, I'm okay with adding it using a build script.)

I personally have no use for WeakShared::*_count, I added them to anticipate a request for them. Removing them for now.


Lables

hi!
is it possible to use labels functionality, similar to "labels" in gmail?

Nope, only starred messages.

Oh, no labels for letters.
I need labels for contacts.
it's like a "mailing group"

#1170


Update several distributions to their latest stable versions

Starting with Fedora, I'll update all distros. This happens in several commits because I want to do it step by step.

Once this is merged, exaile/exaile-testimg#9 is the next step.

@virtuald, @sjohannes : Should we wait for the 4.0 final release or do this now?

Now seems fine.


glibc security

We seem to have unpatched vulnerabilities (noticed by nixpkgs monitor):

http://web.nvd.nist.gov/view/vuln/detail?vulnId=CVE-2014-7817
http://web.nvd.nist.gov/view/vuln/detail?vulnId=CVE-2014-8121

The first one was explicitly claimed to be fixed in 2.21 NEWS, although the CVE entry claims that version still suffers from it.


Improve Battery Charging Icons

As a follow-up to #4 , some entities are a bit different and require a bit more than the boolean .
Two examples of this that I have are my iOS App devices, and my vacuum (Roomba).
Vacuum:
By default, it's using these attributes (example):
battery_level: 100
battery_icon: 'mdi:battery-charging-100'
status: Charging

The main thing here is that the status could also indicate other things than just the battery status, ie stopped, running, etc
The other is the iOS devices, using the official iOS app. These have a battery state sensor, for example: sensor.my_iphone_battery_state where it has the actual state being either Charging or Not Charging. This sensor also has some other attributes, such as the below:
Battery Level: 20
Low Power Mode: false
friendly_name: My iPhone Battery State
icon: 'mdi:battery'
device_class: battery

Further to that, the sensor which is actually used in the battery entity card is the sensor.my_iphone_battery_level. This has an attribute called Battery State: which seems to be referencing the battery state sensor. Attribute example:
Battery State: Charging
Low Power Mode: false
unit_of_measurement: '%'
friendly_name: My iPhone Battery Level
icon: 'mdi:battery-charging-20'
device_class: battery

Most of this can be found in the HA Companion Documentation, and more specifically, the Battery Sensors

Version 1.2.0 added support for specifying custom charging state(s) as well as automatically supporting on, charging or true as valid charging states. Let me know if this does not work as expected.


bug? No commits were scanned

I have this command:
gitleaks  \
    --ssh-key='/keys/ssh/values/bucket' \
    --repo "$line"

I loop over my org's repo urls, we have 100+ repos.
I see this in my logs:
INFO[2019-05-10T23:26:39Z] cloning git@bitbucket.org:interos/metavana-webapp.git
INFO[2019-05-10T23:26:39Z] 0 leaks detected. 0 commits inspected in
INFO[2019-05-10T23:26:39Z] cloning git@bitbucket.org:interos/productionappserver.git
INFO[2019-05-10T23:26:39Z] 0 leaks detected. 0 commits inspected in
INFO[2019-05-10T23:26:39Z] cloning git@bitbucket.org:interos/concepts_adaptive_engine.git
INFO[2019-05-10T23:26:39Z] 0 leaks detected. 0 commits inspected in
INFO[2019-05-10T23:26:39Z] cloning git@bitbucket.org:interos/datavana.git
INFO[2019-05-10T23:26:39Z] 0 leaks detected. 0 commits inspected in
INFO[2019-05-10T23:26:39Z] cloning git@bitbucket.org:interos/products-tools.git
INFO[2019-05-10T23:26:39Z] 0 leaks detected. 0 commits inspected in
INFO[2019-05-10T23:26:39Z] cloning git@bitbucket.org:interos/mrapi.git
INFO[2019-05-10T23:26:39Z] 0 leaks detected. 0 commits inspected in
INFO[2019-05-10T23:26:39Z] cloning git@bitbucket.org:interos/entitycollection.git
INFO[2019-05-10T23:26:39Z] 0 leaks detected. 0 commits inspected in
INFO[2019-05-10T23:26:39Z] cloning git@bitbucket.org:interos/discogsapi.git
INFO[2019-05-10T23:26:39Z] 0 leaks detected. 0 commits inspected in
INFO[2019-05-10T23:26:39Z] cloning git@bitbucket.org:interos/web3.git
INFO[2019-05-10T23:26:39Z] 0 leaks detected. 0 commits inspected in
INFO[2019-05-10T23:26:39Z] cloning git@bitbucket.org:interos/tagvana.wikipedia2.git
INFO[2019-05-10T23:26:39Z] 0 leaks detected. 0 commits inspected in
INFO[2019-05-10T23:26:39Z] cloning git@bitbucket.org:interos/pwp.git
INFO[2019-05-10T23:26:39Z] 0 leaks detected. 0 commits inspected in

so it looks like the clones all worked, ssh-key is good, but never any commits inspected, always 0? why would that be?

I'm not sure what the issue could be. I tried cloning a bunch of private repos using a custom ssh-key path and it worked fine. Gitleaks is recognizing that you have supplied a valid ssh key otherwise you would be seeing this error message: https://github.com/zricethezav/gitleaks/blob/master/src/config.go#L230-L232

@zricethezav I've seen a problem were an unparseable gitleaks.toml file will result in the above instead of erroring out.
Simply adding --repo-config without a valid .gitleaks.toml file in the root of the repository triggers it. When looking at the code I spotted the following line:
https://github.com/zricethezav/gitleaks/blob/3b6c198715026ddfa7d6a68ba371e84c0f5c6197/src/repo.go#L147
Which in the case of an error from updateFromRepo, which looks like it returns the current leaks and nil as the error to the caller instead of err.
By comparison, when auditing a branch further down in the same file at:
https://github.com/zricethezav/gitleaks/blob/3b6c198715026ddfa7d6a68ba371e84c0f5c6197/src/repo.go#L168
I see it returning err in the case that it's not nil.
Looking around the code I see the pattern of testing and then returning nothing for err at:
https://github.com/zricethezav/gitleaks/blob/3b6c198715026ddfa7d6a68ba371e84c0f5c6197/src/repo.go#L155
https://github.com/zricethezav/gitleaks/blob/3b6c198715026ddfa7d6a68ba371e84c0f5c6197/src/repo.go#L198
https://github.com/zricethezav/gitleaks/blob/3b6c198715026ddfa7d6a68ba371e84c0f5c6197/src/repo.go#L335
https://github.com/zricethezav/gitleaks/blob/3b6c198715026ddfa7d6a68ba371e84c0f5c6197/src/repo.go#L350
I'm not very familiar with golang, but given that normally I'd expect to see something output a log message or doing something about the error, are these just silently squashing errors and returning before the scanning gets performed resulting the output above?
Should I log a separate issue for the --repo-config or are they all just the same and can be handled under this issue?

Other issue for --repo-config silently failing with an invalid config file: https://github.com/zricethezav/gitleaks/issues/248

@themills You should use --repo="$line" the following should work:
gitleaks  \
    --ssh-key='/keys/ssh/values/bucket' \
    --repo="$line"

Hello @the1mills !
Has this been solved?
Can you try with newer versions fo gitleaks?
Thank you!

gitleaks is not responsible for cloning in v8+


Running any sqitch command results in an error

After installing sqitch_pg with brew on a macbook, trying to run any command results in the following error for me:
Can’t locate Locale/TextDomain.pm in @INC (you may need to install the Locale::TextDomain module)

I'd appreciate any help in understanding what's going on / fixing the problem. Do I need to manually install a perl module?

Weird, it should have been installed. Yes, try installing manually:
cpanm --local-lib "`brew --prefix`" --notest Locale::TextDomain

I had some success with this just now. For the sake of comparison- here's my brew install sqitch_pg output.  Anyway, working now?
BTW, probably better to post Homebrew issues to /theory/homebrew-sqitch.

Having this same error on Debian 9. This is probably related to CVE-2016-1238. The immediate fix is to export PERL_USE_UNSAFE_INC=1 before installing but this could be a security issue. For more information on a long term fix see:

libintl-perl: CVE-2016-1238
libintl-perl: Fails install when no '.' in @INC

Also check frioux/DBIx-Class-Helpers#81 which is pretty much the same problem. Looks like this needs to be fixed on libintl-perl itself and not on Sqitch

Looks to be a known libintl-perl issue. Thanks for digging, @carragom.

Hey, I just run into this issue, and after running: brew doctor, I realized I had some unlinked dependencies.
Leaving kegs unlinked can lead to build-trouble and cause brews that depend on
those kegs to fail to run properly once built. Run `brew link` on these:
  sqitch_dependencies```

So I run `brew link --overwrite sqitch_dependencies`

I hope this helps anyone getting this issue.


Private YouTube video

This issue is user submitted and needs to be validated:
15.4.1. TCP
Video link goes to https://www.youtube.com/watch?v=WwyJGzZmBe8 which is private and requires a signin

Duplicate of #408


[BUG] Assumption that cache eviction does not happen

From #465: "The existing code assumes that all of the entities are in the cache once the Resolver runs. That process puts all of the entities and their grants into the cache so that by the time the Authorizer retrieves the grants, it fetches them from the cache."
It sounds like that the code really relies on the entities being available from the cache, which means that no eviction must happen. If eviction happens (which must be assumed to happen at any point in time to any cache entry) requests can fail at any time w/ misleading errors and/or wrong information.

At least in the linked PR, all cache lookups invoke entityCache.getOrLoadEntityById - this will validate that the cache entry is present and do the lookup if it is not. The "assumption" mentioned is on the performance characteristics, not the correctness of the code.


MWPW-128105 - stage.adobe.com + MILO libs

Resolve: https://jira.corp.adobe.com/browse/MWPW-128105
According from the @JFernandezAdobe requirement, files which were coming from main-milo-adobecom.hlx.page before, now are coming from  www.adobe.com.
With this change, www.stage.adobe.com will use production code (www.adobe.com), but with ?milolibs param it can be set to use main--milo--adobecom.hlx.page/libs or localhost/libs.
I put the label 'do not merge' because I need confirmation that libs should come from www.adobe.com and not from http://www.stage.adobe.com

libs needs to be assigned for each case otherwise getLibs will be undefined.

Thank you. Please review now

This makes sense and looks ok to me but.. this is also getting more off from original function from milo-college(https://github.com/adobecom/milo-college/blob/main/scripts/utils.js#L16-L30)
Can we do simply add one line of code from original function..? Something like this?: if (branch === 'main' && hostname === 'www.stage.adobe.com') return 'https://www.adobe.com/libs';
So, the function will be like:
export const [setLibs, getLibs] = (() => {
  let libs;
  return [
    (prodLibs, location) => {
      libs = (() => {
        const { hostname, search } = location || window.location;
        if (!(hostname.includes('.hlx.') || hostname.includes('local'))) return prodLibs;
        const branch = new URLSearchParams(search).get('milolibs') || 'main';
        if (branch === 'main' && hostname === 'www.stage.adobe.com') return 'https://www.adobe.com/libs';
        if (branch === 'local') return 'http://localhost:6456/libs';
        return branch.includes('--') ? `https://${branch}.hlx.live/libs` : `https://${branch}--milo--adobecom.hlx.live/libs`;
      })();
      return libs;
    }, () => libs,
  ];
})();

I haven't tested above yet, but it seems same logic to me.

Thank you @seanchoi-dev . Can you, please, review now?

Did you test with my suggested function? I still see the outdated code... like
const stageEnvs = ['localhost', 'hlx.page', 'hlx.live', 'stage.adobe.com'];
if (!stageEnvs.some((env) => hostname.includes(env))) {
  return prodLibs;
}

Wondering if you have reason to keep them.

if (!(hostname.includes('.hlx.') || hostname.includes('local'))) return prodLibs;

With:
if (!(hostname.includes('.hlx.') || hostname.includes('local'))) return prodLibs;
is not covered case when hostname is stage.adobe.com, it will return prodLibs

@seanchoi-dev
Please take a look now.
I added 'stage'
       if (!(hostname.includes('.hlx.') || hostname.includes('local') || hostname.includes('stage'))) return prodLibs;
without that it is not possible test some branch with ?milolibs param

I approved it but do you have the test URL that having the issue you mentioned? I am kinda curious.


NPM Release

Do you have an ETA for the TS 2.0 support making it into NPM?

Hi @mike-morrison ,
The latest version is on our branch called "releases". So you should be able to point your npm dependency to that branch to get the latest. https://github.com/Microsoft/tslint-microsoft-contrib/tree/releases
I'll start the release process now though. The OSS group in Seattle does the actual publish so it can take up to 24 hours to hit npm.


core-js Promise polyfill not working with Shaka Player on IE11

Have you read the FAQ and checked for duplicate issues: Yes
What version of Shaka Player are you using:  2.2.2
Can you reproduce the issue with our latest release version: Yes 2.2.2
Can you reproduce the issue with the latest code from master:  Not tried.
Are you using the demo app or your own custom app: Own app
If custom app, can you reproduce the issue using our demo app: No, not sure how I could.
What browser and OS are you using: IE11 (works in Edge, Chrome), Windows 10
What are the manifest and license server URIs:
(you can send the URIs to<EMAIL_ADDRESS>instead, but please use GitHub and the template for the rest)
What did you do?
Added promise polyfil<EMAIL_ADDRESS>and use Shaka.  eg:
import { shaka } from 'shaka-player/dist/shaka-player.compiled.debug';
import 'core-js/modules/es6.promise';

What did you expect to happen?
Playback to work.
What actually happened?
Get errors and no playback is possible.
IndexedDB.install
InputEvent.install
Patching input event support on IE.
mathRound.install
polyfill Math.round
MediaKeys.install
Using ms-prefixed EME v20140218
PatchedMediaKeysMs.install
MediaSource.install
Using native MSE as-is.
Using native Promises.
VideoPlayPromise.install
Using VTTCue polyfill from 3 argument TextTrackCue.
Unload latency: 0.041
Wrong error type!
load() failed: TypeError: [object Promise] is not iterable!
   "load() failed:"
   {
      [functions]: ,
      __proto__: { },
      message: "[object Promise] is not iterable!",
      name: "TypeError",
      stack: "TypeError: [object Promise] is not iterable!
   at module.exports (eval code:14:34)
   at Anonymous function (eval code:271:7)
   at perform (eval code:61:5)
   at all (eval code:267:5)
   at shaka.dash.MpdUtils.processXlinks (eval code:288:62)
   at shaka.dash.DashParser.prototype.parseManifest_ (eval code:389:214)
   at Anonymous function (eval code:388:374)
   at run (eval code:89:13)
   at Anonymous function (eval code:102:28)
   at flush (eval code:18:9)",
      Symbol()_h.u53ox2ia3cc: undefined,
      Symbol(observable)_k.u53ox2ia3cc: undefined,
      Symbol(react.element)_g.u53ox2ia3cc: undefined,
      Symbol(Symbol.asyncIterator)_i.u53ox2ia3cc: undefined,
      Symbol(Symbol.observable)_j.u53ox2ia3cc: undefined
   }

Followed by 10 - 15 repeats of
Unhandled promise rejection TypeError:  is not iterable!
   "Unhandled promise rejection"
   {
      [functions]: ,
      __proto__: { },
      message: " is not iterable!",
      name: "TypeError",
      stack: "TypeError:  is not iterable!
   at module.exports (eval code:14:34)
   at Anonymous function (eval code:271:7)
   at perform (eval code:61:5)
   at all (eval code:267:5)
   at shaka.dash.MpdUtils.processXlinks (eval code:288:62)
   at shaka.dash.MpdUtils.processXlinks (eval code:288:2)
   at shaka.dash.MpdUtils.processXlinks (eval code:288:2)
   at shaka.dash.MpdUtils.processXlinks (eval code:288:2)
   at shaka.dash.DashParser.prototype.parseManifest_ (eval code:389:214)
   at Anonymous function (eval code:388:374)",
      Symbol()_h.u53ox2ia3cc: undefined,
      Symbol(observable)_k.u53ox2ia3cc: undefined,
      Symbol(react.element)_g.u53ox2ia3cc: undefined,
      Symbol(Symbol.asyncIterator)_i.u53ox2ia3cc: undefined,
      Symbol(Symbol.observable)_j.u53ox2ia3cc: undefined
   }

Note.  If I remove the polyfil then shaka works.
Please let me know if I can help in any way, provide more / clearer details etc.

Shaka Player has its own promise polyfill, which is installed by shaka.poyfill.installAll() if window.Promise isn't defined. Does the problem still happen if you don't import the core-js promise polyfill?

Does the problem still happen if you don't import the core-js promise polyfill?

Yes this still holds true 👍 "Note. If I remove the polyfil then shaka works."
Is it possible that shaka is actually polyfilling more than just promise in promise.js ?  I'm not overly familiar with the promise spec but I'll take a look at https://github.com/google/shaka-player/blob/adb8da4764e3080693252ba0211cd4f936fa27be/lib/polyfill/promise.js

Ah, I didn't see that you had addressed that already. My apologies.
Anyway... I'm looking at es6.promise.js, and it looks like their implementation of Promise.all assumes that promises conform to the iterable protocol. We don't use that protocol, to reduce the number of polyfills we need to maintain.
So maybe it's using our implementation of promises, but core-js has overwritten our implementation of Promise.all? I'll look into it further when I have the spare time.

In case anyone else finds same issue:  our temp fix is to just use the shaka polyfil 😅
// There are issues with core-js promise polyfil and shaka in IE11.
// For now we will rely on shaka promise polyfil.
// See: https://github.com/google/shaka-player/issues/1062
// import 'core-js/modules/es6.promise';

import { polyfill } from 'shaka-player';

polyfill.installAll();

Okay, I've done more testing, and I think this might just be a bug with core-js.
I tried loading only the core-js promise module into a browser, without Shaka Player, and typed
Promise.all([Promise.resolve()])
into the IE browser console, and it threw an exception about [object Object] not being iterable.

@theodab, thanks for looking into this.  Can you please file a bug on core-js?  https://github.com/zloirock/core-js/issues/new
In that bug, please mention google/shaka-player#1062 to link back here for context, and mention @bengeorge and myself.  Thanks!

Looking at core-js's github page, it seems this might be a case of using core-js improperly.
See this and this issue. They warn against using files in core-js/modules/*, as they are internal and not meant to be directly linked unless you really know what you are doing. Instead, they say to include to core-js/fn/promise.

Good find.  Thanks!  @bengeorge, does this help?

import 'core-js/fn/promise'; works perfectly 👍
@joeyparrish @theodab You guys are awesome.  Thanks a bunch for looking into that!
And sorry for wasting your time 😅