DIP0022: Multi-Quorum ChainLocks

Alternative to #85

I think this option provides a much cleaner approach to the documentation.
I'm still reviewing this and comparing it to 0008.  It'll take me having a day without distractions which I have later this week.

Rebased to fix merge conflicts after #87

OK explicit answer to @UdjinM6's question on the other thread

Should be accurate for a single LLMQ mode but yeah, for a multi-quorum mode you'd need to disrupt 2 quorums (or control 3 quorums to produce invalid clsig) at the same time, so I guess you should take all these probabilities roughly to the power of 2 (or 3 respectively) or so? Let's ping @DarrenTa :)

Quick answer: cubing is correct, squaring is not so simple.
p and q are probabilities, and are numbers between 0 and 1
So if the chance of an event happening is p then the chances of it happening three times in a row is p^3 assuming the chance of the event happening does not change.
However if the chance of an event happening is p then the chance of it not happening is 1-p=q.  If the experiment happens three times then there is a ppq chance that the event happens the first two times but does not happen the third time. The chance that the event happens the first and last time but not the second is pqp.  The chance the event does not happen the last two times but not the first is qpp.
The result is the chance the event happens exactly two times is 3p^2q
The chance that the event happens three times is p^3 (these events on these two lines are mutually exclusive)
Thus the chance the event happens at least two times is 3p^2q + q^3

A major assumption in the above comment is that the chance of the event p does not change.
Depending on how the quorums are picked this may or may not be the case.  If each quorum is a simple random selection then the assumption holds.
Alternate selection If the three quorums are picked by listing all MN in an order and picking the first few for the first quorum the next few for the second quorum and a third few for the third quorum then this is not a simple random sample.
If the attacker has less than 40% of the MNs and using the alternate selection the attacker gets over 40% in the first quorum then the attacker will be less likely to have nodes in the second or third quorum compared to a simple random selection.


Angle Conversion Between Degree and Radians

Created two functions, one that converts degree to radian, and another that performs angle conversion from radian to degrees. For both cases, init.py is updated, and generator class adde in mathgen.py

Please check and give me updates @lukew3

hey please give hacktoberfest accepted label

@lukew3 Please add the label hacktoberfest-accepted

please give the label @lukew3


Create a Virtual Yoga Studio

Pull Request Details:

Title: Create a Virtual Yoga Studio
PR Link: #19
Issue: #18

Next Steps:
I'm ready for feedback and any necessary revisions. Please review at your earliest convenience, and feel free to provide comments or suggest improvements.
Thank you for your time and consideration!

Hi User,
Thank you for your pull request. I appreciate your contribution to our project.
I have reviewed your changes and everything looks good. Please be patient, and I'll get back to you as soon as possible.
If you have any questions or need further assistance, don't hesitate to ask.
Best regards,
Jayesh


Twitter v2

from v1

Added sol_buffer_steal_or_copy
added a commit with a integer overflow fix on oauth
Use the returned buffer instead of create string (when the result of the encode is use directly)
save timestamp as long int instead of string
save the result of the encode as slice (using sol_buffer_steal_or_copy) instead of as string to avoid deal with \0
fix typo
added capabilities description
fix integer overflow
removed unecessary include
use SOL_SET_API_VERSION macro
Change the node's category

@barbieri i intend to split the node in several small nodes each one with just one functionality as you asked.
Just sending this version with the suggested fixes to not accumulate things

arr,  i forgot re-use the common code between both process, timeline and tweet. Doing it

@ceolin I would suggest (for next prs) to separate prs with fixes and prs with features.
On this PR you have 3 fixes that are independent from "adding twitter client". Even if you found them while you were writing this client.

@bdilly sure, i used to do this, sorry for this one

closed https://github.com/solettaproject/soletta/pull/1182


[1664] Handle publish errors better

Context
Making validation errors consistent across the course.
Changes proposed in this pull request
This makes the @course.publish method behave more similarly to
.save, where it automatically populates @course.errors with the
correct hash of errors. This is only possible now that the backend gives
us back pointers on the error object returned from publish.
There will be follow-up work to change this from using a flash to doing a
render instead.
Guidance to review
This will break if it's deployed without https://github.com/DFE-Digital/manage-courses-backend/pull/572.

 Errors that display on publish are persisted when users move to about/fees/requirements/salary


Plumb through a list of key-value pair options to the ffmpeg demuxer

I stumbled across the library and it seemed like it would be pretty useful, but for my scenario, I needed a way to set some specific demuxer options (see here for examples).
To that end, I added an optional parameter when loading a file to pass a set of key-value pairs that eventually get passed to avformat_open_input, and figured I'd share the changes if you find them useful.

Hello @boolemancer
Thank you very much. Sorry for the late response, I've been very busy lately.
I'm okay with the demuxer options in the FFmpeg decoder. But I think, we should not explicitly declare FFmpegDecoderOptions in the AudioPlayer. Also, adding IReadOnlyDictionary<string, string> parameter in the IAudioPlayer's Load method seems little bit hackish to me, the IAudioPlayer interface should not know FFmpeg things.
In the current design, we have 2 solutions here:

User need to create custom AudioPlayer class that override CreateDecoder method
Or, we can add an optional IAudioDecoder factory method, or a function that constructs and returns IAudioDecoder instance in AudioPlayer's constructor


Incorrect statement about customer-managed encryption keys for server-side disk encryption

The Azure secure isolation guidance document describes storage encryption and specifically Server-side encryption for managed disks. In this section, the encryption key choices are highlighted.
The customer-managed key description, however, seems to repeat. Likewise, the first description incorrectly refers to Azure Disk Encryption:

"Only key vaults can be used to safeguard customer-managed keys; managed HSMs don't support Azure Disk encryption."

The issue with this statement is that Managed HSMs are supported for server-side encryption. Also see: Customer-managed keys and Disk encryption overview comparison
Please remove the first Customer-managed keys section, as it relates to Azure Disk encryption and not to server-side encryption for managed disks also known as Azure Disk Storage Server-Side Encryption ref.

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

ID: b4d0d7bb-087f-e382-ccd0-822a4b248ff4
Version Independent ID: 4e25b074-96db-3925-7c89-ab1fb9c6c61a
Content: Azure guidance for secure isolation - Azure Government
Content Source: articles/azure-government/azure-secure-isolation-guidance.md
Service: azure-government
GitHub Login: @stevevi
Microsoft Alias: stevevi

@laurensknoll
Thanks for your feedback! We will investigate and update as appropriate.

@KrishnaG-MSFT I have signed-off on a PR to address this issue.


bridges: Remove sender field from message data.

For the reason why, see https://chat.zulip.org/#narrow/stream/127-integrations/topic/bridge.20between.20zulips.

Thanks for merging!


Added Persona 3 Reload support

Game name: Persona 3 Reload
Game package name: SEGAofAmericaInc.L0cb6b3aea_s751p9cej88mt
Used the method referenced by illusion0001 in #114 .

Hi @codeugo - Just happened to look at this and wanted to suggest adding an entry to the main README.md in the "Supported Games" legend/table

Thanks bro I successfully changed my xgp data into steam data.

I modified the readme based on @jakemer advice and fixed the spacing on @revanmj advice.
Let me know if something else is bothering you.

这是来自QQ邮箱的假期自动回复邮件。

您好，我最近正在休假中，无法亲自回复您的邮件。我将在假期结束后，尽快给您回复。

@codeugo one more thing in regards to the README.md - You might want to remove this game's entry under the "Incompatible games" https://github.com/Z1ni/XGP-save-extractor/blob/main/README.md?plain=1#L61

@codeugo one more thing in regards to the README.md - You might want to remove this game's entry under the "Incompatible games" https://github.com/Z1ni/XGP-save-extractor/blob/main/README.md?plain=1#L61

I think I've already done it, but I could be wrong. I'm honestly not used to pull request, so please enlighten me if I didn't remove the game's entry in the "Incompatible Games" list.
https://github.com/Z1ni/XGP-save-extractor/blob/8b241e5a0da2cc46d786c487f60727782c36e6ee/README.md?plain=1#L61

@codeugo My bad! I missed it in the diff - looks good 👍

how do i get my p3r saves from the xbox game pass?

这是来自QQ邮箱的假期自动回复邮件。

您好，我最近正在休假中，无法亲自回复您的邮件。我将在假期结束后，尽快给您回复。

how do i get my p3r saves from the xbox game pass?

Nvm i think i found them. Do i just encrypt them and paste them onto the game's steam save folder?

just to make sure is this the xbox save file?

it worked :D


Is anyone using this, or is there a maintained alternative?

As a chronic forgetter, I keep a ton of apps open in order to avoid overwhelming my TODO.txt - I'll notice the app eventually, which is fine. Later was doing a great job helping me here.
An Issue
Later is not working well, and I uninstalled it; not saying it's broken, as I didn't give a lot of time to debugging. Given that Alyssa isn't maintaining it anymore, is anyone aware of another app that meets the "Read it later" use case for apps? ("Facebook but for woodchucks!")

the closest one you can see is https://zenmode.app


Indent blankline v3

See https://github.com/lukas-reineke/indent-blankline.nvim/tree/v3#migrate-from-version-2

Hi, I'm getting the following selection behavior with default setup:
vip:

With master, I get

Is this intended?


jfbview: mark as broken (upstream issue)

Motivation for this change
doesn't build, incompatible with current libmupdf: https://github.com/jichu4n/JFBView/issues/17
upstream repo appears rather inactive.
cc ZHF #36453

Just merged fix for MuPDF 1.12 to the upstream repo.

@jichu4n thank you. The package can be reactivated in our next release.


Make linting an isolated step?

I was thinking if the linting process was out of the 'serve' process, for 2 reasons:

Speed up the live-reloading
Avoid having linting issues cluttering the terminal

Just a suggestion, Thanks!

This issue was moved to driftyco/ionic-app-scripts#798


[0.9.4] Windows compiled version "DLL load failed"

Hello everyone,
i just downloaded the windows compiled version 0.9.4 and tried to use the "manuel_control.py".
I looks like that there is still a problem with the DLL import.

Traceback (most recent call last):
File "manual_control.py", line 70, in
import carla
File "PythonAPI\carla-0.9.4-py3.7-win-amd64.egg\carla_init_.py", line 7, in
File "PythonAPI\carla-0.9.4-py3.7-win-amd64.egg\carla\libcarla.py", line 7, in
File "PythonAPI\carla-0.9.4-py3.7-win-amd64.egg\carla\libcarla.py", line 6, in bootstrap
File "C:\Users\AppData\Local\Programs\Python\Python37\lib\imp.py", line 342, in load_dynamic
return _load(spec)
ImportError: DLL load failed: Das angegebene Modul wurde nicht gefunden.

My system:
Windows 10
Python 3.7

In #1345  is referenced, that in #1401 the bug was fixed.
But the compiled version includes still the old code.
Where is the newest windows branch?
or
Can someone over a instruction what i have to change inside the compiled version?
Best regards
Atlunch

I could solved it by switching to Anaconda3 64 Bit and changing the font like in #1475 mentioned.
I hade problems installing pygame for Anaconda3 using  "conda install -c cogsci pygame", so i just installed pygame using "pip install pygame" directly inside the "Anaconda Prompt"
`
Wrong: Line 372:  fonts = [x for x in pygame.font.get_fonts() if 'mono' in x]
Correct: Line 372:  fonts = [x for x in pygame.font.get_fonts() if 'arial' in x]
`


GreenGrass lambda

Not sure if it's a bug and not sure if this is the right place to post this but this is the best place I can find in the maze that is AWS.
I'm trying to launch a simple lambda function named iot-function on my raspberry pi in order to discover greengrass. I have the following components running on the device:
aws.greengrass.LocalDebugConsole
aws.greengrass.Cli
aws.greengrass.Nucleus
aws.greengrass.LambdaManager
aws.greengrass.LambdaRuntimes
aws.greengrass.LambdaLauncher

I'm following this guide:
https://docs.aws.amazon.com/greengrass/v2/developerguide/import-lambda-function-console.html
Deploying my component results in a BROKEN state and the iot-function.log logs show the following:
2022-04-12T11:49:34.851Z [INFO] (pool-2-thread-56) iot-function-01: shell-runner-start. {scriptName=services.iot-function-.lifecycle.startup.script, serviceInstance=0, serviceName=iot-function, currentState=STARTING, command=["/greengrass/v2/packages/artifacts/aws.greengrass.LambdaLauncher/2.0.10/lambda-..."]}
2022-04-12T11:49:34.945Z [INFO] (pool-2-thread-59) iot-function: Setting up overlay tmpfs mounts. {serviceInstance=0, serviceName=iot-function, currentState=STARTING}
2022-04-12T11:49:34.969Z [WARN] (Copier) iot-function: stderr. 2022/04/12 12:49:34 unable to setup worker process factories: The following cgroup subsystems are not mounted: devices, memory. {scriptName=services.iot-function.lifecycle.startup.script, serviceInstance=0, serviceName=iot-function, currentState=STARTING}
2022-04-12T11:49:34.973Z [INFO] (pool-2-thread-59) iot-function: Finding mounted cgroups.. {serviceInstance=0, serviceName=iot-function, currentState=STARTING}
2022-04-12T11:49:34.990Z [INFO] (Copier) iot-function: Startup script exited. {exitCode=1, serviceInstance=0, serviceName=iot-function, currentState=STARTING}
2022-04-12T11:49:34.997Z [INFO] (pool-2-thread-54) iot-function: shell-runner-start. {scriptName=services.iot-function.lifecycle.shutdown.script, serviceInstance=0, serviceName=iot-function, currentState=STOPPING, command=["/greengrass/v2/packages/artifacts/aws.greengrass.LambdaLauncher/2.0.10/lambda-..."]}
2022-04-12T11:49:35.181Z [WARN] (Copier) iot-function: stderr. 2022/04/12 12:49:35 could not read process state file /greengrass/v2/work/iot-function/work/worker/0/state.json: open /greengrass/v2/work/iot-function/work/worker/0/state.json: no such file or directory. {scriptName=services.iot-function.lifecycle.shutdown.script, serviceInstance=0, serviceName=iot-function, currentState=STOPPING}
2022-04-12T11:49:35.412Z [ERROR] (pool-2-thread-58) iot-function: error while removing dir	{"path": "/greengrass/v2/work/iot-function/work/worker/0", "errorString": "unlinkat /greengrass/v2/work/iot-function/work/worker/0/overlays: device or resource busy"}. {serviceInstance=0, serviceName=iot-function, currentState=STOPPING}
2022-04-12T11:49:35.552Z [INFO] (pool-2-thread-54) iot-function: shell-runner-start. {scriptName=services.iot-function.lifecycle.startup.script, serviceInstance=0, serviceName=iot-function, currentState=STARTING, command=["/greengrass/v2/packages/artifacts/aws.greengrass.LambdaLauncher/2.0.10/lambda-..."]}
2022-04-12T11:49:35.639Z [INFO] (pool-2-thread-60) iot-function: Setting up overlay tmpfs mounts. {serviceInstance=0, serviceName=iot-function, currentState=STARTING}
2022-04-12T11:49:35.640Z [INFO] (pool-2-thread-60) iot-function: Finding mounted cgroups.. {serviceInstance=0, serviceName=iot-function, currentState=STARTING}
2022-04-12T11:49:35.641Z [WARN] (Copier) iot-function: stderr. 2022/04/12 12:49:35 unable to setup worker process factories: The following cgroup subsystems are not mounted: devices, memory. {scriptName=services.iot-function.lifecycle.startup.script, serviceInstance=0, serviceName=iot-function, currentState=STARTING}
2022-04-12T11:49:35.665Z [INFO] (Copier) iot-function: Startup script exited. {exitCode=1, serviceInstance=0, serviceName=iot-function, currentState=STARTING}
2022-04-12T11:49:35.669Z [INFO] (pool-2-thread-54) iot-function: shell-runner-start. {scriptName=services.iot-function.lifecycle.shutdown.script, serviceInstance=0, serviceName=iot-function, currentState=STOPPING, command=["/greengrass/v2/packages/artifacts/aws.greengrass.LambdaLauncher/2.0.10/lambda-..."]}
2022-04-12T11:49:35.783Z [WARN] (Copier) iot-function: stderr. 2022/04/12 12:49:35 could not read process state file /greengrass/v2/work/iot-function/work/worker/0/state.json: open /greengrass/v2/work/iot-function/work/worker/0/state.json: no such file or directory. {scriptName=services.iot-function.lifecycle.shutdown.script, serviceInstance=0, serviceName=iot-function, currentState=STOPPING}
2022-04-12T11:49:35.893Z [ERROR] (pool-2-thread-55) iot-function: error while removing dir	{"path": "/greengrass/v2/work/iot-function/work/worker/0", "errorString": "unlinkat /greengrass/v2/work/iot-function/work/worker/0/overlays: device or resource busy"}. {serviceInstance=0, serviceName=iot-function, currentState=STOPPING}
2022-04-12T11:49:35.983Z [INFO] (pool-2-thread-60) iot-function: shell-runner-start. {scriptName=services.iot-function.lifecycle.startup.script, serviceInstance=0, serviceName=iot-function, currentState=STARTING, command=["/greengrass/v2/packages/artifacts/aws.greengrass.LambdaLauncher/2.0.10/lambda-..."]}
2022-04-12T11:49:36.072Z [INFO] (pool-2-thread-61) iot-function: Setting up overlay tmpfs mounts. {serviceInstance=0, serviceName=iot-function, currentState=STARTING}
2022-04-12T11:49:36.072Z [WARN] (Copier) iot-function: stderr. 2022/04/12 12:49:36 unable to setup worker process factories: The following cgroup subsystems are not mounted: devices, memory. {scriptName=services.iot-function.lifecycle.startup.script, serviceInstance=0, serviceName=iot-function, currentState=STARTING}
2022-04-12T11:49:36.072Z [INFO] (pool-2-thread-61) iot-function: Finding mounted cgroups.. {serviceInstance=0, serviceName=iot-function, currentState=STARTING}
2022-04-12T11:49:36.097Z [INFO] (Copier) iot-function: Startup script exited. {exitCode=1, serviceInstance=0, serviceName=iot-function, currentState=STARTING}
2022-04-12T11:49:36.103Z [INFO] (pool-2-thread-60) iot-function: shell-runner-start. {scriptName=services.iot-functio.lifecycle.shutdown.script, serviceInstance=0, serviceName=iot-function, currentState=BROKEN, command=["/greengrass/v2/packages/artifacts/aws.greengrass.LambdaLauncher/2.0.10/lambda-..."]}
2022-04-12T11:49:36.265Z [WARN] (Copier) iot-function: stderr. 2022/04/12 12:49:36 could not read process state file /greengrass/v2/work/iot-function/work/worker/0/state.json: open /greengrass/v2/work/iot-function/work/worker/0/state.json: no such file or directory. {scriptName=services.iot-function.lifecycle.shutdown.script, serviceInstance=0, serviceName=iot-function, currentState=BROKEN}
2022-04-12T11:49:36.366Z [ERROR] (pool-2-thread-55) iot-function: error while removing dir	{"path": "/greengrass/v2/work/iot-function/work/worker/0", "errorString": "unlinkat /greengrass/v2/work/iot-function/work/worker/0/overlays: device or resource busy"}. {serviceInstance=0, serviceName=iot-function, currentState=BROKEN}

The lambda function runs on python 3.8.
I also found and ren the greengrass dependency checker which results in:
==========================Checking script dependencies==============================
The device has all commands required for the script to run.

========================Dependency check report for GGC v1.11.x=========================
System configuration:
Kernel architecture: aarch64
Init process: /usr/lib/systemd/systemd
Kernel version: 5.10.103
C library: Debian GLIBC 2.31-13+rpt2+rpi1+deb11u2
C library version: 2.31
Directory /var/run: Present
/dev/stdin: Found
/dev/stdout: Found
/dev/stderr: Found
----------------------------Commands and software packages--------------------------
Python 2.7: Not found
Python 3.7: Not found
Python 3.8 version: 3.8.9
NodeJS 12.x: Not found
Java 8: Not found
wget: Present
realpath: Present
tar: Present
readlink: Present
basename: Present
dirname: Present
pidof: Present
df: Present
grep: Present
umount: Present
mv: Present
gzip: Present
mkdir: Present
rm: Present
ln: Present
cat: Present
cut: Present
/bin/bash: Present

---------------------------------Platform security----------------------------------
Hardlinks_protection: Enabled
Symlinks protection: Enabled

-----------------------------------User and group-----------------------------------
ggc_user: Present
ggc_group: Present

----------------(Optional) Greengrass container dependency check----------------

--------------------------------Kernel configuration--------------------------------
Kernel config file: /proc/config.gz

Namespace configs:
CONFIG_IPC_NS: Enabled
CONFIG_UTS_NS: Enabled
CONFIG_USER_NS: Enabled
CONFIG_PID_NS: Enabled

Cgroup configs:
CONFIG_CGROUP_DEVICE: Enabled
CONFIG_CGROUPS: Enabled

Every dependency looks present, so I'm not sure what's going on here.

Please see: https://github.com/aws-samples/aws-greengrass-samples/issues/41#issuecomment-974845635
You must disable CGroup v2. Greengrass lambda only uses cgroup v1.

The cmdline.txt reads:
console=serial0,115200 console=tty1 root=PARTUUID=b82f4468-02 rootfstype=ext4 fsck.repair=yes rootwait cgroup_enable=memory cgroup_memory=1

correctly configured as described in the docs.
I will add systemd.unified_cgroup_hierarchy=0 as suggested, if that fixes the problem should it not be added to the docs?

Great to hear that it worked for you. It will be added into our troubleshooting soon.


Apple MPS support?

Hello,
Does anyone knows how to use apple "MPS" instead of cpu/cuda device?

Install torch with mps support then it should be enough to choose mps as backend
from mobile_sam import sam_model_registry, SamAutomaticMaskGenerator, SamPredictor

model_type = "vit_t"
sam_checkpoint = "./weights/mobile_sam.pt"

device = "mps"

mobile_sam = sam_model_registry[model_type](checkpoint=sam_checkpoint)
mobile_sam.to(device=device)
mobile_sam.eval()

predictor = SamPredictor(mobile_sam)
predictor.set_image(<your_image>)
masks, _, _ = predictor.predict(<input_prompts>)


dmesg displays different and wrong timestamp

https://stackoverflow.com/questions/52648817/difference-in-time-stamps-between-proc-uptime-and-dmesg
https://serverfault.com/questions/576139/dmesg-time-vs-system-time-time-isnt-correct

Looks fixed after https://github.com/kachick/dotfiles/issues/663#issuecomment-2221995069 🙄


build(cmake): replace TestHeaders.cmake with CMAKE_VERIFY_INTERFACE_HEADER_SETS

See https://cmake.org/cmake/help/latest/variable/CMAKE_VERIFY_INTERFACE_HEADER_SETS.html.

Interesting, thanks!
However, I am not sure if we should require CMake 3.24 already. Maybe this will exclude some of the potential users.
I will experiment with it when I have time. Right now, I am swamped with other non-units-related work. I will probably be back only at the beginning of April, so there will be no major development from my side on mp-units for the next month 😢


Bottom sheet changes

All bottom sheets should behave the way they are behaving in the HS (fading the background, getting pulled down when clicked anywhere outside etc.) A new library has been used for it.
Alerts instead of Bottom Sheets for items like No Internet Modal - list will come from the UX team
Instead of nested bottom-sheets, use screens for level 2 onwards - changes will come from the UX team
Android and iOS behave differently when it comes to bottom sheets. We should also use them accordingly. When using the default component from the library this behaviour should come up.

an Alert would look like very basic UI, it's better to replace it with a snack bar
https://www.npmjs.com/package/react-native-snackbar

For 1.4.0’s focus on account management, I think there’s an opportunity to address the bottom sheets on the primary screens it touches:

Send Screen
Receive Screen
Some of the Pan-account settings screens

@antuz123 I'm going to make a separate issue with that specific focus -- and it might be good to use issue as one we can keep returning to to track the overall progress.

With the changes on feature/send-flow-refactoring, all of the screens in the Send Flow should now be using the latest bottom sheet library/architecture.
That said, there are many other parts of the UI that aren't, and I'm not quite sure that the scope of fully refactoring everything would be small enough to fit in the current sprint (@AliMeer @antuz123)


Add link to Logging Facilities page

@spaceisfun : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@eross-msft : Thanks for your contribution! The author(s) have been notified to review your proposed change.


python311Packages.prometheus-pandas: init at 0.3.2

Description of changes
init prometheus-pandas at 0.3.2

Things done

Built on platform(s)

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

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

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

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

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

[x] Fits CONTRIBUTING.md.

@ofborg eval

All fixed up, please give it another 👀  @h7x4.

@natsukium, can you give it one more 👀 , please?
Not sure what to do about the pyproject = true; thing.

Not sure what to do about the pyproject = true; thing.

Resolved.
Final round, hopefully! :) 🤞 @natsukium plz.


OSX App can't update itself when installed

Update logic is perfectly working when using the portable version.
But, when the Application is installed via .pkg file on OSX (Located under /Applications folder.) Delta or Full updates don't work.
The exception:
An error occurred trying to start process '/Applications/AbpStudio.app/Contents/MacOS/UpdateMac' with working directory '/Applications/AbpStudio.app/Contents/MacOS'. Permission denied
System.ComponentModel.Win32Exception (13): An error occurred trying to start process '/Applications/AbpStudio.app/Contents/MacOS/UpdateMac' with working directory '/Applications/AbpStudio.app/Contents/MacOS'. Permission denied
   at System.Diagnostics.Process.ForkAndExecProcess(ProcessStartInfo startInfo, String resolvedFilename, String[] argv, String[] envp, String cwd, Boolean setCredentials, UInt32 userId, UInt32 groupId, UInt32[] groups, Int32& stdinFd, Int32& stdoutFd, Int32& stderrFd, Boolean usesTerminal, Boolean throwOnNoExec)
   at System.Diagnostics.Process.StartCore(ProcessStartInfo startInfo)
   at System.Diagnostics.Process.Start(ProcessStartInfo startInfo)
   at Velopack.UpdateExe.Apply(IVelopackLocator locator, VelopackAsset toApply, Boolean silent, Boolean restart, String[] restartArgs, ILogger logger)
   at Velopack.UpdateManager.WaitExitThenApplyUpdates(VelopackAsset toApply, Boolean silent, Boolean restart, String[] restartArgs)
   at Velopack.UpdateManager.ApplyUpdatesAndRestart(VelopackAsset toApply, String[] restartArgs)

Probably  starting UpdateMac by osascript with do shell script ... with administrator privileges can solve the problem.
It's not the best way as you mentioned here https://github.com/velopack/velopack/issues/50 but for now it can fix the problem until finding another solution

Also demo has the same result
(https://github.com/velopack/velopack/tree/develop/samples/AvaloniaCrossPlat)

By the way,

✅ Tt works when installed with Install for me Only
❌ It requires admin privileges when installed with Install for all users of this computer

Note that, it's not about the installer, installer works perfectly in both of situations. Only the updater doesn't work when installed for all users because of permission denied.

This was fixed in 0.0.571


Installing the default omnisharp in a version folder

I have added a property defaults in the package json which will include the default omnisharp version being used. This will be used at runtime to launch the default omnisharp. Along with this change we can completely drop the "experimental" suffix for the "latest" omnisharp download

To Do: Also change test-plan.md


Does not work with cordova-plugin-wkwebview since 1.2.0

I haven't updated in a while, and just tried doing so and ran into a problem.
There is some incompatibility in the way https://github.com/nordnet/cordova-hot-code-push/issues/47 was implemented which makes the web server used by the wkwebview plugin not be able to serve the files from "external storage".
The initial startup goes fine, but on the second start of the app it hangs on the splash screen and nothing loads. I connected Safari to the device and can see that the request for index.html returns a 404 from wkwebview's webserver.
The difference is that pre 1.2.0 the webview was being set to an url like http://localhost:12334/Application%20Support/..../www/index.html while the new version attempts to override the wwwFolder and simply set the location to /index.html.
Would it be feasible to revert to the old behavior? Is there any benefit from the new one?

Also note that https://github.com/Telerik-Verified-Plugins/WKWebView/ does not support cordova-ios > 4, so I'm currently using 3.9.1.
For now, I reverted to 1.1.2 of CHCP which seems to not have this issue.

Actually, I just tried 1.2.0 again and it worked. I kept installing newer versions until I found the one where this was broken. 1.2.1 works, but 1.2.2 doesn't.

The difference is that pre 1.2.0 the webview was being set to an url like http://localhost:12334/Application Support/..../www/index.html while the new version attempts to override the wwwFolder and simply set the location to /index.html. Is there any benefit from the new one?

Yes, there are benefits from the new approach. You can look into documentation.

Actually, I just tried 1.2.0 again and it worked. I kept installing newer versions until I found the one where this was broken. 1.2.1 works, but 1.2.2 doesn't.

That's an interesting observation. Difference between these versions should not be big, just some bug fixes. Maybe I "fixed" something else as well...

Actually I think there may have been some caching going on as much as I tried to avoid it. 1.2.0 on the physical device didn't work, but in the simulator it did. I think it was still using the old version somehow as much as I tried to avoid that.
So, probably false alarm, I reverted to 1.1.2 for now.

Yes, there are benefits from the new approach. You can look into documentation.

Not sure I see the part that refers to this in the documentation. The new separate folders per release feature is great, but it doesn't refer to the wwwFolder issue.
From what I can see while debugging the plugin attempts to set the wwwFolder directly to the Application Support release folder, and load /index.html from it and this is somehow incompatible with the cordova server plugin.
If it simply updated window.location to the Application Support/... folder like it did before 1.2.0 there wouldn't have been any problem.
It may be a bug in the cordova webserver plugin, I'm not denying that, but since the previous method worked, I'm just wondering why it was changed.

Sorry for the late reply.

Found just this, no explanation though as to why that is a better approach. :)

Yeah, but if you continue reading:

For each release plugin creates a folder with this name on the external storage, and puts in it all your web files. It is a base url for your project. This approach helps to solve several problems:

Files caching issue. For example, on iOS css files are cached by the UIWebView, and even if we reload the index page - new styles were not applied. You had to kill the app in the task manager to flush it, or do some hacks to change the url of the css file.
Less chances that update will corrupt the existing content, since we are using totally different folders for each release.
But if it is corrupted - we can rollback to the previous version.

So, the main idea behind "new release -> new folder" approach is that it makes plugin more stable and less chances of breaking the app, since releases are now more independent.
I think soon chcp will support cordova-plugin-wkwebview-engine plugin. Then will try to make it work with https://github.com/Telerik-Verified-Plugins/WKWebView/.

I think you are misunderstanding what I mean. The problem is not with the
different release folders but with the way it sets the start wwwfolder on
subsequent app starts.
The older versions would redirect to a deep folder via javascript, the new
versions try to set it from the backend and simply load /index.html
În Vin, 1 apr. 2016 la 10:23 Nikolay Demyankov<EMAIL_ADDRESS>a
scris:

Sorry for the late reply.
Found just this, no explanation though as to why that is a better
approach. :)
Yeah, but if you continue reading:
For each release plugin creates a folder with this name on the external
storage, and puts in it all your web files. It is a base url for your
project. This approach helps to solve several problems:

Files caching issue. For example, on iOS css files are cached by the
UIWebView, and even if we reload the index page - new styles were not
applied. You had to kill the app in the task manager to flush it, or do
some hacks to change the url of the css file.
Less chances that update will corrupt the existing content, since we
are using totally different folders for each release.
But if it is corrupted - we can rollback to the previous version.

So, the main idea behind "new release -> new folder" approach is that it
makes plugin more stable and less chances of breaking the app, since
releases are now more independent.
I think soon chcp will support cordova-plugin-wkwebview-engine
https://github.com/apache/cordova-plugin-wkwebview-engine plugin. Then
will try to make it work with
https://github.com/Telerik-Verified-Plugins/WKWebView/.
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/nordnet/cordova-hot-code-push/issues/127#issuecomment-204285136

The older versions would redirect to a deep folder via javascript

No, it was always reloading index page from the native side. Even in older versions.

The problem is not with the different release folders but with the way it sets the starting wwwfolder on subsequent app starts.

If I remember correctly, I had to switch to setting wwwFolder instead of startPage because of the issue with having # in the index.html path - https://github.com/nordnet/cordova-hot-code-push/commit/ccdaffedbdda60df51fa25fee95e4b82fa0c3ba2.
Can double-check it later. But I suspect, that fix for telerik plugin will be similar to cordova-plugin-wkwebview-engine: set for the local server www folder as /some/path/Library/Application Support/<PACKAGE_NAME>/cordova-hot-code-push-plugin/.

That is it then. With startPage it used to work.

So for Telerik WKWebview users is this a better approach to revert to 1.1.2? Is there any breaking change in later releases?

I have reverted to 1.1.2 despite using custom load options file seems to be xcompatible with 1.3.
@nikDemyankov are there any other issues I should be aware with 1.1.2 until you (if have some more energy) can bring in WKWebview Telerik version alive?
Thanks

@barocsi You can look into changelog or in closed milestones to see, what has changed since v1.1.2. I can try to make a branch, that might fix this issue, but it will work only if index page doesn't contain # in url.

@andreialecu If you want - you can try out current version with telerik wkwebview. To do that - use the following command to add plugin:
cordova plugin add https://github.com/nordnet/cordova-hot-code-push.git#telerik-wkwebview-fix

It should work, if your index page does not have # in it's url.

The branch works. Thanks!

Glad to hear that :)

is this merged in the new release?

@barocsi  Branch telerik-wkwebview-fix is not merged into the master, since there is still a problem with #, if it is used in the index page. But I can merge master into that branch, so you could use it, if needed. But be advised, that this is a fix for telerik version of wkwebview, not the https://github.com/apache/cordova-plugin-wkwebview-engine .

Merged master into telerik-wkwebview-fix branch.

@nikDemyankov Thanks for creating the telerik-wkwebview-fix, it works just as expected.
I'm wonder, could you merge the latest master into that branch again? In particular, getVersionInfo is missing, and it would be awesome to have that the branch without doing a personal fork of the branch.

@kristfal sure, will do :)

@kristfal merged and pushed the changes. Please, check it out. Should work, but I could have miss something...


Disabling hostname checking for connecting via SSL

Attempting to connect to compose.io etcd service using example in SssTest.java. It looks like compose.io uses certificates that do not matchDNS. Is there a way to disable HostnameChecker exposed by ClientBuilder or GrpcSslContexts?
CertificateException No subject alternative DNS name matching etcd found.
	sun.security.util.HostnameChecker.matchDNS (HostnameChecker.java:204)
	sun.security.util.HostnameChecker.match (HostnameChecker.java:95)
	sun.security.ssl.X509TrustManagerImpl.checkIdentity (X509TrustManagerImpl.java:455)
	sun.security.ssl.X509TrustManagerImpl.checkIdentity (X509TrustManagerImpl.java:436)
	sun.security.ssl.X509TrustManagerImpl.checkTrusted (X509TrustManagerImpl.java:252)
sun.security.ssl.X509TrustManagerImpl.checkServerTrusted (X509TrustManagerImpl.java:136)
io.netty.handler.ssl.ReferenceCountedOpenSslClientContext$ExtendedTrustManagerVerifyCallback.verify (ReferenceCountedOpenSslClientContext.java:221)
io.netty.handler.ssl.ReferenceCountedOpenSslContext$AbstractCertificateVerifier.verify (ReferenceCountedOpenSslContext.java:644)
io.netty.internal.tcnative.SSL.readFromSSL (SSL.java:-2)
io.netty.handler.ssl.ReferenceCountedOpenSslEngine.readPlaintextData (ReferenceCountedOpenSslEngine.java:482)
io.netty.handler.ssl.ReferenceCountedOpenSslEngine.unwrap (ReferenceCountedOpenSslEngine.java:1020)
io.netty.handler.ssl.ReferenceCountedOpenSslEngine.unwrap (ReferenceCountedOpenSslEngine.java:1127)

You can set the right authority via the builder:
https://github.com/coreos/jetcd/blob/master/jetcd-core/src/test/java/com/coreos/jetcd/internal/impl/SslTest.java#L46

@lburgazzoli thanks - looks like I need to use a SNAPSHOT to get access to that.

@lburgazzoli How should one set the authority correctly when there are multiple endpoints, each with distinct names and certificates?

I think there was a discussion about this issue on grpc-java about this so I need to check what solution has been implemented and reflect it on jetcd but I had no time so far. The issue is on how the grpc-java NameResolver.

Re-opening as this is still not usable. Authority needs to be set per endpoint.

@agarman I'm not sure about what we can do on jetcd side, see the following conversations:

https://github.com/grpc/grpc-java/pull/2662
https://groups.google.com/forum/#!msg/grpc-io/xtwACYjZ2p8/Vg23_VTSAAAJ
https://github.com/ExampleDriven/spring-boot-grpc-example/pull/3

@agarman @kingpong I think I need to wait for https://github.com/grpc/grpc-java/issues/4469 to be implemented to provide a solution.

@lburgazzoli understood

I updated the library version to the latest version to solve this problem
https://mvnrepository.com/artifact/io.etcd/jetcd-all


Add ability to create Extractors 100% in the GUI

These are living notes, drawing from inspiration

GUI Extractor Requirements:

Launch-able in GUI.

Implementation details

CPU-only for now
Run on your Clowder instance
Unsafe code is okay, just have a big red warning.

Support inference Endpoints, can hit them with super simple curl.

curl -X POST \
    -H "Content-Type: application/json" \
    -d '{"prompt": "Software developers start with a Hello, [MASK]! script."}' \
    http://localhost:8000/

# result:
# {
#     "outputs": {
#         "score": 0.8281897306442261,
#         "sequence": "software developers start with a hello, world! script.",
#         "token": 2088,
#         "token_str": "world"
#     }
# }

Can connect via PyClowder SDK

# from Banana.dev as inspiration, but PyClowder already does this.
from banana_dev import Client

# Create a reference to your model on Banana
my_model = Client(
    api_key="YOUR_API_KEY", # Found in dashboard
    model_key="YOUR_MODEL_KEY", # Found in model view in dashboard
    url="http://localhost:8000",
)

# Specify the model's input JSON
inputs = {
    "prompt": "Software developers start with a Hello, [MASK]! script",
}

# Call your model's inference endpoint
result, meta = my_model.call("/", inputs)
print(result)

# result:
# {
#     "outputs": {
#         "score": 0.8281897306442261,
#         "sequence": "software developers start with a hello, world! script.",
#         "token": 2088,
#         "token_str": "world"
#     }
# }

Custom preprocessing setup() and predict() functions.

Enter code via the GUI, 100% GUI.

# https://github.com/replicate/cog
from cog import BasePredictor, Input, Path
import torch

class Predictor(BasePredictor):
    def setup(self):
        """Load the model into memory to make running multiple predictions efficient"""
        self.model = torch.load("./weights.pth")

    # The arguments and types the model takes as input
    def predict(self,
          image: Path = Input(description="Grayscale input image")
    ) -> Path:
        """Run a single prediction on the model"""
        processed_image = preprocess(image)
        output = self.model(processed_image)
        return postprocess(output)

Warning & limits for Arbitrary code execution

Limit outbound (network) requests from container.
Copy the permissions from MacOS.
Before trying to install an "arbitrary code execution" extractor you have to agree to a big flashy warning & "no guarantees" policy. Just so they understand the concern.

TODO: Investigate Cog as a Docker wrapper to automate job launching. It's designed for exactly this problem.

Maybe start with SegmentAnything model for Xiaoxia Liao to use.


60935: Add "unsaved changes" indicator

Description
Add asterisk char in window title if there's unsaved changes.
https://bz.apache.org/bugzilla/show_bug.cgi?id=60935
Motivation and Context
It is a common and useful feature.
It helps users to see if they haven't saved their changes
How Has This Been Tested?
We have used the software and verified that it works and software runs as it should
Screenshots (if appropriate):
Types of changes

New feature (non-breaking change which adds functionality)

Checklist:

[x ] My code follows the code style of this project.
[ ] I have updated the documentation accordingly.

Codecov Report

Merging #647 (66f7ed4) into master (acdb648) will decrease coverage by 0.02%.
The diff coverage is 13.88%.

@@             Coverage Diff              @@
##             master     #647      +/-   ##
============================================
- Coverage     55.46%   55.43%   -0.03%
- Complexity    10179    10181       +2
============================================
  Files          1046     1047       +1
  Lines         64148    64198      +50
  Branches       7266     7270       +4
============================================
+ Hits          35579    35589      +10
- Misses        26060    26101      +41
+ Partials       2509     2508       -1

Impacted Files
Coverage Δ
Complexity Δ

.../org/apache/jmeter/config/gui/RowDetailDialog.java
11.92% <0.00%> (-0.34%)
1.00 <0.00> (ø)

...src/main/java/org/apache/jmeter/gui/MainFrame.java
40.40% <0.00%> (-0.71%)
13.00 <0.00> (ø)

.../java/org/apache/jmeter/gui/DirtyTitleHandler.java
8.33% <8.33%> (ø)
1.00 <1.00> (?)

...a/org/apache/jmeter/config/gui/ArgumentsPanel.java
44.96% <9.09%> (-1.48%)
30.00 <0.00> (ø)

...rc/main/java/org/apache/jmeter/gui/GuiPackage.java
14.38% <75.00%> (+0.84%)
14.00 <0.00> (ø)

...ache/jmeter/protocol/smtp/sampler/SmtpSampler.java
2.89% <0.00%> (-0.18%)
2.00% <0.00%> (ø%)

.../jmeter/protocol/http/control/DNSCacheManager.java
80.79% <0.00%> (-0.13%)
39.00% <0.00%> (ø%)

...meter/protocol/http/curl/StringArgumentHolder.java
55.55% <0.00%> (ø)
6.00% <0.00%> (ø%)

...ackend/influxdb/AbstractInfluxdbMetricsSender.java
100.00% <0.00%> (ø)
3.00% <0.00%> (ø%)

... and 1 more

Continue to review full report at Codecov.

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

@paivanjerry , would you please create a branch that contains only the change relevant for "unsaved changes" indicator and create a new PR?
Pull requests from master branches are really confusing.


Use colors form the theme

Currently the background of a purescript file is always white even when using a dark theme.

It seams like you can change what theme it should use in the settings. So I'll close this issue due to low priority.


Qingwei/ohlc update contract view

Coverage decreased (-0.08%) to 29.834% when pulling 418e179178cfc405789e71f2b2f9d9d528df2a94 on qingweibinary:qingwei/ohlc_update_contract_view into f9d82ca8472eecfbded8fba0e8ef4ac7e72fcb2a on binary-com:master.

Coverage decreased (-0.08%) to 29.834% when pulling e097741d113df1ea78364756130886ef7822931f on qingweibinary:qingwei/ohlc_update_contract_view into f9d82ca8472eecfbded8fba0e8ef4ac7e72fcb2a on binary-com:master.


IBM-255: Consitency & Style of Expected In & Expected Out & Expected outgoing

https://jira.digital.homeoffice.gov.uk/browse/IBM-255

We made a very explicit decision to render the content how it currently for semantic markup reasons.
Doing this will put cid id's under a column heading that doesn't apply to them, and it stops being a table of data.

The expected display of nested data has changed (IBM-255). Subtitles of nested data are not required. Consistent with the implementation of Beds OOC, where there is no title. Thus, I replicated the choice of implementation (previously approved and on master) of Beds OOC (IBM-86).
Additional benefits received form removing the ul li implementation and adding an additional tr are related to improving readability, reuse of code, & (marginal) reduced pageload time:

reduced amount of DOM elements
reduced amount of css seelctors
consistency, simplicity, and reusability of e2e test steps & feature tests.
consistency of nested elements in the centreGender html file

While I agree with most of that.
My point however is that the data is fundamentally not the same format as the other data in the table, it is a list of CIDs as opposed to a breakdown of detail that fits the existing table headings.
The column heading you're proposing putting a cid id under in this PR is 'Category' which is not semantically valid

'Category' definition (http://www.dictionary.com/browse/category) :
"1. any general or comprehensive division; a class.
2. a classificatory division in any field of knowledge, as a phylum or any of its subdivisions in biology."
I would suggest (with reference to the definition above), that a cid number is equivalent to a task force, as they are both a division of things, where each has a particular shared characteristic.

You're really reaching. A CID 'person id' which refers to an individual human being is not a category or division of things unless you're to dealing on a biological or atomic level, which we aren't.
Furthermore it is fundamentally different in nature to all the other categories in the table which refer to a collection of individuals
.
Your motivation is apparently simplifying markup, sometimes however when you have a table of data, and an unordered list of things that are loosely connected it is correct to semantically have them as such.
To put cids in the table would result in you using a table for the sake of layout, which would be a misuse of a table.
Furthermore your approach would make it incredibly confusing to navigate with a screen reader or similar accessibility device.
Therefore I'm closing this PR.
I would suggest that if you want to fulfil the original of this ticket to write some sass and leave the markup as is. - Though i'm not personally convinced that it's actually a valid thing to work on anyway, and will happily challenge the requirement.


onnxruntime: 0.5.0 -> 1.0.0

Motivation for this change
just had a release ~12 hrs ago
https://github.com/microsoft/onnxruntime/releases
updated some of the metadata, and enabled parallel-building
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[x] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Notify maintainers
cc @
[11:28:31] jon@jon-workstation ~/projects/nixpkgs (bump-onnxruntime)
$ nix path-info -Sh ./result
/nix/store/nh33kjq4nlx9zi6rv6yk5h3px766w5pw-onnxruntime-1.0.0     42.1M
$ nix path-info -Sh ./result-dev
/nix/store/p4dj9jdfzyi2rj0d0zfh5c9ql08vfwj3-onnxruntime-1.0.0-dev         42.4M

@marsam do you mind taking a look?


nimble/mesh: Make use of advertising extensions

With this patch Mesh can make use of multi instances which came with
advertising extension. It creates one advertising instance for
PB-ADV and one for PB-GATT and PROXY.
That said, it is possible to create additional instances which are
used for non mesh purpose. Note that this brings limitation on Mesh
Friendship due to timings.
To make use of it application needs to set BLE_EXT_ADV and
BLE_MULTI_ADV_INSTANCES at least to 1. Mesh will use last available
advertising instance for PB-ADV and second to last for PB-GATT and
PROXY

With latest commit in this PR and  https://github.com/apache/mynewt-core/pull/1078  it back to work.
Issue we had was that controller code start to be more strict and does not allow to mix legacy advertising HCI command with Advertising Extensions. Previous version of this PR was using Legacy HCI to start advertising but Advertising Extenstions to configure advertising instances. It is fixed now.

fixed @andrzej-kaczmarek  comments and rebased


wrong symbol order in sha512-x86_64.pl

https://github.com/openssl/openssl/blob/e9492d1cecf459261f1f5ac0eb03e9c631600537/crypto/sha/asm/sha512-x86_64.pl#L573-L574
https://github.com/openssl/openssl/blob/e9492d1cecf459261f1f5ac0eb03e9c631600537/crypto/sha/asm/sha512-x86_64.pl#L284-L286
obviously,these two codes should exchange?

Marking as inactive, to be closed at the end of the 3.4 dev cycle, barring further input


Modifying object by Iterating over its keys

I'm not sure what is happening so my title is probably off.
Here is the code:
/** @flow */
'use strict';

type Coord = $Exact<{
  x: Array<number>,
  y: Array<number>,
  label: Array<string>
}>;
type Marker = {
  	title: string,
	...Coord
};

const pawn:Marker = {
  	title:"Pawn it is",
  	x: [1,2,3],
  	y: [0,0,0],
  	label: ["x","x","x"]
};
const locations:Coord = {
  	x: [2,2,2],
  	y: [1,0,1],
  	label: ["y","y","y"]
}

// This shows no error:
pawn["x"] = locations["x"];
pawn["y"] = locations["y"];
pawn["label"] = locations["label"];
// But this does:
const keys = Object.keys(locations);
for (const key of keys)
	pawn[key] = locations[key];

Errors are all of this kind:
33: 	pawn[key] = locations[key];
                 ^ array type. Has some incompatible type argument with
33: 	pawn[key] = locations[key];
     ^ array type
Type argument `T` is incompatible:
5:   x: Array<number>,
              ^ number. This type is incompatible with
7:   label: Array<string>
                  ^ string

It seems like flow tries all possible combinations from pawn[A] x locations[B] for A and B in (x,y,label) which is odd since key is a constant and therefore A equals B. Notice that this has the same effect if I replace pawn[key] = locations[key] by pawn[key] = pawn[key].
Any idea on how to achieve that without errors (copy something from an object to another using their respective keys)?

This pattern will not be supported


Piping multiple scrapers

Hello,
what is the suggested method to pipe multiple scrapers?
I have a scaper that extract links from a page and I would like to scape them, extract other links and scape them as well.
What is the best way to do that? Using a queue?

Just use Ruia to achieve what you want, Ruia will automatically crawl

Okay, how to chain multiple scrapers in a sequential way?

Not quite understanding what you mean, can you combine it with code?
Do you mean sequential execution?

Yes, I want to retrieve URLs from a list of pages, then scrape those URLs
to find other URLs and then scrapes those last ones URLs in order to get my
information. How to pass "messages" between scrapers?
Il giorno mer 4 mar 2020 alle ore 14:35 howie.hu<EMAIL_ADDRESS>ha scritto:

Not quite understanding what you mean, can you combine it with code?
Do you mean sequential execution?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/howie6879/ruia/issues/104?email_source=notifications&email_token=ABMUJC5OEUFNYPD365IGDEDRFZKKZA5CNFSM4LBFG6IKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOENX35DI#issuecomment-594525837,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABMUJC5YQVKRFZ5YCJJFUSDRFZKKZANCNFSM4LBFG6IA
.

--
LORENZO MAINARDI

Ruia can achieve what you said, first you should read the docs:
https://docs.python-ruia.org/


ERROR!!! Problem in decrypting application request

image showing problem in decrypting application request
but I put exactly as what described on Package Plz help
how to decrypt request please guide me

@kesamani did you check with test credentials?

yes i used test  credentials

Can you please share this code snippet ?
CcAvenue.cCAvenueInit(
transUrl: '',
accessCode: '',
amount: '',
cancelUrl: '',
currencyType: '',
merchantId: '',
orderId: '',
redirectUrl: '',
rsaKeyUrl: ''
);

CcAvenue.cCAvenueInit(
transUrl: 'https://test.ccavenue.com/transaction/initTrans',
//transUrl: 'https://secure.ccavenue.com/transaction.do?command=initiateTransaction',
accessCode: '<access_code>',
      amount: '1',
      cancelUrl: 'http://<IP_ADDRESS>/merchant/ccavResponseHandler.jsp',
      currencyType: 'INR',
      merchantId: '<merchant_id>',

      orderId: '519',
      redirectUrl: 'http://<IP_ADDRESS>/merchant/ccavResponseHandler.jsp',
      rsaKeyUrl: 'https://secure.ccavenue.com/transaction/jsp/GetRSA.jsp
  );

is your merchantId is valid ?

hey its working now when i used production transUrl instead of test transUrl .
thanks for responding...........


Version 0.13.1

v0.13.1 (2019-10-29)
Bug Fixes

Don't deadlock when collecting and cloning a thread concurrently (d736895)
doc:  Correct the style.css path (2cee5cf)

bors r+


Update GraphQuery location property readOnly=false

MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Contribution checklist:

[x] I have reviewed the documentation for the workflow.
[x] Validation tools were run on swagger spec(s) and have all been fixed in this PR. How to fix?

If any further question about AME onboarding or validation tools, please view the FAQ.
ARM API Review Checklist

[ ] Service team MUST add the "WaitForARMFeedback" label if the management plane API changes fall into one of the below categories.

adding/removing APIs.
adding/removing properties.
adding/removing API-version.
adding a new service in Azure.

Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs.

[ ] If you are blocked on ARM review and want to get the PR merged urgently, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.
Please follow the link to find more details on API review process.

/azp run automation - sdk

Can one of the admins verify this patch?

@xuzhang3 Do you have any background of this change?

@xuzhang3 Do you have any background of this change?

Property marked with readOnyl will be removed in custom json.Marshal.  This parameter is  required  for back-end service and can not be ignored.
Marked with readOnly code:
func (gqup GraphQueryUpdateParameters) MarshalJSON() ([]byte, error) {
	objectMap := make(map[string]interface{})
	if gqup.Tags != nil {
		objectMap["tags"] = gqup.Tags
	}
	if gqup.ETag != nil {
		objectMap["eTag"] = gqup.ETag
	}
	if gqup.GraphQueryPropertiesUpdateParameters != nil {
		objectMap["properties"] = gqup.GraphQueryPropertiesUpdateParameters
	}
	return json.Marshal(objectMap)
}

Remove readOnly :
func (gqup GraphQueryUpdateParameters) MarshalJSON() ([]byte, error) {
	objectMap := make(map[string]interface{})
	if gqup.Tags != nil {
		objectMap["tags"] = gqup.Tags
	}
        if gqup.Location != nil {
		objectMap["location"] = r.Location
	}
	if gqup.ETag != nil {
		objectMap["eTag"] = gqup.ETag
	}
	if gqup.GraphQueryPropertiesUpdateParameters != nil {
		objectMap["properties"] = gqup.GraphQueryPropertiesUpdateParameters
	}
	return json.Marshal(objectMap)
}

/azp run automation - sdk

@xuzhang3 Do you need to mark it as required?

@xuzhang3 Do you need to mark it as required?

How to mark it as required?

@OlhaTkachenko @demyanenko
Can you help review this PR?

@xuzhang3 Is this PR ready be to merged?

@xuzhang3 Is this PR ready be to merged?

Ready!

@akning-ms Please help to merge this PR with breaking change approved. It's a swagger bugfix.


chore(gitignore): exclude .log and .stackdump files

Ignore those pesky .log files that are often generated by node and .stackdump, which is created by ConEmu.

Besides npm-debug.log what other node-related .log files are there ?
I don't remember seeing any.

It's possible that they are coming from ConEmu too. I just deleted them all, so I can't say for sure :)

LGTM (I guess there is no .log that we would want to keep, wherever it's coming from :smiley:)


refactor(window): remove redundant if condition

Description:
I was porting window operator to RxPHP and I stumbled upon this condition inside window.ts:
private openWindow(): void  {
  const prevWindow = this.window;
  if (prevWindow) {
    prevWindow.complete();
  }
  ...

I think this condition is redundant because its always true. The this.window property is set right when the property is created and the only situation it's set to null this.window = null; is in _unsubscribe() method.
I think this was supposed to handle situations where the closing Observable keeps emitting closing signals even after unsubscribing the source Observable. For example like the following diagram:
const source = hot('-1-2-^3-4-5-6-7-8-9-|         ');
const subs =            '^       !                ';
const closings = hot('---^---x---x---x---x---x---|');

However, the closing Observable is automatically unsubscribed at window.ts#59.
All the tests are passing. I didn't add any new tests because I think this is already covered in existing test cases.

Coverage decreased (-0.001%) to 97.687% when pulling d304af294322e1b0a7e9c7e2b57224af626161f7 on martinsik:window-remove-redundant-if into b9f611a7d428c0dd29491d6f2fe182ac36fac04b on ReactiveX:master.

@martinsik if there is a notification from the closing notifier after an unsubscription, then this.window will be null and without that check it will throw. What I'm not certain of, is if that's possible, or more importantly, if it is, do we have test cases to cover it. I feel like it's probably fine, but I don't want to merge this until I'm certain that scenario is covered.
cc @trxcllnt

This is exactly what I was thinking. This should test this situation: https://github.com/ReactiveX/rxjs/blob/master/spec/operators/window-spec.ts#L184
But maybe I'm missing something and this isn't covered by any test case...

This is so old and in conflict, I'm going to close it. Really sorry, @martinsik. Just trying to get the repository straightened out now that I'm on this more full time.


[e2e]: container push branch1 default slsa3

Repo: https://github.com/slsa-framework/example-package/tree/branch1
Run: https://github.com/slsa-framework/example-package/actions/runs/5307380408
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.container.push.branch1.default.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.container.push.branch1.default.slsa3.yml
Trigger: push
Branch: branch1
Date: Mon Jun 19 03:12:28 UTC 2023

Repo: https://github.com/slsa-framework/example-package/tree/main
Run: https://github.com/slsa-framework/example-package/actions/runs/5318092729
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.container.push.branch1.default.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.container.push.branch1.default.slsa3.yml
Trigger: schedule
Branch: main
Date: Tue Jun 20 03:06:01 UTC 2023

Repo: https://github.com/slsa-framework/example-package/tree/branch1
Run: https://github.com/slsa-framework/example-package/actions/runs/5328801364
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.container.push.branch1.default.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.container.push.branch1.default.slsa3.yml
Trigger: push
Branch: branch1
Date: Wed Jun 21 00:49:53 UTC 2023
Tests are passing now. Closing this issue.


[CP Staging] fix inverted values

Explanation of Change

Fixed Issues

$ https://github.com/Expensify/App/issues/52045
PROPOSAL: https://github.com/Expensify/App/issues/52045#issuecomment-2457398326
Tests

Login with a new account (important)
Complete onboarding
Create a new workspace
Go to workspace settings > More features
Enable Workflows
Go to Workflows
Click Connect bank account
Click Connect online with Plaid
Enter magic code
Verify after verification, the Plaid page will open.

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

Offline tests

Same as above
QA Steps

Login with a new account (important)
Complete onboarding
Create a new workspace
Go to workspace settings > More features
Enable Workflows
Go to Workflows
Click Connect bank account
Click Connect online with Plaid
Enter magic code
Verify after verification, the Plaid page will open.

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

PR Author Checklist

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

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

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

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

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

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

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

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is either coming verbatim from figma or has been approved by marketing (in order to get marketing approval, ask the Bug Zero team member to add the Waiting for copy label to the issue)
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

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

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

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

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

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If the PR modifies the UI (e.g. new buttons, new UI components, changing the padding/spacing/sizing, moving components, etc) or modifies the form input styles:

[x] I verified that all the inputs inside a form are aligned with each other.
[x] I added Design label and/or tagged @Expensify/design so the design team can review the changes.

[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.

Screenshots/Videos

Android: Native

https://github.com/user-attachments/assets/0e7be7d1-8772-485a-ac30-49e2f08ba6c9

Android: mWeb Chrome

https://github.com/user-attachments/assets/4648466b-9039-4f2d-bc60-dcc32a5b950b

iOS: Native

iOS: mWeb Safari

https://github.com/user-attachments/assets/da9fb69f-99eb-4398-8297-792f904a8033

MacOS: Chrome / Safari

https://github.com/user-attachments/assets/1cd3a4f4-7112-4b5f-be01-6b9da5509a9c

MacOS: Desktop

https://github.com/user-attachments/assets/d7d5aafb-23a7-478c-9318-8060f3670dbe

Code change looks fine. Will get started on the checklist once the author checklist is completed.

@mananjadhav can you please start when you can? Its a last blocker so we can work in parallel on the author and reviewer checklist

By the guidlines, Adding @hungvu193 as the c+ on the offending pr

On it

Reviewer Checklist

[x] I have verified the author checklist is complete (all boxes are checked off).
[x] I verified the correct issue is linked in the ### Fixed Issues section above
[x] I verified testing steps are clear and they cover the changes made in this PR

[x] I verified the steps for local testing are in the Tests section
[x] I verified the steps for Staging and/or Production testing are in the QA steps section
[x] I verified the steps cover any possible failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[ ] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)

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

[ ] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[ ] MacOS: Chrome / Safari
[ ] MacOS: Desktop

[ ] If there are any errors in the console that are unrelated to this PR, I either fixed them (preferred) or linked to where I reported them in Slack
[x] I verified proper code patterns were followed (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick).
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method
[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is either coming verbatim from figma or has been approved by marketing (in order to get marketing approval, ask the Bug Zero team member to add the Waiting for copy label to the issue)
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

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

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

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

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

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

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

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If the PR modifies the UI (e.g. new buttons, new UI components, changing the padding/spacing/sizing, moving components, etc) or modifies the form input styles:

[x] I verified that all the inputs inside a form are aligned with each other.
[x] I added Design label and/or tagged @Expensify/design so the design team can review the changes.

[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[x] I have checked off every checkbox in the PR reviewer checklist, including those that don't apply to this PR.

Screenshots/Videos

Android: Native

Android: mWeb Chrome

iOS: Native

iOS: mWeb Safari

MacOS: Chrome / Safari

MacOS: Desktop

@mananjadhav Sorry, I will take over this as this came from the PR that I reviewed.

Okay sure @hungvu193. I tried but the resend validate code is taking unexpectedly longer.

Thanks for understanding  🙏

I apologize for the delay, everyone. The videos took longer than expected due to an unexpected error that I started encountering as well.

Yes, if you created many new accounts on a laptop, you will face that issue.

Reviewer Checklist

[x] I have verified the author checklist is complete (all boxes are checked off).
[x] I verified the correct issue is linked in the ### Fixed Issues section above
[x] I verified testing steps are clear and they cover the changes made in this PR

[x] I verified the steps for local testing are in the Tests section
[x] I verified the steps for Staging and/or Production testing are in the QA steps section
[x] I verified the steps cover any possible failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)

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

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

[x] If there are any errors in the console that are unrelated to this PR, I either fixed them (preferred) or linked to where I reported them in Slack
[x] I verified proper code patterns were followed (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick).
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method
[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is either coming verbatim from figma or has been approved by marketing (in order to get marketing approval, ask the Bug Zero team member to add the Waiting for copy label to the issue)
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

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

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

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

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

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

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

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If the PR modifies the UI (e.g. new buttons, new UI components, changing the padding/spacing/sizing, moving components, etc) or modifies the form input styles:

[x] I verified that all the inputs inside a form are aligned with each other.
[x] I added Design label and/or tagged @Expensify/design so the design team can review the changes.

[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[x] I have checked off every checkbox in the PR reviewer checklist, including those that don't apply to this PR.

Screenshots/Videos

Android: Native

Android: mWeb Chrome

iOS: Native

iOS: mWeb Safari

https://github.com/user-attachments/assets/95b34881-4955-421c-9bfc-a5146641798d

MacOS: Chrome / Safari

https://github.com/user-attachments/assets/321fa88a-48b8-43f6-9eee-29194731443f

MacOS: Desktop

https://github.com/user-attachments/assets/edc89c7d-b8e3-4077-81d9-e2f79df8a1a2

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


How to insert a reducer into other reducer where not in root state

I have two pages:
/expense_reimbursement/receipt
/expense_reimbursement/receipt/add

The pages both connect to redux, and I want to insert the second reducer into the first reducer.
I tried follows, but I can only find the first reducer in root state.
store/reducers.js
export const injectSubReducer = (store, {pathString, reducer}) => {
  const pathArr = pathString.split('/')
  store.asyncReducers[pathArr[0]][pathArr[1]] = reducer
  store.replaceReducer(makeRootReducer(store.asyncReducers))
}

Add/index.js
export default (store) => ({
  path: '/expense_reimbursement/receipt/add',
  getComponent(nextState,cb) {
    injectSubReducer(store, {pathString: 'receipt/add', reducer: reducer})
    cb(null, Container)
  }
})

Did you find a solution to this problem? I'm having the same questions...

Try to hold your redux store as flat as possible. For more read here.


CI: forced regenerations of endpoints due to key-value store event causes CI failures

We are reliant in the CI upon endpoints transitioning into "ready" state and staying in that state. However, since Cilium is event-driven, we might receive some sort of key-value store event that triggers endpoint regenerations. This causes failures like the following:

https://github.com/cilium/cilium/issues/3622
https://github.com/cilium/cilium/issues/3467

For a more detailed analysis of how this occurs, see : https://github.com/cilium/cilium/issues/3467#issuecomment-378035614
We need to figure out how to make the CI resilient to such issues. Specifically:

It is entirely possible that we can have endpoints go from being into ready state, to having a key-value store event consumed for an identity, which then forces a policy recalculation for all endpoints, thus putting them into waiting-to-regenerate state. Thus, this isn't a cilium bug; we need a better way in the test framework of having some sort of guarantee that the endpoints are in ready state and that they will not change such state unless the user performs some sort of operation which triggers change of state (policy imported, deleted, endpoint forcibly regenerated, labels changed on endpoints, etc.).

I don't know how feasible this is. Is this feasible with the current state of Cilium?

Long solved


⚠ AlphaRatio has degraded performance

In fed5b01, AlphaRatio ($AR) experienced degraded performance:

HTTP code: 200
Response time: 10167 ms

Resolved: AlphaRatio performance has improved in 655be1e after 5 minutes.


Configuring kotlinx-kover in a multimodule Android project where only the app module has defined flavors

I’m having trouble configuring kotlinx-kover in a multimodule Android project where only the app module has defined flavors.
./gradlew koverHtmlReport fails with
Could not determine the dependencies of task ':app:koverHtmlReport'.
> Could not resolve all task dependencies for configuration ':app:koverExternalArtifacts'.
   > Could not resolve project :core:designsystem:common.
     Required by:
         project :app
      > The consumer was configured to find attribute 'kotlinx.kover.variant.name' with value ''. However we cannot choose between the following variants of project :core:designsystem:common:
          - debugRuntimeElements
          - releaseRuntimeElements

how should I configure it?

Hi,
could you clarify, did you apply the Kover plugin in the :core:designsystem:common subproject?

No, I didn't. Should I do it?

Yes, according docs:

it is necessary to apply Kover plugin in each subproject for which coverage needs to be measured.


Latency stats in BitCraft client

Closed by https://github.com/clockworklabs/com.clockworklabs.spacetimedbsdk/commit/41031248a0c15b57c3ede7a8b7cc22aafb1c378a


Attempt to re-create ChannelBind on failure

Right now when we attempt to create the ChannelBind we don't re-attempt on failure.

Could you please clarify?


Cannot typeText in input with type=number if emulating userAgent

What is your Test Scenario?
I am trying to run my test while emulating the Chrome userAgent according to the spec found here:
https://devexpress.github.io/testcafe/documentation/using-testcafe/common-concepts/browsers/using-chrome-device-emulation.html#emulator-parameters
What is the Current behavior?
When trying to change the value of a number input

Using await t.typeText(input, "12");:
It will type the first number indefinitely.
Using await t.typeText(input, "12", { replace: true });:
the running will hang until the test crashes with the output:
1) Uncaught object "[object Object]" was thrown. Throw Error instead.
What is the Expected behavior?
The number input is able to be changed with or without {replace: true}
What is your web application and your TestCafe test code?

Your complete app code:

  Number of whatever (10-100):

  <input
    type="number"
    id="number-input"
    name="number-input"
    min="10"
    max="100"
  />

My complete test code:

 import { Selector } from "testcafe";

fixture(`User agent - number input bug`).page(`http://<IP_ADDRESS>:8080`);

test("Number input is able to be changed", async t => {
  const input = Selector('input[name="number-input"]');

  await t.expect(input.exists).ok();
  await t.typeText(input, "12");
});

Your complete test report:
testcafe "chrome:emulation:userAgent='foo'" test.js
 Running tests in:
 - Chrome 70.0.3538 / Mac OS X 10.13.6

 User agent - number input bug
 ✖ Number input is able to be changed

   1) Uncaught object "[object Object]" was thrown. Throw Error instead.

      Browser: Chrome 70.0.3538 / Mac OS X 10.13.6

 1/1 failed (17s)

Steps to Reproduce:

Save the html locally and start the saved html file in a local server using something like http-server
Save the test js locally in the same directory as test.js
testcafe "chrome:emulation:userAgent='foo'" test.js
See the error...

Your Environment details:

testcafe version:                   0.23.2
node.js version:                    11.1.0 (also tried 10.8.0 and 9.11.1)
command-line arguments:  testcafe "chrome:emulation:userAgent='foo'" test.js
browser name and version: chrome 70 (also tried chrome 63)
platform and version:          "macOS 10.13.6"

I am having the same issue.

Could you please provide me with the exact user agent string and command line parameters that you are using in the testcafe command?
TestCafe is a cross-browser application, so it uses user agent strings to detect the browser type to define a correct event simulation order and other browser-dependent details that can't be defined by the feature detection.
It means that TestCafe can't work properly if you specify an incorrect user agent string. It can be hard to pass a correct user agent using a command line because user agent strings include special characters that your shell or TestCafe argument parser can remove from the string and treat it in a special way. These modifications will invalidate the user agent string so you need to escape these characters in the way supported by your shell.
The list of special characters contains the following symbols:

\ (backslash)
' (single quote)
" (double quote)
, (comma)
; (semicolon)
: (colon)

The rules of escaping special characters are complex and have minor differences in different shells, so I've created a separate ticket about adding these rules to our documentation: #3116.
Yet, I can give you an example of a properly escaped user agent string for the bash and PowerShell shells. I can't share an example for the cmd shell, because it's very hard to properly escape control characters there and all modern Windows systems (Windows 7+) have PowerShell preinstalled.
PowerShell
testcafe 'chrome:emulation:userAgent=''Mozilla/5.0 (Windows NT 10.0\; Win64\; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/70.0.3538.77 Safari/537.36''' test.js

bash
testcafe $'chrome:emulation:userAgent=\'Mozilla/5.0 (Windows NT 10.0\\; Win64\\; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/70.0.3538.77 Safari/537.36\'' test.js

The issue is not reproduced when I run these examples.

We figured out a workaround by adding an arbitrary string at the end of the legitimate userAgent you provided.
I'm still curious if userAgent is never meant to support arbitrary strings?


Only add a dash when there is a prefix

otherwise the generated name is invalid as a helm release name
(and possibly other uses).
Needs also: https://github.com/epinio/helm-charts/pull/270/files
Fixes #1696

Codecov Report
Base: 62.17% // Head: 51.59% // Decreases project coverage by -10.57% :warning:

Coverage data is based on head (659bccc) compared to base (4d2b028).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files
@@             Coverage Diff             @@
##             main    #1727       +/-   ##
===========================================
- Coverage   62.17%   51.59%   -10.58%
===========================================
  Files         170      170
  Lines       12874    12876        +2
===========================================
- Hits         8005     6644     -1361
- Misses       3858     5342     +1484
+ Partials     1011      890      -121

Flag
Coverage Δ

acceptance-api
48.57% <100.00%> (+0.03%)
:arrow_up:

acceptance-apps
37.48% <100.00%> (+<0.01%)
:arrow_up:

acceptance-cli
?

unittests
8.77% <100.00%> (+0.02%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

internal/names/names.go
98.00% <100.00%> (+0.08%)
:arrow_up:

pkg/api/core/v1/models/namespace.go
0.00% <0.00%> (-100.00%)
:arrow_down:

internal/cli/usercmd/chart.go
0.00% <0.00%> (-75.17%)
:arrow_down:

internal/cli/admincmd/settings.go
0.00% <0.00%> (-68.89%)
:arrow_down:

internal/api/v1/env/show.go
0.00% <0.00%> (-67.57%)
:arrow_down:

internal/cli/usercmd/env.go
0.00% <0.00%> (-66.02%)
:arrow_down:

internal/cli/settings.go
12.12% <0.00%> (-58.59%)
:arrow_down:

helpers/bytes/bytes.go
41.66% <0.00%> (-58.34%)
:arrow_down:

internal/api/v1/env/index.go
0.00% <0.00%> (-57.15%)
:arrow_down:

internal/api/v1/env/unset.go
0.00% <0.00%> (-55.89%)
:arrow_down:

... and 51 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


refactoring ui-scrollpoint structure

rebuilt ui-scrollpoint using extendable controller
redesigned demo
replaced ui-scrollpoint-bottom with configurable ui-scrollpoint-edge

added element.ready(function({ })) wrappers in unit tests
Important Changes

replace ui-scrollpoint-bottom with ui-scrollpoint-edge="bottom"
ui-scrollpoint-action callback parameters reversed

old: function(element, distance)
new: function(distance, element, edge)

Enhancements

edge parameter for action callbacks - "top" or "bottom" depending which edge of the target the scrollpoint hit
multiple edges per scrollpoint.

ex. ui-scrollpoint-edge="['top','bottom']" to trigger when scrollpoint hits top or bottom of its target

mapping edge of screen to edge of element.

ex. ui-scrollpoint-edge="{top: 'bottom'}" triggers when the bottom of the element hits the top edge of target

ui-scrollpoint-edge="view" to trigger when element scrolls out of view (equivalent to ui-scrollpoint-edge="{top: 'bottom', bottom: 'top'}"

I have the ui-scrollpoint-pin directive on a separate branch.  It's ready to merge in here, but I'm not sure if it would be out of the scope of this project.  I could set it up as a separate component if we don't want it in the core ui-scrollpoint module.

See issue #12 . I tried out this refactored branch as well, and had the same failure. I'm excited about all the new features, once we can get the core functionality back on track.

Latest commit addresses issue #12

Great! I'll give it a try.

I also posted a few notes about position: fixed; / sticky elements / pinning feature in #12.

I can confirm that it works now. I hope this is merged soon so that the head is no longer bugged. :+1:

@PowerKiKi just wanted to draw your attention to the fact that a substantial bug (#12) was introduced in the 1.2.1 release, and seems to be fixed by this pull request.

Thanks ! Releasing as 2.0.0 since there is a BC break.

Just wanted to note that I have set up a ui-scrollpoint-pin component at https://github.com/TechNaturally/ui-scrollpoint-pin


metaphlan2-2.96.1-py_0.tar.bz2---database

Hi,
Which database can I use with metaphlan2-2.96.1-py_0.tar.bz2  ? v296 or V30,
Can you put the database in the github?
Thank you

I'd suggest to use v296, but if you can, I'd also suggest you to update to the latest build


Check for null when looking for a git directory and show an appropriate ...

...error. Fixes issue #406.
By fixing I mean it throws an exception describing the issue rather than propagating a null further.

Thanks for the PR