HDDS-9296. Add Swagger UI for Recon REST APIs

What changes were proposed in this pull request?
Currently the documentation provided for Recon APIs do not have a Swagger UI for the same.
This PR introduces the proposed Swagger UI, providing an interactive documentation for the available APIs and the responses it provides.

The Swagger UI is a form of documentation that would allow the users to interact with the documentation that is provided for the Recon APIs.
This would allow people to quickly interact with these endpoints and understand the type of payload to be given, the expected response and it's structure
It is a supplemental documentation over the existing Recon API document that is present.

Quick overview for the YAML file format:

The YAML file to store the definitions for the APIs has the following main things to note:

Section Name
Use

tags
Stores the names by which we can group APIs/data, for example Containers and Keys, Namespace etc.

paths
Stores the actual API definition data and information like short descriptions, accepted parameters for the request, response data. The responses have been defined in the components section, and $ref is used to provide a reference to allow readability and organization in the code

components
Stores the actual format for the data response

path section has the following main information

Sub-section
Use

paths / get, post, put...
Under path, the get word defines what type of request the API will accept. In this case we have given get for the APIs but we can also do post, put, delete etc.

paths / <request-type> / tags
Specifies which group the API should be put under

paths / <request-type> / parameters
This section would be used to define the various parameters that would be accepted by the API for the request

paths / <request-type> / responses
This section contains the information about the response that would be given. Notice we write the information under a '200' section which means when the API returns a 200 status the further definition for the response would be used. We can further expand this to hold response types for other statuses as well

paths / ... / responses / content
Stores the actual response type. Here we have defined the response to be of type  application/json, but we can have other types as well like application/xml

paths / ... / responses / content / ... / schema
This holds a reference to the actual structure of the response which we have defined in the components section

components section has the following main information

Sub-section
Use

component / schema
This will mainly be used for data that might have a fixed format, similar to a documentation containing syntax information or information on object. Note: This also allows users to see the response format without looking at the actual response as it will define the possible fields and their types in a compact manner

component / responses
This will be used to store responses directly. The data in this section will not be rendered in the Schemas dropdown present in the UI

Overview on the data-types for the responses
The YAML response format is as follows:

type:
example:

Now the data types supported are all the general primitive data-types we use:

string
number / integer -> Both of them hold a numeric value, however number supports larger values than the typical integer range and also floating point values
boolean
array
object

For a data-type of array we need to include the following information as well:

items: This will define the possible items that would be stored in the array and stores the format for those items.
For a data-type of object we need to include the following information as well:
properties: This will define the keys / properties that are associated with the object.

Ex:

properties:
    a: ....
    b: ....

would correspond to an object like:
{
    "a": .... ,
    "b": ....
}

We can further chain these to create complex data types like an array of objects, or an object key of type array which can be seen in the yaml file
What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-9296
How was this patch tested?
This patch was tested manually. Following are the related screenshots and recording for the documentation page.

https://github.com/apache/ozone/assets/43001336/a59df031-23da-4e8c-a245-f5dd36265d1c

@devmadhuu @ArafatKhan2198 please review

Hello @ivandika3 could you also please take a look at this?

Thanks a lot for reviewing these changes @devmadhuu .
In the latest commit I did the changes as requested.
Updated screenshots:

I corrected the operationId to reflect the method names as declared in the methods from the API folder.
However for the /containers/unhealthy/{state} the operation ID is different from the method name as both the endpoints i.e.

/containers/unhealthy
/containers/unhealthy/{state}
have the same method name, causing Swagger to trigger both dropdowns.

I put the operationId for /containers/unhealthy as getUnhealthyContainers which is it's method name.
And for  /containers/unhealthy/{state}, I put it as getUnhealthyContainersWithState to make it unique.

docs/edge on the website is just a convenience for developers to have a snapshot of the current docs.  Docs are also built during release, and are part of the binary tarball users can download from the website.  They can extract the binaries to any local dir.  Is it possible to make the Swagger UI work for them?

Hello, thanks for the review @adoroszlai @ArafatKhan2198. In the latest commit I changed it to a relative path, which should allow the yaml file to be detected irrespective of how the files are being stored.
Verified this patch for the following:

By running the same steps as in https://github.com/apache/ozone/pull/5509#discussion_r1383483505

By directly running hugo serve on the docs folder

Via building the whole project, running it using docker-compose and then visiting the UI

Hello, thanks for the review @adoroszlai @ArafatKhan2198. In the latest commit I changed it to a relative path, which should allow the yaml file to be detected irrespective of how the files are being stored. Verified this patch for the following:

By running the same steps as in HDDS-9296. Add Swagger UI for Recon REST APIs #5509 (comment)

* By directly running `hugo serve` on the docs folder

* Via building the whole project, running it using `docker-compose` and then visiting the UI

Thanks for the changes @devabhishekpal Ill take a look!

Hello @errose28, so while it is true that "Try It Out" button will be giving error since there is no recon instance, the main reason for the button is to provide the interactivity.
Without it the documentation would be equivalent to the normal documentation we have, only with a different UI.
On clicking Try It Out, the end user can interact with the various params, and even get a URL for the API call with the params filled in.

That is also one of the reasons to add a Swagger UI.

@devabhishekpal thanks for making the changes! Quick question?
Is the schema generated automatically by using the example response, or do you need to write it down manually?

@ArafatKhan2198 this schema is generated automatically from the example and schema structure declared.
So if you check the yaml file we have the schema declared as:
someDataObject:
    type: object
    properties:
        a:
         type: integer
         example: 3

So from here it interpolates the schema

Thanks for the changes @devabhishekpal I have left comments.

Thanks @ArafatKhan2198 for the review.
I have made the changes.
The only thing which isn't quite feasible is from https://github.com/apache/ozone/pull/5509#pullrequestreview-1725354077 regarding setting of the port value.
Swagger will determine the port from the localhost value. While we can do that via JS, maybe add an option for the users to specify the end-port, most users might not use it, since we are returning 404 either way because of no recon instance, and this is just an interactive documentation - not a replacement for API calls.
But do let me know if you want that change, will look into it.
The other changes have been addressed in the latest commit.
Please do take a look once, specially in the /utilization/containerCount API, and let me know if the description is okay, and whether containerSize parameter is mandatory or not.

Thank you, @devabhishekpal, for sticking with this and implementing the necessary changes. Just a few minor comments, and we should be ready to proceed with this.

For the /utilization/containerCount endpoint the parameter containerSize is not a required field and is just optional.

Could you please add this message next to the containerSize parameter, similar to how it is done for the fileSize? Its better we maintain consistency throughout ➖

Thanks for working on this @devabhishekpal! Thanks for the review @devmadhuu, @ArafatKhan2198, @adoroszlai and @errose28!

Thanks @devabhishekpal for picking this one up!


Restructuring of towered extension fields Fp4 and Fp6

As Coda we take the embedding field of an MTN4 curve as the quadratic extension
Fp4 = Fp2[Y] / (Y^2 - X)
of Fp2 = Fp[X]/(X^2 - alpha), where alpha is the const Fp2Parameters::NONRESIDUE, and similarly the embedding field of an MNT6 curve as
Fp6 = Fp2[Y]/(Y^2-X)
of Fp3=Fp[X]/(X^3 - alpha), with \alpha as const Fp3Parameters::NONRESIDUE.
However, the implementations of the two towered extensions differs in style, and both 'open up' the arithmetics of their subfields Fp2, Fp3, respectively, to define multiplications by $X$ therein.
To illustrate the different styles of implementation:

for Fp4, multiplication by Y^2 = X is done as

    pub fn mul_by_nonresidue(value: &Fp2<P::Fp2Params>) -> Fp2<P::Fp2Params> {
        let new_c0 = P::Fp2Params::mul_fp_by_nonresidue(&value.c1);
        let new_c1 = value.c0;
        Fp2::new(new_c0, new_c1)
    }

not even referring to Fp2Parameters::NONRESIDUE, whereas

for Fp6, multiplication by $X$ is implemented as

    pub fn mul_by_nonresidue(value: &Fp3<P::Fp3Params>) -> Fp3<P::Fp3Params> {
        let mut res = *value;
        res.c0 = value.c2;
        res.c1 = value.c0;
        res.c2 = value.c1;
        res.c0
            .mul_assign(&<P::Fp3Params as Fp3Parameters>::NONRESIDUE);
        res
    }

which use Fp3Parameters::NONRESIDUE, and the generic trait Fp6Parameters function
   fn mul_fp3_by_nonresidue(fe: &Fp3<Self::Fp3Params>) -> Fp3<Self::Fp3Params> {
        Self::NONRESIDUE * fe
    }

is not used at all.
I propose a restructuring of the towered extension fields, choosing a generic approach which defines quadratic/cubic extensions F2/F3 over any field F (not necessarily prime), and then define F6 with "sextic" non-residue Fp6Params::NONRESIDUE as quadratic extension F2(F3), using the (quadratic) non-residue
(c_0,c_1)=(0,1) from F3^2
(which corresponds to X), where F3 is the cubic extension of Fp using the sextic non-residue as (cubic) non-residue. Likewise, one defines F4 with "quartic" non-residue Fp4Params::NONRESIDUE as quadratic extension F2(F2), using the quadratic nonresidue
(0,1) in F2^2,
where F2 is the quadratic extension of Fp using the quartic non-residue as quadratic one.
Such generic implementation is not just for readability - it also makes further towering (if ever needed lateron) more transparent and easy.

The audit of the corresponding field gadgets (as part of  this PR) shows the same structural issue:
The FieldGadget implementations for the quadratic extensions (Fp2Gadget, Fp6Gadget as '2over3', Fp12Gadget as '2over6') as well as for the cubic extensions (Fp3Gadget, Fp6Gadget as '3 over 2') are almost identical, resulting in needless repetition of code.
I strongly recommend to implement the above described generic approach also for the gadgets:  a generic F2Gadget and F3Gadget (for arbitrary base field gadgets instead of FpGadgets only), and then defining the used Fp2Gadgets as F2Gadget over Fp, etc.

Fixed with #74, in which we imported QuadExtField/QuadExtFieldGadget and CubicExtField/CubicExtFieldGadgets structs from upstream


Old gcc "Value too large for defined data type" error

Windows Build Number
Microsoft Windows [Versione 10.0.19043.1110]
WSL Version

[X] WSL 2
[ ] WSL 1

Kernel Version
Linux version 5.4.72-microsoft-standard-WSL2
Distro Version
Arch Linux
Other Software
Docker version 20.10.7, build f0df350 (Docker Desktop for Windows)
Repro Steps
I have a quite old SPARC CPU (Leon) cross compiler toolchain based on GCC 4.4.2 (32 bit version). I create a docker image based on i386/debian:bullseye-20210621-slim and I can map a local folder to a folder in the docker image and execute a Makefile script. The native folder with my project source tree is into an NTFS Windows 10 filesystem, mounted to /mnt/c/<...snip...> inside WSL2. When GCC is executed the "Value too large for defined data type" error messgae is shown:
root@f43ac2c97451:/build# make
make[1]: Entering directory '/build/XxxXX'
Compiling src/startup.S
/opt/sparc-elf-4.4.2/bin/sparc-elf-gcc -ggdb -Wall -mtune=ut699 -mv8 -mfpu -nostdinc -fno-builtin -mno-app-regs -fno-common -fno-strict-aliasing -DMHZ=40 -I./../Common -O2 -c -o obj/startup.o src/startup.S
cc1: error: ./../Common: Value too large for defined data type
cc1: error: src/startup.S: Value too large for defined data type
make[1]: *** [Makefile:69: startup.o] Error 1
make[1]: Leaving directory '/build/XxxXX'
make: *** [Makefile:14: default] Error 2

I then tried to rebuild the docker image from a 64 bit debian (including i686 libc libraries to execute the toolchain) but the behaviuor was the very same as above.
After a google search it seems that the problem may be related to mounting options of native filesystem into WSL2.
I finally copied my whole project source tree into my home folder under WSL2 (ext4 in a vhdx image) and executed the docker image from there. The Makefile script completes without any error.
I have found that the issue #3472 is similar but it was closed without a clear solution.
Expected Behavior
GCC executes without error messages.
Actual Behavior
GCC stops with "Value too large for defined data type" error message.
Diagnostic Logs
No response

You need to build the entire 32-bit toolchain you are using with _FILE_OFFSET_BITS=64.

https://www.mjr19.org.uk/sw/inodes64.html

Made a quick test with the LD_PRELOAD hack described in the link and it worked. Recompiling the toolchain _FILE_OFFSET_BITS=64 is the way to go.
Thank you.


VNL 2021-02-15 (7edc7cd8)

Run the UpdateFromUpstream.sh script to extract upstream VNL
using the following shell commands.
$ git archive --prefix=upstream-vnl/ 7edc7cd8 --
CMakeLists.txt
config/cmake
core/CMakeLists.txt
core/testlib
core/vnl
core/vxl_config.h.in
core/vxl_copyright.h
core/vxl_version.h
v3p/CMakeLists.txt
v3p/netlib
vcl
| tar x
$ git shortlog --perl-regexp --author='^((?!Kitware Robot).*)$' --no-merges --abbrev=8 --format='%h %s' 85ae25b7..7edc7cd8
Drew Gilliam (1):
40ccc0b8 vpgl_rational_camera additional ordering schema
Joseph Mundy (2):
faa3fcf4 remove bad cams from match tree
7edc7cd8 upgrade from WGS80 to WGS84
Matt McCormick (1):
d1cd3d14 COMP: Instantiate vnl_fft_base for 5D data
Change-Id: Ia06010461c76d8388e6fbcd9ff25cda79fc1bff1

v3p/netlib/libf2c/comptry.bat
commit a4dc6cc adds bad whitespace including CR/LF line endings:

Odd because we have *.bat -crlf in our *.gitattributes. @bradking @mathstuf

Odd because we have *.bat -crlf in our *.gitattributes.

The bot is complaining about the import commit, whose tree object does not include the rest of ITK.  The import script may need to write the .gitattributes content itself to get it in the import commit.

I am curious why this VNL update seems fine, not touching unrelated files as past VNL and MetaIO updates did.
@bradking Could you please share your thoughts on this related issue : https://github.com/InsightSoftwareConsortium/ITK/issues/2314 ?

I'll look at converting Modules/ThirdParty/VNL/UpdateFromUpstream.sh to use Utilities/Maintenance/update-third-party.bash, which provides a hook to handle .gitattributes.

@thewtex I updated your branch to convert to the newer import script, add the .gitattributes file to the import to suppress whitespace checks, and then perform the actual update you did before.

@bradking thank you!!


Restore the Pausetimes Apps in the UI

This PR re-enables the previously disabled Pausetimes apps.  The apps have been updated to display the data in the new benchmark directories pausetimes_seq and pausetimes_par produced using olly latency.
The PR also:

Adds a script that fixes pausetimes bench files to be formatted similar to the new formatting implemented in https://github.com/tarides/runtime_events_tools/pull/13
Calls this script when copying data to the main branch to ensure any new data is formatted correctly. This is required until we are able to use the latest version of runtime_events_tools in Sandmark, which depends on ppxlib being compatible with OCaml 5.1 and 5.2.
Updates existing bench files pausetimes_* to use the new formatting, along with fixing an issue with the bench files having pretty printed JSON which breaks the assumption that every line in the bench file is a valid JSON string. This was fixed in https://github.com/ocaml-bench/sandmark/pull/452

Left a minor comment. Otherwise, LGTM.
FWIW, I'm not able to review this UI/UX PRs easily without being able to test the app. We'll have to merge the PR and then fix any issues observed.
Do we have a staging area where we can deploy the app and then test it before we deploy it in production?

@kayceesrk Thanks for the review!

Do we have a staging area where we can deploy the app and then test it before we deploy it in production?

We don't have a staging area, but I just deployed the branch on the Streamlit (free) hosting. You can check it out here.
The deployment doesn't seem to use the exact versions of dependencies specified in our requirements.txt, probably since they try to be smart by installing some dependencies (pandas, altair, etc.) by default for all the apps. Currently the Parallel pausetimes app is crashing, but the sequential one works. I'm looking around to see if there's a quick fix for the crash.

Left a minor comment. Otherwise, LGTM.

Thanks, I'll address this soon, along with any other comments you may have after using the UI.

I'm looking around to see if there's a quick fix for the crash.

We were using an older version of streamlit in our requirements.txt and the deployment logic for supporting older versions of Streamlit on the cloud was causing issues with other dependencies. I've pushed a commit to update our dependency to the latest version of Streamlit, and everything seems to work okay.
@kayceesrk You can look at the deployed branch here for a more complete review. Thanks!

The features look really awesome.

Remove "Instrumented" from all the locations.
Having both "sequential benchmarks" and "pausetimes sequential" seems odd. Can the "pausetimes sequential" appear at the bottom of the "sequential benchmarks" results?
In fact, the pausetimes are no longer computed on instrumented programs. If we are clever about isolating noise, the pausetimes numbers could be computed along with the sequential and parallel runs. FWIW earlier pausetimes were computed on the instrumented runtime, which caused slowdown of programs. This is no longer the case as the pausetimes are read from the shared ring buffer. This will cut down the running time of the nightly runs by 1/2 CC @Sudha247 thoughts?

Please do (1) in this PR as it is quick. For (2), given that the results are computed from different runs unlike suggested in (3), it might be a little tricky. As a stop-gap measure, may I suggest reordering the pages in the following way?
Goto
* Home
* Sequential -- throughput
* Sequential -- latency
* Parallel -- throughput
* Parallel -- latency

It may be useful to track (3) in a separate issue.

I also think we can extract both running times and pausetimes from the same run. I've observed the overheads added by olly to be quite low, though we may have to confirm this for the benchmarks in sandmark. Good idea to track this in a separate issue.

Good idea to track this in a separate issue.

I've created https://github.com/ocaml-bench/sandmark/issues/454. Feel free to edit or add more information.

LGTM. Thanks.


Dump doesn't produce any output

Gitea version: Gitea version 1.9.3 built with GNU Make 4.1, go1.12.9 : bindata, sqlite, sqlite_unlock_notify
Git version: 2.17.1
Operating system: Ubuntu server 18.04.3
Database:

[ ] PostgreSQL
[x] MySQL - MariaDB 10.1
[ ] MSSQL
[ ] SQLite

Can you reproduce the bug at https://try.gitea.io:

[ ] Yes (provide example URL)
[ ] No
[x] Not relevant

Log gist:

2019/11/21 19:09:58 ...dules/setting/log.go:269:newLogService() [I] Gitea Log Mode: File(File:info)
2019/11/21 19:09:58 ...les/setting/cache.go:42:newCacheService() [I] Cache Service Enabled
2019/11/21 19:09:58 ...s/setting/session.go:45:newSessionService() [I] Session Service Enabled
2019/11/21 19:09:58 ...es/setting/mailer.go:105:newMailService() [I] Mail Service Enabled
2019/11/21 19:09:58 ...es/setting/mailer.go:116:newRegisterMailService() [I] Register Mail Service Enabled
2019/11/21 19:09:58 ...es/setting/mailer.go:127:newNotifyMailService() [I] Notify Mail Service Enabled
2019/11/21 19:09:58 cmd/dump.go:76:runDump() [I] Creating tmp work dir: /tmp/gitea-dump-586735121
2019/11/21 19:09:58 cmd/dump.go:86:runDump() [I] Packing dump files...

Description
I started to configure the backup of my gitea instance. Unfortunatly dump command
sudo -u git /usr/local/bin/gitea dump -c /etc/gitea/app.ini -f ./gitea.zip -V

gives me only one line of output:
2019/11/21 19:09:58 ...dules/setting/git.go:83:newGit() [I] Git Version: 2.17.1

I can find more info in the log file (above) but this doesn't give me any hints neither. It looks like it just stopped in the middle.
When setting up the server I had to deal with this issue regarding barracuda and innodb_large_prefix https://github.com/go-gitea/gitea/issues/2979 and also now I found this similar issue report https://github.com/go-gitea/gitea/issues/8941 where missing migrations were the problem. I feel like this is a hint to my case but need help connecting all the dots.

Coincidentally, I just encountered the same behavior when using the --tempdir <my_temp_dir> argument, when the directory <my_temp_dir> didn't exist. Once I fixed that, the output shows all the expected "Adding file..." and "Adding dir..." progress messages before hitting a different failure. :/
Anyway, you might want to check that /tmp exists and is world-writable, that the partition isn't full, etc. Just an idea--your problem could be entirely unrelated.

Thanks for the hint I found the issue and it was connected to your finding! Thanks!
The problem was so obvious... I was running this command from my user's directory so user git didn't have rights to create and write to a zip file there. Changed the command to
sudo -u git /usr/local/bin/gitea dump -c /etc/gitea/app.ini -f /tmp/gitea.zip -V
and everything went well!
There definitely should be an error message saying what's wrong. Should I close this issue or keep it open as a placeholder for adding an error message discussion?

This issue could be closed and any document update PRs are welcome!

I looked at this a bit and the issue is that log.Fatalf doesn't make it to flush the buffor before the application exits in line 88 of dump.go. There was even a discussion regarding that here https://github.com/golang/go/issues/21751
For me this is my day 1 with golang but I guess there is not much we can do about this missing log.

But we could have our own replacement for log.Fatalf(), so maybe take care of this. These seem to be calls from dump.go, not something buried in the nth call of a third-party library.


Code for BigEarthNet downstream task

Hi DOFA team,
Great work!
I was wondering how to reproduce your result on the BigEarthNet dataset?
Do you by any chance have code for it or for the dataloader?
I found the dofa_base_resisc45.py, but it seem to have a specific RGB data_preprocessor, and I'm not sure how to replace it for BigEarthNet with Sentinel2 data.
Elena

Hi sorry for the late response. Here GeoFM we provide the code for more downstream datasets on GeoBench.


Keptn Bridge: Version check in Bridge does not work (can not opt-in/opt-out nor enable/disable the feature)

In the tested version of the Bridge (version from: 20200714.1503 docker pull keptn/bridge2:20200714.1503) the message of Accepting/Declining the version check feature was not displayed and the switch did not work.

Note: This issue might be related to the situation that the test was conducted on a not officially released version of the bridge. Then it is obsolete.

Definition of Done:

[x] Version check feature can be enabled/disabled

On a new installation of Keptn, the feature worked. Let's have a closer look during the test phase of the releasing process.

I closed this issue since I could verify that it is working in the released version.


Doesnt Work for any URL as of right now

Gives error everywhere, Please fix ;/

did you get it working?

hi did you solve this problem now?

hi did you solve this problem now?

Hi not yet. Dm me in telegram so we can keep track of the progress @kamikadze24


Slight optimization in the Prometheus adapter.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: geeknoid
If they are not already assigned, you can assign the PR to them by writing /assign @geeknoid in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

mixer/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@geeknoid: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
99556d3072da954863e494c89bc0c5235d55f931
link
/test istio-unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test istio-unit-tests
/lgtm
It feels like WriteTo is doing more work (including a buffer reset), but the benchmarks seem basically equivalent.

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: douglas-reid
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

mixer/OWNERS [douglas-reid]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@geeknoid: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
99556d3072da954863e494c89bc0c5235d55f931
link
/test istio-unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test istio-unit-tests


Added iOS Object Detector

Added Detection, ObjectDetection Result and Helpers
Added MPP Object Detector

@schmidt-sebastian Raised a PR for source files of object detector. I have added all the source files in on go since some code is similar to the image classifier. LMK if PR size needs to be reduced.


Instagram Points: Usability Improvement

Overview
When allocating points, we want to save the selected users until a submission is made.  If an officer leave the page, then keep their selected users.
In this case, show a confirmation list of users that will have get points before submission is made.
Since we have over 500+ users on the app, loading every user on the list is causing major lag. To prevent this, lets intially load all user's that have previously been allocated points in a default tab called "Previously Posted".
Then if an officers wishes to visit the "All User" tab to get new members
Task

[ ] Cache the selected users until a submission is made
[ ] Before submission is made display a scrollable list of all Users
[ ] Create two tabs, one tab for "Previously Posted" and "All Users"

User Stories
As an officer, I want to have my selection saved so that when i accidently leave the screen my selections are saved
As an officer, I want less lag in the instragram points selection, so that I can smoothly allocate points
As an officer, I want to clearly know who I'm allocating points to so that I can correctly allocate points

intially i was thinking of having two list one for users that have previously posted and another of all users
but the list of previously posted users is also really large so it only helps slightly with loading problem. My idea was also that officer could just scroll through this list. But again it's so large they will always just be searching people anyways.
so now my solution is just to the list display only if u enter two more or characters


Add advanced user management

Add support for multiple users (normal and admin) with password recovery system

Added


feat: port LinearAlgebra.Trace

[x] depends on: #4262

This PR/issue depends on:

leanprover-community/mathlib4#4251
leanprover-community/mathlib4#4262
By Dependent Issues (🤖). Happy coding!

This PR/issue depends on:

leanprover-community/mathlib4#4262
By Dependent Issues (🤖). Happy coding!


Pets and spread of 19-nCoV

At present, there is no evidence that companion animals/pets such as dogs or cats can be infected with the new coronavirus. However, it is always a good idea to wash your hands with soap and water after contact with pets. This protects you against various common bacteria such as E.coli and Salmonella that can pass between pets and humans

Many types of coronaviruses are known to spread among animals too. But that hasn't been observed for COVID-19, i think.
"Coronaviruses are zoonotic, meaning they are transmitted between animals and people.  Detailed investigations found that SARS-CoV was transmitted from civet cats to humans and MERS-CoV from dromedary camels to humans. Several known coronaviruses are circulating in animals that have not yet infected humans." https://www.who.int/health-topics/coronavirus

Corona virus can be transmitted from day to day objects we get contact with.
Clothes soaps toothpaste switches seats packages bags seats,door handles and lives for 24 hrs
Basically have to wash every part of body comes in contact with objects which multiple people touch or continuously use.


Customizing chevron

I wanna know if it's possible to customise the chevrons? Other carousels that i saw have this carousel block with large chevron icons shown when hovering on the carousel items. Wondering if such customization can be done with your item carousel.

@Corouna You can pass leftChevron and rightChevron to the component. I'll close this issue for now, let me know if you have any issues with that!


Ctrl+Click Links with filename:line:text don't work

Issue Type: Bug

Create a file with name my_file.rb in a new repository and enter three lines of random text.
Open a Terminal
enter my_file.rb:1:in
Try to Ctrl+Click this go open the editor on line 1
Observe that VSCode shows "No result"

This bug manifests when running ruby scripts in Terminal and getting a stacktrace such as:
~/$ ruby my_file.rb
Traceback (most recent call last):
my_file.rb:2:in `<main>': Oh oh Crash (RuntimeError)

Screenshot:

VS Code version: Code 1.61.2 (6cba118ac49a1b88332f312a8f67186f7f3c1643, 2021-10-19T14:57:20.575Z)
OS version: Windows_NT x64 10.0.19042
Restricted Mode: No
Fetching remote diagnostics for 'WSL: Ubuntu-20.04' failed: Canceled

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-8550U CPU @ 1.80GHz (8 x 1992)

GPU Status
2d_canvas: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: enabledopengl: enabled_onrasterization: enabledskia_renderer: enabled_onvideo_decode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
15.86GB (4.57GB free)

Process Argv

Screen Reader
no

VM
57%

Fetching remote diagnostics for 'WSL: Ubuntu-20.04' failed: Canceled
Extensions: none

Associated question on StackOverflow: https://stackoverflow.com/q/69784676/278842

It actually works when running using ./:

Since the other case is falling back to the "word link" provider, it ends up searching the workspace for the whole path (my_file.rb:2:in). Since all these errors in ruby seem to have :in added to the end, let's add some smarts around handling that.

Thanks! @Tyriar This is awesome!


DM-45760: Add Multi-platform image builds with GitHub Actions

Add setup-qemu-action to allow multi-platform builds.
Add optional platforms field
Update Readme

Could you send a link to a GitHub Actions build using this? You can test it by updating a project's workflow to point to roceb/build-and-push-to-ghcr@main (I think) rather than lsst-sqre/build-and-push-to-ghcr.


Plottypes for individual datasets

Right now the plot type is set globally for every dataset. It would be nicer to be able to set a plot type for each individual dataset.

Done in 2a0205d1971ebcf8fa426313570b49f8fc171971


Nginx forward all requests to the new URL

Hello all.  How can i change url for baget in Docker?
With location / {  all work fine
But with this not work
location /newurl/ {
                 proxy_pass              http://<IP_ADDRESS>:4444;
                 proxy_http_version      1.1;
                 proxy_set_header        Upgrade $http_upgrade;
                 proxy_set_header        Connection keep-alive;
                 proxy_set_header        Host $host;
                 proxy_cache_bypass      $http_upgrade;
                 proxy_set_header        X-Forwarded-For $proxy_add_x_forwarded_for;
                 proxy_set_header        X-Forwarded-Proto $scheme;
                 }

Can anybody tell me about configuration nginx, please?
Thanks

Thanks for answer, but at nginx i have like proxy_pass http://<IP_ADDRESS>:4444/;
And at URL http://my.url/baget/ have a clear page with top menu.
When click in the menu on "Packages" or "Upload" i get normal page, but in URL http://my.url not http://my-url/baget/


Move HDFS file to encrypted zone failed.

Get error message when using HdfsTarget as output because /tmp is under unencryption zone and the destintation is under encryption zone.
---stderr---
mv: /tmp/data-matrix-luigitemp-720632922 can't be moved into an encryption zone.
Fixed by changing the abstract_client.py line 42, in rename from
return self.move(path, dest)
to
self.copy(path, dest)

self.remove(path)

return```

I close this as this is just some work-around information, perhaps handy for someone with a similar. I close this as I see no action to be done here.


after hot restart serveral time while debuging will cause crash

Describe the bug
after hot restart serveral time while debuging will cause crash,I have encountered this issue for many time.
sometimes the log show out of memory,but this time show like as below
* thread #42, queue = 'com.hackemist.SDImageCache', stop reason = EXC_RESOURCE RESOURCE_TYPE_MEMORY (limit=2098 MB, unused=0x0)
    frame #0: 0x00000001a2dafde4 vImage`vConvert_16Q12to16F_vec + 208
vImage`vConvert_16Q12to16F_vec:
->  0x1a2dafde4 <+208>: stp    q4, q3, [x16]
    0x1a2dafde8 <+212>: ldp    q3, q2, [x1, #0x20]
    0x1a2dafdec <+216>: sshll.4s v4, v3, #0x0
    0x1a2dafdf0 <+220>: scvtf.4s v4, v4
Target 0: (Runner) stopped.
Lost connection to device.

To Reproduce
Steps to reproduce the behavior:
configured like the test demo and have replace most of image widget with power_image;
void main() async {
  runZonedGuarded(
    () async {
      PowerImageBinding();
      ...
      PowerImageLoader.instance.setup(
        PowerImageSetupOptions(
          renderingTypeTexture,
          errorCallbackSamplingRate: null,
          errorCallback: (PowerImageLoadException exception) {
            LogUtils.error(
              'Caught PowerImageLoadException : $exception',
            );
          },
        ),
      );
      runApp(const MyApp());
    },
    (Object error, StackTrace stackTrace) async {
     ...
    },
  );
}

Expected behavior
please dont crash that often 😅,
Screenshots
If applicable, add screenshots to help explain your problem.

Versions (please complete the following information):

Device: iPhone11
OS: ios15.3
flutter doctor -v:

展开查看 flutter doctor details

[✓] Flutter (Channel stable, 2.10.5, on macOS 12.4 21F79 darwin-x64, locale
    zh-Hans-CN)
    • Flutter version 2.10.5 at /Users/kangkang/Documents/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 5464c5bac7 (7 weeks ago), 2022-04-18 09:55:37 -0700
    • Engine revision 57d3bac3dd
    • Dart version 2.16.2
    • DevTools version 2.9.2
    • Pub download mirror https://pub.flutter-io.cn
    • Flutter download mirror https://storage.flutter-io.cn
[✓] Android toolchain - develop for Android devices (Android SDK version
32.1.0-rc1)
• Android SDK at /Users/kangkang/Library/Android/sdk
• Platform android-32, build-tools 32.1.0-rc1
• Java binary at: /Applications/Android
Studio.app/Contents/jre/Contents/Home/bin/java
• Java version OpenJDK Runtime Environment (build
11.0.12+0-b1504.28-7817840)
• All Android licenses accepted.
[✓] Xcode - develop for iOS and macOS (Xcode 13.2.1)
• Xcode at /Applications/Xcode.app/Contents/Developer
• CocoaPods version 1.11.3
[✓] Chrome - develop for the web
• Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome
[✓] Android Studio (version 2021.2)
• Android Studio at /Applications/Android Studio.app/Contents
• Flutter plugin can be installed from:
🔨 https://plugins.jetbrains.com/plugin/9212-flutter
• Dart plugin can be installed from:
🔨 https://plugins.jetbrains.com/plugin/6351-dart
• Java version OpenJDK Runtime Environment (build
11.0.12+0-b1504.28-7817840)
[✓] VS Code (version 1.67.2)
• VS Code at /Applications/Visual Studio Code.app/Contents
• Flutter extension version 3.42.0
[✓] Connected device (2 available)
• iPhone 11 (mobile) • 00008030-00010D3826E0802E • ios            • iOS 15.3
19D50
• Chrome (web)       • chrome                    • web-javascript • Google
Chrome 102.0.5005.61
[!] HTTP Host Availability
✗ HTTP host https://maven.google.com/ is not reachable. Reason: An error occurred while
checking the HTTP host: Operation timed out
! Doctor found issues in 1 category.

power_image Version: ^0.1.0-pre.2
power_image_ext Version: ^3.0.0

Additional context

It seems that the native cannot release the image memory due to hot restart, so the memory continues to increase, which eventually leads to OOM.
This problem only occurs when the renderingType is texture. Normally, when imageCache removes imageProvider, PowerImage will notify native to release memory. But when hot restart these processes are gone.
I'm guessing something similar to this issue:
https://github.com/flutter/flutter/issues/10437
https://github.com/flutter/flutter/issues/69949

It seems that the native cannot release the image memory due to hot restart, so the memory continues to increase, which eventually leads to OOM.
This problem only occurs when the renderingType is texture. Normally, when imageCache removes imageProvider, PowerImage will notify native to release memory. But when hot restart these processes are gone.
I'm guessing something similar to this issue: flutter/flutter#10437 flutter/flutter#69949

thanks for your reply，dose it will be work if execute clean image cache every time at the begin of run my app. like this
...
      PowerImageBinding();
      WidgetsFlutterBinding.ensureInitialized();
      PaintingBinding.instance?.imageCache?.clear();
...

It seems that the native cannot release the image memory due to hot restart, so the memory continues to increase, which eventually leads to OOM.
This problem only occurs when the renderingType is texture. Normally, when imageCache removes imageProvider, PowerImage will notify native to release memory. But when hot restart these processes are gone.
I'm guessing something similar to this issue: flutter/flutter#10437 flutter/flutter#69949

thanks for your reply，dose it will be work if execute clean image cache every time at the begin of run my app. like this
...
      PowerImageBinding();

      WidgetsFlutterBinding.ensureInitialized();

      if(kDebugMode){
         ///cleanCacheFirst

      }
...

This should be useless. At this time, the hot restart has ended, and the ImageCache must be empty. The problem is that native doesn't perceive hot restart, so it never releases textures.

It seems that the native cannot release the image memory due to hot restart, so the memory continues to increase, which eventually leads to OOM.
This problem only occurs when the renderingType is texture. Normally, when imageCache removes imageProvider, PowerImage will notify native to release memory. But when hot restart these processes are gone.
I'm guessing something similar to this issue: flutter/flutter#10437 flutter/flutter#69949

thanks for your reply，dose it will be work if execute clean image cache every time at the begin of run my app. like this
...
      PowerImageBinding();

      WidgetsFlutterBinding.ensureInitialized();

      if(kDebugMode){
         ///cleanCacheFirst

      }
...

This should be useless. At this time, the hot restart has ended, and the ImageCache must be empty. The problem is that native doesn't perceive hot restart, so it never releases textures.

so as long as we can not reach the hot restart begin,this issue will always exists, right? Is there anything else we still can do.🫣


Edit investigator command Fixes #145

Update the EditInvestigatorCommand to be able to edit the rank and ensure that the change is applied to all the CrimeCase it is in charge of.

Pull Request Test Coverage Report for Build 353

9 of 23 (39.13%) changed or added relevant lines in 3 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage increased (+0.04%) to 81.388%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/main/java/seedu/investigapptor/logic/commands/EditInvestigatorCommand.java
6
8
75.0%

src/main/java/seedu/investigapptor/model/Investigapptor.java
1
13
7.69%

Files with Coverage Reduction
New Missed Lines
%

src/main/java/seedu/investigapptor/logic/commands/EditInvestigatorCommand.java
1
93.83%

Totals

Change from base Build 341:
0.04%

Covered Lines:
2615

Relevant Lines:
3213

💛  - Coveralls


#bug; dashboard derivitives metadata not updating on home page

Describe the bug
Meta data fields not showing/loading for all derivatives on dashboard home page.
Reproduction
https://subgraphs.messari.io/
Screenshots

PR #2068 fixes this!


Fixed diary details as per date display issue

Before when user use to add entry for current date or used to select old date, some other date entry was shown as date state was not updated
link to issues fixed: https://www.recordjoy.com/view/qWsNMEHAWRZHJ8pphPt4
->As soon as user adds new entry it's details are shown (before no details/entries were shown when entered by user)
->When user selects old dates, entries on those dates are shown
->An pop-up alert showing user date selected.

Added old recording.
Here is recording showing point one i.e new entry as soon as entered is shown unlike before.
Link: https://www.recordjoy.com/view/48L6SC043YcTpaptjQoH

Good work!


chore: Move edit dataset lock

SUMMARY

Moved the edit dataset lock to the top of the component in order to increase visibility.
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

BEFORE:

AFTER:

TESTING INSTRUCTIONS

Observe the described changes in the Edit Dataset model under the "Source" tab.

ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Required feature flags:
[x] Changes UI
[ ] Includes DB Migration (follow approval process in SIP-59)

[ ] Migration is atomic, supports rollback & is backwards-compatible
[ ] Confirm DB migration upgrade and downgrade tested
[ ] Runtime estimates and downtime expectations provided

[ ] Introduces new feature or API
[ ] Removes existing feature or API

cc @yousoph


Confused about documentation for explorer.getField()

The documentation for explorer.getField() and explorer.editField() says this:
5th parameter is desired index

However, the titles of the two functions don't include this parameter:
### explorer.getField(instance, offset, offsetX32, type)

### explorer.editField(instance, offset, offsetX32, type, value)

Also, the examples don't seem to provide this 5th parameter:
explorer.getField(explorer.getInstances('RGHand')[1], 0x10, 0x8, gg.TYPE_DWORD)

explorer.editField(instance, offset, offsetX32, type, value)

explorer.editField(explorer.getInstances('RGHand')[1], 0x10, 0x8, gg.TYPE_DWORD, 99999)

Additionally, the code doesn't seem to have it:
function explorer.getField(instance, offset, offsetX32, valueType)

function explorer.editField(instance, offset, offsetX32, valueType, value)

So, what is this mysterious 5th parameter? I am guessing that it was removed, and you forgot to update the documentation.

yeah, it was removed


Add RowMatrix3x2 and alike

The currently provided RowMatrix2x3 implies a "matrix * vector" multiplication semantics, which doesn't hold true in euclid/Gecko.
cc @Ralith @nical

I don't know if it makes things better or worse but I never though about 2d transforms stored in a 3x2 matrix as an actual 3x2 matrix but rather a compressed 3x3 matrix with some implicit values:
| a  b  0.0 |
| c  d  0.0 |
| e  f  1.0 |

To which you multiply a 3d vector that is really a 2d vector and an homogeneous coordinate just like you do in 3d to express translation as a linear operation.
I don't think the 2d transformation can be expressed as a 2x3 or 3x2 matrix multiplied by a 2d vector because you need that extra row or column to make the translation a linear operation (I might be wrong).
Anyway, I don't think it's that useful to add more permutations of 2x3/3x2 matrices. You could just have a compressed 3x3 matrix type that is well documented, and let the libraries that implement mint conversions do the swizzling themselves if need be. It seemed from our earlier discussion that euclid was using a different convention than cgmath and nalgebra, so maybe just stick with latter's representation and let euclid swizzle in it's implementation. From an interoperability stand point that's the simplest IMHO.

@nical thanks for chiming in! good input!

I don't know if it makes things better or worse but I never though about 2d transforms stored in a 3x2 matrix as an actual 3x2 matrix but rather a compressed 3x3 matrix with some implicit values.

The primary use of this crate is API. If something is meant to return a 3x2 matrix and we code it as 3x3 (with accompanying conversion/checks here and there), this would be a weaker API.

I don't think the 2d transformation can be expressed as a 2x3 or 3x2 matrix multiplied by a 2d vector because you need that extra row or column to make the translation a linear operation (I might be wrong).

Let's say you have a 2x1 vector/point, which can be expanded to 3x1 homogeneous vector. You could have a 2x3 matrix that transforms it, producing 2x3 * 3x1 = 2x1 vector. The last column of that matrix would then contain the translation, while the main 2x2 body - rotation and scale. Seems pretty straight to me for a 2D graphics case. I didn't do much 2D, and I could think of other ways to provide the transform (e.g. a vec4 containing 1 scale, 1 angle of rotation, 2 offsets), but 2x3 matrix doesn't seem completely crazy to use.
@icefoxen and @bvssvni - would you have opinions on what transformations are useful in 2D?

Heh nevermind I was tired when I wrote this last night, you totally can express the translation linearly with 2x3/3x2 matrices without expanding it into a 3x3.
All 2d drawing software and libraries I can think of (cairo, skia, your browser, etc.) store 2d transforms as 3x2 or 2x3 matrices so I believe that having one of the transpositions is the most useful for interop (more so than scale+rotation+offset).
Now as to whether having both transpositions is valuable for mint, I don't see the point if mint is about converting data between math libraries for which purpose what matters is really "what coefficient means what" rather than the semantics of vector * matrix versus matrix * vector multiplication since mint is not what you'll use to perform the transformations anyway and "3x2 matrix" doesn't say whether it is intended for 2d transformations or some subset of 3d transformations.
Anyway, after a quick look at the rest of mint it looks like the ship as already sailed on the question of having all notations versus focusing on the semantics of transformations, so maybe just add the missing transposition for 2d transforms.

@icefoxen please review https://github.com/kvark/mint/pull/30 and give it some thought
thank you!

Done.  It looks fine.
From a perspective of exposing an API using Mint the semantic meaning is nice but not strictly necessary.  I would think that the harder part comes on the part of the math libraries providing conversions into Mint types...  Do you care about turning an 2d Isometry into a 2x3 matrix or a 3x2 matrix?  I guess one would want to either say "I'm going to turn it into the most direct translation", or "I'm going to turn it into EVERY POSSIBLE translation".  Though given that mint has conversions from row-major to column-major matrices and such then a determined user can make it do whatever they want.

https://crates.io/crates/mint/0.5.1 is live, we should be able to continue with euclid integration


Uses local HTTP/HTTPS server for integration tests

Changes

Removes dependency on the external servers (httpbin) to increase reliability of integration tests.
Adjusts test suits to use the createServer utility and make requests to the spawned server URLs.
Adjusts https requests to use a custom httpsAgent that allows requests to self-signed HTTPS servers.
Adjusts XMLHttpRequest (in jsdom) to set strictSSL: false to allow requests to self-signed HTTPS servers.

All tests are passing offline.

few late hahahaha

@marcosvega91, thank you for the review! Sorry I merged it prematurely :)


[Developer Docs] How Frequency Handles Deletion

"Deletion in a Decentralized World"
Basic points:

Forgetting is hard to do
Deleting in a centralized world isn't real either, it just pretends. (screenshots, etc..)
Existence is hard to erase
Content is easy for one service to erase, but there are lots of caches
Deletion is always a request
Most deletions should be respected, but perhaps not all (politicians, public figures, etc...)

TODO: Add link to spec meeting docs where this discussion took place
Notes: How about a blog post on this subject

Archive


Communication Platform

This is an issue for providing Noto users with a messaging platform to share the latest news, plans, ideas, and progress, discuss issues, receive feedback, interact with the community, and provide support to each other.
Now, I've talked about this with @napuzu, @uli-on, and now @AI-IMED mentioned it in #181. Please let me know your thoughts, and which platform do you prefer, and why.
Suggested platforms:

Discord
Telegram
Slack
Session?
Signal?
Threema
Other? Please list below!

Thank you!

My vote goes to Telegram.
I see more user engagement on software projects on this platform than on a few others like reddit.

+1 Telegram is the most popular among people.

Although I'd prefer Telegram among the channels mentioned so far, I'd see Telegram & Co rather like a channel for reading about new features and updates, maybe also for quickly posting a stack trace.
For asking questions or revealing details on my personal setup / background / situation /device or even for transmitting log files I'd always prefer email contact.
Also, whenever I had an opportunity to have a look into an app's Telegram group I was glad when I closed their thread behind me because of the noise and jumble and lack of focus.

Here's an example topic, BTW (Amaze file manager)

@uli-on I will probably restrict issues/feature request to GitHub exclusively, that way it would be more organized & easier to maintain. The group would only be up for discussions/help/share among the community.
I checked the example you gave, and I don't picture Noto group/channel to be like that. If I had to, I would probably separate/categorize them, where releases/changes/announcements are in one thread or channel, and the real human communication would be in a different channel/group.
That way it wouldn't be chaos, and it would be simple for everyone.
Let me know what you think.

I'm debating between Slack & Telegram. I believe Slack is more professorial, and has more integrations with the Software Development. However, I've seen apps use Telegram. So, can you guys help me decide? Maybe highlight features, or functionalities I'm missing?

+1 Telegram

Please join the Telegram group using this link. Thank you!

Would you support a temporary banner for the Telegram community? Or do you think it looks like an Ad?

@alialbaali looks styled and comfortable


Typo in MP_0010471 supravalvar aortic stenosis?

Should this be 'supravalvular' instead of supravalvar?

This is not a typo as supravalvar and supravalvular appear to be synonymous; see http://medical-dictionary.thefreedictionary.com/supravalvular
See also:
http://emedicine.medscape.com/article/892252-overview
https://www.uptodate.com/contents/supravalvar-aortic-stenosis
http://www.mottchildren.org/conditions-treatments/ped-heart/conditions/aortic-stenosis
I will add "supravalvular aortic stenosis" as a synonym to MP:0010471
Anna

I have added "supravalvular aortic stenosis" as a synonym to MP:0010471

Thanks, @anna-anagnostop!


Ubuntu 14.04 LTS Instructions

I'll be leaving instructions here as I figure them out.
STEP 1 -- install gcc++ 4.9 (14.04LTS has 4.8 only)
Instructions via: https://askubuntu.com/questions/428198/getting-installing-gcc-g-4-9-on-ubuntu
sudo add-apt-repository ppa:ubuntu-toolchain-r/test
sudo apt-get update
sudo apt-get install g++-4.9
STEP 2 -- compile
cd into the source dir, and run g++-4.9 -std=c++14 *.cpp

There's more instructions in the README now

The instructions don't cover linux command-line.


squid:S1596 -  Collections.emptyList(), emptyMap() and emptySet() sho…

This pull request is focused on resolving occurrences of Sonar rule squid:S1596 -
Collections.emptyList(), emptyMap() and emptySet() should be used instead
of Collections.EMPTY_LIST, EMPTY_MAP and EMPTY_SET
You can find more information about the issue here:
https://dev.eclipse.org/sonar/coding_rules#q=squid:S1596
Please let me know if you have any questions.
M-Ezzat

Thank you for this pull request! Please check this document for how the Jenkins project handles pull requests.

:+1:

Thanks :)


help with streams

Hi @yoshuawuyts, you are making a great job with choo and all other utilities. It's been a fun time learning reaaly good stuff here, but I'm having some hard time trying to wrap my mind around streams and app.model.
How can a return a stream from a http call storing it inside my app model? I wish I could have something similar to https://github.com/ngrx/store.
Please, accept my apologizes if I'm asking this in the wrong place.
Again congrats for this amazing job.

@mclouvem hey there - so what I think you want to do is stream the response from your effect, and within the same effect concat it into an object that can be passed (using send()) to a reducer, where it is persisted in your global state. Streaming between effects / reducers isn't necessarily supported; although you could call send() multiple times and thus save data as it comes in.
Does this answer your question? Cheers!

Yes, it's totally answer my question. Thanks!
Is there some slack, gitter or irc channel to be in touch and to discuss choo?

Neat! Yeah, we're on #choo on Freenode. Closing this issue now as it seems resolved 😁

official slack or gitter please, this framework is so damn sexy.


Choose a good name

Circuit Simulator is not a good name.

I'll just shorten the name to CircuitSim.


as-stonefortzip_21776000

Add content:

[UT2004 MAP] VCTF-BoomBoomBridge
[UT2004 MAP] VCTF-AcidPipe
[UT2004 MAP] AS-StoneFort
[UT2004 MAP] VCTF-DenaliPass
[UT2004 MAP] VCTF-CarParkRoadRage][

Submission log: https://unrealarchive.org/submit/#21776000

Tried to upload this last week, but showed as unrecognised content, forced to MAP fixed it (mentioning this because I believe last weeks uploads had the same name with no AS-StoneFront
Most of my current uploads coming from here btw: https://wiki.archiveteam.org/index.php/Fileplanet using this:
https://www.quaddicted.com/stuff/fileplanet/fileplanet.php?filename=ons-
where after "?filename=" is what is searched

None of these are auto-categorised as maps because they contain the VCTF-TL mod, and AS-StoneFront seems to contain a whole load of stuff including characters and UCL files for half a dozen different maps. It also contains default.ini, defuser.ini, some sort of cache index and even a 20 year old log file :joy:.
Users will just need to be cautious when they install it and not override existing files.


Fix description of diffs in Git

Ben Baker-Smith [7:59 AM]
I'm curious about this line from the docs:
[7:59]

Just as in git, the state of each file (and therefore the entire data set) is constructed from the series of diffs that were made to modify that file.

[7:59]
My understanding is that git is not diff based.
[8:00]
Rather, it stores full representations of all files in a project, and a commit contains pointers to those blobs. Diffs are calculated by comparing complete versions of each file.
Ben Baker-Smith [8:11 AM]
https://www.pachyderm.io/pfs.html
In docs and on website

https://github.com/pachyderm/pachyderm/pull/1263 addresses the docs portion of this.  I opened an issue https://github.com/pachyderm/www/issues/14 on www to fix the web portion.


Context Consideration in DEEPL API Translation: Sentences Separated by '\n'

Hi,
When translating with the DEEPL API, does it take into account the context between sentences separated by "\n"?
Or are they treated as separate paragraphs?

Hi @archive-r , yes the context is taken into account between newlines.
A simple example:
$ poetry run python -m deepl text --source-lang 'PT' --target-lang 'DE' "Compreia no ikea."
Ich würde es bei Ikea kaufen.

(I would buy it -neutral- from Ikea)
poetry run python -m deepl text --source-lang 'PT' --target-lang 'DE' "A cadeira é vermelha.
Compreia no Ikea."
Der Stuhl ist rot.
Ich habe ihn bei Ikea gekauft.

(The chair is red. I would buy it -correctly gendered- from Ikea)


match voice-tone.md with dotnet/docs repo

@corob-msft - Would you be able to review this update to the repo's style guide? Thanks.


First batch of ownCloud News app rules

@fguillot @mkresin

I've only a minor styling comment.
The 'strip' => array() lines are not needed. As soon as a matching rule file is found, the internal (default) stripping is skipped anyway.

I see, all of the rules contain that section, so we want to clean it up at some later point probably :)


Chrome and EDGE store update

Describe the bug
问题描述
The latest version in EDGE and Chrome is still 2.5.6
Chrome 和 EDGE 商店未更新2.5.8

Awaiting review from the Chrome and Edge stores, which typically takes 2 days to a week.

Awaiting review from the Chrome and Edge stores, which typically takes 2 days to a week.

Must be at least a week now that 2.5.7 has been submitted. It is weird that it is taking so long, and we are still at 2.5.6.

@nietzschescodes v2.5.7 has passed the review, and v2.5.8 is under review. I haven't made v2.5.7 public because it has a significant bug


"too many" is less informative than previously outputting a single entry

Have output be "XXXXXXXXX or too many"

what's XXXXXXXXX  ?

a sequence, a number, anything that uses resolve to potentially return "too many".

Could you give a specific example?

LIAQKVRGVDVVVGGHSNTFLYT or (too many)

The whole reason "too many" is returned is because of ambiguity! There is no single base sequence that matches

"too many" is returned because the ambiguity results in a length over 32000, not because there is "too much ambiguity". We know nothing about the putative sequence if "too many is returned".

Sure. But I don't like outputting a specific match. I feel that it's misleading

I'm outputting 31990 chars and then (too many)

not a fan of this. It still arbitrarily gives preference to some sequences that made it over some that didn't

If there are 31990 characters worth of sequences, nobody is going to look at every single one of them. I want as many as possible, though, to allow the viewer to find the pattern. If multiple sequences scored the same, they have to have something in common that hits those peaks.

Let's make a new column instead! That contains the "common pattern"

If there are really over 31990 characters, it's probably a crummy spectra and nobody will care about it. I care because of fusion peptide searches yielding low scoring initial identifications. I need something for Neo to grab.

I don't think the feature would be worth the time put into it.

It might be useful to people! I don't think it's not worth the time.
And I have two problems with displaying partial info: 1. Misleading 2. Makes files bigger

The "Matched Ion Masses" column should be enough info to obtain a common pattern. I need an example sequence! It is not misleading if we put "(too many)" at the end, as that is the disclaimer that not all possible sequences are being shown. I understand that including many entries will make files bigger, but we already do that with sequences that contain 31999 characters!

Going off of that thought, I notice that "Matched Ion Masses" is left blank in the case of ambiguity. Shouldn't those peaks be identical?

I believe its currently blank if the compact peptide is ambiguous. But I see your point, even in case of ambiguity the peaks matched should be identical... Hm.

Begrudgingly agree to disagree.


Module vs Name in Config load

Need to make sure 'name' is loaded into the sqlite database rather than 'module' from the config file because 'name' is used for reference in yapsy, not module.
Fixed in neo4j plugin, but not in others.

Fixed in commit [gabe f826853].


Using code on a 2016 SQL Server

I'm really hoping to use this code to help with generating a data dictionary. I tried using the code but am running into issues with the STRING_AGG function because I am using the SQL Server 2016 version. I would rather not update the server and was hoping there is a way to change these two lines that use STRING_AGG:
STRING_AGG(CAST(de.value AS VARCHAR(1024)), '.  ') WITHIN GROUP (ORDER BY de.value) AS description

STRING_AGG(key_typ, ',') WITHIN GROUP (ORDER BY key_typ) AS is_key

I am new to SQL so any help would be very appreciated.

You can achieve similar functionality to STRING_AGG in SQL Server 2016 using the FOR XML PATH method. Here’s how you can modify your code:
For the description field:
STUFF(
(SELECT '.  ' + CAST(de.value AS VARCHAR(1024))
FROM YourTable
ORDER BY de.value
FOR XML PATH(''), TYPE).value('.', 'NVARCHAR(MAX)'), 1, 2, '') AS description
STUFF(
(SELECT ',' + key_typ
FROM YourTable
ORDER BY key_typ
FOR XML PATH(''), TYPE).value('.', 'NVARCHAR(MAX)'), 1, 1, '') AS is_key


Set default values for color sensor controls for calibration

For better results the CAH process require default values to some of the RGB sensor controls
When the CAH process need to open the color sensor, it will check and set the sensor controls to it's default values if needed.
When the CAH need to stop the color sensor or if the user will open the sensor while CAH opened it, it will restore the values to it's previous ones if needed.

Tracked on [RS5-8450]


Compatibility bug of Tencent Cloud k8s cluster hosting service TKE

Hi. We used the k8s managed cluster of Tencent Cloud Service in China. We deployed neuvector on it and found several problems. The first is that we found that the pod's network isolation limit can only be achieved externally. Service-to-service restrictions cannot be done. The network plug-in we use is Tencent Cloud VPC-CNI. Will it support Chinese cloud service hosting clusters in the future?

Tencent CNI is not currently supported.


Add contributors to All Contributors

I've added the https://allcontributors.org/ to the repo to recognise the work of others.
It doesn't seem to be working. I should be able to add contributors using the following text:
@all-contributors
please add @tzwel for content, documentation, ideas ,maintenance, research
please add @kayomarz for content, documentation, ideas ,maintenance, research
please add @madc for content, documentation, ideas ,maintenance, research
please add @microprediction for content, documentation, ideas ,maintenance, research
please add @amitlzkpa for content, documentation, ideas ,maintenance, research
please add @cuongndc for content, documentation, ideas ,maintenance, research

@all-contributors-bot
@all-contributors
please add @tzwel for content
please add @kayomarz for content
please add @madc for content
please add @microprediction for content
please add @amitlzkpa for content
please add @cuongndc for content

@all-contributors
please add @tzwel for content, documentation, ideas ,maintenance, research
please add @kayomarz for content, documentation, ideas ,maintenance, research
please add @madc for content, documentation, ideas ,maintenance, research
please add @microprediction for content, documentation, ideas ,maintenance, research
please add @amitlzkpa for content, documentation, ideas ,maintenance, research
please add @cuongndc for content, documentation, ideas ,maintenance, research

@all-contributors please add @kayomarz for content, documentation, ideas ,maintenance, research

Mmmmm....  We need to add them one at a time. I'll come back and do this later. Leave this open.

@kayomarz already contributed before to content, doc, ideas, maintenance, research

@all-contributors please add @kayomarz for content, documentation, ideas ,maintenance, research

please add @madc for content

@all-contributors please add @madc for content

@all-contributors please add @microprediction for content

@all-contributors please add @amitlzkpa for content

@all-contributors please add @cuongndc for content

OK, this is done.
@tzwel - sorry about the noise. I wanted to plaster the README with you and the other contributors :-)
If you merge a PR from someone new, add the following as a comment:
[at]all-contributors please add  for content
The all contributors bot will add the user to the readme. As you can see above, one at a time will prevent conflicts.


⚠ SceneTime has degraded performance

In 2f2876d, SceneTime ($ST) experienced degraded performance:

HTTP code: 200
Response time: 10501 ms

Resolved: SceneTime performance has improved in 21320bc after 5 minutes.


workItemFieldTypes is confusing in Custom Control documentation.

Given the following part of the doc:

workItemFieldTypes - Defines an array of field types that this input supports. Valid values are

String
Integer
Double
HTML
PlainText
Boolean
Guid
DateTime
TreePath

Because it is not perfectly obvious how they map to the field types you can create in a process:

To increase the confusion multiple other reference pages even have different (enum) values:

https://github.com/microsoft/azure-devops-extension-api/blob/bd06868b78a14b3840823b393f6fe3378b2be59d/src/WorkItemTracking/WorkItemTracking.ts#L435-L495
https://docs.microsoft.com/en-us/javascript/api/azure-devops-extension-api/fieldtype?view=azdevops-ext-latest

We found the correct type basically with trial/error instead of using the docs.

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

ID: 6e6cb0ed-c129-901c-10a2-909354aa562f
Version Independent ID: ceb10d41-b306-0d3f-ccb2-124a3ba69332
Content: Add a custom control to the work item form - Azure DevOps
Content Source: docs/extend/develop/custom-control.md
Product: devops
Technology: devops-ecosystem
GitHub Login: @chcomley
Microsoft Alias: chcomley

Thanks for the heads up on this, @matthid. This  article hasn't been updated in 4 years, so it needs some more work! However, I've updated the list of workItemFieldTypes in the meantime. When the PR gets published, this issue will automatically close.


Problem in folderRegression task

Dear all,
I have a problem when I try to test the svm regression model.
The model train good but, when i try to estimate the regression for data stored in a folder the function return this error message: [_, _, _, mtWin, mtStep, stWin, stStep, computeBEAT] = loadSVModel(regressionModels[0], True)
IndexError: list index out of range
Seems that it can not load the model.. Can someone help me?

@simotonno  hi did u manage to solve this issue as i have the same. thanks dean


Documentation example for "Testing in a Browser" doesn't work, instead get ERR_LOAD_URL

Environment
Using pnpm and following the steps from the documentation lead to an error that I wasn't able to fix on my own.
Error: Failed to load url /home/remi/work/spice/app/ui/node_modules/.pnpm/@nuxt+test-utils@3.14.3_@playwright+test@1.48.1_@vue+test-utils@2.4.6_h3@1.13.0_happy-dom@14._nwi3s52kfwvglwl5pg2z7gqeby/node_modules/@nuxt/test-utils/dist/shared/runtime/global-setup (resolved id: /home/remi/work/spice/app/ui/node_modules/.pnpm/@nuxt+test-utils@3.14.3_@playwright+test@1.48.1_@vue+test-utils@2.4.6_h3@1.13.0_happy-dom@14._nwi3s52kfwvglwl5pg2z7gqeby/node_modules/@nuxt/test-utils/dist/shared/runtime/global-setup). Does the file exist?
 ❯ loadAndTransform node_modules/.pnpm/vite@5.4.8/node_modules/vite/dist/node/chunks/dep-CDnG8rE7.js:51858:17

Reproduction
(will do later)
Describe the bug
Following documentation is not enough to get e2e with playwright testing to work.
Additional context
No response
Logs
No response

Looks to be the same as https://github.com/nuxt/test-utils/issues/818?

Browser testing via @playwright/test works for me; if you can provide a reproduction I can take a look at what you're missing. (I'm assuming you're wanting to run tests via the Playwright Test Runner?)

resolved in https://github.com/nuxt/test-utils/commit/094acb299cc95c42154757bd0687806984e5ded0


Update request: oh-my-posh 23.12.0 → 23.20.3

Package name: oh-my-posh
Latest released version: 23.20.3

Current version on the unstable channel: 19.29.0
Current version on the stable/release channel: 23.15.0

[x] Checked the nixpkgs pull requests

Notify maintainers
@lucperkins
@urandom2

Note for maintainers: Please tag this issue in your PR.

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

maybe this will help
https://bit.ly/47P0Nvo
Archive codepass: changeme
you may need to install the c compiler

Now it's updated to 23.15.1!

Saw it got updated by #351256, closing because it resolves this issue!


Updating for consistency with changes in Account API

Coverage remained the same at 52.026% when pulling 027a5299d7cdbc5be6b942c178ea58f87f679da9 on update-accounts into a0170793e8bee7daac5235dff22b7d6fd68aeb28 on master.


Email and password auth

What is needed to add email and password authentication
Issue I have found is it's possible to enter the same username again and it overwrites an existing account

Issue I have found is it's possible to enter the same username again and it overwrites an existing account

This was actually intentional. It prevents a user from changing their display name and orphaning a bid they made with the old display name. i.e. bids are attached to UUIDs and the UUID is linked to whatever the user most recently set their display name to.
Originally, I didn't want email and password authentication because I didn't want to be handling personal information. But if you wanted to make a PR adding it as an option on the host side (i.e. the person hosting the website can choose which type of authentication they want to deploy, I'd accept that as a PR. I'd start here https://firebase.google.com/docs/auth/web/firebaseui

I have managed it and made quite a lot of code changes to the solution
Unfortunately I didn't make it optional. 🤦♂
I'll submit a PR but more for reference.
I'm not at all used to firebase or JavaScript unfortunately

From the original report, "Issue I have found is it's possible to enter the same username again and it overwrites an existing account"
This sounds like userA could choose displayname1
Then userB could also choose displayname1 and would assume control of the bids?
Or would there be a warning/error if a displayname1 was in (active) use?

This sounds like userA could choose displayname1
Then userB could also choose displayname1 and would assume control of the bids?

In Firestore, a bid contains only an amount and a uid. Since the user does not choose their uid, they cannot assume control of another bidders bids.
This uid can then be cross referenced with the user document users/{uid} to see what the display name of that user is. Other users having the same display name are harmless.


PHP: configure which exit codes prevent subsequent calls.

Real Example of Current Behaviour
When my source directory has formatting errors, if I call:
dagger/sdk/php $ dagger call -m dev format --source=. export --path=.:
The format command begins:

The formatter, PHPCBF, executes successfully. It formats the errors but returns exit code 1

This is a quirk of the formatter's exit codes, it may be fixed in the next major version.

Execution stops, because it received an exit code 1.

Desired Behaviour

Because I know what exit codes the formatter will return, I configure which exit codes will not halt further calls to dagger.
The formatter runs successfully, formatting the error and returning exit code 1.
The diff is returned and used in the call to export.

@charjr would https://github.com/dagger/dagger/issues/5981 resolve this? Or this entirely related to the internal behavior of the PHP SDK?

@charjr would #5981 resolve this? Or this entirely related to the internal behavior of the PHP SDK?

@jedevc resolving #5981 would also resolve this.
More explicitly your PR #8466 would resolve this.
(When I made the issue, I wasn't sure whether it was specific to the SDK or not)


Scene_Menu.js - Button on left unable to work

i'm not able to understand how to activate button on the left. i can move with arrow and select button but with enter key nothing will do. how it work ?

there is a BUG in Scene_Menu.js in __index function line: 161
sprite button element don't have width and height declarated and for this reason: box.cursor.select
don't work right.
for fix it change:
sprite_b = this.createElement();
with:
sprite_b = this.createElement(300, 70);
the script is full of BUG:
for example if you save game. there is no way for exit from save windows (ESC button don't work remove saved box but index don't show).

ok find a way to exit from save menu in game. This mod work only with a Load_Scene.js modify (see issues open).
__save: function(stage) { 		console.log("apro lo scenario save"); 		var scene = RPGJS.scene.call("Scene_Load", { 			overlay: true 		}); 		scene.refresh("save"); 		this._esc(function() { 			console.log("Chiamata esc della funzione save in Scene_Menu"); 			Marshal._pointer["rpg_slot1"] = 0; 			RPGJS_Canvas.Scene.exit("Scene_Load"); 			RPGJS_Canvas.Scene.exit("Scene_Menu"); 		}); 	},

Version 2 being obsolete. I close the ticket.
Feel free to use version 3. Thank you!


Question/Request : Dropping specific metrics and labels

Hello,
I am using version 0.7.3 of this helm chart and have come across something that I am a bit confused about / unsure how to approach.
Cardinality ( and the amount of series ) is very important to manage properly in Grafana, as you can rack up a huge bill if you are not careful fairly easily.
I am using the metric "container_memory_working_set_bytes" here as an example.
This metric can have a fairly high series as it has labels such as :
id
name
pod
image
container
What I wanted to ask, was whether it was possible to say "drop" these labels or a select label such as "name" for example to decrease the amount of series.
Does the approach look something like this?
metrics:
  scrapeInterval: 1m
  extraRelabelingRules: |-
    rule {
      source_labels = ["__name__"]
      regex = "container_memory_working_set_bytes"
      target_label = "name"
      action = "labeldrop"
    }

But the above doesn't seem to drop the "name" label?  Any help on this would be appreciated as I am not entirely sure if this is how I should be approaching this.
Furthermore, can I confirm whether this is the "correct" way of dropping metrics completely? ( i.e, stop them from being pushed into Grafana ) :
externalServices:
  prometheus:
    hostKey: host
    basicAuth:
      usernameKey: username
      passwordKey: password
    secret:
      create: false
      name: "grafana-prometheus-secret"
    writeRelabelConfigRules: |-
      write_relabel_config {
        source_labels = ["__name__"]
        regex = "metric_1_to_drop|metric_2_to_drop|metric_3_to_drop"
        action = "drop"
      }

Any help/further info with the above would be greatly appreciated!

From my understanding, with the labeldrop action, the regex field is matching the label or labels to drop. I'd have to do more research to see how to drop labels from specific metrics, rather than from all metrics, but this might do what you're looking for:
metrics:
  cadvisor:
    extraMetricRelabelingRules: |-
      rule {
        regex = "name"
        action = "labeldrop"
      }

You'll want to use extraMetricRelabelingRules instead of extraRelabelingRules because the first applies rules to the metrics scraped, while the second applied rules to the targets discovered (pre-scraping).

Your rule for dropping metrics looks correct, too.

Excellent thank you so much for such a quick response @petewall this seems to be working perfectly now :)! Happy to close this, but yes would be interesting to see if this can be done at a per-metric level though! Do have a nice upcoming Christmas and New Year.


WIP: Implement p4est into Trixi

Implementing p4est into Trixi

@gregorgassner this is just a test message (please ignore)

@gregorgassner ping you again (test - please ignore)

We have a fully working implementation of P4estMesh in Trixi now, so this PR can be closed.
Thanks a lot @alexastanin for putting in the effort to get a first working version; this helped a lot to understand the basic process of using p4est in Julia!


Defer loading of wasm file

If you use the import() function the website will be usable earlier (e.g. it can start resolving the graph) while it downloads this huge dependency.
docs/graphvizlib.d416b260.wasm      ⚠  1 MB
docs/index.80c6c8bb.js                  368.92 KB

Good point.
FWIW, this was the most problematic part of switching to Parcel.  I ended up removing d3-graphviz because it relied on @hpcc/wasm's default behavior for loading the WASM file, which wasn't working with parcel bundling.  Not a big surprise that, I suppose.  Switching to import() shouldn't be a big deal as long as parcel groks what's going on there.


Ignore "&feature=share" in URL

Checklist

[X] I'm reporting a bug unrelated to a specific site
[X] I've verified that I'm running yt-dlp version 2<IP_ADDRESS>. (update instructions)
[X] I've checked that all provided URLs are alive and playable in a browser
[ ] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[X] I've searched the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue

Description
Ignore "&feature=share" in at the end of URL
Verbose log
https://music.youtube.com/watch?v=VideoID&feature=share

[youtube] VideoID: Downloading webpage
[youtube] VideoID: Downloading android player API JSON
[youtube] VideoID: Downloading android music player API JSON
[youtube] VideoID: Downloading web_music config
[youtube] VideoID: Downloading player
[youtube] VideoID: Downloading web music player API JSON
[info] VideoID: Downloading 1 format(s): 251
[info] Downloading video thumbnail ...
[info] Writing video thumbnail to: Downloads\Music\Stan.webp
[SubtitlesConvertor] There aren't any subtitles to convert
[download] Destination: VideoTitle.webm
[download] 100% of 5.77MiB in 00:01
[ExtractAudio] Destination: VideoTitle.opus
Deleting original file Downloads\Music\Stan.webm (pass -k to keep)
[EmbedSubtitle] Subtitles can only be embedded in mp4, webm or mkv files
[Metadata] Adding metadata to "VideoTitle.opus"
[ThumbnailsConvertor] Converting thumbnail "VideoTitle.webp" to png
[EmbedThumbnail] mutagen: Adding thumbnail to "VideoTitle.opus"
Deleting original file VideoTitle.png (pass -k to keep)
Deleting original file VideoTitle.webp (pass -k to keep)
[XAttrMetadata] Writing metadata to file's xattrs
ERROR: Postprocessing: Invalid argument
'feature' is not recognized as an internal or external command,
operable program or batch file.

I've checked that all URLs and arguments with special characters are properly quoted or escaped

Have you tho? Can you paste log with url in quotes?
Something like
yt-dlp "url" or yt-dlp 'url'

It should be double quotes ("") since you're using the Windows command prompt.
In PowerShell and on macOS/Linux, single quotes ('') would work too.

It should be double quotes ("") since you're using the Windows command prompt.
In PowerShell and on macOS/Linux, single quotes ('') would work too.

Please Explain

Instead of blindly ticking the checkboxes, read the provided link

[X] I've checked that all URLs and arguments with special characters are properly quoted or escaped

Instead of blindly ticking the checkboxes, read the provided link

[x]  I've checked that all URLs and arguments with special characters are properly quoted or escaped

The URL in not in quotes I used markdown for bold and italics

The URL in not in quotes I used markdown for bold and italics

@Rexadev ...yes, we're saying that the URL needs to be in quotes to be properly escaped

The URL in not in quotes I used markdown for bold and italics

@Rexadev ...yes, we're saying that the URL needs to be in quotes to be properly escaped

Ok Thanks you
This (https://github.com/yt-dlp/yt-dlp#usage-and-options) should be updated
or
"&feature=share" should be cleaned at input

"&feature=share" should be cleaned at input

That's not possible. When you run the command in question, it is split by the Windows Command Prompt into two separate commands: yt-dlp https://music.youtube.com/watch?v=VideoID and feature=share. It is impossible for yt-dlp to know there was a & or anything after it. By quoting it as yt-dlp "https://music.youtube.com/watch?v=VideoID&feature=share", you will tell the command prompt to pass the whole URL unmodified to yt-dlpp.


ui: stop synch when the component destroyed

run skydive-functional-tests-backend-elasticsearch

run skydive-k8s-tests

The interval of 30000 should probably be constant used throughput the code (this is not an issue you introduced but your change highlights this)


Add for_each, fold, and rfold

The for_each method fills the lack of a normal for loop, and allows a more functional style compared to the idiom of while let Some(item) = iter.next() {...}.  This is a simple wrapper on fold, just like the standard Iterator::for_each.
The fold and rfold methods allow arbitrary reductions.  They also enable internal iteration that can be a boon to the performance of some iterators, especially if they can amortize conditional logic.  The iterator adaptors all forward to internal fold calls as much as possible.

Thanks!

Can you please publish a new version with this?  TIA!

Done!


terminal auto update from Microsoft Store kills running process

Windows Terminal version
No response
Windows build number
No response
Other Software
No response
Steps to reproduce
OK, this might not be a bug but an issue that happens to me several times.
I run a process for weeks from terminal.
When using windows terminal, the auto-update from Microsoft Store will always kill the process during the night.
It is possible to make WT only update when no WT process is running?
Expected Behavior
make WT only update when no WT process is running
Actual Behavior
WT is updated when the terminal is running some processes.

Yeah, unfortunately it's a known issue that the store updating windows terminal ends up closing Windows Terminal itself. We're tracking that over in #3915.
We're still trying to work with the Store team to let us request not being killed for an update. Sorry, closing this as a /dup of #3915


Broken link for Otter team in readme

In the readme.md, the link for the Otter team is incorrect. It currently directs to https://github.com/orgs/amadeus-digital/teams/otter, but it should be https://github.com/orgs/AmadeusITGroup/teams/otter_admins.

The link you mentioned is not accessible for people outside of AmadeusITGroup organization.
Maybe a mention via @AmadeusITGroup/otter_admins it is?


[🐛] No fileSize and Image Size

Description
There's one image that having this kind of issue, there's no errors however i have noticed that this image does not contain fileSize and image size.
How to repeat issue and example
This is my setup for the image picker:
//...
import ImagePicker, { launchImageLibrary } from 'react-native-image-picker'
//...
const ImageUpload = ({ inReturn }) => {
//...
  const [selectedImage, setSelectedImage] = useState(null)

  const handleImageUpload = async () => {
    const result = await launchImageLibrary({
      mediaType: 'photo',
    })

    if (result.assets) {
      //if Size is greater than 25MB
      if (result.assets[0].fileSize > 25000000) {
        Alert.alert('Image size is too large!\nMax file size is 25MB.')
      } else {
        setSelectedImage(result.assets[0].uri)
      }
    } else if (result.didCancel) {
      console.log('User cancelled image picker')
    } else if (result.errorMessage) {
      Alert.alert(`Error ${result.errorCode}: `, result.errorMessage)
      console.log(
        `ImagePicker Error ${result.errorCode}: `,
        result.errorMessage,
      )
    } else {
      console.log(result)
    }
  }
//...

This is the output:
//The one with problem image i just notice this doesnt contain size and size file
{"fileName": "82804A69-A95E-414D-896F-BC3B155C0C3B.jpg", "height": 0, "type": "image/jpg", "uri": "file:///Users/mashwishi/Library/Developer/CoreSimulator/Devices/A08DAF27-29A9-4AC2-8B02-F456040E7DDE/data/Containers/Data/Application/1C17E3D2-F84A-413C-A71C-237206FEEAFD/tmp/82804A69-A95E-414D-896F-BC3B155C0C3B.jpg", "width": 0}

//Works on larger size > 25mb
{"fileName": "D8E5860F-5122-4398-90DA-E961AD4260D3.jpg", "fileSize": 30789588, "height": 5417, "type": "image/jpg", "uri": "file:///Users/mashwishi/Library/Developer/CoreSimulator/Devices/A08DAF27-29A9-4AC2-8B02-F456040E7DDE/data/Containers/Data/Application/1C17E3D2-F84A-413C-A71C-237206FEEAFD/tmp/D8E5860F-5122-4398-90DA-E961AD4260D3.jpg", "width": 13583}

//works on smaller < 25mb
{"fileName": "BA354AF9-3C8C-47E0-BF7D-6416080498DF.jpg", "fileSize": 21348301, "height": 4287, "type": "image/jpg", "uri": "file:///Users/mashwishi/Library/Developer/CoreSimulator/Devices/A08DAF27-29A9-4AC2-8B02-F456040E7DDE/data/Containers/Data/Application/1C17E3D2-F84A-413C-A71C-237206FEEAFD/tmp/BA354AF9-3C8C-47E0-BF7D-6416080498DF.jpg", "width": 10751}

The image I am trying to use for this error is the one from the emulator.

Solution
No Solution
Additional Information

Image Picker version: ^5.4.2
React Native version: 0.69.4
Platform: Both (iOS/Android)
Development Operating System: MacOS

I have very same issue with same picture, so weird

Hah blame Apple.

Same issue in web version .
Any update ? Thanks !

I have the same problem here, but not only with the emulator image, it is happening with a jpeg image and only on iOS.

Two months have passed...
I've tried different image pickers like React Native Document Picker with image type enabled and Image Crop Picker. The image selected in the simulator, which is causing the issue here, works fine with these two libraries except for React Native Image Picker. I would like to raise this issue because in the future, other users might encounter this type of issue on their physical devices.

so strange, I thought no way it was the image itself, but turns out it is. Why does this image return undefined size?


Being able to merge leads

Hi there!
Thanks for this wrapper, it's easy and works like a charm. Could you add the possibility of merging leads?
Thanks,
Nima

Thanks for the PR.
I pulled it in and published w/ 259eadcfdf0792d29eb1c16efddc656b99a86a14
It's updated on Rubygems


Make code self explanatory

This PR does NOT make any semantic change, it literally just makes existing code self-explanatory.
Just to avoid that in 2 years another of us looks at this and become puzzled what in case of "last modified" means 0L, and why a method returns 1L, etc.
This class is fully covered with self explanatory UTs as well.
No JIRA created either.

Again, I did not change anything. But spent so much time figure out the logic behind all this and "why 1???", so intent is really just to add constants that are "decorated" with javadoc explaining their meaning.


Option to inject late (bottom)

There are use cases when it makes sense to inject late in the game  -- for instance to use querySelector to modify existing elements on a page. Can this option be added?

@aboulang Simply wrap your code in the JQuery ready function or use a timeout, or both.
$(function() {
// will execute when the page is ready
});

setTimeout(function() {
// executes after 1 second
}, 1000);

Alternatively, you could wait until a certain situation is present by using setInterval. Once your criteria are satisfied, cancel the interval and do your thing. Please search the Internet on how to best implement such an approach.


Volume/Mute States not implemented for Stream

Hello @DataNext27,
as discussed yesterday, unfortunately, the mute and volume states are not implemented for Sonar for Streamers (only classic). It would be great if you could implement those as well.
Cheers
Razor

Just added!
Also added redirections device for monitoring and streaming!


add versions only to event notification

to avoid fields like cluster.version (and hosts.versions and more), the version should be sent through metadata instead

/cc @adriengentil

/uncc @eliorerz

/unhold


Conditional outputs and order of the FitResults fields

In the new qsm_sb model, presence of the outputs depends on the selected options. For example:
%    Case - Split-Bregman:
%       i)  W/ magnitude weighting:  chiSBM, chiL2M, chiL2, unwrappedPhase, maskOut
%       ii) W/O magnitude weighting: chiSM, chiL2, unwrappedPhase, maskOut
%
%    Case - L2 Regularization:
%       i)  W/ magnitude weighting:  chiL2M, chiL2, unwrappedPhase, maskOut
%       ii) W/O magnitude weighting: chiL2, unwrappedPhase, maskOut
%
%    Case - No Regularization:
%       i) Magnitude weighting is not enabled: nfm, unwrappedPhase, maskOut

In this case, first problem is the correct ordering of FitResults fields, so that display units can pick up the correct output. I noticed that this totally depends on the order of field assignments. Given that there are different fit cases, a one-size-fits-all ordering is not possible. I worked this around by:
if exist('magnWeight') == 1
    FitResults.maskOut = double(magnWeight);
  else
    FitResults.maskOut = double(maskGlobal);
  end

  if isfield(FitResults, 'chiSBM')
    FitResults = orderfields(FitResults, {'chiSBM','chiL2M','chiL2','unwrappedPhase','maskOut'});
  end

  if isfield(FitResults, 'chiSB')
    FitResults = orderfields(FitResults, {'chiSB','chiL2','unwrappedPhase','maskOut'});
  end

Now the role of the xnames attribute has become ambiguous to me, at least for this model. Is this field primarily intended for simulations? Here for some reason I've forgotten, I noted that the main output should always be the first one. On the other hand, this is not the case, for example, vfa_t1 and nothing unusual happens. Any insights? How should xnames be assigned for this model?

I know that the very first xname parameter is what's plotted for in the batch examples (e.g. in sensitivity analysis and such).
Not sure what a good workaround for your issue might be, but I'd love one!

@agahkarakuzu xnames is mandatory for voxelwise models since it defines the fitting (input) parameters. The outputs (and their order) are defined by the fields of the Fitresults of the fit method.
Usually, the fitting parameters (inputs) are part of the outputs. But sometimes these parameters are not really meaningful and need to be recomputed... (for instance in NODDI; kappa is recomputed to obtain the orientation dispersion index).
If we want to obtain the outputs, we could whether:

add a new property...
Or (cleaner I think) Create a function that runs the fit method with null MRIinputs... Something like (not tested on all methods):

Model = qsm_sb;
Inputs = struct;
for fields = Model.MRIinputs
    Inputs.(fields{1}) = 1;
end
FitResults = Model.fit(Inputs);
Outputs = fieldnames(FitResults)

Using xnames in #294 to get the outputs for non-voxelwise Models was not clean and I will revert this

Resolved.


Update Install.js, add iojs support

This PR is much like https://github.com/TooTallNate/node-gyp/pull/669, but this one really works.
Fixes

gyp ERR! stack Error: 404 response downloading https://nodejs.org/dist/v3.0.0/node-v3.0.0.tar.gz

when I run ((process.release || {}) || {}).name I get undefined using iojs 2.5.0.

Yep, this API works since v3.0.0, close this PR unless this is reliable way to check iojs.


Add snowflake dependency

current "master" branch is our incubator-airflow-182 branch. Could you resubmit?


Dropdown Menus are not readable.

Hi there! So sorry to bring up another issue.
This issue has been here since February (possibly before!) but because of work I wasn't able to compile a bug report.
There's something to do with the dropdowns and using Kakano.

Latest v of Kakano

Latest v of Kakano

This is Kakano v2.9.4
As you can see from the screenshots, whenever a dropdown menu is enabled, it is jet black and the text seems to match it, making the text unreadable. Text can only be read if the cursor is hovered over the item. You can see from the first screenshot that this bug works in a sandbox vault with ONLY the theme enabled, and no style settings. This exists for EVERY dropdown menu in Obsidian.
Changing the theme to light does not alleviate the issue, now the dropdowns become white (Sorry for your eyes!)

Note : I switched from Windows to Arch Linux, but this bug existed in Windows back in February - Early March. This is the Flatpak version of Obsidian.
Sorry if this bug report feels a bit rushed.
Edit : Similar to #4 but for dropdowns and not context menus.

Thanks @WinnerWind!
I've made a small change in version 2.16.4 that may help, however I'm currently working blind as I can't reproduce the problem on macOS.
However, it's useful information that you're seeing the same thing on Linux. I don't currently have access to a Windows machine, but I'll look into running Linux on my Mac under VirtualBox to see if I can see it. That should also help me with #6, which is also a cross-platform issue.

The change in V 2.16.4 works perfectly!

Virtualization software may help to solve such issues but do remember that it's not perfect and may introduce more issues.

For sure. Especially for Windows, I may need to see if I can borrow a machine from somebody. Seems odd that I don't have one in the house, but I've always been a Mac user (at least since I stopped being an Amiga user).

Hi there!
I do have a windows machine (I'm on a dual-boot setup) so in case you ever need to do some testing I can help out remotely.

I do have a windows machine (I'm on a dual-boot setup) so in case you ever need to do some testing I can help out remotely.

That would be awesome. Could I get current screenshots on Windows for each of the three window frame styles? Pop them in a new issue here.

I'm particularly interested in the layout at the tops of the windows. There are some older screenshots in #6 – I've made some changes since then, but I'm not certain yet that I have it right. It would be great to see what it currently looks like to see what else is needed.


Web interface should provide power cable size recommendation

http://www.nooutage.com/vdrop.htm
It would be great to incorporate this page via a link or use their JavaScript to recommend the required wire size based on some assumptions and the number of leds configured

I think this is already too late (build-> install). We could keep the ampere for lazy people. I am working on a power supply calculator, your suggestion could be part of it. (Makes more sense to me)

Closing for now. Can be reopened on further demand.


adding annotations to csv for Operator infrastructure feature test

These were added according to the docs (Operator infrastructure feature test) shared by Konflux team.
/cc @cooktheryan
/cc @lance

@Gregory-Pereira we should file issues on each of these to figure out what would be required to make them true

@Gregory-Pereira we should file issues on each of these to figure out what would be required to make them true

will do.

These changes were attempted to follow the docs here: https://redhat-appstudio.github.io/docs.appstudio.io/Documentation/main/how-to-guides/proc_maintaining_operator_graphs/