Minimize or mitigate the chance of lock contention in worker.go

Small tweak to use a read lock when checking if a work key exists in the running map.
Added a trackWork helper method to minimize the locked lifetime which can possibly introduce contention when synchronizing a shutdown for a worker that takes awhile to finish.

Much Better


Change emit target to ES2022

Looks like we have broad support for ES2022 features. Static initialisers are the slight exception as they are only in Safari since ~2months. We might want to special case them, but otherwise we should be good to go: https://caniuse.com/sr_es13.

On native privates: using them will be faster than weak-map-privates but slower than normal properties (much, much slower for Safari): https://jsbench.me/0gljcn4wzt/2

Some, perf number for some new API

native privates - faster than WeakMap privates but slower than conventional privates
Array.at - slower https://jsbench.me/vhljct96ph/1
ES2022 class definitions - likely faster because of less prototype changes

Upstream issues for private property performance

v8 -> https://bugs.chromium.org/p/v8/issues/detail?id=14127
Webkit -> https://bugs.webkit.org/show_bug.cgi?id=258660


Add xcm schema

Any follow ups?
Please review PR: #86 , before reviewing this PR.
Description

Adds v0-v2 for xcm.
Adds xcm/src.dart served as and entry point to access xcm schema.

Were those fetched from somewhere? If yes, please put the source

Alright, reference added ✅


[Bug] keycloak epic issue

Describe the Bug

keycloak
https://github.com/OHIF/Viewers/issues/4133

Steps to Reproduce
d
The current behavior
d
The expected behavior
d
OS
d
Node version
d
Browser
d

We recently added several recipes for implementing authentication with Keycloak in OHIF. You can find them here:
https://docs.ohif.org/deployment/user-account-control

We released OHIF 3.9 which address this issue


New Radix Themes release and documentation tweaks

Updates Radix Themes and Radix Colors to the upcoming versions
Some documentation tweaks across the board, mostly around how we showcase some component variants and high contrast options
Some design tweaks on the demos
Scroll active sidebar link into view
Clean up Radix Themes example UIs

I diff'd against both sites for all comps, the home page, and the colors pages.

Bless you, appreciate that a ton.

Released 2.0.0 so going to merge now.
I added in some juicy stuff on top of what you've reviewed @andy-hook, feel free to catch up as you guys get back. Main additions:

Beef up Radix Colors homepage
Add links to previous versions of Radix Themes docs
Improve how favicon responds to theme changes
Do a pass on meta titles
Set homepage meta to make sense for all of Radix UI. As of now it doesn't:

Docs tweaks

Added container widths
Tweaked breakpoint descriptions
Remove /colors/docs/tests (almost irrelevant now with the new homepage & a lot of the content there was questionable)
Replace old alert components with callouts


python38Packages.falcon: 2.0.0 -> 3.0.1

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python38Packages.falcon is: "An unladen web framework for building APIs and app backends"
meta.homepage for python38Packages.falcon is: "https://falconframework.org/"
meta.changelog for python38Packages.falcon is: ""
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS

Warning: a test defined in passthru.tests did not pass

/nix/store/vs8l7m1hsvgcbxqj4pk9dqaf71388by9-python3.8-falcon-3.0.1/bin/falcon-bench passed the binary check.

/nix/store/vs8l7m1hsvgcbxqj4pk9dqaf71388by9-python3.8-falcon-3.0.1/bin/falcon-inspect-app passed the binary check.

/nix/store/vs8l7m1hsvgcbxqj4pk9dqaf71388by9-python3.8-falcon-3.0.1/bin/falcon-print-routes passed the binary check.

/nix/store/vs8l7m1hsvgcbxqj4pk9dqaf71388by9-python3.8-falcon-3.0.1/bin/.falcon-bench-wrapped passed the binary check.

/nix/store/vs8l7m1hsvgcbxqj4pk9dqaf71388by9-python3.8-falcon-3.0.1/bin/.falcon-inspect-app-wrapped passed the binary check.

/nix/store/vs8l7m1hsvgcbxqj4pk9dqaf71388by9-python3.8-falcon-3.0.1/bin/.falcon-print-routes-wrapped passed the binary check.

6 of 6 passed binary check by having a zero exit code.

2 of 6 passed binary check by having the new version present in output.

found 3.0.1 with grep in /nix/store/vs8l7m1hsvgcbxqj4pk9dqaf71388by9-python3.8-falcon-3.0.1

found 3.0.1 in filename of file in /nix/store/vs8l7m1hsvgcbxqj4pk9dqaf71388by9-python3.8-falcon-3.0.1

directory tree listing: https://gist.github.com/8a93603c7de3db3699ea7d40c68aa68c

du listing: https://gist.github.com/d79d0a6bc81433c26518bd0d3b658328

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

80 total rebuild path(s)

20 package rebuild(s)

20 x86_64-linux rebuild(s)
20 i686-linux rebuild(s)
20 x86_64-darwin rebuild(s)
20 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
mailman
pylode
python38Packages.falcon
python38Packages.hug
python38Packages.jupyterlab
python38Packages.jupyterlab_server
python38Packages.jupytext
python38Packages.mailman
python38Packages.mailman-hyperkitty
python38Packages.openapi-core
python38Packages.webargs
python39Packages.falcon
python39Packages.hug
python39Packages.jupyterlab
python39Packages.jupyterlab_server
python39Packages.jupytext
python39Packages.mailman
python39Packages.mailman-hyperkitty
python39Packages.openapi-core
python39Packages.webargs

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/vs8l7m1hsvgcbxqj4pk9dqaf71388by9-python3.8-falcon-3.0.1 \
  --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 python38Packages.falcon https://github.com/r-ryantm/nixpkgs/archive/f7d77f85fd2c114ebfb56a75f41b662b0c9cda77.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/vs8l7m1hsvgcbxqj4pk9dqaf71388by9-python3.8-falcon-3.0.1
ls -la /nix/store/vs8l7m1hsvgcbxqj4pk9dqaf71388by9-python3.8-falcon-3.0.1/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 45m and timed out

Maintainer pings
cc @desiderius for testing.

Result of nixpkgs-review pr 134628 at f7d77f85 run on aarch64-linux 1

15 packages failed to build:

mailman
python38Packages.hug
python38Packages.jupyterlab
python38Packages.jupyterlab_server
python38Packages.jupytext
python38Packages.mailman
python38Packages.mailman-hyperkitty
python38Packages.openapi-core
python39Packages.hug
python39Packages.jupyterlab
python39Packages.jupyterlab_server
python39Packages.jupytext
python39Packages.mailman
python39Packages.mailman-hyperkitty
python39Packages.openapi-core

5 packages built successfully:

pylode
python38Packages.falcon
python38Packages.webargs
python39Packages.falcon
python39Packages.webargs

1 suggestion:

warning: python-include-tests
Consider adding a checkPhase for tests, or if not feasible, pythonImportsCheck.
Near pkgs/development/python-modules/falcon/default.nix:28:0:
   |
28 |     description = "An unladen web framework for building APIs and app backends";
   | ^

Note that build failures may predate this PR, and could be nondeterministic or hardware dependent.
Please exercise your independent judgement. Does something look off? Please file an issue or reach out on IRC.


Add $ to the variables in the CLI example.

In this CLI example:
subscription=""
example value is southeastasia
region=""
example value is Standard_E64s_v3
vmSize=""
az vm list-skus --resource-type virtualMachines  --location $region --query "[?name=='$vmSize'].locationInfo[0].zoneDetails[0].Name" --subscription $subscription
the variables [subscription="", region="", and vmSize=""] don't have $. So, if someone pastes values as the comments describe, the user could still receive an error because the variables don't have a $. Users may not understand the need for $ if they're not already familiar with the syntax. I recommend we add $ to the variables or at least add a note to include a prefix for the variables. The Powershell example on the same page has $:
$region = "southeastasia"
$vmSize = "Standard_E64s_v3"
$sku = (Get-AzComputeResourceSku | where {$.Locations.Contains($region) -and ($.Name -eq $vmSize) -and $_.LocationInfo[0].ZoneDetails.Count -gt 0})
if($sku){$sku[0].LocationInfo[0].ZoneDetails} Else {Write-host "$vmSize is not supported with Ultra Disk in $region region"}

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

ID: a4c7ec02-cc18-3f20-704e-79f148f136e8
Version Independent ID: beb667ae-5d75-c979-f621-c75e21453b6a
Content: Ultra disks for VMs - Azure managed disks - Azure Virtual Machines
Content Source: articles/virtual-machines/disks-enable-ultra-ssd.md
Service: virtual-machines
Sub-service: disks
GitHub Login: @roygara
Microsoft Alias: rogarana

Thanks for bringing this to our attention. I have created a PR with the update, changes will be live shortly.
Thanks for improving our documentation! Just so you know, it's also possible to make these changes yourself, if you are interested. Doing so will improve our documentation, and it will add you as a contributor to the document.
In case you're interested, these are the steps to do so:

Navigate to the page of interest
Select the "Edit" button at the top right of the page
Select the "Pencil" icon on the right
Modify the text within the text box in markdown format - more info: https://github.com/adam-p/markdown-here/wiki/Markdown-Cheatsheet
Name your file change and provide a description, if necessary
Select "Propose file change"
Select "Create pull request" on the 'Comparing changes' page
Select "Create pull request" on the 'Open a pull request' page
You can find a GIF that demonstrates the end-to-end process for submitting changes here: https://aka.ms/editsqldocs.

I noticed the CLI example here doesn't have the $ next to the variables yet. I know you wrote the changes would go live shortly, but I'm curious if the changes should've been live now?


Improvements

added

check to see if any webhook environment variable has been set
project name (and project id) to metadata
progress reporting every 5 seconds (configurable with REPORT_STATUS_EVERY)
subfilters for discord and generic webhooks LOGS_FILTER_DISCORD, and LOGS_FILTER_WEBHOOK
seamless log subscription resumption on disconnect
debug logging enabled with DEBUG

fixed

potential runaway for loop in the event of SendWebhooks failing
tiny memory leak

improvements

updated readme.md with more info
uses strings.EqualFold for level filtering
reimplement subscription code
reimplement discord webhooks
send discord and generic webhooks concurrently
add common timestamps to log objects
send an array of log objects instead of singular log objects for the generic webhook
bulk logs are sent as a single discord webhook that contains an embed for each log object

Ready for review


Add a CHANGELOG file

Add a changelog file. What is a changelog file:

https://www.youtube.com/watch?v=Pi8akLplC9M&ab_channel=NickJanetakis
https://keepachangelog.com/en/1.0.0/

Can I suggest that the CHANGELOG.md simply links to the book (or vice versa)? Maintaining two sounds like kind of a pain, whereas you could just link to the generated mdbook file (either the source file or the hosted version (GitHub pages?)).

Can I suggest that the CHANGELOG.md simply links to the book (or vice versa)? Maintaining two sounds like kind of a pain, whereas you could just link to the generated mdbook file (either the source file or the hosted version (GitHub pages?)).

that's the point of this issue https://github.com/John15321/rust-pip/issues/34


Custom Environment Variables for Postgres Pods

This PR adds the ability to configure a config map which is used by the Postgres pods as additional provider for environment variables. This allows to customize the Spilo image and configure it with environment variables. The config map with the additional settings is configured in the operator's main config map.
postgres-operator ConfigMap
apiVersion: v1
kind: ConfigMap
metadata:
  name: postgres-operator
data:
  # referencing config map with custom settings
  pod_env_configmap: postgres-pod-config
  ...

ConfigMap for custom settings
apiVersion: v1
kind: ConfigMap
metadata:
  name: postgres-pod-config
  namespace: default
data:
  MY_CUSTOM_VAR: value
  FOR_POSTGRES: demo

This ConfigMap is then added to the StatefulSet/pods with following spec:
envFrom:
- configMapRef:
    name: postgres-pod-config

Considerations
In our case we changed some scripts in the Spilo image to have a custom backup solution which needs to be configured with additional environment variables. I don't think we can contribute the Spilo changes back as they are very specific for our use cause. Therefore it doesn't make sense to add additional environment settings to the operator ConfigMap and have an explicit mapping in the operator code. And I think having the additional ConfigMap results in a very simple and generic way to allow custom variables.
What do you think? Is it useful for other use cases or against the operator paradigm in general?
Please let me know.
If the setting is omitted in the operator config, nothing is added to the pod.

Coverage decreased (-0.01%) to 7.882% when pulling d55b0fbf925fc1c0b3c90fe3e91b6c3d67126021 on geku:custom-env into d3679bfd4a0c63bf6677f482f3a15a8d69d818dd on zalando-incubator:master.

I like the idea. Let's see what others comment here.
Though I wonder, why are you not building your own Docker Image and setting ENV vars there and just base it on ours? That is our current approach right now, slim last layer with our custom config.
Maybe sometime in the future there is a template for the whole stateful set somewhere that one could customize freely, did this for another operator before. Helps to understand what is created.

The approach looks good, but I'd introduce a more descriptive name instead of pod_env_configmap, perhaps just set_pod_environment_from_configmap?

pod_environment_configmap would be fine with me

@geku Just change the name of the field in the config map to "pod_environment_configmap" and then this looks good to us. Seems this is useful over fiddling with docker images.

Sorry was off a couple of days. I'll change the name and document it as well.

@Jan-M I rebased, changed the name and added some documentation to the readme.
Btw having a template for the pods sounds great but requires quite some more work.

Changes Unknown when pulling 257e0fc0a959f7e395e08136c92aabd63a5c3f31 on geku:custom-env into ** on zalando-incubator:master**.

Changes Unknown when pulling 257e0fc0a959f7e395e08136c92aabd63a5c3f31 on geku:custom-env into ** on zalando-incubator:master**.

looks good to me. I did not know this env from feature.

:+1:

@alexeyklyukin Is there more work needed for the diff between statefulset pod template and the actual set? assuming the env config map is changed and the operator is restarted, does it apply the changes as we are used to?

@Jan-M yes, if you make changes inside the referenced configmap added in this PR the operator without changing the reference itself (i.e. the name of the configmap) the operator won't see the changes, since the statefulset will look absolutely the same, although in reality a pod restart is required to apply the changes within the configmap.

@geku a small fix to make statefulset comparison code aware of the envFrom: https://github.com/geku/postgres-operator/pull/1 (without it existing clusters would not pick up the environment from pod_environment_configmap).

@geku another idea, perhaps we should inject variables from the pod configmap into the resulting statefulset by ourself? That would allow us to detect when those variables have changed and do pods rolling update when necessary. Since that configmap scope is the operator, we can read it only once at start.

@alexeyklyukin I see how adding the env vars directly improves updating existing pods. But am I not sure what happens when the ConfigMap is updated? I guess nothing as the ConfigMap is not watched by the operator.

Changes Unknown when pulling 100caff8026169b7cbc4f609ff1e69bcac3b06a5 on geku:custom-env into ** on zalando-incubator:master**.

This is true for both the conifgmap of the operator itself as for your configmap for the env variables.
For now we actually work with redeploying/restarting the operator on such changes before working on "runtime" config change support.

@geku the operator doesn't watch the configmap, but it does sync each cluster every sync period (5 minutes by default). During the sync the operator will generate the new statefulset definition based on. the updated contents of the configmap and trigger a rolling upgrade if old and new definitions don't match each other.
Here's a PR against your branch implementing this idea: https://github.com/geku/postgres-operator/pull/4
I aimed to make its  behaviour less surprising by sticking to the same rules as envSource: the configmap is only searched in the local cluster namespace, and pod variables have priority over those fetched from the PodEnvironmentConfigMap.

@geku @alexeyklyukin what else is needed here?

@Jan-I just merged the change by @alexeyklyukin into my branch, looks good from my side.

@geku thank you!
Rebased it to the recent master with https://github.com/zalando-incubator/postgres-operator/pull/175 to eliminate conflicts. Going to run a few manual tests before merging.

Closing this PR in favor of the rebased branch on #175

@geku merged your changes, thanks a lot!


When will the power laws solutions be published ? :)

Thank you very much for uploading the solutions.


Provide an API to modify records within files

We need an API to atomically remove some existing files and add new ones to perform "eager" updates and deletes. While RewriteFiles is doing exactly this, it seems to be designed for cases like compaction, where a set of files is replaced but the content does not change.
RewriteFiles cannot be used for updates and deletes because of its logic on retries: it will succeed if all files it tries to remove are still there.
Let's consider an example:

An update starts when the current snapshot is s1.
While we are doing the update, there might be an append that commits first. Therefore, s2 is produced and is marked as the current snapshot.
RewriteFiles detects the change in metadata and retries.
The retry attempt succeeds because the files to be deleted are still there even though the append could add new records that match the update condition. The added matching records won't be taken into account.

We need to either extend/modify the retry logic in an existing operation or provide a new API.

Hi!
I'm curious about the update condition in:
The retry attempt succeeds because the files to be deleted are still there even though the append could add new records that match the update condition.

It looks like this ReplaceFiles.rewriteFiles in a set of DataFiles. Do you mean the condition used to construct this set?

@aokolnychyi, updating the API to support eager rewrites sounds good to me! I think this should probably be a variant of overwrite, though. RewriteFiles produces a replace commit, which signals that the data did not change. Is it possible to update overwrite instead.

@rdblue I am flexible there, let's clarify what's best during review. I've summarized the approach we were taking in this comment.
@johnclara Sorry for the delay, I'll submit what we have by the end of week.


504 Gateway Time-out

My code has been running for months, but today I am experiencing the following errors:
Failed to sign in AuthError: Unexpected error occurred with message: An unknown error occurred
Recovery suggestion: This should not happen. There is a possibility that there is a bug if this error persists. Please take a look at https://github.com/aws-amplify/amplify-ios/issues to see if there are any existing issues that match your scenario, and file an issue with the details of the bug if there isn't. Issue encountered at:
file: /Users/development/Documents/Development/AWSFullStackSocial/Pods/Amplify/Amplify/Categories/Auth/Error/AuthError.swift
function: recoverySuggestion
line: 79
Caused by:
Error Domain=com.amazonaws.AWSServiceErrorDomain Code=0 "
<title>504 Gateway Time-out</title>

504 Gateway Time-out

" UserInfo={NSLocalizedDescription=
<title>504 Gateway Time-out</title>

504 Gateway Time-out

}

The code that runs this is:
    func login(username: String, completion: @escaping (Swift.Result<User, Error>) -> ()) {
        self.isLoading.value = true
            firstly {
                self.servicesProtocol.signIn(username: username)
            }.then { _ in
                self.servicesProtocol.getUser()
            }.done { user in
                self.currentUser.value = user
                self.loginChangeDelegate?.loginChange()
                self.isLoading.value = false
                completion(.success(user))
            }.catch { error in
                self.isLoading.value = false
                completion(.failure(error))
                print("Failed to retreve user details: \(error)")
            }
    }

    func signIn(username: String) -> Promise<AuthSignInResult> {
        return Promise { seal in
            _ = Amplify.Auth.signOut(listener: { result in
                switch result {
                case .success():
                    let password = "Abc@123!"

                    _ = Amplify.Auth.signIn(username: username, password: password, listener: { result in
                        switch result {
                        case .success(let signInResult):
                            if let id = Amplify.Auth.getCurrentUser()?.username {
                                print("Successfully signed in")
                                print("Current user id: \(id)")
                            }
                            seal.fulfill(signInResult)
                        case .failure(let error):
                            print("Failed to sign in \(error)")
                            seal.reject(error)
                        }
                    })
                case .failure(let error):
                    print("Failed to sign out \(error)")
                    seal.reject(error)
                }
            })
        }
    }

    func getUser(for userId: String? = nil) -> Promise<User> {
        return Promise { seal in

            if let id = userId != nil ? userId : Amplify.Auth.getCurrentUser,()?.userId {
                print("Username is: \(id)")
                _ = Amplify.API.query(request: .get(User.self, byId: id) ) { event in
                    switch event {
                    case .success(let result):
                        switch result {
                        case .success(let user):
                            guard let user = user else {
                                print("Unable to find user")
                                return
                            }
                            print("Succesfully retrieved user: \(user)")
                            seal.fulfill(user)
                        case .failure(let error):
                            print("Failed retrieve user: \(error)")
                        }
                    case .failure(let error):
                        print("Failed to get user: \(error)")
                        seal.reject(error)
                    }
                }
            }
        }
    }

I am running the latest everything, including Xcode 12.2, iOS simulator 14.2
Nothing has changed since yesterday / this morning in my app, the code worked perfectly fine.
Thanks.

Same here. Today this error occured for the first time. Unable to download anything due to AuthError.
Highest priority.
Data Failed: Unexpected error occurred with message: An unknown error occurred. This should not happen. There is a possibility that there is a bug if this error persists. Please take a look at https://github.com/aws-amplify/amplify-ios/issues to see if there are any existing issues that match your scenario, and file an issue with the details of the bug if there isn't. Issue encountered at:

hi @mickeysox and @ppoh71 , thanks for reaching out.
Looks like this morning Amazon Cognito experienced an increase in API failures rate in US-EAST-1  region.
I'd recommend follow https://status.aws.amazon.com for updates.

@diegocstn is there an ETA on this? We're also impacted.

@DavidAdade AWS is experiencing a Large Scale Event (LSE). Please follow the AWS Service Health Dashboard.


Fix last message text in LHN includes text and no space above quoted text

Details

Using htmlToText from ExpensifyMark to replace breakline and blockquote closing tag to space ' ', strip out tags.
Fixed Issues

$ https://github.com/Expensify/App/issues/8134
Tests

Open chat
Send quoted text and non-quoted text in a single message. Example:

> Confusing stuff
Tell me about it

Verify on the sender and receiver LHN that there's empty space after the quoted text. Example:
Confusing stuff Tell me about it

[x] Verify that no errors appear in the JS console

PR Review Checklist

Contributor (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 Staging and/or Production testing in the QA steps section
[ ] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[ ] 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 included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] iOS / native
[x] Android / native
[x] iOS / Safari
[x] Android / Chrome
[x] MacOS / Chrome
[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 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 was added in all src/languages/* files
[x] I verified any copy / text that was added to the app is correct English and approved by marketing by tagging the marketing team 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

[ ] 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
[ ] If a new component is created I verified that:

[ ] A similar component doesn't exist in the codebase
[ ] All props are defined accurately and each prop has a /** comment above it */
[ ] Any functional components have the displayName property
[ ] The file is named correctly
[ ] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[ ] The only data being stored in the state is data necessary for rendering and nothing else
[ ] 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)
[ ] 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)
[ ] All JSX used for rendering exists in the render method
[ ] The component has the minimum amount of code necessary for its purpose and it is

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

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

[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.

PR Reviewer Checklist

[ ] I verified the correct issue is linked in the ### Fixed Issues section above
[ ] I verified testing steps are clear and they cover the changes made in this PR

[ ] I verified the steps for local testing are in the Tests section
[ ] I verified the steps for Staging and/or Production testing are in the QA steps section
[ ] I verified the steps cover any possible failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[ ] 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)

[ ] I checked that screenshots or videos are included for tests on all platforms
[ ] I verified tests pass on all platforms & I tested again on:

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

[ ] 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)
[ ] I verified proper code patterns were followed (see Reviewing the code)

[ ] 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).
[ ] I verified that comments were added to code that is not self explanatory
[ ] 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.
[ ] I verified any copy / text shown in the product was added in all src/languages/* files
[ ] I verified any copy / text that was added to the app is correct English and approved by marketing by tagging the marketing team on the original GH to get the correct copy.
[ ] 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.
[ ] I verified the JSDocs style guidelines (in STYLE.md) were followed

[ ] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[ ] I verified that this PR follows the guidelines as stated in the Review Guidelines
[ ] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[ ] 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
[ ] If a new component is created I verified that:

[ ] A similar component doesn't exist in the codebase
[ ] All props are defined accurately and each prop has a /** comment above it */
[ ] Any functional components have the displayName property
[ ] The file is named correctly
[ ] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[ ] The only data being stored in the state is data necessary for rendering and nothing else
[ ] 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)
[ ] 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)
[ ] All JSX used for rendering exists in the render method
[ ] 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

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

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

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

QA Steps

Open chat
Send quoted text and non-quoted text in a single message. Example:

> Confusing stuff
Tell me about it

Verify on the sender and receiver LHN that there's empty space after the quoted text. Example:
Confusing stuff Tell me about it

[x] Verify that no errors appear in the JS console

Screenshots

Web

https://user-images.githubusercontent.com/25520267/165682119-698a583c-28ae-4a84-98bf-ca8f3885db36.mp4
Mobile Web

https://user-images.githubusercontent.com/25520267/165682152-f4f52dde-792b-48b8-a2d1-631e5ad6b69d.mp4
https://user-images.githubusercontent.com/25520267/165682160-8e92ec6a-546a-4546-86b8-12f2dab8d106.mp4
Desktop

https://user-images.githubusercontent.com/25520267/165682184-c84796e2-ad1e-4094-a255-b76eb5beaaee.mp4
iOS

https://user-images.githubusercontent.com/25520267/165682212-bf0400d9-33ed-48c1-b8cf-cdd336a7d1b0.mp4
Android

https://user-images.githubusercontent.com/25520267/165682257-8834fd94-9ebe-44c8-9ebb-9c9e98a3525c.mp4

@mollfpr We can [Hold] on the PR title.

@Luke9389 Any update for the API?

Still on hold. Feel free to take other jobs in the meantime @mollfpr

Took this off HOLD, feel free to continue. Sorry for the long delay here @mollfpr

@Luke9389 No problem! 👍

Please merge main.

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

🚀 Deployed to staging by @Luke9389 in version: 1.1.75-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by @roryabraham in version: 1.1.75-1 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


Template action with ID 'azure:pipeline:create' is not registered.

I have followed and installed Azure Pipeline Actions  using scaffolder-backend-module-azure-pipelines GitHub Repo. Even after following the steps, the desired actions are not being installed in my Backstage.
I followed the standalone server documentation to install Backstage. : https://backstage.io/docs/getting-started/
Followed by I installed the following actions : https://github.com/Parfuemerie-Douglas/scaffolder-backend-module-azure-pipelines
When I try to use the template it gives the following error Template action with ID 'azure:pipeline:create' is not registered.
Can someone help out me, why this actions are not visible in Backstage?
Additionally if tweak and modify template to fetch the pipeline from GitHub, will the template work and create a pipeline?

This work-around fixed it for me: https://github.com/Parfuemerie-Douglas/scaffolder-backend-module-azure-pipelines/issues/42


When using a container generate a loose application relative to a certain root directory.

For now I've hard coded /devmode but perhaps there is a better name.

Can you also open an equivalent PR for ci.gradle?


Add containerized rpmbuild to toolkit

Merge Checklist

All boxes should be checked before merging the PR (just tick any boxes which don't apply to this PR)

[X] The toolchain has been rebuilt successfully (or no changes were made to it)
[X] The toolchain/worker package manifests are up-to-date
[X] Any updated packages successfully build (or no packages were changed)
[X] Packages depending on static components modified in this PR (Golang, *-static subpackages, etc.) have had their Release tag incremented.
[X] Package tests (%check section) have been verified with RUN_CHECK=y for existing SPEC files, or added to new SPEC files
[X] All package sources are available
[X] cgmanifest files are up-to-date and sorted (./cgmanifest.json, ./toolkit/scripts/toolchain/cgmanifest.json, .github/workflows/cgmanifest.json)
[X] LICENSE-MAP files are up-to-date (./SPECS/LICENSES-AND-NOTICES/data/licenses.json, ./SPECS/LICENSES-AND-NOTICES/LICENSES-MAP.md, ./SPECS/LICENSES-AND-NOTICES/LICENSE-EXCEPTIONS.PHOTON)
[X] All source files have up-to-date hashes in the *.signatures.json files
[X] sudo make go-tidy-all and sudo make go-test-coverage pass
[X] Documentation has been updated to match any changes to the build system
[X] Ready to merge

Summary

A developer tool to quickly iterate on building a single RPM through a container based build environment.
Change Log

new make targets, containerized-rpmbuid and containerized-rpmbuild-help
add developer tool to toolkit
cleanup docker images and containers created for the tool on make clean

This tool creates a mariner container, either using the worker chroot as the fs or using upstream Mariner container (depending on the mode), and mounts SPECs, INTERMEDIATE_SRPMS, and out/RPMs from mariner repository at repo_path (or current mariner repo), so the user can build and test a single RPM. The user can choose whether to use locally built RPMs or upstream RPMs to satisfy dependencies. One can use native rpm commands to build packages. Changes made to SPECS/ are synced to the host.
Sample steps to build a spec (say vim)

cd /path/to/CBL-Mariner/toolkit

sudo make containerized-rpmbuild (run make containerized-rpmbuild-help to see optional arguments. This creates a container and lands you inside the container.)

rpm -i /mnt/INTERMEDIATE_SPECS/vim.src.rpm

rpmbuild -ba /usr/src/mariner/SPECS/vim/vim.spec

Please note that one of the purposes of the tool is to not limit the developer to the toolkit while building or testing RPMs. The developer is encouraged to explore and utilise native rpm tools inside the container. The above commands are only a suggestion.

The container shows a welcome screen with sample commands to build RPMs on startup. This welcome screen can be viewed again by running 'show_help'.

Does this affect the toolchain?

NO
Test Methodology

tested building new specs, existing specs

I have not tested if it works on Mainer


Build failure: sysdig

Steps To Reproduce
Steps to reproduce the behavior:

build sysdig

Build log
[ 63%] [MODERN BPF] Building BPF object: /build/source/build/driver/modern_bpf/vfork.bpf.o
clang: warning: ignoring '-fstack-protector-strong' option as it is not currently supported>
clang: warning: argument unused during compilation: '--gcc-toolchain=/nix/store/3s3rjkl3mx0>
[ 63%] [MODERN BPF] Building BPF object: /build/source/build/driver/modern_bpf/write.bpf.o
clang: warning: ignoring '-fstack-protector-strong' option as it is not currently supported>
clang: warning: argument unused during compilation: '--gcc-toolchain=/nix/store/3s3rjkl3mx0>
clang: warning: ignoring '-fstack-protector-strong' option as it is not currently supported>
clang: warning: argument unused during compilation: '--gcc-toolchain=/nix/store/3s3rjkl3mx0>
[ 63%] [MODERN BPF] Building BPF unique object file: /build/source/build/driver/modern_bpf/>
[ 63%] [MODERN BPF] Building BPF skeleton: /build/source/build/skel_dir/bpf_probe.skel.h
[ 63%] Built target ProbeSkeleton
[ 63%] Building C object libpman/CMakeFiles/pman.dir/src/stats.c.o
[ 63%] Building C object libpman/CMakeFiles/pman.dir/src/maps.c.o
[ 64%] Building C object libpman/CMakeFiles/pman.dir/src/lifecycle.c.o
[ 65%] Building C object libpman/CMakeFiles/pman.dir/src/configuration.c.o
[ 65%] Building C object libpman/CMakeFiles/pman.dir/src/ringbuffer.c.o
[ 65%] Building C object libpman/CMakeFiles/pman.dir/src/programs.c.o
[ 65%] Building C object libpman/CMakeFiles/pman.dir/src/sc_set.c.o
[ 65%] Building C object libpman/CMakeFiles/pman.dir/src/state.c.o
make[5]: *** [/nix/store/hlhp0pprnyq96npl2k1b5z9iy2xc4c9q-linux-6.10-dev/lib/modules/6.10.0>
make[4]: *** [/nix/store/hlhp0pprnyq96npl2k1b5z9iy2xc4c9q-linux-6.10-dev/lib/modules/6.10.0>
make[3]: *** [Makefile:22: all] Error 2
make[2]: *** [driver/CMakeFiles/driver.dir/build.make:70: driver/CMakeFiles/driver] Error 2
make[1]: *** [CMakeFiles/Makefile2:656: driver/CMakeFiles/driver.dir/all] Error 2
make[1]: *** Waiting for unfinished jobs....
[ 65%] Linking C static library libpman.a
[ 65%] Built target pman
[ 65%] Linking CXX static library libcri_v1alpha2.a
[ 65%] Built target cri_v1alpha2
[ 65%] Linking CXX static library libcri_v1.a
[ 65%] Built target cri_v1
make: *** [Makefile:156: all] Error 2

Additional context
Add any other context about the problem here.
Notify maintainers
@7c6f434c
Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
[user@system:~]$ nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 6.9.9, NixOS, 24.11 (Vicuna), 24.11.20240716.b21b793`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Lix, like Nix) 2.91.0-dev-pre20240717-6856720
System type: x86_64-linux
Additional system types: aarch64-linux, i686-linux
Features: gc, signed-caches
System configuration file: /etc/nix/nix.conf
User configuration files: /home/sebtm/.config/nix/nix.conf:/etc/xdg/nix/nix.conf:/home/sebtm/.nix-profile/etc/xdg/nix/nix.conf:/nix/profile/etc/xdg/nix/nix.conf:/home/sebtm/.local/state/nix/profile/etc/xdg/nix/nix.conf:/etc/profiles/per-user/sebtm/etc/xdg/nix/nix.conf:/nix/var/nix/profiles/default/etc/xdg/nix/nix.conf:/run/current-system/sw/etc/xdg/nix/nix.conf:/nix/store/y0996ncxalqaj216ngk9iwr5d3f6sdxp-gnome-settings-daemon-46.0/etc/xdg/nix/nix.conf
Store directory: /nix/store
State directory: /nix/var/nix
Data directory: /nix/store/spcn1af2iapk3qaakv6dvnca1brs1mp4-lix-2.91.0-dev-pre20240717-6856720/share`
 - nixpkgs: `/nix/store/2c8gpdgjf3b0krn48k1hfcb340x72bcd-source`

Add a :+1: reaction to issues you find important.

Can't repro on master (09f65ccafc83ce77148375672be19e6746e45db8) and the error isn't in the log output, can you provide additional info?

Hey, thanks for the response - checking again - it only happens if for me if I enable the sysdig-option in my config not building the sysdig-package - maybe this helps to reproduce and depends on the used kernel-version?
I've uploaded the full build log: https://gist.github.com/SebTM/73918e6d0d573fa5cb2bf03c033e908a

Thanks, seems this is the error:
/build/source/build/driver/src/main.c: In function 'drop_nostate_event':
/build/source/build/driver/src/main.c:1653:22: error: implicit declaration of function 'fd_is_open'; did you mean 'finish_open'? [-Werror=implicit-function-declaration]
 1653 |                     !fd_is_open(close_fd, fdt)
      |                      ^~~~~~~~~~
      |                      finish_open

I guess this is already fixed upstream in the drivers (https://github.com/falcosecurity/libs/pull/1884) but needs to be added to nixpkgs to fix 6.10-kernel builds.

https://github.com/NixOS/nixpkgs/pull/329418
fixes it for me


NL_APPVERSION missing in type definitions

The global property NL_APPVERSION is missing when installing @neutralinojs/lib or updating type definitions via npx @neutralinojs/neu update.
Workaround required is:
// @ts-expect-error
await window.setTitle(`App version v${NL_APPVERSION}`);

Hey @shalithasuranga,
I looked into this issue and I found something
    if(!options["version"].is_null()) {
        jsSnippet += "var NL_APPVERSION='" + options["version"].get<string>() + "';";
    }

this is the part of the code that get's the app version but the if statement will not run if the value for version is null.
I think we can solve this by providing a default value if the value of version is null.
do you agree with this approach should i work on it further?

Hello @Sadaf-A, thanks for checking, this is done like that because version is optional for Neutralinojs apps. Btw, seems like we just need to add global TypeScript definition for this one by modifying global.d.ts?

Hello @Sadaf-A, thanks for checking, this is done like that because version is optional for Neutralinojs apps. Btw, seems like we just need to add global TypeScript definition for this one by modifying global.d.ts?

Yes, as I mentioned on discord we can make the NL_APPVERSION optional like this
NL_APPVERSION?: string;

would that be the right approach?

Mm.. Let's try to keep the implementation as it is and update the Ts file :)

Hello @Sadaf-A , seems like what we declare in global.d.ts are not embedded in index.d.ts. Could you please check whether entering window.NL_APPVERSION works file on VS Code intellisense? Thank you :)

Hello @Sadaf-A , seems like what we declare in global.d.ts are not embedded in index.d.ts. Could you please check whether entering window.NL_APPVERSION works file on VS Code intellisense? Thank you :)

Hey @shalithasuranga,
    window.NL_APPID
    window.NL_CVERSION
    window.NL_MODE
    window.NL_OS
    window.NL_PORT
    window.NL_VERSION
    ```
these are the properties being detected in the intellisense. should I look into why other properties are not being detected?

Hello, this was fixed in 5.0.1. Tested with the following scenarios:

If app developers use /// <reference path="./neutralino.d.ts" /> in a TypeScript file, they won't see intellisence issues for the following statements:

NL_VERSION;
NL_APPVERSION;
// ... all globals

If app developers use the Neutralinojs client from NPM, they won't see issues for the following statements:

import { os } from '@neutralinojs/lib';
window.NL_VERSION;
window.NL_APPVERSION;
// .. all globals

@Sadaf-A Could you please check wether it also works for you? Thanks :tada:

Hello, this was fixed in 5.0.1. Tested with the following scenarios:
1. If app developers use `/// <reference path="./neutralino.d.ts" />` in a TypeScript file, they won't see intellisence issues for the following statements:

NL_VERSION;
NL_APPVERSION;
// ... all globals

2. If app developers use the Neutralinojs client from NPM, they won't see issues for the following statements:

import { os } from '@neutralinojs/lib';
window.NL_VERSION;
window.NL_APPVERSION;
// .. all globals

@Sadaf-A Could you please check wether it also works for you? Thanks 🎉

Hey, I'm not getting any TypeScript errors and intellisense is not showing NL_APPVERSION

@shalithasuranga I know this is closed, wouldn't think this merits a new issue...but if I add more globals via config (for example, NL_ENVIRONMENT or something), how would I add definitions for that so the compiler doesn't complain about it?

@shalithasuranga I know this is closed, wouldn't think this merits a new issue...but if I add more globals via config (for example, NL_ENVIRONMENT or something), how would I add typescript definitions for that so the typescript compiler doesn't complain about it?

I thinj the same way as we have added NL_APPVERSION in this PR

@shalithasuranga I know this is closed, wouldn't think this merits a new issue...but if I add more globals via config (for example, NL_ENVIRONMENT or something), how would I add typescript definitions for that so the typescript compiler doesn't complain about it?

I thinj the same way as we have added NL_APPVERSION in this PR

I ended up following this commit: https://github.com/neutralinojs/neutralino.js/commit/1f3604e24db9dc385fe3096d899d8d94d0596a77, and I added it in a globals.ts file I had been using for the app.


Create Map between FHIR message type and SOR-EDI representation

The task is to create a map between the MedCom FHIR message and the code registered in SOR-EDI. As example a map between FDIS91XX and  CareCommunicationMessage version 1.0

Has been created


Promote 1.21.2+k3s1 to stable

Rule of thumb:

new patch, wait ~4 days before mark as stable.
new minor version, wait about 1-2 weeks before we mark the new minor release as stable

Steps:

[x] First, review new GH issues. Ensure there are no serious issues reported by the community or any SUSE staff.
[x] Second, review the rancher-users slack and ensure there are no serious issues reported there.
[x] If no obvious/serious issues exist, modify channel.yaml stable section such that it points to the new release.
[x] Get PR reviewed and then merge the PR. Confirm that the endpoint gets updated (takes up to about 15 mins post CI all being green).
[ ] Ask QA to test so they can ensure that when you install via install.sh it pulls down the correct stable version.

Validated tarball and rpm install
OS: Ubuntu, Centos7, Centos8


Rebase in progress indicator

https://github.com/olivierverdier/zsh-git-prompt/issues/122
Straightforward enough, I guess also do a merging indicator too.

Bash-git-prompt uses something like:
18c78e2|REBASE 1/3|✚1]

Prototype created, only for python for now.


[improve][broker][PIP-149]make getReplicatorDispatchRate method async

Motivation
See #14365
Verifying this change
Make sure that the change passes the CI checks.
Does this pull request potentially affect one of the following parts:
If yes was chosen, please highlight the changes

Dependencies (does it add or upgrade a dependency): (no)
The public API: (no)
The schema: (no)
The default values of configurations: (no)
The wire protocol: (no)
The rest endpoints: (no)
The admin cli options: (no)
Anything that affects deployment: (no)

Documentation
Check the box below and label this PR (if you have committer privilege).
Need to update docs?

[x] doc-not-needed

/pulsarbot run-failure-checks

/pulsarbot run-failure-checks


feat(Text): Add more common utility variants / states as props GM-248

Overview

Props
There are a few commonly used style mixins that either are hard to migrate (like the fade-truncation scss mixin) or require custom overrides (in CSS-in-JS - no-select + font-smoothing + screen-readeronly):
This adds a few of these common patterns and inclusions as variant / state props - to remove common cases where we repeatedly extended a component with identical styles in different locations.
New Box Props

no-select: Toggle user selection / touch callouts

Example
<Box no-select>
  <img src="x" />
</Box>

New Text Props

center: Toggle text-align: center
block: Toggle display: block;
screenreader: Render text outside of document flow, while still visible for screenreaders.
smooth: Enable font-smoothing.
truncate:  Common truncation patterns for limiting text to a single line.

ellipsis: Standard  text-overflow: elipsis
fade: Fades the end of the line into the the background of the page.  (From the monolith mixin: @include fade-truncation)

Example
// Truncation
<Text truncate="ellipsis" />
<Text truncate="fade">{superLongString}</Truncate>

// Screenreader
<Text screenreader>My Hiddent Text!</Text>

// Font Smoothing
<Text smooth />

Typings
Certain properties like  WebkitFontSmoothing we not configurable through variance CSS functions.  So the typings have been expanded here to include the necessary Vendor properties.  This change applies to css variant and states.

PR Checklist

[ ] Related to designs:
[x] Related to JIRA ticket: https://codecademy.atlassian.net/browse/GM-248
[x] I have run this code to verify it works
[ ] This PR includes unit tests for the code change

📬Published Alpha Packages:@codecademy/gamut-icons@5.1.3-alpha.33ae17.0@codecademy/gamut-illustrations@0.17.5-alpha.33ae17.0@codecademy/gamut-kit@0.4.23-alpha.33ae17.0@codecademy/gamut-labs@19.5.1-alpha.33ae17.0@codecademy/gamut-patterns@0.3.5-alpha.33ae17.0@codecademy/gamut-styles@11.2.1-alpha.33ae17.0@codecademy/gamut-tests@2.4.8-alpha.33ae17.0@codecademy/gamut@34.6.1-alpha.33ae17.0@codecademy/konamimojisplosion@0.2.39-alpha.33ae17.0@codecademy/markdown-overrides@0.5.51-alpha.33ae17.0@codecademy/styleguide@41.7.6-alpha.33ae17.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://60c76c57e8a42bbbcc136e57--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-icons@5.2.1-alpha.127bfd.0@codecademy/gamut-illustrations@0.17.6-alpha.127bfd.0@codecademy/gamut-kit@0.4.30-alpha.127bfd.0@codecademy/gamut-labs@19.8.1-alpha.127bfd.0@codecademy/gamut-patterns@0.4.1-alpha.127bfd.0@codecademy/gamut-styles@11.3.1-alpha.127bfd.0@codecademy/gamut-tests@2.4.9-alpha.127bfd.0@codecademy/gamut@34.7.2-alpha.127bfd.0@codecademy/konamimojisplosion@0.2.40-alpha.127bfd.0@codecademy/markdown-overrides@0.5.52-alpha.127bfd.0@codecademy/styleguide@41.9.1-alpha.127bfd.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://60ca682e3c05ef00cacdabae--gamut-preview.netlify.appDeploy Logs


(vlc) Updating resets VLC configuration settings

https://chocolatey.org/packages/vlc
Updates <IP_ADDRESS> and 2.2.6 wiped VLC config settings - this has also happened before, but I can't personally verify with which releases. Can user data be preserved?
VLC settings are stored in ~AppData\Roaming\vlc.

Considering the AppData is user specific, I'm against doing anything inside the script to backup and restore the settings, especially considering the script is running as administrator (some circumstances it may run under the system account, in which case the found AppData folder wouldn't be the correct one).
So unless the installer actually supports an argument to passed to keep the configuration, this likely won't be fixed.

@AdmiringWorm What I don't understand is why the settings are cleared when installed with Chocolatey, but not when installing with the official installer?

It shouldn't, we basically just call the installer using the necessary arguments for silent installs.
We don't even touch the AppData directory at all in the script, so unless the developers of VLC automatically removes the configuration when running a silent install then this sounds like a bug in the installer.

We use the same installer. It might be the upstream bug.

@07416 I'm pretty much of the opinion that this is an upstream bug (as @majkinetor said), possibly because silent arguments are used (I've seen this happen a few times for other softwares, which used bad defaults), and should be fixed there and not in the package.
As such I'm closing this Issue, feel free to re-opening it if you disagree though.


File is a static class

Type of issue
Other (describe below)
Description
Error in documentation, near as I can tell: The text says

... extend the data processing capabilities of the System.IO.File and System.IO.Stream objects, ...

The problem is that System.IO.File is a static class, so calling it an "object", or suggesting that it can be the target of an extension method, is incorrect.
Page URL
https://learn.microsoft.com/en-us/dotnet/csharp/programming-guide/classes-and-structs/extension-methods#extending-predefined-types
Content source URL
https://github.com/dotnet/docs/blob/main/docs/csharp/programming-guide/classes-and-structs/extension-methods.md
Document Version Independent Id
50c65125-3700-c713-ede0-2204e09eb307
Article author
@BillWagner
Metadata

ID: 2a5e0b56-4d67-b219-46fa-d837045e6612
Product: dotnet-csharp
Technology: csharp-fundamentals

Thanks @petterh Your observation is correct.
We should remove the reference to the File class.
I've added this to our backlog, and I've added the help wanted tag in case anyone wants to submit a PR before we get to it.


[BUILD] cleanup unused jenkins files and scripts

Is your feature request related to a problem? Please describe.
Directory:
https://github.com/NVIDIA/spark-rapids/tree/branch-0.4/jenkins
has jenkins files, docker files, and scripts. I believe many of those are not used anymore (like the databricks ones). We should remove any files from here that aren't used.
I'm not sure if one of the Docker files was an example for users. We do already have one for k8s here: https://github.com/NVIDIA/spark-rapids/tree/branch-0.4/docs/get-started. If we want it visible to users perhaps we should document and put it in similar location or create a separate directory for it.

@GaryShen2008

@GaryShen2008

@tgravescs
1, The below 2 Dockerfile are useless, we’ll remove them from GitHub/spark-rapids repo.
Dockerfile.integration.ubuntu16
Dockerfile.ubuntu16
2, The other Dockerfiles/Jenkinsfiles/scripts are essential or spark-rapids prem-merge and integration jobs.
3, We put these Dockerfiles/Jenkinsfiles/scripts in GitHub, so dev team can update them conveniently, instead of making them visible to external users.
--- For example, if someone needs to install a python module for IT, he can change the Dockerfile-blossom.integration.centos7 directly on the GitHub repo.
So after clean useless Dockerfiles in item 1, need we to move the whole Jenkins dir, or some subdirs[databricks] or files out of spark-rapids Github int internal Gitlab repo?

@tgravescs
1, The below 2 Dockerfile are useless, we’ll remove them from GitHub/spark-rapids repo.
Dockerfile.integration.ubuntu16
Dockerfile.ubuntu16
2, The other Dockerfiles/Jenkinsfiles/scripts are essential or spark-rapids prem-merge and integration jobs.
3, We put these Dockerfiles/Jenkinsfiles/scripts in GitHub, so dev team can update them conveniently, instead of making them visible to external users.
--- For example, if someone needs to install a python module for IT, he can change the Dockerfile-blossom.integration.centos7 directly on the GitHub repo.
So after clean useless Dockerfiles in item 1, need we to move the whole Jenkins dir, or some subdirs[databricks] or files out of spark-rapids Github int internal Gitlab repo?

ok, I guess Jenkinsfile.databricks301nightly is still used here, but its inconsistent with the other nightly now so I would rather see it moved out to be consistent.  I do see some of the other scripts are used from here, is there a particular reason to leave them in this repo vs move out to where Jenkins files are?
The problem is that some of the scripts are here and some are in the other repo so Dev doesn't know where to update all the builds, which is confusing. For instance we wanted to add build argument, the scripts updated nightly but not all the IT tests.  It would be have a single scripts for running the IT tests for instance that others could call with parameters if needed.   I don't think it matters which repo this in as much as keeping it consistent and documenting where things are at.

ok, I guess Jenkinsfile.databricks301nightly is still used here, but its inconsistent with the other nightly now so I would rather see it moved out to be consistent.  I do see some of the other scripts are used from here, is there a particular reason to leave them in this repo vs move out to where Jenkins files are?
The problem is that some of the scripts are here and some are in the other repo so Dev doesn't know where to update all the builds, which is confusing. For instance we wanted to add build argument, the scripts updated nightly but not all the IT tests.  It would be have a single scripts for running the IT tests for instance that others could call with parameters if needed.   I don't think it matters which repo this in as much as keeping it consistent and documenting where things are at.

ok, I guess Jenkinsfile.databricks301nightly is still used here, but its inconsistent with the other nightly now so I would rather see it moved out to be consistent.  I do see some of the other scripts are used from here.
The problem is that some of the scripts are here and some are in the other repo so Dev doesn't know where to update all the builds, which is confusing. For instance we wanted to add build argument, the scripts updated nightly but not all the IT tests.  It would be have a single scripts for running the IT tests for instance that others could call with parameters if needed.   I don't think it matters which repo this in as much as keeping it consistent and documenting where things are at.

Hi, I totally agree that put all those stuff in 2 different places could confuse people, but it is required by blossom and security team to move at least jenkins files to internal.
And yes, we may just keep jenkins files and scripts at the same place if possible,

To leave the integration test scripts at github/spark-rapids since developers could still benefit from them (I am assuming most of developers could care about test scripts but not jenkins files) and we put all CSP-related scripts to the internal repo
or to move ALL CICD-related stuff back to internal (people should remember to update test scripts in internal repo after they submit code change to the external repo),
IMHO, I would prefer the first one, any thoughts?

ok, I guess Jenkinsfile.databricks301nightly is still used here, but its inconsistent with the other nightly now so I would rather see it moved out to be consistent.  I do see some of the other scripts are used from here.
The problem is that some of the scripts are here and some are in the other repo so Dev doesn't know where to update all the builds, which is confusing. For instance we wanted to add build argument, the scripts updated nightly but not all the IT tests.  It would be have a single scripts for running the IT tests for instance that others could call with parameters if needed.   I don't think it matters which repo this in as much as keeping it consistent and documenting where things are at.

Hi, I totally agree that put all those stuff in 2 different places could confuse people, but it is required by blossom and security team to move at least jenkins files to internal.
And yes, we may just keep jenkins files and scripts at the same place if possible,

To leave the integration test scripts at github/spark-rapids since developers could still benefit from them (I am assuming most of developers could care about test scripts but not jenkins files) and we put all CSP-related scripts to the internal repo
or to move ALL CICD-related stuff back to internal (people should remember to update test scripts in internal repo after they submit code change to the external repo),
IMHO, I would prefer the first one, any thoughts?

the problem comes in the definition of we put all CSP related scripts...  What does that include, right now that includes scripts for running tests, which I don't want them split.  If we can split and commonize them such that the scripts for running tests are in spark-rapids and other Jenkinsfiles and CSP setup scripts are in private, I'm good with that.

the problem comes in the definition of we put all CSP related scripts...  What does that include, right now that includes scripts for running tests, which I don't want them split.  If we can split and commonize them such that the scripts for running tests are in spark-rapids and other Jenkinsfiles and CSP setup scripts are in private, I'm good with that.

also here, it seems we aren't running on the tests. I'm fixing the integration test jar to include the resources directory, but all the test environment should be using the run_pyspark_from_build.sh where its appropriate.
We may need to modify it to support running against other clusters like EMR

note that this should go alone with https://github.com/NVIDIA/spark-rapids/issues/1640 and basically ideally we commonize to use the same script to run all the tests. see the comments in that issue.

MR on GitLab: https://gitlab-master.nvidia.com/nvspark/blossom-jenkins/-/merge_requests/123

Todo

To move DB IT scripts into GitHub
Make it common to create cluster, setup pytests ... with python API

@NvTimLiu are you still working on commonizing some of the scripts for blossom?

@tgravescs I've done all the common scripts for all the cloud environments.
@jlowe Need we to add some tag similar to the  [-Dpytest.TEST_TYPE="nightly"](https://github.com/NVIDIA/spark-rapids/blob/branch-0.5/jenkins/spark-nightly-build.sh#L25)  for all the integration tests pipelines?

otherwise I suppose we are good to close this issue.

Need we to add some tag similar to the -Dpytest.TEST_TYPE="nightly" for all the integration tests pipelines?

Yes, all of the CI pipelines should be specifying something for TEST_TYPE so we don't allow certain tests that are a bit tricky to setup to keep skipping, otherwise we risk inadvertently never running those tests before we release.

close per PR2059 merged


chore(main): release 5.0.0

:robot: I have created a release beep boop
5.0.0 (2024-02-06)
⚠ BREAKING CHANGES

Combine exporter and endpoint (#293)

Bug Fixes

Combine exporter and endpoint (#293) (e7c0c6c)

This PR was generated with Release Please. See documentation.

close to test release-please workflow


check_archived_root does not have a mutable equivalent

It's currently sound to call check_archived_root and then archived_value_mut, but this would help eliminate an unnecessary unsafe block.

Fixed by 27df1bf182e3fd8f695714fc44a6e9aae320d3af


fix(rmt): set rx filter resolution_hz to rmt_sclk (IDFGH-13418)

Fixes assignment of filter_clock_resolution_hz to rx_channel->base.resolution_hz, which is the clock frequency after the fractional divider RMT_SCLK_DIV (also referenced as rmt_sclk in the TRM), instead of the previously used value from group->resolution_hz, which represents the full undivided frequency from clock selected by RMT_SCLK_SEL (APB, RC_FAST or XTAL).
Before this fix, this mixup resulted in an error signal_range_min_ns too big being returned by the rmt_receive function due to this check failing no matter the resolution_hz used in the configuration of the channel.

NOTE: I couldn't verify this fix's correctness for the ESP32-C5 and ESP32-P4, since I could not find their TRMs (I suppose those are not available yet).

Hi @LonerDan I'm afraid this is not a mistake. The RX channel can filter out glitches in the signals, so it needs a high frequency base clock. The hardware uses the group's clock as the filter's tick clock.

Hi @suda-morris
I was going off of the TRM (more specifically for the ESP32-S3) figure 37-1, where the label rmt_sclk is put after the fractional divider and the same label is used with the arrow going into the filter block. Also, the text about the RX filter mentions that

The filter samples input signals continuously, and detects the signals which remain unchanged for a continuous RMT_RX_FILTER_THRES_CHm rmt_sclk cycles as valid, otherwise, the signals will be detected as invalid.

If that's not really the case, It could be beneficial to clarify this in the documentation.

@suda-morris Ah, I see. Thank you for the correction. I looked closer into the source code and found the part where the rmt_sclk divider is always configured to no division:
https://github.com/espressif/esp-idf/blob/8e4454b285ad39881c5bf3f440d8be617a2f18a8/components/esp_driver_rmt/src/rmt_common.c#L210-L214
In my opinion, this should be configurable since the resolution of the filter is application-specific. In my case, the signal has shortest pulse length around 12 us and there is a lot of short spikes/noise mixed in, so I would like to filter out anything less than say 8 us to limit the number of data rmt rx channel generates for me to process.
I am willing to try to implement this, I just wanted to ask where you think is the best place to incorporate this parameter into? Not sure if rmt_rx_channel_config_t is the correct place since the sclk divider is common to the group.


test canary from fork

test canary from fork to see why it doesnt work in tools
Change Type (required)
Indicate the type of change your pull request is:

[ ] patch
[ ] minor
[ ] major

Does your PR have any documentation updates?

[ ] Updated docs
[ ] No Update needed
[ ] Unable to update docs

/canary


Finalize and test process for standing up decentralized network

Expected Behavior
Process for standing up and running a Celo network in a decentralized way should be finalized and coordinated with external validators.

Done


[PCA] don't normalize input data before applying PCA or provide a parameter to turn off normalization

The default behavior of d4p will normalize input data before applying PCA in both batch and distributed mode.
In sklearn and pyspark, PCA will not normalize data by default. The default behavior of d4p is different from sklearn & pyspark and there is no option in the API to change this behavior.
pca-spmd-pyspark-sklearn.tar.gz

And in distributed mode, d4p will not normalize as batch mode. (maybe another bug). So need to examine all the behaviors and update them in a consistent way.


Fixed "Using with TypeScript"-Link

Just a small correction of a link in README.md

Coverage remained the same at 97.643% when pulling 88d494a74a45c74efb58cf628e79255f386f70df on meisterlampe:master into 8b87e3de3e9595027a4f50b28a11b29d0ebb8386 on kogosoftwarellc:master.

Coverage remained the same at 97.643% when pulling 88d494a74a45c74efb58cf628e79255f386f70df on meisterlampe:master into 8b87e3de3e9595027a4f50b28a11b29d0ebb8386 on kogosoftwarellc:master.

thanks @meisterlampe !


Test(Cypress): Add tests for the landing page and the navbar

Checklist:

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] All the files I changed are in the same world language, for example: only English changes, or only Chinese changes, etc.

Related to #38611

This PR adds e2e tests for the Navbar in fCC's site. Let me know if anything needs to be changed!

@ojeytonwilliams should the sample.env FORUM_LOCATION be 'https://forum.localhost' instead of 'https://localhost/forum'

I'm not 100% sure. But, since we're using the subdomain in reality, https://forum.localhost makes sense to me.

Sure, we could leave it as is and address it in another pr where we would pull these values from a config file.

We should use the real URLs instead of localhost. The idea is to test if the link is valid and reachable.

We should use the real URLs instead of localhost. The idea is to test if the link is valid and reachable.

That's a good idea cause that way we can see if the button is working correctly by clicking it and seeing if it takes us to the right site.

We'll need to update the environment variables to change those links I believe. Any thoughts?

Yes, and we should. I don't see any point keeping them to localhost.

Changing the # Application paths in sample.env should suffice.

Done! Updated the env variables and added a few more tests for the landing page.

This LGTM. @ojeytonwilliams, it would be great if we could get this in so I could adjust the nav tests in the custom nav PR.

Thanks for taking the time to put this together, @Twaha-Rahman, the extra code coverage is so important - this will be a massive help.


Changes

Made some changes. These include stopping your IDE from harassing you, and using the Bukkit Scheduler instead of BukkitRunnables.
i have not tested this, but i dont see a reason for this not to work

why did you replace all bukkit runnables with the scheduler
isn't scheduling deprecated?

i think it's better to use bukkitrunnables since most scheduleSyncDelayedTask methods are deprecated, and even more might become so in the future.


EntityContainer crashes if the entity type is null, making the entire dataset unreadable

I was unable to open a buggy OData stream. It turned out that the problem was that an entity type was null. I was able to fix by
editing class EntityContainer to explicitly trap for this, e.g.
        for entity_set in self._service.schema.entity_sets:
            if not isinstance(entity_set.entity_type, NullType):
                self._entity_sets[entity_set.name] = EntitySetProxy(self._service, entity_set)
            else:
                logging.error("Warning: I am trying to ignore error in:",entity_set)

I dont know if this is the optimal approach, but it allowed me to continue parsing the rest of the data.

Hi @Tunneller would you pls pass a sample snippet of the $metadata of the entity type and the http response leading to this problem? Or ideally, as a new test that would reproduce the problem (check existing unit test suite and create your own mock response).

I am closing this due inactivity, if problem still persist, please reopen with additional data, as requested above.


Fix the parser for use with v18 of autogtp (#40)

The EVENT parser is pretty fragile in the sense that if the initial text sent by autogtp changes at all, it can fail in a mysterious way. I updated the regexp to accept the new text and still be backward compatible with old versions, but it could fail again in the future
if the autogtp start text changes again.
I bumped the version to 1.2.0 and also updated to gradle 6.2 and a newer version of the fatjar plugin. You can now build with java 11. A new l release will need to be created and posted.
I tested against the v16 and v17 versions of the leela-zero distribution.

/review @fsparv


Reset button should also clear Bluetooth bonds

To ensure only a single Bluetooth connection can be established to the blaster, the advertising packets should only be started if the device is not already paired.
Along with this, pressing the reset button should also clear the Bluetooth controller bonds as well as the app data.
Perhaps this also means the reset should be done outside the app control?

This will be enabled by #42

The Bluefruit API doesn’t seem to have any methods exposed to indicate whether the device has been paired.
The connected callback must be setup to check whether the handle has been paired and set a setting on the FRAM chip which can be used for the advertisement decision.


Add selection of Pypi mirrors in the GUI

📝 Description of the feature
In some countries (e.g. China), access to Pypi is a bit slow, especially when downloading huge modules like Tensorflow and other ML modules.
An efficient solution is to setup a mirror  site for Pypi. For example in China you can use https://pypi.tuna.tsinghua.edu.cn.
It would be nice to have a way to set-up a mirror, and even nicer to have a pre-polulated list.
Also, some users might use a private Pypi server in their company, which uses the same concept
💡 Steps for implementing the feature

[ ] Find a list of the best pypi mirror servers
[ ] Update the UI to be able to select one and add your own
[ ] Update the backend to launch pip with this server

🔗 Useful links and references
To use pip with a mirror:
pip install -i https://pypi.tuna.tsinghua.edu.cn/simple some-package
To set as default:
pip config set global.index-url https://pypi.tuna.tsinghua.edu.cn/simple
Documentation of 2 mirrors (both in China):

https://mirror.tuna.tsinghua.edu.cn/help/pypi/
https://mirrors.sustech.edu.cn/help/pypi.html#_2-configure-index-url

I have test the url format of the mirror in PRC.
PYTHON_URLS = {
    'PRC': {
        'python_mirror': 'https://registry.npmmirror.com/-/binary/python/{selected_version}/python-{selected_version}-amd64.exe',
        'pypi_mirror': "https://mirrors.tuna.tsinghua.edu.cn/pypi/web/json/{pkg_name}"
    },
    'Defaults': {
        'python_mirror': "https://www.python.org/ftp/python/{selected_version}/python-{selected_version}-amd64.exe",
        'pypi_mirror': "https://pypi.python.org/pypi/{pkg_name}/json",
    }
}

Hi @hh-wu @jfthuong - this might be a great addition. Currently my support to this project is limited but I will try to do my best to implement it over the next couple of weeks. I do find value in your proposed issue - I understand that some countries might have restrictions and issues with the main server bandwidth. Feel free to open a PR in the meantime to implement this. Otherwise I will try to support it in a couple of weeks. =)


Cache key character with Symfony cache

Hello,
I'm using botman in a composer project, and I had to include Symfony cache in order to do conversations.
There is a small issue with the conversation identifier : https://github.com/mpociot/botman/blob/master/src/Mpociot/BotMan/Message.php#L69
Error: Cache key "conversation:279775023-279775023" contains reserved characters {}()/\@:
Maybe replace : with another character.
Thank you

Fixed in dev-master


Not retina optimized

the page is not rendering properly on retina devices

This was an issue on Messenger.com - it is working for me right now


Status.JSON Reports Incorrect Player Data when Crossplay Enabled

I am using the STATUS_HTTP_HTDOCS status.json file to report if there are connected users. This always reports a value of 0 when running in crossplay mode.
Maybe I need to configure something or it's an issue on my end but this doesn't seem to report the right information about users when operating in crossplay mode.

I suspect that since the player number reporting is based on data returned from the Steam API, this function does not work with crossplay enabled, since it is not using the Steam networking library anymore? Just a guess, but seems plausible.

-crossplay uses Playfab, and from my brief look it doesn't look like there is a similar way to look up the player count info without auth there.


Update google-api-services-sheets to v4-rev20220927-2.0.0

Updates com.google.apis:google-api-services-sheets from v4-rev20220620-2.0.0 to v4-rev20220927-2.0.0.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.google.apis", artifactId = "google-api-services-sheets" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "@monthly" },
  dependency = { groupId = "com.google.apis", artifactId = "google-api-services-sheets" }
}]

labels: library-update, commit-count:1

Superseded by #55.


Method to Replicate Results from Huggingface Spaces

I tried Huggingface Spaces and got wonderful results. Thank you. How can I replicate these results locally?

HF demo locally: https://github.com/PKU-YuanGroup/MoE-LLaVA?tab=readme-ov-file#-demo
CLI inference: https://github.com/PKU-YuanGroup/MoE-LLaVA?tab=readme-ov-file#cli-inference
Our API to use: https://github.com/PKU-YuanGroup/MoE-LLaVA?tab=readme-ov-file#-api

@LinB203 Thank you for your response. There are several models shown, but which model is used in Huggingface Spaces?

@LinB203 Thank you for your response. There are several models shown, but which model is used in Huggingface Spaces?

Phi2.

@LinB203 Thank you. I appreciate your response to my detailed questions.

You're welcome. Btw, if you like our project, please give us a star ⭐ on GitHub for latest update.

@LinB203 Thank you. I appreciate your response to my detailed questions.


cmd/link: incorrect relocations during external linking on linux/386 with musl libc

See https://github.com/golang/go/issues/14476#issuecomment-187554053
Tentatively labeled Go1.6.1, but it probably doesn't meet the line.

Is this a tracking bug for the other being in 1.6.1 or a separate bug? If it's a separate bug maybe it can wait until 1.6.3.

this is a separate bug: external linking doesn't work on linux/386 with
musl libc.
I don't know what's wrong yet....

Given there's no fix, it seems like this should be at least investigated for Go 1.7. It won't be in Go 1.6.3 (which probably won't happen).

This would be nice to fix but it doesn't seem to be happening.  Dropping to 1.8.

The relevant part of Minux's comment from #14476:
An externally linked hello world program couldn't even start:
(gdb) r
Starting program: /root/go.git/src/hello
Program received signal SIGSEGV, Segmentation fault.
0xb7f98656 in do_relocs (dso=dso@entry=0xb7ffd660 , rel=0x80000910,
rel_size=145784, stride=2)
at src/ldso/dynlink.c:407
407 src/ldso/dynlink.c: No such file or directory.
(gdb) bt
#0  0xb7f98656 in do_relocs (dso=dso@entry=0xb7ffd660 ,
rel=0x80000910, rel_size=145784,
stride=2) at src/ldso/dynlink.c:407
#1  0xb7f98ea7 in reloc_all (p=p@entry=0xb7ffd660 ) at
src/ldso/dynlink.c:1115
#2  0xb7f9a5e3 in __dls3 (sp=0xbffffd60) at src/ldso/dynlink.c:1598
#3  0xb7f99e5c in __dls2 (base=0xb7f7b000 "\177ELF\001\001\001",
sp=0xbffffd60)
at src/ldso/dynlink.c:1389
#4  0xb7f97ec5 in _dlstart_c (sp=0xbffffd60, dynv=0xb7ffcef8) at
src/ldso/dlstart.c:147
#5  0xb7f97dbb in _dlstart () from /lib/ld-musl-i386.so.1
Something is wrong with the relocations.

This doesn't seem like it is affecting many people. Moving to Go1.8Maybe.

Is this still a problem?


Feedback for “Importing Video / Audio into TimeBolt”

Hello,
I have paid for your lifetime version. I am using it with a 2020 Mac M1 running Big Sur. I am trying to import an MXF video file from a Canon C300 Mark II, but nothing happens. According to all the tutorial videos, the conversion is supposed to start automatically upon importing the file into your software. Can you help me quickly, please? Otherwise, I will have to use another software.
Thank you.

@renaudderepentigny Are you talking about Nextra?


How to change server url

I cloned your project and tested it on https://voiceping-router-test.netlify.app. using ws://<IP_ADDRESS>:3000. Now how to change this address?
When I typed http:// in the browser it shows:
Welcome to VoicePing Router 1.0.0
Is it correct?
It's written on your github: Welcome to VoicePing Router 2.3.3-LITE

@Timon2023 to change the address you can logout on https://voiceping-router-test.netlify.app/ and input the new vp-router URL in the settings.
Sorry for the confusion on version number. The 1.0.0 is correct.

How to change the server address on the server side?


Traefik timeout annotations do not work

Do you want to request a feature or report a bug?
Bug

What did you do?

I installed Traefik v2.2 on the kubernetes cluster with Helm.
I am using rancher's k3s kubernetes distribution which by default uses Traefik v1.7.19. In order to install version 2.2 instead of 1.7.19 I replaced original /var/lib/rancher/k3s/server/manifests/traefik.yaml with the one including code below and started k3s with: sudo k3s server -no-deploy=traefik (as adviced here https://github.com/k3s-io/k3s/issues/1141)

kind: HelmChart
metadata:
  name: traefik
  namespace: kube-system
spec:
  chart: traefik
  repo: https://containous.github.io/traefik-helm-chart
  set:
    image.tag: "2.2"

The pod started with traefik 2.2
pod description:
Name:         traefik-57f74867d5-z4kjd
Namespace:    kube-system
Priority:     0
Node:         xxxxxxxx
Start Time:   Tue, 12 Jan 2021 15:36:49 +0100
Labels:       app.kubernetes.io/instance=traefik
              app.kubernetes.io/managed-by=Helm
              app.kubernetes.io/name=traefik
              helm.sh/chart=traefik-9.1.1
              pod-template-hash=57f74867d5
Annotations:  <none>
Status:       Running
IP:           xxxx
IPs:
  IP:           xxxx
Controlled By:  ReplicaSet/traefik-57f74867d5
Containers:
  traefik:
    Container ID:  containerd://9b720d5cdb41d8a38211f3d10b7ccc3a491011021f86ec17680c97128fa665b8
    Image:         traefik:2.2
    Image ID:      docker.io/library/traefik@sha256:ea0aa8832bfd08369166baecd40b35fc58979df8f5dc5182e4e63ee6adbe66db
    Ports:         9000/TCP, 8000/TCP, 8443/TCP
    Host Ports:    0/TCP, 0/TCP, 0/TCP
    Args:
      --global.checknewversion
      --global.sendanonymoususage
      --entryPoints.traefik.address=:9000/tcp
      --entryPoints.web.address=:8000/tcp
      --entryPoints.websecure.address=:8443/tcp
      --api.dashboard=true
      --ping=true
      --providers.kubernetescrd
      --providers.kubernetesingress
    State:          Running
      Started:      Tue, 12 Jan 2021 15:36:55 +0100
    Ready:          True
    Restart Count:  0
    Liveness:       http-get http://:9000/ping delay=10s timeout=2s period=10s #success=1 #failure=3
    Readiness:      http-get http://:9000/ping delay=10s timeout=2s period=10s #success=1 #failure=1
    Environment:    <none>
    Mounts:
      /data from data (rw)
      /tmp from tmp (rw)
      /var/run/secrets/kubernetes.io/serviceaccount from traefik-token-fzkqx (ro)
Conditions:
  Type              Status
  Initialized       True
  Ready             True
  ContainersReady   True
  PodScheduled      True
Volumes:
  data:
    Type:       EmptyDir (a temporary directory that shares a pod's lifetime)
    Medium:
    SizeLimit:  <unset>
  tmp:
    Type:       EmptyDir (a temporary directory that shares a pod's lifetime)
    Medium:
    SizeLimit:  <unset>
  traefik-token-fzkqx:
    Type:        Secret (a volume populated by a Secret)
    SecretName:  traefik-token-fzkqx
    Optional:    false
QoS Class:       BestEffort
Node-Selectors:  <none>
Tolerations:     node.kubernetes.io/not-ready:NoExecute op=Exists for 300s
                 node.kubernetes.io/unreachable:NoExecute op=Exists for 300s
Events:          <none>

I deployed on the cluster application with a single endpoint which role is to sleep given amount of seconds (after receiving http request) and respond with status 200 after that.

The application is reachable from outside the cluster through the Ingress.
The timeouts (for application) are defined through Traefik annotations:
transport.respondingTimeouts.readTimeout
transport.respondingTimeouts.writeTimeout
transport.respondingTimeouts.idleTimeout
as described here:
https://doc.traefik.io/traefik/routing/entrypoints/#transport
Ingress description (timeouts are equal to 7 seconds):
Name:             test-timeout
Namespace:        app-qplwhubfno7n
Address:          xxx.xxx.xxx.xxx
Default backend:  default-http-backend:80 (<error: endpoints "default-http-backend" not found>)
Rules:
  Host                                     Path  Backends
  ----                                     ----  --------
  test-timeout-app-qplwhubfno7n.localhost
                                           /   test-timeout:8080 ()
Annotations:                               hsts: false
                                           hsts-include-subdomains: false
                                           transport.respondingTimeouts.idleTimeout: 7
                                           transport.respondingTimeouts.readTimeout: 7
                                           transport.respondingTimeouts.writeTimeout: 7
Events:                                    <none>

I performed http request telling my app to sleep 10 seconds (timeouts are 7 seconds) and return status 200 after that.

What did you expect to see?
My application should return error 504 (gateway timeout) due to timeout (as time of execution of program (10 sec) exceeded 7 seconds).
What did you see instead?
My application returned status 200 after waiting 10 seconds (i.e. Traefik did not returned timeout after 7 seconds).
Output of traefik version: (What version of Traefik are you using?)
Version:      2.2.11
Codename:     chevrotin
Go version:   go1.14.8
Built:        2020-09-07T14:12:48Z
OS/Arch:      linux/amd64

Hello @Kamzs,
thanks for your interest in the project.
You are mixing up two different subjects here. The configuration you have found is configuration that is applied to an entrypoint. Either through a configuration file or through CLI arguments. These settings then take effect for every connection through that EP.
What you've tried to is to attach that setting to an ingress object through annotations, but thats plain not how this works :)
As this is a question, I'll close that issue.

Hello @Kamzs,
thanks for your interest in the project.
You are mixing up two different subjects here. The configuration you have found is configuration that is applied to an entrypoint. Either through a configuration file or through CLI arguments. These settings then take effect for every connection through that EP.
What you've tried to is to attach that setting to an ingress object through annotations, but thats plain not how this works :)
As this is a question, I'll close that issue.


fix: safer process.env access

We use @substrate/txwrapper-core for some tx operations in the Talisman wallet.
As of https://github.com/paritytech/txwrapper-core/pull/298, this package now throws Uncaught ReferenceError: process is not defined because process.env is injected by node, but not by browsers.
Since the environment variables used by this package are all optional, I figured a neat solution might be to just safeguard the process.env access.

Awesome, thanks so much for the contribution. I also didn't realize anyone was leveraging txwrapper-core for the browser. This is great to know and any future changes I'll make sure to keep that in mind so we stay compatible!

You just need to run yarn lint:fix in order to fix the linter.

Will do! It’s late here, will push a fix in the morning :)
Thanks!

Fixed!


Implement new Pubsub to BIgQuery option

Hello!

Vote on this issue by adding a 👍 reaction
If you want to implement this feature, comment to let us know (we'll work with you on design, scheduling, etc.)

Issue details

I would like to suggest the implementation of the new PubSub to BigQuery option as described here: https://cloud.google.com/blog/products/data-analytics/pub-sub-launches-direct-path-to-bigquery-for-streaming-analytics
Affected area/feature

gcp.pubsub.Subscription

Note that the example here doesn't work:
https://www.pulumi.com/registry/packages/gcp/api-docs/pubsub/subscription/
What you ended up getting is a Pull subscription (incorrect type)

Actually the example works.


DBZ-4386 Upgrade impsort-maven-plugin to 1.6.2

https://issues.redhat.com/browse/DBZ-4386

A bit depressing that we need to maintain those versions in two places... :/


HtmlEditor errors on StateHasChanged

I want to add some fancy features on my page, but it requires StateHasChanged. However, the RadzenHtmlEditor component keeps erroring out with "Cannot set property 'innerHTML' of null
Error: Microsoft.JSInterop.JSException: Cannot set property 'innerHTML' of null
TypeError: Cannot set property 'innerHTML' of null
at Object.innerHTML (https://localhost:44388/script/Radzen.Blazor.js:950:21)
at https://localhost:44388/_framework/blazor.server.js:1:70368
at new Promise ()
at e.beginInvokeJSFromDotNet (https://localhost:44388/_framework/blazor.server.js:1:70334)
at https://localhost:44388/_framework/blazor.server.js:1:26441
at Array.forEach ()
at e.invokeClientMethod (https://localhost:44388/_framework/blazor.server.js:1:26411)
at e.processIncomingData (https://localhost:44388/_framework/blazor.server.js:1:24222)
at e.connection.onreceive (https://localhost:44388/_framework/blazor.server.js:1:17307)
at WebSocket.i.onmessage (https://localhost:44388/_framework/blazor.server.js:1:46783)
at Microsoft.JSInterop.JSRuntime.InvokeAsync[TValue](Int64 targetInstanceId, String identifier, Object[] args)
at Microsoft.JSInterop.JSRuntimeExtensions.InvokeVoidAsync(IJSRuntime jsRuntime, String identifier, Object[] args)
at Radzen.Blazor.RadzenHtmlEditor.OnAfterRenderAsync(Boolean firstRender)
at Microsoft.AspNetCore.Components.RenderTree.Renderer.GetErrorHandledTask(Task taskToHandle)

We will need a reproduction of this issue.

I could not reproduce the issue, but I ended up fixing the issue by removing protected override async Task OnParametersSetAsync() in my project


Polygon -> Quad

The class allows only 4 vertices, which is a quad.

I fear confusion with BakedQuad

A person confusing this with BakedQuad won't stay confused for very long.
It's only used in Entity models as part of a Box so I guess we could call it ModelQuad or BoxQuad but IMO those names aren't as good.  Quad is more consistent with the other names in this package.


issues about Mix-Precision training

Thanks for your great work!!
Because of the limitation of GPU, I used pytorch native amp for training.
However, I found some problems with numerical stability while computing giou, NaN or inf.
Is there any good suggestion to solve this problem

Hi~
Does reducing learning rate help?


Allowed getStatements to accept strings

Addresses #128

The tests are fine tests to include, but it probably makes more sense to start adding tests around the _queryStatementsRequestCfg method itself. Could start out with 4, one each for the id versus object forms for those two parameters. That will exercise directly the code that was changed and prevent future regression, and should be simple since that method doesn't have an async form. Granted it is "private" but it is as good as a pass-through so could stand to be tested directly.


Load MOD file example; works right away!

Closes #10

Point of clarification: it's good that this works and that we have an example, but this is not a "point process"... a point process in NEURON occurs at a point (we call h.mod a density mechanism because it's spread out over a region at a certain density).
Here's a Point Process defined in a mod file: http://modeldb.science/getModelFile?model=2730&file=bulbNet/nmdanet.mod
In particular that defines a thing called NMDA, but one does not insert_mechanism("NMDA") (that should be an error)... instead one treats this like an IClamp, and defines, say syn = n.NMDA(axon(0.5)). If it's working, then you should be able to read and set syn.Alpha (default value is 0.072), syn.Beta (default is 0.0066), and syn.e (default is 45).

I think I processed all your comments, but I had to rebase this branch onto branch 48-iterating-over-... in order for the point process example to work as intended (with syn = n.NMDA(axon(0.5))), so ideally that PR is merged first.


Enable string substitution for LabVIEW versions including architecture

The current mechanism for substituting the LabVIEW version used to build a given repo only substitutes the year version. This was fine when we only built for 32-bit LabVIEW, but with the addition of 64-bit support, we need to be able to build nipkg files that install to the correct location based on architecture. This is required in order to build packages for the custom device build tools and LabVIEW scripting APIs and examples.

Closed by #121


Each child in an array or iterator should have a unique key prop

{this.state.complaints.map(comp =>
              <ListItem
                avatar={comp.avatar}
                caption={comp.caption}
                legend={comp.legend}
                key={comp.key}
                comp={comp}
              />
          )}

giving this warning
Warning: Each child in an array or iterator should have a unique "key" prop

You need to provide unique key to each react elements, refer react docs,
this is not a react-toolbox issue
On Wed, Feb 15, 2017, 9:01 PM sudhir<EMAIL_ADDRESS>wrote:

{this.state.complaints.map(comp =>
)}
giving this warning
Warning: Each child in an array or iterator should have a unique "key" prop
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/react-toolbox/react-toolbox/issues/1250, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AAnUA83hBcHyJpyOK7dPaIEKzu0BTcerks5rcxpBgaJpZM4MB22u
.

Yes i understand. ListItem should be taking care of that some how. Might be by taking unique param.

Well passing the key props to your ListItem component is the proper way to go, frameworks should not be taking care of that.
If you want to eliminate passing key altogether, perhaps you could create a wrapper Fn will take care of passing unique key to your components.
Something like this
const renderList = (counter = 0) => (obj) => (
<ListItem
  caption={obj['caption']}
  to={obj['to']}
  activeClassName={GactiveClassName}
  className={GclassName}
  key={`ListItem_${counter++}`}
/>
)
export default renderList()

Usage
const DemoComponent = () => (
<List>
{ array.map((item) => renderList(item)) }
</List>
)

Caution: Generate component in compoentWillMount and not in the render, else on every render call new elements will be created, becuase a new key will be allocated each time.

well i'm passing the key to  which is unique. Please look at top code. Still i observe the error.

Well can you tell me the version of react-toolbox you're using
I'm using "react-toolbox": "^2.0.0-beta.6" and have not faced this issue like you one you're mentioning.
Are you sure this is the only component you're rendering as a list, if yes can you send me the link to your code.

Looks like it was a local issue. Thanks @devarsh !


/jobs top

Hi! When I do /jobs top Fisherman or type any other job name it kicks me out of the server and says

Using Spigot 1.12.2 and Jobs 4.5.3

Tried with other job names but still not working

It throws it out of the game because it puts a long word in the GUI somewhere. That is, in the configuration there is a long word, please send the jobConfig.yml file.

Should be ok with next update