[bug] android WRY_ANDROID_REVERSED_DOMAIN is not set

Describe the bug
fresh install, android dev error "WRY_ANDROID_REVERSED_DOMAIN` is not set"
Reproduction
❯ cargo create-tauri-app
✔ Project name · tauri-app
✔ Choose your package manager · cargo
✔ Choose your UI template · vanilla
❯ cd tauri-app/src-tauri
❯ cargo tauri android init
Installing Android toolchains...
info: component 'rust-std' for target 'aarch64-linux-android' is up to date
info: component 'rust-std' for target 'armv7-linux-androideabi' is up to date
info: component 'rust-std' for target 'i686-linux-android' is up to date
info: component 'rust-std' for target 'x86_64-linux-android' is up to date
Generating Android Studio project...
Info "/home/aminassian/programs/tauri-app/src-tauri" relative to "/home/aminassian/programs/tauri-app/src-tauri/gen/android/tauri_app/tauri_app" is "../../../../"
Info detected host target triple "x86_64-unknown-linux-gnu"
victory: Project generated successfully!
Make cool apps! 🌻 🐕 🎉
❯ cargo tauri android dev
Compiling num_enum v0.5.7 [41s]
error: failed to run custom build command for wry v0.23.4
Caused by:
process didn't exit successfully: /home/aminassian/programs/tauri-app/src-tauri/target/debug/build/wry-3c0c9f18658876a0/build-script-build (exit status: 101)
--- stdout
cargo:rerun-if-env-changed=WRY_ANDROID_REVERSED_DOMAIN
cargo:rerun-if-env-changed=WRY_ANDROID_APP_NAME_SNAKE_CASE
cargo:rerun-if-env-changed=WRY_ANDROID_KOTLIN_FILES_OUT_DIR
--- stderr
thread 'main' panicked at 'WRY_ANDROID_REVERSED_DOMAIN is not set, which is needed to generate the kotlin files for android.', /home/aminassian/.cargo/registry/src/github.com-1ecc6299db9ec823/wry-0.23.4/build.rs:19:9
note: run with RUST_BACKTRACE=1 environment variable to display a backtrace
warning: build failed, waiting for other jobs to finish...
Error Failed to run cargo build`: Command "cargo build --package tauri-app --manifest-path /home/aminassian/programs/tauri-app/src-tauri/Cargo.toml --target x86_64-linux-android --no-default-features --color always" didn't complete successfully, exiting with code 101.
FAILURE: Build failed with an exception.

What went wrong:
Execution failed for task ':app:rustBuildX86_64Debug'.

Process 'command '/home/aminassian/.cargo/bin/cargo-tauri'' finished with non-zero exit value 1

Try:

Run with --stacktrace option to get the stack trace.
Run with --info or --debug option to get more log output.
Run with --scan to get full insights.

Get more help at https://help.gradle.org

BUILD FAILED in 46s
Error Failed to assemble APK: Command "/home/aminassian/programs/tauri-app/src-tauri/gen/android/tauri_app/gradlew --project-dir /home/aminassian/programs/tauri-app/src-tauri/gen/android/tauri_app assembleX86_64Debug --warn" didn't complete successfully, exiting with code 1.
Expected behavior
No response
Platform and versions
Ubuntu 22
Stack trace
No response
Additional context
seem relative to https://github.com/tauri-apps/wry/issues/843

Welcome to Tauri, please next time fill out Platform and versions with the proper output returned by running tauri info. Just Ubuntu 22 isn't helpful in this context.
That said, it's likely a version mismatch between the tauri crate and the CLI. If you're using the CLI from crates.io or npm you must also use tauri from crates.io, if you use the CLI from git then you must also use tauri from git.


feat(resources): Expand MAP20 tag checking to additional AWS services

Add support for Lambda, FSx, EFS, API Gateway, Transit Gateway
Include Network Firewall, VPC Endpoints, Direct Connect, VPC
Add CloudFront, SNS, SQS, Step Functions, QuickSight
Incorporate DMS, DataSync, CloudWatch Logs, and ACM
Update README with new supported services

This expansion enhances tag compliance checking across a broader range of AWS services, improving overall resource management.
Issue #, if available:
Description of changes:

I have incorporated support for 17 additional AWS services, including Lambda, FSx, EFS, API Gateway, Transit Gateway, Network Firewall, VPC services, CloudFront, SNS, SQS, Step Functions, QuickSight, DMS, DataSync, CloudWatch Logs, and ACM. This expansion significantly increases the pack's utility across diverse AWS environments.
The CloudFormation template has been updated to include new AWS Config Rules for each added service. These rules are designed to integrate seamlessly with our existing custom Lambda function for tag checking.
I've enhanced the Lambda function to efficiently handle the new resource types.
For each new Config Rule, I've implemented corresponding remediation configurations. These utilize the existing SSM Automation document to apply the correct MAP 2.0 tag to non-compliant resources, ensuring consistency across your AWS environment.
The README has been thoroughly updated to reflect the expanded scope of the Conformance Pack. It now includes a detailed list of all supported services, providing users with a clear overview of the pack's capabilities.

These enhancements significantly boost the capability of the MAP20 Tag Conformance Pack. Users can now manage and track resources across a much wider range of AWS services, all aligned with MAP 2.0 guidelines.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@aniadev-aws / @isaiahsal can you please review this?


Replace passport-openidconnect with something better

passport-openidconnect, at version 0.0.2, hasn't been updated in years and is responsible (I think) for some of the security alerts identified by Dependabot (https://github.com/ytgov/vue-template/network/alerts).
One option is openid-client, a certified Open ID Connect client for Relying Parties that supports passport.

Max has done work implementing a new library. See 2d23236.


[BL602] 1.add matter ota support, 2. use matter example DeviceInfoPro…

…viderImpl.cpp for adaption, 3.usd matter default dnssd, 4. add compile compile parameters, 5.change product id.
Problem

add matter ota support.
use matter default DeviceInfoProviderImpl.cpp for adaption
use matter default dnssd
add compile compile parameters
change product id

Change overview
add matter ota, add DeviceInfoProviderImpl.cpp for adaption, use matter default dnssd , add compile parameters and change product id.
Testing
How was this tested? (at least one bullet point required)

add matter ota support.
use test event TC-SU to test this case.
add DeviceInfoProviderImpl.cpp for adaption.
use test event TC-TFL to test this case.
use matter default dnssd.
use chip tool commission with bl602.

pr 19845 triggers a restyled error. Guess the reason for this error may be that I deleted the previously restyled file. I can't figure this out, so I'm reposting this pr. https://github.com/project-chip/connectedhomeip/pull/19845

Looks clean, thanks! Fast tracking platform changes.
@jczhang777 Can you please put some simple CI in place to verify this platform builds? Happy to handle that in a follow up, but please prioritize that. If platforms get out of date/broken, we'll likely have to disable them, CI is a great way to handle that.


Pretty transaction result on contract page

Description :
We merged this in SE-2  https://github.com/scaffold-eth/scaffold-eth-2/pull/853, maybe we should port it to Abi ninja too

How do you think this looks?
https://github.com/BuidlGuidl/abi.ninja/assets/108868128/507d6d2a-d718-4352-8f4c-c417ee8de94d
the code is on branch feat/better-tx-result
I have used byte's changes + changed the color + changed one line of code

Completed at #130


Updates needed to TLS description

In this section: https://docs.microsoft.com/en-us/azure/app-service/app-service-web-tutorial-custom-ssl#enforce-tls-1112
Your app allows TLS 1.2 by default, which is the recommended TLS level by industry standards, such as PCI DSS. To enforce different TLS versions, follow these steps:
In your web app page, in the left navigation, select SSL settings. Then, in TLS version, select the minimum TLS version you want.
Enforce TLS 1.1 or 1.2
When the operation is complete, your app rejects all connections with lower TLS versions.
Note that the TLS settings in App Service control the inbound calls only.

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

ID: 8dfb8f30-3236-ad55-eabf-6940a7e16329
Version Independent ID: 8f4385a4-9ac2-38da-d243-ed3c00b0fd36
Content: Bind an existing custom SSL certificate to Azure Web Apps
Content Source: articles/app-service/app-service-web-tutorial-custom-ssl.md
Service: app-service-web
GitHub Login: @cephalin
Microsoft Alias: cephalin

@odvoskin Thank you for the heads up. I was going to try and make the change but there are a two illegal links in the doc preventing me from merging anything. Also, the header, navigation and other items should be updated as well. Since this is a larger cleanup, I've reassigned the issue to @cephalin.

@odvoskin The update should be published in a few hours. #please-close


Reminding users to make backups periodically is a must

We have received two feedbacks about locked account. As we do not hold users' data, we can do nothing for help. Maybe the only thing we can do is reminding users to make backups periodically as Google does.

Done in version 4.6


Message Consumers stop after RabbitMq restart

Is this a bug report?
Yes
Can you also reproduce the problem with the latest version?
Yes, MassTransit 5.2.3.
Environment

Operating system: Windows 10 Pro
Visual Studio version: 2017
Dotnet version: Core 2, Net Framework 4.7.1.
Language version: C# 7.1 or higher
RabbitMq version: 3.7.3

Steps to Reproduce
See repository on https://github.com/jlandheer/MassTransitReconnectTest
Basically create  a client with a static async Task Main(string[] args) startup method and start the receiving bus with await bus.StartAsync();. Stop the RabbitMq service and start it again. The receiver will not receive messages after restart.
When using the synchronous version there is no problem.
Expected Behavior
Client should continue to receive messages after service restart.
Actual Behavior
Client stops receiving messages after service restart.
Reproducible Demo
See repository on https://github.com/jlandheer/MassTransitReconnectTest

Is there a way to work around this issue? We dont necessarily want to do anything async-specific in main, but it would be good to know what to do if someone is forced to async Main in a project.

You can still just call Start, not calling StartAsync won't end the world.

So, I changed your sample to where the async Task Main() doesn't call Console.ReadKey, but instead uses an async-friendly approach to waiting for the time to exit (by awaiting a TaskCompletionSource that is completed when it's time to exit) - and the process continues to work as expected. I have to think this is due to some blocking with Console.ReadKey or something that prevents access to some thread.
Since it works when properly awaiting within the async method, I'm inclined to close this - but it's probably important to show how to handle it properly.

Wow, good catch. Changed it to await Task.Run(() => Console.ReadKey());, and as the French say: voilà!

I had the same problem. Thanks for the tip, I wouldn't have found the solution without you!


document draft status for leaves

The only leaf I could find referencing "deer traps" was the one titled Public Grievance (filename claim-vs-nj, public title Claim vs. NJ); I've marked as a draft as part of this update
review notes

[x] look at files changed to see update to contributor documentation and draft status on claim-vs-nj file
[x] look at the timetree on the render site and confirm that the leaf is not included on the tree

I see that the Public Grievance leaf does not appear, and not apparent in the develop list of files in GitHub. So where do we find drafts?  Perhaps I did not check this correctly?

Thanks @rlskoeser . I'll take a look again when it is in develop.

ok @jhimpele - merging this pull request now, develop should show claim-vs-nj.md as a draft the next time you look


Run in iOS simulator failed

Steps to Reproduce

flutter create .
flutter run

Using iOS simulator failed but success in Android simulator.
Logs

Launching lib/main.dart on iPhone X in debug mode...
Starting Xcode build...
 ├─Assembling Flutter resources...                    1.3s
 └─Compiling, linking and signing...                  0.7s
Xcode build done.                                            3.4s
Failed to build iOS app
Error output from Xcode build:
↳
    ** BUILD FAILED **

Xcode's output:
↳
    While building module 'Flutter' imported from /Users/izayl/Code/flutter_shopping/ios/Runner/GeneratedPluginRegistrant.h:8:
    In file included from <module-includes>:1:
    In file included from /Users/izayl/Code/flutter_shopping/ios/Flutter/Flutter.framework/Headers/Flutter.h:37:
    In file included from /Users/izayl/Code/flutter_shopping/ios/Flutter/Flutter.framework/Headers/FlutterAppDelegate.h:11:
    /Users/izayl/Code/flutter_shopping/ios/Flutter/Flutter.framework/Headers/FlutterPlugin.h:140:58: warning: this block declaration is not a prototype [-Wstrict-prototypes]
                          completionHandler:(nonnull void (^)())completionHandler;
                                                             ^
                                                              void
    1 warning generated.
    1 warning generated.
    === BUILD TARGET Runner OF PROJECT Runner WITH CONFIGURATION Debug ===
    ld: framework not found Flutter
    clang: error: linker command failed with exit code 1 (use -v to see invocation)
Could not build the application for the simulator.
Error launching application on iPhone X.

Analyzing flutter_shopping...
No issues found! (ran in 1.7s)

[✓] Flutter (Channel dev, v0.4.4, on Mac OS X 10.13.2 17C88, locale zh-Hans-CN)
    • Flutter version 0.4.4 at /Users/izayl/Story/flutter-story/flutter
    • Framework revision f9bb4289e9 (10 days ago), 2018-05-11 21:44:54 -0700
    • Engine revision 06afdfe54e
    • Dart version 2.0.0-dev.54.0.flutter-46ab040e58

[✓] Android toolchain - develop for Android devices (Android SDK 27.0.3)
    • Android SDK at /Users/izayl/Library/Android/sdk
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-27, build-tools 27.0.3
    • ANDROID_HOME = /Users/izayl/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1024-b01)
    • All Android licenses accepted.

[✓] iOS toolchain - develop for iOS devices (Xcode 9.3.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 9.3.1, Build version 9E501
    • ios-deploy 1.9.2
    • CocoaPods version 1.5.2

[✓] Android Studio (version 3.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 24.2.1
    • Dart plugin version 173.4700
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1024-b01)

[!] IntelliJ IDEA Ultimate Edition (version 2017.3.1)
    • IntelliJ at /Applications/IntelliJ IDEA.app
    ✗ Flutter plugin not installed; this adds Flutter specific functionality.
    ✗ Dart plugin not installed; this adds Dart specific functionality.
    • For information about installing plugins, see
      https://flutter.io/intellij-setup/#installing-the-plugins

[✓] VS Code (version 1.23.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Dart Code extension version 2.12.1

[✓] Connected devices (1 available)
    • iPhone X • 9FF5BFE6-6A93-468C-B8F9-3CB85416489A • ios • iOS 11.3 (simulator)

! Doctor found issues in 1 category.

success when i reinstall flutter.😔

I was also having this problem. I ran flutter channel and saw I was in beta channel. I then ran flutter channel master to switch over. This fixed the problem for me.

See also the suggestions in https://github.com/flutter/flutter/issues/18340#issuecomment-396457808

Is this still an issue?

@zoechi yes.

@r0hit-gupta
Please add the output of flutter doctor -v.


Fix reverse mu needed calculations

Fixes issue when checking how much MU is needed for the opposition to get in front

Please reopen this issue/pull-request over at therealklanni/hubot-ingress.

Will try to do later today, thanks again


mbidled: init at unstable-2022-10-30

Description of changes

Add mbidled, a program which monitors a mailbox and runs COMMAND on change.
See it's manpage
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/1536


TypeScript - Need to export function return types

Description
Hello,
I need to export the function return types so that I don't have to redefine them in my app. Is it possible to add this feature please?
For example :
public static getInvesmentAggregate(
    data: TDataGetInvesmentAggregate = {},
  ): CancelablePromise<{
    result?: number;
  }> {
...
}

In the code above, I would like a type to be exported.
Thank you

@valentin-harrang try out the latest version of the package. $OpenApiTs is exported from the models. It describes the parameter and return types of each service. Let us know if you have recommendations for improving it.


Double.ToString() truncates large numbers

I have tested this on a number of systems and it seems like it is the same in all version of .Net on all platforms that I have tested
If you take a large number such as<PHONE_NUMBER>45701600d and execute the ToString() method it will truncate the number to "572572217245702000" if using a string format pattern or with none it will give you the sci notation version of the same number.
I have tried many string format patterns and there is no change other than adding leading zeros if you 0 out the length of the original.

@Quinquangle This is not a bug and "Double.ToString()" does NOT perform truncation (of the underlying data).
Double is simply not able to store so many digits and thus the least important digits are truncated/rounded.
Again, this is not a bug, but specified behaviour.
See https://docs.microsoft.com/en-us/dotnet/api/system.double?view=netcore-3.1 how the 64 bits of a double number are used.
Furthermore, on the linked page it is mentioned that

A Double value has up to 15 decimal digits of precision, although a maximum of 17 digits is maintained internally. This means that some floating-point operations may lack the precision to change a floating point value.

On that page there is even an example to highlight the behaviour.
As a further example (that it is not ToString that is the problem, but rather the datatype itself), see this simple sample program where i first print your double number, then in a loop add 1 a million times and print the number again after the increments.
Additionally I perform the same with a long variable.
`double d =<PHONE_NUMBER>45701600d;
        Console.WriteLine("Initial Double Value:" + d.ToString("N"));
        for (int i = 0; i < 1_000_000; i++)
        {
            d++;
        }
        Console.WriteLine("Updated Double Value:" + d.ToString("N"));

        long l =<PHONE_NUMBER>45701600L;

        Console.WriteLine("Initial Long Value:" + l.ToString("N"));
        for (int i = 0; i < 1_000_000; i++)
        {
            l++;
        }
        Console.WriteLine("Updated Long Value:" + l.ToString("N"));`

The output is:
Initial Double Value:572 572 217 245 701 632,00
Updated Double Value:572 572 217 245 701 632,00
Initial Long Value:572 572 217 245 701 600,00
Updated Long Value:572 572 217 246 701 600,00
So you see that even adding 1 a million times did not change the double value, because adding "1" to such a big double number has no effect (due to how it is represented) and thus even adding "1" a million time has no effect.
In contrast, performing the same logic with a long variable worked, because long uses all its 64 bits to directly represent the stored number, i.e. it can represent all integer numbers between its min and max value.


Add readme badges

Lifecycle
License
npm package
CI status
OpenSSF Scorecard
Codecov (currently uknown as there are no tests)

https://github.com/bingenito/eslint-plugin-externalincludes/blob/readme-badges/README.md


MMU support

This PR adds basic MMU support to m1n1.
Still missing:

Tests on real hardware. This has only been verified inside qemu so far
Proxy calls to disable the MMU before e.g. jumping to Linux

My suggestion is to use the defined types (aliases u8, u32, u64, etc.) in types.h so that their use be consistent throughout the code.

My suggestion is to use the defined types (aliases u8, u32, u64, etc.) in types.h so that their use be consistent throughout the code.


fix: remove some toRaw

it was bugging me, this makes more sense

@amritk wait why does this work 🤯 what changed 🤯

Removed a couple toRaws, but Hans pointed out we can do an even better job

Alright now its much better


Does this work with EGL Streams?

Then it would work with NVIDIA drivers, too.

Just a reference if anyone else reading this later wants to give this a shot (I might do so myself when time allows, but no idea when that'll be): Gnome's (or rather, Mutter's) EGLStreams patches

FWIW, EGL stream support appears to have been merged into wlc: https://github.com/Cloudef/wlc/pull/245

@Timidger This seems to work somewhat now (although having some driver instability)
To get it working on fc27 i basically had to install the proprietary drivers, and remove this check:
https://github.com/way-cooler/way-cooler/blob/master/src/main.rs#L121-L125
Not sure if there is a way to check whether the wlc supports egl streams or not, or if it would enough to just bump the version of wlc required.

WLC since... 0.0.7 or so has support egl streams, so the answer given to this issue is out of date. Note that wlroots does _not _ support it, and when we transition to it the support will go away

@ratmice that line you linked just issued a warning, it shouldn't stop the start up sequence

@Timidger Ahh, as I said having driver instability, Its likely that when i tried it and got the warning the device was already in some bad state..


Add some preset comments for list commands.

Based on the comment, we add some preset comments to specific the location for the submitters with working on #1512.

No changes to these commands, just a change of order to make the format reasonable.


Signup Button Displays Login Modal Instead of Signup Modal

Functionality Issue
GSSOC_23_Contributor
when a user clicks on the signup button, the login modal pops up instead of the expected signup modal.
This creates inconvenience for users to signUp.
I will fix the issue, assign it to me.
https://github.com/sourabhsikarwar/Scene-Movie-Platform/assets/97674335/1dbbf318-e029-45f4-98f7-43546345b0d7

Please add a label to me issue as gssoc23

please assign this issue to me


[SIEMINT-122] DDSaaS: Brevo: Integration v1.0.0

What does this PR do?
This is a initial release PR of Brevo integration including all the required assets i.e. Pipeline, Dashboard and Monitors
Additional Notes

This integration is webhook based and so Crawler will not be used.
Datadog team will be required to add UI component for generation of webhook URLs in the configuration tile. Two URLs will be created, with both URLs having the ddsource query parameter set to brevo. The service query parameter will be set to marketing-events for marketing events and transactional-events for transactional events.
The assets are available in our sandbox and can be shared separately with the required teams.

Review checklist (to be filled by reviewers)
[ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[ ] Changelog entries must be created for modifications to shipped code
[ ] Add the qa/skip-qa label if the PR doesn't need to be tested during QA.
[ ] If you need to backport this PR to another branch, you can add the backport/ label to the PR and it will automatically open a backport PR once this one is merged

For logs files, it is looking very good 👍, good job on this. I am pushing it in our test environment and will give you news soon for a potential approval.

/merge

/merge  --cancel

/merge -c

/merge

/remove

@bparmar-crest @thibaultkrebs Can you please take another look? We've approved on our end but are blocked on merging

@apiazza-dd Unfortunately, I cannot access the error links from above comments. Those links might have more info about merge failures.
Maybe @thibaultkrebs can help here !

@bparmar-crest @thibaultkrebs ISorry, we're waiting for another code owner review on our end for the pipeline. We'll let you know when it's merged.

/merge

/merge -c

/merge


feat: adding delete tenant admin endpoint

The change looks good!
It would be great if we add a sequalize transaction for 2 delete queries and even perfect is to have write lock on ClientInformation table so that we can make sure no new entity with the cpoTenantId can be created before we delete that CpoTenant.

omg 😮💨 I remember thinking to my self - "need to wrap in transaction" but forgot - should be there now 👍


Show Pimusic box version on the webpage

I think it is a god idea to show the version on the webpage, for better support and more

Fixed! Thanks


Path point creation doesnt create points

Godot version
eaf306e0b
System information
ArchLinux x11
Issue description
Creating points on a path3d doesnt work.
Steps to reproduce

Create path3d
Create a new curve resource
Click the add point button
Click anywhere
No points!

Minimal reproduction project

It worked in c3539b4561f9b4d7dc4ba1c5859217e7fbf9c6fe


Automate API File Process To Post Every Monday Morning At 9AM

We generate a report that has a record number that goes above 10,000 because of this we will need a file to be posted every Monday at 9AM.

Request app access to shared folder

Waiting on #213

Add registration_status to HealthPro API file


Running AppHost in Release config fails

I created the starter app from the templates. I then built and ran AppHost in Release configuration. When the dashboard came up I was able to see the following stack traces. It's looking for them in the bin/debug folder rather than bin/release. I have no good reason to run it in Release config. It works fine in Debug config. Preview 6 (template from Preview 5).
apiservice
Unhandled exception: System.ComponentModel.Win32Exception (2): An error occurred trying to start process 'Aspire.ApiService\bin\Debug\net8.0\Aspire.ApiService.exe' with working directory 'Aspire.ApiService'. The system cannot find the file specified.
   at System.Diagnostics.Process.StartWithCreateProcess(ProcessStartInfo startInfo)
   at Microsoft.DotNet.Cli.Utils.Command.Execute(Action`1 processStarted)
   at Microsoft.DotNet.Tools.Run.RunCommand.Execute()
   at System.CommandLine.Invocation.InvocationPipeline.Invoke(ParseResult parseResult)
   at Microsoft.DotNet.Cli.Program.ProcessArgs(String[] args, TimeSpan startupTime, ITelemetry telemetryClient)

web
Unhandled exception: System.ComponentModel.Win32Exception (2): An error occurred trying to start process 'Aspire.Web\bin\Debug\net8.0\Aspire.Web.exe' with working directory 'Aspire.Web'. The system cannot find the file specified.
   at System.Diagnostics.Process.StartWithCreateProcess(ProcessStartInfo startInfo)
   at Microsoft.DotNet.Cli.Utils.Command.Execute(Action`1 processStarted)
   at Microsoft.DotNet.Tools.Run.RunCommand.Execute()
   at System.CommandLine.Invocation.InvocationPipeline.Invoke(ParseResult parseResult)
   at Microsoft.DotNet.Cli.Program.ProcessArgs(String[] args, TimeSpan startupTime, ITelemetry telemetryClient)

Thanks @ptasev -- this is from CLI right? (I can repro on CLI, but not in VS)

I actually ran the AppHost project from Rider. I believe it simply ran the executable in the AppHost bin/release folder.

I can reproduce this with dotnet run -c Release.

This is easy to reproduce.

dotnet new aspire-starter -o asp0
cd asp0/asp0.AppHost
dotnet run -c Release

cc @joperezr @eerhardt @danegsta

Fixed by https://github.com/dotnet/aspire/pull/3961 . Tests for this are passing in https://github.com/dotnet/aspire/pull/4168/checks?check_run_id=25072734191 .
Thanks @danegsta .


Fix invalid metadata attribute

@bertvv
Thank you for this explicit and tidy role, have found little issue when running for the first time
ERROR! 'version' is not a valid attribute for a RoleMetadata

The error appears to have been in '/home/----/ansible-hosts/meta/main.yml': line 2, column 1, but may
be elsewhere in the file depending on the exact syntax problem.

The offending line appears to be:

---
galaxy_info:
^ here

Indeed, this (custom) 'version:' field is no longer allowed since Ansible 2.0. It was added to support a 3rd party dependency management tool (librarian-ansible).
Thanks for fixing, any other suggestions/improvements are always welcome!

I released a new version with this change. I also moved test code to a separate branch and fixed a deprecation warning there.


Lack of complete evaluation code

It seems some evaluation code about the class of RefL4Dataset are missing ,could you please update it?

Hi @zhaohongyin, thanks for raising the issue. But could you provide more details on the part you are referring to? Then I'll update accordingly.

@JierunChen I want to evaluate using ref-l4, but I don't see the following modules in the code. Are they planned for an update?
https://github.com/JierunChen/Ref-L4/blob/10a3344fe2c6edbdc6a8aac30120c0fb107f77f9/evaluate_pred.py#L3

Hi @jay-vinin , thanks for the notice. Missing code has been pushed.

Hi @huangb23 , thanks for your interest in our work. Grounding-DINO focuses on open-vocabulary detection and phrase grounding, while our benchmark is tailored for referring expression comprehensive (REC). Phrase grounding is to locate multiple targets for short phrases while REC is to locate a unique object given a generally longer description. Thus, we do not include the Grounding-DINO in our benchmark.


Add ToC and disclaimer

Codecov Report

Merging #77 (aff299c) into main (ea8a489) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #77   +/-   ##
=======================================
  Coverage   65.27%   65.27%
=======================================
  Files          21       21
  Lines        1028     1028
=======================================
  Hits          671      671
  Misses        357      357

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Adding a NATS account server authorizer implementation

Closing #29

Pretty awesome to see this concrete authorizer. It would be really easy to show an enterprise how you can completely restrict the resources running on their host to a fixed group of approved accounts if you want.


backend: update allowed attributes for prefix spec

Add a global list of all attributes of a prefix, also marking which ones
are read-only or read-write, which we derive a second list from that
only contains the writable attributes.
These two lists are then used in all the places where we check input
arguments.
Fixes #667.
@garberg, what do you think of this approach? I didn't apply the same
for pools and VRFs because I wanted input on this one first but if you
think this looks good we should do the same for them as well.

Generally it looks great, a really nice improvement!
But, as the PR is currently implemented, isn't the ability to search for read-only attributes destroyed, as the _prefix_attrs filters out them? Or am I missing something? Perhaps there should be a _prefix_attrs_ro with the read-only attributes and a _prefix_attrs_rw with the writable attributes, which can be concatenated when performing validations in the search_- and list_-functions?

No, because the search functions look in _prefix_spec while add_prefix and edit_prefix look in _prefix_attr, which is a subset of _prefix_spec.

Doh.


Implementors - plan to switch from "Connections" to "DID-Exchange"? And why use DID-Exchange at all?

This issue raises two questions about the new "DID-Exchange" name for the protocol used to establish a connection between agents.
We're interested in knowing when those implementors that have had an Aries agent since the Provo Connect-a-thon are going to be switching from using the "connection" protocol to "did-exchange"? We've decided it's too complicated in ACA-Py to support both, so we're getting ready to just switch over to support more recently created agents that use "DID-Exchange".  We have the IIW Demo that we are still using (and it's important!) and would like to continue to work, so we're wondering what StreetCred is planning @tmarkovski.  Likewise MattrGlobal @tplooker.  Any others?  Our plan is to leave the servers for the IIW Demo using older ACA-Py implementations that support "connection", and when StreetCred updates, we'll update those servers at the same time.
I also note the use of "connection" in our code for the connection object, likewise in aries-staticagent-python (@TelegramSam @dbluhm) and in the aries-framework-dotnet.  Should we really rename to "did-exchange" the thing that creates the connection object that defines the relationship between entities. Connections are about a lot more than DIDs.  My vote would be to revert the change back to "connection" drop "did-exchange" entirely. I believe that other than the name, the protocols are identical.
Anyone else want to go back to "connection", or do we just continue with the new term?

The protocol is about DID exchange. The noun that results from the protocol is a connection. I don't feel any need to change existing nouns in our code or interfaces away from "connection", or to change the name of the protocol to anything other than "DID exchange".

the protocol is about creating and maintaining a connection

I thought it was only about creating. If we want to tear down a connection, how do we do that? If we want to tell the other party that we've updated our DID doc, how do we do that? If we want to add another party to the current 2-party connection, how do we do that? If we want to set policy for our connection, how do we do that? If we want to ask the other party what they think about our connection, how do we do that?

It's just that we can't easily have both in one implementation

I worry a bit about architecture if we can't easily handle name changes or synonyms for a protocol. Going forward, it's at least going to need to be possible to support multiple versions of the same thing. But maybe multiple versions of one name and multiple names are separate issues?

My thought is all of those things will be added to this protocol. I don't see a need to add a new protocol, but to extend this one.  All the better for it to be connections.  Do you think all of those functions need to be different protocols?
I agree that this is not great that the protocol can't be synonymed, but if you think through what this protocol does, you'll likely understand why.  We were able to parallel credential exchange protocols, but since this one is core to the agent itself (vs. on top of the agent) it is difficult.

Do you think all of those functions need to be different protocols?

That was my assumption. But maybe it's a bad assumption. This probably needs further discussion.
The reason I thought this is that the state management for each of these problems is potentially different (e.g., I may have a different error path if I can't successfully tear down/delete a connection than I do if I can't modify the policy of a current one). Also, I thought that these operations might be versioned independently.
Now that I think about it more carefully, maybe there are ways to make my assumption less constraining...

My thinking has also been that a DID Exchange results in a connection and that a name change of the protocol does not necessitate the renaming of objects in code such as StaticConnection in the static agent library. I personally like the descriptiveness of "DID exchange" as that is ultimately what is happening; though it doesn't cover all the details (how could it?), I think it's at least as descriptive as a name like "Diffie-Hellman Exchange." That being said, I'm not totally opposed to another name.
I think the initial establishment of a connection is nuanced enough and sufficiently complex that keeping establishment separate from maintenance feels like a good separation of concerns.
As per our previous discussions on protocol naming, even if we did decide to combine establishment and maintenance into a single protocol, I think "connections" is still ambiguous and potentially confusing. I would recommend keeping the convention that we established of verb-noun combos such as "manage-connection" or "create-connection" or something along those lines.

What I'm asking for is that we don't do an unnecessary name change from connection to did-exchange.  The change is harder than planned, requires coordination amongst implementers, and the value is negligible. "did-exchange" instead of "connections" does not reduce the effort for implementers to understand what the protocol does. It is just a different, equally ambiguous word.
I would like a definitive answer ASAP as the longer we wait the more pain and confusion we inflict on others.
Question: Do we keep the protocol name connections that has been implemented in a number of code bases, or do we switch to did-exchange?

My vote is to keep the protocol name connections.

Others please vote.
I'm not concerned in this issue about whether we add more messages to this protocol for the rest of the connection maintenance, or we create a new protocol. And I don't think the name of the protocol helps in deciding that issue.

Stephen, I agree that the value of the new name's clarity is not huge. I think it's more than negligible, but the difference in magnitude isn't worth quibbling about.
I also agree that name changes should not be done without some deliberation.
In this case, I am inclined to keep the name not because of magnitude, but because I thought we had talked publicly about a general naming strategy and decided that "connections" was a poster child of ambiguity, and that we'd agree to use something better going forward. Here are the slides we used to discuss this on a community call. I believe it was this community conversation that motivated the change to "DID Exchange."
I don't want to come across as unsympathetic, but I'm genuinely puzzled by the "pain and confusion we inflict" comment. How is the string constant that identifies a protocol anything other than a minor detail, declared in one place in code? And why could we not declare in code that string constant A and string constant B are synonyms -- thus allowing a codebase to support both constants at the same time? I would buy the claim that codebases shouldn't generally be oriented towards supporting synonyms, but it certainly must be the case that robust, permanent-use codebases are written to support multiple versions of a protocol, so a multimap behavior must be implemented regardless.
Based on your articulate pushback, and respecting the heavy investment you and your team have made in protocol support, I'm imagining that there's some characteristic of routing a protocol to a handler in code that's more complex than I've understood. I'd really like to know what that is, since it sounds significant. If we can clarify what it is that makes this so hard, and it is as compelling as you imply, then I will cheerfully vote to revert the protocol name. But in the meantime, all I have to go on is my own intuition that this ought to be a 10-min change, and that if it's not, it's the codebase that ought to be refactored to make it so, not the spec. So that's a provisional vote to keep the name "DID Exchange", pending revelations of a deep issue we're not understanding.

My initial feeling is to keep the name as it is, since renaming doesn't bring any immediate (functional) value, though I don't have strong opinion one way or the other. While the renaming for most implementors may be a few minutes of work, there are services in production that will immediately break, which are not as easily updated. If we updated this in the Streetcred mobile app, it would certainly break a number of services that rely on it.
If everyone feels strong about renaming to did-exchange, I suggest introducing a rolling update with backwards compatibility wherever possible, with a broader implementation done during one of the next connect-a-thons, which is where most codebases get heavily updated and synced.

Ah. @swcurran, is the pain you are talking about the pain of updating production deployments? That would make a lot of sense to me; I was only focused on the coding cost when I claimed this should be easy.
If there are production deployments that we don't want to disrupt, then I'm willing to change my vote to "revert the protocol name to 'connection'".

It's a both a code issue and the existing implementations. It's the existing implementations and the fact that (per the developer, who is way smarter than me) it's a non-trivial effort to support both names for the protocol, because it's so core to the agent.  Supporting both credential exchange protocols was relatively straight forward and we've done that. It's because this one is about connections.

Just thought I would supply a concrete version of a rolling upgrade method.
Step 1) Add support for accepting the new protocol type string (in addition to the connections one). This string acts as a synonym for the connections one. Implementations keep their other existing behaviours at this point.
Step 2) After X months, implementations begin sending the new type string.
Step 3) After Y months, implementations cease support for the old type string.
For API surface (where needed): add synonyms for new names and deprecate old ones. After Z months, cease support for deprecated names.
(this method doesn’t support both at once, but allows for a graceful transition.)

Yes, that's the plan generally, and we did that for credential exchange.
The problem (per the devs) is this particular protocol. It's more work than
just a synonym. And there is literally nothing to the change than the name.
On Sat, Sep 14, 2019, 1:59 PM Troy Ronda<EMAIL_ADDRESS>wrote:

Just thought I would supply a concrete version of a rolling upgrade method.
Step 1) Add support for accepting the new protocol type string (in
addition to the connections one). This string acts as a synonym for the
connections one. Implementations keep their other existing behaviours at
this point.
Step 2) After X months, implementations begin sending the new type string.
Step 3) After Y months, implementations cease support for the old type
string.
For API surface (where needed): add synonyms for new names and deprecate
old ones. After Z months, cease support for deprecated names.
(this method doesn’t support both at once, but allows for a graceful
transition.)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/hyperledger/aries-rfcs/issues/214?email_source=notifications&email_token=AAHYRQSVYRU6JYTTOC2Y4T3QJVGEVA5CNFSM4IUN44QKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD6XEDIQ#issuecomment-531513762,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAHYRQSUOPCKNAT5OH2ZBUDQJVGEVANCNFSM4IUN44QA
.

At the risk of muddying the water more, there were other differences from the connectathon version of the connection protocol and RFC 23.   In particular, the body of the request has two name changes: in the 'connection' section, changed 'DID' to 'did' and 'DIDDoc' to 'did_doc'.

Thanks @smithbk - agreed, as your testing found.  Sigh.

We (ACA-Py team/BC Gov) are going to give up fighting against this change. We'll make the conversion to the new protocol the week after IIW and update/redeploy all instances (that we can find) of ACA-Py at that point.  We understand that StreetCred is going to do the same with aries-framework-dotnet the same week.
Let's not do this again.  Breaking changes must have corresponding benefits.


[New game]: spaceship escort

🎮 Game Request

Escort the cargo spaceship.
Select battle starships and place them strategically.
Do not spend more Scriptium than neccesary.

Point down the features
Game points will be awarded on basis of kills
Select program in which you are contributing
GSSoC23
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

Hey @S-ishita 👋!
@kunjgit here,
We have this annoucement to make 📢!!
💥 Special Announcement:
It's the Call to all the designers 🔥 🎨
As you all know that GameZone has crossed 100 games 💗 , it's time that we all come up together and make the UI even better!!
We are redesigning the thumbnails, make sure you come up with the aesthetic and cool ideas 🚀 !
You are supposed to share the 4 different templates that can be used. Make sure that the size is: 1280 px X 720 px.
The frame should be such that there is only a single image of the game that's taking up the major portion of the frame. Any kind of text should not be there.
The design should be catchy and aesthetic! 🔥
Note: You are not supposed to create any issue right now for the same.
You have to share different designs right now in Discord  and Project Admin would be selecting one design and would be assigning the issue. After that only, you have to start working on it.

Hey @S-ishita ! Thank you so much for your raising the issue💗
It’s all yours, you can come anytime again and make some contributions! 🚀
Alone, we can do little, but together we can do so much! 😇


Explore if we can support Gitlab/others

Overview
See https://frictionlessdata.slack.com/archives/C0369HZ2SLT/p1675530494127069?thread_ts=1675523690.362539&cid=C0369HZ2SLT

As we discussed, the ability to use alternative CI/CD pipelines (including self-hosted) would be welcomed by the open source community. At least some instructions, like these on the blog of Tomas Votruba could help. The GitLab docs seem to imply that Actions are not automatically converted on import. However, GitHub provides helpful instructions for going the other way.

Thanks a lot for this info!

This would definitly be a plus. We are currently using FAIRDOM-seek coupled to a python plus frictionless validator. Using frictionless/repository to visualize validation reports would definitely be a plus. We could set this up using our own gitlab instance if need be.

@cpommier
Thanks for sharing! While it's not yet supported by Repository, you can render a validation report within the web app using the report component - https://application.frictionlessdata.io/docs/metadata.html


Cannot hook while Dolphin is using extended memory (MEM1 48mb, MEM2 64mb)

As you can see, I have a version of DME that works (can't even remember where I got it from) that accepts the extended memory, but when I try to read / write from py-dme, it doesn't write. I've tried several addresses since the example above and am unsure on how to get it to hook properly and actually write correctly.

Does it work with the official DME? If its a recent change, this probably
just needs a merge with upstream.
On Wed, 15 Dec 2021, 18:17 ashtif, @.***> wrote:

[image: image]
https://user-images.githubusercontent.com/75453431/146266077-de99784c-4029-40bf-88f2-8205d2529910.png
As you can see, I have a version of DME that works (can't even remember
where I got it from) that accepts the extended memory, but when I try to
read / write from py-dme, it doesn't write. I've tried several addresses
since the example above and am unsure on how to get it to hook properly and
actually write correctly.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/henriquegemignani/py-dolphin-memory-engine/issues/12,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAGYBQG6KJZEAOKIPSSN2OTUREAYDANCNFSM5KEVQVZA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

This still seems to occur, it would be cool to see the restriction removed so we can mess debug builds of games.


[BUG]If the structure with a large number of training samples has a memory exception in the data, it will fail. How to train large data sets, such as how to implement methods such as batch training?

Bug Report Checklist

[ ] I provided code that demonstrates a minimal reproducible example.
[ ] I confirmed bug exists on the latest mainline of AutoGluon via source install.
[ ] I confirmed bug exists on the latest stable version of AutoGluon.

Describe the bug

Expected behavior

To Reproduce

Screenshots / Logs

Installed Versions

# Replace this code with the output of the following:
from autogluon.core.utils import show_versions
show_versions()

The recommended approach is to train on a subset of the data, and then pass data in smaller batches for prediction.


could not load main class while running the java file

whenever i run the java file i was facing an Error that is Could not find the main class
and even when i am performing c program when i am running a c program the c file could notload properly

This sounds like it might be an issue with your code or configuration. If you think it's an issue with vscode, please file it on the repo for the extension you're using


Imports not working

I haven't been able to get this to work, so I tried loading up the Example program and ran into errors right away.
The first three errors have to do with import locations that don't exist:
import 'package:http/http.dart';
import 'package:provider/provider.dart';
import 'player_widget.dart';
"The error message is: Target of URI doesn't exist: 'package:http/http.dart'."

@SSolheim2000 It looks you do not import http package correctly. Try import from instruction from https://pub.dev/packages/http

I was able to get two of them resolved by adding to the dependencies section in pubspec.yaml:
  http: ^0.12.1
  provider: ^4.1.2

I couldn't fine the 'player_widget' page to figure out what line to add there.

@SSolheim2000 What about player_widget? It is a local file?

It is one of the imports you referenced in your example.

@SSolheim2000 If you haven't this file, don't import it.

Then the example doesn't work.

@SSolheim2000 your imports looks like:
import 'package:http/http.dart';
import 'package:provider/provider.dart';

?
Do you get dependencies with flutter packages get or something else?

Here's the example I'm working with: https://pub.dev/packages/audioplayers#-example-tab-
On about the 11th line, there is a call to import 'player_widget.dart'; That import doesn't work.
As for the other two, http and provider, I was able to get them working. I did it in Android Studio, after modifying the dependencies section to include the them, I did the get 'Pub Get' function and now they work fine.
So, I just need to know where to find the dependency information for the 'player_widget'.

@SSolheim2000 See in this folder https://www.github.com/luanpotter/audioplayers/tree/master/example%2Flib

You have to run flutter pub get in the example folder.

I was able to find another project that actually worked and moved on from there. I documented my findings and the code needed here: [https://github.com/SSolheim2000/AudioPlayer_Basic/]
I would take a good look at your documentation and make sure all the steps are included and that someone can actually follow it.

I have the same problem, couldn't find audio widget package.
audio Palyer basic just works for local files and I need audio player for remoteUrl.

I haven't been able to get this to work, so I tried loading up the Example program and ran into errors right away.
The first three errors have to do with import locations that don't exist:
import 'package:http/http.dart';
import 'package:provider/provider.dart';
import 'player_widget.dart';
"The error message is: Target of URI doesn't exist: 'package:http/http.dart'."

Hy you can fix this by doing  import 'package:http/http.dart' as http;

As I mentioned earlier, I was able to get the HTTP and provider imports to work.
As I advised Launpotter, I would double-check and make sure the documentation is correct.

If you want those libs, you have to explicitly add them as deps. They are not transitive from this package.
This is clearly a flutter issue and has nothing to do with audioplayers.

I couldn't use this plugin and the hardest thing that I encountered was that I am missing the nuances in documentation of this plugin.

Yep. I let them know their documentation was incomplete and they did nothing.


Notebooks: Fix Table Generation into Pure Markdown When No thead Exists

Addresses #15140.
Note that it's expected that styles will be lost here during the conversion. Synced with livesite folks and this was deemed acceptable. If anyone thinks that there should instead be a setting, we can chat. However, we already convert tables that have a thead/tbody into pure markdown today (i.e. not persisting all of the styles), so this work merely extends that functionality for more cases.
The following is a typical (very simple, one row table) that gets generated when copying from Office products. Note that there is no thead, only a tbody element:

			test1
			test2
			test3

We weren't handling the case when thead didn't exist at all, or when a cell in a table had a paragraph element in it. Beyond that, just a bit of cleanup.

Pull Request Test Coverage Report for Build 833680749

18 of 19 (94.74%)  changed or added relevant lines in 2 files are covered.
2 unchanged lines in 1 file lost coverage.
Overall coverage increased (+0.009%) to 43.538%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/sql/workbench/contrib/notebook/browser/htmlMarkdownConverter.ts
4
5
80.0%

Files with Coverage Reduction
New Missed Lines
%

src/sql/workbench/contrib/notebook/browser/turndownPluginGfm.ts
2
78.65%

Totals

Change from base Build 833466419:
0.009%

Covered Lines:
26515

Relevant Lines:
55423

💛  - Coveralls

@chlafreniere I added the fix for Component Governance that's causing it to fail, merging main should fix this

@chlafreniere I added the fix for Component Governance that's causing it to fail, merging main should fix this

Thanks @abist. Will merge latest from main shortly.


Prevent Gear Desync Bug

bug Occlude if someone try to add the same Vest/Backpack/Uniform that the unit wearing in one frame
_uniform = uniform player;
removeUniform player;
player addUniform _uniform;

this executed in a unscheuded crate the bug

http://feedback.arma3.com/view.php?id=19829


Refactor code to better account for new ScannerSearchOptions

A bunch of ScannerSearchOptions options were added while writing the documentation, which were previously missing from the collection yet still used from intent extras.
Track down and refactor parts of the code that still use explicit intent extras instead of ScannerSearchOptions options. For example, ScannerService.onBind().

Resolved by #186


Control aircon by half degrees

Is it possible to control the aircon by half a degree?
I can do it via the mitsubishi app however from home assistant when I try set it to a half degree it will just round down to the nearest degree.

Not possible with the ECHONET library sorry. It returns an integer value only, not a floating value. Believe me I have tried...

Reopening this issue to remind myself to update the climate entity to use whole degrees only..

Closing this again because even though I have played around with precision values in HA it doesn't seem to work.


ChromiumFx abandoned

hi,
it seems that the chromiumfx project is abandoned and the question is if cefsharp is the recommened way to use with neutronium now?
Are there plans to keep Neutronium up to date in the future as this is vital for new projects building on this great package!

related https://github.com/NeutroniumCore/Neutronium/issues/170


MarginPool farming

Task: Feature Name:
MarginPool farming
High Level Description
This PR implemented the internal balances accounting and add the excess balance farming functionality.
Code

[x] Unit test 100% coverage
[x] Does your code follow the naming and code documentation guidelines?

Documentation

[x] Is your code up to date with the spec?
[x] Have you added your tests to the testing doc?

LGTM
Can be merged after adding test for farming amount > external balance - storedBalcne
https://coveralls.io/jobs/67334430/source_files/2186019993#L180

Added.


Explain difference to angular-polymer repo

Somewhere we need to display differences from this repo to https://github.com/platosha/angular-polymer
Where should that be and how are the two gonna live together?
In my view: there should only be one repo with different "beta" branches. Also in regard to angular naming convention: I think the repo should be named "angular-polymer". And not bound to a specific version number of angular

@platosha please give your input here. I can do merge requests based on your decision

@BorntraegerMarc agree.
platosha/angular-polymer is a temporary playground for the new DOM renderer. It is intended to be upstreamed here and replace the obsolete PolymerDomAdapter trick in this library.

All clear. Maybe we want to do the renaming already now? As I think the upstreaming will not happen very soon.
And this way we do not confuse all people that are already using the library later on (which I will suspect will be more then at the moment).
Renaming/rebranding stuff is always kinda a hazzle. In my experience it's best to do it as soon as possible...

@BorntraegerMarc Right, makes sense. Done.

gracias :)


Out of Box: Unexpected token <

Tried this out of the box, and got this error:
node_modules/react-geosuggest/src/Geosuggest.jsx Line 222: Unexpected token <
I'm sure it's something with complaining on my end.  Any suggestions?

This is linked to issue #11. Will update the build process to make the JSX transform before publishing to npm.

Thank you, looking forward to using your component.

Fixed and released as 1.4.0.


tests failing with rails 5.2

Originally reported here (has details) https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=918611

It's because the rails app didn't use the assets pipeline, but the rails should enable the assets pipeline by default.
Did you change the default stack or use the api mode to run tests?
https://github.com/fgrehm/letter_opener_web/issues/46#issuecomment-273975173

This is how I run the tests https://salsa.debian.org/ruby-team/ruby-jquery-atwho-rails/blob/master/debian/ruby-tests.rake and it was working with rails 4, but started failing with rails 5.2.
This patch makes the tests to pass, sprockets-rails should be added as a development dependency.
--- a/spec/spec_helper.rb
+++ b/spec/spec_helper.rb
@@ -2,6 +2,7 @@
 require 'rails'
 require 'rails/all'
 require 'rails/generators'
+require 'sprockets/railtie'

 class TestApp < Rails::Application
   config.root = File.dirname(__FILE__)

require 'rails/all'

Did require 'sprockets/railtie'

@ichord I don't have much idea about rails or ruby (I only manage with basic understanding of ruby to maintain debian packages). So not sure why rails/all is not enough. May be about order of requires?

Maybe rails/all raised a LoadError before require 'sprockets/railties', not so sure.
I saw you fix the bug and close the report, so that i will close this issue.
I think the better way to import javascript, even for rails, is using packing toolings such as webpacker.


Inplace arithmetic fails

Causes a segfault (!):
import starsim as ss

def jump_age(sim):
    """ Arbitrary intervention to reduce people's ages in the simulation """
    if sim.ti == 20:
        sim.people.age += 1000 # sim.people.age = sim.people.age + 1000 works

sim = ss.Sim(interventions=jump_age)
sim.run()

The inplace arithmetic works well in Python but numpy arrays get mutated by " += ".
https://stackoverflow.com/questions/35910577/why-does-python-numpys-mutate-the-original-array

@cliffckerr - I spent an hour today looking at this and have a root cause identified.
We can boil it down to the following case
x = Arr([1,2,3])
x += 1000

This replicates the segfault
It appears what is happening it  iadd(=+) call ufunc with add, and kwargs out out=(self). This means the out Kwarg is of type FloatArr in this case. This ends up in a infinite recursion somehow, maybe repeated called to add on the out kwargs?
If I add the following code to the Arr.array_ufunc before the return, things work fine
        # Handle the 'out' parameter in kwargs
        if 'out' in kwargs:
            out = kwargs['out']
            if isinstance(out, tuple):
                kwargs['out'] = tuple(x.values if isinstance(x, Arr) else x for x in out)
            elif isinstance(out, Arr):
                kwargs['out'] = out.values

This makes some sense looking here https://numpy.org/doc/stable/user/basics.subclassing.html#array-ufunc-for-ufuncs
I am not sure if we want to do a more extensive fix like and look at other parameters and situations this could arise. I also know some others things are in flux so not sure if this already being addressed somewhere else.

This version adapted from the numpy example seems to work for all our tests
 def __array_ufunc__(self, ufunc, method, *inputs, out=None, **kwargs):
        args = []
        in_no = []
        for i, input_ in enumerate(inputs):
            if isinstance(input_, Arr):
                in_no.append(i)
                args.append(input_.values)
            else:
                args.append(input_)

        outputs = out
        out_no = []
        if outputs:
            out_args = []
            for j, output in enumerate(outputs):
                if isinstance(output, Arr):
                    out_no.append(j)
                    out_args.append(output.values)
                else:
                    out_args.append(output)
            kwargs['out'] = tuple(out_args)
        else:
            outputs = (None,) * ufunc.nout

        info = {}
        if in_no:
            info['inputs'] = in_no
        if out_no:
            info['outputs'] = out_no

        results = self.values.__array_ufunc__(ufunc, method, *args, **kwargs)
        if results is NotImplemented:
            return NotImplemented

        if method == 'at':
            if isinstance(inputs[0], Arr):
                inputs[0].info = info
            return

        if ufunc.nout == 1:
            results = (results,)

        results = tuple((np.asarray(result)
                         if output is None else output)
                        for result, output in zip(results, outputs))
        if results and isinstance(results[0], Arr):
            results[0].info = info

        return results[0] if len(results) == 1 else results

@RomeshA what do you think the right behavior here would be? Logically I'd assume that FloatArr += 1 should return a FloatArr, but if it has to cast to a (plain) NumPy array and back that could be slow (...or not?)...


[BUG] Authentication is not working

Description of the bug
Hey, if I click the Google login button and then back then it redirects to the dashboard page but I am not login. I think this issue needs to fix.
Can you please assign it to me?
Please add screenshots (if applicable)
No response
Add any other context about the problem here
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

@SubhamB2003 how are you gonna fix this issue? Can you please explain a little bit?

Yes if I am successful in login then only it redirects to the dashboard page otherwise it stays on the sign in page.
Can you please assign it to me

@SubhamB2003 how are you gonna fix it?

@SubhamB2003 how are you gonna fix it?

I changed the fallback URL to "/" currently it was "/dashboard"

@SubhamB2003 is it working fine locally?


Added pre-commit hook to test whitespaces and formatting

Fixes #162.
At the moment, whitespace failures have to be fixed manually, while
formatting failures can be fixed by running
git clang-format

@dalg24 Could you please see if this works for you?
The interface is not super user-friendly just yet, but it's a start.

Updated PR.

Another update.

This looks good to me overall.  The only thing I don't like so much is that the scripts directory should not become a garbage collector.  I thought it was originally meant for example configuration scripts.  This is not a big deal though, we can move things around later.
Also another thought, when this PR and #173 get merged, we need to decide whether or not to remove check_format_cpp and check_no_trailing_whitespaces_or_tabs from the test suite on Jenkins, and, more generally, whether we want to deprecate DataTransferKit_ENABLE_ClangFormat.

The only thing I don't like so much is that the scripts directory should not become a garbage collector. I thought it was originally meant for example configuration scripts.

I don't disagree, but I don't know a better option. We would occasionally need some pseudo-random scripts to put somewhere.


assert account balance <= min balance in newly created accounts?

We disable it explicitly in https://github.com/MinaProtocol/mina/pull/6722. Can this check be added back if we were to allow new time locked accounts?

This has been done for the berkeley upgrade.


Refresh failure alert

Is your feature request related to a problem? Please describe.

Describe the solution you'd like

Describe alternatives you've considered

Additional context

Hi there, is there anyway we can have an alert when the page stops refreshing

I'd like additional things related to this:

An Alert when it pops up "You are about to be logged out."

A Click Time Randomizer.  "Click every x seconds +/- a random number between xxxx and xxxx ms."

Local file storage of things like:
- preferred dates (so I don't have to reload them every time I'm logged out.)
- user information


Adding ability to switch between front and rear facing camera.

This will allow the user to switch between front and rear-facing camera and record the video by adding one camera toggle button.  (See the screen shot.)

The developer can also open front-facing camera by default by passing VideoCaptureActivity.EXTRA_CAMERA_FACING parameter.

The developer can also disable camera toggle switch by passing false to allowFrontFacingCamera while creating CaptureConfiguration. By default, camera toggle switch is on.

Awesome @kevalpatel2106 !
I'm a bit busy currently, but I will try to merge things next week.

@JeroenMols Thank you very much.

Coverage decreased (-4.1%) to 74.161% when pulling 9cd66203cee55d2159d49020cf7c30e0d2f0a8ad on kevalpatel2106:master into<PHONE_NUMBER>1c86e66e5a6e5cc7807ab97c23d81b on JeroenMols:master.

Current coverage is 70.51% (diff: 52.83%)

Merging #34 into master will decrease coverage by 9.24%

@@             master        #34   diff @@
==========================================
  Files            13         16     +3
  Lines           583        685   +102
  Methods           0          0
  Messages          0          0
  Branches         52         78    +26
==========================================
+ Hits            465        483    +18
- Misses          102        178    +76
- Partials         16         24     +8

Powered by Codecov. Last update 5741790...9cd6620

Awesome @kevalpatel2106 and @danielgomezrico !
Overall:

Very much appreciate the short concise commits
Thanks for sticking to the same coding style
Very cool animation! Is that why you restart the activity of was there another limitation?
Commits look pretty good, and all test are running -> Am going to do a final check to see if the bigger picture still "fits"
Added few comments to the commit
There is an issue where my preview is  upside down (nexus 6p). -> I'll look into that

Will try to wrap this up asap, sorry for the delay already gentlemen!

Hi guys, I did some minor refactors and merged the code into master.
Thanks again for your awesome contribution @kevalpatel2106  and @danielgomezrico.
Could you install the latest app from Google play and test if the orientation is fine in portrait?

Could you install the latest app from Google play and test if the orientation is fine in portrait?

@JeroenMols Sure. Let me try the app from the google play.

Very cool animation! Is that why you restart the activity of was there another limitation?

Actually, I was not able to switch between front and rear-facing camera while the preview is running continually. So, I looked into the Google Camera app and I figure out that they are restarting the activity while changing the camra. So, I did the same thing and code works. I added the animations to override default android activity transition animation as they were not looking pretty cool. :-)
Thank you for your feedbacks.

@kevalpatel2106 I just fixed an issue this morning -> after camera switch, a new activity was started on top of the old one. When confirming recording the result however would not be propagated back and you would get a "recording failed".
Fixed that this morning, so you'll have to wait for the 1.2.2 app to be live on Google play.

@JeroenMols Ok. I'll wait for that.

@kevalpatel2106 App should be live. (had to uninstall it to get the updated version though)

@JeroenMols I tested it. It is working as expected in my device Motorola G3 Android 6.0.1.

Awesome!

@kevalpatel2106 @kevalpatel2106 Guys I tested on Nexus 5X and the camera is upside down too.

@caipivara Now it's resolved, right?


Workspace settings should inherit user settings when first initialized

Currently, when opening a folder for the first time, vscode creates a .vscode/settings.json file, aka workspace settings. This file is populated with default settings, which makes user settings pretty...useless.
I spent a lot of time carefully crafting my user settings only to have to re-apply the same changes for every new workspace. As-is, user settings only apply to individual files outside of workspaces, which makes them pretty useless.
It would be much better if, upon first opening a folder, the workspace settings were inherited from the user settings and could then be tailored to the specific workspace form there, rather than starting 'from scratch' everytime.

/confirmPlease

Please perform the following three tasks to diagnose the root cause of the issue:

[ ] 1.) Disable Extensions

Select View and pick Command Palette...
Run Developer: Reload With Extensions Disabled
👉 See if the issue reproduces

[ ] 2.) Disable Configuration

Select View and pick Command Palette...
Run Profiles: Create a Temporary Profile
👉 See if the issue reproduces

[ ] 3.) Try VS Code Insiders

Download VS Code Insiders
Install and Run it
👉 See if the issue reproduces

Then pick one of the three resolutions depending on which step has helped:

Disabling my Extensions helped
Please run the command Start Extension Bisect and follow the instructions to find the extension that is causing this issue.

Please report the issue to the extension causing this.

Disabling my configuration helped
Please report back more details about your configuration, including settings.

Using VS Code Insiders has helped
✅ This likely means that the issue has been addressed already and will be available in an upcoming release. You can safely use VS Code Insiders until the new stable version is available.

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


Cleanup language support on TTS

Description:
After we support language also on service call, I make a small refactory of language support for future. Now it is also possible to check language on service call and  protect cache System for produce not correct files.
If the code does not interact with devices:

[x] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[x] Tests have been added to verify that the new code works.

@balloob done

@balloob done. I have remove this code while I had implement this check on service but I was blind to see that was not needed anymore.


Disallow properties not in schema

I can't seem to find anything about this in the documentation.
Is there a way to throw a validation error if a property not in the schema is present in the payload?
eg if uuid is "disallowed" and its present in the JSON being validated, it errors as invalid

Please read JSON schema specification or docs/json-schema.md . For this particular problem you need “required” keyword

Please read JSON schema specification or docs/json-schema.md . For this particular problem you need “required” keyword

@epoberezkin 'required' is the wrong keyword. That defines properties that must be included. I want to make sure no extra properties not in the schema are rejected.
Turns out I needed additionalProperites: false

@epoberezkin 'required' is the wrong keyword. That defines properties that must be included. I want to make sure no extra properties not in the schema are rejected.
Turns out I needed additionalProperites: false


[Feature Request]: Lower log-level for caught error in StorageApiWritesShardedRecords

What would you like to happen?
We log an error here:
https://github.com/apache/beam/blob/e5e3cb25ca4fc2e31c10eb3dbda8289c6bfc7140/sdks/java/io/google-cloud-platform/src/main/java/org/apache/beam/sdk/io/gcp/bigquery/StorageApiWritesShardedRecords.java#L447-L448
But I've seen that these errors can be benign, e.g. ALREADY_EXISTS: The offset is within stream, expected offset ... can happen when re-establishing a stream, but doesn't indicate a failure, so logging it as an error is misleading.
Issue Priority
Priority: 2
Issue Component
Component: io-java-gcp

@reuvenlax should we lower the log level here? I'm not sure if there are other cases where logging an error is appropriate, but it seems like at least in the case I noted it's not.


Add cache support

Add an abstraction that gives opportunity to hook up different cache providers for application data caching.
As an example Java caching API can be used (https://www.javadoc.io/doc/javax.cache/cache-api/latest/index.html)

There are three areas that need to be resolved:

What annotations to use in sources
How to choose caching implementation (and is there an existing SPI we could use for it)
Implement interceptors in SE and in MP (very very similar to Fault Tolerance implementation)

Currently there is no MP Cache specification

design in Helidon

annotations
SPI
SE API

first SPI implementation could be JCache (to support all existing JCache impls)
JCache -
https://www.javadoc.io/doc/javax.cache/cache-api/1.0.0/index.html
https://github.com/jsr107/jsr107spec

Coherence implements JCache using a dedicated library
others support it as well - https://www.baeldung.com/jcache

Then we could have a dedicated native impl for each provider
we would need to have our own SPI with default implementation using JCache but supporting native impl instead

Is JCache alive?
Probably not - not upgraded to jakarta
Helidon SE
Either use a dedicated API (Similar to FT), or just use directly cache, as you control everything
Cache cache = Cache.create(...);
 Cache.execute(Object... keys, Supplier<Single> supplier);
String user;
cache.execute(user, () -> createMessage(user))
.forSingle(response::send);

Removing from in-progress, as we do not have major feedback on this.


Add getKeyByCurve method

Description
Add new method for get private key by curve type
How to test
Will add test case
Types of changes

New feature (non-breaking change which adds functionality)

Checklist

[x] Create pull request as draft initially, unless its complete.
[x] Add tests to cover changes as needed.
[x] Update documentation as needed.
[x] If there is a related Issue, mention it in the description.

If you're adding a new blockchain

[x] I have read the guidelines for adding a new blockchain.

Could you explain use case where this method is needed?

Could you explain use case where this method is needed?

A few blockchains allow different curves, polkadot / flow

I understand now the motivation. Conceptually it's fine with me.
I have some issue with the implementation:

current version of getKeyByCurve does not work correctly with all curves, namely with special PrivateKeyTypeDoubleExtended (Cardano)
adding the special handling also to getKeyByCurve would mean code duplication.

Hence my proposal, to build getKey on top of getKeyByCurve.  I think implementation is nicer this way.
Code (and diff) would look like:

PrivateKey HDWallet::getKey(TWCoinType coin, const DerivationPath& derivationPath) const {
    const auto curve = TWCoinTypeCurve(coin);
    return getKeyByCurve(curve, derivationPath);
}

PrivateKey HDWallet::getKeyByCurve(TWCurve curve, const DerivationPath& derivationPath) const {
    const auto privateKeyType = getPrivateKeyType(curve);
    const auto node = getNode(*this, curve, derivationPath);
    switch (privateKeyType) {
        case PrivateKeyTypeDoubleExtended: // special handling for Cardano
            {
                if (derivationPath.indices.size() < 4 || derivationPath.indices[3].value > 1) {
                    // invalid derivation path
                    return PrivateKey(Data(192));
                }
                const DerivationPath stakingPath = cardanoStakingDerivationPath(derivationPath);

                auto pkData = Data(node.private_key, node.private_key + PrivateKey::size);
                auto extData = Data(node.private_key_extension, node.private_key_extension + PrivateKey::size);
                auto chainCode = Data(node.chain_code, node.chain_code + PrivateKey::size);

                // repeat with staking path
                const auto node2 = getNode(*this, curve, stakingPath);
                auto pkData2 = Data(node2.private_key, node2.private_key + PrivateKey::size);
                auto extData2 = Data(node2.private_key_extension, node2.private_key_extension + PrivateKey::size);
                auto chainCode2 = Data(node2.chain_code, node2.chain_code + PrivateKey::size);

                return PrivateKey(pkData, extData, chainCode, pkData2, extData2, chainCode2);
            }

        case PrivateKeyTypeDefault32:
        default:
            // default path
            auto data = Data(node.private_key, node.private_key + PrivateKey::size);
            return PrivateKey(data);
    }
}

Since there is PR became inactive, I took it over in #2473 .

Hi @catenocrypt
Sorry for the late reply, got stuck on other stuff.  :(
Thanks for raising new PR, I will loop in that one.


⚡ use HdrHistogram to aggre

What is the purpose of the change
Brief changelog
Verifying this change

Checklist

[x] Make sure there is a GitHub_issue field for the change (usually before you start working on it). Trivial changes like typos do not require a GitHub issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[ ] Each commit in the pull request should have a meaningful subject line and body.
[ ] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[ ] Check if is necessary to patch to Dubbo 3 if you are work on Dubbo 2.7
[ ] Write necessary unit-test to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add sample in dubbo samples project.
[ ] Add some description to dubbo-website project if you are requesting to add a feature.
[ ] GitHub Actions works fine on your own branch.
[ ] If this contribution is large, please follow the Software Donation Guide.

sample as follow:

import org.HdrHistogram.ConcurrentDoubleHistogram;
import org.HdrHistogram.ConcurrentHistogram;

public class HistogramExample {
    public static void main(String[] args) {
        // 创建 ConcurrentHistogram 对象
        ConcurrentDoubleHistogram histogram = new ConcurrentDoubleHistogram(2);
        // 向直方图中添加数据
        histogram.recordValue(1);  // 假设添加了多个值
        histogram.recordValue(2);  // 假设添加了多个值
        histogram.recordValue(3);  // 假设添加了多个值
        histogram.recordValue(4);  // 假设添加了多个值
        histogram.recordValue(5);  // 假设添加了多个值
        histogram.recordValue(6);  // 假设添加了多个值
        histogram.recordValue(7);  // 假设添加了多个值
        histogram.recordValue(8);  // 假设添加了多个值
        histogram.recordValue(9);  // 假设添加了多个值
        histogram.recordValue(10);  // 假设添加了多个值

        double valueAtPercentile = histogram.getValueAtPercentile(80.0);
        System.out.println(valueAtPercentile);
    }
}

result is：
8.0546875

Codecov Report

Merging #12868 (fff4104) into 3.2 (1dd248a) will increase coverage by 2.46%.
The diff coverage is n/a.

:exclamation: Current head fff4104 differs from pull request most recent head 4fdbd9c. Consider uploading reports for the commit 4fdbd9c to get more accurate results

@@             Coverage Diff              @@
##                3.2   #12868      +/-   ##
============================================
+ Coverage     66.98%   69.44%   +2.46%
  Complexity        2        2
============================================
  Files          1782     1648     -134
  Lines         71819    68370    -3449
  Branches      10258     9987     -271
============================================
- Hits          48106    47480     -626
+ Misses        19085    16318    -2767
+ Partials       4628     4572      -56

see 172 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

After the research is completed, we will raise the PR


Legacy VTK format has undefined data formats in vtk_to_numpy_dtype_name

The code for legacy VTK has these types in vtk_to_numpy_dtype_name:
vtk_to_numpy_dtype_name = {
    "float": "float32",
    "double": "float64",
    "int": "int",
    "vtktypeint8": "int8",
    "vtktypeint16": "int16",
    "vtktypeint32": "int32",
    "vtktypeint64": "int64",
    "vtktypeuint8": "uint8",
    "vtktypeuint16": "uint16",
    "vtktypeuint32": "uint32",
    "vtktypeuint64": "uint64",
}

But out of these, VTK actually uses only float, double, int, vtktypeint64 and vtktypeuint64 (plus some more, like unsigned int). So if you write an array with e.g. vtktypeint32, VTK will give an unsupported data type error.
The relevant code in VTK is here: https://github.com/Kitware/VTK/blob/b9f028db9f801258d4c1c0c97716ea7a3c14c289/IO/Legacy/vtkDataReader.cxx#L1546-L1828
(I actually did not encounter this problem while using meshio, it might even be a non-issue in practical code paths. But I thought that I'd let you know...)

The code looks quite a bit different now, and it got fixed somewhere along the way. Thanks for letting me know anyway!


Build /register endpoint

In the /api package, create the /register endpoint to the api so that workers can phone home.
After a struct is defined for passing back and forth in the /sniffer package, extend the api endpoint to accept that data and log it into the SQLite database.

Finished in 0b0686fdbc6e090edc72d15aab4dc772d94c1ef2


Ampersand getting encoded as &

Ampersand in content targeted is getting converted to &. Is there an option I am missing that would avoid this? Thanks

Sorry, it turns out that my implementation was the issue. I was using this with Drupal behaviors and it was getting re-applied a few times. Thank you for this library.

You're welcome
Thank you for the feedback!


delete_squashed_migrations can't delete multi-line "replaces" list

When Migration.replaces defined as multi-line list, like
    replaces = [
        ("lost", "0001_initial"),
        ("lost", "0002_alter_category_options"),
    ]

delete_squashed_migrations fails with
CommandError: Couldn't find 'replaces =' line in file /code/src/lost/migrations/0001_0012_squashed.py. Please finish cleaning up manually.

It completes successfully when replaces definition is done on a single line.
Tested with django-extensions = "3.1.5"

Also got bitten by this today.

Same here... I'm pretty sure this is because of the adoption of black for management commands in Django 4.1.

Same here... I'm pretty sure this is because of the adoption of black for management commands in Django 4.1.

This is because of https://github.com/django-extensions/django-extensions/blob/main/django_extensions/management/commands/delete_squashed_migrations.py#L121

Also got bitten by this today.

you can do this
black -l 1000000 <app>/migrations/*.py
python manage.py delete_squashed_migrations <app>
black  <app>/migrations/*.py


git mv .github/workflows/tests.yml .github/workflows/ci.yml

According to the style guide. And so that the repo auto-configurer can detect GitHub Actions.

A single non-Jenkins CI config file will not in any way affect the e2e tests, so I'm skipping them.


feat: backfill id-token permissions for github actions

Sadly github is too smart for this. It will not let you update permissions for an action file when that action file doesn't already have the correct permissions.


Add iflight f411 rx target

New bord: iflight f411 rx
I can't find the name of the board IFLIGHTF411RX in https://github.com/betaflight/unified-targets
BTFL_cli_Baby_Nazgul 73_20211026_145412.txt

indeed there does not seem to be a matching unified target for this board...not the best performance on iflights part.

support for the CC2500_LNA_EN_PIN situation landed

support for the CC2500_LNA_EN_PIN situation landed

What do you mean ?

we currently do no support USE_CC2500_PA_LNA without a CC2500_LNA_EN_PIN, however this is easy enough to add. please remove CC2500_LNA_EN_PIN

I pushed a change to the firmware to make it possible have the above situation work correctly - so once you remove CC2500_ANT_SEL_PIN, USE_CC2500_DIVERSITY and CC2500_LNA_EN_PIN we should be good to go.

Is it good now ?


Bit.Client.Web.BlazorUI & Bit.Tooling.BSwup nuget packages contain irrelevant contents

Before project structure refactor, these projects had
        <IncludeContentInPack>false</IncludeContentInPack>

But since refactoring, they're relying on Bit.Build.props file for such settings which doesn't provide such a setting.


Cross-compiling for ARMV7?

Hi! There are some suggestions in the README that this perf-version was designed with cross compilation in mind etc and there are some very brief instructions on how to cross compile it. Could you please expand this into a working example?
For example, on Ubuntu 20.04 you can install gcc-9-arm-linux-gnueabihf and in it you can find a gcc (the "linker" for Rust .config) but where do you find the "sys-root"?

You might not necessarily need to specify the sysroot. (It depends on how the compiler was configured/built.)
For Ubuntu Focal something like this should work I think:

Install the cross compiler: apt install g++-arm-linux-gnueabihf libc6-dev-armhf-cross
Install Rust.
Install Rust's ARM target: rustup target add armv7-unknown-linux-gnueabihf
Put this in your .cargo/.config:

[target.armv7-unknown-linux-gnueabihf]
linker = "/usr/bin/arm-linux-gnueabihf-gcc"

Compile with cargo build --release --target=armv7-unknown-linux-gnueabihf.

Thanks, yeah I eventually managed to compile it but I had to spend many hours scourging stack overflow confused posts from a lot of other people who also had problems cross-compiling with Rust :) (not compiling not-perf in particular)
The main issue is that when you install the Ubuntu 20.04 rust packages you don't get rustup and they don't provide the necessary pre-built std/core crates for armv7 either, so you get confused and nothing works and the various tutorials on the net are confusing if you try to use the Debian packages; there are multiple ways to set this up it seems. And you just get weird error messages from rust about the std/core crates missing and if you don't know anything about Rust this just adds to the confusion :)
I ended up having to remove all of the Ubuntu rust-related packages, and instead use the rust shell script to set it up, do the rustup target add and added the cross compiler to the .config like you wrote and then it compiled.

However, while it compiled properly, I never got any binaries. In the release dir there is only a couple of libs it seems, any suggestions?

~/tmp/not-perf/target/armv7-unknown-linux-gnueabihf/release$ ls -l
total 7904
drwxrwxr-x 28 bjorn bjorn    4096 Jul  7 18:05 build
drwxrwxr-x  2 bjorn bjorn   16384 Jul  7 18:06 deps
drwxrwxr-x  2 bjorn bjorn    4096 Jul  7 17:11 examples
drwxrwxr-x  2 bjorn bjorn    4096 Jul  7 17:11 incremental
-rw-rw-r--  1 bjorn bjorn    2290 Jul  7 18:06 libnperf_core.d
-rw-rw-r--  2 bjorn bjorn 8058580 Jul  7 18:06 libnperf_core.rlib

Try cargo build --release --target=armv7-unknown-linux-gnueabihf --workspace. The root dir defines the nperf_core crate it seems. The cli is found in a different crate in a subdirectory. --workspace causes the whole workspace to be built instead of just the root nperf_core.

Ah, right, I forgot to update README after that change! Yes, the bin crate is no longer in the root; if you go into the cli directory and build from there it will build properly.

Thanks both, going into cli/ and running the build with the appropriate --target produced the nperf binary (in the top-level target/ structure). And it seems to execute on my target.
Now for finding out how to use the tool :)