index-gff does not seem to respect --compress option

bionorm index-gff --compress Aeschynomene_evenia/CIAT22838.gnm1.ann0.ZM3R/aesev.CIAT22838.gnm1.ann0.ZM3R.gene_models_main.gff3
File "/home/localhost/adf/.local/pipx/venvs/bionorm/lib/python3.8/site-packages/sh.py", line 865, in handle_command_exit_code
raise exc
sh.ErrorReturnCode_1:
RAN: /erdos/adf/bin/tabix -p gff Aeschynomene_evenia/CIAT22838.gnm1.ann0.ZM3R/aesev.CIAT22838.gnm1.ann0.ZM3R.gene_models_main.gff3
STDOUT:
STDERR:
[tabix] the compression of 'Aeschynomene_evenia/CIAT22838.gnm1.ann0.ZM3R/aesev.CIAT22838.gnm1.ann0.ZM3R.gene_models_main.gff3' is not BGZF

or perhaps the option description is simply misleading?
--compress  Compress with bgzip before indexing.
but if I run it on an uncompressed file it is perfectly happy whereas if I feed it a pre-compressed file I get
2021-01-10 15:34:05.542 | ERROR    | bionorm.index:index_gff:84 - Uncompress Aeschynomene_evenia/CIAT22838.gnm1.ann0.ZM3R/aesev.CIAT22838.gnm1.ann0.ZM3R.gene_models_main.gff3.gz befor indexing.
the --help could probably be more --helpful

or perhaps the option description is simply misleading?
--compress  Compress with bgzip before indexing.
but if I run it on an uncompressed file it is perfectly happy whereas if I feed it a pre-compressed file I get
2021-01-10 15:34:05.542 | ERROR    | bionorm.index:index_gff:84 - Uncompress Aeschynomene_evenia/CIAT22838.gnm1.ann0.ZM3R/aesev.CIAT22838.gnm1.ann0.ZM3R.gene_models_main.gff3.gz befor indexing.
the --help could probably be more --helpful

similarly with
bionorm index-fasta
it appears that if --compress argument is given it does not compress the given file, whereas if it is not given then it does compress it.

similarly with
bionorm index-fasta
it appears that if --compress argument is given it does not compress the given file, whereas if it is not given then it does compress it.


Supports prompting whether functions imported by imports are being used

Clear and concise description of the problem
Supports prompting whether functions imported by imports are being used
Suggested solution
Supports prompting whether functions imported by imports are being used
AutoImport({
imports:['a','b','c','d']
})

If a and c are not used, a message is displayed indicating that A and c are not used
Alternative
No response
Additional context
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guide.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.

This is considered out-of-scope and we don't have a plan for that. For reference, Nuxt DevTools is able to list unused imports https://nuxt.com/blog/introducing-nuxt-devtools#imports, which means you should be able to implement sometime similar with the existing APIs if you want to.


Configuration for default proxy use & authentication

Currently there is no way for the app to work from behind a proxy. I have added configuration enabling use of default system proxy and authentication method to app.config.

Thanks @singular0! I assume this is fine for users that don't have a proxy configured?


Stern bash completions clash with kubectl completions

Note: this issue was imported from https://github.com/wercker/stern/issues/137, but it was originally created by drzero42...
I just spent some time troubleshooting a weird problem I was experiencing with kubectl on my system. I noticed that recently I could not get kubectl to autocomplete names of deployments, statefulsets, replicasets and pods. It would gladly autocomplete namespaces, crds, storageclasses, secrets, configmaps and others. As most of my cluster don't have anything in the default namespace, but rather put everything into other namespaces, at first I thought name completion was totally broken, but it turned out that it was just looking in the default namespace, even though I was trying to tab-complete the name of a pod in a different namespace, eg: kubectl -n kube-system describe pod <tab><tab>.
I load bash completions from my .bashrc with source <(kubectl completion bash).  I decided to try loading the kubectl completions by hand in my terminal, which made it work. So I went to my .bashrc and started experimenting with the order that I loaded the completions in, and quickly discovered that if stern completions are loaded after kubectl completions, then I can't autocomplete names from any other namespace than default.
I am in no way an expert on how to write bash completion functions, but a quick look at the output of stern --completion bash and comparing to the output of kubectl completion bash, I see stern defining a number of functions that kubectl also defines. Thus, if the stern bash completions are loaded after the kubectl completions, it screws them up. It would be great if the stern bash completions could work without fiddling with kubectl completions ;)

I have the same issue. is it fixed ? if yes, would you pleaset let me know which version has the fix.

It has been fixed on v1.12.0 and later.

Thanks for your quick reply!


Is 1.7.10 still an option?

A lot of the modded servers are still running on 1.7.10 and it would be absolutely wonderful if you could give this a backport to 1.7.10 as that version completely missed out.
Glad your back by the way!

See discussion under #1249 and #1248

That's just sad. Can't wait until my favorite mods go to 1.8.9
-Isaac
On 7/2/2016 9:19:10 PM, Ross Allan<EMAIL_ADDRESS>wrote:
See discussion under #1249 [https://github.com/nallar/TickThreading/issues/1249] and #1248 [https://github.com/nallar/TickThreading/issues/1248]
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub [https://github.com/nallar/TickThreading/issues/1255#issuecomment-230129979], or mute the thread [https://github.com/notifications/unsubscribe/AF42h6cnH-AYpQBpnTygXLSOogKT7T3mks5qRw4OgaJpZM4JDx1E].

May I just point this out.
1.2k mods vs 47.
http://modlist.mcf.li/version/1.7.10
http://modlist.mcf.li/version/1.8.9

Number of mods for a specific version of Minecraft should never be the determining factor for which version of the Forge API you build a new piece of software for. All in all, stick to the past for to long and you're bound to be in a situation where nothing progresses at all.


kube-dns sidecar container doesn't expose metrics

Is this a request for help?: NO

Is this an ISSUE or FEATURE REQUEST? (choose one): ISSUE

What version of acs-engine?: ALL

Orchestrator and version (e.g. Kubernetes, DC/OS, Swarm) Kubernetes
What happened: Running a kubernetes cluster
What you expected to happen: kube-dns should expose metrics in Prometheus format as in other cloud providers
How to reproduce it (as minimally and precisely as possible): Install cluster using acs engine
Anything else we need to know: If you check the original kube-dns addon https://github.com/kubernetes/kubernetes/blob/master/cluster/addons/dns/kube-dns/kube-dns.yaml.base you can see that the sidecar container exposes a metrics endpoint on port 10054. This metrics are very useful and monitoring tools like prometheus-operator try to scrape it by default.
Related issues:
https://github.com/Azure/AKS/issues/345

We're also hitting this now. It appears that kube-dns on AKS/ACS does not include the sidecar container.
Also of possible interest: https://github.com/kubernetes/dns/blob/master/docs/sidecar/README.md

https://github.com/Azure/acs-engine/pull/3373 fixed the issue


[Function Request] Export media by type

In export page, add the new option to allow user can choose the media type that want to export like:
( ) Include Video
(v) Include Image

Media type filtering is now available in nightly build.


PRIO - Certificate for Ticketing can not be found

Describe the bug
After scanning the Code from the Booking-Portal and selecting an endpoint, the related certificate does not show up in the List of certificates.

Expected behaviour
With the same booking code and the same certificate, the certificate is visible on android:

data used for test

Host Machine OS (Windows/Linux/Mac):

Technical details

iPhoneX
iOS 14.4
Version <IP_ADDRESS> (5)

fixed in wallet 1.2.3(6)


Error Page for transclude: noSlot

Do you want to request a feature or report a bug?
Bug.
What is the current behavior?
Angular threw an error due to misuse of transclude and transclude-slots. The generated error url linked to a not existing page. https://docs.angularjs.org/error/$compile/noslot?p0=foo&p1=bar
Note: This is not the originial generated url. I replaced the content of the query parameters p0 and p1.
If the current behavior is a bug, please provide the steps to reproduce and if possible a minimal demo of the problem via https://plnkr.co or similar (template: http://plnkr.co/edit/tpl:yBpEi4).
http://plnkr.co/edit/Sj4LpgEdzeNmKh3LThXR
What is the expected behavior?
A functional error page.
What is the motivation / use case for changing the behavior?
Helping developer.
Which versions of AngularJS, and which browser / OS are affected by this issue? Did this work in previous versions of AngularJS? Please also test with the latest stable and snapshot (https://code.angularjs.org/snapshot/) versions.
Happened on 1.5.11, I guess on every browser/OS. When you change the angular version on the plunkr to 1.6.2, it also appears.
Other information (e.g. stacktraces, related issues, suggestions how to fix)
I think its a regression bug, but I'm really not sure.

@Narretz I'd like to take a stab at updating this. I found https://github.com/angular/angular.js/wiki/Writing-AngularJS-Documentation for information on how angular handles docs, but no specific information on how error pages are generated. Do you have any additional documentation you suggest I read?

Hi @JoshuaJWilborn, thanks for your interest. The wiki info is incomplete (and we don't maintain it ...)
Fortunately, creating an error page is very simple: just ceate a file with the error name in this folder: https://github.com/angular/angular.js/tree/c80fa1cfe13b3b08b934e7e8ee073661f2b1ab8b/docs/content/error/%24compile
You can use one of the existing errors as a guideline, e.g. reqslot which is also transclusion-related.


RF builder custom templates: Surprise (fuzzy?) translation happening

I discovered this in the course of creating a custom template. It appears that we are automatically translating custom RF tiers, which is unexpected, and problematic in that it results in a mishmash of languages for languages other than the one in which the template was created.

To reproduce:

Create a custom RF template that includes a mix of unique tiers and tiers used in our standard templates such as Goal or Activity.

Add at least one indicator to the Goal and/or Activity tier.

Go to the program page.

Note that these words are translated, whereas the unique tier names are not.

Did you check anywhere else for this (e.g. IPTT or program page?)  It may be happening there too.  I suspect this is just the general behavior of the translation machinery, that it's seeing a word on its list of things to translate and translating it.  This may not bite us all that often.  I'm guessing that if a user has set their language to something other than English, that they'll be entering non-English values for the tier names.  This isn't related to fuzzy translations, those are guesses that the system makes when it sees new translations.
All that said, I think this is going to be a heavy lift to fix. There is a context mechanism that's meant to help with a somewhat similar problem but it doesn't really help us here.  As an example, when we added the descriptive text about number displays to the profile, the word "Period" was used differently than we'd used it in, say, periodic targets.  I added a context tag to the profile page that told the translation machinery to translate Period differently there than elsewhere.  But here we are differentiating how we want to translate in the exact same context.  To do this, I think we'll need to ensure we pass the template type to wherever these tier names are displayed (which is a lot of places now) and put a conditional around the display to say whether it should be translated or not.  Checking other parts of the application to see if it's happening there will help confirm whether or not this problem is widespread.  If it's not, I may be overestimating the LoE.  I'd also really like to see what @camgnostic has to say about it.

@sanjuroj You're right that it's everywhere, and must be a function of the translation machinery.
Clarification: My understanding was that a string needs to have some form of translation tags around it, for it to be picked up by the translation machinery. So I'm wondering: Do we have translation tags around the custom RF tiers, and if so, why? (I'm sure it's more complicated than this, but in my mind, it'd be a matter of removing those tags.)

For those displays, the app is currently blind to what type of tier name it is.  It can't tell custom tags from standard ones, it just is told "here's a tier name, try to translate it".  So we'd have to add that ability to differentiate to every place it's displayed.

Thanks for the clarification, @sanjuroj.
@camgnostic what do you think? Is this a high level of effort?


InputEventMouseMotion.Relative gives incorrect values if MouseMode is set to Captured.

Godot version
Godot_v3.4.2-stable_mono_win64
System information
Windows 10
Issue description
For some reason, when the MouseMode is set to Captured, getting the mouse's relative position returns an abnormally low value at all times. Anywhere from 10 to 100x lower than it ought to be.
Steps to reproduce
I've provided a sample project along with this bug. Move your mouse and watch the Mouse Delta value change. Hitting Space will change the mouse mode from Captured to Visible and back again. Observe the difference in values depending on the mode.
Minimal reproduction project
MouseDeltaTest.zip

You seem to incorrectly assume there can be only one InputEventMouseMotion during a single frame. What you're showing is Relative mouse movement of just the last such event occured during the given frame. Instead of overriding your MouseDelta (in the minimal reproduction project) you can accumulate the result for the current frame instead (since you're already zeroing it out at the end of _Process):
    public override void _Input(InputEvent @event)
    {
        if (@event is InputEventMouseMotion inputEvent)
        {
            // this:
            MouseDelta += inputEvent.Relative;

            // instead of:
            //MouseDelta = inputEvent.Relative;

            ...

And it should "fix the issue".
Not sure why mouse motion events occur more often (and thus with smaller values) in MouseMode.Captured than in MouseMode.Visible though.

Not sure why mouse motion events occur more often (and thus with smaller values) in MouseMode.Captured than in MouseMode.Visible though.

Raw input is used when the mouse mode is set to Input.MOUSE_MODE_CAPTURED.


[FEATURE]: [Square] Use connector_response_reference_id as reference to merchant

:memo: Feature Description

Reference id are used to map transactions in the connector’s dashboard.
Hyperswitch manages several reference ids, such as payment_id, attempt_id, and connector_transaction_id for a single transaction.
However, merchants may encounter uncertainty when determining which ID to utilize in the connector dashboard to identify the payment.

:hammer: Possible Implementation

When we receive a response from the connector for the payment, we deserialize it and populate the response field in RouterData.
For the TransactionResponse type, we must fill the connector_response_reference_id with a corresponding reference id for the merchant to identify the transaction.
One might need to have exposure to api docs of the connector for which it is being implemented to decide what to fill in connector_response_reference_id.
You can check this PR for further reference https://github.com/juspay/hyperswitch/pull/1735

🔖 Note: All the changes needed should be contained within hyperswitch/crates/router/src/connector/
:package: Have you spent some time checking if this feature request has been raised before?

[X] I checked and didn't find a similar issue

:package: Have you read the Contributing Guidelines?

[X] I have read the Contributing Guidelines

:sparkles: Are you willing to submit a PR?

@AkshayaFoiger
I would be delighted to work on this issue
so kindly assign me this
Thank You : )

Hey @Kota-Karthik, I see that you're already assigned to #2351. For now, we'll leave this issue open to any other interested contributors. You may pick this up once your work on #2351 is completed, and if no other contributors have expressed interest on this. I hope that's fine by you

@AkshayaFoiger I would love to work on this issue.

I'm interested in doing this issue so kindly assign me this
Thank You : )

Hey @manirht, I see that you're already assigned to #2248. For now, we'll leave this issue open to any other interested contributors. You may pick this up once your work on #2248 is completed, and if no other contributors have expressed interest on this. I hope that's fine by you

Hey @work-mohit, sure! I've assigned this to you.
In case you have any queries, you can ask them on this issue thread, or on our discord server, or on slack whichever you are comfortable with

Hey @AkshayaFoiger
Will you please assign this me if Mohit isn't taking this up?
Thanks in advance!

Hi @AkshayaFoiger
I would like to work on it, if its available

Hey @VedantKhairnar, sure! I've assigned this to you.
In case you have any queries, you can ask them on this issue thread, or on our discord server, or on slack whichever you are comfortable with

Hi @AkshayaFoiger I would like to work on it, if its available

Hello @bhimeshagrawal, since this issue has already been assigned, could you kindly review #2205, or there are additional similar issues that require attention, I would be pleased to assign them to you

Hi @AkshayaFoiger I would like to work on it, if its available

Hello @bhimeshagrawal, since this issue has already been assigned, could you kindly review #2205, or there are additional similar issues that require attention, I would be pleased to assign them to you

Sure


Merge in code from Adafruit_CircuitPython_DisplayIO_Layout v1.17.0

Merges in code from https://github.com/adafruit/Adafruit_CircuitPython_DisplayIO_Layout v1.17.0
Adds the new example file for plotting lines
Fixes example to use the updated library name

Tested on Adafruit CircuitPython 7.1.1 on 2022-01-14; Adafruit Pybadge with samd51j19, examples work as expected, though I admit I didn't check the advanced example entirely since it seems to be made for a larger screen than the PyBadge.

The CI can be rerun after this PR is merged:  https://github.com/adafruit/Adafruit_CircuitPython_Display_Text/pull/165


release process is undocumented

Release process is completely missing. How do tagged releases + GitHub release work? How is the CHANGELOG.md (currently empty) generated? How do builds end up on PyPi?
Note: Follow-up issue is #179 to simplify the release process.

High priority because not having it is big friction for anyone fixing bugs or enhancing ocean.py.

Updates:

5 days ago Samer added the file RELEASE_PROCESS.md
today I linked to it from developers.md.

So at least now we have a basic release process! I can turn off the "high" priority label.

Also note how like 10 required steps are all automated behind one simple command in the above example. If the release process requires more than 3 steps, please automate it

This is what's needed to close the issue.

This is what's needed to close the issue.

I moved this to a new issue: #179. Therefore we can close this issue.


Command to request pet status

I have finalized the implementation of the management command and admin action to request pet status.
Now it's possible to request pet status of unsolved cases, just by a "python manage.py request_unsolvedcases_status" in terminal. And request status, of any type of case, by a action in  PetAdmin.

Hi @paulo-romano, thanks for the PR.
Can you fix the broken test before I review this?

@paulo-romano could you submit the README as a separated PR?

@paulo-romano as we talked about some time ago, I'll close this PR as I began to do it myself. :)
Thanks!


Turtle.FilePath -> FilePath

Need this for using turtles find followed by readFile

You might be interested in this so answer:
http://stackoverflow.com/questions/34659580/with-haskells-turtle-library-how-to-extract-filename-from-filepath/

I think a better solution would be to re-export Filesystem.readTextFile from the system-fileio library.  How does that sound?

I think that's a great solution.
On Jan 30, 2016 7:47 PM, "Gabriel Gonzalez"<EMAIL_ADDRESS>wrote:

I think a better solution would be to re-export Filesystem.readTextFile
from the system-fileio library. How does that sound?
—
Reply to this email directly or view it on GitHub
https://github.com/Gabriel439/Haskell-Turtle-Library/issues/138#issuecomment-177356621
.


"babylon" cannot be recognized as a valid HTML tag

When I used '@ babylonjs/viewer' in VueJs,
According to the document description, I need to create a "babylon" tag, and my code looks like this:
<template>
  <babylon extends="minimal" model=" https://playground.babylonjs.com/scenes/Rabbit.babylon "></babylon>
</template>
<script setup lang="ts">
  import * as BabylonViewer from '@babylonjs/viewer';
  import {onMounted} from 'vue';
  onMounted(() => {
      BabylonViewer.viewerManager.getViewerPromiseById("babylon-viewer").then(function (viewer) {
          viewer.onEngineInitObservable.add(function () {
              viewer.loadModel({
                  title: "Helmet",
                  subtitle: "BabylonJS",
                  thumbnail: "https://www.babylonjs.com/img/favicon/apple-icon-144x144.png",
                  url: "https://www.babylonjs.com/Assets/DamagedHelmet/glTF/DamagedHelmet.gltf",
              });
          });
      });
  })
</script>
Unfortunately, I received a warning:
[Vue warn]: Failed to resolve component: babylon
If this is a native custom element, make sure to exclude it from component resolution via compilerOptions.isCustomElement.
I have read the source code of the viewer and did not find a declaration for the custom tag 'babylon'. This issue has now prevented me from continuing to write my code.
If there is any way to solve this problem, I suggest you explain it in the official documentation.
Thank you, good luck to you.

As the issue template states, we use issues to track our progress and track validated bugs. We ask devs to join our forum and ask the question there so we can get some examples and information answered before we define anything as a bug.
I would appreciate if you asked the question there, and also provided a project we can debug (instead of just a code snippet).
To answer you direct question, this feels like a vite issue not accepting custom tags, which HTML5 does accept. A quick search would have found this - https://github.com/vitejs/vite/issues/153, which is probably what you need to do.
Closing this issue, will answer in the forum if you have any further questions.


Build getting Failed: Support cordova-cli 9.0.0 - requireCordovaModule deprecation

I am getting following error when creating build for android and ios
Installing "phonegap-plugin-barcodescanner" for android
Subproject Path: CordovaLib
Subproject Path: app
Using "requireCordovaModule" to load non-cordova module "glob" is not supported. Instead, add this module to your dependencies and use regular "require" to load it.

Anything with this pull request?


Is Cargo.toml current?

Running cargo minimal-versions check gives a bunch of errors:
error[E0432]: unresolved import `std::path::AsPath`
  --> /Users/maximilian/.cargo/registry/src/github.com-1ecc6299db9ec823/gcc-0.3.0/src/lib.rs:50:26
   |
50 | use std::path::{PathBuf, AsPath};
   |                          ^^^^^^
   |                          |
   |                          no `AsPath` in `path`
   |                          help: a similar name exists in the module: `Path`

error[E0433]: failed to resolve: could not find `old_io` in `std`
   --> /Users/maximilian/.cargo/registry/src/github.com-1ecc6299db9ec823/gcc-0.3.0/src/lib.rs:328:10
    |
328 |     std::old_io::stdio::set_stderr(Box::new(std::old_io::util::NullWriter));
    |          ^^^^^^ could not find `old_io` in `std`

error[E0433]: failed to resolve: could not find `old_io` in `std`
   --> /Users/maximilian/.cargo/registry/src/github.com-1ecc6299db9ec823/gcc-0.3.0/src/lib.rs:328:50
    |
328 |     std::old_io::stdio::set_stderr(Box::new(std::old_io::util::NullWriter));
    |                                                  ^^^^^^ could not find `old_io` in `std`

error[E0425]: cannot find function `set_exit_status` in module `env`
   --> /Users/maximilian/.cargo/registry/src/github.com-1ecc6299db9ec823/gcc-0.3.0/src/lib.rs:327:10
    |
327 |     env::set_exit_status(1);
    |          ^^^^^^^^^^^^^^^ not found in `env`

(and thank you for the excellent library! It's really helpful over at PRQL!)

Could you give more details? I can’t understand the title, and also the description seems not related to this repo?

If you wanna know is this repo use the latest version dependencies, yes we are, we always keep them up to date https://deps.rs/repo/github/wangfenjin/duckdb-rs

Sorry for the lack of description.
My claim is that attempting to use the library with the minimum possible dependencies in Cargo.toml doesn't work. That's what https://crates.io/crates/cargo-minimal-versions attempts to validate — that if Cargo.lock is the same as Cargo.toml, does the library still compile?
When I run cargo +nightly update -Z minimal-versions to set the minimum versions allowed, and then look for gcc, which seems to be the cause of the error, I find:
gcc v0.3.0
[build-dependencies]
└── miniz-sys v0.1.7
    └── flate2 v1.0.0
        [build-dependencies]
        └── libduckdb-sys v0.7.1 (/Users/maximilian/workspace/duckdb-rs/libduckdb-sys)
            └── duckdb v0.7.1 (/Users/maximilian/workspace/duckdb-rs)

So maybe it's worth attempting to upgrade flake2 or transitively raising upstream?

Trying to fix the issue, but seems another error reported in arrow pkg.
To be honest, the error message from this tool is totally UN-understandable...Very hard to find the clue to fix the issue

Very hard to find the clue to fix the issue

OK, though is it:

So maybe it's worth attempting to upgrade flake2 or transitively raising upstream?

I already fix this issue in #135
There report another one in arrow-rs, could you help identify the issue?
run in https://github.com/apache/arrow-rs
cargo minimal-versions test

I run this command in my project and in arrow-rs project, both report the same error

In #135 I see the test failing because nightly isn't installed...

What do you see if you run this, but replacing gcc with the location of the new failure?

When I run cargo +nightly update -Z minimal-versions to set the minimum versions allowed, and then cargo tree -i gcc and look for gcc, which seems to be the cause of the error, I find:

Updated the CI, but it will fail because I check the result locally
you can clone the arrow-rs or #135 branch

https://github.com/PRQL/prql/actions/runs/4395201277/jobs/7696819175
You can also check your pipeline log

https://github.com/wangfenjin/duckdb-rs/actions/runs/4395501064/jobs/7697404026

Any clue how to fix this?

https://github.com/wangfenjin/duckdb-rs/actions/runs/4395501064/jobs/7697404026

It looks like this is coming from the arrow crate. What requires that dependency?

I put an issue into arrow: https://github.com/apache/arrow-rs/issues/3849

The issue is I'm not sure it's their problem or it's because I misuse it. For the first issue reported of flate2 or gcc, actually it's our cargo.toml don't make the flate2 as optional
That's why I said the error message is too misleading

arrow-rs has the same issue when running the steps on their repo — so it is an issue there too!


ClickHouse connector: can not use () after from closure

the following sql statement can not be executed by ClickHouse
WITH
  "default_CUSTOMER" AS (
   SELECT
     "default_CUSTOMER"."C_ACCTBAL" "C_ACCTBAL"
   , "default_CUSTOMER"."C_ADDRESS" "C_ADDRESS"
   , "default_CUSTOMER"."C_COMMENT" "C_COMMENT"
   , "default_CUSTOMER"."C_CUSTKEY" "C_CUSTKEY"
   , "default_CUSTOMER"."C_MKTSEGMENT" "C_MKTSEGMENT"
   , "default_CUSTOMER"."C_NAME" "C_NAME"
   , "default_CUSTOMER"."C_NATIONKEY" "C_NATIONKEY"
   , "default_CUSTOMER"."C_PHONE" "C_PHONE"
   FROM
     (
      SELECT
        "default_CUSTOMER"."C_ACCTBAL" "C_ACCTBAL"
      , "default_CUSTOMER"."C_ADDRESS" "C_ADDRESS"
      , "default_CUSTOMER"."C_COMMENT" "C_COMMENT"
      , "default_CUSTOMER"."C_CUSTKEY" "C_CUSTKEY"
      , "default_CUSTOMER"."C_MKTSEGMENT" "C_MKTSEGMENT"
      , "default_CUSTOMER"."C_NAME" "C_NAME"
      , "default_CUSTOMER"."C_NATIONKEY" "C_NATIONKEY"
      , "default_CUSTOMER"."C_PHONE" "C_PHONE"
      FROM
        (
         SELECT
           "C_ACCTBAL" "C_ACCTBAL"
         , "C_ADDRESS" "C_ADDRESS"
         , "C_COMMENT" "C_COMMENT"
         , "C_CUSTKEY" "C_CUSTKEY"
         , "C_MKTSEGMENT" "C_MKTSEGMENT"
         , "C_NAME" "C_NAME"
         , "C_NATIONKEY" "C_NATIONKEY"
         , "C_PHONE" "C_PHONE"
         FROM
           "default"."CUSTOMER" "default_CUSTOMER"
      )  "default_CUSTOMER"
   )  "default_CUSTOMER"
)
, "default_ORDERS" AS (
   SELECT
     "default_ORDERS"."O_CLERK" "O_CLERK"
   , "default_ORDERS"."O_COMMENT" "O_COMMENT"
   , "default_ORDERS"."O_CUSTKEY" "O_CUSTKEY"
   , "default_ORDERS"."O_ORDERDATE" "O_ORDERDATE"
   , "default_ORDERS"."O_ORDERKEY" "O_ORDERKEY"
   , "default_ORDERS"."O_ORDERPRIORITY" "O_ORDERPRIORITY"
   , "default_ORDERS"."O_ORDERSTATUS" "O_ORDERSTATUS"
   , "default_ORDERS"."O_SHIPPRIORITY" "O_SHIPPRIORITY"
   , "default_ORDERS"."O_TOTALPRICE" "O_TOTALPRICE"
   FROM
     (
      SELECT
        "default_ORDERS"."O_CLERK" "O_CLERK"
      , "default_ORDERS"."O_COMMENT" "O_COMMENT"
      , "default_ORDERS"."O_CUSTKEY" "O_CUSTKEY"
      , "default_ORDERS"."O_ORDERDATE" "O_ORDERDATE"
      , "default_ORDERS"."O_ORDERKEY" "O_ORDERKEY"
      , "default_ORDERS"."O_ORDERPRIORITY" "O_ORDERPRIORITY"
      , "default_ORDERS"."O_ORDERSTATUS" "O_ORDERSTATUS"
      , "default_ORDERS"."O_SHIPPRIORITY" "O_SHIPPRIORITY"
      , "default_ORDERS"."O_TOTALPRICE" "O_TOTALPRICE"
      FROM
        (
         SELECT
           "O_CLERK" "O_CLERK"
         , "O_COMMENT" "O_COMMENT"
         , "O_CUSTKEY" "O_CUSTKEY"
         , "O_ORDERDATE" "O_ORDERDATE"
         , "O_ORDERKEY" "O_ORDERKEY"
         , "O_ORDERPRIORITY" "O_ORDERPRIORITY"
         , "O_ORDERSTATUS" "O_ORDERSTATUS"
         , "O_SHIPPRIORITY" "O_SHIPPRIORITY"
         , "O_TOTALPRICE" "O_TOTALPRICE"
         FROM
           "default"."ORDERS" "default_ORDERS"
      )  "default_ORDERS"
   )  "default_ORDERS"
)
SELECT tmp.*
FROM
  (
   SELECT
     "C_NAME"
   , SUM("O_TOTALPRICE") "total_revenue"
   FROM
     ("default_CUSTOMER" "C"
   INNER JOIN "default_ORDERS" "O" ON ("C"."C_CUSTKEY" = "O"."O_CUSTKEY"))
   GROUP BY "C_NAME"
   ORDER BY "total_revenue" DESC
)  tmp
LIMIT 1

after removing the () after FROM, the SQL can be execute
WITH
...
SELECT tmp.*
FROM
  (
   SELECT
     "C_NAME"
   , SUM("O_TOTALPRICE") "total_revenue"
   FROM
     "default_CUSTOMER" "C"
   INNER JOIN "default_ORDERS" "O" ON ("C"."C_CUSTKEY" = "O"."O_CUSTKEY")
   GROUP BY "C_NAME"
   ORDER BY "total_revenue" DESC
)  tmp
LIMIT 1

Note: All the TO-MANY calculated fields can not be supported by the ClickHouse connector for now.

I have filed an issue in https://github.com/tobymao/sqlglot/issues/3735 but they don't have plan to fix it currently. So, we're going to close this issue after #649 .


PRL-687 MIAM Screen Refinement

Before creating a pull request make sure that:

[x] commit messages are meaningful and follow good commit message guidelines
[ ] README and other documentation has been updated / added (if needed)
[x] tests have been updated / new tests has been added (if needed)

Please remove this line and everything above and fill the following sections:
JIRA link (if applicable)
https://tools.hmcts.net/jira/browse/PRL-687 (MIAM details screen )
Change description
Does this PR introduce a breaking change? (check one with "x")
[ ] Yes
[ ] No

I wasn't aware of the  test team confirmation about these changes , raised a pull request accidentally closing it accordingly.


V22F-906 incorrect intervention

the fix was to update the struct in SDALib, since these values got added to DAISI as well, The clientbb PR with the same ID is also needed for this

Merging with Sjoerd's permission


To show scrollbar in float windows

Scrollbars don't show in float windows such as Lazy, Mason or LspInfo.
I didn't find any option to control it. Is this configurable? If not, could you please add it as a feature?

"I didn't find any option to control it. Is this configurable?"

Hi @huge-pancake, it's not currently possible for scrollbars and/or signs to appear in floating windows. There's no configuration option for that functionality.

"If not, could you please add it as a feature?"

I'll consider adding this to a future version of the plugin. If it were a simple update, I'd add it quickly, but I believe there are various assumptions throughout the code that scrollbars don't appear in floating windows.

I see, hope things work out soon :smile_cat:

I've added an experimental option, scrollview_floating_windows, that can be set to v:true to enable scrollview for floating windows.
For floating windows that use nvim_win_set_hl_ns, the background color of scrollview signs will be incorrect (#105).
VimScript
:let g:scrollview_floating_windows = v:true

Lua
-- with vim.g
vim.g.scrollview_floating_windows = true,

-- with a setup() function
require('scrollview').setup({
  ...
  floating_windows = true,
  ...
})


Allow extension of the Native Library Resolver / Locator

I'm writing a plugin that deals with the native side of gradle, in particular, dependency management.
In order to make linking libraries provided by the plugin easier on the user-side, I'm attempting to make these libraries available through the Map notation:
lib library: "<name>"
After some digging, I found that the code responsible for finding these libraries is the job of the DefaultLibraryResolver, which in turn uses an instance of the LibraryBinaryLocator interface to find the binaries matching the given name. These are internal classes, of course.
Expected Behavior
To be able to add a LibraryBinaryLocator (or some other suitable interface that isn't internal) to this process in order to locate libraries registered by plugins that don't fit into the prebuilt library infrastructure.
Current Behavior
No such ability exists, to my knowledge.
Context
The plugin in question fetches native libraries from maven, or other sources. These binaries are not able to be registered as prebuilt libraries as they carry some extra attributes we need to access during compile. Being able to add a locator for these libraries would be the best way to ensure they get used in a fashion users are familiar with (the Map notation)
Steps to Reproduce (for bugs)
N/A
Your Environment
N/A

Thanks @JacisNonsense for the feature request.
I'm assuming you are using the software model native plugin. Due to a change of direction regarding the software model, we are introducing new native plugins. Those new plugins use Gradle's dependency management and can already pull binaries from Maven. We still have some polishing to do with those plugin before making them stable, but I still encourage you to have a look at the native samples using those sample and see if that can solve your issues.
I'm going to close this issue and point you to the Gradle native repository to get a feel of any opened issues that you may require for your project to work on the new plugin. Don't hesitate to vote on the issues that are important to you.


Jdm/bump pq

Thanks for your help! We finished this in https://github.com/chef/automate/pull/2858 and some other PRs


[Suggestion] Post Categories

I thought this would be a nice progression for this project, I'm typically finding that I want a category for a blog post and not just a list of tags
I understand that tags can be used in it's place

I think having tags is enough, you can assign one or many tags to your post :)

@themsaid I'm new to Wink, so please bear with me. As for categories, they can be parts of fancy URLs like my.blog/travel/trip-to-cyprus, or: my.blog/recipes/my-favorite-soup, where travel and recipes are category slugs.

@Lloople Sorry, but semantically tags are other thing than categories: category is something broad, like "travel", "gadgets", "clothing", "recipes"; tags, on the other part, are useful for searching, and one tag per post has kinda little sense.

Ignore the wording, use tags as categories or as tags, wink isn’t enforcing how you do it 🙂
Sent from my iPhone

On Nov 15, 2018, at 10:14 PM, Andre Polykanine<EMAIL_ADDRESS>wrote:
@Lloople Sorry, but semantically tags are other thing than categories: category is something broad, like "travel", "gadgets", "clothing", "recipes"; tags, on the other part, are useful for searching, and one tag per post has kinda little sense.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

What i have done, is hardcoding in an array which are the tags i use as categories, and then in a foreach loop thru the tags, display as the category the first tag that is in that array and if not call it "general" category, not "pretty", requires you to be careful when adding tags, but its the only way i can think about, as i have several tags per post and then categories.
Seems like we have gotten to use to Wordpress and its categories and tags, i believe Ghost does the same as well? 😀

I like this package but I can not use because there is no category functionality.
Tag is fine. Let' me explain
I have blog I am posting things like
news, snippets, tutorials etc => this are categories or we can say type.
Tag like laravel ,vuejs ,react etc can be used in any post. but Type is only one for per post.
Post => belongs to one => Category/Type/OrList
Post => many to many => Tags

Agreed!


[issue] Support a mask in hostnames

Thank for your awesome project. Will your project support a mask in hostnames, like test.com,*.test.com.
Thank

@spravo I developed this feature at version 2.3.3
I configured that a bit different, you just have to put . front the domain like .github.com and not *.github.com
Please check it solves your requeriments

@mageddo, I checked.
You did awesome work, thank you


Very slow resizing when lots of content is in output panel

Steps to Reproduce:

Set "typescript.tsserver.trace": "verbose",
Work in vscode code base for a while in order to generate a lot of TS logs
Open output panel on bottom
Resize the sidebar to trigger a resize of the output panel

bug
Very bad resizing performance. Closing the output panel fixes this. Here is the log:
Profile-20171213T161858.json.txt
Main culprit seems to be splitLinesCollection._constructLines:

When changing the width of a word-wrapped editor, the wrapping points have to be recomputed.
So, the entire buffer is walked over, character-by-character to recompute the wrapping points.
How large was the TypeScript log (number of lines, size on disk?)

~3MB at 50,000 lines or so. Some lines be very long however
For TS specifically, we usually only care about the most recent ~1000 lines. Any way to easily limit the buffer size

@mjbvz this comes close to the output log viewer feature request we have been discusing in the standup
fyi @sandy081

Duplicate of #21196


Saving file output correctly

Hey I was wondering how I'm am supposed to use the -o command correctly. I get the file output but that file seems have issues. The file is always encodes as h264 MPG-4 part 10. What file extention should I be using to save these files? .mp4? Any web based video player (tried all the common jquery players and HTML 5 video tags) can't load the file, I read in another issue thread that the file could be missing metadata (seeking information)? The output files load fine in VLC and windows media player regardless of extention once downloaded locally.
Tldr:  How can I "fix" a video recoded from an HLS twitch stream to be able to be read by a web based video player (flowplayer, jPlayer or HTML 5 video embed) and what file extention should I be using for these files?

I can't insert the "  |  " in win shell.. any ideas?


support multiple glob patterns for collectCoverageFrom

This PR address issue #5198.
Summary
Improves the behaviour of collectCoverageFrom CLI option by allowing passing multiple glob patterns.
For example:
jest --collectCoverageFrom foo/**/*.js bar/**/*.js

Test plan
I added a simple integration test that covers this use case.

Codecov Report

Merging #5537 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #5537   +/-   ##
=======================================
  Coverage   61.71%   61.71%
=======================================
  Files         213      213
  Lines        7149     7149
  Branches        4        4
=======================================
  Hits         4412     4412
  Misses       2736     2736
  Partials        1        1

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 55c342a...f251fe7. Read the comment docs.


Control startup order / readiness check for services

Firstly, so awesome to see this project, i was real sad to see project tye get fewer and fewer updates because i found it really useful and perhaps took a larger dependency on it that you should for a preview project :)
For tye and our current, rather hacky local development orchestration we have an issue where some services need to be running for other services to start correctly.
It would be great if aspire started the projects in the order they where defined and also had the option to wait for health checks to pass before starting the next service
Perhaps this is already the case , if so, my appologies, but i didnt find anything in the docs about that. i guess it should already have the data to work like this since the services are defined in sequence in code and also health checks and dependencies can be defined. i guess some services wont have health checks in the same sense, like a database container for example, so maybe they need to have custom method supplied to see if they are ready

Controlling startup order of services is something we generally do not want to offer be because notion does not exist in reality when you deploy. There are a few tasks and jobs that you do want to run in some order (migrations for example), but we’re not convinced that adding startup order generally is a positive .

I agree that that is not the typical case in production but when running locally i dont think it uncommon for the database to not be running when you start, where as in production that would be very unusual.
Since the information about dependencies is already it seems friction could be reduced by using it :)
I guess the issue could be mitigated by having restart policies for services, that typically does exist in production but is not something that aspire does at the moment as far as i understand

In the next release, these startup errors will go away for the most part because we have a proxy between services that we can make work without your code having to retry or delay.
That said, aspire components are resilient to connection failures by default and retry connections for transient failures (up to some limit). In a way it makes sure you are building with an understanding that the network can fail at any time and should recovery from transient errors.

In the next release, these startup errors will go away for the most part because we have a proxy between services that we can make work without your code having to retry or delay.
That said, aspire components are resilient to connection failures by default and retry connections for transient failures (up to some limit). In a way it makes sure you are building with an understanding that the network can fail at any time and should recovery from transient errors.

Is this applicable to SQL container and running migrations? Currently, we do not use EF migrations. With docker-compose, we are able to control startup order so the migration project doesn't run until the SQL container is ready. For now, added a delay to the migration project to ensure the sql container is ready

It would be nice if we can toggle containers created in AppHost as a required dependency before projects are started. For us, a SQL container is only applicable for local dev. When deployed, our apps are targeting Azure SQL and don't have the same issue. Migrations run in our CI/CD pipeline, and we will retry the pipeline if it fails.

Here's a spike for this feature based on the latest preview (6 at the time of writing) https://github.com/davidfowl/WaitForDependenciesAspire

Here's a spike for this feature based on the latest preview (6 at the time of writing) https://github.com/davidfowl/WaitForDependenciesAspire

I've tested it in our applications (used it to wait for database setup/seeding across Postgres, Redis and Kafka topics before services start up so they are all in a known state) for local development and it worked really well. Have you considered spinning that out into a nuget as an optional Aspire feature for local development?

Adding to this just to point out how helpful it would be for #4177. Local LLM hosting may involve downloading many-gigabyte models before startup if not already cached locally, which can easily take 10+ minutes depending on connection speed. If developers don't realise this is going on, the only indication they might get is errors from dependent projects that are trying to call the local LLM service. They will likely start debugging and rechecking configuration, and may shut down the AppHost many times in the process which stops the download, when all they need to do is just wait.
Similarly, waiting for readiness could be considered a prerequiste for a data seeding mechanism. For example, seeding a Qdrant vector DB can take a minute or two even with just 10k records or so. If you don't realise what's going on and just see it as errors, you'll waste a lot of time debugging.

This happens all of the time with integration tests. We need something that causes blocking on database spin up/readiness etc. for development which can be ignored in production: .WaitForReadyDev() or something so it's really obvious what it's doing.

I was trying to test this with Dapr and RabbitMQ but even it stills doesn't solves the issue, it seems like Dapr is started in a different way with the sidecars first and the sidecars fails because the rabbitmq is not responding to the requests at the time that the sidecar is trying to connect to it.
@davidfowl Any hints on how to solve that with Dapr and RabbitMQ ?
This is how I was trying to setup:
var builder = DistributedApplication.CreateBuilder(args);
var rabbitMq = builder.AddRabbitMQ("event-bus").WithHealthCheck();

builder.AddDapr(o =>
{
    o.EnableTelemetry = true;
});

var daprRabbit = builder.AddDaprComponent(name: "pubsub", type: "pubsub.rabbitmq",  new DaprComponentOptions()
{
    LocalPath = "../Services/Infrastructure/Dapr/"
}).WaitFor(rabbitMq);

//Other initialization parts here

var accountService = builder.AddProject<Projects.Ms_Account>("account-service")
    .WithEndpoint(name: "account", port: 8002, scheme: "http")
    .WithReference(accountDb)
    .WithDaprSidecar(new DaprSidecarOptions
    {
        AppId = "account-service",
        DaprGrpcPort = 50002,
        DaprHttpPort = 3502
    })
    .WithEnvironments("SystemSettings")
    .WithReference(daprRabbit)
    .WithReference(rabbitMq)
    .WaitFor(postgres)
    .WaitFor(rabbitMq);

Yes, this is required here too.
When I use docker compose and depends_on, the container startup will wait until the dependency is up and running. When a container has a configured healthcheck, then the dependent containers will only start up when the healthcheck of the dependency returns fine.
That way, i.e. a migrations container can wait until the database is up and running, the API can wait for the migrations and the container that has the data seed script (which uses the API) can wait until the API reports back healthy, and all other services that need the data can wait for the seed container to be started up.
So, this "wait for a certain service to report back healthy until this is started" just like in docker compose would be really great. There are lot of scenarios that would be enabled with that functionality.

We also possibly have a need for this. For local development, we have to incorporate a custom identity server container whose implementation is not entirely under our control. It does not participate directly with aspire service discovery other than injecting connection strings to the database. This identity server implementation reads configuration data from the database on startup (outside of our control), so it must be able to connect to the database on startup. The database container does have a health check defined, and we use a docker compose dependency to ensure the database container is up, running, and healthy before starting the identity server container.
We could probably get away with using a longer retry/timeout setting for the connection string (although this would be more fragile in practice than waiting for a health check), but the default Aspire behavior when adding a SqlServer resource reference is to generate a connection string without any retry/timeout settings. So, we'd have to customize the connection string, and that's a bit more complicated than it needs to be just to support (unavoidable) resource dependencies for local development.

Controlling startup order of services is something we generally do not want to offer be because notion does not exist in reality when you deploy. There are a few tasks and jobs that you do want to run in some order (migrations for example), but we’re not convinced that adding startup order generally is a positive .

@davidfowl So what's the solution for running migrations right now? I tried to sample from playground but I stuck:
Npgsql.PostgresException (0x80004005): 57P03: the database system is starting up

Controlling startup order of services is something we generally do not want to offer be because notion does not exist in reality when you deploy.

Unless you're using initcontainers, which EF migrations is a perfect example of when you might use init containers.

Controlling startup order of services is something we generally do not want to offer be because notion does not exist in reality when you deploy.

Unless you're using initcontainers, which EF migrations is a perfect example of when you might use init containers. Init containers are also a thing in azure container apps

They're also a thing in Kubernetes and every other cloud platform that does orchestration. Aspire needs to be able to model these.

Closing this in favor of: https://github.com/dotnet/aspire/issues/5275


Can I decorate a path operation function

First check

[x] I added a descriptive title to this issue.
[x] I used the GitHub search to find a similar issue and didn't find it.
[x] I searched the FastAPI documentation, with the integrated search.
[x] I already searched in Google "How to X in FastAPI" and didn't find any information.
[x] I already read and followed all the tutorial in the docs and didn't find an answer.
[x] I already checked if it is not related to FastAPI but to Pydantic.
[x] I already checked if it is not related to FastAPI but to Swagger UI.
[x] I already checked if it is not related to FastAPI but to ReDoc.

Description
I want to decorate my path operation function using a custom decorator. I want to check for some information within the Request body for every request and based on the information, Permit or disallow the user from accessing certain endpoints.
I'm wondering if this can be done, And if so, How? I tried a few options but I'm not able to get it working as expected.
This is my decorator function:
def decorator(permissions):
    def wrapper(function, *args, **kwargs):
        async def executor():
            return await function(*args, **kwargs)
        return executor
    return wrapper

And this is how I am using it

@apiRouter.get("/example-end-point")
@decorator(["CAN_DO_SOMETHING"])
async def do_something():
    """
    Do whatever needs to be done
    """
   return {"payload": {}}

I am able to decorate the function in the sense that the do_something function executes every time the end-point is called, along with decorator code. BUT, I face problems such as

Decorating the path operation function prevents me from utilizing FastAPI functionality. For example, Adding a mandatory query parameter to the above API,

async def do_something(query: str)

Returns an ASGI Internal Server Error instead of a 422 / Validation Error; EVEN when I pass the query parameter.
I am assuming that the query parameters are not "passed" to the end-point function triggering this error, but I'm curious as to why FastAPI does not handle it with a 422, instead throwing an ASGI Exception with a TypeError as
TypeError: do_something() missing 1 required positional argument: 'query'

A cryptic warning pops up at times with the below stack trace

WARNING: Executing <Task finished coro=<RequestResponseCycle.run_asgi() done, defined at /project/env/lib/python3.7/site-packages/uvicorn/protocols/http/httptools_impl.py:373> result=None created at /project/env/lib/python3.7/site-packages/uvicorn/protocols/http/httptools_impl.py:261> took 3.402 seconds

which I assume occurs when a finished co-routine object is awaited.
I tried searching online but could not find much regarding fastapi custom decorators OR the warning <Task finished...done>.
Would appreciate any advice I get.
Thanks a bunch!
Environment

OS: Ubuntu 16:
FastAPI Version [e.g. 0.59.0]:
Python version: 3.7.7

Try using built-in functools.wraps() that preserves function signature. In your case I would rewrite code like that:
from functools import wraps

def decorator(permissions):
    def outer_wrapper(function):
        @wraps(function)
        async def inner_wrapper(*args, **kwargs):
            return await function(*args, **kwargs)

        return inner_wrapper

    return outer_wrapper

Try and let me know if it helps.

@HarrySky Using functools.wrap() solves my problem completely! Thanks a bunch!
Also, Would you have any idea why using functools solves my problem? From what I understand, using functools preserves my function signature, But would the function signature be something that is depended on by FastAPI?

@subritc as far as I know, FastAPI checks endpoint function signature on startup so that it knows what kind of data should be provided to the endpoint, that's why it was not providing query: str argument - it simply did not detect it :smile:

Got it. Without the functools, FastAPI was throwing a 422 for parameters *args and **kwargs which was the signature of the wrapping decorator function, But using functools preserves the signature for my endpoint function.
Thanks a lot @HarrySky :)

Another great way of doing such kind of things, is using the amazing depends. You can create an simple function and tell that a request depends on that function execution, that is, doing checks or all that you need before executing inner_wrap.
PS: you call use depends beautifully by:
permissions = Depends(auth)

@router.get(
    "/path",
    depends=[permissions]
)

Thanks for the help here @HarrySky! :clap: :bow:
And yeah, all what @HarrySky said :point_up_2:
Thanks for reporting back and closing the issue @subritc  :+1:
Also, as @gabriel-viviani says, depends would be the best way to go to handle shared logic like permissions.


Use cloudpickle instead of pickle

I am writing some unit tests using uvtrick. If you define objects inside a function scope, pickle can't pickle them.
cloudpickle  "makes it possible to serialize Python constructs not supported by the default pickle module from the Python standard library."
Shall we use that instead? Happy to provide a PR if you want.

Here's an example error message you get when not using cloudpickle:
Traceback (most recent call last):
  File "/tmp/tmp311h9dlh/pytemp.py", line 55, in <module>
    args, kwargs = cloudpickle.loads(Path('/tmp/tmp311h9dlh/pickled_inputs.pickle').read_bytes())
                   ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
ModuleNotFoundError: No module named 'tests'

I have been using cloudpickle happily in many projects. The only downside is the extra dependency (see the PR)

No experience with dill yet

Closed via #23


Cyberpion logo update

@cyberpion-yotam When we check svg file in svglint, the logo is not come proper frame and color of logo also didnt get display.

you can change the viewbox (viewBox="0 0 2092 2012") value of the logo to make fit in frame.
Thanks,
Aditi Maheshwari

@v-admahe
I changed it so you can clearly see in the logo in the linter you sent


ESP-IDF version 4.0 (IDFGH-3472)

Good Morning,
I am using ESP-IDF version 4.0 and I don't find adc.c file. I want to find the body of the function adc_i2s_mode_init(). In another versions like 4.1 and 4.2, this function is included in the adc.c file but I don't know how it works in 4.0 version.

It is here, just in a different file:
https://github.com/espressif/esp-idf/blob/4c81978a3e2220674a432a588292a4c860eef27b/components/driver/rtc_module.c#L1383-L1415

Thanks for reporting, feel free to reopen if you still have the issue. Thanks.


Invisible FOBs

When I resume a multiplayer game all the FOBs still exist in game logic but they are not visible (and there is no way to collide with them). But as I said: all the functions are available further ...
Sorry if it was reported already.
Happens without mods too.
It happens every time I resume a saved mission. A FOB I/we built is only visible in the same session.

Please report which version you play, which map, from where you downloaded the missionfile, which preset you use, which exact edits you've made and it would be also the best if you provide one server.rpt where the issue happen.
With this information it's much easier to help. Without it starts every time with a question and answer game. ;)


Example is at the wrong position in the text

This is in the text:
Remarks
[...]
Do not use a semicolon as a statement terminator after GO.

Permissions
GO is a utility command that requires no permissions. It can be executed by any user.

[the following is a code example]
-- Yields an error because ; is not permitted after GO
SELECT @@VERSION;
GO;

The example should be after the last sentence in Remarks. It has nothing to do with Permissions and should be moved up two paragraphs.

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

ID: 960815fc-a05d-b887-e5a8-e4aef6761000
Version Independent ID: 4af0d0ae-c858-9b92-3058-a2e43f60bd8d
Content: GO (Transact-SQL) - SQL Server
Content Source: docs/t-sql/language-elements/sql-server-utilities-statements-go.md
Product: sql
Technology: t-sql
GitHub Login: @rothja
Microsoft Alias: jroth

@kaesaecracker Thank you for the feedback.
The change has been made and merged.


ENH: fly will return uid

Description

This PR make fly function return uid instead of None
Motivation and Context

https://github.com/bluesky/bluesky/issues/1332
How Has This Been Tested?

In the context of bluesky tutorial  binderFly Basic section  https://try.nsls2.bnl.gov
Script:
import bluesky.plans as bp
def custom_fly(flyer):
    uids = (yield from bp.fly(flyer))
    print('uid: ',uids)
    return [uids]

def get_data_using_uid(uids):
    for uid in uids:
        print(db[uid].table(db[uid].stream_names[0]))

def big_plan(flyer):
    uids = (yield from custom_fly(flyer))
    get_data_using_uid(uids)

RE(big_plan([ifly]))

Output:
uid:  d5466512-b070-438a-8643-39ec6fe58ee4
                                 time         x
seq_num
1       2020-04-30 21:56:19.784460068  0.003487
2       2020-04-30 21:56:19.784499884  0.003527
3       2020-04-30 21:56:19.784512043  0.003539
4       2020-04-30 21:56:19.784520864  0.003548
5       2020-04-30 21:56:19.784529924  0.003557
('d5466512-b070-438a-8643-39ec6fe58ee4',)

Thanks for tackling this, @ke-zhang-rd!


Google pay shows processing forever if google pay button is embedded in iframe and webpage is viewed on mobile

Describe the bug A clear and concise description of what the bug is.
I have the google pay button working on a form, but if that form is embedded on another page and that page is being viewed on a mobile device the google window never loads credit cards and instead shows a loading spinner and says processing forever
To Reproduce Steps to reproduce the behavior:

Load google pay react button within an iframe on mobile device
Click on google pay button
google pay window hangs on "processing" view

Expected behavior A clear and concise description of what you expected to happen.
google window finishes processing and shows credit cards
Screenshots If applicable, add screenshots to help explain your problem.

Component information:

Component

[X] React component (@google-pay/button-react)
[ ] Custom element (@google-pay/button-element)
[ ] Angular component (@google-pay/button-angular)

Component version (e.g. 1.0.0):

Environment:
only appears to occur on chrome from a mobile view (whether an actual phone or using the mobile view from dev tools)
Google button must be embedded within an iframe.
Was using the react google pay button in the TEST environment

@jacob-pecile-CH do you still experience this issue? If yes, could you maybe send a link where I can reproduce the issue?

Closing due to inacticity.


flash: Add program/erase parallelism support for STM32F4x

The implementation uses the same approach as STM32F1x.
Program/erase speed can be set by setting 'write-block-size' flash property to 1, 2, 4 or 8.

Wouldn't it be most efficient for the driver to select the largest possible write size?

Max write size depends on MCU voltage.

every buffer address and size needs to be two bytes aligned

Unaligned buffer is handled with memcpy() in 224-226. The compiler emits very efficient code - for 4 byte write, it's just 4 instructions (logical or with logical shift). The size and flash offset need to be aligned as it is in flash_stm32f1x.c.

Wouldn't it be most efficient for the driver to select the largest possible write size?

Max write size depends on MCU voltage.

Hm. Interesting, could not find the info in STM32F4 docs, but not a platform specialist here.

every buffer address and size needs to be two bytes aligned

Unaligned buffer is handled with memcpy() in 224-226. The compiler emits very efficient code - for 4 byte write, it's just 4 instructions (logical or with logical shift). The size and flash offset need to be aligned as it is in flash_stm32f1x.c.

You have two alignments here: source and destination; memcpy aligns source buffer,  you could also use here UNALIGNED_GET .
What I mean by changing write alignment is that it is probably most efficient, although I do not know the platform, to write by the highest possible chunk.
For example when you have 'write-block-size` == 2, then it is expected that user will give you offset and size of buffer aligned to 2;
but notice that buffer aligned to 2 will, if the length is >4, at some point align to 4 and even to 8, if size is >8.
Because you control the size, by which the data is being copied it chunks, at run-time, you can change it as you wish.
For example N length buffer, aligned at 2, may be copied by sizes 2 + 4 +  N * 8 +  4 + 2, or 2 + (N + 1) * 8 + 2.

Wouldn't it be most efficient for the driver to select the largest possible write size?

Max write size depends on MCU voltage.

Hm. Interesting, could not find the info in STM32F4 docs, but not a platform specialist here.

It's in the STM32F412 reference manual RM0402 Rev 6, 3.5.2 Program/erase parallelism.

You have two alignments here: source and destination; memcpy aligns source buffer, you could also use here UNALIGNED_GET

Is UNALIGNED_GET preferred over memcpy()?

What I mean by changing write alignment is that it is probably most efficient, although I do not know the platform, to write by the highest possible chunk.

Yes, it would be the most efficient, but in this case, the chunk size depends on applied voltage, so the write-block-size property is a maximum chunk size (we trust that the user read the manual).
Fun fact:
While configuring number of flash wait states (3.4.1 Relation between CPU clock frequency and Flash memory read time), Zephyr silently assumes that the voltage is between 2.7V - 3.6V. I haven't checked it, but I suspect that Zephyr won't boot when low voltage is applied to the MCU.

Wouldn't it be most efficient for the driver to select the largest possible write size?

Max write size depends on MCU voltage.

Hm. Interesting, could not find the info in STM32F4 docs, but not a platform specialist here.

It's in the STM32F412 reference manual RM0402 Rev 6, 3.5.2 Program/erase parallelism.

Thanks, now I can see this. Very interesting.

You have two alignments here: source and destination; memcpy aligns source buffer, you could also use here UNALIGNED_GET

Is UNALIGNED_GET preferred over memcpy()?

Check what will give you better results. I suspect that UNALIGNED_GET will give you smaller footprint, smaller stack usage and quicker execution as it does not require doing all the function call/return stuff.

What I mean by changing write alignment is that it is probably most efficient, although I do not know the platform, to write by the highest possible chunk.

Yes, it would be the most efficient, but in this case, the chunk size depends on applied voltage, so the write-block-size property is a maximum chunk size (we trust that the user read the manual).

Nice to know I am not the only one.

Fun fact: While configuring number of flash wait states (3.4.1 Relation between CPU clock frequency and Flash memory read time), Zephyr silently assumes that the voltage is between 2.7V - 3.6V. I haven't checked it, but I suspect that Zephyr won't boot when low voltage is applied to the MCU.

I guess user has to check that before sending board to the field.

You probably want to update copyrights.

LGTM. I have left comment on possible usage of `UNALIGNED_GET, but that is up to the committer to validate.

Clang compiles it to the same instructions. It's just a decision between
value = UNALIGNED_GET((flash_prg_t *)data + i);

and
memcpy(&value, (const uint8_t *)data + i * sizeof(flash_prg_t), sizeof(flash_prg_t));

I must admit that UNALIGNED_GET looks nice, but let the code owner decide :smile:

You probably want to update copyrights.

Google is already in copyrights. Should I change the year?

ST people decide then.

UNALIGNED_GET is already in use in some others STM32 flash drivers, and personally I like it, so indeed, would be a good move.

Changed memcpy() to UNALIGNED_GET().


seTimeoutInterval crash

[self.mutableObservedChangedKeyPaths addObject:keyPath];    when it comes to this code,it crashes,and that is when it was running after setTimeoutInterval;

I have this problem too. Have AFNetworking resolved it?
btw:I writed the network request code in custom thread.

For general help, please ask on StackOverflow.

Me too, nobody fix it?


tgpt: 1.10.0 -> 2.0.2

Diff: https://github.com/aandrew-me/tgpt/compare/refs/tags/v1.10.0...v2.0.2
Changelog: https://github.com/aandrew-me/tgpt/releases/tag/v2.0.2
Description of changes

Things done

Built on platform(s)

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

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

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

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

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

[ ] Fits CONTRIBUTING.md.

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

1 package built:

tgpt


Automatic clean up of old files?

vcpkg\downloads and other maps are full of old files, is there a way to clean them up automatically?
Ideally, Windows would support a Useful, but may be deleted flag on files for purposes like this. Perhaps you could forward this feature request? ;)
A Uesless, should be deleted flag would be nice as well while we're at it.

One idea that might save a little space, have an option to remove the .obj files from the buildtrees folder for builds that successfully install. The source and build tools would still be available for reference or if a change is required. I don't see much value in keeping .obj files after a successful install and it may be a way to save some space. The more general issue of removing old files would probably require the need to track downloads and builds processes from older versions of the vcpkg and portfiles, making things less clean then the current vcpkg design. I sometimes do a clean build from an new vcpkg of the libraries I use. If successful and it passes some tests, I archive the older vcpkg directory and remove after a sufficient time.

Iirc you can just delete the build trees folders without problems.

Hi @OlafvdSpek, thanks for reporting this issue!
You can manually delete the port folder that is not required to be archived under buildtrees and the port source package that does not need to be archived under downloads.

I know, but manual != automatic. ;)

Just wanted to add my +1 to this issue - I'm currently installing the Point Cloud Library and my vcpkg repo is up to +14GB and still climbing. Out of that there is only around 750MB of downloads and another 1.7GB of built/installed binaries... Is it safe to prune the entire buildtrees directories once builds are completed assuming I'm not planning on reading through a couple hundred thousand lines of peripheral code?

I recommend using the --clean-after-build flag.
PS D:\vcpkg\vcpkg> .\vcpkg.exe help install
Example:
  vcpkg install zlib zlib:x64-windows curl boost

Options:
    --dry-run                                Do not actually build or install
    --head                                   Install the libraries on the command line using the latest upstream sources
    --no-downloads                           Do not download new sources
    --recurse                                Allow removal of packages as part of installation
    --keep-going                             Continue installing packages on failure
    --x-use-aria2                            Use aria2 to perform download tasks
    --clean-after-build                      Clean buildtrees, packages and downloads after building each package
    --x-xunit=...                            File to output results in XUnit format (Internal use)
    --triplet <t>                            Set the default triplet for unqualified packages
    --overlay-ports=<path>                   Specify directories to be used when searching for ports
    --overlay-triplets=<path>                Specify directories containing triplets files
    --vcpkg-root <path>                      Specify the vcpkg directory to use instead of current directory or tool directory
    --scripts-root=<path>                    Specify the scripts directory to use instead of default vcpkg scripts directory

@Rastaban On every invocation of install / upgrade? Not a solution IMO.

@OlafvdSpek, agree that there has to be an automatic solution, but in the meantime maybe this type of vcpkg.ps1 script could help by adding --clean-after-build command at the end on every passing of install?
#vcpkg.ps1
for ( $i = 0; $i -lt $args.count; $i++ ) {
  if ($args[$i] -eq "install") { # add extra default options to install
    $args += "--clean-after-build"	# Clean buildtrees, packages and downloads after building each package
  }
}
&'\path\to\vcpkg.exe' $args

I recommend using the --clean-after-build flag.
PS D:\vcpkg\vcpkg> .\vcpkg.exe help install
Example:
  vcpkg install zlib zlib:x64-windows curl boost

Options:
    --dry-run                                Do not actually build or install
    --head                                   Install the libraries on the command line using the latest upstream sources
    --no-downloads                           Do not download new sources
    --recurse                                Allow removal of packages as part of installation
    --keep-going                             Continue installing packages on failure
    --x-use-aria2                            Use aria2 to perform download tasks
    --clean-after-build                      Clean buildtrees, packages and downloads after building each package
    --x-xunit=...                            File to output results in XUnit format (Internal use)
    --triplet <t>                            Set the default triplet for unqualified packages
    --overlay-ports=<path>                   Specify directories to be used when searching for ports
    --overlay-triplets=<path>                Specify directories containing triplets files
    --vcpkg-root <path>                      Specify the vcpkg directory to use instead of current directory or tool directory
    --scripts-root=<path>                    Specify the scripts directory to use instead of default vcpkg scripts directory

As --clean-after-build would remove all downloaded files, it makes vcpkg download msys2 again and again 🤔


there is an bug on scss import

hey guys,
I installed node-sass, sass-loader to use sass in my project,
but when I try to use primer-markdown, there is an Bug
ERROR in ./~/css-loader!./~/vue-loader/lib/style-rewriter.js?id=_v-bb18f99e&scoped=true!./~/sass-loader?indentedSyntax!./~/vue-loader/lib/selector.js?type=style&index=0!./src/components/widget/Article.vue
Module build failed:
@import "primer-support/index.scss";
^
      File to import not found or unreadable: primer-support/index.scss
Parent style sheet: /Users/yozman/sysc/node_modules/primer-markdown/index.scss
      in /Users/yozman/sysc/node_modules/primer-markdown/index.scss (line 1, column 1)
 @ ./~/vue-style-loader!./~/css-loader!./~/vue-loader/lib/style-rewriter.js?id=_v-bb18f99e&scoped=true!./~/sass-loader?indentedSyntax!./~/vue-loader/lib/selector.js?type=style&index=0!./src/components/widget/Article.vue 4:14-298 13:2-17:4 14:20-304

the code is like
// Article.vue
<template lang="jade">
  div(:class.once="$options.name" v-html="innerHTML")
</template>

<script>
export default {
  data () {
    return {
      innerHTML: ''
    }
  },
  name: 'W__Article',
  props: { args: Object },
  watch: {
    'args.content' (val) {
      let scoped = this.$el.attributes[0].name
      this.innerHTML = val.replace(/(<[^/]+?)([> ])/g, `$1 ${scoped}$2`)
    }
  }
}
</script>

<style lang="sass" scoped>
@import ~primer-markdown/index.scss

.W__Article
  @extend .markdown-body
</style>

// node_modules/primer-markdown/index.scss
@import "primer-support/index.scss";
@import "./lib/markdown-body.scss";
@import "./lib/headings.scss";
@import "./lib/lists.scss";
@import "./lib/tables.scss";
@import "./lib/images.scss";
@import "./lib/code.scss";

when I change it to @import "~primer-support/index.scss" it works.
how can I fix it to works on my project?

This is not a bug. It's just how css-loader works.

@chrisvfritz
but the source is on package primer-markdown not mine,
primer-markdown require primer-support,
could u tell me what can I do in my project ?

You don't need to add the ~ in front of primer-support. You can instruct sass-loader to include all node packages in order to be able to resolve primer-support/index.scss properly.
webpack config:
const path = require('path');
const webpack = require('webpack');
const nodeModulesDirectory = path.resolve(__dirname, './node_modules');

module.exports = {
  .....
  module: {
    loaders: [
       { test: /\.css$/, loader: 'style-loader!css-loader' },
       { test: /\.scss$/, loader: `style-loader!css-loader!sass-loader?includePaths[]=${nodeModulesDirectory}`}
    ]
  }
};


Add speed/effort parameter

I believe this parameter is necessary for balancing between speed and quality.
Parameter references: https://github.com/strukturag/libheif/blob/2dff6b4282d5e03feb5bd8dea5b09710e635cf68/libheif/heif_encoder_aom.cc#L70

or maybe add a higher level API to set a custom parameter would be great

Before I merge I have some suggestion.
The most of users do not really know anything about speed/effort and how it works. Especially for AV1 where each encoder provides it own implementation of effors/speed.
So, my suggestion to introduce small brief description how it works, what direction will increase/decrease an effect and might be good to introduce speed parameter as enum in kotlin with readable names or otherwise protect the parameter with @IntRange(0, 20).

I add new enum value and fix the checking, in case of using x265 instead of aom I don't know much about the implementation of it

I add new enum value and fix the checking, in case of using x265 instead of aom I don't know much about the implementation of it

Actually you don't have to expose -1 to kotlin, you may pass it just in c++ to avoid suspicious enum values. However that's ok.  I think I'll do that changes myself it'll be easier


Updating Advanced Paste Docs

This PR includes doc updates for the new paste actions going into Advanced Paste in the upcoming PowerToys release.

Good catch! For now, a name with a timestamp is automatically generated.

I'm merging the suggestions so we can go live. We can circle back with updates later if necessary.


feat: add support for external materialization

@jwills I added tests too. So if you are ok with my answers to your comments, we can merge.

@tomsej ack, I just merged in @AlexanderVR's concurrency test fix that caused a conflict for you, sorry about that! Otherwise I am good to go, let's merge this and prep a new release!

NP, @jwills! I fixed that and merged this PR.


CS:GO: Move to lib/env

Make CS use lib/env for config.
Prepare config for reloading.
Refactor CS to follow same internal structure as PS and sciond
fixes #1835
fixes #1898
fixes #1910

This change is

go/cert_srv/main.go, line 77 at r2 (raw file):
Previously, Oncilla wrote…
Sure. Will do.

Agree with @lukedirtwalker, this would be really nice to have in the other services.

go/cert_srv/main.go, line 121 at r2 (raw file):
Previously, scrye (Sergiu Costea) wrote…
I noticed the commit text mentions this fixes #1910, but the reconnection function is not actually initialized. This is done via InitInfraEnvironment (which basically consists only of initializing the reconnecter).
You probably want something similar to how SD and PS do it:
environment = infraenv.InitInfraEnvironment(config.General.TopologyPath)

Wait. Issue #1910 is about IsCS bool not being set.
Do you mean #1898?
But that is about reconnecting to dispatcher.
infraenv.InitInfraEnvironment only sets the reload function, no?

go/cert_srv/main.go, line 72 at r7 (raw file):

		return 1
	}
	defer log.LogPanicAndExit()

Use env.CleanupLog


Add tags to generated lambdas

The template yaml defines the lambdas that needs to be deployed https://github.com/intuit/Trapheus/blob/master/template.yaml#L182 . We need to add tags as to those lambdas as shown here https://github.com/aws/serverless-application-model/blob/master/versions/2016-10-31.md#example-awsserverlessfunction .
We can add the tags of trapheus-<<functionName>>`` , for instance for RenameLambdaFunction the corresponding tag would betrapheus-rename```

@stationeros please assign this to me

@sandipnahak I have merged the PR now. Kindly star us to show some support


How can we get a unique id after vector recall

As far as I know, currently the only thing returned in SearchResult.NodeScore is node, a value that represents the index of the RandomAccessVectorValues.
How can we get a unique id after vector recall, such as the docid returned in ES, where docid can be customized at index time.

You can store a mapping between nodeId <-> docid in an external system (database, filesystem, in-memory hashtable etc).

You can store a mapping between nodeId <-> docid in an external system (database, filesystem, in-memory hashtable etc).

There isn't a separate place to store it in the index, so this seems to be the only way for now. I understand that the order of this index is supposed to be determined after the entire index is complete, how exactly should this be handled? I looked through the test cases and didn't see the corresponding code

nodeId is the ordinal of the vector value provided by RandomAccessVectorValues. It's already known upfront. You already provide an instance if RandomAccessVectorValues to JVector. You should just keep a mapping unique id for every ordinal.
But note that, if you delete some values then ordinals can change and you need to update the mapping too.

nodeId is the ordinal of the vector value provided by RandomAccessVectorValues. It's already known upfront. You already provide an instance if RandomAccessVectorValues to JVector. You should just keep a mapping unique id for every ordinal.
But note that, if you delete some values then ordinals can change and you need to update the mapping too.

In the way you said, I wrote a test case with the following code, please help me to see if it is correct or not
@Test
    public void testNodeDocIdMapping() {

        var sourceData = List.of(new TestVectorItem(new float[]{-1, -1}, 11L),
            new TestVectorItem(new float[]{1.5f, 1.4f}, 13L),
            new TestVectorItem(new float[]{0.9f, 0.9f}, 14L),
            new TestVectorItem(new float[]{1, 1}, 12L));

        Map<Integer, Long> nodeMap = new HashMap<>();
        List<VectorFloat<?>> rawVectors = new ArrayList<>();
        for (int i = 0; i < sourceData.size(); i++) {
            rawVectors.add(vectorTypeSupport.createFloatVector(sourceData.get(i).getVectors()));
            nodeMap.put(i, sourceData.get(i).getDocId());
        }

        var vectors = new ListRandomAccessVectorValues(rawVectors, 2);
        var builder = new GraphIndexBuilder(vectors, VectorSimilarityFunction.EUCLIDEAN, 2, 2, 1.0f, 1.0f);

        try (var graph = builder.build()) {
            var qv = vectorTypeSupport.createFloatVector(new float[]{0.5f, 0.5f});
            var results = GraphSearcher.search(qv, 4, vectors, VectorSimilarityFunction.EUCLIDEAN, graph, Bits.ALL);
            SearchResult.NodeScore[] nodes = results.getNodes();
            for (SearchResult.NodeScore nodeScore : nodes) {
                int node = nodeScore.node;
                float[] ff = (float[]) rawVectors.get(node).get();
                float score = nodeScore.score;
                System.out.println("float[0]=" + ff[0] + ", float[1]=" + ff[1] + ", node=" + node + ", score=" + score + ", docId=" + nodeMap.get(node));
            }
        }
    }

    public static class TestVectorItem {
        public float[] vectors;
        public long docId;

        public TestVectorItem(float[] vectors, long docId) {
            this.vectors = vectors;
            this.docId = docId;
        }

        public float[] getVectors() {
            return vectors;
        }
        public long getDocId() {
            return docId;
        }
    }

That looks reasonable to me.

That looks reasonable to me.

tks


Add load testing support

Use nbomber or k6

https://github.com/ForrestTech/Exemplum/commit/c1cb3e12fa20462b0609d06a415736e9082ba9ad


JXcore SM latest version isn't working on CI

We should only need this for the 3 tests and no where else. I'm hoping that thaliproject/Thali_CordovaPlugin#1025 will make this issue go away.


golang 1.19 tags aren't properly populated

This project now has go 1.19 tags, but the openssl branch hasn't been pushed up, and the tags aren't pointing at a ready to build golang source. So isn't currently working the way the previous tags and branches were.
Was this a mistake, or is there still work pending for the 1.19 releases?

@twk3 we've started to take a different approach from Go 1.19 forward.  Instead of hosting a full Go source tree with the openssl changes dispersed throughout, we've refactored all of the openssl code into its own standalone module, which can be vendored directly into an upstream Go tree.  This helps both with porting the changes to new major versions, as well as making the changes more clear between golang-fips and upstream Go.
The go1.19-fips-release branch and corresponding Go 1.19 tags contain the setup scripts and patches required to vendor the openssl module into an upstream Go 1.19 tree.  Currently there are two ways to set it up.
If your build environment has network accessibility, you can run the scripts/full-generate-repo.sh script to generate a Go tree that is patched to use the vendored openssl module.
The other option is to download the upstream Go sources and apply 000-initial-setup.patch and 001-initial-openssl-for-fips.patch directly to the upstream Go source tree whose version corresponds to the tag.
We probably need to document this more thoroughly.  Let's leave this issue open until the user experience is more clear.

@dbenoit17 thanks for clarifying
I went ahead and accounted for this in our build system. For now I'm using full-generate-repo.sh But it sounds like vendoring it in  and apply the patches would actually be better long term.
I think ideally an example of how the vendoring is intended to be pulled into the upstream checkout. And a new script for applying the patches (would currently just be the last part of the full-generate-repo scriot). In case patch sorting becomes important and is something that ends up accounted for in the script, seems better than having all the builders needing to see the problem and re-apply the sorting.


Logging macro incompatibility (OTP >= v21)

An issue arises if the new(-ish) logging macros (kernel/include/logger.hrl) are included alongside wf.hrl, specifically, ?LOG is redefined.  I cannot see a 100% backwards compatible solution to this problem as logger:?LOG accepts different arguments than wf:?LOG.  (It's entirely possible my understanding of how includes works is faulty or incomplete.)
If this is something that should be addressed in Nitrogen, I'd propose a solution like
%%% LOGGING %%%
-ifndef(debug_print).
-define(debug_print, true).

-ifndef(LOGGER_HRL).
-define(PRINT(Var), error_logger:info_msg("DEBUG: ~p~n~p:~p~n~p~n  ~p~n", [self(), ?MODULE, ?LINE, ??Var, Var])).
-define(LOG(Msg, Args), error_logger:info_msg(Msg, Args)).
-define(DEBUG, error_logger:info_msg("DEBUG: ~p:~p~n", [?MODULE, ?LINE])).

-else.
%% logger.hrl has been included
%% ?LOG has already been defined
-define(WFLOG(Msg, Args), ?LOG_INFO(Msg, Args)).
-define(PRINT(Var), ?LOG_INFO("DEBUG: ~p: ~p~n", [??Var, Var])).
-define(DEBUG, ?LOG_INFO("DEBUG: ~p:~p~n", [?MODULE, ?LINE])).
-endif.

-endif.

which allows for both hrl files to be included, in this order:
-include_lib("kernel/include/logger.hrl").
-include_lib("nitrogen_core/include/wf.hrl").

This is an incomplete (and not well tested) solution!  wf:info/1,2; wf:warning/1,2; and wf:error/1,2 will continue to use the legacy logging infrastructure.  Further, the alternate macros do not take advantage of the enhanced metadata the new logging facility provides (which would require, I think, a custom log formatter).
I'm happy to submit a PR addressing this issue, either along these lines or a better solution arising from discussion.  Before continuing however, I think it's important to acknowledge the compatibility issue that will arise and determine how it should be managed.  It seems to be worth noting that the incompatibility is triggered by the developer opting to include the new macros; the default logging should just work in the vanilla case.

Hey @bunnylushington, your proposed solution is good.
For the sake of consistency, I'd go with ?WF_LOG only because the other ?WF macro (?WF_IF) uses ?WF_ as the prefix instead of ?WF (without the underscore).
In this situation, I'm not super concerned with backwards compatibility being a problem, since it's a pretty easy s/LOG/WF_LOG kind of solution.

Hola @bunnylushington and @choptastic, is anyone working on this one? We are migrating some production pages to a new system built on Erlang and Nitrogen and just discovered this conflict. It would be really helpful to be able to use the OTP logger macros without having to refactor all the Nitrogen stuff into a separate module. I'd be happy to try my hand at a PR if that would help get the ball rolling, though I'm admittedly not sure how to address the deeper questions about wf:info et al using the legacy logging setup.


Addition of a Flutter Quickstart

Summary
Addiiton of a Flutter quickstart that contains a sample dashboard, 3 alerts, an install guide (that includes a sample project as getting flutter working with the native agents is an in-depth process), and some assets.

Pre merge checklist

[x] Did you check you NRQL syntax? - Does it work?
[x] Did you check your dashboard image quality? -  Do they look good?
[x] Did you check that your alerts actually work?
[x] Did you include an InstallPlan and Documentation reference?
[x] Did you check your descriptive content for voice, tone, spelling and grammar errors?
[x] Did you attach images of your dashboards to the PR so we can see them working?

Screenshots
Attach images of any visual changes, such as a dashboard here.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Jake Landers seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

Closing this out as it's about to become stale


Not able to run detox test (Error: connect ECONNREFUSED)

Description
I followed the "Getting Started" guide carefully step by step, on a completely new React Native project (react-native init).
Running detox build works fine, but detox test does not. It results in this error:

I am not behind any specific/corporate proxy, and I have never met NPM connection errors like that before.
Steps to Reproduce

Followed the "Getting Started" docs
Install all the latest dependencies as of this date today
Init a new React Native project
Run detox build and then detox test

Detox, Node, Device, Xcode and macOS Versions

Detox: 7.1.0
Detox CLI: 7.0.0
Node: 8.9.4
Device: iPhone 7, 11.2
Xcode: 9.2, build 9C40b
macOS: 10.13.3 (17D102)

Device and verbose Detox logs
node_modules/.bin/mocha e2e --opts e2e/mocha.opts  --loglevel verbose

detox info 11:45:45: server listening on localhost:51743...
detox ERR! ws onError: Error: connect ECONNREFUSED ::1:51743
  1) "before all" hook

  0 passing (30s)
  1 failing

  1) "before all" hook:
     Error: connect ECONNREFUSED ::1:51743
      at Object._errnoException (util.js:1022:11)
      at _exceptionWithHostPort (util.js:1044:20)
      at TCPConnectWrap.afterConnect [as oncomplete] (net.js:1182:14)

  0 passing (30s)
  1 failing

  1) "before all" hook:
     Error: connect ECONNREFUSED ::1:51743
      at Object._errnoException (util.js:1022:11)
      at _exceptionWithHostPort (util.js:1044:20)
      at TCPConnectWrap.afterConnect [as oncomplete] (net.js:1182:14)

child_process.js:644
    throw err;
    ^

Error: Command failed: node_modules/.bin/mocha e2e --opts e2e/mocha.opts  --loglevel verbose
    at checkExecSyncError (child_process.js:601:13)
    at Object.execSync (child_process.js:641:13)
    at runMocha (/Users/jesperlund/dev/mobile-client/node_modules/detox/local-cli/detox-test.js:72:6)
    at Object.<anonymous> (/Users/jesperlund/dev/mobile-client/node_modules/detox/local-cli/detox-test.js:50:5)
    at Module._compile (module.js:643:30)
    at Object.Module._extensions..js (module.js:654:10)
    at Module.load (module.js:556:32)
    at tryModuleLoad (module.js:499:12)
    at Function.Module._load (module.js:491:3)
    at Function.Module.runMain (module.js:684:10)```

Something is blocking connection. Perhaps you have a firewall application that is preventing the server port opening.


SCT-1147 Implement create and get namespace

Pull Request Test Coverage Report for Build 60

26 of 30 (86.67%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.2%) to 94.203%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/jgikbase/idmapping/storage/mongo/id_mapping_mongo_storage.py
23
27
85.19%

Totals

Change from base Build 59:
-0.2%

Covered Lines:
260

Relevant Lines:
276

💛  - Coveralls

Looks okay to me, can't think of any changes


initial draft of dump doc clarification

Summary
Organize the documentation on collecting and analyzing dumps using .NET. Created a singular page that connects to all dump documentation. Fill in missing gaps.
Fixes #2905

And add the new files to the TOC at https://github.com/dotnet/docs/blob/main/docs/toc/tools-diagnostics/toc.yml.

I can only find docs/toc.yml which does not look like the correct file to add things to.

And add the new files to the TOC at https://github.com/dotnet/docs/blob/main/docs/toc/tools-diagnostics/toc.yml.

I can only find docs/toc.yml which does not look like the correct file to add things to. @gewarren

Sorry, it moved to https://github.com/dotnet/docs/blob/main/docs/navigate/tools-diagnostics/toc.yml.

Sorry, it moved to https://github.com/dotnet/docs/blob/main/docs/navigate/tools-diagnostics/toc.yml.

I added the new files!


Add .pgm to list of supported image file formats

Tested on https://groups.google.com/d/msg/digits-users/v46QtzlPSCw/-tMSse0ICwAJ


Mass creation of invite codes by an admin for external use

We need the ability to generate a number of invite codes in the admin panel that can just be copied (or downloaded) via a text file. For example, if I want to just generate a few dozen codes to send around not via the platform.
This feature should be available through the admin panel. It's separate from distributing invite codes for current users to share.

Display new invite codes in a large text field, one invite code per line
So they can be copied whole


visionOS "full screen" player

Is your feature request related to a problem?
This may be less of a problem and more of a question. Videos only seem to be playable inline on visionOS. Is there something I need to do or could do to get a "full screen" player.
What solution would you like?
Getting either the system player or the player you get from youtube.com would be perfect.
What alternatives have you considered?
I've tried forcing allowsInlineMediaPlayback to false (via a fork) and that does play sound but blurs the entire window interface. Using the .web fullscreenMode goes one step further and resizes the window to the ratio of the video and does play the video but it is blurred. Finally, I've noticed that combining the two, if the user presses the full screen button you get the blurred video like before but pressing the progress bar will play the video in its own window.
To recap, that's with allowsInlineMediaPlayback to false and the following configuration:
.init(automaticallyAdjustsContentInsets: true, allowsPictureInPictureMediaPlayback: true, fullscreenMode: .web, autoPlay: false, progressBarColor: .white, showControls: true, keyboardControlsDisabled: false, enableJavaScriptAPI: false, showFullscreenButton: true, showAnnotations: false, useModestBranding: true, origin: nil, playInline: false, showRelatedVideos: false)
Any additional context?
No response

Hi @frlefebvre,
I've also experienced this issue and it seems like a strange WebKit bug on visionOS because setting the fullscreenMode to .web should work fine as it enables the native HTML5 fullscreen API.
I think thats the reason why Christian Selig the author of the Juno app on visionOS uses a custom interface (opened in a new window) for fullscreen which adds an extra ornament view below the player interface.

Ah that's a shame! I guess I'll keep opening the link in Safari for the time being. Have you filed a radar for it yet I could reference in my own?

Thats a good workaround 👍
No, I haven't submitted a feedback for it yet.


Factorize matrices for BBMBBMVariableEquations1D

Same as #108, but for the variable bathymetry case.

It looks like some tolerances and/or reference values may need to be adjusted

Yes, this was also the case for the other PR. I will do that.


FORMS-14013 - Fixing Dropdown that had a selected value without default/placeholder set

Description & Screenshots of the change are present in the jira comment
https://jira.corp.adobe.com/browse/FORMS-14013
Description

Related Issue

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):
Types of changes

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

Checklist:

[ ] I have signed the Adobe Open Source CLA.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes and the overall coverage did not decrease.
[ ] All unit tests pass on CircleCi.
[ ] I ran all tests locally and they pass.

Created new PR with test cases: https://github.com/adobe/aem-core-forms-components/pull/1203


Remove min and max idea for bar lines

min and max methods are not used by now in GtPlotterBarChartLine

Code was removed with some examples


Gyrosim: Fixed constant looping

Gyrosim would call measure continuously because the write_checked_reg
was failing. There is no need to check faked reg writes in the
simulator so that code was removed.
The delay that was added to the simulator to pace the gyrosim reads
was also removed now that the source of the problem was determined.
Signed-off-by: Mark Charlebois<EMAIL_ADDRESS>

@mcharleb Thanks Mark! Will give that a try.

How does this PR relate to the other sim / work queue PRs?

Both sets of changes are needed. Two separate issues