Error building project on MacOS

Hi,
I am trying to build the Verus project on my Mac following the instructions on the README, after running tools/update-rust.sh I encounter an error. Here's what I get :
rustc exited with signal: 6
error: could not compile `rustc_middle`

Caused by:
  process didn't exit successfully: `/Users/loris/verus/rust/build/bootstrap/debug/rustc --crate-name rustc_middle --edition=2021 compiler/rustc_middle/src/lib.rs --error-format=json --json=diagnostic-rendered-ansi,artifacts --crate-type lib --emit=dep-info,metadata,link -C opt-level=3 -C embed-bitcode=no -C debuginfo=0 -C metadata=ed56d0b053bcfdbd -C extra-filename=-ed56d0b053bcfdbd --out-dir /Users/loris/verus/rust/build/x86_64-apple-darwin/stage1-rustc/x86_64-apple-darwin/release/deps --target x86_64-apple-darwin -C incremental=/Users/loris/verus/rust/build/x86_64-apple-darwin/stage1-rustc/x86_64-apple-darwin/release/incremental -L dependency=/Users/loris/verus/rust/build/x86_64-apple-darwin/stage1-rustc/x86_64-apple-darwin/release/deps -L dependency=/Users/loris/verus/rust/build/x86_64-apple-darwin/stage1-rustc/release/deps --extern bitflags=/Users/loris/verus/rust/build/x86_64-apple-darwin/stage1-rustc/x86_64-apple-darwin/release/deps/libbitflags-084bf38c02022602.rmeta --extern chalk_ir=/Users/loris/verus/rust/build/x86_64-apple-darwin/stage1-rustc/x86_64-apple-darwin/release/deps/libchalk_ir-65a3476fe8ea65e7.rmeta --extern either=/Users/loris/verus/rust/build/x86_64-apple-darwin/stage1-rustc/x86_64-apple-darwin/release/deps/libeither-39c15c6d8d6de952.rmeta --extern gsgdt=/Users/loris/verus/rust/build/x86_64-apple-darwin/stage1-rustc/x86_64-apple-darwin/release/deps/libgsgdt-6698b3a0ddbab467.rmeta --extern polonius_engine=/Users/loris/verus/rust/build/x86_64-apple-darwin/stage1-rustc/x86_64-apple-darwin/release/deps/libpolonius_engine-653e0aa6ed61bb60.rmeta --extern rand=/Users/loris/verus/rust/build/x86_64-apple-darwin/stage1-rustc/x86_64-apple-darwin/release/deps/librand-e47352adffa82242.rmeta --extern rand_xoshiro=/Users/loris/verus/rust/build/x86_64-apple-darwin/stage1-rustc/x86_64-apple-darwin/release/deps/librand_xoshiro-5a4c17239dddc73b.rmeta --extern rustc_rayon=/Users/loris/verus/rust/build/x86_64-apple-darwin/stage1-rustc/x86_64-apple-darwin/release/deps/librustc_rayon-7888addd4e07c3e8.rmeta --extern rustc_rayon_core=/Users/loris/verus/rust/build/x86_64-apple-darwin/stage1-rustc/x86_64-apple-darwin/release/deps/librustc_rayon_core-fb7beb3cafa2e4af.rmeta --extern rustc_apfloat=/Users/loris/verus/rust/build/x86_64-apple-darwin/stage1-rustc/x86_64-apple-darwin/release/deps/librustc_apfloat-a10adcd75b890b89.rmeta --extern rustc_arena=/Users/loris/verus/rust/build/x86_64-apple-darwin/stage1-rustc/x86_64-apple-darwin/release/deps/librustc_arena-02897dc85c1a3658.rmeta --extern rustc_ast=/Users/loris/verus/rust/build/x86_64-apple-darwin/stage1-rustc/x86_64-apple-darwin/release/deps/librustc_ast-cd09a9326f104fd4.rmeta --extern rustc_attr=/Users/loris/verus/rust/build/x86_64-apple-darwin/stage1-rustc/x86_64-apple-darwin/release/deps/librustc_attr-3c3fa44c40717821.rmeta --extern rustc_data_structures=/Users/loris/verus/rust/build/x86_64-apple-darwin/stage1-rustc/x86_64-apple-darwin/release/deps/librustc_data_structures-1061aa8f295bf009.rmeta --extern rustc_errors=/Users/loris/verus/rust/build/x86_64-apple-darwin/stage1-rustc/x86_64-apple-darwin/release/deps/librustc_errors-ab612a9bd647f704.rmeta --extern rustc_feature=/Users/loris/verus/rust/build/x86_64-apple-darwin/stage1-rustc/x86_64-apple-darwin/release/deps/librustc_feature-29adda405d71debd.rmeta --extern rustc_graphviz=/Users/loris/verus/rust/build/x86_64-apple-darwin/stage1-rustc/x86_64-apple-darwin/release/deps/librustc_graphviz-6b9cac4911dbb6f0.rmeta --extern rustc_hir=/Users/loris/verus/rust/build/x86_64-apple-darwin/stage1-rustc/x86_64-apple-darwin/release/deps/librustc_hir-4ce5ef9fab936309.rmeta --extern rustc_index=/Users/loris/verus/rust/build/x86_64-apple-darwin/stage1-rustc/x86_64-apple-darwin/release/deps/librustc_index-24ac7dc9892ce8c4.rmeta --extern rustc_macros=/Users/loris/verus/rust/build/x86_64-apple-darwin/stage1-rustc/release/deps/librustc_macros-e3e3f4c4d54030f0.dylib --extern rustc_query_system=/Users/loris/verus/rust/build/x86_64-apple-darwin/stage1-rustc/x86_64-apple-darwin/release/deps/librustc_query_system-3eff3c0b926a5993.rmeta --extern rustc_serialize=/Users/loris/verus/rust/build/x86_64-apple-darwin/stage1-rustc/x86_64-apple-darwin/release/deps/librustc_serialize-bbc7cc88ffd427df.rmeta --extern rustc_session=/Users/loris/verus/rust/build/x86_64-apple-darwin/stage1-rustc/x86_64-apple-darwin/release/deps/librustc_session-04eb547175547562.rmeta --extern rustc_span=/Users/loris/verus/rust/build/x86_64-apple-darwin/stage1-rustc/x86_64-apple-darwin/release/deps/librustc_span-a81d5ff9e8fbbd5c.rmeta --extern rustc_target=/Users/loris/verus/rust/build/x86_64-apple-darwin/stage1-rustc/x86_64-apple-darwin/release/deps/librustc_target-7ec14da5fe7d92b0.rmeta --extern rustc_type_ir=/Users/loris/verus/rust/build/x86_64-apple-darwin/stage1-rustc/x86_64-apple-darwin/release/deps/librustc_type_ir-e6a42e601be596a1.rmeta --extern smallvec=/Users/loris/verus/rust/build/x86_64-apple-darwin/stage1-rustc/x86_64-apple-darwin/release/deps/libsmallvec-1aa785a6627c59db.rmeta --extern tracing=/Users/loris/verus/rust/build/x86_64-apple-darwin/stage1-rustc/x86_64-apple-darwin/release/deps/libtracing-9cc860bd2fb03c61.rmeta -Zsymbol-mangling-version=v0 -Zmacro-backtrace -Zosx-rpath-install-name<EMAIL_ADDRESS>-Csplit-debuginfo=unpacked -Ztls-model=initial-exec -Zunstable-options '-Wrustc::internal' -Cprefer-dynamic -Cllvm-args=-import-instr-limit=10 -Z binary-dep-depinfo -L native=/Users/loris/verus/rust/build/x86_64-apple-darwin/stage1-rustc/x86_64-apple-darwin/release/build/psm-14e7f5a490b013b1/out` (exit status: 254)
warning: build failed, waiting for other jobs to finish...
error: build failed
Build completed unsuccessfully in 0:50:20

Can someone please help me troubleshoot this issue?
Thank you!

Hi @LorisPalma,
Thanks for the report. Unfortunately there is an libc allocator issue in macOS ( https://github.com/rust-lang/rust/issues/92173 ) and my understanding is that it is baked into the early stage0 compiler binary that is downloaded to build the version of rust we are using. I am not aware of a direct fix. Fortunately, it only crashes the compiler sometimes and re-running the last command of tools/update-rust.sh by hand once or twice should work:
In the rust directory, run:
$python_command ./x.py install -i(where $python_command is python or python3 depending on which is available on your system)
This will only re-attempt the build of the parts of rustc that failed (rustc_middle in this case) and their dependencies, which is more likely to succeed.
Please ask again if you encounter further issues.

FYI, we are currently working on switching to a much more recent (and pre-compiled) rustc distribution, so it will hopefully be much easier to get set up in the future.

Thank you for your answer.
Unfortunately, even after running the python3 ./x.py install -i 3-4 times, it didn't succeed.
As a temporary solution, I'll probably use a VM with Ubuntu on my Mac.

IIRC, in the past I've had better luck leaving the -i off. You might try that and see if it helps.

Simply running python3 ./x.py install worked just fine.
Thank you so much!

Hello,
Sorry to bother again, but I can't manage to build the verifier.
When I run ./tools/cargo.sh build I get :
    Updating git repository `https://github.com/utaal/getopts.git`
./tools/cargo.sh: line 22:  4017 Segmentation fault: 11  VERUS_Z3_PATH=/Users/loris/verus/source/z3 DYLD_LIBRARY_PATH=/Users/loris/verus/source/../rust/install/lib/rustlib/x86_64-apple-darwin/lib RUSTC=../rust/install/bin/rustc RUSTDOC=../rust/install/bin/rustdoc ../rust/install/bin/cargo build

and when I run RUSTC=../rust/install/bin/rustc ../rust/install/bin/cargo build I also get :
    Updating git repository `https://github.com/utaal/getopts.git`
zsh: segmentation fault  RUSTC=../rust/install/bin/rustc ../rust/install/bin/cargo build

I don't know if I'm doing something wrong, can someone please help me ?

Hi @LorisPalma,
Sorry for not being a whole lot of help here. I don't think any of us have any idea why this is happening.
However, we've completely revamped the build system on the branch main_new, which is going to be the default branch soon. The new system no longer requires building rustc from source, so I'm hopeful you might have more luck with it.

The new build worked like a charm, thank you for the answer.

@LorisPalma a quick note: we're still tweaking the new build, so please re-activate the environment every time you pull, so you're sure you have the latest vargo.


Milliseconds Example Added From processing.org to p5js.org

Fixes #962
Changes:
-Added Milliseconds example in Inputs From processing.org to p5js.org

renamed the examples of input folder to maintain order similar to processing/examples/input

Screenshots of the change:

Link to original example added. Thanks

Thank You very much 😊


Changed watch statement in group by scenario to be watchCollection since...

... using just watch will only check if the reference of the array changes (as opposed to watchCollection that looks for additions/subtractions to the array
I think this helps the situation described in https://github.com/dotansimha/angularjs-dropdown-multiselect/issues/32

All watches have been removed


OLS-712: TODO for OLS-712

Description
TODO for OLS-712
Type of change

[ ] Refactor
[ ] New feature
[ ] Bug fix
[ ] CVE fix
[ ] Optimization
[x] Documentation Update
[ ] Configuration Update
[ ] Bump-up dependent library
[ ] Bump-up library or tool used for development (does not change the final image)

Related Tickets & Documents

Related Issue #OLS-712
Closes #

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

Project coverage is 92.98%. Comparing base (0322da5) to head (01532ee).
Report is 6 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #1074      +/-   ##
==========================================
- Coverage   95.36%   92.98%   -2.39%
==========================================
  Files          55       55
  Lines        2353     2351       -2
==========================================
- Hits         2244     2186      -58
- Misses        109      165      +56

see 6 files with indirect coverage changes


Getting unauthorized while adding custom header

While trying to add custom header using the createApiWithRequestInterceptor
MailgunMessagesApi mailgunMessagesApi = MailgunClient.config(API_KEY)
            .createApiWithRequestInterceptor(MailgunMessagesApi.class,
                MailgunRequestInterceptor.builder()
                    .addProperty("h:X-My-Header", "my_custom_header")
                    .build()
            );

we get the following error (No error when using createAPI method)
feign.FeignException$Unauthorized: [401 Unauthorized] during [POST] to [https://api.mailgun.net/v3/[my_verified_domain]/messages?h%3AX-My-Header=my_custom_header] [MailgunMessagesApi#sendMessage(String,Message)]: []
        at feign.FeignException.clientErrorStatus(FeignException.java:224) ~[feign-core-12.2.jar:?]
        at feign.FeignException.errorStatus(FeignException.java:203) ~[feign-core-12.2.jar:?]
        at feign.FeignException.errorStatus(FeignException.java:194) ~[feign-core-12.2.jar:?]
        at feign.FeignException.errorStatus(FeignException.java:171) ~[feign-core-12.2.jar:?]
        at feign.codec.ErrorDecoder$Default.decode(ErrorDecoder.java:92) ~[feign-core-12.2.jar:?]
        at feign.ResponseHandler.decodeError(ResponseHandler.java:136) ~[feign-core-12.2.jar:?]
        at feign.ResponseHandler.handleResponse(ResponseHandler.java:70) ~[feign-core-12.2.jar:?]
        at feign.SynchronousMethodHandler.executeAndDecode(SynchronousMethodHandler.java:114) ~[feign-core-12.2.jar:?]
        at feign.SynchronousMethodHandler.invoke(SynchronousMethodHandler.java:70) ~[feign-core-12.2.jar:?]
        at feign.ReflectiveFeign$FeignInvocationHandler.invoke(ReflectiveFeign.java:96) ~[feign-core-12.2.jar:?]

Are there any other ways to include a custom header?
Note: Removed the domain name from the error and we are using the 1.1.1 sdk version

@arundaskd, thanks for raising this issue!
Sorry for the late reply.
Based on your description you are trying to add a custom header but on the code snippet I see that you are adding form property .addProperty("h:X-My-Header", "my_custom_header")
If you read a Javadoc for MailgunRequestInterceptor you can see the description and difference

Also, you can find the same here https://github.com/mailgun/mailgun-java?tab=readme-ov-file#client-configuration
So can you please try to add your custom header in this way?
MailgunMessagesApi mailgunMessagesApi = MailgunClient.config(API_KEY)
            .createApiWithRequestInterceptor(MailgunMessagesApi.class,
                MailgunRequestInterceptor.builder()
                    .addHeader("X-My-Header", "my_custom_header")
                    .build()
            );

And please let us know if everything works as expected!

@vitalii-chornobryvyi-swi There was no more error when using addHeader, but I couldn't find this header in the mailgun logs nor in the original email headers.
We want to add List-Unsubscribe header with custom value.

Let me clarify something. Are you expecting to see List-Unsubscribe header in your letter for example like on this screenshot?

If I understand correctly, then can you please do like this

And please let us know if everything works as expected!

@vitalii-chornobryvyi-swi Worked like charm. Thanks a lot 🙏

Anytime, thank you for your proactivity and communication


Throw error when missing commas in Array and Object expressions

Fixes #605
Formerly:
[1 2 3]; // [1, 2, 3];

let [a b c] = []; // let [a, b, c] = [];

({ a: 1 b: 2 c: 3 }); // ({a: 1, b:2, c: 3});

let { a b c } = {}; // let { a, b, c } = {};

let [...rest, a] = []; // let [...rest_0] = [];

[...[],]; // [...[],,];

All of these cases save the last should throw (it was simply parsing incorrectly). Now they do:
[1 2 3]
// Error: unexpected token
// __2__ 3

let [a b c] = [];
// Error: unexpected token
// __b__ c

({ a: 1 b: 2 c: 3 })
// Error: unexpected token
// __b__ : 2 c : 3

let { a b c } = {};
// Error: unexpected token
// __b__ c

let [...rest, a] = [];
// Error: Rest element must be last element in array
// , a

[...[],]; // [...[]];

@disnet I realized after the fact that I pushed this branch to the main repo. Do you want me working in here or should I delete the branch and resubmit from my repo?

This isn't quite right yet. I messed up on the with the spread operator.

It's ready to review.

This looks great! Are there some test262 tests we can enable now to catch this?

btw creating branches on this repo is totally fine

Are there some test262 tests we can enable now to catch this?

If not, we should add some.
We need to start running the fail and early tests. I'm sure that would give us tons of work to do 😄

Do you want me to submit a PR to enable the early and fail tests before merging this one? It would be nice to make sure we have a regression test in place.

Yeah that would be great
On Wed, Mar 22, 2017 at 7:32 AM Gabe Johnson<EMAIL_ADDRESS>wrote:

Do you want me to submit a PR to enable the early and fail tests before
merging this one? It would be nice to make sure we have a regression test
in place.
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/sweet-js/sweet.js/pull/650#issuecomment-288417250,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAAJOmN-MpqX-SqJ4yf2w7W5c_HLKwxUks5roTEFgaJpZM4MhhPs
.

These aren't bugs w/ the new code, I think the problem is here.  We should be a checking to see if = follows.

While ☝ is true, the problem is greater. The transformDestructuring trick won't work as the rules for what can follow a rest element are different than those for a spread element.
This is going to bite us again once object rest/spread lands.
I propose a refactor that uses double lookahead for in instances where there might be an AssignmentExpression and deals with the bindings right away.
As far as this PR goes, do you want to hold on it? Or merge it, comment out the offending tests and proceed with a refactor?

Let's merge it now and then refactor.


[BUG] Corp list in Intel window will not scroll w/ mouse wheel

In the corp selection popup box used to select the owning group a structure in the structure intel panel does not operate with the mouse scroll wheel. Only arrow keys can be used to select enteries on an extended result list.
Version: 1.4.3

This works for me in the current stable v1.5.1 Chrome browser


How to test batch?

I would like to ask what commands should be used or what code should be changed if testing multiple batch runs of the converted model?

summarize.py and run.py are both supported multiple batch runs.


[No QA] Update copy on terms page

Details
This PR updates 2 copy sections for the terms page of the VBA flow, specifically the updates in this comment. I've also added a screenshot to confirm that expensify.cash is not on the terms page and new.expensify.com is shown instead.
Fixed Issues
$ https://github.com/Expensify/Expensify/issues/167278
Tests
Go through the VBA flow, confirm that the copy is updated.
QA Steps
No QA
Tested On

[x] Web
[ ] Mobile Web
[ ] Desktop
[ ] iOS
[ ] Android

Screenshots

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to production by @sketchydroide in version: 1.1.22-2 🚀

platform
result

🤖 android 🤖
cancelled 🔪

🖥 desktop 🖥
cancelled 🔪

🍎 iOS 🍎
cancelled 🔪

🕸 web 🕸
cancelled 🔪

🚀 Deployed to staging by @aldo-expensify in version: 1.1.22-2 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by @yuwenmemon in version: 1.1.23-1 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
failure ❌

🍎 iOS 🍎
failure ❌

🕸 web 🕸
success ✅

🚀 Deployed to production by @yuwenmemon in version: 1.1.23-1 🚀

platform
result

🤖 android 🤖
failure ❌

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
failure ❌

🕸 web 🕸
success ✅


[REF-1996] Setting radix accordion.root.color_scheme gives javascript warning

Describe the bug
client.js:2 Warning: React does not recognize the colorScheme prop on a DOM element. If you intentionally want it to appear in the DOM as a custom attribute, spell it as lowercase colorscheme instead. If you accidentally passed it from a parent component, remove it from the DOM element.
To Reproduce
rx.accordion.root(
        rx.accordion.item(
            header="First Item",
            content=The First accordion item's content",
            font_size="3em",
            value="item_1",
        ),
        rx.accordion.item(
            header="Second Item",
            content="The second accordion item's content",
            font_size="3em",
            value="item_2",
        ),
        rx.accordion.item(
            header="Third item",
            content="The third accordion item's content",
            font_size="3em",
            value="item_3",
        ),
        collapsible=True,
        color_scheme='indigo',
    )

Expected behavior
Screenshots
Specifics (please complete the following information):

Python Version: 0.9.2
Reflex Version: 0.4.0
OS: debian 11
Browser (Optional):

Additional context
REF-1996

Thanks for reporting, will fix it up for 0.4.1


Change alc_layout_id to layout 1

The default alc_layout_id layout 7 did not work on my Asus Rog Maximus XI Hero so I change it to layout 1.

@hcwilhelm Thanks for the PR, I'll test it locally during the weekend and get back to you.

@hcwilhelm Actually, it works for me as well. I'll leave it to check if all is stable over the weekend.


Dirty staging branch

Changes Unknown when pulling 6f4bf9404c97cf97f321474b9e271ea1fa631ea5 on dirty-staging-branch into ** on staging-branch**.


invite to speak feature

an "invite to speak" button on their profile like the "make speaker" button.
when you invite someone in the audience to speak, a pop-up with the option to "join as speaker" or deny the request should present to the listener.

Yeah. You shouldn't be forced to talk.


withNetworkConnectivity makes componentDidUpdate fire with initial render

According to the React docs:

componentDidUpdate() is invoked immediately after updating occurs. This method is not called for the initial render.

https://reactjs.org/docs/react-component.html#componentdidupdate
However, when I use export default withNetworkConnectivity()(YourComponent), the componentDidUpdate() event is called even though I'm not changing state nor am I changing my network connectivity status. If I remove withNetworkConnectivity then componentDidUpdate is no longer called.
Is this expected behaviour?
This discussion may be relevant:

withNetworkConnectivity HoC subscribes to connectivity change events after the component that it wraps mounts. Being that component the top most component, there is an implicit delay since React & RN have to render the whole app for the 1st time.

Is this expected behaviour?

Yes, it could happen. You almost got to the point in the paragraph you quoted at the end. The component that withNetworkConnectivity wraps it first mounts and right after that it subscribes to connectivity events. It always performs an initial check to detect initial connectivity, calling setState internally. That causes your component to re-render, hence executing componentDidUpdate afterwards.
Even though the prop isConnected that gets injected to your component may not change, I think React will call the lifecycle method if you don't put into place any optimisations such as using a PureComponent or leveraging shouldComponentUpdate hook.


Dependency hell for stream_chat_v1 project

flutter pub get fails to resolve versions because of version constraints in pubspec.yaml. I took at a stab at trying to manually resolve the versions but the rabbit hole goes too deep for me. The project is currently unusable.

We're in the process of migrating to nullsafety and releasing a v2.0 of our package
You can try the new version in the develop branch


Show user location on Map

Hello,
would it be somehow possible to show the location of the user on the map?

The user's location should automatically be visible by turning on location services and giving the right permission's to the app.


[WIP] Blog starter example

This example will help to start off a blog using Next.js and MDX. It's still a work in progress, but I hope with this open pull request I can get more eyes on it and probably someone will want to contribute and help.
Visit the work-in-progress demo
Inspiration
This example is highly inspired by and uses code from:

@mxstbr 's blog
gatsby-remark-prismjs
rehype-prism
react-refractor

Progress

[x] get a post object from a list of MDX files
[x] add syntax-highlighting using Prism
[x] add support for line-highlighting like Gatsby
[ ] add test to make sure the syntax-highlighting behaves correctly
[ ] add pagination
[ ] add frontmatter support
[ ] sort the posts object
[ ] decide on the default styling/theme
[ ] extract mdx-prism into its own package or make a pull request to https://github.com/mapbox/rehype-prism that adds line-highlighting
[ ] add RSS feed
[ ] add plugin API?

I'm going to close this PR until it's ready 👍


Navigation bar Issue #14

Description
Added a Navbar which include links to different pages on the website, such as the home page, about us page, contact us page and other relevant pages.
Fixes # (Add Navigation Bar) #14
Type of change
Removed the Contribute section and added to navbar , media queries to be defined in the next commit after a review.
Please delete options that are not relevant.

[✔] 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 not work as expected)

[] This change requires a documentation update

Checklist:

[✔] My code follows the style guidelines of this project
[✔] I have performed a self-review of my own code
[✔ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[✔ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules

Hi @yuvrxj-afk
please remove the package-lock.json file

📝 Changed routes:

/

(view full size)

Commit 6bb2c53ff9d2945d2587ffc53b7bf335c4ca19e5 (https://testcase-generator-j87rbb94t-ank1traj.vercel.app).

@yuvrxj-afk Aren't we going to add navigation bar in every routes and also I think navigation bar is not responsive. Correct me if I am wrong.

We need to remove components from index.js and I mean that create nav/description/cards separately , not in a single file, then It won't be so complex.

@yuvrxj-afk This should be done completely. If you don't have bandwidth/time you can convey that. What is required is complete closure of the issue. Not by part by part.

I'm having some work , will try to commit soon @ank1traj In between if anyone can contribute that would be so great.

will be adding rest pages in issues , where others can contribute to navbar linked pages.

@yuvrxj-afk Could you please resolve the conflicts?

@ank1traj would you please review?

@ank1traj would you please review?

Dropped some comments. Have a look.

@ank1traj would you please review?

Dropped some comments. Have a look.
Sentry ones?

Dropped some comments. Have a look.
Didn't get it. Sentry ones?

Dropped some comments. Have a look.
Didn't get it. Sentry ones?

First deployment is getting failed due to searchbar not found

Second if you see the review one. You will get my comment. Even if you scroll this PR you will get that.

@yuvrxj-afk what was the need of search bar? What it's providing here?

You mentioned the search bar in the navbar issue's description, so created the component, which can be used later with functionality.

Surely, i should make one more commit , we'll add it later with functionalities as we have to fetch as well.


Provide resistances and immunity information when viewing a Pokemon

When clicking on a Pokemon's icon to view more information about that Pokemon, only its type-weaknesses are shown. Ideally this screen should also show resistances and any immunities. This has been a feature enhancement requested by a few people already.
Current Pokemon info look-up screen

Notes on implementation

I suspect we can remove the extra filler information here, such as BST, Weight, and Evolution details. This information is already available on the previous main Tracker screen. One case for keeping the Evolution is when you want to look up a Pokemon that isn't currently on screen, such as trying to remember how a Clamperl evolves.
The icon of the Pokemon and its current types must remain on this info lookup and should not be removed
It's important that weaknesses and levelup move information remains on this first screen, as it should be easily accessible from a single click. It's also important that the notes section is available on this first screen. In other words, when a user clicks on the Pokemon icon to learn more about a Pokemon, they shouldn't then have to click on something else to see any of this information; it should have already been revealed to them.

what about as a rough idea for an initial implementation, arrows to cycle weaknesses/resists/immunities? Arrows either here or on either side of the text like the route encounter areas

To chime in on the filler info, i kinda like being able to look up the BST within this menu occasionally (rather than looking it up on an external site), not dying to keep it there but i think some people probably like being able to look it up in the tracker. Not sure how useful the weight info is though

Noting also that the highest number of resistances is water/steel at 12 in gen 3. You can only get that through randomized typing though.

Leaving a note that the data is will be available after the mGBA branch merge in (at some point in the future). Will still have the question on how best to show it all on the pokemon display page.

Resolved as of Tracker 7.3.0 release.


std::out_of_range

Freebayes version: v0.9.14-30-g6ac638c-dirty.
Example command line: freebayes -r chrEBV:0-171823 -f GCA_000001405.15_GRCh38_full_analysis_set.fna file.bam
Program exits with:
terminate called after throwing an instance of 'std::out_of_range'
what():  basic_string::substr
Aborted (core dumped)

Hello,
I'm getting this error today on v0.9.21 running discovery mode against 1kgp data:
freebayes -f /reference/genome.fa.gz  /shared/data/samples/NA12878/NA12878.chrom11.ILLUMINA.bwa.CEU.low_coverage.20121211.bam  > /tmp/na12878_chrom11.vcf &
The crash is immediate.
Was the root cause of this issue every understood? What was the fix?
Thanks,
Sergei.


CSP issue

Hi, using this module I'm getting a "Refused to execute inline script because it violates the following Content Security Policy directive [...]" in the browser console.
I can see that the whole script is wrapped using the $secureRenderer->renderTag method on line 25 in view/frontend/templates/scripts.phtml and that's ok
But I get the error on line 19 where "_document.body.append(speculationScript);".
Shouldn't any created script element have a nonce attribute attached to it to avoid any issue?
Thanks

What is your Magento version?
In my case, it's 2.4.7 and ->renderTag  -> Magento/Csp/Helper/InlineUtil.php added 'nonce' attribute by itself.
public function processTag(TagData $tagData): TagData
    {
        //Processing tag data
       ...

            //Adding required policies.
            ...

            if ($tagData->getContent()
                && !empty(self::$tagMeta[$tagData->getTag()]['hash'])
                && $this->isInlineDisabled(self::$tagMeta[$tagData->getTag()]['id'])
            ) {
                /** create new tagData with a nonce */
                if ($tagData->getTag() === 'script') {
                    $nonce = $this->nonceProvider->generateNonce();
                    $tagAttributes = $tagData->getAttributes();
                    $tagAttributes['nonce'] = $nonce;
                    $newTagData = new TagData(
                        $tagData->getTag(),
                        $tagAttributes,
                        $tagData->getContent(),
                        $tagData->isTextContent()
                    );

                    $tagData = $newTagData;

but it's only if isInlineDisabled("script-src") returns true
<csp>
           <policies>
               <storefront>
                   <scripts>
                       <inline>0</inline>
                   </scripts>
               </storefront>
           </policies>
       </csp>

My Magento version is 2.4.6-p6.
I get the nonce attribute added to that script and it's correct. But if that script creates a new script tag and appends it to the DOM, the new script tag needs a nonce itself or it will get flagged for CSP violation.
Hope this is more clear.

Thank you. I will research it
https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers/Content-Security-Policy/script-src#allowing_speculation_rules

I've made a quick test, changed  vendor/swissup/module-speculation-rules/view/frontend/templates/scripts.phtml
as following:

added
$cspNonceProvider = \Magento\Framework\App\ObjectManager::getInstance()->get('\Magento\Csp\Helper\CspNonceProvider');
$nonce = $cspNonceProvider->generateNonce();
on line 10;

Added
speculationScript.setAttribute("nonce", "$nonce");
on line 17

On checkout, the error I was seeing in my original post is gone.
Use of objectManager is not good practice but it was the fastest way to figure out.
You'll probably need to add the CspNonceProvider to the script.phtml block class and add the nonce attribute to the dynamically generated script tag.
Hope this helps


chore: upgrades

upgrade to mypy==1.11.* caused a failure. Seems to be resolved in the 1.12 dev release. for now excluding 1.11.* versions

use a the 0.0.999 sunset release of waylay-beta (instead of the 9.9.9b9 prerelease): dependency on prereleases breaks e.g. the uv-installer.

make freeze-deps-all

something wrong here: runtime dependencies now pop up in dev deps?

verified with pip

installing waylay-beta, then waylay-sdk-core downgrades waylay-beta to 0.0.999
upgrading afterwards waylay-beta with pip install --upgrade waylay-beta  raises a warning, but still installs it:

ERROR: pip's dependency resolver does not currently take into account all the packages that are installed. This behaviour is the source of the following dependency conflicts.
waylay-sdk-core 0.2.2.post3+git.bfae957b.dirty requires waylay-beta==0.0.999, but you have waylay-beta 0.7.2 which is incompatible.


podboat file naming

Newsboat version (copy from newsboat -v):
podboat 2.11.1
Steps to reproduce the issue:

standard podboat configuration

2.enqueue a podcast in newsboat (e.g. WTF with Marc Maron)

download file with podboat

Other info you think is relevant:
Some podcasts have poor naming conventions- rather than end with the file extension, they will often end with ?. I'm not sure why they're named this way, and it's not a podboat bug, however, it would be useful to add a feature which enables the user to name the files themselves when Downloading somehow (like a pattern  "-.mp3" if this feature doesn't already exist

Thanks for the report. Unfortunately, Podboat doesn't get much attention, so it's hard to tell when we'll get around to fixing this.


fix(wayland): fix compile error wrt LV_WAYLAND_CLIENT_SIDE_DECORATIONS

attach_decoration is called by create_decoration and uses defines that are reserved for decorations

Hi Thomas,
Thank you for the fix! You might have seen that in LVGL v9 we have added the drivers directly to the LVGL repo. However the Wayland driver is not ported to v9 yet.
Maybe do you have time and interest to contribute with that? :slightly_smiling_face:


Slow by default

In the code we have a button that lets us go slow rather than the standard fast. Steven wants the button to let you go fast, and the standard procedure to be slow.

In Drive.cpp deleted the " * speedpercent " in the button-setting speed section (lines 143-147) and moved it to the standard driving section of code (lines 99-110). Nate is testing.

TESTED AND FIXED!!!! :laughing: :rainbow: In branch SwitchStandardFastToSlow.

(That last one means it works if that wasn't clear)


Problem with Github Copilot

I want to use both LuaSnip and Copilot suggestions at the same time, but the snippets didn't show up when i enable Copilot, until I disable it again, snippets came back. And yup, i changed the key bindings to ensure that they aren't using a same key
Any ideas to solve this?

Do you use snippets and copilot via nvim-cmp? If so, I don't think luasnip can do anything about it..
AFAIK it's possible to change source-ordering (and maybe even completely disable some source if another has items), but I don't really know how that works.
You might have more success asking in eg. the neovim matrix room or in cmps issues/discussions.

thanks


Remove unused provider queries

These are not used in the codebase
Summary
Provide a brief overview of the changes and the issue being addressed.
Explain the rationale and any background necessary for understanding the changes.
List dependencies required by this change, if any.
Fixes #(related issue)
Change Type
Mark the type of change your PR introduces:

[ ] Bug fix (resolves an issue without affecting existing features)
[ ] Feature (adds new functionality without breaking changes)
[ ] Breaking change (may impact existing functionalities or require documentation updates)
[ ] Documentation (updates or additions to documentation)
[x] Refactoring or test improvements (no bug fixes or new functionality)

Testing
Outline how the changes were tested, including steps to reproduce and any relevant configurations.
Attach screenshots if helpful.
Review Checklist:

[x] Reviewed my own code for quality and clarity.
[ ] Added comments to complex or tricky code sections.
[ ] Updated any affected documentation.
[ ] Included tests that validate the fix or feature.
[ ] Checked that related changes are merged.

coverage: 46.787% (+0.01%) from 46.774%
when pulling 4219476f7b662cf358d766cd54605eee5d16d97c on delete-unused-provider-queries
into c6961d9c120e01c1cddd6388eab410421f637d8a on main.


Update README.md

Updated LastItemUserModifiedDate from LastItemModifiedDate as the latter is not an accurate representation of user activity on the lists. The "LastItemUserModifiedDate" represents what is actually reported on the SharePoint web UX and consequently more accurate in what would be expected in this report.

Hey @getalex , thank you 👏 for your contribution - it is highly appreciated. This should be live shortly 😊
Have you registered for the PnP Recognition program, your submission will qualify for a badge :-)
Its opt in so please register here: https://pnp.github.io/recognitionprogram

Hey @getalex , thank you 👏 for your contribution - it is highly appreciated. This should be live shortly 😊
Have you registered for the PnP Recognition program, your submission will qualify for a badge :-) Its opt in so please register here: https://pnp.github.io/recognitionprogram

Thank you for the kind words - I just registered, wonder if there's anyone to reach out to or would it retroactively issue a badge? would be a cool item to have for sure.

Typically badges are done by David Warner, in batches, so you would be in the next run, when he runs one. But absolutely you will get a badge.


rqt_robot_monitor: 1.0.2-1 in 'eloquent/distribution.yaml' [bloom]

Increasing version of package(s) in repository rqt_robot_monitor to 1.0.2-1:

upstream repository: https://github.com/ros-visualization/rqt_robot_monitor.git
release repository: https://github.com/ros2-gbp/rqt_robot_monitor-release.git
distro file: eloquent/distribution.yaml
bloom version: 0.9.7
previous version for package: 1.0.1-1

rqt_robot_monitor
* fix for changed data type of diagnostic level (#26 <https://github.com/ros-visualization/rqt_robot_monitor/issues/26>)

Holding for the upcoming Eloquent sync.


[TODO] Re-evalulate the current state of apotheosis loot tables

Also see about introducing affix loot pools into standard pools (chests/simple_dungeon, etc)

I have a question, How do I put loot from another mods in the aphoteosis chests?. I'd like to add loot from electroblob's wizardry and its addons in the chests under the apothosis spawners but I don't know how to do that

This should be handled on discord or a new issue.


Not working on macOS Monterey

Hi,
I've updated the MacOS for Monterey and now it's not working.
Both accessibility and Screen Recording are enabled for S3D and nothing happens for some reason, at least for me.
I tried many times to run from terminal and runs without issues, naturally given the proper system permissions.
It's been some time since S3D is not updated, so maybe this is an S3D issue, but not sure.
Anyone else faced the same issue?
Many Thanks!

Any time I have encountered similar problem, it was related to Simplify3D config.
what is exact text from "Additional terminal commands for post processing"?
what do the debugging files in the INSTALL_DIR reveal?

Cannot reproduce


This is an example why I hate MSDN

20 pages of water and no even single example how to write and use.

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

ID: a4ffec9b-27ef-0f8c-5fcf-58347950feaf
Version Independent ID: 8dadd327-8781-e1ef-e2b3-f75f416524e6
Content: Materialized View pattern - Cloud Design Patterns
Content Source: docs/patterns/materialized-view.md
Service: architecture-center
Sub-service: cloud-fundamentals
GitHub Login: @dragon119
Microsoft Alias: pnp

Hi @Lbaralageen Thank you for your feedback! We will review and update as appropriate.

@Lbaralageen, this pattern doesn't require 'code'. It just pre-populate a table for a complex query to make it faster.

Hi @Lbaralageen We will now proceed to close this thread for now. If there are further questions regarding this matter, please reopen it and we will gladly continue the discussion.


Running the BackgroundService of Widget every minute

Hello,
I have done some changes with your superb widget. I only got stuck at updating the widget even though my phone is not using the App or in off condition.
How do I update BackgroundTask or similar service every minute or so? Can you hint out at something and I will implement at my end. I don't want to take your much time.
Thanks

I already asked and answered a very much related question in stackoverflow 👇
https://stackoverflow.com/questions/39946431/what-is-the-best-way-to-schedule-a-task-in-react-native
I'm afraid this is the best I can do


CoPilot is not working as expected

Copilot does not return the correct information when users query

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

ID: fdc86775-f71a-5828-52dd-997d9adf6a6f
Version Independent ID: fdc86775-f71a-5828-52dd-997d9adf6a6f
Content: AI Copilot overview - Power Apps
Content Source: powerapps-docs/maker/canvas-apps/ai-overview.md
Service: powerapps
Sub-service: canvas-maker
GitHub Login: @mduelae
Microsoft Alias: tapanm

Thanks for your comment, @Veda1232. This issue has been assigned to the doc author for review.


One pool to rule them all!

An attempt to move the pooled-writer to using a single thread pool instead of one for compressing and one for writing.
Tests pass, including the long prop-test.  However, some cleanup is needed and likely this should be benchmarked against the prior version using real code.
@sstadick Not sure if you're interested in looking at this, but figured you might be interested in how the single-pool solution works.

Agreed on the wait-loop.  Looking at the prior code, I think the writer threads were often in a similar wait loop since they iterated over all the writer_rx channels doing try_recv, but the compressors were better about just blocking to receive.  I couldn't find a way around that that didn't introduce deadlocks though!


api should return not only active findings for product

Hello Team,
we want to integrate DefectDojo with an internal system via API. That system needs to know all the findings of a product: active, not active, false positive & so on. Can you change the behaviour of the Product API https://cr.rete.poste/api/v2/products/{product_id}/ so that it will be able to return all the findings's id ?
Best,
Sal
Hello Team,
if I send a request to https://cr.rete.poste/api/v2/products/{product_id}/, I get only active findings in findings_list, is it correct ?
Best,
Sal

Please come to slack to discuss these kinds of questions https://owasp-slack.herokuapp.com/ #defectdojo channel

I have done another test (see image below)

If I invoke product API with product_id 245 I get this response (see abstract below)
{
"id": 245,
"findings_count": 1,
"findings_list": [
3897,
3898,
3900,
3901
],
"tags": [],
"product_meta": [],
"name": "TEST_PRODUCT_1",
"description": "TEST_PRODUCT_1",
As you can see there is't finding set as a false positive which have 3899 id

Yes, that's what confirmed above. It only returns active findings.

would it be possible to change this behavior returning all findings with their status?

I suggest to create a new github issue or update this one with a clear description of what you want to be implemented.

Hello Team,
we want to integrate DefectDojo with an internal system via API that needs to know all the findings of a product (active, not active, false positive & so on). Can you chage the behavior of the API https://cr.rete.poste/api/v2/products/{product_id}/ so that IDs of all findings are returned ?
Best,
Sal

I updated the title and description for you

In case someone is wondering how to work around this issue:
class DefectDojoAPI:
    def __init__(self):
        self.url = os.getenv('DD_URL')
        self._headers = {'Authorization': 'Token ' + os.getenv('DD_API_KEY'),
                         'Content-type': 'application/json'}
        self._ssl_verification = os.getenv('DD_SSL_VERIFY', 'true').lower() == 'true'

        retries = requests.adapters.Retry(total=5, backoff_factor=0.1, status_forcelist=[500, 502, 503, 504])
        self._session.mount('http://', requests.adapters.HTTPAdapter(max_retries=retries))
        self._session.mount('https://', requests.adapters.HTTPAdapter(max_retries=retries))

        self._product_url = self.url + '/api/v2/products/'
        self._dojo_group_url = self.url + '/api/v2/dojo_groups/'
        self._test_url = self.url + '/api/v2/tests/'
        self._engagement_url = self.url + '/api/v2/engagements/'
        self._finding_url = self.url + '/api/v2/findings/'
        self._notes_url = self.url + '/api/v2/notes/'
        self._users_url = self.url + '/api/v2/users/'

        self._all_products = []

    def _get_paginated_items(self, url, url_params=None):
        items = []
        while url:
            # we explicitly pass the url_params only for the first page since next page url already contains them
            r = self._session.get(url, headers=self._headers, params=url_params if not items else None, verify=self._ssl_verification)
            r.raise_for_status()
            response = json.loads(r.text)
            items.extend(response['results'])
            url = response['next']
        return items

    def prefetch_products_in_scope(self):
        self._all_products = self._get_paginated_items(self._product_url, {
            'not_tags': ['divize-security'],
            'prefetch': ['authorization_groups', 'members', 'prod_type']
        })
        # workaround since findings_list only contains active findings
        for product in self._all_products:
            product['_findings'] = self._get_paginated_items(self._finding_url, {
                    'test__engagement__product': product['id'],
                    'prefetch': ['found_by', 'mitigated_by', 'notes', 'reporter', 'review_requested_by', 'reviewers', 'risk_acceptance_set'],
                    'related_fields': 'true',
                    'o': 'last_status_update'
                })


Potential accessibility problem with purple visited links

What

I have recently conducted research with cognitively impaired users of Assistive Technology on our service which was built using the Gov.uk design system. Our service, which is in Public Beta, can be found at: https://www.support-with-employee-health-and-disability.dwp.gov.uk/support-with-employee-health-and-disability

Why

Cognitively impaired participants found it hard to distinguish purple visited links from the standard black running copy, so could not easily pick them out. They asked if a different colour might be used for visited links.

Anything else

I have attached a slide from my report covering this potential issue. Could someone please let me know if there is any leeway in the Gov.uk guidelines which would enable our team to fix the issue for our users?

I've asked on https://github.com/alphagov/govuk-design-system-backlog/issues/64 whether other services have encountered the same issue, which should help us corroborate this finding.

Thank you, @stevenjmesser.

Hi @JonathanCulling I'm doing some research into this issue and would be very interested to hear if, or how, you took this forward?


useQuery initialData + staleTime should be respected if sibling component uses same queryKey and enabled false

Describe the bug
When useQuery is called multiple times with the same queryKey, the behavior of enabled, initialData, and staleTime seem off.
useQuery({ queryKey, queryFn, enabled: false });
useQuery({ queryKey, queryFn, initialData: {}, staleTime: Infinity });

I expect the above code to not trigger queryFn because the first call is disabled, and the second has infinitely cached initialData. But queryFn ends up actually getting triggered
Your minimal, reproducible example
https://codesandbox.io/p/sandbox/sparkling-lake-4zgjk5
Steps to reproduce
See codesandbox link
Expected behavior
I expect query is running! to not print to console, but it does print to console
How often does this bug happen?
Always
Screenshots or Videos
No response
Platform

OS: Mac Sonoma 14.7
Browser: Chrome 131.0.6778.86

Tanstack Query adapter
react-query
TanStack Query version
v5.62.0
TypeScript version
No response
Additional context
If the calls happen in reverse order, then everything works as I'd expect. This does not trigger queryFn:
useQuery({ queryKey, queryFn, initialData: {}, staleTime: Infinity });
useQuery({ queryKey, queryFn, enabled: false });

initialData runs when the query is created in the query cache. In the first order of calls, the query is created without initial data. When the second observer then gets added, it picks up the already existing cache entry, which doesn’t have any data. That’s why initialData “doesn’t do anything” here anymore, and the query runs because it has no data yet.
In the second order, initialData is written to the cache (because it’s passed on the query creation), so staleTime will see this as valid data and not trigger a fetch.
All in all, I’d say the current behaviour is expected.

Ok, thanks @TkDodo
I cannot easily reorder the existing calls. So my workaround is to add a 3rd useless useQuery to hydrate the cache with initialData before the existing calls. But that is awkward.
Is there anyway to not create that first cache entry? I can't conditionally call a hook, and it sounds like you're saying enabled:false is expected to create an entry.


Add r escape character

Fixes https://github.com/plyara/plyara/issues/123

Much needed fix, kindly approve

Looks good. Sorry for the long wait!


Losing Control After Take-Off

Hello, when we command from PC and press spacebar to take-off we camera visual freezes and program not responds. How can we solve that ? Thanks in advance!

Cna you please provide any logs? Looks like smth with recognition module itselt


Convert more easy modules to ES6 modules

Some initial results from starting to teach modzulator (#10018) how to modify node_tests.

Merged, thanks @andersk!


[QUESTION] How to call AppendKV when ColMap is wrapped in an array

Hello!
Thank you for the wonderful library.
Could you please help me? I want to call the method AppendKV for ColMap that is wrapped inside a ColArr. My goal is to set a specific order of keys in each of these maps.
Here is an example of what I'm trying to achieve:
eventsAttr := proto.NewArray(
  proto.NewMap[string, string](
    new(proto.ColStr).LowCardinality(),
    new(proto.ColStr),
  ),
)

eventsAttrArr := make([][]proto.KV[string, string], 0, len(events))
eventsAttrArr = append(eventsAttrArr, []proto.KV[string, string]{
  {Key: "one", Value: "specific"},
  {Key: "two", Value: "order"},
})

eventsAttrArr = append(eventsAttrArr, []proto.KV[string, string]{
  {Key: "one", Value: "totally"},
  {Key: "two", Value: "specific"},
  {Key: "three", Value: "order"},
})

// somehow add all eventsAttrArr to eventsAttr (?)

Could you provide an example of how this can be done?
Thank you in advance for your assistance!
P.S.
Is it maybe possible to achieve this using *proto.ColArr[[]proto.KV]? Will it serialize correctly to Array(Map(LowCardinality(String), String))?

Used something like this is
         eventsAttr := proto.NewArray(
		proto.NewMap[string, string](
			new(proto.ColStr).LowCardinality(),
			new(proto.ColStr),
		),
	)
	for _, event := range span.Events {
		attrs := make([]proto.KV[string, string], 0, len(event.Attributes))
		for _, kv := range event.Attributes {
			attrs = append(attrs, proto.KV[string, string]{Key: string(kv.Key), Value: value})
		}
		eventsAttr.Data.(*proto.ColMap[string, string]).AppendKV(attrs)
	}
	eventsAttr.Offsets = append(b.eventsAttr.Offsets, uint64(b.eventsAttr.Data.Rows()))

Wouldn't it be convenient for Array to contain methods AddElement and NextRow?


[TEST] Update Single replica node down test case

What's the test to develop? Please describe
After 1.2.5 & 1.2.6 release test  Single replica node down, I need to update single-replica-node-down test case and make it clear enough
Describe the items of the test development (DoD, definition of done) you'd like

[ ] update case. Get it reviewed.

Additional context
https://longhorn.github.io/longhorn-tests/manual/pre-release/ha/single-replica-node-down/

Node restart/down scenario with Pod Deletion Policy When Node is Down set to default value do-nothing.
The workload pod will get stuck in the terminating state.


Specify download folder?

Many options, complex ones....  but no option to set the downloads folder?

There is option to set downloads folder: link

OK it sounded like it was only the folder name created as part of the process, not the whole path.
I solved this by setting the "start in" properties menu on windows, to my desired download location.


Releases on GitHub not up to date

latest ist 1.3.0, but on PyPI it's 1.3.2

Hi @kabouzeid,
Thank you for your feedback.
This is expected, but due to some mismatch in our python wheels delivering process.

Our last code version was 1.3.0, but we had some issues with our python wheels, so we generated two new minor versions on PyPI (that explains the jump to version 1.3.2).

Our pipeline to generate our python wheels has been improved and we don't expect this issue to happen any longer.

You can find our new release here https://github.com/facebookresearch/projectaria_tools/releases/tag/1.3.3 and see that we have a match for the tag and PyPI version https://pypi.org/project/projectaria-tools/1.3.3/


✅ Improve internal validators test coverage

Change Summary
pydantic/_internal/_validators.py is covered as much is possible with exception of #1569
Related issue number
None so far
Checklist

[x] Unit tests for the changes exist
[x] Tests pass on CI and coverage remains at 100%
[x] Documentation reflects the changes where applicable
[ ] changes/<pull request or issue id>-<github username>.md file added describing change
(see changes/README.md for details)
[x] My PR is ready to review, please add a comment including the phrase "please review" to assign reviewers

please review

please review


Module not found error when importing umfpack MacOS Monterey M1 chip

I installed umfpack using pip install scikit-umfpack and tried importing the umfpack module

from scikits import umfpack

I'm running into the following error:
Traceback (most recent call last):
  File "/Users/christianadib/pyfrbus_package/venv/lib/python3.9/site-packages/scikits/umfpack/_umfpack.py", line 18, in swig_import_helper
    fp, pathname, description = imp.find_module('__umfpack', [dirname(__file__)])
  File "/Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/imp.py", line 296, in find_module
    raise ImportError(_ERR_MSG.format(name), name=name)
ImportError: No module named '__umfpack'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/Users/christianadib/pyfrbus_package/venv/lib/python3.9/site-packages/scikits/umfpack/__init__.py", line 12, in <module>
    from .umfpack import *
  File "/Users/christianadib/pyfrbus_package/venv/lib/python3.9/site-packages/scikits/umfpack/umfpack.py", line 130, in <module>
    from . import _umfpack as _um
  File "/Users/christianadib/pyfrbus_package/venv/lib/python3.9/site-packages/scikits/umfpack/_umfpack.py", line 28, in <module>
    __umfpack = swig_import_helper()
  File "/Users/christianadib/pyfrbus_package/venv/lib/python3.9/site-packages/scikits/umfpack/_umfpack.py", line 20, in swig_import_helper
    import __umfpack
ModuleNotFoundError: No module named '__umfpack'

The readme states clearly that you need to install suitesparse before this library. Here is what worked for me:

Install homebrew
brew install swig suite-sparse
Create a config file with the location of the files homebrew just installed. It is a .ini file, the name doesn't matter; I used custom-config.ini:

[properties]
umfpack-libdir = '/opt/homebrew/Cellar/suite-sparse/7.6.0/lib/'
umfpack-includedir = '/opt/homebrew/Cellar/suite-sparse/7.6.0/include/suitesparse/'

This assumes you install version 7.6.0.

Install scikit-umfpack and tell it where the config file is:

pip install scikit-umfpack -Csetup-args=--native-file=$PWD/custom-config.ini

This assumes that the custom-config.ini file is in your current working directory.


Bonus event should be from "GoodDollar" and not unknown

happens in two cases I think

we wait until bonus is done on backend, then we add the TX to the feed, so the Transfer events maybe arrives before we enqeueuTX to feed so its not updated correctly.
If bonus was redeemed directly on blockchain not through wallet.
ie in console call api.reedemBonus() so wallet doesn't call .enqueTX, the bonus tx will be added to the wallet (since we listen to blockchain events) but the from is "Unknown"
modify handling of bonus receipt/event to set from="GoodDollar"

The easiet solution I think is to identify Transfer from the SignupBonus contract address and mark them as bonus. So no need for enqueueTX.
Definition of Done

[x] if bonus is redeemed not via dashboard then bonus tx in feed says "GoodDollar" in the from field
[x] bonus feed card should have the correct icon
[x] remove enqeueuTX from bonus claim process

The link to GoodServer PR - https://github.com/GoodDollar/GoodServer/pull/120

Checked on QA Env
Look as per requirements.
User receive the bonuses from Goodgollar.


Create tool for downloading game data

GameData\reports\blocks.json

Any more information on this?
This is pretty vague, even if I think it can be reproduced easily

@Geolykt There is no GameData folder

Things such as reproduction steps are better for the future (I haven't run this, so idk how easily this can reproduced, but I'd just remark this in case it's as that easy)

Silly me, I forgot to add instructions for required resources.
You can get blocks.json here

This issue will probably remain open until I create some data report generator.

@TechnologicalPizza Thanks

@TechnologicalPizza  I have another question... Where this directory "GameData\reports\blocks.json" should be created ?

The server expects that to be in it's working directory. Looking at ServerMain.cs should make it clear.


Migration generates primary key column as "jsonb" when using typedef

Describe the bug
If we have:
export type Id = number;

@Entity()
export class User {
  @PrimaryKey()
  id: Id;

  @Property()
  name: string;
}

instead of:
@Entity()
export class {
  @PrimaryKey()
  id: number;

  @Property()
  name: string;
}

the column generated in the migration will be "id" jsonb not null instead of "id" serial primary key.
To Reproduce
Steps to reproduce the behavior:

Create a couple files with the following code:

// id.ts
export type Id = number;

// user.ts
@Entity()
export class User {
  @PrimaryKey()
  id: Id;

  @Property()
  name: string;
}

Generate migration: npx mikro-orm migration:create --initial
Go to the generated Migration file, and check the id column type. It will state "id" jsonb not null.

Expected behavior
The migration should interpret the type of the property as a number.
Versions

Dependency
Version

node
16.9.1

typescript
4.3.5

mikro-orm
5.1.3

your-driver
postgresql

How does your ORM config look like?

Not that I know of. This is my config:
export default {
  type: 'postgresql',
  host: process.env.DB_HOST,
  port: parseInt(process.env.DB_PORT),
  password: process.env.DB_PASS,
  user: process.env.DB_USERNAME,
  dbName: process.env.DB_NAME,
  entitiesTs: ['src/**/*.model.ts'],
  entities: ['dist/**/*.model.js'],
  migrations: {
    path: './src/db/migrations',
  },
}

That's weird, I can't reproduce this with reflect metadata 🤷
Please provide executable repro

I'm importing 'reflect-metadata' too :thinking:
I managed to reproduce it in a reduced repo: https://github.com/javiermrz/mikroorm-bug-report/blob/main/docker-compose.yml
Just run:
npm i
docker-compose up -d
npx mikro-orm migration:create --initial
You should see the generated migration file in src/db/migrations

Interesting, it fails because the type is defined in another file. Unfortunately, this is a won't-fix, as it's the reflect-metadata package giving us wrong type (object). If you define the Id type in the same file, it works fine, but apparenly types defined elsewhere won't be resolved.


person_detection and face_mask_detection

Hello, may I ask what are the hardware devices and models of the demo OpenMV Cam H7?
I want to repeat that.
Thank you very much!!

Hi @Super-Karl,
Feel free to try our example demos on OpenMV Cam H7:
https://github.com/mit-han-lab/tinyengine/tree/main/examples/openmv_person_detection
https://github.com/mit-han-lab/tinyengine/tree/main/examples/openmv_vww
You can find the models here with the tflite format: https://github.com/mit-han-lab/tinyengine/tree/main/assets

Hi @meenchen ,
Thank you for your reply, I want to ask about the specific model of the development board and the model of the display screen that is required to implement the above demo.
I'm going to buy these hardware resources now, and I see that I may still need lithium batteries?

Sure, please check out the following links:
https://openmv.io/products/openmv-cam-h7


An issue within the readme example

Hi,
Thanks for doing so much of the heavy lifting in terms of providing a solution that allows features like cloud-init to be useful.
One thing you have the following example in your readme.md
docker build -t iso-generator . &&
docker run -it --mount type=bind,source="$(pwd)",target=/app iso-generator
ubuntu-autoinstall-generator.sh -a -u user-data.example -n jammy
The version of ubuntu-autoinstall-generator.sh over at covertsh/ubuntu-autoinstall-generator does not have an -n option.
Currently, it is not clear if you have a typo in your example or maybe an extended version of the script.
Thanks again

tanks for pointing that out - I added the -n option to my branch to allow you to specifiy an ubuntu code-name (like bionic, focal or jammy)
Here's the function that variable is used with:

Thanks for the quick update to the examples. As a side note your example user-data.basic seems to be a very personal example.

Yeah that's an accurate description, I wanted to add something that contained more variety of modules used than many other cloud init tutorials and ended up using a mildly sanitized version of my own homelab's config.
I'm not really sure how best to approach providing examples for this project since there's so many possibilities. I'm definitely open to feedback and suggestions in that regard.
There is a companion repo to this one that focuses more on cloud-init and has some more examples, as well as some private templates I could clean up and provide if that would be helpful.
You can see some of that work here https://github.com/cloudymax/Scrap-Metal/blob/main/virtual-machines/user-data.sh though it is not documented enough for a release at this time.
My other thought was to simply link to a few other projects and blogs like this one from Puget systems    that also cover the topic as an attempt to lessen my own responsibility for maintaining documentation.

OK, I think adding much of your reply into the readme.md would provide enough background for most people. I am guessing that anyone who finds your repo is looking for a starting point that they can then build on.
As it is a personal example I think the only changes I would suggest are
- document the password you have set
- remove the font change.

Those 2 changes would at least allow someone to build an iso with your example and then log in to it. The font change can become 'interesting' depending on the test environment - with your settings I end up with a 30x20 screen :(


Remove second author from event-015

The second author from this event has been in touch to say they will no longer be involved and asking to be removed from the web pages.
This is ready to merge.

hey @cc-a! thanks for this! Can we make this from a branch within this repository? Sorry, but we need to regenerate the PDF and this is not yet possible for forks (see #347)


Add support for Bedrock Custom Import Models

Now the custom import model is GAed, we need to evaluate the cost performance using FMBench with customized model arn.

Thanks for this idea! We will be working on it and keep you posted. To benchmark a custom model, we would simply have to write a predictor (https://aws-samples.github.io/foundation-model-benchmarking-tool/byo_rest_predictor.html) that supports the prediction format for custom model imports (nothing too complex, the function would be similar to what is given in https://aws.amazon.com/cn/blogs/machine-learning/amazon-bedrock-custom-model-import-now-generally-available/, using the converse API.


Release version 0.1.2

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


⚠ MorethanTV has degraded performance

In 7985497, MorethanTV ($MTV) experienced degraded performance:

HTTP code: 502
Response time: 1583 ms

Resolved: MorethanTV performance has improved in 4c3d3ae.


docs.flutter.io missing open search description (doesn't tab-search in chrome)

https://www.chromium.org/tab-to-search
Maybe this is a dartdoc bug?  Donno.  @sethladd

This is likely an easy fix?

Issue moved to flutter/website #1823 via ZenHub


Low resolution video

The videos appear to be low res and don't look nearly as good as they do on my AppleTV/TV combo at home

https://drive.google.com/drive/folders/0B50igKAfYyPCcDdQV25yc21sQzQ
These are the hi-res versions of the videos. Download them and place them in the Aerial cache folder

The videos that John's screensaver plays on your Mac are exactly the same videos from Apple's servers that play on your Apple TV.
The videos linked above aren't higher resolution. Someone just took Apple's videos and ran a sharpness filter over them. As Cristian, the guy who created them, says "This is by no means a resolution increase and they still look blurry, but the improvements are noticeable."

Also, see that the result of the discussion on Issue #257 (the one where Cristian provides the link to the enhanced videos) is that the improvement will be done programatically via OpenGL/Metal filtering, so that battery impact is small, and no additional downloads are required.


Release 0.1.18

XRTK - Mixed Reality Toolkit Change Request
Overview
Update to 0.1.18 to deploy UPM packaging issue and related fixes
Target of the change:
Is this enhancement for:

Core (core framework, interfaces and definitions)

Hold releasing for Controller mapping bugfix


Justify context-free search

Restricting search to regular grammar is a performance optimization. Recursion in functions was also once considered a controversial performance compromise. Times change.
https://vanemden.wordpress.com/2014/06/18/how-recursion-got-into-programming-a-comedy-of-errors-3/

https://stackoverflow.com/a/25841718/187223

Regular Expression Matching Can Be Simple And Fast

https://stackoverflow.com/questions/57667534/how-can-i-use-a-parser-in-haskell-to-find-the-locations-of-some-substrings-in-a#comment101804063_57667534

https://stackoverflow.com/a/4161173/187223


Specify environment variable separator

Motivation
I was attempting to deploy Canvas Data Loader in a Docker container that uses environment variables for conv. The default separator in config-rs for environment variables is _. This causes problems with hierarchical configuration. For example, cdl_canvasdataauth_api_secret is parsed as canvasdataauth.api.secret.
I've specified the separator to be __ such that cdl__canvasdataauth__api_secret parses as canvasdataauth.api_secret
Test Plan

Confirm that environment variables are parsed properly.
Confirm that there is no impact on default/local file configuration.

Next Steps
cargo test passes (but I didn't add new tests, so that's not surprising)

 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.

Have updated per review feedback. I'm not sure if it's of any interest, but I also have a Dockerfile that I could contribute.


Add preserve stop times flag to avoid normalizing stop_sequence

Checklist

[x] Appropriate branch selected (all PRs must first be merged to dev before they can be merged to master)
[x] Any modified or new methods or classes have helpful JavaDoc and code is thoroughly commented
[x] The description lists all applicable issues this PR seeks to resolve
[x] The description lists any configuration setting(s) that differ from the default settings
[x] All tests and CI builds passing

Description
This PR adds a FeedSource#preserveStopTimes flag to avoid the stop_sequence normalization that typically happens when making a snapshot for the editor.
re conveyal/gtfs-lib#283.
Note: needs pom.xml updated to<EMAIL_ADDRESS>in order for build to pass (conveyal/gtfs-lib#285 needs to be merged and released first).
Note: there is no corresponding datatools-ui PR to set this flag (primarily just so that users don't "try this at home"). It must be done with a MongoDB update statement like:
db.getCollection('FeedSource').update({"_id": "YOUR_TEST_FEED_ID"}, {"preserveStopTimes": true})

OK, just waiting on the gtfs-lib release. I'll merge this once I've updated the pom.

:tada: This PR is included in version 3.7.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Problem with local desktop PR

warn: No match for commit 84690c850e986d19789c415009e5ce3177dfa16d found at https://github.com/twindle-co/twindle.git
warn: Are you sure you pushed 'HEAD' there?
Hİ, when i try to do my second PR i get these errors i've checked on stackoverflow and they say i have to push first. But when i try to push permission is denied remotely. Can anyone help ?

As mentioned in the Documentation, you needed to do sync your forked repo first with the main repo before making your local changes, like:
git remote add upstream https://github.com/twindle-co/twindle.git
git fetch upstream
git merge upstream/main
git push origin main

Then make your local changes and then do git add, commit and then finally do git push.

I'm facing the same problem. I synced the local with upstream, than made my changes, but same problem goes.

I cannot create a pull request from commit f9f8bc7408841a3669b415e360f72ead11f8b918 to upstream

Hey, can you pls check whether ur local and remote repo is in sync via this command  git status -uno  .

Hey, can you pls check whether ur local and remote repo is in sync via this command git status -uno .

It prints
Your branch is up to date with 'origin/main'

Nice. Then try to create a pull request via a web interface(If you had made any new changes). Are you still getting an error while doing that? If so pls let me know the error message

Nice. Then try to create a pull request via a web interface(If you had made any new changes). Are you still getting an error while doing that? If so pls let me know the error message

I did successfully by using the github's website, but I'm wondering why it is not working from terminal. We'll see in future PR's, how will it go. @Rakesh-4 Thank you anyway

Nice. Post me that error let me have a look.

I'm facing the same problem. I synced the local with upstream, than made my changes, but same problem goes.

I cannot create a pull request from commit f9f8bc7 to upstream

That was the error I was getting from terminal, but none occurred using GitHub's website

I was having this error, but none occurred in GitHub's website

As of February 2020, you can now use GitHub CLI and then easily Create a pull request from command line like:

It's not an error just a warning saying the changes you made isn't for remote server as it won't be available for others to pull your changes so you need to push you changes to the remote server first and the warning will go away.

Thanks guys, I'll try a dummy PR to solve this problem, as soon as I can.

@johnjacobkenny can this be closed now since it is fixed? 😁

Sure, please close if it's fixed!

I can close it hehe.


Add NUL bytes to the end of String allocations in MRB_API string FFI functions

This PR is a continuation in the same theme as #1969. #1969 was an incomplete fix for the buffer overread that only targeted strings produced from symbols. This PR targets all strings to ensure they have a trailing NUL byte.
The upstream version of mrb_str_new_capa always allocates capa + 1 bytes and sets the last byte of the allocation to zero.
Prior to this commit, Artichoke did not maintain this invariant. This results in the same style of buffer overread as in https://github.com/artichoke/artichoke/pull/1969 because mrb_str_dup is another plausible pathway to get an RSTRING_PTR from mrb_class_path:
https://github.com/artichoke/artichoke/blob/7bf3f033e020dd60f4b5fd5974dcb19af2617f6d/artichoke-backend/vendor/mruby/src/class.c#L2123-L2127
Commits in this PR ensure all String FFI routines route through mrb_str_new_capa to maintain this invariant, or otherwise ensure a trailing NUL byte is present similar to the mitigation in https://github.com/artichoke/artichoke/pull/1969.
This was observed as similar corruption/garbage bytes in class names in the spec suite:
For example, these subsequent runs as identified in https://github.com/artichoke/artichoke/pull/1969#issuecomment-1193400913:
Method#inspect: .XXXXXXX: "#<Method: MethodSpecs::MySub(MethodSpecs::MyMod)#bar>"
.XXXXXXX: "#<Method: MethodSpecs::MySub(MethodSpecs::MyMod)#bar>"
.XXXXXXX: "#<Method: MethodSpecs::MySub:\xB8\xF7\x7F(MethodSpecs::MyMod)#bar>"
X.XXXXXXX: "#<Method: MethodSpecs::MySub(MethodSpecs::MyMod)#bar>"
X.XXXXXXX: "#<Method: MethodSpecs::MySub(MethodSpecs::MyMod)#bar>"
.XXXXXXX: "#<Method: MethodSpecs::MySubW(MethodSpecs::MyMod)#bar>"

Method#inspect: .XXXXXXX: "#<Method: MethodSpecs::MySub(MethodSpecs::MyMod)#bar>"
.XXXXXXX: "#<Method: MethodSpecs::MySubmethods'-\xBC\xF7\x7F(MethodSpecs::MyMod)#bar>"
X.XXXXXXX: "#<Method: MethodSpecs::MySub(MethodSpecs::MyMod)#bar>"
.XXXXXXX: "#<Method: MethodSpecs::MySubn(MethodSpecs::MyMod)#bar>"
X.XXXXXXX: "#<Method: MethodSpecs::MySub(MethodSpecs::MyMod)#bar>"
X.XXXXXXX: "#<Method: MethodSpecs::MySub(MethodSpecs::MyMod)#bar>"

Additional fixes target mrb_str_new and mrb_str_new_cstr:
To maintain the invariant that all string byte buffers have a trailing NUL byte, Artichoke has added logic to mrb_str_new_capa to always allocate an extra byte and write a 0 to it.
To avoid repeating this error prone logic in multiple places, implement mrb_str_new in terms of mrb_str_new_capa and a raw memcpy, remembering to fixup the length in the RString*.
Implement mrb_str_new_cstr with mrb_str_new by finding the ptr and len with Rust's CStr.
Results and Testing
Compared to the latest trunk, all observed instances of this bug have been fixed, as measured by the Core spec suite.
trunk
$ cargo run --bin spec-runner -q -- all-core-specs.toml > trunk.txt
$ rg haystack trunk.txt
40646:ArgumentError: regex crate utf8 backend for Regexp only supports UTF-8 haystacks
40660:ArgumentError: regex crate utf8 backend for Regexp only supports UTF-8 haystacks
40674:ArgumentError: regex crate utf8 backend for Regexp only supports UTF-8 haystacks
40880:ArgumentError: regex crate utf8 backend for Regexp only supports UTF-8 haystacks
40894:ArgumentError: regex crate utf8 backend for Regexp only supports UTF-8 haystacks
40908:ArgumentError: regex crate utf8 backend for Regexp only supports UTF-8 haystacks
40922:ArgumentError: regex crate utf8 backend for Regexp only supports UTF-8 haystacks
40936:ArgumentError: regex crate utf8 backend for Regexp only supports UTF-8 haystacks
40950:ArgumentError: regex crate utf8 backend for Regexp only supports UTF-8 haystacks
41685:ArgumentError: regex crate utf8 backend for Regexp only supports UTF-8 haystacks
70970:ArgumentError: regex crate utf8 backend for Regexp only supports UTF-8 haystacks
71027:ArgumentError: regex crate utf8 backend for Regexp only supports UTF-8 haystacks
$

mem-unsafety branch
$ cargo run --bin spec-runner -q -- all-core-specs.toml > mem-unsafety.txt
$ rg haystack mem-unsafety.txt
$

now that the latest report has run, all of these in the full spec suite are fixed. The only one left is a real bug with a binary haystack in strscan:
$ rg 'ArgumentError: regex crate utf8 backend for Regexp only supports UTF-8 haystacks' reports/exceptions/latest.json
7194:    "StringScanner#getch is multi-byte character sensitive ERROR\nArgumentError: regex crate utf8 backend for Regexp only supports UTF-8 haystacks\n/artichoke/virtual_root/src/lib/strscan.rb:272:in match\n/artichoke/virtual_root/src/lib/strscan.rb:272:in scan_full\n/artichoke/virtual_root/src/lib/strscan.rb:264:in scan\n/artichoke/virtual_root/src/lib/strscan.rb:112:in getch\n/artichoke/virtual_root/src/lib/library/stringscanner/getch_spec.rb:19:in protect\n(eval):165:in all?\n(eval):590:in each\n/artichoke/virtual_root/src/lib/library/stringscanner/getch_spec.rb:35\n(eval):590:in each\n/artichoke/virtual_root/src/lib/spec_runner.rb:250:in run_specs\n(eval):1",


Input: Support chorded key-presses

Today our InputManager does not handle chorded key presses, like inputManager.bind("jj", () => { ... }).
It'd be great to be able to handle this, as it would be one less case where a user has to 'eject' from Oni's input configuration to go to their init.vim.
Some considerations:

We'd need a setting for how long to 'wait' for the binding to complete
We'd need to decide how to handle the case where both j and jj are bound - AFAIK, Vim actually waits for the time with a single press, unless it is cleared by another key press that is unmapped.

Isn’t chorded jk and jj repeated?

Sorry, bad example on my part - updated to reflect jk instead to make it clearer.

Can this be closed @bryphe?


[SDK-3090] Remote screenshotter service support

This is an implementation of a command similar to the matchRemoteImageSnapshot command from IAM-76 / https://github.com/Appboy/platform/pull/46841 - rewritten on a fork of a more actively maintained, ts-based, image comparison cypress plugin (with a comparison/baseline-updating UI!).  It's working for me locally integrated into platform.  I'm not able to commit my platform side yet due to some tsconfig issues that I hope running the CI workflow (which is in github) on this repo will allow me to repair.

look at this, it's even mad at me for hurting the test coverage! I love this repo!

i think i have to merge it for github to pick up the workflow revision, all but the step that requires codeclimate are green


timesync: synchronization verification

SMIP: https://github.com/spacemeshos/SMIPS/issues/55
Listen to broadcasted blocks and verify that the layer, according to local clock, matches a layer specified in a block.

[ ] define details after a discussion

This is not going to be implemented.


device_tracker consider_home only used from first device-tracker platform

Make sure you are running the latest version of Home Assistant before reporting an issue.
You should only file an issue if you found a bug. Feature and enhancement requests should go in the Feature Requests section of our community forum:
Home Assistant release (hass --version): 0.27 (and previous)
Python release (python3 --version): 3.4.2
Component/platform: device_tracker
Description of problem:
When multiple device_trackers are configured, the "consider_home" value is only used from the first device_tracker. If it isn't configured for this tracker, configuring it on another device_tracker has no effect.
Expected:
consider_home is being used or at least some warning is issued.
Problem-relevant configuration.yaml entries and steps to reproduce:
- platform: bluetooth_tracker
- platform: bluetooth_le_tracker
  track_new_devices: no
  interval_seconds: 480
  consider_home: 2000
  scan_duration: 15
- platform: locative

Additional info:
This seems to be a design issue of the device_tracker subsystem in general. The device tracker is initialized only once when initializing the first device_tracker platform.
I'm not sure what would be the best approach to fix this.

Duplicate of #1328 I believe.

It is a known issue, only the first platform is supported. I wanted to add a warning in #2908's first commit but the methods proposed in #1328
Which option would you prefer? I can work on a PR
Option 1:
device_tracker:
  consider_home: 80
  track_new_devices: True
  platforms:
    - platform: a
    - platform: b

Option 2:
Do it per platform.
In this method we'll have to save the original source platform in known_devices.yaml to be able to apply consider_home for the loaded devices (potentially a bigger change, but more flexibility)
CC: @home-assistant/developers

Just an opinion here but I'd love to see the consider_home option put down into the known_devices.yaml file so I can apply it on a device by device case.  Some sleep more than others, some don't sleep at all and with the option at the platform level, I have to choose based on the worst offender (iOS devices).
https://community.home-assistant.io/t/consider-home-entry-inside-known-device-yaml/3495

Did #3743 take care of this issue?

Appears like it, thanks for bumping to remind us to close!


fieldalignment analyses does not work

local golspcfg = {
    settings = {
	    gopls = {
	        analyses = {
                    fieldalignment = true,
    	        },
    	        staticcheck = true,
	    },
    },
}

require('go').setup({
    lsp_cfg = golspcfg,
    lsp_on_attach = function()
        require('lsp_signature').on_attach()
    end;
})

nvim does not show the warnings about struct field alignment.
When I use the same config, but directly with lspconfig, the warning is displayed.
Edit: also using lsp_on_attach causes the codelens not to be displayed.

Thanks. there was a bug in the code. The fix is pushed through. Could you take another look?

Thanks. Now it works correctly.


HIP32: Split DCs Among All Transferers

Author(s): @PapaOwl
Start Date: 2021-06-03
Category: Economic
Initial PR: https://github.com/helium/HIP/pull/194
Tracking Issue: this
Status: In Discussion
Discord channel: #hip-32-split-dcs on https://discord.gg/helium
Rendered view:
https://github.com/helium/HIP/blob/master/0032-split-dcs.md
Summary:
This proposal changes the DC reward mechanism. Instead of only going to the fastest responder, the DC reward is split among all hotspots that tried to perform this duty in a timely fashion.

Closing 6 months later


localhost - Site cannot be reached

I am seeing these issues:
npm WARN optional Skipping failed optional dependency /chokidar/fsevents:
npm WARN notsup Not compatible with your operating system or architecture<EMAIL_ADDRESS>Also another error at the end of the log file below when I tried to run npm start.
I am running Windows 10, 64bit.
node v6.3.1
npm  v3.10.3
git v2.9.2
npm_log.txt

logfile output attached.
Windows 10 64 bit
git version 2.8.1.windows.1
node v6.3.1
npm 3.10.3
kcdoutput.txt

Hi friends! Thank you for reporting this. Unfortunately, the instructions I had them send to you were wrong. Copy/paste error on my part. The bit about npm start and localhost:8080 were for my webpack workshop (today). If you were able to get to this point without error, then you're good.
As for the warnings you're seeing @cdelstad, those are probably fine. If your setup:fem command doesn't show any errors, then you're good :)

I don't see a Todo app - is that part of the webpack instructions as well?

Yes it is.. 😅 sorry about that!

In my case, setup:fem ran just fine without giving any errors. However, I can not open localhost:8080. It says 'Connection Refused'.

Yeah, sorry, you should have received an updated email indicating that the localhost bit was a mistake. You're good to go!


feat: enable findings by default

Closes https://github.com/getappmap/appmap-intellij-plugin/issues/148

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

v0.16.0
GitHub release

Your semantic-release bot :package::rocket:


Add Mercedes me component

The Mercedes me component offers integration with the Mercedes me cloud service and provides presence detection as well as sensors such as doors, tires, windows, and service interval.
This component provides the following platforms:

Binary Sensors - such as windows, tires, doors, lock
Sensors - such as fuel status, service interval, remaining km...
Device tracker - to track location of your car

Pull request in home-assistant.github.io with documentation (if applicable): home-assistant/home-assistant.github.io#4464
# Example configuration.yaml entry
mercedesme:
  username: email
  password: password

Checklist:

[x] The code change is tested and works locally.

If user exposed functionality or configuration variables are added/changed:

[x] Documentation added/updated in home-assistant.github.io

If the code communicates with devices, web services, or third-party tools:

[x] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[x] New dependencies have been added to the REQUIREMENTS variable (example).
[x] New dependencies are only imported inside functions that use them (example).
[x] New dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
[x] New files were added to .coveragerc.

Thanks for the contribution! 🌟

Oof. Thanks for the follow-up @MartinHjelmare. Sorry for the premature merge!

can you please check the issue reported here:
https://community.home-assistant.io/t/0-63-entity-registry-sql-sensor-mercedes-cars/42886/4?u=elradix


Implement StopAsync RPC for Aspire Server

We don't do anything with this RPC today.  It's possible we should, perhaps like aborting all inflight connections via cancellation and then waiting for them to complete before shutting ourselves down?
Alternately, we can just have the person running us terminate the process.
When we do this, we need to think about how to implement Close for our implementation of jsonrpc2.Stream.  Today it is a no-op but that also may not be correct.
We just need to think through with the VS folks what we want the shutdown behavior to be.  I don't think we need to engineer anything complex to start with - but it might be nice to not worry about the server being terminated rudely.

@ellismg @weikanglim Not in scope for VS 17.10 Preview 2. Moving it to March iteration. Feel free to pull it in, if need be.

@abpiskunov @anvillan is this required for VS 17.10 Preview 3?

I believe yes, it will be already 3rd preview and we need to wrap up all known feature gaps and leave only bug fixes for final previews before build.


Slider paint incorrectly when the track shape is rectangular

Description

The Slider paint incorrectly when the track shape is rectangular, it has a wrong offset.
There have a gap between the active track and inactive track, If you increase the height, you can clearly see the gap.

This gap was added intended, see #31564, but this process is different from RoundedRectSliderTrackShape, so I think we should remove it.
Related Issues
None.
Tests
I added the following tests:
See files.
Checklist
Before you create this PR, confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I signed the CLA.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I updated/added relevant documentation (doc comments with ///).
[x] All existing and new tests are passing.
[x] The analyzer (flutter analyze --flutter-repo) does not report any problems on my PR.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Did any tests fail when you ran them? Please read Handling breaking changes.

[x] No, no existing tests failed, so this is not a breaking change.
[ ] Yes, this is a breaking change. If not, delete the remainder of this section.

[ ] I wrote a design doc: https://flutter.dev/go/template Replace this with a link to your design doc's short link
[ ] I got input from the developer relations team, specifically from: Replace with the names of who gave advice
[ ] I wrote a migration guide: https://flutter.dev/go/breaking-changes-template Replace this with a link to a pull request that adds the migration guide to https://flutter.dev/docs/release/breaking-changes

@HansMuller Hi, please review this change at your convenience, thanks very much!

The RangeSlider also have a gap when the track shape is Rectangular, I fixed at #64627

CC @clocksmith @JoseAlba


.end reports client-side errors if they occur in the callback

Version 2.0.0
Assuming you have the code:
Request.get(/* location that returns with 2xx */)
  .end((error, response) => {
    if (error) {
      console.log(error)
      return
    }
    if(response.ok) {
      console.log('ok')
      nonExistantThing.go();
    }
  })

You get the following log messages after making a single request:
ok
ReferenceError: nonExistantThing is not defined(…)

When the error occurs in the callback, it's still in the second try/catch block in the Request function (client.js:504-517, specifically line 506). What happens is that it swallows the JS error, and re-reports it through the same callback function. When this happens, you lose any context for the error (Stack trace, line number, etc).
My testing indicates that moving the callback functions out of the try block fixes the issue, but I don't know if the callbacks are in the try block for a reason.

That was done for #985 I think we can shuffle things around a bit


Assigning a variable to itself when defining that variablee

Elm seems to not catch the issue in which a variable is defined using a self-asignment or a calculation that involves itself.
A modified version of the Hello World Example:
import Html exposing (text)

main =
  let x = x
  in text x

As you can see, this is a clear mistake, but the compiler doesn't seem to detect this. It's quite an annoying error as it's hard to debug such issues when finding in your code.
An even more rediculous example (based on the previous example) that compiles is:
 let x = y
      y = x
  in text y

It's the same kind of error, whereby you use a variable before it is defined.
Would be great if such code get's flagged as an error with a message such as "Error: Using variable 'x' before it is defined at Line ?"

I'm pretty certain this is already covered by other issues? Have you looked?

I looked with keywords such as variable and I only found stuff like variable-shadowing. This wasn't reported as far as I'm aware. I'm sorry if it was and you can mark it as duplicate in such case.

I was thinking about various issues with recursive definitions. The following one seems the most encompassing one at the moment: https://github.com/elm-lang/elm-compiler/issues/873. At its bottom it lists several related ones.
Maybe your example adds some new aspect, so instead of closing the issue right away, maybe you can first look through the referenced ones to see whether it is already covered.

Yeah, it's a known issue. Even x = x at the top level creates a value of type a, which you can then use as any type and wreak havoc. Thanks for reporting.

I'm working on a fix for this, since it's related to DeadCode elimination. Please close this issue, since it's covered by #873 .


Strandtest.py not driving LED's when X Server running - Pi 3 B+

I have successfully gotten the firmware to run on the ESP8266 and software on my desktop, but upon trying the Raspberry Pi version running Raspbian Buster, I noticed that when using Visualization.py, and Strandtest.py for that matter, while running the X server, the LED strip does not respond. No illumination whatsoever. I use the same circuit and triple checked it thinking at first that it had to be my doing. If I boot into CLI only, and when I originally tested it I used Raspbian Buster Light, so CLI only, it works perfectly and immediately. I did notice that when unplugging the data pin and plugging it back in quickly and repeatedly that the LED strip does somewhat react to the Strandtest.py output but its jittery and as soon as the connection is solid, whatever state the LED's were at for that split second, they remain, until I repeat that process, or unplug them. Not sure if that's notable or not. I'm not quite sure what the problem is considering it works perfectly without the X server. Even after starting X from it working in CLI only, it may work once, but then a few minutes later, I see the same issue. Anyone else having this issue?
Edit: I re-installed the OS and all of the requirements and upon testing Visualazation.py I did get some feed to the LED's however it would jitter and skip. Shortly theirafter I stopped the program and attempted Standtest.py again, which caused no change to the existing state of the LED's. I unplugged the strip and plugged it back in, which cleared the LED's, but thereafter there were no LED's illuminated from ether program. Again, rebooting into the CLI allows Standtest.py to work perfectly.

Sounds like the program is hanging up on something while driving the LEDs, possibly whatever library or driver is used stops responding?


Multi-format copy and paste

This was very frustrating! I spent a lot of time trying to come up with a more "just works" API, but it ended up getting hairy and ultimately I ripped it out and tried to just do the simplest API I could.
In any case, this is working on mac, windows and gtk. The only bug I'm aware of is on GTK; if you put multiple formats on the clipboard, and one of them is text, the clipboard doesn't seem able to retrieve the text. I'm not totally sure what's going on there, but I'm not going to fight it right now, it's a pretty marginal case all in all.
Anyway: this covers the common case (text) and the special case required by runebender (multi-format vector data) comfortably. There may be future limitations; in particular I think doing copy/paste for images is going to be a headache and a half, but we can worry about that at some future date.

/bloat


Unexpected output in stderr while running Powershell EncodedCommand without pseudo-tty

Background and Description
I'm trying to run Write-Error boomed with pwsh -EncodedCommand VwByAGkAdABlAC0ARQByAHIAbwByACAAYgBvAG8AbQBlAGQA, and I'm expecting the following output:
Write-Error: boomed

And it works pretty well. However, running this command without pseudo-tty, will cause it to print some meaningless characters. I'm using ssh -T localhost xxx to run without tty, and ssh -tt localhost xxx to run with tty. And I'm getting the following test results:

with tty (-tt)
without tty (-T)

Windows powershell 5.1
good
messed up

Linux powershell 7
good
messed up

Steps to reproduce
Run
ssh -T localhost pwsh -EncodedCommand VwByAGkAdABlAC0ARQByAHIAbwByACAAYgBvAG8AbQBlAGQA

Expected output in stderr
Write-Error: boomed

Actual output in stderr
#< CLIXML
<Objs Version="<IP_ADDRESS>" xmlns="http://schemas.microsoft.com/powershell/2004/04"><S S="Error">_x001B_[91mWrite-Error: _x001B_[91mboomed_x001B_[0m_x000A_</S></Objs>

Environment data
Name                           Value
----                           -----
PSVersion                      7.1.3
PSEdition                      Core
GitCommitId                    7.1.3
OS                             Linux 5.15.10-hardened1-1-hardened #1 SMP PREEMPT Fri, 17 Dec 2021 20:16:31 +0000
Platform                       Unix
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

# Note that this problem also exists in powershell 5.1 on Windows.

Visuals

What have I tried
I have tried setting $WarningPreference = "SilentlyContinue". After setting this variable, I can run  [Console]::Error.WriteLine('xxx') with pwsh -EncodedCommand without any issue. But Write-Error xxx is still messing up the terminal.
Ref
https://github.com/ansible/ansible/issues/67964
https://stackoverflow.com/questions/51634308/start-process-using-encodedcommand-outputs-to-standard-error

The template is preventing me from adding more information. Please wait before I added more info...

I have finished my modification to my post. You may look at it.
(the issue template was preventing me from adding more details... So I have to modify it after creation)

@WG-Remoting
Remoting working group: removing WG-Remoting label as this is not a PowerShell remoting issue.  Instead, this appears to be an issue with PowerShell running within a SSH remoting session.

It looks like due to the logic here: https://github.com/PowerShell/PowerShell/blob/bcde806724b765dcf8a854cadacfac60a7bb1031/src/Microsoft.PowerShell.ConsoleHost/host/msh/ConsoleHost.cs#L1349-L1364
The workaround is to specify the outputformat:
ssh -T localhost pwsh -EncodedCommand VwByAGkAdABlAC0ARQByAHIAbwByACAAYgBvAG8AbQBlAGQA -outputformat text

The problem is because there is no tty, pwsh interprets that as non-interactive.  Console is redirected as it's redirected by ssh.  No output format was specified and the command was encoded.  So I don't see a way to change this without breaking existing usage.

Seeing similar behavior in Azure DevOps pipeline etc. after moving from powershell.exe -command .. to -encodedCommand(https://github.com/pester/Pester/commit/3d05da013946589b0cdf77c1c2daf966fafe23b1). Suddenly getting clixml output instead of text for InformationRecord, WarningRecord etc. We've not specified -NonInteractive but I guess it's auto-detected somehow in Azure Pipelines.
If this is intended behavior, please update about_PowerShell.exe / about_pwsh.exe as it really is unexpected.


random question re flutter/dart

Does anyone know of any effort out there to make a turbo-flutter project?
So instead of building directly on top of IOS or Android ... build on top of flutter/dart?
Why ... for an existing flutter app this would provide a convenient way to add turbo bits and pieces.

I was thinking about this today too but couldn’t find much on it. If I get some time, it’d be cool to dig into how Turbo (now Hotwire) and Flutter could work together.