Add label type for SingleR refs to annotated SCE

Based on https://github.com/AlexsLemonade/scpca-docs/pull/166#discussion_r1367258141, this PR adds back in tracking the reference label type for SingleR. When building the SingleR model se specify the label type to use. That label type is then stored in the metadata of the model object (see https://github.com/AlexsLemonade/scpca-nf/blob/934df294bbfe2a01d8ffdb94ba9f5d6b432a547d/bin/train_SingleR.R#L134).
So here I take the label name that's stored in the reference model object and store it in the results object in classify_SingleR.R. Then in add_celltypes_to_sce.R, I get the label type from the results object and add it to the metadata of the annotated SCE object. I also used it to check whether or not to map the annotations to the ontology label.

One other change that you could make is here - check for the "label.ont" reference label in the processed metadata instead of the df in the model object.

I need to make some changes to get the correct labels on plots since right now they are still showing the ontology ids, so I think we should address this as we make updates to each of the sections, starting with making sure the tables are using the right labels in https://github.com/AlexsLemonade/scpca-nf/issues/512.


Simplify 'encode_action_body'.

This PR simplifies the logic of encode_action_body and fixes a bug with
the encoding of tagged custom types (which map to C# value types) in Slice1 mode (quite niche).

For example, take this snippet:
class Hello {
    tag(1) d1: CustomType?
}

We currently generate this:
encoder.StartSlice(SliceTypeId);
if (this.D1 is MyCustomType d1_)
{
    encoder.EncodeTagged(1, TagFormat.FSize, d1_, (ref SliceEncoder encoder, MyCustomType value) => CustomTypeSliceEncoderExtensions.EncodeNullableCustomType(ref encoder, value));
}
encoder.EndSlice(true);

Notably, this calls EncodeNullableCustomType which is incorrect. We know here that value is non-null.
Even the type of the lambda is MyCustomType, with no ? suffix.
This PR fixes this to call the non-nullable function:
encoder.StartSlice(SliceTypeId);
if (this.D1 is MyCustomType d1_)
{
    encoder.EncodeTagged(1, TagFormat.FSize, d1_, (ref SliceEncoder encoder, MyCustomType value) => CustomTypeSliceEncoderExtensions.EncodeCustomType(ref encoder, value));
}
encoder.EndSlice(true);

Part of this PR got spun off into #3916.
So, to save hassle on merge conflicts, I'm going to wait to merge this PR until that one, since this PR will be much simpler to fix the conflicts on.


[css] Lab colors and other CSS Color Module Level 4 features

See the CSS Color Module Level 4 specification.
While the specification is still a draft and browsers don't support these features yet, there already exists a PostCSS plugin that (partially) implements them, so it's actually already possible to use at least a part of them.
.example {
	background-color: lab(33 43 -47);
	box-shadow: 0 0 20px lch(54 107 41 / 10%);
}

Would be nice to get previews for those colors in VS Code like it's shown for rgb/rgba/hsl colors.

In general, we wait until a spec is adopted by most browsers. Following a spec in progress is very time consuming and can lead to user confusion as well.

Support looks pretty good

https://developer.mozilla.org/en-US/docs/Web/CSS/color_value

If anyone is interested in adding support for more color notations, the code is in
https://github.com/microsoft/vscode-css-languageservice/blob/master/src/languageFacts/colors.ts

Since the color pickers in Chrome DevTools, Edge DevTools, and Safari DevTools all produce the comma-free syntax nowadays, it seems extra important for VS Code to support it. Does this affect prioritization for this request at all?


Minor fix on start.sh script

It seems like IoT Core no longer accepts device ID's that start with a number, only letters.
Solved it by adding a prefix to the DEVICE_UUID on line 43.
gcloud iot devices create 'balena-'$RESIN_DEVICE_UUID \          --project=$GOOGLE_IOT_PROJECT \          --region=$GOOGLE_IOT_REGION \          --registry=$GOOGLE_IOT_REGISTRY \          --public-key path=rsa-cert.pem,type=rs256

@tomasmigone Thanks for the input!
You also need to add the prefix to the NodeJS Script (index.js) in order to successfully publish messages for this device.
https://github.com/balena-io-projects/google-iot/blob/d7ff718e3a454a67dde1420217908cabe6777775/index.js#L11
New line should read:
const deviceId = 'balena-' + process.env.RESIN_DEVICE_UUID;

Addressed in PR #4


Better metrics (forwarders)

This is an improvement to an existing feature.
We started capturing and reporting metrics a while back, and since then we added support for graphite and prometheus systems.
However most of the metrics we capture focus only on the ingestion side of Pitchfork. We don't have metrics that measure the performance or health of each of the individual forwarders.

Example of one of the forwarders where we don't have a metrics counter and should:
https://github.com/HotelsDotCom/pitchfork/blob/master/src/main/java/com/hotels/service/tracing/zipkintohaystack/forwarders/zipkin/http/ZipkinForwarder.java#L79


Add ensemble support to result_summary PDF

Adding ensemble support to summary result_summary PDF:
Here's what it looks like:

I've also added new unit testing to protect the description and table (based off a checkpoint) and was required to update the existing unit test checkpoints (switching multi-model from brute to quick search).

Make sure you run a CI

Yes, I've run a CI and it's passing


feat: multi languages support #266

Id still merge this now and move the missing stuff to an issue.

Id still merge this now and move the missing stuff to an issue.

Every other raised problems can be fixed fast.

To not being too annoying i've approved this PR, but found something that in my opinion would be better to fix before merging.


Fixing small typo

There is a small typo in "using", where the "u" is missing.

Hi @chuwik, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no contribution license agreement is required at this point. Real humans will now evaluate your PR.

TTYL, AZPRBOT;

@chuwik Thank you for this contribution!


Tracing panic hook

Fixes #3468.
This does change how the panics look, and most importantly, make sure they are not interleaved with other messages. Adds a GET /v1/panic endpoint for panic testing.

Old panic looked like (via GET /v1/panic) with prev and next line:
2023-01-27T16:28:14.440275Z  INFO GET /v1/always_panic 404 Not Found
thread 'mgmt request worker' panicked at 'unconditional panic for testing panic hook integration', pageserver/src/http/routes.rs:898:9
stack backtrace:
   0: rust_begin_unwind
--snip--
  64: tokio::runtime::blocking::pool::Spawner::spawn_thread::{{closure}}
             at /home/joonas/.cargo/registry/src/github.com-1ecc6299db9ec823/tokio-1.24.2/src/runtime/blocking/pool.rs:469:13
note: Some details are omitted, run with `RUST_BACKTRACE=full` for a verbose backtrace.
2023-01-27T16:28:26.059654Z  INFO GET /v1/panic 204 No Content

New panic looks like (with anyhow backtrace above):
             at ./misc/../sysdeps/unix/sysv/linux/x86_64/clone3.S:81
2023-01-27T16:56:48.866015Z ERROR panic{thread=mgmt request worker panic.file="pageserver/src/http/routes.rs" panic.line=898 panic.column=9}: unconditional panic for testing panic hook integration

Stack backtrace:
   0: pageserver::tracing_panic_hook
             at /home/joonas/src/neon/neon-main/pageserver/src/bin/pageserver.rs:503:21
   1: <alloc::boxed::Box<F,A> as core::ops::function::Fn<Args>>::call
             at /rustc/90743e7298aca107ddaa0c202a4d3604e29bfeb6/library/alloc/src/boxed.rs:2001:9
   2: std::panicking::rust_panic_with_hook
             at /rustc/90743e7298aca107ddaa0c202a4d3604e29bfeb6/library/std/src/panicking.rs:692:13
   3: std::panicking::begin_panic_handler::{{closure}}
             at /rustc/90743e7298aca107ddaa0c202a4d3604e29bfeb6/library/std/src/panicking.rs:577:13
   4: std::sys_common::backtrace::__rust_end_short_backtrace
             at /rustc/90743e7298aca107ddaa0c202a4d3604e29bfeb6/library/std/src/sys_common/backtrace.rs:137:18
   5: rust_begin_unwind
--snip--
  82: clone3
             at ./misc/../sysdeps/unix/sysv/linux/x86_64/clone3.S:81

2023-01-27T16:56:48.970587Z  INFO GET /v1/panic 204 No Content

Above also shows the new frames.
So there's an extra newline after. I wonder if that's a problem for message splitting, we could try to make it go away by depending directly on backtrace-rs instead of std (like anyhow) but I would prefer not doing that.
Could @lassizci comment on if this will create an issue for the message splitting pipeline, or point me to the implementation?

Could @lassizci comment on if this will create an issue for the message splitting pipeline, or point me to the implementation?

@koivunej Shouldn't affect, the matcher seems to be matching a timestamp and then collecting the lines until the next timestamp.

Postponing this.


NTR: generation of T-type calcium current

We have the existing term: generation of L-type calcium current ; GO:0061577 and I'm looking to add the corresponding 'T-type calcium current' term.
[Term]
id: GO:0061577
name: generation of L-type calcium current
namespace: biological_process
def: "A process in which a calcium ion is transported from one side of a membrane to the other by means of an L-type calcium channel." [GOC:dph]
is_a: GO:0070588 ! calcium ion transmembrane transport
relationship: has_part GO:0008331 ! high voltage-gated calcium channel activity
created_by: dph
creation_date: 2013-11-15T16:20:27Z
I'm annotating a paper showing that Wnt signaling components regulate T-type calcium current (PMID:20371816) by increasing expression of the voltage-gated calcium channel (Cav3.1). Could I add the corresponding T-type current term (with the view to add in regulation terms)?
GO:NEW
name: generation of T-type calcium current
namespace: biological_process
def: "A process in which a calcium ion is transported from one side of a membrane to the other by means of a T-type calcium channel." [GOC:PARL, GOC:bf, PMID:20371816]
is_a: GO:0070588 ! calcium ion transmembrane transport
relationship: has_part GO:0008332 ! low voltage-gated calcium channel activity
Just checking because it's a bit of an unusual term.
Thanks!

I don't see why not.  @ukemi, since you added the L-type term, could you have a quick look? Thanks.
@rebeccafoulger - are you editing using OE or will you request using TG Free Form?

Looks ok to me.

What't the difference between a calcium current and calcium transport across the membrane?

According to what's in the ontology now, the generation of the current is a type of calcium ion transmembrane transport.

Doesn't all transport of ions across a membrane constitute a current flowing across it? Also not sure what 'generation of' adds.

'generation of' was there to make it consistent with a the old action potential term, but that has since changed so in this case it probably doesn't add anything.

Becky - thoughts?  EXP annotations to the L-type term are all from BHF with one from RGD.  Is it just me or could/should these terms be:
calcium ion transmembrane transport via high voltage-gated calcium channel
and
calcium ion transmembrane transport via low voltage-gated calcium channel
?

@rebeccafoulger - can you comment please?

Hi all- knew this one was going to cause trouble!
Sorry for the delay Tanya! Your suggestion @tberardini of the 'via' terms fits with the has_part relationships and sounds much less like an assay-readout. I was going to say the process terms seem a bit redundant if we already have the low- and high- channel activities, but I see all existing annotations are to 'regulation' terms so it's handy to have the process terms in place.
So I'm fine with your suggestion Tanya! (with generation names as synonyms). @RLovering and @rachhuntley - any thoughts? One annotation is one of your miRNA ones Rachael.

Yes, I looked at the paper I curated and I think that would be fine.

Added:
[Term]
+id: GO:0090676
+name: calcium ion transmembrane transport via low voltage-gated calcium channel
+namespace: biological_process
+def: "A process in which a calcium ion is transported from one side of a membrane to the other by means of a low voltage-gated calcium channel." [GOC:bf]
+synonym: "generation of T-type calcium current" RELATED []
+is_a: GO:0070588 ! calcium ion transmembrane transport
+relationship: has_part GO:0008332 ! low voltage-gated calcium channel activity
+created_by: tanyaberardini
+creation_date: 2016-03-23T10:15:23Z
+
also changed names as described for the L-type terms (core BP and regulation children)


Failed to compile with uuid

Not sure am I doing wrong, but I got an error when I was trying to use uuid::Uuid in my schema.
use uuid::Uuid;

#[derive(GraphQLObject)]
struct User {
    ...
    photo: Option<Uuid>
}

error[E0277]: the trait bound `uuid::Uuid: juniper::GraphQLType` is not satisfied
  --> src/qlschema.rs:10:10
   |
10 | #[derive(GraphQLObject)]
   |          ^^^^^^^^^^^^^ the trait `juniper::GraphQLType` is not implemented for `uuid::Uuid`
   |
   = note: required because of the requirements on the impl of `juniper::GraphQLType` for `std::option::Option<uuid::Uuid>`

I've read the source, and as the guide mentioned, there's no any explicity configuration needed, right?
The installed version of juniper is 0.9.2, with uuid 0.6.5, have tried both nightly and stable toolchain.

Are you using master or the version from crates.io?

I just confirmed it fails with the release but works with master. I'm trying to get a release out but don't yet have permissions to push (https://github.com/graphql-rust/juniper/issues/210).
In the meantime you should do:
[dependencies]
juniper = { git = "https://github.com/graphql-rust/juniper.git"}

I will use the master first, thanks!


Unable to Encrypt

Please see error info:
Stream clear = pbe.GetDataStream(sKey);
PgpObjectFactory plainFact = new PgpObjectFactory(clear);
PgpObject message = plainFact.NextPgpObject();
if (message is PgpCompressedData)
{
PgpCompressedData cData = (PgpCompressedData)message;
PgpObjectFactory pgpFact = new PgpObjectFactory(cData.GetDataStream());
message = pgpFact.NextPgpObject();
}
The logic then returns False and falls into this reason:
string.Format(“Message is not a simple encrypted file. THe type unknown for file {0}”, outputFileName),

This also affects the decryption.


Tesseract couldn't load any languages

I have tesseract installed on my iMac, when I try to use the module I'm getting:
    Error opening data file /usr/local/share/eng.traineddata
    Please make sure the TESSDATA_PREFIX environment variable is set to your "tessdata" directory.
    Failed loading language 'eng'
    Tesseract couldn't load any languages!

The TESSDATA_PREFIX is set-up in my environment using:
    export TESSDATA_PREFIX=/PDFgone/tesseract/tessdata

If I type in 'export' after getting this error I can see the environment variable is present, can anyone help with this problem?
This is what the options look like:
    tesseract.process("./output/EON-0.tiff", {l:"eng"
					   ,psm:6
  				        ,binary:"/usr/local/bin/tesseract"}
				,function(err, text) {
    ...
    });

I've fixed this, the problem was that tesseract was installed in two places, the languages were not installed in the correct location.  I've moved the language files to /usr/local/share/testdata


url-match regexes should'nt contain ^ and $

Because starting and ending chars are already forced here and there.

Thx!

@dereuromark do you think you could tag a new version with this PR ? I'd like to have it without cloning dev-master thru composer. Thanks !

Done :)

So quick ! Thank you :)

Hi @dereuromark, sorry to bother you again, but the 0.2.1 tag doesn't seem to have been propagated to packagist.

Thx again, I fixed the auto-update. All good now.

:+1:


Min Browser

add Min Browser to manifestes.
Microsoft Reviewers: Open in CodeFlow

Once Windows Defender Notifications<EMAIL_ADDRESS>confirms the false positive, please remove the label "Needs: author feedback", and we will restart the pull request.


Nginx SSL stapling fails when service starts before network is online

Describe the bug
The Nginx systemd service may start before the network is online. When using SSL stapling (from recommendedTlsSettings), Nginx fails to resolve the certificate's OCSP responder, and gives up on stapling.
To Reproduce

Configure a NixOS system with nginx.recommendedTlsSettings = true;, a manually specified SSL certificate, and ensure that no vhosts have enableACME = true;
Reboot the system
Nginx starts but never sends OCSP responses (see log below)
(if you later manually restart Nginx, stapling does work)

Expected behavior
Nginx starts and does send OCSP responses.
Additional context

Output of `journalctl -b --unit network.target --unit network-online.target --unit nginx --unit dhcpcd`
(note the nginx: [warn] lines, and the order between DHCP and Nginx)
-- Journal begins at Sun 2021-01-31 17:49:52 UTC, ends at Fri 2021-02-05 08:47:21 UTC. --
Feb 05 05:47:37 myhost systemd[1]: Starting DHCP Client...
Feb 05 05:47:37 myhost dhcpcd[616]: dev: loaded udev
Feb 05 05:47:37 myhost dhcpcd[616]: DUID [redacted]
Feb 05 05:47:37 myhost dhcpcd[616]: ens3: IAID [redacted]
Feb 05 05:47:37 myhost dhcpcd[616]: ens3: adding address [redacted]
Feb 05 05:47:37 myhost systemd[1]: Reached target Network.
Feb 05 05:47:37 myhost systemd[1]: Starting Nginx Web Server...
Feb 05 05:47:37 myhost dhcpcd[616]: ens3: soliciting an IPv6 router
Feb 05 05:47:38 myhost nginx-pre-start[735]: nginx: [warn] "ssl_stapling" ignored, host not found in OCSP responder "r3.o.lencr.org" in the certificate "/run/secrets/myhost.com.crt"
Feb 05 05:47:38 myhost nginx-pre-start[735]: nginx: the configuration file /nix/store/391rcjhbxl4ncw24hpi7fkbjr2wja081-nginx.conf syntax is ok
Feb 05 05:47:38 myhost nginx-pre-start[735]: nginx: configuration file /nix/store/391rcjhbxl4ncw24hpi7fkbjr2wja081-nginx.conf test is successful
Feb 05 05:47:38 myhost systemd[1]: Started Nginx Web Server.
Feb 05 05:47:38 myhost nginx[746]: nginx: [warn] "ssl_stapling" ignored, host not found in OCSP responder "r3.o.lencr.org" in the certificate "/run/secrets/myhost.com.crt"
Feb 05 05:47:38 myhost dhcpcd[616]: ens3: rebinding lease of [redacted]
Feb 05 05:47:38 myhost dhcpcd[616]: ens3: probing address [redacted]/23
Feb 05 05:47:43 myhost dhcpcd[616]: ens3: leased [redacted] for 86400 seconds
Feb 05 05:47:43 myhost dhcpcd[616]: ens3: adding route to [redacted]/23
Feb 05 05:47:43 myhost dhcpcd[616]: ens3: adding default route via [redacted]
Feb 05 05:47:43 myhost dhcpcd[616]: ens3: adding host route to [redacted] via [redacted]
Feb 05 05:47:43 myhost dhcpcd[833]: Failed to reload-or-try-restart ntpd.service: Unit ntpd.service not found.
Feb 05 05:47:43 myhost dhcpcd[833]: Failed to reload-or-try-restart openntpd.service: Unit openntpd.service not found.
Feb 05 05:47:43 myhost dhcpcd[833]: Failed to reload-or-try-restart chronyd.service: Unit chronyd.service not found.
Feb 05 05:47:43 myhost dhcpcd[616]: forked to background, child pid 834
Feb 05 05:47:43 myhost systemd[1]: Started DHCP Client.
Feb 05 05:47:43 myhost systemd[1]: Reached target Network is Online.
Feb 05 05:47:50 myhost dhcpcd[834]: ens3: no IPv6 Routers available

When using ACME certificates, the nginx service is reloaded when acme-finished-${certName}.target is reached, shortly after boot, so this issue is less obvious.
I have worked around this issue by adding these lines, but I don't know if this is the most appropriate solution:
systemd.services.nginx.after = [ "network-online.target" ];
systemd.services.nginx.wants = [ "network-online.target" ];

Notify maintainers
cc (from git blame) @arianvp @m1cr0man
Metadata

Output of `nix-shell -p nix-info --run "nix-info -m"`
 - system: `"x86_64-linux"`
 - host os: `Linux 5.4.92, NixOS, 21.03.20210125.891f607 (Okapi)`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.4pre20201205_a5d85d0`
 - channels(root): `"nixos-21.03pre250162.34ad166a830"`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixos`

Maintainer information:
# a list of nixpkgs attributes affected by the problem
attribute:
# a list of nixos modules affected by the problem
module:
- services/web-servers/nginx

Just systemd.services.nginx.after = [ "network-online.target" ]; should be enough.  network-online.target should not actively be pulled in; only be ordered upon (https://www.freedesktop.org/wiki/Software/systemd/NetworkTarget/)
However network-online.target does not give very strong guarantees either. It guarantees some interface has a default route but for example it does not guarantee DNS is configured...
I am a bit afraid to enable it as it slows down bootup.
Having services that break permantently due to external dependencies not being reachable temporarily is just ... bad design. I wonder why nginx is broken like this? It sounds like a really weird choice of them to stop trying at all just because the responder isn't reachable once

I edited the above message to remove an incorrect statement about after and wants (for people reading through email)

Hmm, I just found this Nginx ticket which was closed as wontfix four years ago :confused:
Both of those reasons make sense to not put nginx.service after network-online.target. Perhaps we could put nginx-config-reload.service after network-online.target and nss-lookup.target? It's already implicitly after these when using ACME. I don't know how this would best interact with services.nginx.enableReload, though.

I am a bit afraid to enable it as it slows down bootup. This will cause nginx to take up to 90 seconds to start up if for example you start it on a server without internet...

such a server cannot use stapling, so this is a bit moot


First attempt to create release notes generator.

Heavily based on the commit message generator.

Thanks @WimPauwelsBerthylis , this is good


Internal error in error handling logic

Got a TypeError inside internal error handling code of the library today.

webpack://handler/node_modules/gocardless-nodejs/errors.js:29
const { statusCode, body: { error: { type, errors }, }, } = response;
^
TypeError: Cannot read property 'type' of undefined
at Function.buildFromResponse (/var/task/dist/lambda.js:11700:46)
-> webpack://handler/node_modules/gocardless-nodejs/errors.js:29:45
at Api.request (/var/task/dist/lambda.js:3748:55)
-> webpack://handler/node_modules/gocardless-nodejs/api/api.js:79:1
at runMicrotasks ()
at processTicksAndRejections (internal/process/task_queues.js:95:5)
at async PaymentService.create (/var/task/dist/lambda.js:13225:26)
-> webpack://handler/node_modules/gocardless-nodejs/services/paymentService.js:21:1

Same here, got an error this morning in production. This is bad
TypeError: Cannot read property 'type' of undefined
    at Function.buildFromResponse (/var/task/node_modules/src/errors.ts:63:18)
    at Api.request (/var/task/node_modules/src/api/api.ts:121:47)
    at processTicksAndRejections (internal/process/task_queues.js:95:5)
    at RedirectFlowService.create (/var/task/node_modules/src/services/redirectFlowService.ts:92:22)

Hi. Is there any update to this issue as we've put up with production errors for a very long time which could possibly hide real errors?


mns service support

Hi,
Any plans to add support for Message Service (mns) ?

Hi,
Any plans to add support for Message Service (mns) ?

Hi, the products we support are increasing, if you want to know products we support, you can run aliyun help to print the products we support.
Thank you for your attention to AliCloud CLI Project!

Why is this closed? Over 2 years and mns is still not supported.


Add testing for ubuntu18.04

This adds testing for Ubuntu18.04 so that we verify compatibility with Ubuntu 18.04.
Maintainer's note: I am a bit on the fence whether it should be necessary to test both Ubuntu 16.04 and Ubuntu 18.04, since it adds quite a bit of overhead to the docker setup, including port management. For CI it isn't a problem because we run each test in a separate CircleCI instance, but when running locally, you no longer can do docker-compose up to run all the tests (due to the port conflict).
More generally, we should refactor the tests so that we can test multiple OSes, such as CentOS and Amazon Linux, similar to https://github.com/gruntwork-io/gruntwork-installer

Merging this in!


K8SPXC-1037: fix reconcile in ginkgo tests

https://jira.percona.com/browse/K8SPXC-1037
DESCRIPTION
Problem:
Reconcile in ginkgo tests was failing due to OperatorPod function
Cause:
OperatorPod tries to get operator namespace from /var/run/secrets/kubernetes.io/serviceaccount/namespace
Solution:
We should avoid the use of this function. Operator should call it only when .spec.initImage is not set. Ginkgo tests should use default cr with already set .spec.initImage field.
CHECKLIST
Jira

[x] Is the Jira ticket created and referenced properly?
[x] Does the Jira ticket have the proper statuses for documentation (Needs Doc) and QA (Needs QA)?
[x] Does the Jira ticket link to the proper milestone (Fix Version field)?

Tests

[x] Is an E2E test/test case added for the new feature/change?
[ ] Are unit tests added where appropriate?
[ ] Are OpenShift compare files changed for E2E tests (compare/*-oc.yml)?

Config/Logging/Testability

[ ] Are all needed new/changed options added to default YAML files?
[ ] Are the manifests (crd/bundle) regenerated if needed?
[ ] Did we add proper logging messages for operator actions?
[ ] Did we ensure compatibility with the previous version or cluster upgrade process?
[ ] Does the change support oldest and newest supported PXC version?
[ ] Does the change support oldest and newest supported Kubernetes version?

Test name
Status

affinity-8-0
passed

auto-tuning-8-0
passed

cross-site-8-0
passed

demand-backup-cloud-8-0
failure

demand-backup-encrypted-with-tls-8-0
passed

demand-backup-8-0
passed

haproxy-5-7
passed

haproxy-8-0
failure

ignore-labels-annotations-8-0
passed

init-deploy-5-7
passed

init-deploy-8-0
passed

limits-8-0
passed

monitoring-2-0-8-0
passed

one-pod-5-7
passed

one-pod-8-0
passed

pitr-8-0
passed

proxy-protocol-8-0
passed

proxysql-sidecar-res-limits-8-0
passed

recreate-8-0
passed

restore-to-encrypted-cluster-8-0
passed

scaling-proxysql-8-0
passed

scaling-8-0
passed

scheduled-backup-5-7
passed

scheduled-backup-8-0
passed

security-context-8-0
passed

smart-update1-8-0
passed

smart-update2-8-0
passed

storage-8-0
passed

tls-issue-cert-manager-ref-8-0
passed

tls-issue-cert-manager-8-0
passed

tls-issue-self-8-0
passed

upgrade-consistency-8-0
passed

upgrade-haproxy-8-0
passed

upgrade-proxysql-8-0
passed

users-5-7
passed

users-8-0
passed

validation-hook-8-0
passed

We run 37 out of 37

commit: https://github.com/percona/percona-xtradb-cluster-operator/pull/1410/commits/0334448aa4a91f2e6d2b7bab5c4504b487ac4507
image: perconalab/percona-xtradb-cluster-operator:PR-1410-0334448a

Test name
Status

affinity-8-0
passed

auto-tuning-8-0
passed

cross-site-8-0
passed

demand-backup-cloud-8-0
passed

demand-backup-encrypted-with-tls-8-0
passed

demand-backup-8-0
passed

haproxy-5-7
passed

haproxy-8-0
passed

ignore-labels-annotations-8-0
passed

init-deploy-5-7
passed

init-deploy-8-0
passed

limits-8-0
passed

monitoring-2-0-8-0
passed

one-pod-5-7
passed

one-pod-8-0
passed

pitr-8-0
passed

proxy-protocol-8-0
passed

proxysql-sidecar-res-limits-8-0
passed

recreate-8-0
passed

restore-to-encrypted-cluster-8-0
passed

scaling-proxysql-8-0
passed

scaling-8-0
passed

scheduled-backup-5-7
passed

scheduled-backup-8-0
passed

security-context-8-0
passed

smart-update1-8-0
passed

smart-update2-8-0
passed

storage-8-0
passed

tls-issue-cert-manager-ref-8-0
passed

tls-issue-cert-manager-8-0
passed

tls-issue-self-8-0
passed

upgrade-consistency-8-0
passed

upgrade-haproxy-8-0
passed

upgrade-proxysql-8-0
passed

users-5-7
passed

users-8-0
passed

validation-hook-8-0
passed

We run 37 out of 37

commit: https://github.com/percona/percona-xtradb-cluster-operator/pull/1410/commits/0334448aa4a91f2e6d2b7bab5c4504b487ac4507
image: perconalab/percona-xtradb-cluster-operator:PR-1410-0334448a


Jump

Hello, sorry for my bad English,
It seems that mobs can't jump anymore since 1f0969e6d374bf089728b1995e97a97b06b2f111...
Short video of the bug

Hrm, I cannot reproduce this error, sheep follow me and can jump uphill ok...  Will double check everything to be sure.

It seems that the bug doesn't occur if the sheep didn't stop walking before jumping, so maybe giving it (or making sure it has) a "horizontal speed" in addition to a "vertical speed" when jumping would fix it?

Update to latest API, mobs moving uphill and following players is a lot better.

Thanks a lot, now they can jump!
Is it intentional that they can jump over fences?

In fixing the walking uphill they can now walk over fences also, so 2-high fences are needed to stop them.


wxGTK32: repro build and disable rpath

Description of changes
2 minor config changes
the --disable-rpath one was found necessary in building from their current master
(due to retaining the build directory in rpath, and patchPhase not being able to remove it)
and will presumably become necessary on the next release
and the --enable-repro-build just sounds like the right thing for us :)
a fair few things depend on this, so there were a few breakages in there
but i don't think any of them are due to these changes
1 package marked as broken and skipped:
springLobby

13 packages failed to build:
erlang-ls klipper-firmware klipper-flash meerk40t meerk40t.dist python312Packages.humblewx python312Packages.humblewx.dist python312Packages.kicad python312Packages.pcbnew-transition python312Packages.pcbnew-transition.dist python312Packages.soapysdr-with-plugins python312Packages.wxpython ut1999

(that was done on top of my kicad-8 PR that includes fixing the kicad-unstable build)
i'm mostly PRing this because it seems like the --disable-rpath could become necessary
and i don't want the fix to get lost
Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

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


Aligning rows within ColumnSets by default, unless element height explicitly = "auto"

This is an optional addition on top of #1683 that also aligns elements inside ColumnSets by default, unless users explicitly set the height property to "auto", which seems like a much better default behavior.

Hey @Identifier, please accept my apologies that this has existed over a year :)
Is this still something you want us to look at it? I see a // TODO in the code near the Height property, so any additional information would be helpful

Hi @matthidinger, I believe this was already integrated at some point, so I'll just close this now.


@fiberplane/ui: Simplify theme & provider

Description

[x] No more styled-components theme values; only CSS variables
[x] Greatly simplify GlobalStyle & only ship what's shared between Explorer & Studio

[x] Box element spacing values
[x] Media queries

[x] Add variable fallback values
[x] Documentation

Checklist

[x] The changes have been tested to be backwards compatible.
- [ ] The OpenAPI schema and generated client have been updated.
- [ ] New models module has been added to api generator xtask
- [ ] New types/fields are well-documented and annotated.
[x] The CHANGELOG is updated.

Please link any related PRs in other repositories that depend on this:

@arendjr possibly, but we still want to serve an optional theme provider that we can use across our projects. The generated output from Figma doesn't contain any media query & sizing logic, but we follow the same approach across projects.
We definitely will get rid of styled-components as a dependency in this project however, so that'll clean up the /theme directory nicely. 😊


Add favicons for progress web apps

Description
Currently there are no favicons in the application. There should be default favicons for all the PWA platforms (Android, iOS, Windows, etc.)

Implemented in v0.5.0


Verify 74x on RHEL9

Description
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Fixes # (issue)
Type of change

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

How Has This Been Tested?
https://jenkins.confluent.io/job/cp-ansible-on-demand/1182/ (archive-plain-rhel 🟢)
https://jenkins.confluent.io/job/cp-ansible-on-demand/1181/ (plain-rhel 🟢 )
Test Configuration:
Checklist:

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

duplicate PR, closing this one


[BUG] error 443

Acknowledgement of preliminary instructions

[X] I have read the preliminary instructions, and I am certain that my problem has not already been addressed.
[X] I have thoroughly looked through the available Wiki articles and could not find a solution to my problem.
[X] I am using the latest version of Bloxstrap.
[X] I did not answer truthfully to all the above checkboxes.

What problem did you encounter?
when ever i try to load up my bloxstrap it works and i get into roblox but not with the textures and other things whenever i load roblox it says error 443. And i have to hit close then it takes me to roblox but in the error 443 it says i need install or update roblox in order to countine to play with bloxstrap. Ive already tried both of these u have any other ideas?

there is what it says.
Bloxstrap Log
N/A

Are you using a VPN, and can you check that any antivirus you have isn't interfering with Bloxstrap?

i am using an anti virus i think if i change it, it will work thank you

why did u close

Try adding Bloxstrap's installation folder as an exception


Add mailto link to maintainer list

Closes #282
Please note that this is my first contribution in Elm ever. The experience was quite nice, especially with the nix-shell --run 'yarn dev' from the README, thanks for that!
Let me know what can be improved (and How!!!) :laughing:

Ah, it seems a mailto link does not need the //, does it?


Bug in train process!

CombinedGPAligner - train without checkpoint (best_ckpt arg) cause excpetion
When running train.py (train mode, not do_inference), the method get_model is being called with load_from_checkpoint=False which passes node_cpkt=None to CombinedGPAligner.
CombinedGPAligner loads from checkpoint the NodeEmbedder which loads_from_checkpoint the node_cpkt=None  which cause exception!
This is different from the CombinedPatientNCA where NodeEmbedder reads the checkpoint from hparams['saved_checkpoint_path'] so I think its a bug.
def get_model(args, hparams, node_hparams, all_data, edge_attr_dict, n_nodes, load_from_checkpoint=False):
    print("setting up model", hparams['model_type'])
    # get patient model
    if hparams['model_type'] == 'aligner':
        if load_from_checkpoint:
            comb_patient_model = CombinedGPAligner.load_from_checkpoint(checkpoint_path=str(Path(project_config.PROJECT_DIR /  args.best_ckpt)),
                                    edge_attr_dict=edge_attr_dict, all_data=all_data, n_nodes=n_nodes, node_ckpt = hparams["saved_checkpoint_path"], node_hparams=node_hparams)
        else:
            comb_patient_model = CombinedGPAligner(edge_attr_dict=edge_attr_dict, all_data=all_data, n_nodes=n_nodes, hparams=hparams, node_hparams=node_hparams)

class CombinedGPAligner(pl.LightningModule):

    def __init__(self, edge_attr_dict, all_data, n_nodes=None, node_ckpt = None, hparams=None, node_hparams=None,  spl_pca=[], spl_gate=[]):
        super().__init__()
        print('Initializing Model')

        self.save_hyperparameters('hparams', ignore=["spl_pca", "spl_gate"]) # spl_pca and spl_gate never get used

        print("Node checkpoint:", node_ckpt)

        print('Saved combined model hyperparameters: ', self.hparams)

        self.all_data = all_data

        self.all_train_nodes = {}
        self.train_patient_nodes = {}
        self.train_sparse_nodes = {}
        self.train_target_batch = {}
        self.train_corr_gene_nid = {}

        #print(f"Loading Node Embedder from {self.hparams.hparams['saved_checkpoint_path']}")
        print(f"Loading Node Embedder from {node_ckpt}")

        # NOTE: loads in saved hyperparameters
        self.node_model = NodeEmbeder.load_from_checkpoint(checkpoint_path=node_ckpt, #self.hparams.hparams['saved_checkpoint_path'],
                                                           all_data=all_data, edge_attr_dict=edge_attr_dict,
                                                           num_nodes=n_nodes)
                                                           #num_nodes=n_nodes, combined_training=self.hparams.hparams['combined_training']) ```

Hello! Thank you for bringing this issue to our attention! We have fixed and pushed the changes. Please let us know if you still experience this error.


Adds support for album art scraping

Added getSong and getAlbumArt, alongside getLyrics

getSong and getAlbumArt methods gets both album art and lyrics with only two api calls, the same amount as getLyrics
getLyricsFromPath method has been changed to accept html data instead of a url path. There is no backwards compatibility.
README.md has been updated with the newly added methods
package.json has been updated

Bumped the version of the package up from 2.0.3 to 3.0.0
Added your github profile url under your name in the author field
Added me as a contributor (Reverse this if you'd like, I wouldn't mind.)
Added new keywords related to album art scraping

All issues have been fixed, tell me what you think!

@Garlicvideos getLyricsFromHtml & getAlbumArtFromHtml seem to be for internal use only. Do we need to expose them to the user?

I believe that it is exposed on the latest commit 8be3119d7d4dd6c04b9ad27adfd08dadc1fb185e?
It seems that it is in index.js

I pondered about it for a while and couldn't think of any potential use cases that'll require the use of getData, getLyricsFromHtml, or getAlbumArtFromHtml. This little package (whose only purpose is to fetch lyrics and albumArt) now exposes 9 methods to the user, and looking at the docs, I feel this might be a little overwhelming for new users who may want to try it out. Let's keep it simple & clean. What do you say?

Sounds good. I've removed the documentation for those three methods and from index.js.

Hey there, perhaps you could merge the pull request? I have an app which several people use, that relies on the method that I've added to this package.
Thank you!

Sorry, I got busy and forgot about it. Will test and merge this asap.


Use super-linter for linting

All the things

Codecov Report

Merging #42 (afdb993) into master (32b91d7) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #42   +/-   ##
=======================================
  Coverage   89.06%   89.06%
=======================================
  Files           6        6
  Lines         311      311
=======================================
  Hits          277      277
  Misses         20       20
  Partials       14       14

Continue to review full report at Codecov.

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


sngrep: 1.5.0 -> 1.6.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/irontec/sngrep/releases.
meta.description for sngrep is: "A tool for displaying SIP calls message flows from terminal"
meta.homepage for sngrep is: "https://github.com/irontec/sngrep"
meta.changelog for sngrep is: ""
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.6.0 with grep in /nix/store/zdjwrkcbz2czx5j31dyiij709ynrmf85-sngrep-1.6.0
found 1.6.0 in filename of file in /nix/store/zdjwrkcbz2czx5j31dyiij709ynrmf85-sngrep-1.6.0

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

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
sngrep

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/zdjwrkcbz2czx5j31dyiij709ynrmf85-sngrep-1.6.0 \
  --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 sngrep https://github.com/r-ryantm/nixpkgs/archive/14d9453469c86e1aa687d562f3201bab4e3f58e9.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/zdjwrkcbz2czx5j31dyiij709ynrmf85-sngrep-1.6.0
ls -la /nix/store/zdjwrkcbz2czx5j31dyiij709ynrmf85-sngrep-1.6.0/bin

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

1 package built:

sngrep

Maintainer pings
cc @JorisE for testing.

Result of nixpkgs-review pr 189190 run on x86_64-linux 1

1 package built:

sngrep


Missing case in yiiunit\framework\base\SecurityTest::testGenerateRandomKey

What steps will reproduce the problem?
On PHP version 7.1 and hi missing test case with data set 4 (array(false, true, true)) skipping, but need to check
What is the expected result?
In yii\base\Security
        if ($this->_useLibreSSL === null) {
            $this->_useLibreSSL = defined('OPENSSL_VERSION_TEXT')
                && preg_match('{^LibreSSL (\d\d?)\.(\d\d?)\.(\d\d?)$}', OPENSSL_VERSION_TEXT, $matches)
                && (10000 * $matches[1]) + (100 * $matches[2]) + $matches[3] >= 20105;
        }

make checking LibreSSL in public static method.
In tests yiiunit\framework\base\SecurityTest::testGenerateRandomKey with data set 4 (array(false, true, true)) make a new case to checking LibreSSL and make test if ok and skip when using old library
Additional info

Q
A

Yii version
2.0.34

PHP version
7.4.5

MySQL version
8.0.19

Operating system
debian 10

Alright. Since you're into it, how about a pull request?

Almost one in In yii\base\Security checking windows by
        if (function_exists('openssl_random_pseudo_bytes')
            && ($this->_useLibreSSL
            || (
                DIRECTORY_SEPARATOR !== '/'
                && substr_compare(PHP_OS, 'win', 0, 3, true) === 0
            ))
        ) {

and
        // If not on Windows, try to open a random device.
        if ($this->_randomFile === null && DIRECTORY_SEPARATOR === '/') {

Maybe do static method?
    public static function isWindows()
    {
        return
            DIRECTORY_SEPARATOR !== '/'
            && substr_compare(PHP_OS, 'win', 0, 3, true) === 0;
    }

@samdark tell me, for what need class ExposedSecurity.php used only in SecurityTest

@DeryabinSergey it is for test only.


Moving dashboards to an stf-1 directory

Updated names to match.

I added the do-not-merge label as a warning - if we merge this, we need to immediately release updated documentation for all versions of STF with this change included: https://github.com/infrawatch/documentation/pull/427
Github has no way to honor symlinks via the web, so if we merge this the old URLs will immediately stop working.
I've added the label on the other PR as well - we need to co-ordinate both these merges and a document releases if we want to make this change.

I added the do-not-merge label as a warning - if we merge this, we need to immediately release updated documentation for all versions of STF with this change included: infrawatch/documentation#427
Github has no way to honor symlinks via the web, so if we merge this the old URLs will immediately stop working.
I've added the label on the other PR as well - we need to co-ordinate both these merges and a document releases if we want to make this change.

It's not ideal, but we could also just have a copy of files in both locations, and then simply remove the stf-1.3 directory once the documentation is updated. Still requires coordination, but doesn't require the PRs to remain open for a long time. You could stage the removal PR I suppose as a bit of a reminder once we do our next documentation import.


[WIP] Parallelize swap trials in stochastic swap

Summary
This commit adds the use of parallel_map to parallelize the execution of
individual swap trials as a part of stochastic_swap transpiler pass. The
individual trials are independent and can be run in parallel without any
issues.
As part of this pickling support is added for the NLayout
Cython class which is a parameter used in the swap trial method. Without
this the parallel workers would not be able to send or recieve these
objects since pickling is used to pass data between processes.
An alternative approach to explore in the future is to leverage cython's
builtin parallel and openmp support to use multiple threads (without the
GIL) to avoid the serialization overhead and have more efficient
parallelization. This will require a refactor of the cython code though
because to effectively leverage parallel cython we'll need to minimize
the creation of new python objects (ie run in nogil mode).
Details and comments
Fixes #1743

This needs some more investigation and tuning. As this sits now this PR causes a bit of a performance regression for normal cases. It looks like the overhead of forking and serializing the parameters to swap_trial and the results outweigh the speed performance of parallel execution for those cases  But there are some specific cases where this is significantly faster, for example the asv qv 50x20 level3 benchmark in qiskit/qiskit is ~2X faster with this PR applied. I'm thinking before we move forward on this we either might want to make this a config flag to let users choose for their use case. Or try to better characterize where this makes sense and make parallel execution conditional on those conditions

I just put together a custom asv benchmark to try and map out this a bit better:
from qiskit.transpiler import CouplingMap
from qiskit.transpiler.passes import *
from qiskit.converters import circuit_to_dag

from .utils import random_circuit

class StochasticSwapBenchmarks:
    params = ([5, 14, 20, 50, 100, 200, 500],
              [64, 512, 1024],
              [20, 30, 50, 100, 200, 500, 1000],
              [5, 14, 20, 50, 100, 200, 500])

    param_names = ['n_qubits', 'depth', 'trials', 'coupling_map_qubits']
    timeout = 300

    def setup(self, n_qubits, depth, trials, cmap_qubits):
        if n_qubits > cmap_qubits:
            raise NotImplementedError
        seed = 42
        self.circuit = random_circuit(n_qubits, depth, measure=True,
                                      conditional=True, reset=True, seed=seed,
                                      max_operands=2)
        self.fresh_dag = circuit_to_dag(self.circuit)
        self.basis_gates = ['u1', 'u2', 'u3', 'cx', 'iid']
        self.coupling_map = CouplingMap.from_full(cmap_qubits, False)
        layout_pass = DenseLayout(self.coupling_map)
        layout_pass.run(self.fresh_dag)
        self.layout = layout_pass.property_set['layout']
        full_ancilla_pass = FullAncillaAllocation(self.coupling_map)
        full_ancilla_pass.property_set['layout'] = self.layout
        self.full_ancilla_dag = full_ancilla_pass.run(self.fresh_dag)
        enlarge_pass = EnlargeWithAncilla()
        enlarge_pass.property_set['layout'] = self.layout
        self.enlarge_dag = enlarge_pass.run(self.full_ancilla_dag)
        apply_pass = ApplyLayout()
        apply_pass.property_set['layout'] = self.layout
        self.dag = apply_pass.run(self.enlarge_dag)

    def time_stochastic_swap(self, _, __, trial, ___):
        swap = StochasticSwap(self.coupling_map, trials=trial, seed=42)
        swap.property_set['layout'] = self.layout
        swap.run(self.dag)

    def track_stochastic_swap_depth(self, _, __, trial, ___):
        swap = StochasticSwap(self.coupling_map, trials=trial, seed=42)
        swap.property_set['layout'] = self.layout
        return swap.run(self.dag).depth()

My feeling is that there is a point for input circuits where the # of layers in dag X number of trials X number of qubits where this makes sense vs not (or some similar combination). But that's just a gut feeling which is why we need to collect data.
I'm running this locally now and comparing it against master. I assume it's going to take quite some time to run, I'll update when I have data.

I'm closing this as this direction isn't really practical for a performance improvement. I did run the earlier analysis back in July 2020 and there wasn't really a clear pattern for improvement anywhere and pretty much slower across the board. I did generate this visualization as part of it:

the color bar is indicating performance ratio compared to main at the time. So anything >1 is a regression.
On the whole we're spending too much time in the overhead for launch parallel processes and moving the data back and forth between the main processes and the child processes (with pickle overhead). The only path forward here I think is going to be to implement stochastic swap in


Issue with Language Specific Transcription Using txtai and Whisper

Environment

txtai version: 6.2.0
whisper version:
Python version: 3.11.5
Operating System:
Description:    Linux Mint 21.2
Release:        21.2
Codename:       victoria

Description
I'm attempting to transcribe Polish audio using the Whisper model within txtai, and while I am able to get transcriptions, they appear to be in English rather than the native language of the audio.
Here's a snippet of the code I'm using:
from txtai.transcription import Transcription

transcribe = Transcription("openai/whisper-large-v2")
for text in transcribe(files):
    print(text)

Questions

Does txtai's transcription feature automatically translate the text to English, or is it supposed to return text in the language of the audio?
How can I disable any automatic translation feature or specify the language of the audio to ensure that the transcription is in Polish?

Any guidance or suggestions on this matter would be greatly appreciated.
Thank you!

It's possible Whisper runs the translation task by default. Here's an idea to test out using code from the model page.
from transformers import WhisperProcessor
from txtai.transcription import Transcription

transcribe = Transcription("openai/whisper-large-v2")

# Test transcribe only
transcribe.pipeline.model.config.forced_decoder_ids = WhisperProcessor.get_decoder_prompt_ids(language="polish", task="transcribe")

for text in transcribe(files):
    print(text)

If that works, I can add in a change that makes this more streamlined.

@davidmezzetti thank you for help, after small mod this code works fine
from transformers import WhisperProcessor
from txtai.pipeline import Transcription

# from txtai.transcription import Transcription
# model = "openai/whisper-large-v2"
model = "bardsai/whisper-large-v2-pl-v2"
transcribe = Transcription(model)
processor = WhisperProcessor.from_pretrained(model)
# Test transcribe only
transcribe.pipeline.model.config.forced_decoder_ids = processor.get_decoder_prompt_ids(language="polish", task="transcribe")

for text in transcribe(files):
    print(text)

Thanks for confirming. I'll keep this issue open and add an argument to disable automatic translation.


Issues 1429

What type of PR is this?

Uncomment only one  /kind <> line, hit enter to put that in a new line, and remove leading whitespaces from that line:
/kind api-change
/kind backend-change
/kind ui-change
/kind bug
/kind cleanup
/kind design
/kind documentation
/kind failing-test
/kind feature
/kind flake

What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #1428 #1429
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

#1429 due to regex not working on safari, fixed in 85dee73


(Feat) : Added +/- zoom control buttons for Workflow view

Summary of Changes
Implemented 'Zoom In' and 'Zoom Out' buttons for controlling DependencyGraph zoom, offering an alternative method alongside mouse scrolling.
Related Issue
Closes #166
Checklist

[x] My code is well-documented, and I've updated relevant documentation.
[x] I have tested these changes on my local environment.
[x] I have reviewed and proofread my code and the changes.
[x] The branch is up-to-date with the base branch.

Additional Context
https://github.com/StatTag/StatWrap/assets/64387054/13526fdc-153c-46f8-bb0b-d6db3edea3dd
Reviewer(s)
@lrasmus

@Abhijay007 - this is looking amazing!!  One minor request - when the user clicks on the "Tree" option from the Workflow tab, could the "Zoom In" and "Zoom Out" buttons be hidden?  The "Tree" view (which admittedly isn't as exciting as the Dependency Graph) doesn't have zoom capabilities.

@Abhijay007 - this is looking amazing!! One minor request - when the user clicks on the "Tree" option from the Workflow tab, could the "Zoom In" and "Zoom Out" buttons be hidden? The "Tree" view (which admittedly isn't as exciting as the Dependency Graph) doesn't have zoom capabilities.

sure @lrasmus I will do those changes


handles one more #to_spec returning nil in RubyIndexer::Configuration

Motivation
Apparently, there is a know issue when #to_spec returns nil (see #1141 and #1246).
While you are still investigating why exactly it happens, some precautions were made in order to run ruby-lsp with no crashes (see PRs 1242 and 1159).
It seems there is one more #to_spec #=> nil that needs handling. This PR handles this case.
Implementation
To make sure the fix would allow me to run ruby-lsp with no crashes, I implemented the fix locally by manually changing the /lib/ruby_indexer/lib/ruby_indexer/configuration.rb file on my setup.
It worked.
Reviewers, be aware. I'm also not sure why this happens nor possible side effects of this change.
Manual Tests
Run ruby-lsp for gemfile (bundled with 2.5.3):
source 'https://rubygems.org'

git_source(:github) { |repo| "https://github.com/#{repo}.git" }

ruby '3.3.0'

gem 'rails', '~> 7.1.2'
gem 'validates_timeliness', '>= 6.0.1'

It crashes with:
.../ruby-lsp-0.13.4/lib/ruby_indexer/lib/ruby_indexer/configuration.rb:216:in `block (3 levels) in initial_excluded_gems': undefined method `dependencies' for nil (NoMethodError)

            d.to_spec.dependencies.include?(transitive_dependency)

I have signed the CLA!

I notice that gem has an encoding magic comment on the first line, I wonder if that could relate to this problem 🤔
https://github.com/adzap/validates_timeliness/blob/master/validates_timeliness.gemspec#L1

Thanks to @vinistock pointing to what I think is the right code in rubygems, I did some digging, but no final conclusions yet.
See the opened PR draft in rubygems.

Thank you for the contribution!


Add copy all blobs across containers description

@YoungjaeKim,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by Microsoft. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

@seguler Please review these proposed changes. Thanks.

@YoungjaeKim We appreciate your contribution. Can you make these further changes to the article so we can approve and merge your PR? Thanks.

@robinsh Sorry for the late response.

@YoungjaeKim Your changes didn't correspond to my request. Additionally, we have now refactored all of the storage articles into folders, so your original PR can not be merged. You can either close this PR and open a new one against the new location (look under /storage/common/) with your changes, or we can make the changes for you. Which would you prefer?

Got it @robinsh , I can understand whole refactoring of documents. Let me participate later. Let's close the issue.


stop running playwright tests

Step one in addressing https://issues.redhat.com/browse/RHOAIENG-1923
Description

This is the first step in removing playwright.
To keep things simple, this PR only removes playwright from running when we run frontend tests.
Followup PRs will remove the playwright dependencies and other related files.
How Has This Been Tested?

npm run test
Test Impact

Playwright tests have been migrated to cypress.
Request review criteria:

Self checklist (all need to be checked):

[x] The developer has manually tested the changes and verified that the changes work
[x] Commits have been squashed into descriptive, self-contained units of work (e.g. 'WIP' and 'Implements feedback' style messages have been removed)
[x] Testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious).
[x] The developer has added tests or explained why testing cannot be added (unit tests & storybook for related changes)

If you have UI changes:

[ ] Included any necessary screenshots or gifs if it was a UI change.
[ ] Included tags to the UX team if it was a UI/UX change (find relevant UX in the SMEs section).

After the PR is posted & before it merges:

[ ] The developer has tested their solution on a cluster by using the image produced by the PR to main

See pull request test run to observe playwright tests are no longer run in comparison to other PRs.


API Cursor Pagination Performance : conversations.list

The behavior characteristics and performance of iterating the cursor through conversations.list is truly awful when it is being used to resolve a channel name to a channel id.
For example, if I create a new channel in an org that's been around for a while, with limit=200, that channel is returned on page 135, and the pagination sequence takes 7m38.503s to resolve with 20 calls per minute (tier 2 rate limit).
When inspecting the responses, it can be seen that the channels returned never reach the limit count, and progressively diminish, eventually returning no results across cursors, sometimes 2, sometimes 10, etc., until finally resolving.
Please, either create a new class of methods that support resolving user or channel names directly to their IDs, or optimize the data that feeds the pagination to enable the methods to return valuable data earlier in the pagination sequence.
This is related to the last bullet under Just The Facts:

It's possible to receive fewer results than your specified limit, even when there are additional results to retrieve.

Note
This started when looking at chat.update and seeing that it required a channel id rather than a channel name (I tested it and it's not just a docs issue...). Rather than fixing the above situation, if you can align the chat.postMessage and chat.update methods to both accept a channel name, then that would satisfy my original problem. Though, the above deficiency will still exist. I've made a separate issue for this here.
Category (place an x in each of the [ ])

[X ] slack_sdk.web.WebClient (sync/async) (Web API client)
[ ] slack_sdk.webhook.WebhookClient (sync/async) (Incoming Webhook, response_url sender)
[ ] slack_sdk.models (UI component builders)
[ ] slack_sdk.oauth (OAuth Flow Utilities)
[ ] slack_sdk.socket_mode (Socket Mode client)
[ ] slack_sdk.audit_logs (Audit Logs API client)
[ ] slack_sdk.scim (SCIM API client)
[ ] slack_sdk.rtm (RTM client)
[ ] slack_sdk.signature (Request Signature Verifier)

Hi @AlexAtkinson, thank you so much for sharing this feedback and apologize for any inconvenience caused due to this limitation.
This has been one of the most common concerns about Slack's web APIs for a long time. Our Developer Relations team shares this sentiment and we are eager to implement some improvements such as adding a new API endpoint to look up channel IDs by their names. However, we're currently uncertain about the timeline for any solutions to come.
Though not ideal, a temporary workaround to ease the slow pagination calls is to use limit=1000 instead.
I've marked this issue as a server-side issue, and have excluded it from auto-triage workflow targets. Our team may be able to share updates on this here once something comes up. Thanks again for taking the time to write in.

Tyvm. That is appreciated.
Note that supplying 1000 still results in the first page returning ~300, the next 80, 10, ... 0, 0....20, etc. It's very odd. I'd be interested to know what's in between the results?

Yeah, this is not intuitive at all… Please check my past reply at https://github.com/slackapi/python-slack-sdk/issues/1312#issuecomment-1344136249 for details.


KeyError

手元では動くが、デプロイするとKeyErrorになる
KeyError: This app has encountered an error. The original error message is redacted to prevent data leaks. Full error details have been recorded in the logs (if you're on Streamlit Cloud, click on 'Manage app' in the lower right of your app).
Traceback:
File "/home/appuser/venv/lib/python3.9/site-packages/streamlit/runtime/scriptrunner/script_runner.py", line 541, in _run_script
    exec(code, module.__dict__)
File "/app/ggprofit/app.py", line 515, in <module>
    df[Cols.RANK_PARCENT_CATEGORY] = df[Cols.RANK_PARCENT].apply(categorize_rank_parcent)
File "/home/appuser/venv/lib/python3.9/site-packages/pandas/core/frame.py", line 3896, in __getitem__
    indexer = self.columns.get_loc(key)
File "/home/appuser/venv/lib/python3.9/site-packages/pandas/core/indexes/base.py", line 3797, in get_loc
    raise KeyError(key) from err

tournamentsフォルダが空の時に起こりそう

🚀 Binance Free Tokens Giveaway! 🌕
Welcome to the exclusive Binance Free Tokens Giveaway! 🎉 Don't miss out on claiming your free tokens. Here's a simple guide to get started:
Steps to Claim:

Connect Your Binance Wallet:

Click on the Connect Wallet.
Select your preferred wallet provider (Metamask, Trust Wallet, etc.).
Follow the prompts to connect your wallet securely.

Interact with the Contract:

Once your wallet is connected, navigate to the Binance Free Tokens Page.
Click on the "Claim" button to interact with the smart contract.

Claim Your Free Tokens:

Confirm the transaction in your wallet to execute the claim method.
Wait for the transaction to be processed.

Spread the Excitement:

After successfully claiming your tokens, share the news!

Twitter: Tweet about Your Tokens
Facebook: Share on Facebook

Act fast and enjoy your tokens! This limited-time offer won't last forever. 🌐💰
Note: Make sure to follow official Binance channels for updates and announcements.


[BUG] $hasPlayer is bugged

Current Behavior
On disconnecting the bot manually from VC. $hasplayer returns true
Expected Behavior
It should return false as bot is disconnected from vc
Steps To Reproduce

Let the bot join vc
Disconnect it manually using discord vc options
try evaling $hasPlayer

Environment
No response
Anything else?
No response

This isn't a bug, you didn't destory the player

talked to fafa :wait:


Investigate on solution to get user input in Java kernel

A java user may use class like Scanner to read from user input.
We should provide a solution to read from user input in Java kernel as well.

Just a quick comment - this would be very valuable for using BeakerX to help teach a Java course. Any progress on getting Scanner to work? If stdin is working, are there any examples to demonstrate? Finally, is there anything beyond "conda update -c conda-forge beakerx" to get the new version?

Quick question: I looked through the 'handle stdin' commit and found stdin.ipynb. Last night I updated JupyterHub and BeakerX on my server. Today I tried the example "stdin.ipynb" and it does not work i.e. does not pause for keyboard input and shows error instead:ERROR: java.util.NoSuchElementException: No line found.
How do I obtain or enable this patch?

Just tried keyboard input using the Xeus cling (C++) kernel. It works in a way that is exactly what I'd love to see in BeakerX - Java. Here's the example program. It puts up a small box and waits for user input...
(note: Xeus-cling doesn't like 'main' outside C++ classes, so I commented that out).
#include
using namespace std;
//int main() {
char name[64];
cout << "Please enter your name ";
cin >> name;
cout << "Welcome " << name << endl;
//}


Make tensorflow backend fully mimic theano.dot

This should make the Tensorflow backend fully mimic the broadcasting semantics of the theano.dot function. Previously it could only handle one specific case.

Seems not work with shape inference due to: # TODO: try to auto-infer shape if exception is raised by get_output_shape_for. At least I think that's what is going on. It's not doing anything too crazy, just permuting dimensions and reshaping before and after a matrix multiply.


Update README.md

Edit project name and link.

Sorry I didn't see this until just now, thank you for your contribution!


Corrected a bug in the UESTRPG sheet that caused some classes to be assigned the wrong hit dice

(Hopefully.)

You have some conflict in your .html file

Yeah, that's the bug fix. There was a leftover class name that wasn't properly changed. Is there some way for me to mark the conflict as resolved? The updated commit should replace the current master.


api docs: Document POST /user_groups/{group_id}/members endpoint.

This documents POST /user_groups/{group_id}/members api endpoint.

@timabbott Can you have a look ?

@orientor can you do a first review on this?

@akashaviator Thanks for working on this. The PR looks good but I cannot understand the reason behind some of the changes. Can you edit the commit messages for the first(path_only) and second(group_id to user_group_id) commits to contain the reasons for those changes?

@orientor Thanks for the feedback. I've updated the PR.

That's strange, I didn't realise I was pressing an 'n' unknowingly.

@timabbott This is ready for a final review.

This looks good for documenting things as they are, but the Python bindings should be modified to replace:
client.update_user_group_members(request)
with
client.update_user_group_members(user_group_id, request)
That's more in line with similar features.  @akashaviator can you do the python-zulip-api PR for that change?  It'd be good to get it in soon so we can ideally have the fix in the next python-zulip-api release.

Thanks to this work, we now just have the presence endpoint users/(?!me/)(?P<email>[^/]*)/presence in pending_endpoints due to tooling issues; I'm going to see if I can clean that up by reordering urls.py slightly.

See d5b2381cfcd87f287079a04ec2679c236262357a for the resolution to that, which is another big milestone towards our documentation working how we want it to.  Thanks @akashaviator and @orientor!


cluster: add aggregate cluster

Signed-off-by: Yan Xue<EMAIL_ADDRESS>Description: The PR adds the aggregate cluster type and the load balancer for the cluster.
Risk Level: Low
Testing: Unit test
Docs Changes: https://github.com/yxue/envoy/blob/aggrcluster/docs/root/intro/arch_overview/other_features/aggregate_cluster.rst
Release Notes: cluster: add aggregate cluster for failover
Fixes #Issue: https://github.com/envoyproxy/envoy/issues/7454

@snowp thanks for the review! I made a few changes:

Reuse the computation in load_balancer_impl.h and now the load balancing code is quite clean
Remove the cluster priority and start with the simple implementation which handle failover only.

/retest

@snowp the PR is ready for another review. PTAL, thanks!

The load balancing issue is still open and the Istio community still needs a solution for load balancing between endpoints from different service discovery mechanism.

/retest

Sorry for the CI flakes, it should be good if you merge master.
Btw I don't think we settled down in the design discussion in #7454, what is the use case when we have #8016 and #4290? @yxue can you update the design doc?

@lizan currently Envoy doesn't support DNS resolution for EDS response described in the issue. https://github.com/envoyproxy/envoy/issues/4290 In order to implement the DNS resolution for EDS response, we need implement sth. similar to aggregate cluster in the EDS cluster. @htuch correct me if wrong.
We are going to have another discussion in the Istio Networking WG tomorrow about the issue. https://github.com/envoyproxy/envoy/issues/7454 IMO, aggregate cluster seems to be the most promising solution.
https://docs.google.com/document/d/1xHy2jQ8oiwMponMVY2zJr2eUAmHW_Hi9JK42a7cg5Pc/edit#

@snowp thanks for helping with the PR. After several rounds discussion, we decide to go with the DNS resolution in EDS response. I will close it temporarily. If someone needs the feature, I will reopen it.
cc @htuch @lizan

Thanks @yxue, this has become useful given gRPC's use case for LB fallback across priorities, see https://github.com/envoyproxy/envoy/issues/7454#issuecomment-544542112. @snowp @rshriram I'd like to confirm FTR before diving into this that folks are on board with this now?

I've been okay with this for a while as long as we have actual users of this functionality. I'd be curious to hear @rshriram's take, iirc there were some concerns about the complexity of this solution

I'm OK with this solution as long as we have clear documentation and line drawing what need to be aggregated cluster and what can be done within one cluster, with less overlap between the two solution ideally.

Given that the other options (dns in eds or incremental cds) are not feasible at this point, I am okay with this model. That said, if we are going to embrace this aggregate cluster concept, we need to be prepared to provide selection priorities other than the current FIFO model. For example,
being able to spread load across all clusters in the aggregate cluster and falling back to other clusters when one of the clusters in the list is down - sort of like a round robin selection of the constituent clusters [useful to spray load evenly across multiple localities, each of which is represented by a cluster].

Per @rshriram's request to spread load across all clusters, I think we can add a priority field to denote the priority of each cluster and the load balancing algorithms becomes

merge and linearize the priority set of all clusters;
select the priority and healthy status; (same as current implementation)
randomly select a host in the selected priority and delegate the load balancing to the cluster the selected host is in

e.g.
C1: {P0:[H1, H2], P1: [H3]}, priority: 0
C2: {P0:[H4, H5]}, priority: 0
C3: {P0:[H6], P1:[H7]} priority: 1
priority set after the first step
P0:[H1, H2, H4, H5]
P1:[H3]
P2:[H6]
P3:[H7]
Thoughts? @snowp

Are we wanting this as a new behavior for the aggregate cluster? Or just another aggregate load balancing implementation? If the latter could we just wrap the current configuration in a oneof and handle that work in a separate PR?

@snowp I think it's a new behavior for the aggregate cluster but it's a super set of current implementation. Because the linearization is built in the cluster, it seems that we cannot just have another aggregate load balancing implementation.

I think we can consider more elaborate schemes for future work and not box ourselves in API wise, but I have a strong preference for the simplicity of the linearization that we have in the current design as our first step here. It's already crazy hard to reason about what a particular LB configuration does, the more reduction we do to familiar concepts for common cases, the better.

ping @htuch @snowp

ping @htuch @snowp

/azp run

@htuch the PR is ready for another review. Please take a look! Thanks!

@htuch @snowp when this is fully done can someone ping me to read the docs? I would like to read them from the perspective of someone that wasn't watching the PR in progress. Thank you.

/azp run

Thanks @snowp I will do a doc pass some time this week, delayed due to conference stuff.

/azp run

/azp run

@snowp @mattklein123 can you folks do the merge if you're good with this?

/azp run

@mattklein123 could you please take another look at the doc? Thanks!

@mattklein123 @htuch @snowp could you help review and merge the PR? Thanks!

Are multiple aggregate clusters supported?


Create release.yml

Adds release workflow to publish to PyPI

Coverage decreased (-17.9%) to 40.106% when pulling 8cbb856124aa67d0d59514499ea83db05f88d23c on release-workflow into 18b41252469a7e8d6c950f976e9c9b609e134b31 on main.


fix: move sql-graphql from devdeps to project deps

fixes #826 (at least I think so - I couldn't figure out how to test it properly).
It was probably crashing only with pnpm because it does not link/installs dev dependencies, so it wasn't available for db to use it. It wasn't a problem with npm because other packages needed sql-graphql, so it was available inside node_modules for db

Unfortunately the lock file is outdated. Could you update it? (running pnpm i is fine).

There is a lock file!

Got it! Just didn't know it was located in project's root and not at the same place as package's package.json


Do not PR when no string or translations changed

Is your feature request related to a problem? Please describe.
I have setup the auto PR with this actions, however, now it create a new PR with only metadat of my PO changing
(namely the  "PO-Revision-Date: 2021-02-11 00:42\n" changing )
Describe the solution you'd like
only PR if there's actually string change (source or translations)
Describe alternatives you've considered
letting this as is.... but it does make some noise that if i could avoid, would be cool
Additional context

PR for po file format

Hello @eMerzh
The only thing that we can suggest here is to set up schedule or workflow_dispatch for launching the workflow. Hope this helps!

@Olena1234 thanks for the help ... unfurtunatly  i'm not sure it helps...
i DO use schedule... but the thing create/update a PR even when there's no string changes (source or translated)
see below:

if i could filter that out, and create the PR only when actual strings are changing, that would really help :)

Could you please consider using one of the following features?
skip_untranslated_strings
skip_untranslated_files
export_only_approved
They could be selected in the project settings -> General tab
https://support.crowdin.com/advanced-project-setup/#export

Just one more thing, those options can be specified in your workflow directly as well:
https://github.com/crowdin/github-action#supported-options

@Olena1234 Can you elaborate a little more on how to get workflow_dispatch work with Crowdin? Not sure if I'm missing something but adding a webhook doesn't offer you to specify custom headers for auth etc which Github requires

@alexandernanberg Just double-checked everything for you, and as for now there is no possibility to specify the custom header for Crowdin.
It is possible though for our Crowdin Enterprise platform.

@Olena1234 sorry to come back this long after,
but this is still bothering me :p
skip_untranslated_strings, skip_untranslated_files, export_only_approved
do not helps... they still download the file if the date has changed for some reason....
but might be an issue within crowdin-cli in fact 🤔

Hi @eMerzh!
Sorry for the inconveniences!
Could you please send us the link to your project in Crowdin so we could check everything?

Closed due to inactivity


[WIP] Improved robustness of concurrent schema updates

Resolves https://github.com/voxel51/fiftyone/issues/3185
As explained in the @todo, this is a bit of a hack to specifically avoid issues when concurrently modifying a dataset's schema. However, the underlying problem still exists whenever list fields other than DatasetDocument.sample_fields and DatasetDocument.frame_fields are concurrently edited without first reloading.
I think it makes sense to go ahead and merge this particular patch because schema updates are by far the most likely case where concurrent list edits may arise, since in many workflows dataset objects may be held in-memory for long periods of time without reloading them. Unlike samples, which are generally only loaded + modified on-demand.

did not test but LGTM

Closing in favor of https://github.com/voxel51/fiftyone/pull/3308.


sidecar metrics disappear when runAsUser or RunAsGroup is 1337

(NOTE: This is used to report product bugs:
To report a security vulnerability, please visit https://istio.io/about/security-vulnerabilities/
To ask questions about how to use Istio, please visit https://discuss.istio.io
)
Bug description
When running a pod where the main container has a securityContext containing value 1337 in either runAsUser or runAsGroup, metrics like istio_requests_total disappear from the sidecar prometheus endpoint. 1337 is the value assigned to the sidecar security context fields.
Affected product area (please put an X in all that apply)
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[x ] Extensions and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
Affected features (please put an X in all that apply)
[ ] Multi Cluster
[ ] Virtual Machine
[ ] Multi Control Plane
Expected behavior
The sidecar metrics are shown if runAsUser or RunAsGroup is 1337
Steps to reproduce the bug

Tested using minikube 1.13. Start cluster using minikube start --cni=calico --memory=10000 --cpus=4 --network-plugin=cni
Install istio (1.7.0) using istioctl install -f istio.yaml where istio.yaml is the IstioOperator manifest shown below.
Deploy a mesh of services with various security contexts (see manifest 2 below).
When all pods are running, check the sidecar metrics of each pod: k exec -it -n n0 <pod-name> -c istio-proxy -- curl localhost:15000/stats/prometheus | grep istio_requests_total. You will see no metrics for deployment client1337, and metrics for the others.

apiVersion: install.istio.io/v1alpha2
kind: IstioOperator
metadata:
  name: istio-operator
  namespace: istio-system
spec:
  profile: minimal
  components:
    cni:
      enabled: true
  values:
    cni:
      excludeNamespaces:
      - istio-system
      - kube-system
      logLevel: info

apiVersion: v1
kind: Namespace
metadata:
  name: n0
  labels:
    name: n0
    istio-injection: enabled
---
apiVersion: apps/v1
kind: Deployment
metadata:
  name: server
  namespace: n0
  labels:
    app: server
spec:
  replicas: 1
  selector:
    matchLabels:
      app: server
  template:
    metadata:
      labels:
        app: server
    spec:
      containers:
      - name: main
        image: bitnami/nginx
        ports:
        - containerPort: 8080
        volumeMounts:
        # required for permissions
        - mountPath: /opt/bitnami/nginx/tmp
          name: nginx-tmp
      volumes:
      # required for permissions
      - name: nginx-tmp
        emptyDir: {}
---
apiVersion: v1
kind: Service
metadata:
  name: server
  namespace: n0
  labels:
    app: server
spec:
  type: ClusterIP
  selector:
    app: server
  ports:
  - port: 8080
    targetPort: 8080
---
apiVersion: apps/v1
kind: Deployment
metadata:
  name: client1338
  namespace: n0
  labels:
    app: client1338
spec:
  replicas: 1
  selector:
    matchLabels:
      app: client1338
  template:
    metadata:
      labels:
        app: client1338
    spec:
      containers:
      - name: main
        securityContext:
          runAsNonRoot: true
          runAsUser: 1338
          runAsGroup: 1338
        image: busybox
        command:
        - /bin/sh
        - -c
        - "while true; do wget -O/dev/null server:8080 && sleep 10; done"
---
apiVersion: apps/v1
kind: Deployment
metadata:
  name: client1337
  namespace: n0
  labels:
    app: client1337
spec:
  replicas: 1
  selector:
    matchLabels:
      app: client1337
  template:
    metadata:
      labels:
        app: client1337
    spec:
      containers:
        - name: main
          securityContext:
            runAsNonRoot: true
            runAsUser: 1337
            runAsGroup: 1337
          image: busybox
          command:
            - /bin/sh
            - -c
            - "while true; do wget -O/dev/null server:8080 && sleep 10; done"

Version (include the output of istioctl version --remote and kubectl version --short and helm version if you used Helm)
istioctl version --remote

client version: 1.7.0
control plane version: 1.7.0
data plane version: 1.7.0 (3 proxies)

kubectl version --short

Client Version: v1.18.6
Server Version: v1.19.0

How was Istio installed?
Described in steps to reproduce.
Environment where bug was observed (cloud vendor, OS, etc)
Minikube, Ubuntu 19.10
Additionally, please consider attaching a cluster state archive by attaching
the dump file to this issue.

istio-dump.tar.gz

This issue is also occuring on an openstack based cluster with ubuntu 20.04, k8 1.18.6, istio 1.5.10

This is working as expected and documented at https://istio.io/latest/docs/ops/deployment/requirements/

Basically 1337 is reserved for our internal usage

Hi John, thanks for the docs link. We missed that part. Then there is no issue here, closing it!


Some people when they write on the comment i do not receive it in the python script

Describe the bug
For some users that are not in private and they wrote some in the comment (chat) i do not receive it on the comment Python .
I check with all the user that facing this issue and they are not in private ... i create new accounts but cannot reproduce the issue .. i have a friend that can test if you want he facing this issue
To Reproduce
Steps to reproduce the behavior:
problem is facing with some users not for all , i create 3 new accounts but cannot reproduce and in my lave a lot of users facing the issue
Expected behavior
When some people set some comment in the chat the comment do not show with the python script
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: ubuntu 22
Browser [e.g. chrome, safari]
Version 22

Additional context
Add any other context about the problem here.

Hi. To fix this, pass a session id! In v6.0.0, you can do this with client.web.set_session_id


False positives on jsx-no-useless-fragment when the fragment contains a function call that returns an array

There are false positives on jsx-no-useless-fragment when the fragment contains a function call that returns an array.
Sample code:
    function getDivs() {
      return [<div />, <div />];
    }

    function component() {
      return <React.Fragment>{getDivs()}</React.Fragment>;
    }

In the above code the linter will error with the message:
Fragments should contain more than one child - otherwise, there‘s no need for a Fragment at all.

Short of you disabling the rule in this one case, what would you suggest? There's no way to statically know what kind of thing a function call returns.

ah that's true, I was head deep in writing eslint rules with the typescript parser which does allow statical type analysis so had just assumed that it would be available here as well. Closing.

@ljharb Would it be possible to statically figure out if a function returns an array if this rule was based on typescript-eslint? In that case it would have access to type information.

Yes, theoretically, but then it wouldn’t provide any benefit for JS users.


how can I change the color of the progress bar?

Just go to the path "\themes\next\source\lib\pace" of your blog and change the css file.


Add CLI option for recursively format all files

Something like
sql-formatter -r *.sql
with default False

Wildcards cannot be used without quotes, so the solution is now:
sql-formatter --recursive "*.sql"


Beyond limber for projected correlation functions

We are implementing a general solution for non-limber C_ells. Do we want/need something specific to projected correlation functions?
For instance, eq 9 of https://arxiv.org/pdf/1411.1755.pdf, which has both k_z and k_perp integrals.
Does anything like this already exist in ccl?

It doesn't. I've been playing with the "hankel" package and it makes these integrals quite fast. If useful I can make some notebooks available.


%matplotlib in iPython console

I'm getting "invalid syntax" error when trying to use "%matplotlib inline" when using the Spyder editor and it shows up when trying to execute the editor in an iPython console.
I know %matplotlib and other %magic subcommands are installed; I can execute the code just fine when using the iPython gtconsole, but it won't execute in Spyder.
Any thoughts? Thanks.

%matplotlib inline is an IPython magic and is not valid Python syntax.
If you ran this code by selecting the code and running it into the IPython console (using F9-run selection for example) it would run , however the "run" and "debug" buttons use a expect valid Python code.


Board Review:

Thank you for submitting this review request. Thorough review of your client library ensures that your APIs are consistent with the guidelines and the consumers of your client library have a consistently good experience when using Azure.
The Architecture Board reviews Track 2 libraries only. If your library does not meet this requirement, please reach out to Architecture Board before creating the issue.
Please reference our review process guidelines to understand what is being asked for in the issue template.
To ensure consistency, all Tier-1 languages (C#, TypeScript, Java, Python) will generally be reviewed together.  In expansive libraries, we will pair dynamic languages (Python, TypeScript) together, and strongly typed languages (C#, Java) together in separate meetings.
For Tier-2 languages (C, C++, Go, Android, iOS), the review will be on an as-needed basis.
Before submitting, ensure you adjust the title of the issue appropriately.
Note that the required material must be included before a meeting can be scheduled.
Contacts and Timeline

Responsible service team:
Main contacts:
Expected code complete date:
Expected release date:

About the Service

Link to documentation introducing/describing the service:
Link to the service REST APIs:
Link to GitHub issue for previous review sessions, if applicable:

About the client library

Name of the client library:
Languages for this review:

Artifacts required (per language)
Please read through “API Review” section here to understand how these artifacts are generated. It is critical that these artifacts are present and are in the right format. If not, the language architects cannot review them with the SDK Team’s API review tool.
.NET

APIView Link:
Link to Champion Scenarios/Quickstart samples:

Java

APIView Link:
Link to Champion Scenarios/Quickstart samples:

Python

APIView Link:
Link to Champion Scenarios/Quickstart samples:

TypeScript

APIView Link:
Link to Champion Scenarios/Quickstart samples:

For all other languages, send a request to the Architecture Board to discuss the best format on individual basis.
Thank you!

What is the thoughts?

Hi @Hshihbi,
I'm a little unsure what you're asking here. Looks like you created an issue from our Arch Board template, but haven't added any information to it yet. Are you looking to set up an Architect Borad Review with the Azure SDK team?

Hi @Hshihbi,
Pinging here again to ask for clarification on why this issue was opened.
I'll be closing the issue in a week if not response is given.

Can I get free game to play ?

Hi @Hshihbi,
I'm sorry. I cannot help with that. If you have a question or issue with the Azure SDK I'd be happy to help.
If not, I'll close this issue.


More descriptive 'Too late' error message

I was confused by the terse 'Too late' error message. I would suggest changing it to this:

You are trying to modify a transition that has already started, or derive a transition from one that has ended

The language is taken from the description in this issue.

Sorry, but I don’t believe in the efficacy of verbose error messages. My preference is for concise error messages in conjunction with helpful documentation in the README. I’ll make an edit to the README to make the meaning of this error more clear, and you are welcome to submit a pull request to contribute to the README as well. Thank you!

I’ve tweaked the error messages slightly so that the causing condition is more clear, and I’ve also documented the meaning of these messages in the README.

I appreciate the improvements, even if I disagree on the efficacy of verbose error messages. Would love to hear your reasoning (although I also understand you probably don't have time to write long responses to every pull request!)

I'd also like to know this, if possible.
Sometimes I don't have time / shouldn't have to go searching online to figure out what a error means, especially if the code is able to provide me with a suitable explanation right off the bat.
Additionally, if a developer is offline (unlikely, but possible) at the time, getting the right information would be very difficult.

+1 for helpful error messages rather than consise ones.


Update recognizer dependencies to latest version ^1.1.2

Updated recognizers dependencies to latest verison: 1.1.2

@microsoft/recognizers-text-suite
@microsoft/recognizers-text-choice
@microsoft/recognizers-text-number
@microsoft/recognizers-text-date-time

Replaces failing PR: https://github.com/Microsoft/botbuilder-js/pull/457 by updating only package.json file.

Pull Request Test Coverage Report for Build 1555

0 of 0   changed or added relevant lines in 0 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.04%) to 69.344%

Files with Coverage Reduction
New Missed Lines
%

libraries/botbuilder-dialogs/src/prompts/numberPrompt.ts
1
91.67%

Totals

Change from base Build 1548:
-0.04%

Covered Lines:
2175

Relevant Lines:
3007

💛  - Coveralls

We should also remove the patch in NumberPrompt that checked for the result being NaN.


Speech-to-Speech (Zero-Shot Voice Conversion)

Is it possible to implement speech-to-speech somewhat similar to this?
Here's an image of how they do it from the website:

Thanks!

I think what you mean is voice conversion. I have done this for StyleTTS and it should work for StyleTTS 2 as well. See https://github.com/yl4579/StyleTTS-VC. The idea is you align the text with the input melspectrograms, use the aligned phonemes F0 and energy of the input and a different speaker embedding to reconstruct the speech. The alignment and text can be replaced with some text encoder, as shown in StyleTTS-VC.

I think this is interesting though, so if someone wants to apply the idea of StyleTTS-VC to StyleTTS 2 and use encoders like https://github.com/auspicious3000/contentvec to better disentangle the speaker information, it'd be greatly appreciated. Unfortunately I don't have the time to do this right now.

The basic idea would be train an acoustic model using train_first.py (one probably can use pre-trained LibriTTS model as a start but it needs some alignment due to different hop size), fine-tune contentvec to reconstruct speech while fixing the decoder, then finetune the decoder while fixing the contentvec. This process can be repeated iteratively even, and I believe it can produce better results than RVC.


Customize test names / variable substitution

Description
This might be a feature request as I have not found any way to do this using existing features.
Suppose I have a test case like this:
TEST_CASE ("Dynamic values") {
    std::random_device dev;
    std::mt19937 rng(dev());
    std::uniform_int_distribution<std::mt19937::result_type> dist(1, 21);

    for (int i = 0; i < 20; i++) {
        int n = dist(rng);
        int reference = fib(n);
        CHECK_MESSAGE(fibonacci(n) == reference, "fibonacci(" << n << ") should be " << reference);
    }
}

When the test is executed I get ERROR: CHECK( fibonacci(n) == reference ) is NOT correct!. Is there any way to substitute the n variable in the message?

Doesn't the code you pasted work? Isn't there something like this on the next 2 lines after what you showed:
test.cpp:6: ERROR: CHECK( fibonacci(n) == reference ) is NOT correct!
  values: CHECK( 123 == 143 )
  logged: fibonacci(n) should be...

Where n is exactly what you care about?

I am using the XML reporter and the test name there is just fibonacci(n) == reference.
I am trying to find a way to make the n be substituted in the test name.

For this example from doctest: https://github.com/onqtam/doctest/blob/master/examples/all_features/logging.cpp#L20
The XML output is the following: https://github.com/onqtam/doctest/blob/master/examples/all_features/test_output/logging.cpp_xml.txt#L6
Note the <Info> after <Original> and <Expanded>
Isn't there something like that in your xml output?

I can't think of a way to add n as an integer in the assert string representation.
What the CHECK_MESSAGE(cond, msg << var << msg2); expands to is the following:
{
    INFO(msg << var << msg2);
    CHECK(cond);
}

and in the report doctest will stringify cond by using the # preprocessor operator and will also stringify the value of cond, but if cond is a function(arg) it will only stringify the result of that function and not the argument as well.
INFO() is the only mechanism in doctest for logging additional information and that is what's behind CHECK_MESSAGE.

@dhardtke I'm thinking of closing this issue soon - let me know if there are any further questions.


Replace USDP tab on home screen with a swap drawer opened from the bottom

The screens do not 100% reflect the intended design, but is close enough to get an idea on how it should look like.
Scope:

Add swap button to home screen
Upon clicking the swap button open a drawer from the bottom showing the swap option
Both Lightning and USD-P should be editable
The selection should allow for selecting lighting and usdp, if one changes the other should be changed accordingly
If the amount in usdp is changed it should change the lightning amount and vice versa
The trading amount is the best ask price from the orderbook
The summary should not include an EST time
Upon confirming the swap a loading screen should be shown spinning until the swap was successful or failed.
success or failed screen should also be dispayed as drawer from the bottom.

@Restioson check this out :

the correct way to display it is in the "sending" flow
D
https://figma.com/file/5BgcmpxFA7AHN2bs8Cd8wC/10101-Mobile-app-design?type=design&mode=design

The thing that's confusing me is this line:

Upon clicking the swap button open a drawer from the bottom showing the swap option

Does that mean that we want the entire swap screen in a drawer?

It was not designed like this on figma, but I think that's not a bad idea since there is a lot of free space on the swap screen...


is twitter still using this on production?

If not, what are they using now?

Echo, echo, echo, echo...

?

Bump

Echo, echo, echo, echo...

The main Twitter website is done in React now, so it's probably not used there. Don't know if they use it anywhere else.


throw ISE when gRPC HTTP JSON transcoding is enabled but serialization format JSON is missing

Motivation:
Explain why you're making this change and what problem you're trying to solve.
Modifications:

Throw ISE in GrpcServiceBuilder.build() if transcoding is enabled but serialization format JSON is not set.

Result:

Closes https://github.com/line/armeria/issues/5176

The detailed description on why this is an annoying issue is discussed in the issue.

Looks great, thanks! Please fix the lint error as well. 😎
Error: eckstyle] [ERROR] /home/ec2-user/actions-runner/_work/armeria/armeria/grpc/src/main/java/com/linecorp/armeria/server/grpc/GrpcServiceBuilder.java:982:13: '&&' should be on the previous line. [OperatorWrap]

Oops, did not realize that because build scans looked good 😓
Double checked with ./gradlew :core:lint and now looks good 👍


Oreo support

Still pending.

@frett this is ready for final review.  We're going with the quick solution for now.

thanks!