[Feature] Connect to RayCluster via GCS port rather than Ray client in compatibility test

Search before asking

[X] I had searched in the issues and found no similar feature requirement.

Description
Based on my observation, the GCS FT bugs (#634 and #638) are easily triggered when we connect to RayCluster via GCS port. Hence, we temporarily connect to RayCluster via Ray client. However, the Ray community plans to de-emphasizes Ray client. Therefore, we need to update ray.init when #634 and #638 are fixed. See https://github.com/ray-project/kuberay/pull/844#issuecomment-1364091941 for more details.
Use case
No response
Related issues
#634
#638
#844
Are you willing to submit a PR?

[X] Yes I am willing to submit a PR!

The flakiness caused by ray.init() is because it shuts down the process with exit code 1 rather than throwing an exception when it fails to connect to the cluster if the cluster is not ready.

To clarify, if we use sys.exit(1) to exit the process, it will raise an exception that can be caught by BaseException. However, when the process exits due to QuickExit, no exception is thrown. As a result, the process will terminate after the first call of ray.init() and will not attempt a retry. On the other hand, if the cluster is not ready, Ray client will throw a Python exception.
https://github.com/ray-project/kuberay/blob/4fbdb9e130d30b33395522ff9c682dcc2b91d381/tests/config/ray-cluster.ray-ft.yaml.template#L243-L253

Logs for ray.init failure.2023-08-26 11:38:30,134	INFO worker.py:1330 -- Using address <IP_ADDRESS>:6379 set in the environment variable RAY_ADDRESS
2023-08-26 11:38:30,135	INFO worker.py:1459 -- Connecting to existing Ray cluster at address: <IP_ADDRESS>:6379...
2023-08-26 11:38:30,203	INFO worker.py:1640 -- Connected to Ray cluster. View the dashboard at http://<IP_ADDRESS>:8265
[2023-08-26 11:38:39,210 E 244 258] core_worker_process.cc:216: Failed to get the system config from raylet because it is dead. Worker will terminate. Status: GrpcUnavailable: RPC Error message: failed to connect to all addresses; RPC Error details:  .Please see `raylet.out` for more details.
command terminated with exit code 1

Ray may need to have consistent behaviors to thrown exceptions between different methods for connecting with Ray.


Add firebase

Tracking Info
Resolves parts of #15
Changes
Added firebase to frontend and verify tokens in backend.
Testing
Tested with logging in with<EMAIL_ADDRESS>and password abc123, then verifying the account's info in compass.
Confirmation of Change
See the testing section.

Waiting on @parth4apple to change the .env firebase variable to be named NEXT_PUBLIC_FIREBASE_CONFIG instead, because for some reason you can't read env variables that don't start with NEXT_PUBLIC.,


feat: +SECURITY.md

Features

add SECURITY.md
Fixs #967

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

Project coverage is 80.63%. Comparing base (5cae13f) to head (e4203e5).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #970      +/-   ##
==========================================
- Coverage   80.69%   80.63%   -0.07%
==========================================
  Files         226      226
  Lines       13178    13178
==========================================
- Hits        10634    10626       -8
- Misses       2544     2552       +8

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

LGTM


feat: Add soundScript for menu buttons

This script enhances the user experience of buttons by adding audio feedback when they are clicked or hovered over. The script leverages the Unity EventSystems and AudioSource components to seamlessly integrate sound effects into button interactions.
To achieve this functionality, the script utilizes the EventTrigger component attached to the button to capture mouse events. Upon detecting a click or hover event, the script checks the button's intractable state, ensuring that only enabled buttons trigger sound playback.
If the button is enabled, the script plays the corresponding sound clip, either a click sound for button clicks or a hover sound for mouse hovers. The script ensures that only the appropriate sound is played, preventing unintended audio cues.

Tested and working.


I18n of numbers and dates

Formatting numbers such as "Install size 2,345 kB", this should use locale-aware formatting (decimal and thousands separator). This can be done with `number.toLocaleString(cockpit.language). Obviously this should be put into a wrapper in core/utils.js with a simple fallback when not running under Cockpit.
Any dates/times such as "Last modified date 01/12/17" should be formatted using moment.js, which deals with proper i18n (assuming one calls moment.locale(cockpit.language) initially) and also human-friendly relative timestamps like "a few moments ago" or "6 hours ago" where no perfect precision is wanted.

Tracked by https://issues.redhat.com/browse/COMPOSER-1753


TZInfo::PeriodNotFound exception when using Time.zone

Issue description
When using Time.zone instead of ENV['TZ'], Cron#next_time throws a TZInfo::PeriodNotFound exception.
How to reproduce
require 'fugit'
require 'active_support/core_ext/time/zones'
Time.zone = 'America/Santiago'  # it works with ENV['TZ'] = 'America/Santiago'
t = EtOrbi::EoTime.parse('2021-08-18 01:00:00')
puts Fugit::Cron.parse('0 8 15 * *').next_time(t).to_s

Error and error backtrace (if any)
 11: from /Users/pascal/.rvm/gems/ruby-2.7.2/gems/fugit-1.5.1/lib/fugit/cron.rb:251:in `next_time'
 10: from /Users/pascal/.rvm/gems/ruby-2.7.2/gems/fugit-1.5.1/lib/fugit/cron.rb:251:in `loop'
  9: from /Users/pascal/.rvm/gems/ruby-2.7.2/gems/fugit-1.5.1/lib/fugit/cron.rb:261:in `block in next_time'
  8: from /Users/pascal/.rvm/gems/ruby-2.7.2/gems/fugit-1.5.1/lib/fugit/cron.rb:107:in `inc_day'
  7: from /Users/pascal/.rvm/gems/ruby-2.7.2/gems/et-orbi-1.2.4/lib/et-orbi/make.rb:59:in `make_time'
  6: from /Users/pascal/.rvm/gems/ruby-2.7.2/gems/et-orbi-1.2.4/lib/et-orbi/make.rb:108:in `make_from_array'
  5: from /Users/pascal/.rvm/gems/ruby-2.7.2/gems/et-orbi-1.2.4/lib/et-orbi/make.rb:44:in `parse'
  4: from /Users/pascal/.rvm/gems/ruby-2.7.2/gems/tzinfo-2.0.3/lib/tzinfo/timezone.rb:648:in `local_to_utc'
  3: from /Users/pascal/.rvm/gems/ruby-2.7.2/gems/tzinfo-2.0.3/lib/tzinfo/timestamp.rb:142:in `for'
  2: from /Users/pascal/.rvm/gems/ruby-2.7.2/gems/tzinfo-2.0.3/lib/tzinfo/timezone.rb:652:in `block in local_to_utc'
  1: from /Users/pascal/.rvm/gems/ruby-2.7.2/gems/tzinfo-2.0.3/lib/tzinfo/timezone.rb:500:in `period_for_local'
     TZInfo::PeriodNotFound (2021-09-05 00:00:00 is an invalid local time.)

Expected behaviour
The timezone setting in Time.zone should be used as fallback when ENV['TZ'] is nil and the correct output ("2021-09-15 08:00:00 -0300") should be returned.
Context
Darwin Pascals-BigPro.fritz.box 20.6.0 Darwin Kernel Version 20.6.0: Wed Jun 23 00:26:31 PDT 2021; root:xnu-7195.141.2~5/RELEASE_X86_64 x86_64
ruby 2.7.2p137 (2020-10-01 revision 5445e04352) [x86_64-darwin20]
[:env_tz, nil]
(secs:1631862880.412856,utc~:"2021-09-17 07:14:40.4128561019897461",ltz~:"CEST")
(etz:nil,tnz:"CEST",tziv:"2.0.4",tzidv:nil,rv:"2.7.2",rp:"x86_64-darwin20",win:false,rorv:nil,astz:nil,eov:"1.2.4",eotnz:#<TZInfo::TimezoneProxy: Africa/Ceuta>,eotnfz:"+0200",eotlzn:"Africa/Ceuta",eotnfZ:"CEST",debian:nil,centos:nil,osx:"zoneinfo/Europe/Zurich")
"1.5.1"
[:now, 2021-09-17 09:15:13.611169 +0200, :zone, "CEST"]

)

Thanks.
Moved to https://github.com/floraison/et-orbi/issues/31

The time zone in Time.zone is used and it is what is causing the problem in ::TZInfo::TimeZone.local_to_utc. Back here.

https://github.com/codez/delayed_cron_job/issues/30

Hello,
I've just released https://rubygems.org/gems/fugit/versions/1.5.2
It solves the issue (at least for me). If you spot anything wrong, please do tell me.
Thanks and best regards!

Thank you very much for your work!

@codez je t'en prie, merci vielmal again!


Redisson script caching

According to the Redis documentation, script caching is recommended, using EVALSHA and EVAL only if Redis server does not have the script already loaded, but I don't see this behavior in Redisson when using EVAL.
This seems to be supported in Spring spring-data-redis using the default ScriptExecutor.
Is there any script caching option I am not aware of? Is this going to be implemented if missing?

Config.useScriptCache settings added. Please refer to wiki for more details https://github.com/redisson/redisson/wiki/2.-Configuration#usescriptcache

Awesome, thanks for the quick changes!

@alturkovic
Thank you! Redisson architecture affords that ;)

Awesome work @mrniko

@jackygurui
Thank you!

if this has better performance, why isn't it true by default? Any gotchas?


Tuxedo application

Project Abstract
We propose to create Tuxedo, a framework for developing Substrate runtimes based on the Unspent Transaction Output (UTXO) model.
There are essentially two models for creating blockchain state machines:

Account System as seen in Ethereum, Polkadot, and others
UTXO System as seen in Bitcoin, Monero, Cardano, and others.

FRAME has been very successful in making the Accounts system accessible to Substrate developers. Tuxedo would make the UTXO model similarly accessible and begin to create a diversity of runtime frameworks, thus strengthening the entire Substrate Ecosystem.
Grant level

[ ] Level 1:  Up to $10,000, 2 approvals
[x] Level 2:  Up to $30,000, 3 approvals
[ ] Level 3:  Unlimited, 5 approvals (for >$100k: Web3 Foundation Council approval)

Application Checklist

[x] The application template has been copied and aptly renamed (project_name.md).
[x] I have read the application guidelines.
[x] A BTC, Ethereum (USDC/DAI) or Polkadot/Kusama (aUSD/USDT) address for the payment of the milestones is provided in the application.
[x] The software delivered for this grant will be released under an open-source license specified in the application.
[x] The initial PR contains only one commit (squash and force-push if needed).
[x] The grant will only be announced once the first milestone has been accepted (see the announcement guidelines).
[ ] I prefer the discussion of this application to take place in a private Element/Matrix channel. My username is: @_______:matrix.org (change the homeserver if you use a different one)

 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.

@JoshOrndorff dismissed stale reviews from @takahser and @Noc2 via 3c0b41c

Oh oops. I didn't mean to dismiss your reviews. That happened automatically. Could I get your approvals again :pray: Sorry about that.

About a year ago I was conducting the Technical Feedback Process for the Ecosystem Success team at Parity.
While talking with Enjin/Efinity, @Lohann relayed to me that they could greatly benefit from a native UTxO support on Substrate.
Gaming apps impose very high throughput requirements, they experienced difficulty synchronizing concurrent transactions, especially when game specific actions needed to be executed sequentially.
On account-based ledgers, every failed operation is reverted. For the gaming application, that imposes the requirement of a fallback logic that greatly increases its complexity.
As a consequence, dealing with failed concurrent transactions became a highly consequential optimization issue in their pipeline.
An UTxO-based ledger that does not impose the need to process reverted transactions would help them achieve their desired throughput without overbloating the gaming apps with fallback logic for failed txs.

In my limited knowlege of them, I think the context of zero knowledge proving systems, UTXOs will be far more reasonable and opimizable to use vs. account bases systems.
This work may prove foundationally critical to rollups and privacy tools as well.
Will share with the ZKP on Polkadot chat to get some others commenting on that front.

Thanks for the support everyone :pray:
We plan to start work on Monday 13 Feb. We're still wrapping up the latest academy right now.


Publish C# 11 speclets

Add the C# 11 speclets to the C# TOC and the list of published specifications.
Fixes #31408
Fixes #31458

/cc @jcouv @333fred


Could tarpauling set an environment variable to detect in build.rs?

I have a build.rs for my project and it somehow fails when tarpauling tries to execute it as part of the build. Otherwise it doesn't fail. It's not needed for tarpaulin to work, so I'm thinking of having the build.rs just return when it detects a tarpaulin run.
Would it be possible for tarpaulin to just set a simple env variable like TARPAULIN=1 or something? I think it can make sense in more cases than just mine.

Yeah sure, I had seen the build.rs issue with some repos and not others so was hoping to get to the bottom of it and solve it properly but this seems a reasonable compromise

There's now a version with this in the develop branch if you want to already start using it


Fix russian short form of minutes

Hey, @michael-lynch Could you merge it please? It is weird to see wrong form there. It's like 2 mistakes in 3-letters word.


Print json result for sushi errors when user specify --json

Description

As title. It's in MVP of mainnet of SushiChain.
Related Issues

https://github.com/SushiChain/SushiChain/issues/214

Checklist

[x] Every CI jobs finished successfully

awesome :)


Update references to idev_csr_ready

RTL Issue
Once this bit is present in the soc_ifc registers structures these locations in the sw repo need to be updated:
drivers/src/soc_ifc.rs: 82
drivers/test-fw/src/bin/status_reporter_tests.rs: 39
rom/dev/tests/helpers.rs: 53
In PR 475 these are being changed from 0x0800_0000 to 0x0100_0000. Should refer to them by name.

In addition to the above locations, sw-emulator/lib/periph/src/soc_reg.rs:111 has an independent definition of the FlowStatus register that also needs to be kept in sync.

Fixed by: #590


recover images time stamp problem

Hello, I'm using e2calib for imu-ecam chain calibration and I recover images from event camera successfully. Kalibr calculates the intrinsic of my event camera correctly(reprojection error RMS 0.15pix) but when it comes to extrinsic, the calibration can not converge  with huge reprojection error and wrong extrinsic results. The event camera is synchronized with IMU using hardware trigger. I check the synchronization for a few times but every thing goes well. So I wonder if I miss some details about your project in taking stamps for events and hope you can help me. THX

Hello @LauZanMo,
It's important to note that a low reprojection error doesn't necessarily indicate accurate calibration. To ensure accuracy, it's essential to have comprehensive coverage of detected corners within the camera frame, as this makes it more challenging to achieve a low reprojection error.
In our experience, we encountered similar issues when attempting IMU-eCam calibration with a low-cost IMU, specifically one integrated within the camera case. We haven't tested this process with a higher-quality IMU. Have you had the chance to try it with a better IMU?

Prophesee EVK4 HD(event camera) and ADIS16465(IMU) is mounted on my test suite. However, I use kalibr intrinsic result to run pure vo (ORB3) in the e2calib recover images and the trajectory result is bad with only pure rotation, as I only do pure translation during the calibration. Maybe the intrinsic of output is also wrong although the reprojection error is low. I think the problem may comes from the wrong stamps of events. Do you have any idea?

When you look at the report from Kalibr, do you see good coverage of the detected pattern? That is, has the pattern been detected throughout the whole field of view?


Emit close event on archive close

It can be a little frustrating to manage hyperdrives (for example in the SDK) when you don't know when an application will be closing them.
This PR propagates the close event from the metadata hypercore to the hyperdrive.

9.16.0


Use Rocky Linux 8 and vcpkg to build RPM packages

Motivation
See https://lists.apache.org/thread/7o8hpv1gtoffvzx053wm0ss2s9xt0795, we will discard the support for CentOS 7 and old std::string ABI.
Modifications

Switch from centos:7 image and GCC 4.8 to rockylinux:8 image and GCC 8 to build RPM packages.
Use vcpkg to install dependencies according to vcpkg.json (dependencies.json will be deprecated in future)
Add a new script to build libpulsarwithdeps.a for dependencies installed by vcpkg
Link to libstdc++ and libgcc_s libraries statically

This patch has been verified in my own fork, see https://github.com/BewareMyPower/pulsar-client-cpp/pull/32#issuecomment-2172077631


Threshold for scaling out and Threshold for scaling in

Proposal
There should be a threshold for scaling out, and another for scaling in.
Use-Case
This way you can implement something like "If there are more than 1000 messages queued, scale OUT another pod. If there are more than 20 but less than 1000 messages, don't do anything. If there are less than 20, scale IN one pod"...
or perhaps "If each of my pods are dealing with more than 100 req/s, scale out. If they are handling between 20 and 99 don't change, if they each and all of them are handling less than 20 req/s scale in by 1"
Is this a feature you are interested in implementing yourself?
Maybe
Anything else?
No response

Any feedback, anyone?

Hi, thanks for the proposal, I do understand the reasoning, unfortunately it is very hard to achieve as the scaling itself is driven by HPA and it doesn't support such behaviour. Do you have any specific ideas how to do that?

hey @zroubalik, I opened this one: https://github.com/kubernetes/autoscaler/issues/6990


Plots with matplotlib

Dear developers,
Somehow I experience this bug that the plot turns black once I move around to other cells. I saw that you were using matplotlib as well in your examples, did you have a certain configuration for your gui?
Best,
v.

Hi,
Are you using kitty terminal by any chance?
The kitty image implementation is a work in progress and needs some work - I wrote a bit more about it in this issue 12.

Quick update - the kitty graphics system has been entirely re-written, so this should be fixed in the v1.0.0


test: add more tests to improve coverage

Codecov Report

Merging #209 (b14158d) into main (b2966b7) will increase coverage by 1.54%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #209      +/-   ##
==========================================
+ Coverage   86.97%   88.51%   +1.54%
==========================================
  Files         115      115
  Lines        7330     7330
==========================================
+ Hits         6375     6488     +113
+ Misses        955      842     -113

Flag
Coverage Δ

e2e.cbl
88.79% <100.00%> (+1.55%)
:arrow_up:

unit.cbl
2.62% <0.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

packages/cbl/lib/src/service/cbl_service_api.dart
96.35% <100.00%> (+7.43%)
:arrow_up:

packages/cbl_ffi/lib/src/libraries.dart
67.74% <0.00%> (-3.23%)
:arrow_down:

packages/cbl/lib/src/document/fragment.dart
95.94% <0.00%> (+2.02%)
:arrow_up:

packages/cbl_ffi/lib/src/replicator.dart
86.82% <0.00%> (+2.39%)
:arrow_up:

...ckages/cbl/lib/src/replication/ffi_replicator.dart
89.83% <0.00%> (+3.38%)
:arrow_up:

packages/cbl/lib/src/database/ffi_database.dart
98.67% <0.00%> (+4.63%)
:arrow_up:

packages/cbl/lib/src/database/proxy_database.dart
90.06% <0.00%> (+6.62%)
:arrow_up:

packages/cbl/lib/src/errors.dart
92.00% <0.00%> (+8.00%)
:arrow_up:

packages/cbl/lib/src/query/ordering.dart
66.66% <0.00%> (+13.33%)
:arrow_up:

... and 2 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b2966b7...b14158d. Read the comment docs.


Update stageprompt.js

Update to enable support for the latest Google Analytics Events api inline with https://github.com/alphagov/stageprompt

Closing in favour of https://github.com/alphagov/govuk_frontend_toolkit/pull/421 – this repo is automatically updated when govuk_frontend_toolkit changes, so the change should be made there.


align MSRV check with 'rust-version'

the MSRV check should match the 'rust-version' specified in Cargo.toml

as you can see from the failed test, the rust-version in Cargo.toml is not correct. I'll have to bump the version to get this to build

i've left the commits in there for ease of review, but you probably want to squash merge this

Thanks @danieleades, can you squash the commits ? We don't have squash merge enable for better descriptive commit messages.

Thanks @danieleades, can you squash the commits ? We don't have squash merge enable for better descriptive commit messages.

 It gives you the option of setting the commit message when you merge, no?


State machine testing

This is a new version of https://github.com/proptest-rs/proptest/pull/257 with some refinements.
Accompanying blog post (it contains the guide to be included in the Proptest book): https://tzemanovic.gitlab.io/posts/state-machine-testing-with-proptest/
A summary of differences from #257:

StateMachineTest::init_test takes abstract_state by immutable reference instead of an owned value.
StateMachineTest::apply_concrete and StateMachineTest::invariants additionally take abstract_state immutable reference, which can be used for comparison with the ConcreteState.
The StateMachineTest::invariants are also checked on the initial state
StateMachineTest::test_sequential now takes proptest::test_runner::Config, which can be used enable printing of the transitions before they're applied in the test with verbose = 1 or higher to make it easier to follow what the state machine does and to aid in diagnostics. Because of this change, the support for state machine test setup without the prop_state_machine! macro has been dropped as it's not possible to access the config value without it.
Improved the state machine's rustdoc.
Some internal refactors have been applied in the state machine strategy implementation.
Added tests for the sequential strategy implementation.

Found and fixed an issue in which an index of a rejected simplification wasn't getting cleared from SequentialVecTree::shrinkable_transitions, which could cause an infinite loop.

Added a new example state_machine_echo_server.rs
Added a Proptest Book page.

StateMachineTest::apply_concrete and StateMachineTest::invariants additionally take abstract_state immutable reference, which can be used for comparison with the ConcreteState.

Yes! Bravo! 👏

We're still using the version over at https://github.com/proptest-rs/proptest/pull/257
Is there any way to run setup/cleanup code per run?
I'd like to delete resources that I create during the execution of my sequence of transitions. I'd like not to put too many things esp effectful code in impl Drop.
See for example in an Erlang implementation: https://proper-testing.github.io/apidocs/
?SETUP(<Setup_fun>, <Prop>)
    Adds a setup <Setup_fun>ction to the property which will be called before the first test. This function has to return a finalize function of arity 0, which should return the atom ok, that will be called after the last test. It is possible to use multiple ?SETUP macros on the same property.

The hard part is running code after a whole sequence of calls is executed and before the model struct is being dropped.

I see - you should be able to do that when you override the default test_sequential after the for loop that goes through the transitions. I'm not sure how common this is, but maybe there could be an additional method to allow that, e.g. tear_down

I do this all the time. Inset set up code before the loop in test_sequential and teardown afterwards. Works fine

I am very excited about this.
One suggestion: the heap example is slightly too simple.
In particular, the key differentiator in my mind of stateful testing vs. non-stateful testing is the ability to generate transitions based on current state. For the heap example, I could just do a normal proptest and generate a sequence of Pop/Push commands.
On the other hand, an example of a key-value map, rather than a heap, where you insert values and might have transition to get existing keys, shows the extra value you get from having a model state. Generating a Transition::Get(existing_key) is where the stateful testing really lets you express new kinds of tests. And the echo server  example does that, but with more complexity.
Second suggestion: the docs could probably benefit from a walkthrough of implementing a concrete, simple example (e.g. the key-value map). In the four-quadrant documentation model this is missing the Tutorial quadrant.
Neither of these seem like blockers, since they can always be done as a second pass.

Overriding test_sequential for the purpose of teardown i.e. inserting code means duplicating the rest of the code, at the risk of this duplication getting stale.
I'd much prefer passing a config option or an additional teardown trait with a default noop impl. Please.

Thanks to everyone who worked on this!

thanks for the additional reviews @cameron1024 and @rex-remind101! I've added couple fixups from the suggestions. Please let me know if you think this ready and I'll squash them

force-pushed to squash the fixups https://github.com/proptest-rs/proptest/compare/bd6d95cc76c65b593da11d7135d057c64d90cb00..d85ccf2bdc3a8813e1d7346880ddb4fbceddbe05

Wow, this is so different from the original branch that I've been using for a year.

Wow, this is so different from the original branch that I've been using for a year.

It’s most just renames (ref to the description) and being separated to its own crate

So far I've failed to understand how to get it into my project…I use proptest = 1.2.0 in cargo toml (rather than my old fork) and everything breaks. I read the docs…no mention of how to include the new crate. I search on crates.io…can't find it.
Is it "released" yet?

So far I've failed to understand how to get it into my project…I use proptest = 1.2.0 in cargo toml (rather than my old fork) and everything breaks. I read the docs…no mention of how to include the new crate. I search on crates.io…can't find it.
Is it "released" yet?

Yeah, it will be released as v0.1.0, but until then you can use it with proptest-state-machine = { package = "proptest-state-machine", git = "https://github.com/proptest-rs/proptest.git", tag = "v0.1.2" }

Yeah, it will be released as v0.1.0, but until then you can use it with proptest-state-machine = { package = "proptest-state-machine", git = "https://github.com/proptest-rs/proptest.git", tag = "v0.1.2" }

Ah, so grateful for your help, many thanks! It will be great to get off this fork 😊

thank you for raising this @russelldb , it has been released https://crates.io/crates/proptest-state-machine

Thank you SO much for working on this @tzemanovic This is AMAZING!


index.html: Update text - fixes #1

:star: PR built and available in a preview environment gmacario-node-http-hmi-repository-pr-2 here

LGTM, merging PR


Unable to build apk for Android with Flutter version 3.7.0

🐛 Bug Report
Getting error while creating build for android
FAILURE: Build failed with an exception.

What went wrong:
Execution failed for task ':geolocator_android:compileReleaseJavaWithJavac'.

Compilation failed; see the compiler error output for details.

Expected behavior
Reproduction steps
Configuration
Pacakge and flutter version
geolocator: ^9.0.2
Version: 3.7.0 flutter
Platform:

[ ] :robot: Android

Hi! I have a similar issue, I updated my flutter project to v3.7.
And when running it on android emulator I get the same failure when building:
FAILURE: Build failed with an exception.

What went wrong:
Execution failed for task ':geolocator_android:compileDebugJavaWithJavac'.

There're some deprecations in the code:
warning: [deprecation] stopForeground(boolean) in Service has been deprecated
stopForeground(true);
^
warning: [deprecation] getPackageInfo(String,int) in PackageManager has been deprecated
.getPackageInfo(context.getPackageName(), PackageManager.GET_PERMISSIONS);
^
1 error
2 warnings
I'm currently using geolocator ^8.2.1, couldn't update to the last version, but as I see it on de documentation it's from the las 5 months. Should be updated to the last Flutter version.
flutter doctor
Doctor summary (to see all details, run flutter doctor -v):
[√] Flutter (Channel stable, 3.7.5, on Microsoft Windows [Versi¢n 10.0.19044.2604], locale es-CL)
[X] Windows Version (Unable to confirm if installed Windows version is 10 or greater)
[√] Android toolchain - develop for Android devices (Android SDK version 32.1.0-rc1)
[√] Chrome - develop for the web
[√] Visual Studio - develop for Windows (Visual Studio Community 2022 17.1.0)
[√] Android Studio (version 2021.1)
[√] VS Code (version 1.75.1)
[√] Connected device (3 available)
[√] HTTP Host Availability
! Doctor found issues in 1 category.

Maybe Related issue: https://github.com/Baseflow/flutter-geolocator/issues/795
Try out one of those solutions: https://github.com/Baseflow/flutter-geolocator/issues/795#issuecomment-888179840
https://github.com/Baseflow/flutter-geolocator/issues/795#issuecomment-1671218143

Thanks for pointing this issue out @jb3rndt, this has been solved in version 4.2.1 of the geolocator_android package. Please run flutter pub upgrade to download the latest version.

I just had the issue with 4.2.1 and 4.2.2. After downgrading to 4.2.0 it disappeared.

Logs:
[        ] /Users/jberndt/.pub-cache/hosted/pub.dev/geolocator_android-4.2.1/android/src/main/java/com/baseflow/geolocator/location/FusedLocationClient.java:91: error: cannot find
symbol
[   +1 ms]     LocationRequest.Builder builder = new LocationRequest.Builder(0);
[        ]                    ^
[        ]   symbol:   class Builder
[        ]   location: class LocationRequest
[        ] /Users/jberndt/.pub-cache/hosted/pub.dev/geolocator_android-4.2.1/android/src/main/java/com/baseflow/geolocator/location/FusedLocationClient.java:91: error: cannot find
symbol
[        ]     LocationRequest.Builder builder = new LocationRequest.Builder(0);
[        ]                                                          ^
[        ]   symbol:   class Builder
[        ]   location: class LocationRequest
[        ] 2 errors
[        ] FAILURE: Build failed with an exception.
[        ] * What went wrong:
[        ] Execution failed for task ':geolocator_android:compileDebugJavaWithJavac'.
[        ] > Compilation failed; see the compiler error output for details.
[        ] * Try:
[        ] > Run with --debug option to get more log output.
[        ] > Run with --scan to get full insights.

Am I doing something wrong?

@jb3rndt, this seems a different error. In this case the compiler is complaining about not being able to find the LocationRequest.Builder class which has been introduced in Android API 31 and is replacing the deprecated LocationRequest.create method since Android API 33.
This could be related to the Android licenses you haven't accepted (according to the flutter doctor -v output you supplied). Can you try running flutter doctor --android-licenses and accepting the licenses and try again? If this doesn't work please create a new issue and I would be happy to help and dive into this.

It had nothing to do with the licenses, but I found some other reasons. I'll open a new issue. Thank you :)

If you

I just had the issue with 4.2.1 and 4.2.2. After downgrading to 4.2.0 it disappeared.
Logs:
Am I doing something wrong?
flutter doctor -v

If you downgrade the geolocator_android version to 4.2.0, then the map loading would get stuck in the application, you might get rid of the build issues.


Experimental: Allow setting TX power for edge

https://github.com/Koenkk/zigbee2mqtt/issues/2253
@Koenkk @danielwelch please merge this

Hello, it is duplicate of #276

@dmitrybabeshko now i see it, but no one merge


Add dotnet tool to control LivePreview Analyzer

https://github.com/SilkyFowl/Avalonia.FuncUI.LiveView/commit/e2c7270f425df1251e22ca8599731cf2e5020bd6


LinalgRefactor - rename gpumemory data method

To distinguish from data_matrix method #3382

Happy to merge once green

@karlnapf ready : )


Can't access storage account from Azure Storage Explorer

Preflight Checklist

[X] I have installed the latest version of Storage Explorer.
[X] I have checked existing resources, including the troubleshooting guide and the release notes.
[X] I have searched for similar issues.

Storage Explorer Version
latest version
Regression From
no
Architecture
x64
Storage Explorer Build Number
No response
Platform
Windows
OS Version
Windows 10
Bug Description
{
"message": ""{\n  \"name\": \"Error\",\n  \"message\": \"self signed certificate in certificate chain\",\n  \"stack\": \"Error: self signed certificate in certificate chain\\n    at TLSSocket.onConnectSecure (_tls_wrap.js:1484:34)\\n    at TLSSocket.emit (events.js:310:20)\\n    at TLSSocket._finishInit (_tls_wrap.js:927:8)\\n    at TLSWrap.ssl.onhandshakedone (_tls_wrap.js:697:12)\",\n  \"code\": \"SELF_SIGNED_CERT_IN_CHAIN\"\n}""
}
Steps to Reproduce
Launch Storage Explorer
By default doesn't connect
Actual Experience

Expected Experience
No response
Additional Context
No response

Hi, it sounds like you may be behind a proxy. First, please upgrade to the latest version of storage explorer if not already done so. You can download it from https://storageexplorer.com.
After you update, we recommend:

Try using system proxy, which you can read about in our networking documentation: https://docs.microsoft.com/en-us/azure/storage/common/storage-explorer-network
Try setting up proxy manually. You can read the proxy section in our troubleshooting guide for additional help with that: https://docs.microsoft.com/en-us/azure/storage/common/storage-explorer-troubleshooting?tabs=Windows%2C2004#proxy-issues

If neither of those help feel free to come back here to let us know what happaned.

Closing due to lack of response. If you are still encountering this issue we suggest:

try the above troubleshooting steps
open an Azure support ticket via the portal
open a new issue here


[Addition]: Mixed PantherX/RisingHF cluster (16 HS)

Hotspot b58 Addresses
112AGRhjetsQKBr55gxQAocZGDCNjWG7ZSwsKmsxURSi7HxCpsZ7
112DuF8rPeA9koT9VQjkihgxoyWEbm1eHXHNtkY46TMeDEfKpREg
112goRTKHN5VcV5gRAehp3orAMMdg2WSTzNNLAgnCvRMbiz9fnyh
112SydcGfsi9Usq3Gdmq1t7i3oeUJjjCsBCVjou32EAPo5PTjAiy
112Y1Vw48ADj75XMQ3mwRdHzvFPh14yPhKeMtmtZNX3vqMS6LJeR
114mEWZF1VXNH164FyZxGq52LNkp1zk1ojCHxQa5hZJZTCRomyM
11kvD3QzJUfEHyQWy9mM5mgEEX6h2xd2J98So27aqBycRk7e3EE
11rscj5hcX6RrRENUNJySPg5VBiWZPy8VGdoFrWzGMnBDvAzp82
11SUzMSx3Wv3aJBVVTHQMoDwNDG2h8YQzWy4XwrLCCojD9dxxv3
11UfYgEHFTrigxA9ZpeuzgFCuoPotfBhQBi79TnSNXQf6JA3ps7
11UmqXyA6LqcY5J9UvdjuStqiBfz3kiMkpZcZx7KCYy4UuiVewq
11UqnRN3px4Pz9aVDXfpY6BW6kaGMgsWF4gX5EWL5xy9dCNiQHd
11VeSybtVjoL43gkFJVAZwRcVGCfjR3HTSZ4tiAaicMFhnCqSm3
11VYDLZYErYcMCFkjCKxWrK5bs4C6wtKr3wputjpQRoiMudj2hU
11yGreCWboFFLb4tzYsvd9MALuiz7xyEgM9Jm4U5eMHfPJcKW1k
11zovjBD2kcZKeoQt6QiN54zkNgNxXXN2xTDZVTZwDq6PTSNjtY
Discord Handle
No response
Reason(s)
These Hotspots are only witnessing their own brand, have very often a unlikely (perfect) placement, same assertion date-range and usually there are enough real hotspots arround but these do not get witnessed at all.
Below some proof of the cluster(s):

Wallet address 1：13vURcoH8pYUEWRGtfPXFnLkzeWuCnfUKjWFDJ93r4pdna1y4x1
Hotspot b58 Addresses
11nC26pT8yju88nnehqnAsRWDszg4mUgnD2EdroLt7FFGvCzomz
112VZA9A5j8FjpTKkmrMK8qRQ9orB2Gy4gLUQrhAgVtnSVmAepJH
11Ufv7oKyCkDzDh5nHsQSszGo1zeuyy9v2nKc9FgK8Q1hfD9LUW
112roYeWH7HvRn45Jf9zBFE9qyVJGhWKk6yNgLC5invnF5PGFVY7
112aMzszBSgdM76bDZ6DLdNR6uo2z4dc36Gfr7tKHoNhh8KxwdRL
11LYLzfeit4ktQ9SBopqP6RaSnoL8aE5ZWHWFvwXnXecYGDtNcQ
1121Es5197vC6whn6wRc5wMobciZ8JZyCuHk1T1PKhwR8TcQ7Hkj
112LWyZ2288Qc3Y3inodxXBMjnqiWiBwFLtxhQYhVyhXvWVfT2v6
11pykiCc2ctvTRncdxsELcLCqY5x6PU9xFXepDWrr59p1M5e1iM
11RkvARHvY9scVCBsgDChrLtQsBqrFFFYdgnNkTzxL12hF3WeTm
11cpiuau1m1yXSRX6zDG8hH4PGHog2yEYcJibNPKt5g8By1P9Cz
Wallet address 2：14kYFB121sDYUBFnLHNw7YdCNjipY6frMgbQZuMGeGEZpcky9Ji
Hotspot b58 Addresses
115g1ct8HbgEo4uCL5kTPk5A1hvnkwTx3uhcBeUzh4Smn8vZmBi
112BYX7FX4fAYyMxVQu7P7WET6vVPJqANDaR4rPM9RafuDZQnKTb
11VwefZqaqsWXHeEk6FxcvkfKoXzPZgRTrbryEzshZH9crqK8TT
118omh3f2pkpTE891ay7hmo1Xa8xqszccbb8xB9wxGiEi9xA2m8
112hP9sQ6a3nrT2JyWUqqujG6eXLEhNf8zYnD85BodBL6HaJTtxw
1127SUJAtFqenb2jM2daGV6KvzHFL4MkPg4ogRRkuUQ1yC4ftsZH
11PbpDp7tQnSRuiAGHBNH1E5JgCHd4nixiXfodoWHybbibdg7wY
112X5kXTXMLjfZnpk72FKHJGWzeD2Me5DyoVQNB2wMqG13Prib9c
119FTqX62VcvcZcuUA5uj748LTeLgZbfGNJNwv1HFaa8fGaCwXw
11yAv4KH7MYZCwAyWZTtCZT9YoXgLQovVgjxzYKzNqBtgdMhe7y
1122M1S24Y13nyp43dbhxeSJghk1bkNRkqzjZFixQRVVfYfL7pHJ
Reasons:
The 22 machines are all of the same brand and are assigned to two wallets. All verification behaviors only occur in the two wallets, and the assertion time is the same. Do you pretend not to see these?
These Hotspots are only witnessing their own brand, have very often a unlikely (perfect) placement, same assertion date-range and usually there are enough real hotspots arround but these do not get witnessed at all.

@berserkrrr
Can you give me a feedback?

@6CornerDefender

@6CornerDefender
It's important to note that you're hiding the real deployment, while the cheater continues to validate the bad project! ! !

@6CornerDefender
@berserkrrr


Fix email & migrations docs; update changelog

Correct user guide errors on migration & email pages.
Update changelog, getting ready for RC.1

Hello Jim,
I spotted this wrong assumption based on the comment above the variable
/*
|--------------------------------------------------------------------------
| Enable/Disable Migrations
|--------------------------------------------------------------------------
|
| Migrations are disabled by default for security reasons.
| You should enable migrations whenever you intend to do a schema migration
| and disable it back when you're done.
|
*/
public $enabled = true;

@lonnieezell @MGatner Contradictory user guide text pointed out. Which is correct - the text or the setting?

Text is wrong - migrations are enabled by default. That was a change with lonnie's refactor and we probably just missed the comment, good catch by fefo-p

Fixed this with #2157


[WIP] Refactoring

Connects to: https://github.com/resin-io-playground/node.analytics/issues/4
Connects to: https://github.com/resin-io-playground/node.analytics/issues/6
Connects to: https://github.com/resin-io-playground/node.analytics/issues/10
Connects to: https://github.com/resin-io-playground/node.analytics/issues/17

@izavits A massive refactoring PR is going to be really hard to merge. Can you send separate PRs to tackle each of the issues separately, and thus gets the library polished in an incremental way?

yes, sure. I'll close this one and I'll send separate PRs. :-)


Specifying a directory in staticpaths shouldn't migrate files one by one

When specifying a directory in staticpaths, staticjinja should copy the entire directory, and all sub files in one operation. Currently the files in the directory are copied one by one, which for a large number of files can be extremely slow.

This is true. Can you be more specific as to "extremely slow"? How long for how many files?
Doing them all at once would require a larger re-structuring, it is nice how right now the code only operates on a template-by-template basis, and we don;t have to worry about bulk operations.
Perhaps another solution would be to use async/await, I believe that copying files would be a good application for that, and it might need less refactoring.


Out of memory

Hello.
Running program as Administrator, disk C and other disks has more then 20+ gb of free space, also have 7 of 16gb free ram, BUT when trying to download several .mov's( each is 40-50mb) in a row from discord channel - it said that there was an error and throw me a bunch of debug info:
runtime: out of memory: cannot allocate 301989888-byte block (540606464 in use)
fatal error: out of memory

goroutine 256 [running]:
runtime.throw({0x1183d5f, 0xd})
        C:/Program Files/Go/src/runtime/panic.go:1047 +0x4d fp=0xb4bf4f0 sp=0xb4bf4dc pc=0xab6cad
runtime.(*mcache).allocLarge(0x25609c8, 0x11edc000, 0x1)
        C:/Program Files/Go/src/runtime/mcache.go:236 +0x1be fp=0xb4bf518 sp=0xb4bf4f0 pc=0xa93f7e
runtime.mallocgc(0x11edc000, 0x0, 0x0)
        C:/Program Files/Go/src/runtime/malloc.go:1053 +0x3eb fp=0xb4bf550 sp=0xb4bf518 pc=0xa8b85b
runtime.growslice(0xbb1c000, 0xe57c001, 0xe57c000, 0x1, 0x10b2d40)
        C:/Program Files/Go/src/runtime/slice.go:266 +0x3ec fp=0xb4bf57c sp=0xb4bf550 pc=0xacb78c
io.ReadAll({0x53f58690, 0xb0b4510})
        C:/Program Files/Go/src/io/io.go:699 +0x8f fp=0xb4bf5bc sp=0xb4bf57c pc=0xaf139f
main.downloadRequestStruct.tryDownload({{0xb22ed20, 0x66}, {0xb5570f8, 0x17}, {0x0, 0x0}, {0xb026900, 0x2f}, 0xb6d2750, {0x3750280, ...}, ...})
        P:/Projects/Go/src/github.com/get-got/discord-downloader-go/downloads.go:727 +0xf66 fp=0xb4c0138 sp=0xb4bf5bc pc=0x1027e86
main.downloadRequestStruct.handleDownload({{0xb22ed20, 0x66}, {0xb5570f8, 0x17}, {0x0, 0x0}, {0xb23c270, 0x2e}, 0xb6d2750, {0x3750280, ...}, ...})

        C:/Program Files/Go/src/runtime/proc.go:387
runtime.forcegchelper()
        C:/Program Files/Go/src/runtime/proc.go:305 +0xcf fp=0xb057ff0 sp=0xb057fdc pc=0xab9a0f
runtime.goexit()
        C:/Program Files/Go/src/runtime/asm_386.s:1326 +0x1 fp=0xb057ff4 sp=0xb057ff0 pc=0xae4e41
created by runtime.init.5
        C:/Program Files/Go/src/runtime/proc.go:293 +0x23

and other 34+ goroutine. Is there any way to fix this?

This is happening consistently and not just once?

Tried 3 channels - every channel with videos and images have that error. Error can appear after 40+(randomly) downloaded images+videos or after start of first file download - I didn't found any viable pattern. But every time its "out of memory" with random numbers of memory allocation errors.
Now tried run it again on channel, which I downloaded some time ago, but got same error on new files downloading:

Same issue here, with same error.


ESP32 crashes when enabling a second serial port

Originally posted by nospamsam2011 January  9, 2023
Good day All,
I was curious if anyone else has experienced startup issues when creating second serial port on the ESP32?  I ask because as soon as I create a second serial port (Serial1 or Serial2), the ESP32 crashes.  The IO pins do not conflict with the HUB75 pins and so I figured before I start digging deep, I would see if anyone else had a similar experience and if so what they had to do to enable a second serial port.
Thanks in advance!

The link you provided indicates the the discussion has been deleted... Can you provide a different link or why was the discussion deleted?

Discussion has been moved to an issue

Can you provide a sample sketch that causes this crash so I can replicate / test?

Good day mrfaptastic,
My apologies for the response delay!  Digging deeper I found that the 2nd serial port issue is dependent upon the Rx/Tx pins selected.  If I use GPIO's 32/35 then the serial port crashes.  To rule out any issue with your library, I created a simple sketch which is simply the Serial and Serial2 ... and sure enough the GPIO Pins selected for Tx/Rx can crash the system.  Consequently, the Serial2 issue has nothing to do with your library.
My apologies for mistakenly correcting this issue!
Cheers,
Sam

As a follow up to this and if anyone else get's caught by this issue...  Although the ESP32 names pins as GPIO, some pins are input only.  Effectively GPIO Pins from GPIO34 and up can only be used as inputs.  In my case I had GPIO35 set as serial2's Tx (output) which caused the system crash.  This is annoying, as these pins should have been called GPI insted of GPIO... Sadly, this means a board change to remap the Tx pin to another pin... argh...
You can see the various pin functionality on Page 60 of the ESP32 technical Reference Manual (Revision 4.8).
Cheers,
Sam

Lots of useful information on this site ...
https://randomnerdtutorials.com/esp32-pinout-reference-gpios/
In particular that page for GPIO.
Cheers
Daz


WIP: PowerShell Archive Support

PowerShell Archive Support
On of biggest desired features of mine has always been a PSProvider for Archive/ZipFile Support in powershell. So for the past few months I have been working on creating a PSProvider that supports opening and active manipulation of ZipFiles.
Motivation
The ability to utilize a ZipFile like a FileSystemProvider opens the door for Automation components by allowing automated tasks to work directly inside an Archive instead of FileSystem.
This implimentaiton would allow Sys admins to Preview/Modify configuration of tools like Nuget or log files.
User Experience
Prelimnary code has been uploaded to my Github under PS1C
Current Code base is Previewable on:
https://github.com/romero126/ps1c

It can be checkedout on Gallery for Windows Only Currently;
Install-Module PS1C
Import-Module PS1C
New-PSDrive -Name TestDrive -PSProvider ZipFile -Root "ZipFile.Zip" -ErrorAction "Stop"
Push-Location TestDrive:\
# Go to town

Specification
The Desire to feel as close to the FileSystem Provider as possible sets a very natural feel to the user experience.
Testing Requirements
Code must maintain rigourous testing standards and must meet all Applicable FileSystem Provider based tests
Current Provider Varience and Language Constraints
Currently PSProviders locked down certain features/functionality natively.

Invoke-Item is locked down to the FileSystemProvider

Both & and .
Invoke-Command
Invoke-Expression

Get-Content -Tail is limited to FileSystemProvider because BackReader is not publicly available.

Example of Use
PS>Install-Module PS1C -Force
PS>Import-Module PS1C
PS>New-PSDrive -Name NewDrive -PSProvider ZipFile -Root "$Pwd\ZipFile.zip"

Name        Used (GB)  Free (GB) Provider   Root                                CurrentLocation
----        ---------  --------- --------   ----                                ---------------
NewDrive                         ZipFile    C:\Users\Romero126\Documents\Softw…

PS>Push-Location NewDrive:\
PS>Get-ChildItem

LastWriteTime             Length   Name                      FullName
-------------             ------   ----                      --------
8/12/2019 3:04:26 PM      50       _InvokeCommand.ps1        TestDrive:\_InvokeCommand.ps1
8/3/2019 9:18:32 PM       4        Binary.txt                TestDrive:\Binary.txt
9/26/2019 12:33:16 PM     17       dynamicfile.txt           TestDrive:\dynamicfile.txt
9/26/2019 12:33:16 PM     14       dynamicfile2.txt          TestDrive:\dynamicfile2.txt
8/3/2019 9:18:02 PM       191      Multiline.txt             TestDrive:\Multiline.txt
9/27/2019 12:11:08 PM     0        newDirectory              TestDrive:\newDirectory\
8/7/2019 10:42:52 PM      62       Test.txt                  TestDrive:\Test.txt

PS>Get-Content Test.txt
8/12/2019 10:07:17 PM : 8ab0b829-9b71-4c11-9cfa-33d1295901db
PS>("{0} : {1}" -f (Get-Date).ToString(), (New-Guid)) | Set-Content Test.txt -PassThru
9/27/2019 2:32:19 PM : 44ef9073-71ec-41c4-8013-a4ce5cc2832d
PS>Get-Content Test.txt
9/27/2019 2:32:19 PM : 44ef9073-71ec-41c4-8013-a4ce5cc2832d
PS>Get-Item NewTestFile.txt

LastWriteTime             Length   Name                      FullName
-------------             ------   ----                      --------
9/27/2019 2:34:02 PM      11       NewTestFile.txt           TestDrive:\NewTestFile.txt

PS>Get-Item NewTestFile.txt | Get-Content
NewItemData
PS>

Hey @anmenaga,
I am very interested in your feedback. What do you think about this PR?
Its getting quite a bit closer to a final product, I am very interested in if integrating this would be good.
Currently it works on PWSH Core only.


FEAT:  export/import the plugins configuration (aka. our prompts)

Adding a feature to export/import the configuration (aka. our prompts) should be great

Hi!
Thank you for the feature request. I will definitely consider it to improve the extension.

i am confused how "our prompts" are the same as "the configuration". I want to suggest a feature to let us export our chat history. I want to keep track of all my interactions with chatgpt so it would be nice to be able to export the chat history conversations in the same format that openai gives us when exporting data from their site.

@carlrobertoh Thanks for the awesome plugin! Configuration import and export can definitely work well for sharing the plugin and usage in other languages or occasions! Looking forward to new version with such functions!


Allow the injector control plane service to request any identity from sentry

It is required that we allow the injector to request any identity from sentry in Kubernetes mode so that it is able to inject the certificate and private key as environment variables for v1.11 daprds.
PR updates sentry so it will accept the injector requesting any certificate identity. Adds an option so that the Kubernetes validator signals for this certificate to have the largest possible validity duration which is needed as these certificates will be used by daprds for its entire duration.
A follow-up PR will include the changes in injector to actually make use of this change.
/cc @mukundansundar

/ok-to-test

/test-version-skew

/ok-to-test


[$500] when adding amount details to a failed smartscan, default currency isn’t local

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

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

Head to any chat
Create request money
Choose Scan with faulty picture (e.g. a book)
Wait till scan fails
Click on the Amount details required

Expected Result:
the amount to input would default to your local currency
Actual Result:
the amount to input defaults to USD$
Workaround:
unknown
Platforms:

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

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

Screenshots/Videos
Add any screenshot/video evidence
https://github.com/Expensify/App/assets/38435837/12b42847-8b4e-4bef-bf42-19c9f8bbfd08

View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01afd06e1036c5deb0
Upwork Job ID:<PHONE_NUMBER>099211776
Last Price Increase: 2023-12-18

Proposal
Please re-state the problem that we are trying to solve in this issue.
when adding amount details to a failed smartscan, default currency isn’t local
What is the root cause of that problem?
The component CurrencySymbolButton receives the property currencySymbol to show the currency symbol, this information comes from the EditRequestPage component. The EditRequestPage component uses the method ReportUtils.getTransactionDetails to retrieve the transaction details and one of the information is currency.
https://github.com/Expensify/App/blob/19ca74491ec7dc60bf8e3103058add651964533c/src/pages/EditRequestPage.js#L78-L85
To retrieve the currency, is used the methods from ReportUtils file: getTransactionDetails -> getCurrency, in the getCurrency there is a logic to retrieve the currency from the transaction object, and the value returned from the method getCurrency will be the information from the transaction property modifiedCurrency that has the default currency when the scanning fails.
https://github.com/Expensify/App/blob/19ca74491ec7dc60bf8e3103058add651964533c/src/libs/TransactionUtils.ts#L280-L286
What changes do you think we should make in order to solve the problem?
We should change the getCurrency to verify if the transaction has an error, if so, we need to get the currency from the currency property in the transaction object.
For example:
function getCurrency(transaction: OnyxEntry<Transaction>): string {
    const currency = transaction?.modifiedCurrency ?? '';
    const hasFieldErrors = hasMissingSmartscanFields(transaction);

    if (hasFieldErrors || _.isEmpty(currency)) {
        return transaction?.currency ?? CONST.CURRENCY.USD;
    }

    return currency;
}

What alternative solutions did you explore? (Optional)
N/A
POC
https://dev.new.expensify.com:8082/r/5473945671459503

This just needs a simple backend fix because once the scan fails, it sets the transaction.modifiedCurrency from '' to USD by the server. Also, the transaction.currency remains unchanged.
Check whether the backend is setting the transaction.modifiedCurrency to USD.
@rushatgabhane, could you confirm this?

Not exactly the same, but the currency is changed from the local one to USD here too, so I thought it good to mention the issue for reference sake.

https://github.com/Expensify/App/issues/29709

Bump on this one @rushatgabhane

@rushatgabhane - any update here?

Bumped in Slack - https://expensify.slack.com/archives/C01GTK53T8Q/p1703871901016549

@wlegolas 's proposal LGTM https://github.com/Expensify/App/issues/33258#issuecomment-1861932910
🎀 👀 🎀

hmm, @Tony-MK that is a good point, but I think that change would affect also the Old website leaving the currency empty, and that may break things there.

Sorry, I didn't know that.
Good job @wlegolas. You deserve it.
Thanks @aldo-expensify 😄

Hi everyone, I created the PR with the changes https://github.com/Expensify/App/pull/33853
If you have any questions or suggestions, please let me know.

This still stuck on me, I have done some progress in reproducing the original problem in dev. The Expensiwork/smartscan was a bit tricky to get to work.
I need to do more testing to decide where is the best place to fix this (backend/frontend).

Hi @aldo-expensify did you have time to execute your tests to check the better place to fix this issue?
If there is something that I can help, please let me know.

I found the code in the backend where we are setting the modifiedCurrency when smartscan fails and it is new code that was introduced for the new app. I don't think this behaviour existed for the Old app and it seems like a bug introduced by mistake. So I still think the right fix would be to stop setting the modifiedCurrency incorrectly instead of ignoring the bad currency in the frontend.
This issue is kind of low priority, so I have been getting pulled to other higher priority issues.
I think what we should do is:

I finds the right fix in the backend, and we close your PR
Pay you something (half?) because you still spent some time investigating, coding and testing your solution.

These don't have to necessarily happen in that order.

I found the code in the backend where we are setting the modifiedCurrency when smartscan fails and it is new code that was introduced for the new app. I don't think this behaviour existed for the Old app and it seems like a bug introduced by mistake. So I still think the right fix would be to stop setting the modifiedCurrency incorrectly instead of ignoring the bad currency in the frontend.
This issue is kind of low priority, so I have been getting pulled to other higher priority issues.
I think what we should do is:

I finds the right fix in the backend, and we close your PR
Pay you something (half?) because you still spent some time investigating, coding and testing your solution.

These don't have to necessarily happen in that order.

Hi @aldo-expensify thank you for sharing your results.
If the fix should be done by the backend, for me doesn't have a problem, I don't know how it's the process (just close the PR or pay something for my time), so I leave it open to you to understand the best way to close this issue.

Not resolved yet!

Reopening for handling some payment for @wlegolas for the work done in the PR: https://github.com/Expensify/App/pull/33853 (even if it wasn't merged because we preferred a backend solution in the end)

Payment summary:
We agreed to pay half of the original job price since the PR wasn't merged but the job was assigned and @wlegolas put work into it.

Contributor: $250 @wlegolas (Offer sent in Upwork)

@lschurr me too please, i had reviewed the issue and PR

Ah, thanks @rushatgabhane. I'll add you to the payment summary

All set.

$250 approved for @rushatgabhane

Hi folks.
Thank you so much for giving me this payment ❤


Add fingerprint to soft timeouts in Celery

Hmm, why is this much different than what @tkaemming attempted to do with https://github.com/getsentry/sentry/pull/2415/files? I don't remember why that didn't work out well. I just know it needed to be reverted.

This is why I write those verbose commit messages: getsentry/sentry@1a818cedc95e53f671d46c21d0c19aad79f5ee06

This had no effect, since SoftTimeLimitExceeded exceptions were being caught and logged by sentry.utils.safe.safe_execute without being reraised.

@mattrobenolt this helps everyone not just Sentry, and we can improve the situation on our end (not everything is wrapped in safe_execute)

Sorry, I meant, it didn't solve our issue.


Fix sig-storage flakiness: Use a smaller image for test

Tests enable all feature gates, so these tests enable the new
online resize. The disks get expanded to the largest size, causing
disk pressure that leaks into the next running test.
What this PR does / why we need it:
Example of failure due to disk pressure.
The previous running test was "[Serial]with NFS Disk PVC using ipv4 address of the NFS pod not owned by qemu".
Release note:

NONE

the same tests fail on both lanes, that is suspicious

/retest

/retest

Tests enable all feature gates, so these tests enable the new
online resize.

For completeness: in general also make sure that tests which require a feature gate have a skip statement in case a feature gate is not enabled. We strive for, but do not verify it right now, that the test suite can cope with deliberately globally disabled gates.

/retest

@maya-r does not look like this PR fixes the issue? The lanes fail permanently.

@rmohr it might not be sufficient, I think it's an improvement to avoid the disk pressure issues on the nodes. @oshoval has proposed to ignore the transient warning seen in the failures in https://github.com/kubevirt/kubevirt/pull/6761

/cherry-pick release-0.44


Modengine 2, which brings up the cmd that closes automatically approx 100 milliseconds after launching it.

I have a problem similar to some of the issue i saw, but here's what I have:
for the past few hours, I've been trying to find solutions to my problem, but nothing (via Steam)
To be honest, I'm completely lost with all the answers I've read
If someone can help, I'm desperate.
Issue:

Game file Location : "C:\steam\steamapps\common\ELDEN RING\Game"

"modengine2" Location : "C:\Users\yura\OneDrive\Рабочий стол\ModEngine-<IP_ADDRESS>-win64"
Рабочий стол = (Dekstop) in RU

how my "config_eldenring" look like :
mods = [
{ enabled = true, name = "default", path = "mod" }
]

Move all the content in the same folder with ME2 launcher to the game's directory, change ME2 launcher's name to "start_protected_game.exe" (backup the original one), set it to launch as admin and run it.

Move all the content in the same folder with ME2 launcher to the game's directory, change ME2 launcher's name to "start_protected_game.exe" (backup the original one), set it to launch as admin and run it.

"I'm sorry, but if I understand correctly, I should take the first folder ("ModEngine-<IP_ADDRESS>-win64")
to put it where the game files are located in ("C:\steam\steamapps\common\ELDEN RING\Game")? If not, I'm not sure which launcher you're referring to specifically. Excuse me."

Just place all the ME2 folder’s contents inside of the game’s directory (…ELDEN RING\Game) and rename the modengine2_launcher to “start_protected_game.exe”
I had the same exact issue as yours, we even share the devices language.

Just place all the ME2 folder’s contents inside of the game’s directory (…ELDEN RING\Game) and rename the modengine2_launcher to “start_protected_game.exe”
I had the same exact issue as yours, we even share the devices language.

omg tysm for your help that ' s finnaly working 💯 Thank you for being able to spare some of your time.


README: add missing maintainers.

Sorry I missed you until now @Bo98 and @zachauten!

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.

[x] Have you successfully run brew tests with your changes locally?

May I humbly ask to be listed as well 🙄

@bayandin Whoops, yes, sorry. This process is far too manual, my apologies.

Thanks!


<EMAIL_ADDRESS>Vulnerability within<EMAIL_ADDRESS>

Is this repo still being maintained?
nyc has the following dependency tree based on semver
<EMAIL_ADDRESS>├─┬<EMAIL_ADDRESS>│ ├─┬<EMAIL_ADDRESS>│ │ ├─┬<EMAIL_ADDRESS>│ │ │ └──<EMAIL_ADDRESS>│ │ └──<EMAIL_ADDRESS>│ └──<EMAIL_ADDRESS>└─┬<EMAIL_ADDRESS>└──<EMAIL_ADDRESS><EMAIL_ADDRESS>has a vulnerability
https://github.com/advisories/GHSA-c2qf-rxjj-qqgw

No response? 👀

I also need a fix .. any update ?

@coreyfarrell are you able to comment if this repo is still maintained?

Did anyone find any alternative?

@jaws97 After realizing that nyc is not really maintained any longer, our project switched to C8
https://github.com/bcoe/c8


Need help with S3 to MySQL to Reshift

I am trying to present few tools like Airflow as POC for Data Orchestration to incorporate into few projects
Sample:
I got python script which does collecting files from S3 --> Validates --> loads into staging into MySQL --> Runs few Procs ---> Calls R packages for Analytics ---> final tables Dumps into Redshift
Can you please point some examples how to drive above activities connecting using Airflow.

Please use the mailinglist for these kind of questions


Query<->Mutation Chicken<->Egg problem

A painful paradigm I'm coming across is how sometimes you need to call a mutation before you query that same data.
This is a problem because mutations need to return something (although that could change depending on the response for https://github.com/graphql/graphql-js/issues/410.
The options we have are:

Return something & virtually ignore it. The query that comes later will get that information & store it.
Always query before a mutation is called.

@jordanh any other viable patterns you see?

For now, the cleanest pattern is to make the type a scalar, like Boolean. That way, we don't have to return anything. In doing so, we don't have to listen for that return prematurely. See createTeam in parabolinc/action


Remove --use-mirrors from pip command

As this is not supported, and so breaks Travis tests.

This is failing due to issues with html5lib whose latest version currently requires a minimum version of setuptools; I think they've fixed that recently, so this should be revisited when they release a new version.


Track Purge count in ReversePurgeItemHashMap

I want to add a purgeCount field to the the ReversePurgeItemHashMap.
I also want to expose this in the ItemsSketch object
My use case is this: I want to be able to tell if my Stream is an enum (i.e. contains < k distinct values). This can be achieved by creating a ItemsSketch with capacity k, and then seeing if the map is ever purged.
I am happy to implement this; what do you guys think?

The ItemsSketch depends on the hashCode() to determine uniqueness.  Why not just extract the hashCode() of each item and feed it to a Theta UpdateSketch of size k.  If the # of unique items is < k, the answer is exact.  If greater than k, it will be approximate.  This will be much smaller, faster and much more accurate (if # > k). No need to rewrite anything.
If you also want to track the counts of number of duplicates of each item, then use a Tuple Sketch.
It is not clear why you are focusing on the FIS?

The number of purges is an implementation detail. I don't think it should be exposed, and I don't think that you can draw any conclusions from the number. It seems that you don't really need the number, but need to know if any purges happened at all. For this you can use getMaximumError(). If it is zero, than no purges happened (given the current implementation).
On the other hand, as Lee suggested, perhaps, you just want a Theta sketch to estimate the cardinality of your stream.

I'm confused about your idea. If I feed N items into a FIS with capacity k, won't the underlying map get purged down to size k? I want to be able to determine if the map has been purged or not.
One other thought: I see that the ItemSketch provides the streamLength; it seems like if the sum of the frequency estimates equals the streamLength, then the frequent items span the stream? Or does the fact that the frequency estimates are approximate make this a bad idea?
I was focusing on the FIS out of selfishness; this could make sense elsewhere.

Ah ok, yes that is perfect! Thanks

So you must be using the FIS for other reasons ?

I would argue that for cardinality estimation the right tool is Theta sketch. If your sketch size is K, and the cardinality happens to be not more than K, the sketch will be in the exact counting mode, and you can learn this fact from the sketch by calling isEstimationMode(). If this method returns false (meaning that the sketch is not in estimation mode yet) the lower and upper bounds will be equal to getEstimate()

I see. That seems like a viable alternative. It seems a bit odd that the theta sketch API supports this isEstimationMode, but that the FrequentItems doesn't have a similar boolean for IsExact (although I see how the error can be interpreted to achieve this).
You're also right; I'm using the FIS to get frequent items, and perhaps I'm overloading it a bit in order to get a cardinality estimation out of it.
I'm going to pursue the FIS maxError approach and will validate it's behavior (i.e. maxError == 0 iff no purges occur).

maxError == 0 iff no purges occur

That is a fact given the current algorithm. Purge means subtracting the mean value (estimated) from all the counts, keeping entries with positive counts only and increasing the max error by that amount. So purge always increases max error.

Great. I'm going to close this out.

I'm using the FIS to get frequent items, and perhaps I'm overloading it a bit in order to get a cardinality estimation out of it.

This is a limited cardinality estimator. If getMaxError() == 0, cardinality is exactly getNumActiveItems(), but if getMaxError() > 0, you have no idea.


[Bug]: Gradio gets a ERROR:    Exception in ASGI application

What happened?
ERROR:    Exception in ASGI application
Console logs
ERROR:    Exception in ASGI application
Traceback (most recent call last):
  File "/usr/local/lib/python3.9/dist-packages/uvicorn/protocols/websockets/websockets_impl.py", line 254, in run_asgi
    result = await self.app(self.scope, self.asgi_receive, self.asgi_send)
  File "/usr/local/lib/python3.9/dist-packages/uvicorn/middleware/proxy_headers.py", line 78, in __call__
    return await self.app(scope, receive, send)
  File "/usr/local/lib/python3.9/dist-packages/fastapi/applications.py", line 271, in __call__
    await super().__call__(scope, receive, send)
  File "/usr/local/lib/python3.9/dist-packages/starlette/applications.py", line 125, in __call__
    await self.middleware_stack(scope, receive, send)
  File "/usr/local/lib/python3.9/dist-packages/starlette/middleware/errors.py", line 149, in __call__
    await self.app(scope, receive, send)
  File "/usr/local/lib/python3.9/dist-packages/starlette/middleware/gzip.py", line 26, in __call__
    await self.app(scope, receive, send)
  File "/usr/local/lib/python3.9/dist-packages/starlette/middleware/exceptions.py", line 79, in __call__
    raise exc
  File "/usr/local/lib/python3.9/dist-packages/starlette/middleware/exceptions.py", line 68, in __call__
    await self.app(scope, receive, sender)
  File "/usr/local/lib/python3.9/dist-packages/fastapi/middleware/asyncexitstack.py", line 21, in __call__
    raise e
  File "/usr/local/lib/python3.9/dist-packages/fastapi/middleware/asyncexitstack.py", line 18, in __call__
    await self.app(scope, receive, send)
  File "/usr/local/lib/python3.9/dist-packages/starlette/routing.py", line 706, in __call__
    await route.handle(scope, receive, send)
  File "/usr/local/lib/python3.9/dist-packages/starlette/routing.py", line 341, in handle
    await self.app(scope, receive, send)
  File "/usr/local/lib/python3.9/dist-packages/starlette/routing.py", line 82, in app
    await func(session)
  File "/usr/local/lib/python3.9/dist-packages/fastapi/routing.py", line 289, in app
    await dependant.call(**values)
  File "/usr/local/lib/python3.9/dist-packages/gradio/routes.py", line 436, in join_queue
    session_info = await websocket.receive_json()
  File "/usr/local/lib/python3.9/dist-packages/starlette/websockets.py", line 133, in receive_json
    self._raise_on_disconnect(message)
  File "/usr/local/lib/python3.9/dist-packages/starlette/websockets.py", line 105, in _raise_on_disconnect
    raise WebSocketDisconnect(message["code"])
starlette.websockets.WebSocketDisconnect: 1006

Is there an existing issue for this?

[X] I have searched the existing issues and checked the recent builds/commits

Are you using the latest version of the colab notebook?

[X] Yes, i am using the latest version from the repo

Which colab notebook did you use?
abyss_orange_mix_3_webui_colab
Which repo branch did the colab you use comes from ?
v2.0
Which tunnel did you use ?
--share (Gradio Tunnel)
What browsers do you use to access the WebUI ?
Google Chrome

what were you doing before this error happened? generating stuff? txt2img? img2img?

img2img, or any options in that tab.

for img2img, maybe decrease the resolution a little bit. For png info, use this extension.
We'll figure out the problem soon.

thanks for the info @BunE-exe ❤ gradio fixed connection error

In mi case I'm facing the same error!

@blackpro98

Hi @blackpro98 👋 please tell me  which type of colab is giving you the error lite stable or nightly ?


PowerShell sample bug

Hi, adding the Post-LogAnalyticsData function into an Azure Automation runbook produced an "Object reference not set to an instance of an object." when executing the Invoke-WebRequest.
Initialising $TimeStampField = [DateTime]::UtcNow.ToString("r") variable fixed this issue for me.
Suggest updating the Post-LogAnalyticsData Powershell sample.
Thanks John

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

ID: 1295e1b3-5aed-3aee-4331-8b81e7a7e544
Version Independent ID: 3c00a5ee-7093-1e8b-9004-8fed3f82960e
Content: Azure Monitor HTTP Data Collector API - Azure Monitor
Content Source: articles/azure-monitor/platform/data-collector-api.md
Service: azure-monitor
Sub-service: logs
GitHub Login: @bwren
Microsoft Alias: bwren

@JBiasilloTM Thanks for the feedback. We shall evaluate and update the document as appropriate.

@JBiasilloTM I see that the $TimeStampField is initialized to an empty string in the quoted sample, due to which I did not encounter the error while executing the sample as is.

Do you think we can still improve it further?

Hi
The code works OK when running from my desktop powershell session too but I got the “object not set …”issue when running from Azure Automation runbook so there’s something different in the desktop and automation hosting environment.
Did you try from running the code from an Azure Automation runbook?
Kind regards,
John Biasillo / Senior Cloud Consultant
M: 07599 549208
E: john.biasillo<EMAIL_ADDRESS>From: BhargaviAnnadevara-MSFT<EMAIL_ADDRESS>Sent: 18 December 2020 06:36
To: MicrosoftDocs/azure-docs<EMAIL_ADDRESS>Cc: John Biasillo<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/azure-docs] PowerShell sample bug (#67817)
@JBiasilloTMhttps://github.com/JBiasilloTM I see that the $TimeStampField is initialized to an empty string in the quoted samplehttps://docs.microsoft.com/en-us/azure/azure-monitor/platform/data-collector-api#powershell-sample, due to which I did not encounter the error while executing the sample as is.
[image]https://user-images.githubusercontent.com/44050820/102582750-3c88e080-4129-11eb-93cc-4291f8436465.png
Do you think we can still improve it further?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/azure-docs/issues/67817#issuecomment-747899679, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ARBTIMGBGBKCWLWUVKASCHTSVLZ4JANCNFSM4U7HKO3Q.
Trustmarque Classification: Confidential
This email is sent on behalf of Trustmarque Solutions Limited a wholly owned subsidiary of Capita plc. Trustmarque is a company registered in England and Wales. Company Number: 02183240. Registered Office: 65 Gresham Street, London EC2V 7NQ. VAT Number: 618184140 GB.
This email is intended solely for the addressee, is strictly confidential and may also be privileged. It is for the exclusive use of the intended recipient(s) only, and must not be re-transmitted in an amended form without our consent. If you are not the addressee please do not read, print, re-transmit, store or act in reliance on it or any attachments. Instead, please reply to this email and then immediately permanently delete it.
Security and reliability of emails are not guaranteed. We operate anti-virus programs but you must take full responsibility for virus checking this email (including all attachments). Capita plc and its subsidiaries do not accept any liability in respect of any damage caused by any virus which is not detected. Capita plc and its subsidiaries may monitor email traffic data and also the content of emails for compliance purposes and to protect its business.

@JBiasilloTM Yes, I was able to run it without any errors both from my machine and a PowerShell automation runbook.
I copied the PS snippet into the runbook as is and replaced the $CustomerId and $SharedKey with my LA Workspace ID and Primary Key, respectively, and executed it. The run completed successfully with a 200 status code.
I was able to reproduce this error as soon as I commented the line that initialized the $TimeStampField variable:
$TimeStampField = ""

Could you please run it passing the -Debug parameter to the Invoke-WebRequest cmdlet this time? Do let me know what you see.

Hi
I think I see how the issue has come about.  I added the Post-LogAnalyticsData as a nested function in my Automation runbook.  The code sample initialises $TimeStampField variable outside of the function (but I didn’t do this in my “main” runbook processing).  I think initialising the $TimeStampField variable inside the function or adding a $TimeStampField function parameter would may the code more readably / robust (or at least make it less likely for someone else making a Post-LogAnalyticsData copy & paste error 😊).
Thanks for your responses.
Kind regards,
John Biasillo / Senior Cloud Consultant
M: 07599 549208
E: john.biasillo<EMAIL_ADDRESS>From: BhargaviAnnadevara-MSFT<EMAIL_ADDRESS>Sent: 18 December 2020 13:55
To: MicrosoftDocs/azure-docs<EMAIL_ADDRESS>Cc: John Biasillo<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/azure-docs] PowerShell sample bug (#67817)
@JBiasilloTMhttps://github.com/JBiasilloTM Yes, I was able to run it without any errors both from my machine and a PowerShell automation runbook.
I copied the PS snippet into the runbook as is and replaced the $CustomerId and $SharedKey with my LA Workspace ID and Primary Key, respectively, and executed it. The run completed successfully with a 200 status code.
I was able to reproduce this error as soon as I commented the line that initialized the $TimeStampField variable:
$TimeStampField = ""
Could you please run it passing the -Debug parameter to the Invoke-WebRequest cmdlet this time? Do let me know what you see.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/azure-docs/issues/67817#issuecomment-748096509, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ARBTIMEKYWDPKIUUDTHUMMTSVNNL5ANCNFSM4U7HKO3Q.
Trustmarque Classification: Confidential
This email is sent on behalf of Trustmarque Solutions Limited a wholly owned subsidiary of Capita plc. Trustmarque is a company registered in England and Wales. Company Number: 02183240. Registered Office: 65 Gresham Street, London EC2V 7NQ. VAT Number: 618184140 GB.
This email is intended solely for the addressee, is strictly confidential and may also be privileged. It is for the exclusive use of the intended recipient(s) only, and must not be re-transmitted in an amended form without our consent. If you are not the addressee please do not read, print, re-transmit, store or act in reliance on it or any attachments. Instead, please reply to this email and then immediately permanently delete it.
Security and reliability of emails are not guaranteed. We operate anti-virus programs but you must take full responsibility for virus checking this email (including all attachments). Capita plc and its subsidiaries do not accept any liability in respect of any damage caused by any virus which is not detected. Capita plc and its subsidiaries may monitor email traffic data and also the content of emails for compliance purposes and to protect its business.

@JBiasilloTM Oh, thanks for the additional context. I hope you would be able to get it to work now, but please feel free to paste your code here if you need any further help in ironing out the issue. :)

@JBiasilloTM Hope the above inputs help. Are there any further questions here?

@JBiasilloTM We will now proceed to close this thread as resolved, but if there are any further questions regarding the documentation, please tag me in your reply and we will be happy to continue the conversation.


MiddlewareFunc return value

modify  #66

Thanks your pr, but need to inform: middleware returning the next function is necessary

      let h = handler;
      let temp = h;
      for (const m of middleware) {
        h = m(h);
        if (h(c) === void 0) {
          h = temp;
        } else temp = h
      }

how does it looks

@wk989898 We shouldn't call h. It is a next function

if middleware returning undefined ?

if middleware returning undefined ?

if m(h) returning undefined ,h will be undefined

@wk989898 Even if the handler exists, the context cannot be passed on correctly, so this is meaningless

you are riht,maybe deepCopy  could solve it .

@wk989898 May #69 is related
We should find a way to bypass the handler and pass the context.

      let h = handler;
      let MiddlewareStack = []
      let temp
      for (const m of middleware) {
        h = m(h);
        MiddlewareStack.push(h)
      }
      while (MiddlewareStack.length > 0) {
        h = MiddlewareStack.pop() as HandlerFunc
        if (temp = h(c)) return temp;
      }
      return handler(c)

what do you think

I don't think it's necessary to continue the discussion now, in short middleware must return next function.


[Suggestion]: Request to add compatibility with kubejs mod

General Info

[ ] I would like to propose new content
[ ] I would like to propose changes
[X] I would like to propose another thing

Suggestion Description
I like dimensional painting, can you add compatibility with kubejs please? So that kubejs can customize textures, transfer locations, transfer worlds...

I would also like to see this compatibility added


Add documentation for cpp

@AlirezaMorsali Could you go through all the comments I added and ensure they are reasonable? Most of them are from the Python doc. It would be nice if you can format the display in the html into a better shape, e.g. "bold" some keywords or whatever.

You can add some style using the Javadoc special commands.
Here is another link for more info on that.
In particular with @papram you can specify parameter, @return the return values and @see.

@AlirezaMorsali Could you do a proofread for the documentation and comments what need to improve? The current documentation is already complete without extra formatting.

After modification:

@AlirezaMorsali Do you think this PR is ready to merge?

I tired to add links to see also but it required more work. Also, we may need to add more explanations which we can do in another go.
I think it's a good start for now.


inner_loop assumes the use of stateful metrics

Hi Christian,
I've been loving this library – it makes quickly spinning up a training loop so much easier and really reduces the amount of boilerplate code I'm writing.
Perhaps I'm not using it correctly, but I think that the current implementation of inner_loop implicitly assumes that the metrics being used are stateful. In particular, this code
logs = log_history[-1] if len(log_history) > 0 else Logs()

only makes sense if the metrics are stateful, and the last entry added to the history is a value that represents the final version of the metric after accumulating over the whole test dataset and then computing the result. If metrics are not stateful then this results in taking only the last batch-wise estimates. (This behavior is especially problematic when the last batch is smaller than the rest!)
Have I missed an obvious way around this issue? Happy to provide some minimum working examples if that would help.
If you think that this is an actual issue, I'd be happy to take a stab at a fix. Perhaps adding an aggregate option to inner_loop.

Hey @JamesAllingham , thanks for reporting this. You are right, this assumes metrics (and all other collections in Logs) are stateful. Metrics from both clu.metrics and jax_metrics (are supported) and are stateful which is mostly what I've used.
Feel free to send a proposal / PR, happy to review it.


DTSSTCI 353

Jira link (if applicable)
https://tools.hmcts.net/jira/browse/DTSSTCI-357
Change description
Make changes to upload documents page

Checklist

[ ] commit messages are meaningful and follow good commit message guidelines
[ ] README and other documentation has been updated / added (if needed)
[ ] tests have been updated / new tests has been added (if needed)
[ ] Does this PR introduce a breaking change

Closed - will merge this work in the same PR as DTSSTCI-450 (https://github.com/hmcts/sptribs-dss-update-case-web/pull/101)


add  Δ° pint preprocessor

Changes

add text preprocessor for handling Δ° in unit notation

I have tested it for  Δ°C, Δ°F and Δ°Re but I think we should stick to the long serialization format for now @marscher @vhirtham

I'm just curios why this preprocessor is mandatory, if we do not even request to export a "delta" unit in the first place.

I'm just curios why this preprocessor is mandatory, if we do not even request to export a "delta" unit in the first place.

just for convenience so we can use e.g. Q_(150, "Δ°C") since the default format everywhere outside of serialization is still 'short' so Δ° might still pop up in console outputs, plots etc.

Shouldn't we add a test for preprocessors? I think at least the delta case is not covered yet.

Shouldn't we add a test for preprocessors? I think at least the delta case is not covered yet.

good idea, I will integrate it in one of the other testcases

please note the changes I made to unit checking in weldx/tests/test_core.py @vhirtham
using .is_compatible_with for unit checking looks the most straight forward to me


Add Plek draft prefixes

This commit adds env vars for draft-content-store, draft-government-frontend, draft-manuals-frontend and draft-specialist-frontend to direct Plek to prefix all requested hostnames with “draft-“ to ensure they point to the draft versions of the respective apps.

looks good to me too! 👍


Tool for checking storage slots before a deploy

We had an incident where, due to a mistake, storage slots were shifted during the deployment of a new contract. Writes were then issued and caused a data corruption.
To prevent this from happening in the future, we should have a tool we run prior to a deployment to check the storage slots of the current contracts on the master branch vs the ones of the contracts on Mainnet.

CC @DanielVF @tomlinton

Hardhat does some kind of upgrade checking here in this module.
https://hardhat.org/plugins/hardhat-upgrades.html

A relevant article: https://blog.coinbase.com/usdc-v2-upgrading-a-multi-billion-dollar-erc-20-token-b57cd9437096 with pointers to USDT storage slots unit tests.

A relevant article: https://blog.coinbase.com/usdc-v2-upgrading-a-multi-billion-dollar-erc-20-token-b57cd9437096 with pointers to USDT storage slots unit tests.

If the hardhat stuff is good enough, great.  An alternative might be a simple tool using the antlr4 parser and looking for storage def changes.

If the hardhat stuff is good enough, great.  An alternative might be a simple tool using the antlr4 parser and looking for storage def changes.

Solution for this here: https://github.com/OriginProtocol/origin-dollar/pull/536

Solution for this here: https://github.com/OriginProtocol/origin-dollar/pull/536


Handle the operator policy message changes

Specifically, the CSV name is now listed in the message.

/hold for WIP status

/hold for WIP status

It's WIP since I didn't test it locally, so I'm letting CI validate it :laughing:

This is blocked by https://github.com/open-cluster-management-io/config-policy-controller/pull/234.

/retest

/unhold

@JustinKuli this is ready for review now


refactor and make less tornado

Following #393 , this moves some common code into Source, and takes the chance to make some of the sources async/await rather than tornado (the loop is still tornado running asyncio for now...). Kafka sources not touched, since they are more complex.

Should it always be await asyncio.gather(*self._emit), and if yes, should this be folded into emit itself? _emit returns a list, and that list will be all coroutines if the pipeline has a loop - which all Sources now enforce.

cc @roveo

Nice! This won't even break any existing custom sources, because most of them implement their own .start() method.

Should it always be await asyncio.gather(*self._emit)

This looks like an idiom that should be hidden from whoever is using this API. Maybe Source could implement custom .emit() that's always async (and doesn't require asynchronous=True).
Something like
async def emit(self, x, metadata=None):
    await asyncio.gather(*self._emit(x, metadata=metadata))

and then one will just
await self.emit(x) in _run implementation.
But then other node types will still have to use asynchronous=True, which is not uniform.  It would be nice to have one emit for synchronous/tornado coro emitting and one for async.

@chinmaychandak , please see the small kafka changes here, required to get things passing again.

LGTM!


Prepend after any tags

Hi,
I use the first few lines of my notes for plain text tags. Wondering if it could be possible to skip any lines that contain tags before prepending to a note?
Thanks 😊

I think that makes the api too complex and a bit too specific to your individual use case. So I probably won't implement this unless I see a greater need in this. I will leave this issue open though to allow others to support this feature request.


Use time-style only if time is provided

Closes #6169
This is the output as expected
└─$ target/debug/coreutils du -s --time-style=banana
2449284 .

└─$ target/debug/coreutils du -s --time --time-style=banana
du: invalid argument 'banana' for 'time style'
Valid arguments are:
- 'full-iso'
- 'long-iso'
- 'iso'
Try 'target/debug/coreutils du --help' for more information.

└─$ target/debug/coreutils du -s --time --time-style=iso
2449476 2024-04-01      .

The changes in [tests/by-util/test_env.rs](https://github.com/uutils/coreutils/pull/6170/files#diff-6846bdbc752ed473a430a7b6a01e94472b73373c35685c50841a69cc2d658525) have been done by cargo fmt.

could you please add a test to make sure we don't regress ?
thanks

Sure, I will try to write a test.

Should the test be in here https://github.com/uutils/coreutils/blob/1bb5111cf70a944b089232b3cfabb6403cbd2e0a/src/uu/du/src/du.rs#L1111 or in this https://github.com/uutils/coreutils/blob/main/tests/by-util/test_du.rs file and how do I run test on my local machine cargo test test_invalid_time_style ?

I think this test only really makes sense as a high-level integration test, i.e. "just call it and see what happens". Tests like that should go into …/by-util/test_du.rs, you're right! :)
cargo test has slightly weird semantics in how it interprets things. There are many correct ways to test it, I suggest that you call it like cargo test test_du in this case. Experiment around, that'll be more helpful than trying to explain it :)

You can run the du tests with cargo test --features "du" --no-default-features

I tried the tests on previous and new code the previous code fails and new one passes the tests.
Old

New

I say this comment not as a uutils-maintainer but a private person:
Have you noticed how easy-to-read and self-contained the changeset is? Have a look: https://github.com/uutils/coreutils/pull/6170/files
In fact, it's easier to read than the individual commits, which fix each other and undo prior things!
So if someone comes along later and wants to understand the history of uutils (e.g. to trace down a bug, or to review a PR, or just pure curiosity), then it would have been easier if the PR were just a series of atomic commits (in this case: a single commit). Also, there's this thing called "bisecting", which is essentially binary search on the history of a project. Bisecting can answer a question like "Which commit introduces the bug that causes XYZ to fail?" in an incredibly short time! However, it has a much harder job if the intermediate commits do not compile. :(
For these reasons (and probably more), I'm a big fan of rebasing PRs before they are merged. (Perhaps I do it a bit excessively.) This project doesn't seem to care too much about it, which is why I say this as a private person, not as a maintainer:
Please take a look at man git-rebase, and try it out yourself! :)
In this case, if you still have the branch checked-out, you could do git rebase -i 1bb5111cf70a944b089232b3cfabb6403cbd2e0a (the commit hash is needed here because origin/main now includes your PR, so trying to rebase it on the actual main-branch would be meaningless.)

I say this comment not as a uutils-maintainer but a private person:
Have you noticed how easy-to-read and self-contained the changeset is? Have a look: https://github.com/uutils/coreutils/pull/6170/files
In fact, it's easier to read than the individual commits, which fix each other and undo prior things!
So if someone comes along later and wants to understand the history of uutils (e.g. to trace down a bug, or to review a PR, or just pure curiosity), then it would have been easier if the PR were just a series of atomic commits (in this case: a single commit). Also, there's this thing called "bisecting", which is essentially binary search on the history of a project. Bisecting can answer a question like "Which commit introduces the bug that causes XYZ to fail?" in an incredibly short time! However, it has a much harder job if the intermediate commits do not compile. :(
For these reasons (and probably more), I'm a big fan of rebasing PRs before they are merged. (Perhaps I do it a bit excessively.) This project doesn't seem to care too much about it, which is why I say this as a private person, not as a maintainer:
Please take a look at man git-rebase, and try it out yourself! :)
In this case, if you still have the branch checked-out, you could do git rebase -i 1bb5111cf70a944b089232b3cfabb6403cbd2e0a (the commit hash is needed here because origin/main now includes your PR, so trying to rebase it on the actual main-branch would be meaningless.)

Actually, I was relying on you to merge pr with rebase(This option is available on github when merging prs). But I will keep it in mind in future.

And you're absolutely right in relying on that! It's my fault for not clicking that button; I have talked with the maintainers in the meantime, and will use it in the future. :)


Windows installer does not include clangd-indexer and clangd-index-server

The windows installer at https://releases.llvm.org/ does not include clangd-indexer and clangd-index-server
System information
Output of clangd --version: 16.0.0
Editor/LSP plugin: N/A
Operating system: Windows 64 bit

We have our own releases of clangd-indexer and clangd-index-server in the clangd_indexing_tools package on our releases page, e.g. https://github.com/clangd/clangd/releases/tag/18.1.3.
I don't think anyone has proposed adding them to the llvm-project release packages. Such a proposal could be made, but given that those packages are pushing 1GB in size, there might be pushback to adding in optional binaries of a subproject like these.

I just saw a post on the forum about 3th party executables, so I'll raise it there.
I'm fine with closing this without any changes as some exes are available even though they are hard to be discovered


cjs import problem

I am trying to import this in a node.js environment for later call in a jquery context. But simple import doesn't work. Please help!

@shurinskiy  , where is $ defined here ? , you need to do import $ from 'jquery'

$ was defined earlier, by webpack externals. This works for all imports except rateyo.


Bandyc

I finished drawing a pretty house and answered all the questions, can I be pulled please? 😅

Excellent job Cody! You'll need to do two unwritten down tasks as your last steps:

Put your PDF of your Google Doc in the folder "PDFs"
Put your hw03_questions.md file in the "Answers" folder, and make sure it is named bandyc_hw03_questions.md


chore(deps): update dependency com.google.cloud:google-cloud-shared-config to v1.9.0

…onfig to v1.9.0
Thank you for opening a Pull Request! Before submitting your PR, there are a few things you can do to make sure it goes smoothly:

[ ] Make sure to open an issue as a bug/issue before writing your code!  That way we can discuss the change, evaluate designs, and agree on the general idea
[ ] Ensure the tests and linter pass
[ ] Code coverage does not decrease (if any source code was changed)
[ ] Appropriate docs were updated (if necessary)

Fixes #<issue_number_goes_here> ☕
If you write sample code, please follow the samples format.

close in favor of https://github.com/googleapis/google-auth-library-java/pull/1434


Chapter 06 - filteredList

You start the section of code with a ternary on reminders -
const filteredList = reminders ? [...reminders] : undefined;
Then you then redefine filteredList with a call to the function
const filteredList =  filterList(reminders, selectedFilter)
In the repo code the first definition is removed.  The problem is if no reminders an error is thrown plus we are still calling to RemindersList component.
Outside of initializing reminders with an empty array (and than bypassing Proptypes defult) what can be done ?
Unless I misread something this works -
 const filteredList = reminders     ? filterList(reminders, selectedFilter)     : undefined;

Good catch! Yeah, my first instinct was to initialize reminders with an empty array, but that defeated what I wanted to do with the default prop. Anyway, I like your solution and I'll update the repo. Thanks


Parameterize/inject config from gradle

Expected Behavior
I want to apply the same detekt.yml config file to all subprojects in my project, but update/override the parameters to the applied rules, on a per-module basis.
Current Behavior
Currently, there seems to be no way to express in the config file that I want each subproject to have basically the same ruleset, but with some sub-project-specific parameters - such as rootPackage.
My options seem to be to either disable the new InvalidPackageDeclaration rule, introduce with 1.21-RC1, or to to separately apply the plugin to each sub-project, copy-pasting the entire config file, just to change that one parameter. Of course, I can write custom code to merge/template my detekt config.yml, but it seems to me that a way to inject parameters by subproject would broadly useful.
Perhaps, something like:
tasks.withType<io.gitlab.arturbosch.detekt.Detekt>().configureEach {
  detekt.rule("InvalidPackageDeclaration").parameter("rootPackage").set("$group.${project.name}")
}

Context
In my project, we follow the kotlin coding conventions and our "directory structure follows the package structure with the common root package omitted". Each sub-project in the project has a different common root pacakage. When applying the new InvalidPackageDeclaration rule, I want to supply a different rootPackage for each sub-project.
Further considerations
When using the IntelliJ detekt plugin, I would want to be sure that IntelliJ could also recognise the injected rule parameters.
Am I missing a simpler fix here?

Ok, so I've realised that there is indeed a simpler fix.  I can have a config file in the root directory and then override it at the submodule level with a minimal config file. e.g.

a/build.gradle.kts

detekt {
    config = files("../config/detekt/detekt.yml", "config/detekt/detekt.yml")
}

/config/detekt/detekt.yml

<snip>
naming:
  active: true
<snip>
  FunctionParameterNaming:
    active: true
    parameterPattern: '[a-z][A-Za-z0-9]*'
    excludeClassPattern: '$^'
    ignoreOverridden: true
  InvalidPackageDeclaration:
    active: true
#    rootPackage: ''
    requireRootInDeclaration: true
  LambdaParameterNaming:
    active: false
    parameterPattern: '[a-z][A-Za-z0-9]*|_'
<snip>

a/config/detekt/detekt.yml

naming:
  InvalidPackageDeclaration:
    rootPackage: 'com.example.a'

b/config/detekt/detekt.yml

naming:
  InvalidPackageDeclaration:
    rootPackage: 'com.example.b'

This solves the issue for me, so feel free to disregard. :-)

Not a nice thing to require Detekt config in each of the potentially many modules.

Gradle gives you plenty of options to don't repeat that configuration. The easiest:
subpojects {
  detekt {
    // Your magic here
  }
}