[occm] Octavia OVN Provider create listener error

/kind bug
What happened:
Setting lb-provider=ovn works, and allows the user to provision a Octavia OVN LoadBalancer, but breaks while creating the Listener. The configuration looks like so:
[LoadBalancer]
lb-provider=ovn
use-octavia=true
floating-network-id=180941c5-9e82-41c7-b64d-6a57302ec211

However, it fails to create the Listener with the following error:
I1205 11:33:24.169540       1 event.go:291] "Event occurred" object="awx/awx-web-lb" kind="Service" apiVersion="v1" type="Normal" reason="EnsuringLoadBalancer" message="Ensuring load balancer"
E1205 11:33:24.517822       1 controller.go:275] error processing service awx/awx-web-lb (will retry): failed to ensure load balancer: failed to create listener for loadbalancer 6c670e04-7c66-4f1a-9a01-9ad2ba7e2a92: Expected HTTP response code [] when accessing [POST https://openstack.bne-home.net:13876/v2.0/lbaas/listeners], but got 501 instead
{"faultcode": "Server", "faultstring": "Provider 'ovn' does not support a requested option: OVN provider does not support allowed_cidrs option", "debuginfo": null}
I1205 11:33:24.518078       1 event.go:291] "Event occurred" object="awx/awx-web-lb" kind="Service" apiVersion="v1" type="Warning" reason="SyncLoadBalancerFailed" message="Error syncing load balancer: failed to ensure load balancer: failed to create listener for loadbalancer 6c670e04-7c66-4f1a-9a01-9ad2ba7e2a92: Expected HTTP response code [] when accessing [POST https://openstack.bne-home.net:13876/v2.0/lbaas/listeners], but got 501 instead\n{\"faultcode\": \"Server\", \"faultstring\": \"Provider 'ovn' does not support a requested option: OVN provider does not support allowed_cidrs option\", \"debuginfo\": null}"

What you expected to happen:
When a operator sets OVN as the provider, we will need to adjust the API request sent to Octavia to ensure the request is valid.
How to reproduce it:

Ensure you have Octavia OVN installed on OpenStack;
Add the above mentioned options to the [LoadBalancer] section in cloud.conf
Create a Service with type: LoadBalancer
Observe the LoadBalancer itself is created, but it will fail while creating the listener, because it's trying to use the allowed_cidr option which is not supported by the OVN provider

Anything else we need to know?:
It looks like there are efforts to support this in the ovn_octavia_provider:
https://opendev.org/openstack/ovn-octavia-provider/src/branch/master/ovn_octavia_provider/driver.py#L61-L69
But this breaks for Kubernetes users attempting to use it. So I believe we either need to block lb-provider=ovn, or we need to create the listener without allowed_cidrs.
We can consider this a feature request if you prefer. But since it breaks for users, I raised this as a bug. Let me know if you need any additional logs
Environment:
Kubernetes environment version is: v1.19.0
Server Version: 4.6.0-0.okd-2020-11-27-200126
Kubernetes Version: v1.19.0-rc.2.1077+43983cda8af930-dirty

This is the image I'm using for occm:
https://hub.docker.com/layers/k8scloudprovider/openstack-cloud-controller-manager/latest/images/sha256-e9d3de3c0cbc6edca78f0e5ca0cdb7643af10225feed88fa6c1bdd90fc90139d?context=explore

OpenStack version: Victoria

Others:
Octavia Version:

[root@overcloud-controller-0 ~]# podman exec octavia_api rpm -qa | grep octavia
python3-octavia-lib-2.2.0-0.20201118104745.fe022cd.el8.noarch
openstack-octavia-common-7.1.0-0.20201203065445.9d38db9.el8.noarch
python3-ovn-octavia-provider-0.4.0-0.20201118135846.5b0715d.el8.noarch
python3-octavia-7.1.0-0.20201203065445.9d38db9.el8.noarch
python3-octaviaclient-2.2.0-0.20201112084714.78c08ed.el8.noarch
openstack-octavia-api-7.1.0-0.20201203065445.9d38db9.el8.noarch

I agree we could check lb-provider to only allow octavia(for backward compatibility) and amphora, amphorav2 is still not production ready though.

I think it is probably best if we check for lb-provider if it's NOT octavia or amphora, then we default to octavia or amphora? That way we aren't breaking if a use does specify something else and still giving our selves a chance at working.
Worst case scenario that I can think of then is that they didn't deploy the Amphora image (Like me who is just using OVN) and Octavia will fail to create the Load Balancer. But at least there wont be a Active load balancer in OpenStack that appears to have worked?

if it's NOT octavia or amphora, then we default to octavia or amphora?

We only need to check when occm is starting up, if lb-provider is not supported, fail and exit.

Hey,
Sorry, I might have needed to include something extra in my commit message to have it linked to this Issue. Please advise if I'm missing something there. The relevant PR is:
https://github.com/kubernetes/cloud-provider-openstack/pull/1344
And some artifacts from my change:
I1209 03:51:37.075114       1 controllermanager.go:127] Version: v0.0.0-master+$Format:%h$
W1209 03:51:37.075575       1 openstack.go:306] failed to read config: Unsupported LoadBalancer Provider: ovn
F1209 03:51:37.075641       1 controllermanager.go:131] Cloud provider could not be initialized: could not init cloud provider "openstack": Unsupported LoadBalancer Provider: ovn
goroutine 1 [running]:

Hmm, not too sure why that test is failing. Is it possible to get more info from this check?
https://logs.openlabtesting.org/logs/44/1344/0b75c75f8b258edf131dea106ed5e4f53c96c7c2/cloud-provider-openstack-acceptance-test-lb-octavia/cloud-provider-openstack-acceptance-test-lb-octavia/7fd0fa1/job-output.txt.gz
2020-12-09 04:27:44.815992 | TASK [Wait for openstack-cloud-controller-manager up and running]
2020-12-09 04:32:37.756204 | ubuntu-bionic | ERROR
2020-12-09 04:32:37.756822 | ubuntu-bionic | {
2020-12-09 04:32:37.756913 | ubuntu-bionic |   "attempts": 24,
2020-12-09 04:32:37.756992 | ubuntu-bionic |   "delta": "0:00:06.699446",
2020-12-09 04:32:37.757115 | ubuntu-bionic |   "end": "2020-12-09 04:32:37.701453",
2020-12-09 04:32:37.757191 | ubuntu-bionic |   "msg": "non-zero return code",
2020-12-09 04:32:37.757264 | ubuntu-bionic |   "rc": 1,
2020-12-09 04:32:37.757373 | ubuntu-bionic |   "start": "2020-12-09 04:32:31.002007"
2020-12-09 04:32:37.757447 | ubuntu-bionic | }


Add tests for comments counter

In this branch I add tests to the comment counter

Hi Team,
While you made a great effort in this project, unfortunately, I cannot proceed to review your code.
Invalid Code Review Request

[ ]  You have submitted a project, that has incorrect gitflow/ Github flow. Kindly note that the gitflow of this project should have merge request flow from development into main. Your team members are supposed to give reviews to each other and merge their feature branches into the dev branch, which your submission from dev branch into main. Kindly view this article: curriculum-transversal-skills for further instructions on the same.
[ ]  Kindly also ensure that all required linter checks are showing/ correctly passing on your Pull Request

Your Code Review Request will be marked as invalid in your Dashboard, so please submit a new one once you are ready 🙏
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear. Please, remember to tag me in your question so I can receive the notification.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.
Invalid Code Review Request does not count into the code reviews limit.

Thank you @rloterh , my bad indeed.


Add screenshots for v6 web interface

What does this PR aim to accomplish?
Add a few screenshots of the updated web interface.

Dashboard updated design;
new Query Log page and Advanced Filter;
Domain Management details;
new online API docs.

By submitting this pull request, I confirm the following:

I have read and understood the contributors guide, as well as this entire template. I understand which branch to base my commits and Pull Requests against.
I have commented my proposed changes within the code and I have tested my changes.
I am willing to help maintain this change if there are issues with it later.
It is compatible with the EUPL 1.2 license
I have squashed any insignificant commits. (git rebase)
I have checked that another pull request for this purpose does not exist.
I have considered, and confirmed that this submission will be valuable to others.
I accept that this submission may not be used, and the pull request closed at the will of the maintainer.
I give this submission freely, and claim no ownership to its content.

[x] I have read the above and my PR is ready for review. Check this box to confirm

Can you compress the gif - it's over 1 MB?

I did. It was over 3 MB and I reduced it to the current size.


Increase lsm_batch_multiple to 64.

This is roughly the value we plan to use in production, so we should be benchmarking this configuration as early as possible.
This increases memory usage and increase the latency spike from the final phase of compaction, but we need to deal with both regardless.
https://github.com/tigerbeetledb/tigerbeetle/issues/491 and https://github.com/tigerbeetledb/tigerbeetle/issues/490 can help with memory usage.
https://github.com/tigerbeetledb/tigerbeetle/issues/269 will reduce the latency spike, as will any optimization that improves compaction performance in general.
Pre-merge checklist
Performance:

[x] Compare zig benchmark on linux before and after this pr.

lbm=4
1643655168 allocated
1223 batches in 116.60 s
load offered = 1000000 tx/s
load accepted = 85766 tx/s
batch latency p00 = 5 ms
batch latency p10 = 26 ms
batch latency p20 = 27 ms
batch latency p30 = 32 ms
batch latency p40 = 36 ms
batch latency p50 = 42 ms
batch latency p60 = 112 ms
batch latency p70 = 142 ms
batch latency p80 = 172 ms
batch latency p90 = 217 ms
batch latency p100 = 411 ms

lbm=64
2989330432 bytes allocated
1223 batches in 93.51 s
load offered = 1000000 tx/s
load accepted = 106942 tx/s
batch latency p00 = 5 ms
batch latency p10 = 26 ms
batch latency p20 = 26 ms
batch latency p30 = 27 ms
batch latency p40 = 31 ms
batch latency p50 = 31 ms
batch latency p60 = 32 ms
batch latency p70 = 36 ms
batch latency p80 = 37 ms
batch latency p90 = 42 ms
batch latency p100 = 3624 ms

OR

[ ] I am very sure this PR could not affect performance.

bors d+
(d+ since its a draft, but this change LGTM).

Oops. I guess the button defaults to whatever I picked last?

bors r+

bors retry


Correcting small issues

Just a small correction

The last commit tries to solve of the get_ready_states result show in precompilation.


Classes gone when restarting UE Editor

Hello,
When making the widgets, I can choose the "Focusable" user widget class. When I save everything and restart my Editor/Computer, the parent classes of the widgets are getting lost:

I followed the installation and included everything in Build.cs file (I am also using your other helpers like SPUD etc... and they all work).
I already deleted all cache folders (Binaries, DerivedDataCache, Intermediate etc...) and fully rebuild the project in VS code etc...
Any ideas?
(I'm using UE5.1, the SUDS example project also works with this version)

I've started to occasionally get this recently in one of my example projects and I haven't been able to figure out why, except that simply restarting the editor usually fixes it. I've recently added a plugin entry for StevesUEHelpers to my project to see if that makes a difference - I've seen the problem once since then but it seems to be less often? I've never had this problem until 5.1 so I think it might be an editor bug, it's just hard to reproduce since it goes away on its own.

I'm starting to wonder if it's a timing problem, perhaps changing the LoadingPhase might help

Hello,
I can confirm that adding StevesUEHelpers to .uproject modules list and changing the LoadingPhase to "PreDefault" works like a charm.
{
	"Name": "StevesUEHelpers",
	"Type": "Runtime",
	"LoadingPhase": "PreDefault"
}

Thanks, I've changed the LoadingPhase, hopefully this goes away for good this time!


WIP: use gobwas/ws instead of gorilla/websocket

Related to https://github.com/knative/pkg/pull/2730, https://github.com/knative/serving/pull/13932, https://github.com/knative/serving/issues/13824
Changes
https://github.com/gorilla/websocket is deprecated and unmaintained.
We should switch to something that is and ensure everything works as expected (ie. draining in queue proxy etc).
replaced gorilla/websocket with gobwas/ws

:broom: Update or clean up current behavior

/kind deprecation
Fixes #800
Release Note
/kind deprecation

Hi @dprotaso,
I've created a WIP in the downstream serving repo to test the changes I've made.
Could you have a look?
Thanks,


Adding a resources page

Why are these changes needed?
Placeholder for listing growing number of resources
Related issue number (if any).

I do like the idea of having this resource page, but I saw @Bytes-Explorer 's reservations about doing this. I discussed it with Nirmit this morning and his general direction is not to make the README page longer than what it needs to be and by not having a separate resource page for these links, the README keeps getting longer. In any case, let's discuss the pros/cons in our Tuesday meeting, before approving or not.

I do like the idea of having this resource page, but I saw @Bytes-Explorer 's reservations about doing this. I discussed it with Nirmit this morning and his general direction is not to make the README page longer than what it needs to be and by not having a separate resource page for these links, the README keeps getting longer. In any case, let's discuss the pros/cons in our Tuesday meeting, before approving or not.

I am also in favor of compact top level README.  Let's discuss.

I dont have a strong opinion here. My point is that it should be easy for the user to find the information and usually I have seen repos have this information in the top level readme. @touma-I Aanchal - pls share your suggestions too.

@sujee Need to sync with upstream and update branch with latest before editing latest.

done.  please check again. thx


[ENHANCEMENT] Camunda Optimize connection with AWS Opensearch using IRSA

Hello Team,
We are currently testing Camunda in our development environment and are aiming to enhance our setup for production using AWS Opensearch Service. We've noticed that most of Camunda's components support connecting to Opensearch using IRSA, with the exception of Optimize and it needs username and password https://docs.camunda.io/docs/self-managed/setup/deploy/amazon/amazon-eks/irsa/#amazon-opensearch-service
Could you kindly provide any information on when Optimize might support this functionality?
Thanks

@camunda/optimize , @grlimacan    Can you link a camunda/optimize ticket for supporting IRSA in AWS?

@camunda/optimize , @grlimacan Can you link a camunda/optimize ticket for supporting IRSA in AWS?

There you go: https://github.com/camunda/camunda-optimize/issues/12696

or if you prefer the public issue: https://github.com/camunda/issues/issues/739

I'm going to close this issue since there is already a feature request for the Optimize team.


dotnet-sdk: properly copy license files

Description of changes
After changing license files to be in /share/doc in https://github.com/NixOS/nixpkgs/pull/194061, the combining script required fixes.

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)
[x] 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

[x] 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/1300

@GrahamcOfBorg build dotnet-sdk_3 dotnet-sdk_5 dotnet-sdk_6 dotnet-sdk_7


console docs: change instructional steps to lists

fixes #345
pls note im not adding any new content in this pr, i am changing instructional steps that are presented in paragraphs to lists to make them easier to consume and use, more details in #345 issue description.

@cnunciato im gonna merge this so i can follow up with the transfer pr. super happy to address any concerns in a future pr.


Settings screen - "Remove current wallet" make it in red color

Please make "Remove current wallet" in red color
https://zpl.io/bep6zrD

checked


⚠ SolarOscp has degraded performance

In 1f349dc, SolarOscp ($SOLAROSCP_URL) experienced degraded performance:

HTTP code: 200
Response time: 184 ms

Resolved: SolarOscp performance has improved in 9a58944 after 10 minutes.


Let’s frontend!

These changes are the only ones I’m confident making without being able to build the site locally (see #42)

If it helps at all, I tried to make the commits understandable at an atomic level.

Closing this, since #76 updates the image optimization, and the unused components/typo in the README have since been fixed.


Provide Application class instead of QualityMattersApp

Related to #78 and https://github.com/google/dagger/issues/328
Unfortunately, this does not remove warning from Dagger, but it makes design of modules a little bit better and easier to reuse in other projects.
Initially, I thought that it removed the warning because I did assemble without clean and annotation processing was skipped.

See https://travis-ci.org/artem-zinnatullin/qualitymatters/builds/113676485#L1561

@vanniktech PTAL :)

:+1: the problem does still exist though

Yeah, I'll update issue in Dagger


Parameter 'Alias' doesn't affect local tables  if  distributed materialized is used.

Parameter 'Alias' doesn't affect local tables  if  distributed materialized is used.
Steps to reproduce

Create model with name 'test'
Set follow parameters:
{{ config(
alias='alias_for_test_table',
materialized='distributed_incremental',
)
}}
run dbt model.
The distributed table will be called 'alias_for_test_table' but local table will be called 'testlocal'
Change model's name to 'new_test_name'.
Run dbt model with parameter --full-refresh
get an error: "DB::Exception: Table new_test_namelocal doesn't exist."

Expected behaviour
The distributed table is called 'alias_for_test_table'  and local table is called 'alias_for_test_tablelocal'
Configuration
Environment
Plugins:

clickhouse: 1.4.8 - Up to date!

dbt version: 1.4.7

dbt-clickhouse version:1.4.8

Python version:

Operating system: MacOS

I will take a look.

commit a fix with PR https://github.com/ClickHouse/dbt-clickhouse/pull/186
waiting for review


Add a fish shell completion file for the Godot editor

Follow-up to #34957 and #35602. Our completion game is almost complete now :wink:
To test it, save godot.fish into ~/.config/fish/completions/ (create if needed) then restart your shell.
Preview

Thanks!

Cherry-picked for 3.2.1.


Is Bash really required for hooks?

Can I use another executable program, e.g. Go script*, typescript (with shebang indicating ts-node), or even compiled program, as long as it is directly executable, or do I have to use bash?

https://blog.cloudflare.com/using-go-as-a-scripting-language-in-linux/

Presently shell script or windows batch is required because we inject some code to allow us to capture exported environment variables.
If you didn't care too much about capturing exported environment variables, it would be possible to use anything executable I believe!

Thanks!
Out of curiosity: Why inject environment variables, why not set it before calling the script? If that were done/possible, we could use any script/executable.

@newtack the issue is extracting exported environment variables not injecting them! We figure out what variables a script exports and make them available to subsequent hooks.


test-docker-ubuntu2110-armv8l-1 : 13:28:02  ./get.sh: line 595: /home/jenkins/workspace/Test_openjdk8_hs_extended.functional_arm_linux/openjdkbinary/j2sdk-image/bin/java: No such file or directory

Test build failed at the setup stage. It only happened to this machine. test-docker-ubuntu2110-armv8l-1:
13:28:02  ./get.sh: line 595: /home/jenkins/workspace/Test_openjdk8_hs_extended.functional_arm_linux/openjdkbinary/j2sdk-image/bin/java: No such file or directory/home/jenkins/workspace/Test_openjdk8_hs_extended.functional_arm_linux/openjdkbinary/j2sdk-image/bin/java: No such file or directory

https://ci.adoptopenjdk.net/job/Test_openjdk8_hs_extended.functional_arm_linux/131/console
https://ci.adoptopenjdk.net/job/Test_openjdk8_hs_special.functional_arm_linux/35/
https://ci.adoptopenjdk.net/job/Test_openjdk8_hs_extended.openjdk_arm_linux_testList_1/25/
https://ci.adoptopenjdk.net/job/Test_openjdk8_hs_extended.openjdk_arm_linux_testList_0/25/

Think we may have already corrected this problem... this machine was mislabelled with hw.arch.aarch32 but is a 64-bit machine, label now been changed to hw.arch.aarch64 so should not receive arm_linux jobs anymore.

Agreed - my fix last week should have resolved this. I'll close this, but if it shows again (unlikely!) we can re-open.


error Failed to launch emulator. Reason: No emulators found as an output of emulator -list-avds.

Description
% npx react-native run-android
warn Package react-native-navigation contains invalid configuration: "dependency.assets" is not allowed,"dependency.hooks" is not allowed. Please verify it's properly linked using "react-native config" command and contact the package maintainers about this.
info Starting JS server...
/bin/sh: /Applications/ADT/sdk/platform-tools/adb: No such file or directory
info Launching emulator...
error Failed to launch emulator. Reason: No emulators found as an output of `emulator -list-avds`.
warn Please launch an emulator manually or connect a device. Otherwise app may fail to launch.
info Installing the app...

> Configure project :react-native-reanimated
Native libs debug enabled: false
Android gradle plugin: 7.0.4
Gradle: 7.5.1
building Reanimated2
WARNING:The specified Android SDK Build Tools version (23.0.1) is ignored, as it is below the minimum supported version (30.0.2) for Android Gradle Plugin 7.0.4.
Android SDK Build Tools 30.0.2 will be used.
To suppress this warning, remove "buildToolsVersion '23.0.1'" from your build.gradle file, as each version of the Android Gradle Plugin now has a default version of the build tools.
WARNING:The specified Android SDK Build Tools version (23.0.1) is ignored, as it is below the minimum supported version (30.0.2) for Android Gradle Plugin 7.0.4.
Android SDK Build Tools 30.0.2 will be used.
To suppress this warning, remove "buildToolsVersion '23.0.1'" from your build.gradle file, as each version of the Android Gradle Plugin now has a default version of the build tools.

> Task :app:processDebugMainManifest FAILED
[com.android.support:animated-vector-drawable:28.0.0] /Users/tenna/.gradle/caches/transforms-3/233cd41ab5d43779d1ddd64b4614d29a/transformed/animated-vector-drawable-28.0.0/AndroidManifest.xml Warning:
        Package name 'android.support.graphics.drawable' used in: com.android.support:animated-vector-drawable:28.0.0, com.android.support:support-vector-drawable:28.0.0.
[androidx.versionedparcelable:versionedparcelable:1.1.1] /Users/tenna/.gradle/caches/transforms-3/dc0f11691095d4bbc8003ad8f5bb847f/transformed/versionedparcelable-1.1.1/AndroidManifest.xml Warning:
        Package name 'androidx.versionedparcelable' used in: androidx.versionedparcelable:versionedparcelable:1.1.1, com.android.support:versionedparcelable:28.0.0.

See http://g.co/androidstudio/manifest-merger for more information about the manifest merger.

Deprecated Gradle features were used in this build, making it incompatible with Gradle 8.0.

You can use '--warning-mode all' to show the individual deprecation warnings and determine if they come from your own scripts or plugins.

See https://docs.gradle.org/7.5.1/userguide/command_line_interface.html#sec:command_line_warnings
102 actionable tasks: 2 executed, 100 up-to-date
/Users/tenna/Desktop/aitem/android/app/src/main/AndroidManifest.xml:24:18-86 Error:
	Attribute application@appComponentFactory value=(androidx.core.app.CoreComponentFactory) from [androidx.core:core:1.7.0] AndroidManifest.xml:24:18-86
	is also present at [com.android.support:support-compat:28.0.0] AndroidManifest.xml:22:18-91 value=(android.support.v4.app.CoreComponentFactory).
	Suggestion: add 'tools:replace="android:appComponentFactory"' to <application> element at AndroidManifest.xml:12:5-54:19 to override.

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':app:processDebugMainManifest'.
> Manifest merger failed with multiple errors, see logs

* 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 6s

error Failed to install the app. Make sure you have the Android development environment set up: https://reactnative.dev/docs/environment-setup.
Error: Command failed: ./gradlew app:installDebug -PreactNativeDevServerPort=8081
/Users/tenna/Desktop/aitem/android/app/src/main/AndroidManifest.xml:24:18-86 Error:
	Attribute application@appComponentFactory value=(androidx.core.app.CoreComponentFactory) from [androidx.core:core:1.7.0] AndroidManifest.xml:24:18-86
	is also present at [com.android.support:support-compat:28.0.0] AndroidManifest.xml:22:18-91 value=(android.support.v4.app.CoreComponentFactory).
	Suggestion: add 'tools:replace="android:appComponentFactory"' to <application> element at AndroidManifest.xml:12:5-54:19 to override.

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':app:processDebugMainManifest'.
> Manifest merger failed with multiple errors, see logs

* 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 6s

    at makeError (/Users/tenna/Desktop/aitem/node_modules/execa/index.js:174:9)
    at /Users/tenna/Desktop/aitem/node_modules/execa/index.js:278:16
    at processTicksAndRejections (node:internal/process/task_queues:96:5)
    at async runOnAllDevices (/Users/tenna/Desktop/aitem/node_modules/@react-native-community/cli-platform-android/build/commands/runAndroid/runOnAllDevices.js:109:5)
    at async Command.handleAction (/Users/tenna/Desktop/aitem/node_modules/@react-native-community/cli/build/index.js:142:9)
info Run CLI with --verbose flag for more details.

I have my android device connected to M1 Mac.
Also have emulator running.
% adb devices
List of devices attached
RFCR71VND0N	device
emulator-5554	device
Version
0.7
Output of npx react-native info
% npx react-native info
warn Package react-native-navigation contains invalid configuration: "dependency.assets" is not allowed,"dependency.hooks" is not allowed. Please verify it's properly linked using "react-native config" command and contact the package maintainers about this.
info Fetching system and libraries information...
System:
OS: macOS 12.5
CPU: (8) arm64 Apple M1
Memory: 117.31 MB / 8.00 GB
Shell: 5.8.1 - /bin/zsh
Binaries:
Node: 17.0.0 - ~/.nvm/versions/node/v17.0.0/bin/node
Yarn: 3.2.3 - ~/.yarn/bin/yarn
npm: 8.0.0 - ~/.nodebrew/current/bin/npm
Watchman: 2<IP_ADDRESS> - /opt/homebrew/bin/watchman
Managers:
CocoaPods: 1.11.3 - /usr/local/bin/pod
SDKs:
iOS SDK:
Platforms: DriverKit 21.4, iOS 16.0, macOS 12.3, tvOS 16.0, watchOS 9.0
Android SDK:
API Levels: 23, 27, 28, 29, 30, 31, 32, 33
Build Tools: 28.0.2, 29.0.2, 30.0.2, 30.0.3, 31.0.0, 33.0.0
System Images: android-31 | Google APIs ARM 64 v8a
Android NDK: Not Found
IDEs:
Android Studio: 2021.2 AI-212.5712.43.2112.8609683
Xcode: 14.0.1/14A400 - /usr/bin/xcodebuild
Languages:
Java: 18.0.2 - /usr/bin/javac
npmPackages:
@react-native-community/cli: Not Found
react: ^18.2.0 => 18.2.0
react-native: ^0.70.1 => 0.70.1
react-native-macos: Not Found
npmGlobalPackages:
react-native: Not Found
Steps to reproduce
don't know
Snack, code example, screenshot, or link to a repository
I can give access to my repo if that'll help.

Hi @ChurikiTenna Thanks for reporting this issue.
The issue is this error in the console:
/bin/sh: /Applications/ADT/sdk/platform-tools/adb: No such file or directory

You probably have an environment variable set wrong. yarn react-native doctor should help you fix this.
Moreover, this issue is actually unrelated to React Native core but to our CLI. You can get better support by opening this issue against this other repository:
https://github.com/react-native-community/cli/issues


Floating Action button without Ripple-effect issue

Floating Action Button with pmd-ripple-effect works fine, but when i try to remove the pmd-ripple-effect from the buttons i face strange issue, the data-title of the rest of buttons becomes showing even if i don't hover over them...
Floating Action example with pmd-ripple-effect :

works fine!
Floating Action example without pmd-ripple-effect :

you can see the issue!
Hopefully i fixed this by adding overflow:hidden to the buttons:
.pmd-floating-action-btn{   overflow: hidden; }
I attached some demos
floating_actions_issue.zip

Hello,
Thank you for bringing this to our notice.

@ExillustX: This issue is resolved.


loading remote context failed

Hi,
I have the following issue when I tried to sign my VC :
 at async Object._createSignData [as createVerifyData] (file:///Users/adrien/Desktop/Cassiope%CC%81/VC/api_sign_vc/node_modules/@digitalbazaar/bbs-2023-cryptosuite/lib/sign.js:84:7) {
 details: {
   code: 'loading remote context failed',
   url: 'https://w3id.org/security/data-integrity/v2',
   cause: Error: Document loader unable to load URL "https://w3id.org/security/data-integrity/v2".
       at documentLoader (file:///Users/adrien/Desktop/Cassiope%CC%81/VC/api_sign_vc/node_modules/@digitalbazaar/vc/lib/documentLoader.js:16:9)
       at /Users/adrien/Desktop/Cassiopé/VC/api_sign_vc/node_modules/jsonld-signatures/lib/documentLoader.js:48:12
       at /Users/adrien/Desktop/Cassiopé/VC/api_sign_vc/node_modules/jsonld-signatures/lib/documentLoader.js:48:12
       at ContextResolver._fetchContext (/Users/adrien/Desktop/Cassiopé/VC/api_sign_vc/node_modules/jsonld/lib/ContextResolver.js:166:25)
       at ContextResolver._resolveRemoteContext (/Users/adrien/Desktop/Cassiopé/VC/api_sign_vc/node_modules/jsonld/lib/ContextResolver.js:117:45)
       at ContextResolver.resolve (/Users/adrien/Desktop/Cassiopé/VC/api_sign_vc/node_modules/jsonld/lib/ContextResolver.js:50:33)
       at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
       at async api.process (/Users/adrien/Desktop/Cassiopé/VC/api_sign_vc/node_modules/jsonld/lib/context.js:87:20)
       at async api.expand (/Users/adrien/Desktop/Cassiopé/VC/api_sign_vc/node_modules/jsonld/lib/expand.js:214:17)
       at async jsonld.expand (/Users/adrien/Desktop/Cassiopé/VC/api_sign_vc/node_modules/jsonld/lib/jsonld.js:322:18)
 }

I don't know how to do to resolve the pb because it's an intern pb form the github project. Any idea ?
Thanks a lot.

Hey @abonnivard, thanks for asking!
One option for resolving this is with a custom documentLoader:
https://github.com/digitalbazaar/vc?tab=readme-ov-file#custom-documentloader
Using that, you can download the context file (essentially caching it), and load it via your custom document loader--thereby avoiding the network dependency (and latency). You'll of course want to separately track future revisions to that context file to be sure you have the latest, but using the custom documentLoader approach means you won't be doing that on every request.
Hope that helps!
🎩

Closing as the answer has been given here and it seems this has been resolved.


Pillow Resizing Option for FFCV Dataset Creation

This PR adds the ability to use Pillow to resize images when creating a RGBField, which allows resizing images using antialiasing. It also adds an option to change the OpenCV interpolation, and a min_resolution setting.
I also changed the smart_threshold argstring to reflect what the code does. Perhaps the code the original argstring is referring too didn't get merged from a branch? Here is the current smart_threshold code:
if self.smart_threshold is not None:
    if image.nbytes > self.smart_threshold:
        write_mode = 'jpg'

@warner-benjamin is this ready to be merged?

It's ready to go. I realized forgot to push my doc changes and fixed it yesterday.


MM-33209 Downgrade serialize-error

The Selenium tests that QA use run on Firefox 54, and the new version of serialize-error fails to load on there because it uses slightly newer syntax (https://caniuse.com/mdn-javascript_statements_try_catch_optional_catch_binding, supported in Firefox 58+). Selenium will be upgraded or removed at some point, but this will fix it in the meantime.
Ticket Link
https://mattermost.atlassian.net/browse/MM-33209
Related Pull Requests
https://github.com/mattermost/mattermost-webapp/pull/7586

Cherry pick is scheduled.

Error trying doing the automated Cherry picking. Please do this manually
+++ Updating remotes...
Fetching upstream
Failed to add the RSA host key for IP address '<IP_ADDRESS>' to the list of known hosts (/app/.ssh/known_hosts).
From github.com:mattermost/mattermost-redux
 * [new branch]        MM-20441_call-path     -> upstream/MM-20441_call-path
   4c6af352..757cc488  MM-32655               -> upstream/MM-32655
 * [new branch]        MM-32691               -> upstream/MM-32691
   f0a679c2..43723c72  SectionsToSubSections  -> upstream/SectionsToSubSections
   0bac39f5..3599510c  cloud                  -> upstream/cloud
   de420382..e82ee2e5  feature/cloud-apps     -> upstream/feature/cloud-apps
 * [new branch]        fix_feature/cloud-apps -> upstream/fix_feature/cloud-apps
   041dc987..e0bc76cd  master                 -> upstream/master
Fetching origin
Failed to add the RSA host key for IP address '<IP_ADDRESS>' to the list of known hosts (/app/.ssh/known_hosts).
+++ Updating remotes done...
!!! 'upstream/release-5.33' not found. The second argument should be something like upstream/release-0.21.
    (In particular, it needs to be a valid, existing remote branch that I can 'git checkout'.)

+++ Returning you to the master branch and cleaning up.

/cherry-pick cloud

Cherry pick is scheduled.


Add a dependabot configuration for this project

I merged even though the build is failing, sicne the failure is unrelated to this change. I'll follow up with a PR that tries to fix the failure

Success! (Once #62 merged)


My two cents

I've been using conda since it was initially released and always stuck with it (even during the slow solver years). I have manually constructed hundreds of conda environments with pip packages over the years and my general process was always to identify which packages are not available as conda packages then examine their dependency tree to identify any of their dependencies that are conda packages. I then install everything in the dependency tree I can from conda. After that I manually pip install all of the pypi packages with the --no-deps flag. This results in a conda environment where everything that can possibly be installed by conda is present and then only a small number of top level pip packages are installed. This generally allowed conda update --all to work followed by a manual pip install -U --no-deps for the pypi packages. This, in general, prevents clobbering things in the environment.
I write this because I suppose my first try to make a tool such as conda-pypi would be to mimic this manual process. But I imagine that this is difficult to pull off with the name mismatchs in conda/pypi and probably many other corner cases.
Feel free to close this, I just felt compelled to write it down.

I didn't notice the docs until after I wrote this and read through all the issues. I see now that this tool does essentially what I just described (and more). Congrats! Looking forward to trying it out.


Issue 197/refactoring-routes

This PR includes the refactoring of React Routes for PASS and the optimization of current routing logic.

I couldn't find much about it so want to ask here, is the replace attribute used to replace the navigation history?

Though I don't think a typical user would use the back button a lot from their browser for PASS, it's simply there to reset the local history stack if a user logs out or refresh.

The only other thing that I'd like to consider is whether or not you felt it appropriate if we move the rest of the routes into the routes folder with this PR?
Forms, Inbox, and UserSection

Think we can do that in this PR.

All looks good to me now.


libc: Move atexit/on_exit to user space

Summary

Remove dedicate atexit implemention since atexit and on_exit share the same infrastructure
Add a new syscall SYS_nxtask_onexit for drop to user space and exexute cleanup functions

Impact
Should be NONE
Testing
Tested on maix-bit, both PROTECTED & FLAT build
See #1263

@no1wudi https://github.com/no1wudi requested your review on: #3951
https://github.com/apache/incubator-nuttx/pull/3951 libc: Move
atexit/on_exit to user space.

I will try to make some time to look at the PR this weekend.

Looks like you will have to update some defconfig files due to the on_exit/atexit configuration changes.

LGTM. There are some CI issues that need to be fixed before merging. defconfig files need to be updated, I think.

All CI build passed in my repository, maybe we should re-trigger workflow.

@patacongo @xiaoxiang781216 I've just add a new TLS like structure task_local_s to storage all user space data in task_group_s, please take a look, maybe we could move pthread_cleanup & tls destructor and more data into it.

@patacongo @xiaoxiang781216 I've just add a new TLS like structure task_local_s to storage all user space data in task_group_s, please take a look, maybe we could move pthread_cleanup & tls destructor and more data into it.
And then save many syscall like tls_get/set xxx since them moved to user space.

It's better to move the pointer from task_group_s to tls_info_s, so all threads in userspace can access this key struct directly without any syscall.

It's better to move the pointer from task_group_s to tls_info_s, so all threads in userspace can access this key struct directly without any syscall.

Done, the task_info_s has been moved into task_group_s and it will be allocated by kumm_malloc at runtime.

It's better to move the pointer from task_group_s to tls_info_s, so all threads in userspace can access this key struct directly without any syscall.

@xiaoxiang781216  Yes, tls_info_s would save a copy of  this pointer, and all user space can access it by tls_info_s.
Put the task_info_s here is for re-using reference counting of task_group_s.

@yamt In old implementation #3858 introduce a issue #3868, and it had been reverted , could you have a test with this PR ?

@patacongo @xiaoxiang781216 @yamt
I'll close this PR and re-organize the patch in clear way since it's hard to review.


Delete Message - Listener in different platforms

Hi ,
This would be regarding delete message.
If I delete message from qmunicate-web is there any way I can be informed in qmunicate-android that those particular messages are deleted , so that we can delete from ormlite in android. Our requirement is to keep android and web in sync , hence the question.

you can realize this functionality via normal messages but without parameter saveToHostory=1. In your case user which deletes message should send message with additional parameters, for example (Android code):
            QBChatMessage messageAboutDelete = new QBChatMessage();
            messageAboutDelete.setProperty("notification_type", NOTIFICATION_DELETE_MESSAGE);
            messageAboutDelete.setProperty("delete_message_id", messageId);
            messageAboutDelete.setProperty("delete_message_dialog_id", dialogId);

On other side when this message will received just check "notification_type" and remove message from ormlite.

Ok thanks !


Problems downloading the checkpoints

Hi, I downloaded all the checkpoints, both with simple download, both with the Yandex.Disk app, but all archives seems to be corrupted.
I tried to open them with WinRAR, 7zip and PeaZip, same result.
Can you please help me?
Thank you.

You should not open them.


Missing Linux support (Delphi 11 Alexandria - Enterprise)

Skia4Delphi Ver. <IP_ADDRESS>
There is no define about TGrCanvasGL (or something like TSkCanvasRasterLinux) in line 1263 of Skia.FMX.Graphics when define Linux64 as target OS:
"  {$IF defined(MSWINDOWS)}
if GlobalUseSkiaRasterWhenAvailable then
Result := TSkCanvasRasterWindows
else
Result := TGrCanvasGL;
{$ELSEIF DEFINED(MACOS)}
if GlobalUseMetal then
Exit(TGrCanvasMetal)
else
{$IFDEF IOS}
Result := TGrCanvasGL;
{$ELSE}
Result := TSkCanvasRasterMacOS;
{$ENDIF}
{$ELSE}
Result := TGrCanvasGL;    <----------------------- THIS if LINUX64 DEFINED
{$ENDIF}
"
TGrCanvasGL is defined in Skia.FMX.Canvas.GL, but for Linux nothing is declared.
Demo is not compiled for Linux64 (FMXLinux is installed), I miss something ?

Thank you so much for reporting! The problem has been fixed in new release v3.0.1. But it is worth mentioning that the binary was pre-built for systems based on Debian, such as Ubuntu. If you need it for another distro it will be necessary to recompile the Skia binary. For more informations: https://github.com/skia4delphi/skia4delphi/blob/main/Documents/BUILD.md


Updated free-courses-en.md

Added a free course on Kotlin for beginners from Udemy
What does this PR do?
Add resource(s)
For resources
Description
Updated the Kotlin section with a free Udemy course on Kotlin for beginners
Why is this valuable (or not)?
This course is suitable for beginners and also covers OOP
How do we know it's really free?
The webpage labels it as a free course
For book lists, is it a book? For course lists, is it a course? etc.
Course
Checklist:

[x] Read our contributing guidelines.
[x] Search for duplicates.
[x] Include author(s) and platform where appropriate.
[x] Put lists in alphabetical order, correct spacing.
[x] Add needed indications (PDF, access notes, under construction).
[x] Used an informative name for this pull request.

Follow-up

Check the status of GitHub Actions and resolve any reported warnings!

@eshellman This PR was merged successfully however Hacktoberfest has now excluded this repository from participation as it did  not follow the rules of the event, Can you clarify?


linux: enable TCP_CONG_ADVANCED on aarch64-linux

Motivation for this change
TCP_CONG_ADVANCED is enabled by default on x86_64 in the upstream.
Although it is not the case for aarch64, many distributions, such as
Debian, Fedora and Gentoo, choose to enable it in their
distribution kernel.
With this patch, aarch64 users can choose many other TCP congestion
algorithms, which may improve their network performance.
I have tested the 5.10.76 kernel and it builds and runs well.
I also run several iperf tests. The fastest speed I get using the default value (cubic) of net.ipv4.tcp_congestion_control is 38Mbps, while it is 268Mbps using bbr, which needs this patch to be applied.
This PR fixes https://github.com/NixOS/nixpkgs/issues/141051.
Things done

Built on platform(s)

[ ] x86_64-linux
[x] 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 wip"
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.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

[x] Fits CONTRIBUTING.md.

Hi, could you please review my pr? Thanks in advance. @jonringer @lovesegfault @SuperSandro2000 @samueldr @arcnmx @TredwellGit @Ma27 @Mic92

Given we're building generic kernels for all platforms, and kernels with as-close-as-possible feature sets, is there a reason this would be enabled only on aarch64? Shouldn't this be enabled on all platforms? (e.g. Without the conditional.)
I know that it's enabled in the x86_64 defconfig, but having it re-define it as =y here is fine. In addition that would add support for it for all platforms intrinsically.
So yeah, since it's part of the x86_64 defconfig already, and other distros enable it, I agree with the general principle here that it would be fine to enable for all platforms.

I have removed the condition checking.

I have changed to target the staging branch.

@GrahamcOfBorg build linuxPackages.kernel


Clock speed?

Such a cool project!
How many MHz is the RISC-V core?
Thanks!

Timing closed around 50 to 60Mhz in Skywater PDK 130nm targeted for Efabless Shuttle project

Cool, thanks!


HTML

HTML
User Story
As a bootcamp student,  I want prework notes to be on a webpage
in order to easily find and read information
Acceptance Criteria
GIVEN a Prework Study Guide website
WHEN I visit the website in my browser
THEN I see four boxes titled HTML, CSS, Git, and Javascript and associated notes

HTML content added per Acceptance Criteria


Rails 5 compatibility

Have you checked it against Rails 5? After running the following:
Feed.all.calculate_all(:visits_count)

I get:
fatal: exception reentered
from /home/vasilakisfil/.rvm/gems/ruby-2.3.1/gems/pry-0.10.4/lib/pry/pry_instance.rb:355:in `eval'

Feed is a relatively simple model.

@vasilakisfil Sorry, I missed the issue.
gem should be working with rails5, thanks to @StefSchenkelaars
But your exception message seem quite weird.
can you please check again with 0.2.0 version of the gem?


API Resources

When building an API, you may need a transformation layer that sits between your models and the json responses that are actually returned to your application's users.  Has any feature known in adonisjs that can easily work with this method? One sample here or in laravel5.5

Closing since no response from the issue reporter and not actionable as well


update report preview message to use workspace name, not admin name

Details

use workspace name for subscript avatar, tooltip
show {workspace name} owes $xxx message for all money requests in workspace chat

Fixed Issues
$ https://github.com/Expensify/App/issues/18858
PROPOSAL: N/A
Tests

Login account A with policyExpenseChat beta
Invite B (with policyExpenseChat beta) to any workspace
Login account B on another device
On invited workspace chat, request money
Verify that in IOU message, shows workspace avatar for subscript, workspace name for tooltip, report preview message says {workspace name} owes xxx
Also verify 5. in "workspace chat with B" on account A

NOTE: A is admin, B is invited member (not admin)

[ ] Verify that no errors appear in the JS console

Offline tests
Same as Tests
QA Steps
Same as Tests

[ ] Verify that no errors appear in the JS console

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android / native
[x] Android / Chrome
[x] iOS / native
[x] iOS / Safari
[x] MacOS / Chrome / Safari
[x] MacOS / Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If a new component is created I verified that:

[x] A similar component doesn't exist in the codebase
[x] All props are defined accurately and each prop has a /** comment above it */
[x] The file is named correctly
[x] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[x] The only data being stored in the state is data necessary for rendering and nothing else
[x] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[x] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[x] All JSX used for rendering exists in the render method
[x] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[x] I have checked off every checkbox in the PR author checklist, including those that don't apply to this PR.

Screenshots/Videos

Web
https://github.com/Expensify/App/assets/97473779/f0136c14-d290-4b94-9ad1-3fc486484206

Mobile Web - Chrome

Mobile Web - Safari

Desktop
https://github.com/Expensify/App/assets/97473779/f0136c14-d290-4b94-9ad1-3fc486484206

iOS

Android

Is anyone against me putting the CP Staging label on this, given that it's pretty high value polish for EC3 to distinguish the workspace case from consumer? Let's give it the best chance of making it.

Ready for review @mananjadhav

Tushat, I had the same issue on staging now when I created a Group and sent message my avatar was missing so maybe its not related

Hey @rushatgabhane,
Good spot. I think we've seen that elsewhere so we can ignore that for this PR review 👍

🚀 Cherry-picked to staging by https://github.com/Julesssss in version: 1.3.14-8 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

@Expensify/applauseleads please QA this PR and check it off on the deploy checklist if it passes.

🚀 Deployed to production by https://github.com/yuwenmemon in version: 1.3.14-14 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Cherry-picked to staging by https://github.com/AndrewGable in version: 1.3.28-2 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

@Expensify/applauseleads please QA this PR and check it off on the deploy checklist if it passes.

🚀 Deployed to production by https://github.com/AndrewGable in version: 1.3.28-5 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


node unhealth

{
"code": "ServiceUnavailable",
"message": "connection failed"
}

hi can you help me

Hi, For faster reply to your issues please to join discord to request help.  - https://discord.com/channels/856325968096133191/862674505193488415
The error seems like your catapult server is down.  What does docker ps -a shows?  Post this output to discord.

@Wayonb i slove it ,thanks


Add tests for var_value for univariate distributions

Summary
This modifies the distribution test suite so that var_value<Matrix> is also tested.
Tests
All the standard distribution tests but using var_value<Matrix>
Side Effects
Nope
Release notes
Tests distributions are var_value<Matrix> compatible
Checklist

[x] Math issue #1805

[x] Copyright holder: Steve Bronder
The copyright holder is typically you or your assignee, such as a university or company. By submitting this pull request, the copyright holder is agreeing to the license the submitted work under the following licenses:
- Code: BSD 3-clause (https://opensource.org/licenses/BSD-3-Clause)
- Documentation: CC-BY 4.0 (https://creativecommons.org/licenses/by/4.0/)

[x] the basic tests are passing

unit tests pass (to run, use: ./runTests.py test/unit)
header checks pass, (make test-headers)
dependencies checks pass, (make test-math-dependencies)
docs build, (make doxygen)
code passes the built in C++ standards checks (make cpplint)

[x] the code is written in idiomatic C++ and changes are documented in the doxygen

[x] the new changes are tested

Greencheck woooo. We got time to polish this a bit more since we aren't trying to merge varmats this release now.

Greencheck woooo. We got time to polish this a bit more since we aren't trying to merge varmats this release now.

Alright I think with the division thing all that's left to do is sort out the integer value of and we are good. Personally idt it hurts and from the math library it is possible to have a var there

Alright I think with the division thing all that's left to do is sort out the integer value of and we are good. Personally idt it hurts and from the math library it is possible to have a var there

I cleaned out the test changes that didn't need to change and removed the value_ofs (gradients aren't going to work even if we do value_of if someone passes in a var). This pull is all good to me.

All good to me as well! though idt I can approve because I'm the author


Use nanoseconds as time base (#1)

Fix library to use nanosecond tick, for better platform-independence

Modify examples to use ns time base

Thanks @XijiaoLi !

Adding a note here that this is just upstreaming the commit introduced by this PR: https://github.com/ssm-lang/ssm-runtime/pull/1


List slack apps for developers

I created a list of 50 slack apps for developers, happy to rewrite them in the same awesome format as you guys have if you think this would make sense?
https://github.com/axolo-co/slack-apps-for-developers
Let me know and I can easily create a PR,
Cheers

Looks like a useful list. It would be welcome here.
https://github.com/sindresorhus/awesome/blob/main/pull_request_template.md


7 improve readme.md

Updated README.md file to better follow the README standard
Added MIT license

LGTM


fix: promise wrapper race/all

Overview

Updating the promise wrapper race and all methods to accept any object that implements the Iterable interface.
Related Issue(s)

https://issues.newrelic.com/browse/NEWRELIC-8148
Testing

Added unit tests that check race and all methods work with standard arrays, a customer iterable and a native iterable Set.

Codecov Report

Merging #477 (6f6035f) into main (6575325) will increase coverage by 3.46%.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##            main     #477      +/-   ##
=========================================
+ Coverage   6.93%   10.40%   +3.46%
=========================================
  Files        121      121
  Lines       3980     3980
  Branches     987      987
=========================================
+ Hits         276      414     +138
+ Misses      2924     2812     -112
+ Partials     780      754      -26

Impacted Files
Coverage Δ

src/common/wrap/wrap-promise.js
84.50% <100.00%> (+84.50%)
:arrow_up:

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


[blocked on removing trusty] Upgrade to mypy 0.700

This comprises three commits, which might be squashed (or simplified given the provision numbers), upgrading to 0.660, 0.670 and then 0.700. I separated them since an initial attempt at 0.700 failed, leaving:

0.660 requires no changes
0.670 was crashing mypy due to an accented variable name in api_test_helpers.py
0.700 would also have crashed, but also needed some minor type updates in models.py

This fixes #12058.
I can't say at this point if there is a significant performance improvement, but I got a core dump from the mypy crash!
I was going to report the bug in 0.670+, but that file still currently has python 2.x function type hints - which could have caused the issue based on some initial trials. As it is, just changing the variable name is fine.

This just failed circleci due to a tooling dependency on 3.5+ for mypy.

Just pushed the rebased last commit after the first two were cherry-picked by Tim.
This may wait on removal of trusty support.

The typed_ast crash on Python 3.4/3.5 is fixed by typed_ast 1.3.5.

Updating typed_ast to 1.3.5 does/did appear to resolve this :+1:
I have an updated commit to squash into the existing commit.
However, on killing the daemon, removing the mypy cache (not used now?) and re-installing typed_ast 1.3.1, it also doesn't crash now! I did rebase and re-provision, so perhaps something else changed in that time from the Zulip side?

I would be surprised if we changed anything on the Zulip side relevant to this.

Trusty is now removed so we can unblock this.

Also by now you could try mypy 0.711.

@neiljp do you have time to update this PR?

Closing in favor of #12850, thanks @neiljp!

The performance improvement here is huge!  With 0.711, mypy is now only barely our slowest linter even if the daemon wasn't running (a ~4x improvement), and if the daemon is, it's like 2x faster.
Huge thanks to the mypy team for this performance work!


Deadlock when failing to deserialize a CrossDomainInvokeRequest

This is an interesting issue that I encountered when loading a custom performance metric that depended on a version of TraceEvent that was different that the one that the Logger process depended upon. In my case, when deserializing a CrossDomainInvokeRequest, an exception is thrown by the runtime for the assembly version mismatch which is propegated here:
while ((parentMessage = await ReadNextRequestAsync()) != null)
{
    Task throwaway = HandleInvokeRequest(parentMessage);
}

Since the faulted task is awaited, it continues to unwind to here:
var throwaway = ListenToParentDomainAsync();

where the task is never awaited, so the exception stops propagating. At this point, the child procdomain is waiting to be unloaded, but it will never be unloaded because ListenToParentDomainAsync unwound past the call to this.Unload() and is no longer listening to the parent domain. Meanwhile, the parent domain is waiting for a response from the child domain, which will never come (since it's not listening).
Simple repro:

Insert an exception here so that this method begins to unwind.
Run a test suite as normal. The test runner gets stuck waiting for the trace to begin:

xunit.performance Console Runner (64-bit .NET 4.0.30319.42000)
Copyright (C) 2015 Microsoft Corporation.

Creating output directory: .
Discovering tests for C:\Users\segilles\Documents\Visual Studio 2015\Projects\Microsoft.Performance.GC\Microsoft.Performance.GC.Tests\bin\Debug\Microsoft.Performance.GC.Tests.dll.
Discovered a total of 1 tests.
Starting ETW tracing. Logging to .\Crash.etl

It will never progress past this point because it's waiting on the child procdomain which is partially faulted.
I looked into fixing this but there are a couple factors that make this tricky.

Allowing the child procdomain to die via async void is not good because it leaks a ETW session in the Logger process, which is unacceptable.
Allowing the child procdomain to immediately unload on a fault causes the parent procdomain to wait forever on a response, still deadlocking.

Perhaps it would be possible to introduce a sort of protocol where the child procdomain could inform its parent (or vice versa) that it has faulted and will no longer respond to messages? Unfortunately, the scenario that would cause someone to run into this issue is probably quite common - the version mismatch in TraceEvent.

Closing prior to archive.

Closing prior to archive.


Replace synchronized under core/client with ReentrantLock

Motivation:
synchronized should be removed because they should be avoided when using virtual threads
resolves part of https://github.com/line/armeria/issues/4510
Modifications:

Replaced all synchronized  under core/client package with ReentrantLock

@minwoox @ikhoon I added the @GuardedBy as suggested, but it seems to throw an error even where lock is required 😢  I'll look into it again later to see what the problem may be 🙏

but it seems to throw an error even where lock is required 😢 I'll look into it again later to see what the problem may be

I don't know how it works but some of the warnings are disappeared if we inline the methods.
And Let's use @SuppressWarnings("GuardedBy") for the false positives.

Thanks a lot, @j-min5u! 😄


Feature/quality

Add quality improvements

Coverage remained the same at 98.827% when pulling cccdaea06d4e7e83695f235a376626efaeb64f5a on feature/quality into ad404abe17df3ad692fba3dc502c4f6aa1dc79c1 on master.


Acrylic theme

Hi Guys,
Awesome app firstly,  luv using this app.
A nice added touch will be to include and Acrylic theme in your theme settings where it displays the current pc desktop background.  With Bing wallpaper rotating images on the desktop that will create nice added effect.

Hi @Pinox thanks for using Ambie! Unfortunately, Microsoft updated their guidelines to say that acrylic is only recommended for short-lived, transient surfaces like flyouts. This means that acrylic is no longer recommended for the background of an app. Thus, I will keep the current design as is for now unless more and more people want to try seeing acrylic. Also note that on Windows 11, the app's background is using Mica which is the new recommended design.
Sorry if I'm being a downer, but just wanted to be honest and set expectations.

Thanks for reply @dpaulino. It's not a downer, it's just your app is a great showcase for how a beautiful app should look like ;))
I thought Mica & Background Acrylic will be an good addition as it's now available in the Windows App SDK 1.1.0 but I forgot this is a UWP app and not a WinUI 3 app. Just out of interest are you planning to upgrade it to WinUI 3 in the future or does the benefits of UWP (.net native) still outweight WinUI3 ?

Right yeah, Ambie is a c# xaml uwp app. I am not planning to update Ambie to WinUI 3 because Ambie must run on Xbox and uwp's performance/memory usage is still better than WinUI 3 (at least last I checked).


updated README and images

this commit represents adding images and modifying README.md.

good job, Kev!

README edit  approved


shutdown with count down latch

Notify - wait mechanism is fragile,  Main.class is global, anyone could wait on or notify it, which makes shutdown fragile too.
Shutdown with count down latch can avoid evil notify on Main.class(which causes abnormal shutdown) or evil wait on Main.class(which causes non-stop).

@johnraychina I think it is better to put running.countDown() outside the for loop, how do you think?

@qinliujie Sometimes container.stop() may throw exceptions,
it denpends on how you want shutdown the application:

I prefer to solve the exceptions, and shutdown again;
If running.countDown() is outside of the loop,  main thread is stopped while container is still running, then the application is in a very strange status.

why do Main.class start so many container? --" for( Container container: containers ) { .....}
BUT the countDownLatch just count one

@bigcai
Because Multiple containers is added AFTER my fork, and this branch is is conflict with master.

Since simple object lock will be ok to avoid evil wait on Main class, I 'll just create another pull request.

@johnraychina
Current logic do have problem.

Main.class.wait() may never receive a notify.
But because of running flag, i think the main process will not shutdown unexpectedly.

Looking forward to your new PR

I have fixed this, see Fixed #520 change Main.class wait/notify to lock/condition


IGNITE-19958 Remove unused bouncycastle version in pom

Thank you for submitting the pull request to the Apache Ignite.
In order to streamline the review of the contribution
we ask you to ensure the following steps have been taken:
The Contribution Checklist

[ ] There is a single JIRA ticket related to the pull request.
[ ] The web-link to the pull request is attached to the JIRA ticket.
[ ] The JIRA ticket has the Patch Available state.
[ ] The pull request body describes changes that have been made.
The description explains WHAT and WHY was made instead of HOW.
[ ] The pull request title is treated as the final commit message.
The following pattern must be used: IGNITE-XXXX Change summary where XXXX - number of JIRA issue.
[ ] A reviewer has been mentioned through the JIRA comments
(see the Maintainers list)
[ ] The pull request has been checked by the Teamcity Bot and
the green visa attached to the JIRA ticket (see TC.Bot: Check PR)

Notes

How to Contribute
Coding abbreviation rules
Coding Guidelines
Apache Ignite Teamcity Bot

If you need any help, please email<EMAIL_ADDRESS>or ask anу advice on http://asf.slack.com #ignite channel.

@petrov-mg Could you please take a look?

We actually have dozens of unused Maven dependency versions after some Ignite modules were moved to a separate repository Ignite Extensions. It might be better to clear them all in one ticket rather than create one for each unused version.
Although I have no objection to the current patch.


Add a wiki: Zero Ranger Wiki

An anonymous user has submitted a new wiki via our submission form.
Link to origin wiki: https://zeroranger.fandom.com
Link to destination (independent) wiki: https://zeroranger.miraheze.org

Added in v3.7.0 (now available for Chrome, release pending for Firefox).


add Harmonization

Adding a custom Harmonization class. The model is explained in  this paper and is available under this repo. It is a type of model that gets deep neural networks to rely on similar aspects of images as humans do while doing object recognition.

@candemircan Thanks for this PR! I think this will definitely be a useful addition to the current models in thingsvision. I know Drew's work quite well. Could you add tests for at least two of the models, and update the README.md similarly to how you've already changed the docs? I think you need to rebase before you do any of the above.

hey Lukas :) thanks for the quick response. Just made the changes as you requested. Hope it works out fine!
Best,
Can

@candemircan, thanks a lot for the changes! Could you rebase one more time and submit the PR again? I had to accept a different PR before I could accept yours.

Also, could you do me a favor and remove the folder build/lib after you've rebased? This folder was accidentally added during the previous PR.

@candemircan I see that the unittests are not passing. Did you rebase correctly?

@LukasMut I believe I did, but a comma was missing. Now I fixed it. Let's see if it passes now

@candemircan Unfortunately, the unittests are still failing. I think there's a module missing for one of the harmonization models. Please see yourself when and why the tests are failing.

@candemircan I see that you've added the module to the requirements.txt file, but I am wondering whether it has to be added to the setup.py and environment.yml files too? Unfortunately, the tests are still failing for some reason. Could you test locally what's going on?

@candemircan Will you submit a new PR? You could have just pushed another commit to this PR. :)

Hi Lukas,
I’ll push another commit to this PR later :) just closed this for now as it kept running new tests despite me not making any changes yet.
22 Feb 2023 at 16:20 by @.***:

@candemircan https://github.com/candemircan>  Will you submit a new PR? You could have just pushed another commit to this PR. :)
—
Reply to this email directly, > view it on GitHub https://github.com/ViCCo-Group/thingsvision/pull/130#issuecomment-1440242913> , or > unsubscribe https://github.com/notifications/unsubscribe-auth/APEIP2QNT6USBKU7PBORRHDWYYVCHANCNFSM6AAAAAAVATEIGE> .
You are receiving this because you were mentioned.> Message ID: > <ViCCo-Group/thingsvision/pull/130/c1440242913> @> github> .> com>

@candemircan Sounds good to me! I can stop automatic testing for this PR until you push another commit.

Hi again

@Alxmrphi, do you have any idea what's going on with the mac-os unittests?

@Alxmrphi, do you have any idea what's going on with the mac-os unittests?

My gut feeling tells me that it has something to do with the
TensorFlow version, but I am not 100% sure. One could also try to pip install git+https://github.com/leondgarse/keras_cv_attention_models rather than keras_cv_attention_models.

I'm scratching my head at this one, too. How is it passing the unit tests but not the code coverage? I downloaded and ran the unit tests locally and they fail on the harmonization model tests. Quick fixes to install libraries, more failures in different places. The first hurdle it fails at is not having access to keras_applications.

@candemircan do you use keras-cv-attention-models or keras_cv_attention_models for installing the module via pip? I think it needs to be the former (although the Harmonization repo uses the latter).

Hi Lukas,
I was using the latter, but you are right that the package is named as the former in pypi. Should I change it and make another commit?

@candemircan do you use keras-cv-attention-models or keras_cv_attention_models for installing the module via pip? I think it needs to be the former (although the Harmonization repo uses the latter).

Hi Lukas,
I was using the latter, but you are right that the package is named as the former in pypi. Should I change it and make another commit?

@candemircan Let me see whether I can figure this out. I feel like that error is not on our end.

I'm scratching my head at this one, too. How is it passing the unit tests but not the code coverage? I downloaded and ran the unit tests locally and they fail on the harmonization model tests. Quick fixes to install libraries, more failures in different places. The first hurdle it fails at is not having access to keras_applications.

@Alxmrphi, I see that this error is happening within the harmonization module. See the issue that references this PR.


set limit for event name to be 512

Fixing https://github.com/Microsoft/ApplicationInsights-dotnet/issues/61

:shipit:


netpbm: 11.7.1 -> 11.8.1

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for netpbm is: Toolkit for manipulation of graphic images
meta.homepage for netpbm is: https://netpbm.sourceforge.net/
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 11.8.0 with grep in /nix/store/992s0m4jlfcvbiibzac0i4vdamj9fwh9-netpbm-11.8.0-bin
found 11.8.0 in filename of file in /nix/store/992s0m4jlfcvbiibzac0i4vdamj9fwh9-netpbm-11.8.0-bin

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

96 total rebuild path(s)

96 package rebuild(s)

First fifty rebuilds by attrpath

asciidoc-full
asciidoc-full-with-plugins
auto-multiple-choice
bicgl
bicpl
cantor
clevis
conglomerate
cqrlog
denemo
diffoscope
diffoscopeMinimal
disorderfs
dosbox-x
ezminc
fbcat
foomatic-db-ppds
foomatic-db-ppds-withNonfreeDb
fped
frescobaldi
geeqie
img2pdf
kakounePlugins.rep
kstars
labplot
latex2html
libsForQt5.cantor
lilypond
lilypond-unstable
lilypond-unstable-with-fonts
lilypond-with-fonts
luksmeta
manga-cli
mpvScripts.convert
mup
netpbm
nhentai
ocrmypdf
oobicpl
paperless-ngx
pcb
pdfarranger
pfstools
plasma5Packages.cantor
pnglatex
polymake
protontricks
pynac
python311Packages.abjad

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/992s0m4jlfcvbiibzac0i4vdamj9fwh9-netpbm-11.8.0-bin \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

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

Or:
nix build github:r-ryantm/nixpkgs/ef361b3c76ef314dfcd5daf9f4d15df7ce2c4a7a#netpbm

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/992s0m4jlfcvbiibzac0i4vdamj9fwh9-netpbm-11.8.0-bin
ls -la /nix/store/992s0m4jlfcvbiibzac0i4vdamj9fwh9-netpbm-11.8.0-bin/bin

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

3 packages marked as broken and skipped:

ezminc
spring
springLobby

3 packages failed to build:

pynac
sage
sageWithDoc

77 packages built:

asciidoc-full
asciidoc-full-with-plugins
asciidoc-full-with-plugins.dist
asciidoc-full.dist
auto-multiple-choice
bicgl
bicpl
cantor
clevis
clevis.man
conglomerate
cqrlog
denemo
diffoscope
diffoscope.dist
diffoscope.man
diffoscopeMinimal
diffoscopeMinimal.dist
diffoscopeMinimal.man
disorderfs
dosbox-x
fbcat
foomatic-db-ppds
foomatic-db-ppds-withNonfreeDb
fped
frescobaldi
frescobaldi.dist
geeqie
img2pdf (python312Packages.img2pdf)
img2pdf.dist (python312Packages.img2pdf.dist)
kakounePlugins.rep
kstars
labplot
latex2html
lilypond
lilypond-unstable
lilypond-unstable-with-fonts
lilypond-with-fonts
luksmeta
manga-cli
mpvScripts.convert
mup
netpbm
netpbm.bin
netpbm.dev
nhentai
nhentai.dist
ocrmypdf (python312Packages.ocrmypdf)
ocrmypdf.dist (python312Packages.ocrmypdf.dist)
oobicpl
paperless-ngx
pcb
pdfarranger
pdfarranger.dist
pnglatex (python312Packages.pnglatex)
pnglatex.dist (python312Packages.pnglatex.dist)
polymake
protontricks
protontricks.dist
python311Packages.abjad
python311Packages.abjad.dist
python311Packages.img2pdf
python311Packages.img2pdf.dist
python311Packages.ocrmypdf
python311Packages.ocrmypdf.dist
python311Packages.pnglatex
python311Packages.pnglatex.dist
singular
sng
sng.man
solfege
steamtinkerlaunch
tang
tang.man
tuxpaint
xplanet
yad

Maintainer pings

[!TIP]
As a maintainer, if your package is located under pkgs/by-name/*, you can comment @NixOS/nixpkgs-merge-bot merge to automatically merge this update using the nixpkgs-merge-bot.

Add a :+1: reaction to pull requests you find important.

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for netpbm is: Toolkit for manipulation of graphic images
meta.homepage for netpbm is: https://netpbm.sourceforge.net/
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 11.8.1 with grep in /nix/store/80xacjn7msql3lf4ylk8q6lsmmrav6k6-netpbm-11.8.1-bin
found 11.8.1 in filename of file in /nix/store/80xacjn7msql3lf4ylk8q6lsmmrav6k6-netpbm-11.8.1-bin

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

96 total rebuild path(s)

96 package rebuild(s)

First fifty rebuilds by attrpath

asciidoc-full
asciidoc-full-with-plugins
auto-multiple-choice
bicgl
bicpl
cantor
clevis
conglomerate
cqrlog
denemo
diffoscope
diffoscopeMinimal
disorderfs
dosbox-x
ezminc
fbcat
foomatic-db-ppds
foomatic-db-ppds-withNonfreeDb
fped
frescobaldi
geeqie
img2pdf
kakounePlugins.rep
kstars
labplot
latex2html
libsForQt5.cantor
lilypond
lilypond-unstable
lilypond-unstable-with-fonts
lilypond-with-fonts
luksmeta
manga-cli
mpvScripts.convert
mup
netpbm
nhentai
ocrmypdf
oobicpl
paperless-ngx
pcb
pdfarranger
pfstools
plasma5Packages.cantor
pnglatex
polymake
protontricks
pynac
python311Packages.abjad

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/80xacjn7msql3lf4ylk8q6lsmmrav6k6-netpbm-11.8.1-bin \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

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

Or:
nix build github:r-ryantm/nixpkgs/188834d68635b4c05de6ad37270113acb7768548#netpbm

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/80xacjn7msql3lf4ylk8q6lsmmrav6k6-netpbm-11.8.1-bin
ls -la /nix/store/80xacjn7msql3lf4ylk8q6lsmmrav6k6-netpbm-11.8.1-bin/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
nixpkgs-review took longer than 120m and timed out

Maintainer pings

[!TIP]
As a maintainer, if your package is located under pkgs/by-name/*, you can comment @NixOS/nixpkgs-merge-bot merge to automatically merge this update using the nixpkgs-merge-bot.

Add a :+1: reaction to pull requests you find important.


feat: redirect unauthenticated user to the login page

Description
I moved the login check to main.ts. The link redirection is now done in auth.ts. Not sure if this is the best way to do it - I was just focused on getting auth operations outside of the Vue components.
In this version you won't be automatically redirected to see passwords for adam and ursala (localhost:8078) it will go straight to the keycloak login. I assume that's fine.
Resolves #113
Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide reproduction instructions and list any relevant details for your test configuration

[ ] Test A
[ ] Test B

Checklist:

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

Redirecting to /app and bypassing the landing page is a potential issue.  The current landing page is for Uncharted developers to remember what users they have available and have a link to the only UI at the time the landing page was written, however, in the future (now?) that could/should change.  The landing page should be a menu of UI/Tools that can be selected to use, or simply a info dump as to what TERArium is, this might be pointless to developers but could be instrumental to actual users.

Thanks @Nithos
I believe once we removed the authentication from the front-end we will not need this. Or we could use the staging keycloak with a test user specifically for this purpose. But this means using secrets within test. Is this possible?

Thanks @Nithos I believe once we removed the authentication from the front-end we will not need this. Or we could use the staging keycloak with a test user specifically for this purpose. But this means using secrets within test. Is this possible?

Authentication is getting removed? How?
Currently this is just an intercept that mocks a response from keycloak. This response can be anything really it does not matter, however currently 'auth' is extracting some meta information from the token (name, email) and eventually it could extract the roles that user has access to (analyst, admin, etc..) So if we want to mimick that we just need to encode whatever we need within that token used for tests.
This was meant to use the dummy user we have but can be changed, and it can also be loaded from a local env if needed. But I think for test purposes we just need to generate ones that meet our needs for the tests only, for now at least. Again this may need to change in the future.


Fix syntax warnings

Exception handling (due to https://github.com/abapGit/abapGit/pull/5638)
Usage of secondary keys
Select FAE with LOB

After:

thanks 👍


Tls communication fails while using the engine tpm2tss with OpenSSL 3.1.0

If I try to establish a tls communication with a server using for example this command:
```
$ openssl s_client -connect example.com:8883 -servername example.com -key 0x81000001 -cert client.crt -keyform engine -engine tpm2tss
```

I get this error.
$4057BF5CB17F0000:error:0A00041B:SSL routines:ssl3_read_bytes:tlsv1 alert decrypt error:../ssl/record/rec_layer_s3.c:1584:SSL alert number 51
I tested also the tools mosquito and curl and I got the same problem. The both tools use also the openssl engine functionality.
Is this issue already known or the openssl3 supports further the usage of the engine and for me specially the tpm2tss engine.
the tested version is: OpenSSL 3.1.0 14 Mar 2023 (Library: OpenSSL 3.0.2 15 Mar 2022)

I'd suggest at least upgrading the library (or try to use LD_LIBRARY_PATH) to the 3.0.8 or 3.1.0 version as there were some fixes related to engines since 3.0.2.

marking as inactive, will be closed at the completion of 3.4 dev without further input


Maximum number of ENV variables in docker versin 1.9.0

Hi,
When I user more than 10 environment variables in docker file and pass them to the bash scripts as command line arguments it is messing the environment variables that are over 10.
But when I use less then 10 every thing is working as it is supposed to.
Is there a limitation on the number of ENV variables or is it a bug?
Docker version is 1.9.0
Host is CentOS 7.1

Hi!
Please read this important information about creating issues.
If you are reporting a new issue, make sure that we do not have any duplicates already open. You can ensure this by searching the issue list for this repository. If there is a duplicate, please close your issue and add a comment to the existing issue instead.
If you suspect your issue is a bug, please edit your issue description to include the BUG REPORT INFORMATION shown below. If you fail to provide this information within 7 days, we cannot debug your issue and will close it. We will, however, reopen it if you later provide the information.
This is an automated, informational response.
Thank you.
For more information about reporting issues, see https://github.com/docker/docker/blob/master/CONTRIBUTING.md#reporting-other-issues

BUG REPORT INFORMATION
Use the commands below to provide key information from your environment:
docker version:
docker info:
uname -a:
Provide additional environment details (AWS, VirtualBox, physical, etc.):
List the steps to reproduce the issue:
1.
2.
3.
Describe the results you received:
Describe the results you expected:
Provide additional info you think is important:
----------END REPORT ---------
#ENEEDMOREINFO

@eemin can you share with us your Dockerfile and exact command line you use to build, and/or run the image?  I'm not seeing this issue.

Well to figure out what is going on I made a very simple script and the  test the situation in clean Ubuntu base image . Every thing there is working as it is supposed to be.
So this is something else I have to figure it out, either from the image or the scripts  that are used in it. Image was prepared by a colleague for the company.
Sorry to waste some space.

@eemin no problem, hope you're able to resolve your issue!


[1.12] Bump Mesos to nightly 1.7.x d8acd9c

High-level description
This is a routine bump to the latest Mesos and mesos-modules.
Related JIRA Issues
Checklist for all PRs

[ ] Included a test which will fail if code is reverted but test is not. If there is no test please explain here:
[x] Read the DC/OS contributing guidelines
[x] Followed relevant code rules Rules for Packages and Systemd

Checklist for component/package updates:
If you are changing components or packages in DC/OS (e.g. you are bumping the sha or ref of anything underneath packages), then in addition to the above please also include:

[x] Changelog:
https://github.com/apache/mesos/compare/71685b5c807e4d4d1a5ebe33a3c3c0a2f824b9b4...d8acd9cfacd2edf8500f07f63a8837aa0ddd14ba
https://github.com/dcos/dcos-mesos-modules/compare/3f87e3d7d4706decdc20a046fa280a47c2e2954a...e6cd9e3844a666752b4758e383a5d46e965a27db

[ ] Test Results: [link to CI job test results for component]

[ ] Code Coverage (if available): [link to code coverage report]

@mesosphere-mergebot bump-ee

@mesosphere-mergebot label Ready For Review


FAILS for ligature lookup I don't understand

Apparently the font file is corrupted or malformed, and I don't know why. Maybe the feature order?
🔥 FAIL Failed to lookup ligatures. This font file seems to be malformed. For more info, read: googlefonts/fontbakery#1596 [code: malformed]
Happening there: https://github.com/kosmynkab/Bona-Nova

I think I've seen this before. I will have to take a look. If I recall correctly, it probably means fontbakery is actually not interpreting it correctly while the file is good. (but I have to inspect the file to be sure)

Here’s another one: https://github.com/google/fonts/pull/3326

One more remark: I did check the fonts of the above PR with fontbakery today using the latest version directly from the git repo, and didn't see this and a few other FAILs except the avar one which is a legitimate FAIL here.

I think https://github.com/googlefonts/fontbakery/pull/3246 will solve this.

Apparently the font file is corrupted or malformed, and I don't know why. Maybe the feature order?

I had opened an Issue for this a while ago.
Happy if Denis's PR solves it!

Can this issue be closed?

This can be closed. This has been fixed by #3246.


Add DeleteAfter tag to Search test resource groups

closes #11046
if you just want to see the non-recording changes, they are all in https://github.com/Azure/azure-sdk-for-python/commit/7a924c578efc3b273587bce06d7e399a6b7644e1

@scbedd @KieranBrantnerMagee I have filled https://github.com/Azure/azure-sdk-for-python/issues/11137 as a follow up


group commands with prefix

Hi,
How i can group commands  with prefix?
operations for group1

script.py myprefix list
script.py myprefix create ARG
script.py myprefix delete ARG
script.py myprefix update ARG1 ARG2

operations for group2

script.py myprefix2 list
script.py myprefix2 create_other ARG ARG2
script.py myprefix2 update_foo ARG1 ARG2

i can create one command with many optional args, but it not better way.
Can i split functions list, create, delete, bul call with prefix?

You are thinking of command groups: http://click.pocoo.org/5/commands/

`import click
@click.group()
def cli(debug):
click.echo('Debug mode is %s' % ('on' if debug else 'off'))
@cli.command()
def sync():
click.echo('Synching')
if name == 'main':
cli()
`
sync command calling from cmd as: script.py sync, but i need call with prefix script.py foo sync.

Then create a new sub-group using cli.group.

Also please don't abuse the issue tracker for support requests. I will lock this issue now.


optional component: set up https+letsencrypt

https://community.openhab.org/t/securing-openhab-with-free-ssl-lets-encrypt-certificates/7918

Fixed in https://github.com/ThomDietrich/openhabian/pull/26 and following.
Thanks @BClark09


Option to control blank border on each side of the category axis?

Hi,
I can't figure out how to get rid of gap on each end of the X axis (before and after the first and last X values): http://i.imgur.com/TntkyC6.png
My code: http://jsfiddle.net/bz9mLome/
For example, Baidu's ECharts has a boundaryGap option which allows exactly that. I couldn't find anything similar in C3, so would appreciate any workarounds and also put in a feature request.
Thank you.

You can set the x axis's padding to 0 in order to get rid of that gap.
axis: {
  x: {
    padding: {
      left: 0,
      right: 0,
    }
  }
}

@olivierguerriat if things were that simple in life... :) I tried all possible solutions before posting the issue. You can use my JSFiddle to test the padding. Appears to be the case only when x type is category.

Putting the padding to -0.5 does the trick. But it's not the cleanest configuration…

That's a neat hack! Thank you!
I'll use it for the time being but I do agree that it's not the cleanest configuration. 0 should be smack at the intersection of axes. Perhaps it's due to the way category types are rendered. It'd be nice to have a boundaryGap-type of configuration option though.

Workaround present
When margin set to 0,  the charts should start from beginnging of the axis


oop-ext v0.5.1

I'm just pointing the meta.yaml to oop-ext v0.5.1 and adding my Github handle to the maintainers on the readme.

Checklist

[X] Used a fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.


extract instruction names and descriptive prose

In addition to mnemonics, instructions need human-friendly names and descriptions.
Unfortunately, these are not present in the RISC-V Sail specification, so this issue is a sort of placeholder / reminder to pursue adding these to the Sail specification hosted at https://github.com/riscv/sail-riscv.

Deprecating / archiving project.


[Bug]: RawSource from webpack-sources is not assignable to rspack's Source

System Info
System: Mac
Details
When i use RawSource, ts says it cannot be assigned to rspack's Source.
Seems like the return type of sourceAndMap()'s definition is not compatible with webpack-sources:
webpack-sources:
sourceAndMap(options?: MapOptions): SourceAndMapResult;

export interface SourceAndMapResult {
    source: string | Buffer;
    map: RawSourceMap | null;
}

rspack:
sourceAndMap(options?: MapOptions): {
    source: string | Buffer;
    map: Object;
};

Reproduce link
No response
Reproduce Steps
Copy this to editor and see what ts says:
import { RawSource } from "webpack-sources";
import type { Compilation } from "@rspack/core";

function addFile(compilation: Compilation, filename: string, content: string) {
  compilation.emitAsset(filename, new RawSource(content));
}

Replace RawSource that is imported from webpack-sources with compiler.webpack.sources.RawSource.


Support gptq  quantization

Contributions are welcome. You can add the support for GPTQ here
https://github.com/sgl-project/sglang/blob/71b54eea7d21a2bb1d8ef340e7002983a29b1d5f/python/sglang/srt/managers/router/model_runner.py#L282-L286
We reuse the quantization and linear method in vLLM, so you can learn how to use them in the vLLM code
https://github.com/vllm-project/vllm/blob/3dad94448583a835230ae68c726d4b67e46845f2/vllm/model_executor/model_loader.py#L47

done by #141


Revision: Remove stale industry demand rules?

It seems that the rules build_industrial_energy_demand_per_country and build_industrial_demand are not used anymore.
Is that correct?

Yes, that is correct. They were replaced by build_industry_energy_demand_per_node


How to get the length of the response body before gzip decompress

I'm going to count the bytes received by every request, using the codes below,
long length = response.body().bytes().length;

but the length is much greater then the actual body size and actually is the uncompressed body size.
I've execute a request and  get the length=4143.
The overview of this request in Charles,

Response	1.07 KB (1,095 bytes)
Header	364 bytes
Cookies	44 bytes
Body	731 bytes
Uncompressed Body	4.05 KB (4,143 bytes)
Compression	82.4% (gzip)

Is there any way to get the length of the response body before gzip decompress?

Two options:

Disable gzip with this header: Accept-Encoding:identity.
Use response.networkResponse() and ask its Content-Length. Unfortunately this isn’t guaranteed to be present.

I've found an solution,
okBuilder.addNetworkInterceptor(new Interceptor() {
            @Override
            public Response intercept(Chain chain) throws IOException {
                Response response = chain.proceed(chain.request());
                ForwardingSource forwardingSource = new ForwardingSource(response.body().source()) {
                    long bodySize;

                    @Override
                    public long read(Buffer sink, long byteCount) throws IOException {
                        long bytesRead = super.read(sink, byteCount);
                        if (bytesRead != -1) {
                            bodySize += bytesRead;
                        }
                        return bytesRead;
                    }
                };
                Response.Builder responseBuilder = response.newBuilder();
                responseBuilder.body(new RealResponseBody(response.headers(), Okio.buffer(forwardingSource)));
                return responseBuilder.build();
            }
        });

But the bodySize smaller then the data in Charles about 10 bytes. Is it the correct way?