Fzf on Windows doesn't accept keyboard input

[X] I have read through the manual page (man fzf)
[X] I have the latest version of fzf
[X] I have searched through the existing issues

Info

OS

[ ] Linux
[ ] Mac OS X
[X] Windows
[ ] Etc.

Shell

[ ] bash
[ ] zsh
[ ] fish
[X] Powershell

Problem / Steps to reproduce
I just updated to 0.46.0, and fzf doesn't accept any input (arrow keys, letters).
Version 0.45.0 works, I downgraded and tested.
I am using the Windows terminal with the latest powershell version.
Installed using the winget install fzf command.
Steps to reproduce
Any query with fzf (even just the directory search, fzf with no args doesn't work.)

Thanks for the report. Though I don't have a Windows PC to test it, I think da753fc should fix it.
I have replaced the binaries on https://github.com/junegunn/fzf/releases/tag/0.46.0, can you manually download the one for your platform and test if the problem is fixed?

I also encountered the issue, it now works with the updated binaries, thank you!

@junegunn I confirm it works, thanks a lot!

@junegunn I can confirm it's fixed; thanks!


Does bug prediction support human developers? findings from a google case study

http://dl.acm.org/citation.cfm?id=2486838

Proprietary google code


Add better starting instructions

There are some instructions currently in the README, but I'm not sure how to do any of the things they mention:

Update setenv to use your xtensa enabled rustc, then simply run source setenv. From then on, you can just call xargo build or use the built in flash script to build and flash to the esp.

It would be great if the instructions were spelled out in full (i.e. with full commands), and perhaps a sample project added, showing how to flip a few GPIOs or whatnot. I think that would be much more helpful in getting newbies started.

This has been an ongoing process, but I'd like to think the instructions are much easier to follow now.
In particular, its gotten considerably easier to build the entire toolchain with https://github.com/MabezDev/xtensa-rust-quickstart#recommended-build-method
I will close this now, but feel free to follow up if you feel its still not at good as it should be!


Update/improve documentation

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (701617a) 95.53% compared to head (78ccf11) 95.55%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #79      +/-   ##
==========================================
+ Coverage   95.53%   95.55%   +0.02%
==========================================
  Files          51       51
  Lines        3938     3938
==========================================
+ Hits         3762     3763       +1
+ Misses        176      175       -1

Flag
Coverage Δ

unittests
95.55% <ø> (+0.02%)
:arrow_up:

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

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


build: Bump Node.js to v16

as node12 is EOL since 2022-04-30.

This should not be breaking anything,
as all the runners are running this in node16 anyway.


Android Support

Hi! This is an awesome project. I've been looking for a database implementation I can share with Desktop/Server and Android. Will this work with Android?

Maybe a guide on how to get started with Android?

Hello @bangonkali!
Thanks for checking the project out! I made the README more clear on Android support.
I have build a specific Android version of RocksDB using the setup at: https://github.com/marykdb/rocksdb-android and use that in this Kotlin Multiplatform project.
I also added a quick usage example to the README. The API is the same as the JVM RocksDB version. So you can just add the dependency to a Kotlin enabled Android project and then you can start using this project as in the example.
RocksDB is a lower level key value database which is used as the foundation of multiple databases. You work more with the raw byte arrays. I myself try to implement a higher level database on top of this which works across platforms, but this is not published yet.
If you have any questions please ask. 😄


Replace ItemList with custom list in add_node_popup

#219

Hmm I'm not sure I like the new behavior with the filter (the filter remains, but is not always applied when showing the popup).

Hmm @bonbonmiel, thanks for that contribution, I modified a few details and added quick connect for inputs. Please tell me if you like the result (or if you find bugs I'd have introduced) so I can merge this PR.

Hey,

The filter was purposely not filtering when quick-adding a node because the options have already been filtered though a compatibility pass (so I figured it was better not to automatically re-filter them through the current search).

I'm personnally happy with the ui and the feature set but there was feedback in the issue too so maybe it could be polished a bit before merging (e.g. the filter behavior).
(also for the record, quick-connect for inputs closes #99.)

There is also a bug where custom nodes you just added do not show in the list before you restart the program but I guess this doesn't have anything to do with this popup.

Regarding paterial spray, I didn't use it seriously for now. I'll sure give feedback when I do!

Well, that's Material Spray, I just typed a bit too fast (and too far from the center of keys) when commiting.


Possible error(s) in extended Kalman particle filter update

I think there may be some errors in the update step of the extended Kalman particle filter, but it is also possible that I am misunderstanding some things.
Below is a screenshot from page 56 of https://books.google.co.za/books?id=zABIY--qk2AC&pg=PA37&source=gbs_toc_r&cad=3#v=onepage&q&f=false

The likelihood seems to be based on the sampled state $x_{k}^{i}$, and not the filtered EKF estimate $x̂_{k}^{i}$. However in the code the likelihood is computed from the filtered EKF estimate propagated_state instead of the sample updated_state.

Should the prior not be computed as $\mathcal{N}(x_{k}^{i}; x_{k-1}^{i} + B_ku_k, Q)$ i.e. using the state propagated by the process model and not the filtered EKF estimate? In the code the mean used is  propagated_state and not $x_{k-1}^{i} + B_ku_k$.

Finally, should the particle states not be set to updated_state instead of propagated_state in line 176? From the screenshot above the EKF filtering step makes use of the previous particle's sampled state $x_{k-1}^{i}$ and not $x̂_{k-1}^{i}$.

https://github.com/jelfring/particle-filter-tutorial/blob/8d1304ec23e634353d108a2795f6c492b330344b/core/particle_filters/extended_kalman_particle_filter.py#L161-L176
If the current implementation is correct, can an explanation be provided? I am just trying to learn :)

The likelihood seems to be based on the sampled state xki, and not the filtered EKF estimate x̂ki. However in the code the likelihood is computed from the filtered EKF estimate propagated_state instead of the sample updated_state.
Should the prior not be computed as N(xki;xk−1i+Bkuk,Q) i.e. using the state propagated by the process model and not the filtered EKF estimate? In the code the mean used is  propagated_state and not xk−1i+Bkuk.

Short answer: you are right. The naming of some of the variables is confusing, which makes it hard to understand the code and which, even worse, led to some mistakes that are not easy to spot. Thanks for pointing this out!

Finally, should the particle states not be set to updated_state instead of propagated_state in line 176? From the screenshot above the EKF filtering step makes use of the previous particle's sampled state xk−1i and not x̂k−1i.

Yes indeed.
@Paulkie99 , would you be willing to create a PR that fixes the problem? I just created this branch which I believe should solve the problem (if you prefer and agree I can merge that branch instead).


Tag a new release?

There're a couple of fixes since 3.1.0 — could you do a 3.1.1 for these?

Tagged as 3.2 because the default config file changed.

Thanks!


Add Leaky ReLU activation function

It seems the Leaky ReLU activation function is missed in KotlinDL. I would love to contribute this if it sounds good.

@Rishit-dagli Actually, it has been already implemented as an activation layer; see here.

Ah, got it, I was looking for an activation function. Thanks!


sqltest

Avatar,Planet Bomber,and Spy Probe will report the "NFCoosen" in the page fleetStep1
Flying Death are not displayed in page fleetTable
In addition,i find some ship name are not conformity with their text，such as the text for ”Star Fighter“ are the destroyer's，and very much ship and buildings are have no text only has the name，I don't know if you didn't have time to modify it or didn't notice it,so i reminder.
Sorry for this is my first time to use the github，if I do something wrong please tell me.
At last,i'm translating the game into Chinese,i want to know is there anything need to take care？
thanks very much

I wanted to write a story for the game and connect with the ships. But lately I've been out of time. Now I'm working on version 3.x.x. According to the plan, it will be released at the end of June-August this year. You can help by simply reporting bugs and fixing them. Thank you.


FC Networks - Assign Bandwidth

Hi,
When creating an FC Network do you know if there is currently an option to set the Preferred/Max bandwidth?
Cheers
Mark

At this time there is no option to set this via Terraform, sorry. All the available options should be documented in this repo.

Hi,
Is there any chance that this is going to be updated to allow these options to be set in the future?


[Snyk] Upgrade @tanstack/react-query-devtools from 4.20.4 to 4.20.9

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to upgrade @tanstack/react-query-devtools from 4.20.4 to 4.20.9.
:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 1 version ahead of your current version.
The recommended version was released 22 days ago, on 2023-01-04.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:
🧐 View latest project report
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

Codecov Report
Base: 80.00% // Head: 80.00% // No change to project coverage :thumbsup:

Coverage data is based on head (8f096eb) compared to base (e77b208).
Patch has no changes to coverable lines.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #483   +/-   ##
=======================================
  Coverage   80.00%   80.00%
=======================================
  Files          31       31
  Lines         365      365
  Branches       60       60
=======================================
  Hits          292      292
  Misses         67       67
  Partials        6        6

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.


StorageImageReadWithoutFormat should not be required for feature level 11_0

DXVK requires StorageImageReadWithoutFormat for feature 11_0 support. It shouldn't be required. Only the following 3 formats require the UAV typed load support in the 11_0 feature level: R32_UINT, R32_SINT, R32_FLOAT.

It might be true that this feature should technically not be required, but since there's no way to determine the exact UAV format accurately at compile time, I don't really see how to work around that without ruining performance at draw time.
Just assuming that a UAV with a float return type is R32_FLOAT when a load instruction is used sounds like an incorrect assumption.

09151f36168bad2630a3c642a8f4401a65e87ed5 should work on GPUs which do not support this capability as long as the application does not use typed loads for any other format.


command not found: inferTraceBugs

i install infer from brew ,but the  inferTraceBugs not found

Is this infer 0.9.2?

Infer version v0.8.1

Upgrading to 0.9.2 with brew should fix this. Please reopen if it doesn't.

Infer version v0.13.1 also has this problem

@crespoxiao it has been replaced by infer explore. See #881.

thank you. " infer explore --html"  works     @jvillard


Add Pnm Kaimei riddles support

Add support for Pop'n Music Kaimei Riddles.
It's mainly based on the same code as peace, but with more (documented) params and additional code in (un)format_profile in order to manage the Kaimei! MN tanteisha event and new values in player profile.

Tested this all out locally, seems to work perfectly. Thanks again!


ci: Introduce stack leak detection for BPF host routing

To cover https://github.com/cilium/cilium/pull/35098.

It can be something like "drop all iptables rules" installed for bpf host routing, so if any stack leak takes place, connectivity breaks.

Using bpftrace to probe skb->sk == NULL at native egress hook might be another possible solution.


[Breadcrumb] Do not preserve messages in the intermediate segments

I have the following message while I did not perform any action as my last navigation behavior.

Steps to reproduce:

Go to patient devices.
Remove device assignation
Now the message from the print screen appear and it is OK.
Go to patient devices again.
Click on trial subjects from breadcrumb menu
Now the message from the print screen appear and it is NOT OK.

Proposed solution: in BreadcrumbNavigatorController

cleanup the message if the segment is not the active/last one

@Mastaleru I pushed the solution for this. Please check this commit: https://github.com/PharmaLedger-IMI/eco-iot-services/commit/9ebb639bbb38377fa5e8546ae179e9010f4a8a25

@cosmintuca check my comment on your commit https://github.com/PharmaLedger-IMI/eco-iot-services/commit/9ebb639bbb38377fa5e8546ae179e9010f4a8a25#r80177631

@Mastaleru I've made an update on the changes in this commit: https://github.com/PharmaLedger-IMI/eco-iot-services/commit/4d3e807a0689b827a288ef2605f31d9a5972efd8


Loading data batch Killed

I am training custom data in linux 20 "ns-train nerfacto --data [data path] " and getting below error
The viewer bridge server subprocess failed. Please check the log file outputs/destdir/nerfacto/2023-02-07_074307/viewer_log_filename.txt.
You likely have to modify --viewer.zmq-port and/or --viewer.websocket-port in the config to avoid conflicting ports.
Try modifying --viewer.websocket-port 7007

That means something is probably already running on port 7007. Try adding --viewer.websocket-port 7008 to your command.

Thanks for the reply
Now, I understood the problem that it was using 7007 port as I was running training command multiple times  as It was not working in the first time.
First time training process atuomatically stops without any error it is just showing Killed while loading data command output shown below
Sending ping to the viewer Bridge Server...
Successfully connected.
Sending ping to the viewer Bridge Server...
Successfully connected.
[NOTE] Not running eval iterations since only viewer is enabled. Use --vis wandb or --vis tensorboard to run with eval
instead.
Disabled tensorboard/wandb event writers
[08:38:35] Auto image downscale factor of 1                                                 nerfstudio_dataparser.py:314
Skipping 0 files in dataset split train.                                         nerfstudio_dataparser.py:165
Skipping 0 files in dataset split val.                                           nerfstudio_dataparser.py:165
Setting up training dataset...
Caching all 288 images.
Loading data batch ━━━━━━━━━━━━╺━━━━━━━━━━━━━━━━━━━━━━━━━━━  31% 0:00:39Killed

What is the correct way to end training for a certain port?

The server should be killed when the training process ends (ctrl + c), however in some rare cases when the training crashes the server doesn't get closed. When that happens, I just work with a different port ;), not sure the best way to kill that process when that happens.

Thanks for the prompt response. I tried ctrl + c on several ports and none of them was closed properly and the port becomes unusable :(
How many other ports could we possibly use?

The nuclear option is killall -9 python, or on Windows, taskkill /im python exe /f

Thanks for the reply Now, I understood the problem that it was using 7007 port as I was running training command multiple times as It was not working in the first time.
First time training process atuomatically stops without any error it is just showing Killed while loading data command output shown below
Sending ping to the viewer Bridge Server... Successfully connected. Sending ping to the viewer Bridge Server... Successfully connected. [NOTE] Not running eval iterations since only viewer is enabled. Use --vis wandb or --vis tensorboard to run with eval instead. Disabled tensorboard/wandb event writers [08:38:35] Auto image downscale factor of 1 nerfstudio_dataparser.py:314 Skipping 0 files in dataset split train. nerfstudio_dataparser.py:165 Skipping 0 files in dataset split val. nerfstudio_dataparser.py:165 Setting up training dataset... Caching all 288 images. Loading data batch ━━━━━━━━━━━━╺━━━━━━━━━━━━━━━━━━━━━━━━━━━ 31% 0:00:39Killed
Please help me resolving this issue

How you finally solve this problem?


Added support for configuring elasticsearch http.host property

We would like to be able to configure the elasticsearch http.host property. This allows us to better control the external visibility of the HTTP service exposed by elasticsearch. For example, by specifying "<IP_ADDRESS>" we can make it visible only by other processes running on the same host, e.g. a reverse proxy.

good one, thanks!


Can't import aspose on my Mac

I've already installed aspose-words.
$ pip install aspose-words

Requirement already satisfied: aspose-words in /Library/Frameworks/Python.framework/Versions/3.10/lib/python3.10/site-packages (24.5.0)

& python is installed on my machine:
$ brew info python
==><EMAIL_ADDRESS>stable 3.12.3 (bottled)

Installed
<EMAIL_ADDRESS>(3,296 files, 66.2MB) *

However, I'm unable to import aspose in any python script:
import aspose.words as aw
...

err:
$ /opt/homebrew/bin/python3 /Users/alaa/Developer/BASS/Docs/aspose-convertCHM2HTML.py
Traceback (most recent call last):
  File "/Users/alaa/Developer/BASS/Docs/aspose-convertCHM2HTML.py", line 1, in <module>
    import aspose.words as aw
ModuleNotFoundError: No module named 'aspose'

I have an ARM64 M1 MBA with macOS 14.5
& I'm using VSCode here.
I tried to import the package using different python versions, but the issue persisted.

It's an issue related to an externally managed environment (caused by VSCode and Brew here) & has nothing to do with the package itself.
Honestly, running Python could be a mess in macOS ...


Hygiene linting for extensions + new rule

Enable hygiene linting for ADS extensions
Fix hygiene errors in the extensions
Added new rule for not allowing underscores in keys for localize calls
Fix the fix rule for double quoting localize call args so that it actually fixes it correctly

@udeeshagautam Any concerns here? The new rule to not allow underscores is applicable to all the localize calls correct?

Yes any localize call key parameter. Looks good to me.


bam in mounted directory doesn't load with 'path'

Hi,
Thanks for this software!
I am trying to use igv-notebook to view alignments against bacterial genomes but I'm having trouble adding .bam tracks. These files all work as expected on the desktop app.
I am working within VSCode using the Jupyter extension. I am running notebook 6.4.11 and my python kernel is 3.9.7. I open the folder where the .ipynb is stored and can access the track files from that location relatively.
I am using the following code:
`import igv_notebook
igv_notebook.init()
fm043 = igv_notebook.Browser(
{
"reference": {
"id": "FM_043",
"name": "FM_043",
"fastaPath": "flye/assembly.fasta",
"indexPath": "flye/assembly.fasta.fai"
},
"locus": "contig_5:18,000-30,000"
}
)
fm043.load_track(
{
"name": "FM043_flye",
"path": "annotation/flye_igv.bed",
"format": "bed",
"type": "annotation",
"height":120,
"indexed": False,
"color": "red"
}
)
fm043.load_track(
{
"name": "FM043_flye",
"path": "align/flye_alignment.sorted.bam",
"indexPath": "align/flye_alignment.sorted.bam.bai",
"format": "bam",
"height": 240,
"type": "alignment",
"color": "blue"
}
)`
This works for the genome and annotation track! The bam file though fails to load even after 30 minutes. Sometimes a 408 error is output which suggests the server connection is too slow.

When I try to repeat the same process but instead downloading the relevant files locally, the .bam file still doesn't load. The bam file in this case is ~200Mb which I wouldn't expect to be problematic. In the /Examples section, there is reference to the url approach functioning better than the path approach. Is that all that is going on here?

I don't know anything about the VS code Jupyter extension.   However you can't use true local paths with Jupyter Lab.  In Jupyter Notebook you can, but it is not as effecient as using a relative URL and letting the Jupyter server itself handle the file loading.
If you want to use the local file ("path") option, have you tried using absolute paths to the files?   That would be the first thing I would try.   If using relative paths use "urls", although its sometimes tricky figuring out what the root path is for a Jupyter server.

Thanks for the quick reply!
I just tried it with 'url' instead of 'path' and I am actually getting the same behavior.
fm043.load_track(
{
"name": "bam",
"url": "/Volumes/.../flye_alignment.sorted.bam",
"indexURL": "/Volumes/.../flye_alignment.sorted.bam.bai",
"format": "bam",
"height": 240,
"type": "alignment",
"color": "blue"
}

When I try the tutorial example with hg19, it loads in a second so I don't think it's a problem with VSCode.

For sure you can't use a an absolute path with "url" and "indexURL".   I was suggesting you try the absolute paths with the "path" / "indexPath" options if that's what you want to do.
The url will be relative to your Jupyter file tree root, generally the startup directory of the Jupyter server.  If you are running Jupyter lab there is actually a user interface to find the URL.  Is your data in the Jupyter file tree (i.e. under the root directory of the Jupyter server)?


Fixed Feedback on 0.12 UI #7071

Describe your changes :

I worked on the https://github.com/open-metadata/OpenMetadata/issues/7071#issuecomment-1236061998

Type of change :

[x] Bug fix
[x] Improvement

Frontend Preview (Screenshots) :

Checklist:

[x] I have read the CONTRIBUTING document.
[ ] I have commented on my code, particularly in hard-to-understand areas.
[ ] I have added tests that prove my fix is effective or that my feature works.
[x] All new and existing tests passed.

Reviewers

@ShaileshParmar11 There's one warning for unused variable can you check?


Cloud Firestore: document whether calling .snapshots multiple times with the same query will have the reads be shared or not

It is unclear whether calling the method .snapshots() (and consequently .listen(...)), of theQuery and DocumentSnapshot classes, multiple times and with the same "logical" query (i.e. not necessarily the same Query or DocumentSnapshot object, but with identical querying parameters) will:

multiply the amount of reads, or
have the optimization where the reads are shared among all listeners.

This is very useful information for properly minimizing the read cost and client network bandwidth usage.

IMO, the amount of reads should go up, as this is how Streams work. You need to share the Stream yourself - e.g. by using ValueConnectableStream from RxDart.

cc @Salakar, looks familiar to a discussion we had previously
So if my understanding is correct, Firebase shares the reads across snapshots streams.
The first value pushed on a new stream is from the offline cache. And following values are pushed to all snapshots at once.

In a nutshell; if you create two different Query instances with the exact same parameters, the Query.snapshot() observers will read from the database twice (AFAIK). As mentioned, you should share the Stream instance in your code created by the Query.snapshot().


Is vue3.0 supported?

When will vue3.0 be available?

swiper js currently supports vue js 3 by default .
[https://swiperjs.com/vue](SwiperJs Vue 3)

Swiper Js support Vue.js 3 out of the box .
Swiper Vue.js Components

You don't need  this component for vue 3 . swiper already support vue 3 out of the box.
Swiper Vue.js Components

[[> This package is based on the famous swiper

As of late 2020 they support Vue.js out of the box! (Though you must at least use Vue.js version 3). So you should migrate to that if you can.
More info here: https://swiperjs.com/vue

](https://github.com/surmon-china/vue-awesome-swiper/issues/780#issuecomment-842295010)](https://github.com/surmon-china/vue-awesome-swiper/issues/780#issuecomment-842295010)

As of late 2020 they support Vue.js out of the box! (Though you must at least use Vue.js version 3). So you should migrate to that if you can.
More info here: https://swiperjs.com/vue

As of late 2020 they support Vue.js out of the box! (Though you must at least use Vue.js version 3).
More info here: https://swiperjs.com/vue

As of late 2020 they support Vue.js out of the box! (Though you must at least use Vue.js version 3).
More info here: https://swiperjs.com/vue

Thank you for your reply. I have checked that Swiper supports Vue3.0

v5.0.0


Fix typos

PR created using https://github.com/tbroadley/github-spellcheck-cli.

Coverage remained the same at 89.613% when pulling<PHONE_NUMBER>fb0aec0baa139f7fd01e5a4f5430d6 on tbroadley:fix-typos into 201074fe12181cc3ea162575c5082fbc42e0075d on fireworq:master.

Thanks! Your commit was merged via #11 .

No problem!


820 fix links

@thomas-neitmann i dont think i need to run devtools::documents() as i only updated R/data.R but let me know if i'm mistaken.
Thank you for your Pull Request! We have developed this task checklist from the Development Process Guide to help with the final steps of the process. Completing the below tasks helps to ensure our reviewers can maximize their time on your code as well as making sure the admiral codebase remains robust and consistent.
Please check off each taskbox as an acknowledgment that you completed the task or check off that it is not relevant to your Pull Request. This checklist is part of the Github Action workflows and the Pull Request will not be merged into the devel branch until you have checked off each task.

[x ] Code is formatted according to the tidyverse style guide
[x ] Updated relevant unit tests or have written new unit tests - See Unit Test Guide
[x ] If you removed/replaced any function and/or function parameters, did you fully follow the deprecation guidance?
[x ] Update to all relevant roxygen headers and examples
[x ] Run devtools::documents() so all .Rd files in the man folder and the NAMESPACE file in the project root are updated appropriately
[x ] Address any updates needed for vignettes and/or templates
[x ] Update NEWS.md
[x ] Build admiral site pkgdown::build_site() and check that all affected examples are displayed correctly and that all new functions occur on the "Reference" page.
[x ] Address or fix all lintr warnings and errors - lintr::lint_package()
[x ] Run R CMD check locally and address all errors and warnings - devtools::check()
[x ] Link the issue so that it closes after successful merging.
[x ] Address all merge conflicts and resolve appropriately
[x ] Pat yourself on the back for a job well done!  Much love to your accomplishment!

This PR includes site link fixes since moving to pharmaverse website and also in the README now included ref="main" as per discussion on developer Slack since we renamed master branch.

@thomas-neitmann as i was updating README i took the opportunity to add a pharmaverse badge as per our draft site page: https://pharmaverse.org/contribute/badges/. over time we're hoping this will be like a mark of honour for recommended packages

@rossfarrugia I do think you want to run devtools::document(). The data.R file generates these ,rd files

adae.Rd
adcm.Rd
adex.Rd
adsl.Rd
advs.Rd
ex_single.Rd

Each of those contains the only roche-gsk link.

thanks @mstackhouse think i've got that working now but @thomas-neitmann can double check next week. i had issues with devtools::documents() saying Error: 'documents' is not an exported object from 'namespace:devtools'. but devtools::check() worked and the .Rd files got updated fine. i think its that i need to update my RStudio project to link the new repo. shout if anyone's seen this before and has any other advice!

@rossfarrugia  no "s" :)
https://devtools.r-lib.org/reference/document.html

thanks @mstackhouse - thats what happens when i work late on a friday... ;)
will make those updates now @thomas-neitmann

@thomas-neitmann this one should be good to go now. had to resolve some merge conflicts for README and NEWS. this vignette really helped https://pharmaverse.github.io/admiral/articles/git_usage.html

because this one touches so many programs in so many places, every time we make a new merge to develop we're likely to get more merge conflicts until this one itself merges :)

@bms63 @thomas-neitmann can whoever has earliest availability please review and merge this one? i saw the "roche-gsk" comment come up again from Ben's review of #875 - we're always going to get these and merge conflicts until we merge this PR that affects so many programs.


BZ-1274394: add reproducer

Added reproducer for BZ-1274394.
Later I found that the failing test (LogicalInsertTest#testChangeElement) uses the class Person which does not define its equals and hashcode methods correctly. When I added the age attribute to the equals method, the tests passes.
This seems to be a change between Drools 6.2.x and 6.3.x, is this intentional?

As agreed this test case is invalid due to that wrong equals method. When the equals method is fixed the test succeeds as expected.


deadbeef/plugins: add the waveform seekbar plugin

Fixes https://github.com/NixOS/nixpkgs/issues/52612.
Things done

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

[ ] 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 nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Assured whether relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

I don't know how to do nix-build of a deadbeef plugin.

@GrahamcOfBorg eval

the checksum is required

But how do I know the valid checksum?

@suhr we use the trust-on-first-use model for most sources by specifying an incorrect checksum on the first use or using something like nix-prefetch-url.


export full name math helpers

I just did a simple rename in VS Code for Div, Mul, and Sub. Would you rather me add additional Divide, Multiply, Subtract exports on top of the existing?
closes #349

Hi and thanks for the PR. As discussed previously, please do not rename these, but merely re-export them under the longer names? (export const Multiply = Mul, etc.) - thanks!

@hmans my bad, updated!

The one build failure was something that was broken in main. I'm merging this, thanks!


PR Process Can Be Automatted

@unfoldingWord/devs  @birchamp
@kintsoogi and @theNerd247  created the following steps for submitting a PR for a JS library that is a dependency in another app. I'm making a note of some issues I've discovered to start a discussion here.
Problems

a PR is merged BEFORE it passes a QA process
only a single consuming app is tested. Changes made to a library may work for one app but induce new bugs in other apps.

Improvements

slowly start introducing a qa process for libraries that's independent of the consuming apps (this could be as simple as unit testing)
create separate tickets for consuming apps and give a separate qa process for those (this idea closely relates to #499)

Current QA Process

in library
let $betaVersion = $incrementPrereleaseVersion $newLatestVersion
let $newLatestVersion = $incrementVersion $latestPublishedVersion

publish $betaVersion to npm from local machine
update package.json to $newLatestVersion
Create a commit and push
Create PR to merge into main/master branch
let LibPrLink = PRlink
let PRLibDesc = <write pr description>
mark PR as draft
add reviewers to PR
set PR status to in review
let AppReviewLinks = LibPrLink

∀.app ∈ consumer-apps
1. checkout develop branch
2. create a new branch
3. update the <lib>@$latestPublishedVersion to <lib>@$betaVersion in app/package.json
4. create a commit and push
5. create a PR
let PRLink
let PRdescription = <write pr description> + <write testing steps>
AppReviewLinks += PRLink
These are done as part of the create new PR process

mark PR as draft
add reviewers to PR
set PR status to in review
PRdescription += <netlify deploy preview link>
LibPRDesc += PRdescription

∀pr ∈ lib. if reviewPass pr (merge pr) (fix pr)
in library
1. pull and checkout to <main> branch
2. publish $newLatestVersion to npm from local machine
∀pr ∈ consumer-apps
1. if reviewPass pr

update <lib>@$betaVersion to <lib>@$newLatestVersion (using yarn add)
commit and push
merge pr
else (fix pr)

∀pr ∈ lib <> consumer-apps >> setAsInQAStatus pr
Send discord message to QA (DM to Elsy, Daniel) with LibPrLink <> first AppReviewLinks <> IssueLink <> Git ShA (first consumer-app)
∀pr ∈ lib <> consumer-apps. if passQa pr (celebrate) (create new issues and start dev proces over)

@theNerd247 @kintsoogi could this be simplified by having a RCL QA pipeline and an integration pipeline in zenhub?  Then we could have QA test the component prior to integration.
Also in the consumer app steps above, step 7 needs to include moving the issue to the QA pipeline in Zenhub. The QA team checks that to see what's ready for testing.


Cannot call vcpkg from an outside directory to install a package (since commit e79f0dc5)

Describe the bug
Some change in commit e79f0dc5328f28b2b3942e2cd0e9b0c1accca4a1 introduced a bug that makes calling vcpkg from outside directories fail.
Assuming vcpkg is cloned and bootstrapped in directory ~/projects/vcpkg, the following behavior happens:
/ $ ~/projects/vcpkg/vcpkg install eigen3
error checking existence of file /home/user/projects/vcpkg/vcpkg/.vcpkg-root: Not a directory

Seems it looks for .vcpkg-root in the wrong location (off by one).
The problem also appears when the vcpkg directory is put on the PATH.
What should happen: installs eigen3 package (or any other package that's specified)
This works with commits made on master before e79f0dc5328f28b2b3942e2cd0e9b0c1accca4a1.
Environment

OS: Linux (Ubuntu 16.04, but issue should also be present on other versions/distributions).
Compiler: GCC 9.1

To Reproduce
Steps to reproduce the behavior:

Clone and bootstrap vcpkg
Go to a directory above and call the vcpkg install ... executable with an absolute path.
The error message "error checking existence of file /home/user/projects/vcpkg/vcpkg/.vcpkg-root: Not a directory" is emitted.

Expected behavior
Package installation should succeed.

Currently checking this out

Oh, that was dumb. I fixed the code on Windows, but didn't finish it on Unix.

Fixed by #7619


Jason.DecodeError UTF8

Hi, I'm getting this error when doing Req.get!/1:
Request: GET /
** (exit) an exception was raised:
    ** (Jason.DecodeError) unexpected byte at position 542: 0xD1

Is there a way to avoid it?, I'm fetching a JSON that contains an "Ñ" as a value.

Thanks for the report. You can set decode_body: false and manually decode the body. I'm gonna close this in favour of similar https://github.com/wojtekmach/req/issues/295.


Conditional deployment does not work with Bicep role Assignment

Bicep version
Bicep CLI version 0.24.24
Describe the bug
I am trying to conditionally deploy a roleAssignments associated with a policyAssignments. The condition for both roleAssignments and policyAssignments evaluates to false. The policyAssignments does not deploy as expected. However, roleAssignments is still trying to deploy. The deployment fails because the role assignment is trying to use the attributes of policy assignment which does not exist.
Error: "details":[{"code":"PolicyAssignmentNotFound","message":"The policy assignment 'XXX Assignment' is not found."}]}]}]}}
Expected behavior
When the condition for roleAssignments is false, the resource must not be deployed.
To Reproduce
Steps to reproduce the behavior:

Create a policyAssignments and roleAssignments in Bicep. Use "SystemAssigned" identity type for policy assignment.
The role assignment is assigning a role (can be any role) to the policy Assignment identity.
Use a condition for both resources and make sure the condition evaluates to false.
Try deployments

Additional context
Here are the API versions I use in the code:
Microsoft.Authorization/roleAssignments@2022-04-01
Microsoft.Authorization/policyAssignments@2023-04-01

This is a Microsoft.Authorization RP issue, please open a support ticket with that RP to streamline the issue


Fix build

For some reason the test suite needs to have cwd set to root dir. This needs to be explicit because of some change some other PR made.

@macobo Is this still relevant? What was broken about the build?

@kibertoad Not sure; I think I was trying to build from source years ago and ran into this issue.
If everything is stable, feel free to close this PR.

Thank you for clarification!


If "element not visible" error, inform user if there are similar elements on the page

Look for issue #201 for more examples on this topic

It is hard to implement. Looks like complicated and not so useful feature.


ModSecurity false positive file editor addon

hello thank you for your work !
i have a false posisitive detection with the file editor addon :
2021/04/12 12:02:49 [error] 254#254: *15730 [client ****] ModSecurity: Access denied with code 403 (phase 4). Matched "Operator Ge' with parameter 4' against variable TX:OUTBOUND_ANOMALY_SCORE' (Value: 5' ) [file "/usr/local/owasp-modsecurity-crs/rules/RESPONSE-959-BLOCKING-EVALUATION.conf"] [line "128"] [id "959100"] [rev ""] [msg "Outbound Anomaly Score Exceeded (Total Score: 5)"] [data ""] [severity "0"] [ver "OWASP_CRS/3.3.1"] [maturity "0"] [accuracy "0"] [tag "anomaly-evaluation"] [hostname "****"] [uri "/api/hassio_ingress/oWD9BImRi4BTOXJ1RhmwmIK_uDnAHbXok9mT7iJMiU4/"] [unique_id "1618221768"] [ref ""] while sending to client, client: ****, server: ***, request: "GET /api/hassio_ingress/oWD9BImRi4BTOXJ1RhmwmIK_uDnAHbXok9mT7iJMiU4/ HTTP/2.0", upstream: "http://****/api/hassio_ingress/oWD9BImRi4BTOXJ1RhmwmIK_uDnAHbXok9mT7iJMiU4/", host: "****", referrer: "https://****/core_configurator/dashboard" 2021/04/12 12:02:49 [alert] 254#254: *15730 header already sent while sending to client, client: ****, server: ****, request: "GET /api/hassio_ingress/oWD9BImRi4BTOXJ1RhmwmIK_uDnAHbXok9mT7iJMiU4/ HTTP/2.0", upstream: "http://***/api/hassio_ingress/oWD9BImRi4BTOXJ1RhmwmIK_uDnAHbXok9mT7iJMiU4/", host: "****", referrer: "https://***/core_configurator/dashboard"
thank you
ragards

Thanks for reporting a false positive detection. Is there any specific action while using File Editor addon that triggers the false positive (i.e. opening a file, saving a file, etc.)?
I'm the meantime, I hope you have seen the local workarounds for bypassing modsecurity so you can successfully use File Editor addon: https://github.com/jhampson-dbre/nginx-proxy-waf/blob/main/nginx_proxy/DOCS.md#known-issues-and-limitations

thank you for your reply,
yes i see it ! and i'm in DetectionOnly mode for wait
for File editor, it's when i open the addon, therefore i cant use this, it doesn't open

Thanks for the info. I am able to reproduce this on my system so I should be able to create and test an exclusion rule.
Can you confirm that in DetectionOnly mode you are able to use File Editor? Nothing should be blocked by Nginx Proxy WAF in DetectionOnly mode, so if you are still having problems make sure the Nginx Proxy WAF addon has been restarted after changing the configuration to DetectionOnly.

hello,
yes, in DetectionOnly, mode it work, i can open File Editor Addon but in "On" mode it don't work

This should be resolved in v3.0.16 just released. Please let me know if you find otherwise. Thanks again for reporting this false positive!

i just update your nginx addon and it work !! thank you


minor: show failure of sonar build

This PR should show failing sonar build.

Closed, we can refer to this later if need be


Transition between pages issue

Hello everyone,
Environment
nuxt : 2.15.8
nuxt-gsap-module:  1.7.2
Reproduction
Reproduction on the link below :
https://codesandbox.io/s/animation-between-page-test-go4k6h?file=/layouts/default.vue
Describe the bug
I used GSAP for transitions between my pages.
The problem is when you leave about, home is not loaded and we see body background. Conversely, when you are on home and you go on about, home is still alive and "about" move above home
Another thing is between about and news, the two pages "push" each other.
Is it possible to "keep alive" home page in both cases ? and to prevent "pushing" ?
Additional context
I tried to reproduce animations in this website : https://www.mirrormirror.fr/
Not sure if it's a issue or not, can someone enlighten me ?

We are approaching the Nuxt 2 EOL date (June 30, 2024) - see this article for more information. This is advance warning that I'm going to close this issue then, as it's currently marked as a Nuxt 2 related bug.
If it's a critical or security issue, please do comment and let me know, in case it is possible to address it before the EOL date.
If it's a an issue you think is relevant for Nuxt 3, please feel free to open a fresh issue (or just comment here so I can update labels, etc.). 🙏
Thank you for your understanding and for your contribution to Nuxt. ❤

It's the day, at last. Nuxt 2 is now marked end-of-life.
My apologies we never got round to resolving this bug in the 2.x branch, but we do have to draw a line somewhere. Again, if you think this is still relevant for Nuxt going forward, please feel free to open a fresh issue (or just comment here so I can update labels, etc.). 🙏


Don't throw oob exception when setting numeric indexes on TAs

Relevant spec section: https://tc39.es/ecma262/multipage/ordinary-and-exotic-objects-behaviours.html#sec-typedarraysetelement
It should only throw if the backing buffer is detached, which test262 tests for.
Fixes: https://github.com/quickjs-ng/quickjs/issues/645

SGTM, I'll wait for your PR, apply it on top and add a couple of tests for it!

You already fixed this one Ben :-) Closing!

@bnoordhuis Updated, can you PTAL once more?


walk doesn't work on Fused Exprs

We don't introspect lists of expressions

Can you elaborate a bit on what you want to change? I think it is generally a good thing that we don't make any promises to dive in to list-based operands to find Expr objects. So, I think I'd rather add special handling for Fused expressions where necessary, and avoid promising this behavior everywhere.


How to download the pre-trained models from OSF

Thanks for the code shared!
I have a question about the 6th step in the "Installation" block:
I'm a new to the linux, there are difficults for me to download files in linux language. May I download the same model straightly in the OSF?

Oh thanks for your help. 😳
After the issue published, I notice that the DeepLabCut version you used needs a relatively lower version tensorflow pack(1.15), the cuda version corresponds to our GPU(A2000) only supports the tensorflow pack higher versions higher than 1.17.
And unfortunately my python skills are not good enough to support me to modify the contents of the functions in the package.
So it's a pity that we can't use the package any more.😟😟😟

Hi @LaylaQAQ, thanks for letting me know - I am sorry to hear about the TensorFlow issue. I want people including you to be able to use this package with newer tensorflow and deeplabcut versions and will work on updating MesoNet to accommodate these!

Hi @bf777 , thanks for updating the supporting versions, now there is a stable problem throughout this and the previous version: after I press the "predict brain regions using landmarks" button, it always reports this error: ModuleNotFoundError: No module named 'keras.legacy_tf_layers'. I checked up the GPT and it say: the new version of keras don't support 'legacy_tf_layers' method any more. So may there is some way to solve it?

Hi @LaylaQAQ , I have set the tensorflow and keras versions to 2.12.0 to attempt to address this issue in a newly pushed version #44. The "predict brain regions using landmarks" button offers a custom workflow that is less reliable (it is based on the checkboxes you select above it). If you can test one of the default pipelines on the right side of the GUI, especially 1 or 2, they should give you comparable results for the calcium image alignment.

Hi@bf777 , I set up the environment again, and now troubles with tensorflow and keras are solved perfectly, thanks very much. There is another problem: I press the '1-atlas to brain' button, it error :'FileNotFoundError: [WinError 3] The system cannot find the path specified: 'C:\Users\user\Desktop\mesonet\atlas-DongshengXiao-2020-08-03\training-datasets\iteration-2\UnaugmentedDataSet_atlasAug3', below in the pic is my config in the GUI. May I need to change something in my env to reset the path? Or a manual DLC step is necessary?

Hi @LaylaQAQ , which folder were you in when you ran the script get_default_models.sh? You may need to run this script, which downloads the files listed in the error message, from within the directory in which it is located (as opposed to running the script from another directory). This will ensure that all external files are downloaded into the dlc subfolder of the mesonet directory (the paths in the directory are relative paths, which may have caused the issue). Alternatively, if you already ran this script, you can look at the script here: https://github.com/bf777/MesoNet/blob/master/mesonet/get_default_models.sh and move each of the files listed into the corresponding subfolders in your mesonet folder.

Hi@bf777, here is another question that, in 'Read me' -> 'installation' -> '6. Run bash get_default_models.sh to download...', in step 5, I'm in the '/Mesonet' folder, but when I input 'bash get_default...' it errors: '/usr/bin/bash: get_default_models.sh: No such file or directory', so I 'cd mesonet' again. And this time in /Mesonet/mesonet path, it downloads a 'models' folder. But it seems not in the path you noted in readme. Is that right? Which 'mesonet' should the 'models' be in? Very thanks.

@LaylaQAQ Yes, you should do it in the mesonet directory within the main MesoNet git repository folder, where the bash script is (check to see in which folder the script is contained). I will make sure this is clearer in the readme.

I am closing this issue due to inactivity.


Add optional parentheses IR

Summary
This PR adds a custom IR for content that may need to be parenthesized to make it fit.
Test Plan
cargo test

Current dependencies on/for this PR:

main

PR #7475   👈

This comment was auto-generated by Graphite.


Typescript types?

Why this error in a Svelte Typescript project?
Could not find a declaration file for module 'svelte-flatpickr'. 'C:/project/node_modules/svelte-flatpickr/dist/index.cjs.js' implicitly has an 'any' type.
  Try `npm i --save-dev @types/svelte-flatpickr` if it exists or add a new declaration (.d.ts) file containing `declare module 'svelte-flatpickr';` (ts)

That's an extremely common warning that happens with any dependency that doesn't have types. Please feel free to contribute types to the @types project as indicated in the warning if you'd like to resolve it. Or just ignore the warning like I always do for untyped deps.

You don't want to add them to this project?

Sure, feel free to submit a .d.ts if you'd like, that's better I suppose.

I started this here: https://github.com/jacobmischka/svelte-flatpickr/tree/add-types
Would appreciate anyone more familiar with external type definitions and/or svelte typings to take a look!

Fixed in 200fcc7ec2b6c573de4796c9a155516f640f9fe7, published in 3.2.0. Related issues and suggestions expected and welcome!


Creating Database Container throws DockerApiException

Describe the bug
When creating a Database Container (e.g. NpgSql or MsSql) throws a DockerApiException with the following Response:
code=BadRequest, response={"message":"invalid filter 'reference'"}
Container Registration:
private readonly PostgreSqlTestcontainer _database = new TestcontainersBuilder<PostgreSqlTestcontainer>()
        .WithDatabase(new PostgreSqlTestcontainerConfiguration
        {
            Database = "test",
            Username = "user",
            Password = "password",
        })
        .Build();

Stacktrace
    Index Failed: Docker.DotNet.DockerApiException: Docker API responded with status code=BadRequest, response={"message":"invalid filter 'reference'"}
Docker.DotNet.DockerApiException
Docker API responded with status code=BadRequest, response={"message":"invalid filter 'reference'"}

   at Docker.DotNet.DockerClient.HandleIfErrorResponseAsync(HttpStatusCode statusCode, HttpResponseMessage response, IEnumerable`1 handlers)
   at Docker.DotNet.DockerClient.MakeRequestAsync(IEnumerable`1 errorHandlers, HttpMethod method, String path, IQueryString queryString, IRequestContent body, IDictionary`2 headers, TimeSpan timeout, CancellationToken token)
   at Docker.DotNet.ImageOperations.ListImagesAsync(ImagesListParameters parameters, CancellationToken cancellationToken)
   at DotNet.Testcontainers.Clients.DockerImageOperations.ByPropertyAsync(String property, String value, CancellationToken ct)
   at DotNet.Testcontainers.Clients.DockerImageOperations.ExistsWithNameAsync(String name, CancellationToken ct)
   at DotNet.Testcontainers.Clients.TestcontainersClient.RunAsync(ITestcontainersConfiguration configuration, CancellationToken ct)
   at DotNet.Testcontainers.Containers.TestcontainersContainer.Create(CancellationToken ct)
   at DotNet.Testcontainers.Containers.TestcontainersContainer.StartAsync(CancellationToken ct)
   at DotNet.Testcontainers.Containers.ResourceReaper.GetAndStartNewAsync(Guid sessionId, IDockerEndpointAuthenticationConfiguration dockerEndpointAuthConfig, String ryukImage, TimeSpan initTimeout, CancellationToken ct)
   at DotNet.Testcontainers.Containers.ResourceReaper.GetAndStartNewAsync(Guid sessionId, IDockerEndpointAuthenticationConfiguration dockerEndpointAuthConfig, String ryukImage, TimeSpan initTimeout, CancellationToken ct)
   at DotNet.Testcontainers.Containers.ResourceReaper.GetAndStartDefaultAsync(IDockerEndpointAuthenticationConfiguration dockerEndpointAuthConfig, CancellationToken ct)
   at DotNet.Testcontainers.Clients.TestcontainersClient.RunAsync(ITestcontainersConfiguration configuration, CancellationToken ct)
   at DotNet.Testcontainers.Containers.TestcontainersContainer.Create(CancellationToken ct)
   at DotNet.Testcontainers.Containers.TestcontainersContainer.StartAsync(CancellationToken ct)
   at Tests.Backoffice.Xunit.IAsyncLifetime.InitializeAsync()
   at Xunit.Sdk.ExceptionAggregator.RunAsync(Func`1 code) in /_/src/xunit.core/Sdk/ExceptionAggregator.cs:line 90

If it helps, the DockerImageOperation part tries to assemble a reference Filter with the image tag ghcr.io/psanetra/ryuk:2021.12.20
Additional Information
Testcontainers 2.1.0
Docker Desktop 4.12.0 (85629)
Windows 10 19044.1889
Running on WSL2 Engine

I cannot reproduce it with Docker Desktop for Mac. I'll have access to a Windows machine sometimes next week. Then I can double-check if version 4.12.0 breaks something.
Can you disable the resource reaper and test if it is able to pull the database images (TestcontainersSettings.ResourceReaperEnabled = false)?

Thank you for your Response.
I put that in the ModuleInitializer annotated Block of xUnit, but now it fails with postgres:11.14

I tried to debug further to see what the outcoming request URI is, which the Docker Engine API receives.
http://localhost:2375/images/json?all=1&filters={%22reference%22:{%22postgres:11.14%22:true}}
This looks pretty fine to me and also trying to use other API Versions like v1.12 or v1.40 doesnt seem to change the error message.
This could be a parsing error in Docker Desktop, since the specification for the /images/json endpoint hasn't changed since.

I have opened an issue over at moby/moby/issues/44161
I suggest we'll wait for them to respond, since this seems to be an issue with docker engine.

So it seems, that the containerd integration on Windows is the issue. I think i can close this as resolved.

Thanks for keeping me up to date.


We might be able to "not" also fork fastai-docs

As pointed in #512 by @KushajveerSingh .
We should use absolute path for fastai-docs in .gitmodules, so we need to only fork fastai2.
[submodule "docs"]
	path = docs
	url = https://github.com/fastai/fastai-docs
	branch = master

This is also confirmed by the answers to my question on stackoverflow.
How to set .gitmodules so that contributors don't have to fork also submodule?
It should be worth trying it.

That then forces https mode, but some folks (like me!) want to use ssh.


help update Ink to react 18, and use suspense for async ops in <Dev/>

closing this in favor of https://github.com/cloudflare/workers-sdk/issues/2714


Design discussion: pixi global manifest and CLI changes

Typical usage
This is a starting point for the pixi global manifest and how the CLI has to change.
pixi global install python=3.11.*

Will create a new manifest at some global location, create a new environment with the implied name python and expose all binaries of the python package.
[[envs]]
name = "python"

[envs.dependencies.python]
spec = "3.11.*"
expose_binaries = "auto" # This will expose python, python3 and python3.11

Injecting dependencies
pixi global install pip --name=python

Will inject pip into the existing environment, named python.
Binaries will not be exposed by default when installing into an existing environment.
[[envs]]
name = "python"

[envs.dependencies.python]
spec = "3.11.*"
expose_binaries = "auto" # This will expose python, python3 and python3.11

[envs.dependencies.pip]
spec = "*"

Specify which binaries to expose
pixi global install python=3.10.* --name=python_3_10 --expose-binary="python3.10=python"

Will create another environment of name python_3_10, add dependency python and expose python with name python3.10.
[[envs]]
name = "python"

[envs.dependencies.python]
spec = "3.11.*"
expose_binaries = "auto" # This will expose python, python3 and python3.11

[envs.dependencies.pip]
spec = "*"

[[envs]]
name = "python_3_10"

[envs.dependencies.python]
spec = "3.10.*"
expose_binaries = {"python3.10"="python"}

Discussion points:
How to change exposed binaries from the CLI?
Proposal: Let pixi global set instead of add exposed_binaries
[[envs]]
name = "python_3_10"

[envs.dependencies.python]
spec = "3.10.*"
expose_binaries = {"python3.10"="python"}

Running this
pixi global install python=3.10.* --name=python_3_10 --expose-binary="python_old=python"

would lead to
[[envs]]
name = "python_3_10"

[envs.dependencies.python]
spec = "3.10.*"
expose_binaries = {"python_old"="python"}

Multiple exposed binaries could be set like this:
pixi global install python=3.10.* --name=python_3_10 \
    --expose-binary="python3.10=python" --expose-binary="python_old=python"

How to deal with existing packages?
Proposal: If there's no manifest present, every pixi global commands offers to create a new manifest filled with the information of the existing environments
Which API should we add to update packages?
❔

Not needed anymore, now that https://github.com/prefix-dev/pixi/pull/1757 is merged


Clean up the druid sync api.

Cleans up the druid sync api to pass the data as the json blob rather than 3 separate attributes.
Reviewers:

@mistercrunch
@ascott
@vera-liu

LGTM


Improve packaging: use setup.cfg

(Also please remove the  .DS_Store file).

OK, I pushed a few commits to illustrate what this could look like if we moved all config from setup.py to setup.cfg and also included requirements.txt there.
There is some improvement with respect to the current approach, but I also can't say that there is currently a problem that really needs fixing. There is a small risk of install issues if I messed up somewhere in setup.cfg.
This is ready for review now. Looking for other opinions.

I'm going to close this as it's become stale and no one is available to sync this.


Now EventCallback only allows parameterless callback. Update is needed?

[EDIT by guardrex to manually add the metadata]
Is it a typo?

EventCallback is weakly typed and allows any argument type.

It should be

EventCallback is weakly typed and only allows no argument type.

or more precisely

EventCallback is weakly typed and only allows callbacks (or event handling methods) with no paramater.

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

ID: aa724f22-3a49-c7f6-1500-ac2d8dde5894
Version Independent ID: 5e8ef502-ccf9-659d-f25a-0f8f3c75f6e0
Content: ASP.NET Core Blazor event handling
Content Source: aspnetcore/blazor/components/event-handling.md
Product: aspnet-core
Technology: aspnetcore-blazor
GitHub Login: @guardrex
Microsoft Alias: riande

@pstricks-fans ... Please open issues using the This page feedback button and form at the bottom of a topic. I'll add the metadata to your opening comment. When you use the button and form, we don't need to add it manually to issues.

Artak, the source is ...
https://github.com/dotnet/aspnetcore/issues/6351#issue-395755868
It's in the Invoking EventCallback manually section of the remarks, which reads ...

Invoking EventCallback manually
You can invoke an EventCallback like: await callback.InvokeAsync(arg).
Yes, EventCallback and EventCallback<> are async. We think that this will work well, because it's now much easier to use async thanks to the other improvements we made in this area. EventCallback<> is strongly typed and requires a specific argument type to invoke. EventCallback is weakly-typed and will allow any argument type.

Sorry. I clicked the "this page" button but  I accidentally removed the template provided by the github. Thank you.

@guardrex Did you intend to ping Artak on this?
FWIW, I think Ryan's comments are accurate. EventCallback has an InvokeAsync(object arg) method but EventCallback<TValue> has an InvokeAsync(TValue arg) method. The former is weakly-typed because you can pass anything in place of the object, but the latter is strongly-typed because the value passed in must be assignable to TValue.

Artak sees these when he has time to triage issues. By not pinging him, he's not disturbed with an email; but by addressing him by name, readers know (and he knows) that I'm addressing my remarks to him.
Thanks for calling out the detail on it. I'll add/modify the text to clarify it.


python: add @comment.inner

# test
--- cic ---
# |

and
#test
--- cic ---
#|

where | denotes the cursor.
I added the first as 'special case', because I think that is (and should be) how most programmers use comments: always having a space after the comment marker. Adding this as a special case skips having to add the space every time after something like cic.

Looks really good, thank you!


Target_org_apache_cxf_common_spi_NamespaceClassGenerator loads classes that are not registered for reflection

Target_org_apache_cxf_common_spi_NamespaceClassGenerator tries to load several classes such as
org.apache.cxf.jaxb.EclipseNamespaceMapper
org.apache.cxf.jaxb.NamespaceMapper
org.apache.cxf.jaxb.NamespaceMapperRI
org.apache.cxf.common.jaxb.NamespaceMapper
out of which only org.apache.cxf.jaxb.NamespaceMapper is registered for reflection and thus only that execution path will ever be executed.
I wonder which of the following is the right solution:
A. Register all named classes
B. Register all subclasses of com.sun.xml.bind.marshaller.NamespacePrefixMapper - we need to check whether this catches all the named classes
C. Change Target_org_apache_cxf_common_spi_NamespaceClassGenerator to always return org.apache.cxf.jaxb.NamespaceMapper
The bottom line is that I have never seen the Could not create a NamespaceMapper compatible with Marshaller class ... message even though I removed the org.apache.cxf.jaxb.NamespaceMapper registration. I wonder what kind of test would hit the substituted method?

Hi @ppalaga In cases like this, I have generally followed what seems like the 'safest' approach to minimize any potential issues.  I couldn't find any unit tests in the CXF codebase that deal with org.apache.cxf.jaxb.NamespaceMapper ... my suggestion would be to either do option A, or even better, remove the substitution all together given you couldn't even hit the method.
We likely have quite a bit of leftover experimental code from the initial days of this extension, that could possibly be cleaned up ... this seems to fall into that category.

Oh, indeed, removing Target_org_apache_cxf_common_spi_NamespaceClassGenerator would be the most elegant solution :) Let's try it!


Pass redacted headers to request and response logger

I want to add a unit test, but we haven't yet developed any means to capture the log output.

Oops, this was supposed to be against 0.17.


Added to latest master version. Will be in next release

https://github.com/AlexxIT/XiaomiGateway3/releases/tag/v2.1.1


Events are not logged at production

I updated the @amplitude/react-native from 2.6.0 to 2.11.0. All goes well at debug mode and release mode on simulator. But there are no logs after the app has been released.
@amplitude/react-native: 2.11.0
react-native: 0.67.3

Hi @mtnt , Sorry for the late reply.
Several questions related to this issue.

does that issue happens on both Android and iOS?
which platform did you release your app?

A side note: We are working on enabling the setLogCallback in Amplitude-ReactNative. That should be helpful for debugging the application. I will update here if that feature got released.


[Question] how does Polaris Scales?

Hi guys,
I was wondering if there were any tests (with numbers) about how well Polaris can scale?
Background of my question
If I want to introduce an open source project like Polaris into a project at my company, I need to have some kind of proof that it the module will support scaling.
do you guys have anything like that by any chance?

Nothing on paper. Keep in mind Polaris can only handle one request at a time. Due to its use of a single runspace.
It was an early design decision to replicate expressjs's design pattern but it seems like more folks want something that can handle multiple requests at the same time.

See https://github.com/PowerShell/Polaris/issues/127

I would treat it the same as if you were trying to scale node or php-fpm and use a reverse proxy like nginx or IIS to handle load balancing between several instances or something like Azure App Service that would handle scaling Polaris for you.


Defined REST APIs to update PromotionResult for existing deployments

References #19

@tommysullivan this is relevant to our discussion today

This looks great to me, I'm going to mergey merge


looking for WallPanel Legacy api

Hi, I'm just looking for a download link for WallPanel legacy ( support Webview on older device )

Just look at the list of repositories on my Github profile.


Iam roles with proxy

NOTE: Please read the CONTRIBUTING.md guidelines before submitting your patch,
and ensure you followed them all:
https://github.com/Kong/kong/blob/master/CONTRIBUTING.md#contributing
Summary
This PR is an extension to the PR #2777 raised by @Mgutjahr. Extended the plugin to enabled proxy option for enterprises where requests are routed via enterprise proxy.
Full changelog
Added plugins config flag proxy_url and proxy_scheme to kong.plugins.aws-lambda.schema
Added self_check function to check whether proxy_url is set, if yes, check whether proxy_scheme is set.
Updated kong.plugins.v4.lua to use openssl library instead of crpto
Updated kong.plugins.iam-role-credentials.lua to make <IP_ADDRESS> to go via proxy only.
Added check for proxy url. if set, http connect uses connect_proxy else uses normal connect. Also changed the upstream_body attributes to match/align that of aws lambda implementations i.e upstream_body.httpMethod, upstream_body.headers, upstream_body.path, upstream_body.queryStringParameters and upstream_body.body

Hi @rkbasa,
Thank you for your contribution! We will need some time to review it as we are sorting out a few other priorities right now, but I just wanted to drop you a note that it isn't being ignored.
Cheers

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

closing this, has been added to the aws-lambda plugin after it was spun out into its own git repo.


How do you use this library when your model uses long primary keys

My models use long primary keys. I'm unable to configure multitenancy in my EF entities due to this constraint where TKey : IEquatable<TKey>.  Adding a migration results in the error: "The binary operator NotEqual is not defined for the types 'System.Int64' and 'System.Object'."

Great. I'll test it out within the week and revert. Appreciate the quick response.

Hi, I just realized I never got back. I tested it and it's working great. Thanks a lot.


0500 - Keyboard Row

Description of the Problem
Given a List of words, return the words that can be typed using letters of alphabet on only one row's of American keyboard like the image below.

Example
Input: ["Hello", "Alaska", "Dad", "Peace"]
Output: ["Alaska", "Dad"]

Note

You may use one character in the keyboard more than once.
You may assume the input string will only contain letters of alphabet.

Code
class Solution:
    def findWords(self, words: List[str]) -> List[str]:

Link To The LeetCode Problem
LeetCode

Hey,
I would like to do this.
Can you assign this to me?


Some Minor Suggestions

Adding readonly access modifier to private fields set in constructors.
For projects targeting .NET 6 and C# 10 language version, suggesting the shift to file-scoped namespaces.

TIL how to filter out whitespace changes in github :D

TIL how to filter out whitespace changes in github :D

😆 ... yeah, I usually try to make separate commits for that kind of thing. Seemed small enough (says the guy who didn't have to review ha).

I forced that through and turned off the code signing, that is not what I meant to have as a restriction

I thought this is what code signing added:

I'll have to figure that out later


Enforce EdgeState invariant

An EdgeState can only be added as an incoming edge if it is properly
tracked as an outgoing edge in the node it originates from.
Before these changes there was a code path were an edge could be added
as incoming while it was no longer outgoing.
Draft PR to get the changes through CI while crafting an integration test.

@bot-gradle test RFM

OK, I've already triggered ReadyForMerge build for you.


Plugins block apply plugins automatically despite "apply false" for precompiled script plugins

In gradle 6.6.1, and probably before, plugins {} block apply plugins automatically, despite apply false statements:
//buildSrc/build.gradle
plugins {
  id 'groovy-gradle-plugin'
}

//buildSrc/src/main/groovy/pluginA.gradle
plugins {
 id 'pluginB' apply false
}

//buildSrc/src/main/groovy/pluginB.gradle
throw new GradleException('plugin B was applied')

//build.gradle
plugins {
 id 'pluginA'
}

The command
./gradlew :help

will fail with the exception plugin B was applied.
The problem is definitely in the transitivity of the plugins declarations: once a plugin is applied, all the plugins defined transitively in the plugin chains' plugins {} block will be applied, regardless of the apply statement.

apply false in precompiled scripts plugins {} block doesn't make sense and should either not compile or throw.

Yes. This should throw an error.


Add contact info to README.md

Update the README.md file to add a Questions? section with the following info:
Twitter: @CodeDotGov
Email<EMAIL_ADDRESS>LinkedIn: code-gov
Join our #opensource-public Slack channel: https://chat.18f.gov/

I can take this one.


Dispatch: Simplify DispatchData.append(_: UnsafeBufferPointer)

The deprecation warning guides us towards UnsafeRawBufferPointer, so sure, let's use that.
No intended functionality change.

@swift-ci Please test

cc @madcoder @mwwa


Code Splitting incorrectly merging chunks

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

What is the current behavior?
I have sections on a page. Only 1 will be showing at a time. I want to Code split so the chunk loads when the panel is active (react component mounts)
Take the following example:
<panel id="foo">
  <lazy>{() => import("foo.js")}</lazy>
</panel>
<panel id="bar">
  <lazy>{() => import("bar.js")}</lazy>
</panel>

Lazy is a wrapper that only executes the callback when the component mounts.
I expect this to generate 2 chunks, however webpack is generating foo and bar into the same chunk file.
I assume that webpack is incorrectly assuming that because it is in the same file, it must be loaded at the same time.
This assumption is wrong. Webpack should treat any code split separately when they are in callbacks.
I tried to use require.ensure too and give different names:
<panel id="foo">
  <lazy>{(load) => require.ensure([], (require) => {
                        load(require("foo.js"));
                    }, "foo")}</lazy>
</panel>
<panel id="bar">
  <lazy>{(load) => require.ensure([], (require) => {
                        load(require("bar.js"));
                    }, "bar")}</lazy>
</panel>

This was a less clean approach, but let me name my chunks, so that was nice, but it still has the same problem, they go into foo
Copy of the lazy load logic can be found here in the require.ensure form: https://gist.github.com/aikar/cc61d652c0d24aad4fc1fa5987abc6d2
What is the expected behavior?
I expect that the code is split into multiple parts.
If this is a feature request, what is motivation or use case for changing the behavior?
Improve code splitting
Please mention other relevant information such as the browser version, Node.js version, webpack version and Operating System.
Webpack 2.3.1.

Nevermind. It looks like the dependencies of these views were simply shared too much that was the ultimate cause as using it in other places I was able to get multiple splits.

@aikar can u show how u create dynamic import for chunks?
My question on SO


[IB] Re-added Command Permissions

This is a pull to add the permission system as it was.
Without having customizable permission nodes, everything feels messy; this is especially true in Eco where there are 100+ commands to account for.
With this system, I am able to organize the permissions into groups based on the accessibility, use, ect, (See Discord Chat).
Even with these additions, this is optional. If a plugin opts to set the permissions field to null, then it will use the system currently in place to create the permission based on the context.

Command permissions were removed intentionally. The main reason for this is having consistent commands permissions. The permission should be just Eco.(command). There is no need for grouping.


Inefficient $response->file() should be removed

The $response->file() function should be removed or at least not be mentioned so heavily in the samples in the Readme. It is very bad practice to send files by just outputting them with php, php was not made for this and it should be preferred to redirect to the file using Location header.
It may work fine for small files, but it will make a lot of trouble for large files, especially with slow connections, since they will hit the max execution time of a php script. (This can be changed, but defaults to a very short time, about 30 seconds)
See "Why does readfile() exhaust PHP memory?" and "PHP readfile() and large downloads" and "Readfile() and large files"

@Rican7 Is this going to be removed or reworked? I was trying to use $request->file() with an external url and it doesn't work at all, if you're keeping it I might go ahead and change it and submit a pull request.


use the right protoc executable regardless of protobuf provider

cherry pick grpc/grpc#12411 to make it compile without protoc.

LGTM


[SPARK-19956][Core]Optimize a location order of blocks with topology information

What changes were proposed in this pull request?
When call the method getLocations of BlockManager, we only compare the data block host. Random selection for non-local data blocks, this may cause the selected data block to be in a different rack. So in this patch to increase the sort of the rack.
How was this patch tested?
New test case.
Please review http://spark.apache.org/contributing.html before opening a pull request.

Can one of the admins verify this patch?

The fix LGTM, I think it is nice to have such topology priority. CC @mridulm .

Hi, @jerryshao @mridulm Thanks for your review, I have updated the code.

Ok, thanks a lot.

Hi, @cloud-fan @zsxwing Can you take a look? Thanks a lot.

retest this please

LGTM

Test build #76453 has started for PR 17300 at commit 56f5231.

Test build #76453 has finished for PR 17300 at commit 56f5231.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/76453/
Test FAILed.

Merged build finished. Test FAILed.

retest this please

Will merge when tests pass.

retest this please

Test build #76473 has started for PR 17300 at commit 56f5231.

Thanks both of you for review, I have addressed the comments and modified the test case. Please help calling jenkins for test, because I can't trigger that.  Thanks again.

Test build #76473 has finished for PR 17300 at commit 56f5231.

This patch fails from timeout after a configured wait of `250m`.
This patch merges cleanly.
This patch adds no public classes.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/76473/
Test FAILed.

Merged build finished. Test FAILed.

retest this please

retest this please

Test build #76500 has started for PR 17300 at commit a3e979f.

That is weird, jenkins should have restarted build when I commented ...

Test build #76500 has finished for PR 17300 at commit a3e979f.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/76500/
Test FAILed.

retest this please

Test build #76502 has started for PR 17300 at commit a3e979f.

Test build #76502 has finished for PR 17300 at commit a3e979f.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/76502/
Test PASSed.

Merged build finished. Test PASSed.

LGTM

Thanks for your review.

thanks, merging to master!

Thanks for merging @cloud-fan, this PR kept dropping form my list ...

Thanks @cloud-fan @mridulm @gatorsmile


vault kv secrets issuer

Hi,
I would like to know if it's possible to get certificate from vault kv secrets ?
I only see examples with PKI.
Thank you.
Guillaume

Hi @inteon,
Thank you for your quick response.
We will try to implement a custom issuer based on https://github.com/bank-vaults/vault-env, which already used Vault KV secrets.
Guillaume


Netflix Error check the log for more information. Raspbian + RaspberyPi B3

I'm submitting a ...

[ ] bug report
[ ] feature request
[X] support request

General information
I install Netflix from zipfile from here, and after install Netflix will not open. I get information "Netflix Error check the log for more information"
Addon version used
Version 0.14.0
CastagnaIT Respository
Installation

[X] I'm using this Netflix Repo
[ ] I'm using other Netflix Repo (Please specify which)
[ ] I'm using a different source (Please tell which)

Operating System

[ ] Linux (x86/x64)
[ ] OSX (x86/x64)
[ ] Windows (x86/x64)
[X] Linux (ARM) ---------- Raspberry Pi B3
[ ] Android

Additional informatin on the environment
[Descripe your environment a bit more detailed, are you using LibreElec f.e.]
I use Raspberry pi B3 with installed raspbian from RetroPie.org.uk it is original raspbian from RaspberryPi.org but with some applications cut out.
I install kodi by use sudo apt-get install kodi
Debug log
https://pastebin.com/mtv0Fqqr

you forgot to install Cryptdome, read the readme on the main page

OMG. Thank You very much for help me with this :)
But I have another issue. To watch some in Netflix I need inputstream.adaptive but i don't have VideoPlayer InputStream category in My-Addon section :/

goto menu addon->install from repository->select kodi rep.->scroll to video player inputstream install it


Retro: Launch Week 1

We ran our first Launch Week from March 21st-25th this year.
Please share your feedback in this public Issue, namely:

What went well
What could have been better
Open questions and ideas for next time

What went well

We got a bunch of functionality out of the door in a short space of time.
We shipped it, and can learn from it.

What could have been better

Not much engagement from people outside of our own team.
Didn't really get everyone involved (especially me and the customer success team IMHO). I think we captured about 65% of the stuff the company is working on. Specifically, we were trying to get Squeak! out as a growth-oriented product, and we coudl have used the customer success team to get existing customers more excited 1/1

Open questions and ideas for next time
If app team feel this helped them, then I'd be happy to repeat and would aim that the entire company is more focussed on this.
If the app team didn't feel this made much difference, I'd propose either (i) not doing this again, due to low external engagement or (ii) having a very different way of building hype / engagement from what we did this time

Went well

Company aligned around shipping a lot of great product work.
Lots of high quality content from a range of team members, inc. engineers.
Pushed us to fix a lot of things like Docs and updating the Website product pages.
Lots of awareness, including a viral HN post from @macobo (over 5k views) and 773k Twitter impressions for reasonable spend.

Do better

Bucketed under 'it didn't generate that much external hype otherwise':

It didn't generate a record week of signups (415 vs. 446). Did we fail, or was signups not the right target metric?
We didn't succeed in doing any community type stuff like a HogTalk - we couldn't secure a speaker.
More cross-promo with other companies to get the word out, retweets etc. would have helped.

Product release for self-host customers missed the launch week (though I think impact was minor tbh).

Open questions

The big one - should we do this again for ~plugins~ apps? This is a whole-product initiative rather than a marketing one, so it would be good to get wider feedback on this.

Went good

Got something out every day
Hackernews post
Got lots of internal engagement during production

Could have gone gooder

No real external hype or engagement (1)
Did not really line up with the actual release of promised features (2)
Fact that (2) didn't result in mass confusion is indicative of the scale of (1)

Ideas

Sign-ups does feel like the wrong metric, as I don't think anyone will sign-up to PostHog just because we have a data management suite. Better metric: Github stars as indicator of excitement.
If GitHub stars is the metric, plugins could make a good topic for the future.
If we were to repeat this, I think we need a clearer idea of promotion for each day beyond simple social posts. E.g. We need to be bringing in influencer support, driving to a clear goal.
Weird idea: "When we get to X Github stars, we'll do Launch Week 2!". Track this openly with a star-meter.
We could probably do a better job of reporting success internally each day. It felt at times like we were ticking boxes, then on to the next one without sharing excitement around the launch.

Putting in writing what I mentioned in sprint meeting, namely that one of / the main metric ought to be usage / awareness of the features we're announcing / promoting. This isn't exclusive to launch weeks, obviously, and basically agree that we should over commit to them or rule out doing another in future. Launch weeks and option, not a necessity.


Always starting SDWebImage operations from the main queue

Fixes issue https://github.com/rs/SDWebImage/issues/414.
Some fix is slated for release in rs/SDWebImage 4.0.0, but our crash rate for this bug is too high to ignore until it's patched.

Hi man, I got here from the main SDWebImage repo. My app suffers from the same issues and I saw you reference. Can you please confirm that this is a solid fix for the issue?
Thanks


Enhance step 3 of replication protocol  "Find Common Ancestry" (GET /source/_local/replication-id) to return Database update_seq too in response header, if a dbseq=true request header is provided in the request.

Enhance step 3 of replication protocol  "Find Common Ancestry" (GET /source/_local/replication-id) to return Database update_seq  too in response header, if a dbseq:true header is provided in the request.
Summary
As of now the first three steps of replication protocol is required to determine if a replication is required or not :

Verify Peers (HEAD /peers )
Get Peers Information (GET /peers to know update_seq of source and target)
Find Common Ancestry (GET /source/_local/replication-id)

For a client side replicator, these three steps (hence 3 HTTP requests) are needed to determine if replication is required or not.
However such clients (say browser based) only step 3:  GET /source/_local/replication-id,  can be sufficient  to determine if replication is required or not with remote server, provided:
If in step-3 we have a provision to provide DB update_seq, and a status text for 404 of what went wrong.(Wrong like DB do not exist/local doc do not exist).
In many cases a client side replicator won't have rights to create DB at remote CouchDB server. For such cases, without effecting any other cases, if this enhancement is done, we can save HTTP calls to the remote couchdb server.

Desired Behaviour
If in request "Find Common Ancestry" (GET /source/_local/replication-id) , if we provide a request header dbseq=true, it must return a response header update_seq: DB_UPDATE_SEQ, along with usual reply.
This will make step 2 a bit redundant.
And for making step 1 redudant, step 3 status text can be modified to give more information for 404, if its DB which do not exist or local doc.
Header based approach, can make old client work as they use to work, and new clients which are sure about source and target existence (say one time during mobile application start), can only do step 3 and know if they need to replicate or not.
Now this may be not a big change, but instead of 3 HTTP request, such clients will only require 1 HTTP request to determine if replication is required or not.
Possible Solution
To make replication protocol work all good with old clients, if for request GET /source/_local/replication-id if a dbseq:true header is provided, then along with the usual reponse, an addtional header update_seq too must be provided.
And if the DB do not exist, then along with 404 status text should be "DB do not exist", and if the local doc "_local/replication_id" do not exist then a corresponding message must be provided.
Additional context
My mobile app (with a client side replicator) do not have rights to create target at remote couchdb server, and do checks existence of remote DB once during initialization.
Currently while replicating , it needs to first get update_seq from DB and then get local replication doc.
If we can just enhance step3 (GET /source/_local/replication-id) to provide update seq of db too, then we can reduce first 3 steps to one step at client side replication.
The aim is to reduce multiple request to remote server, when it can be done in one single request.
So by a small enahcement such clients can determine requirement for replication in a single step.

If this is implemented, I will suggest dbseq=true to be a query param instead of request header, as its used as a query in here.
Correcting the requirement as per that.

If this is implemented, I will suggest dbseq=true to be a query param instead of request header, as its used as a query in here.
Correcting the requirement as per that.

Hi team,
Any comment on this ?

As far as db / doc not found, I think error reason is already provided:
 % http $DB/mydb2/doc4
HTTP/1.1 404 Object Not Found

{
    "error": "not_found",
    "reason": "Database does not exist."
}

% http $DB/mydb1/doc2
HTTP/1.1 404 Object Not Found

{
    "error": "not_found",
    "reason": "missing"
}

As for dbseq=true we have a working thing for it in main with using local_seq=true parameter. Notice
% http $DB/mydb1/doc1'?local_seq=true'

{
    "_id": "doc1",
    "_local_seq": "00000008d5c93d5a00000000",
    "_rev": "1-4b8a35d3f70a5962f86c6dd06ceb599c",
    "a": "b"
}

Not for 3.x series that will return just the shard local sequence which is not very useful for our purpose here:
% http $DB/mydb1/doc1'?local_seq=true'

{
    "_id": "doc1",
    "_local_seq": 1,
    "_rev": "1-4b8a35d3f70a5962f86c6dd06ceb599c",
    "a": "b"
}

So perhaps we could add a ?seq=true parameter which would return the full db seq on 3.x and then on main it would do the same as local_seq.

As far as db / doc not found, I think error reason is already provided:
 % http $DB/mydb2/doc4
HTTP/1.1 404 Object Not Found

{
    "error": "not_found",
    "reason": "Database does not exist."
}

Yes, my mistake. I some how missed it.

As for dbseq=true we have a similar idea already with the local_seq=true parameter:

Works only for normal docs, but for local docs (replication logs) it won't return anything. Morever the _local_seq is not DB update sequence which is used to determine if replication is required or not.

We return the doc update sequence along with a body. I wonder if we can co-opt this feature to return the current dbseq. Something like?
% http $DB/mydb1/doc1'?db_seq=true'

I would suggest it to be for local docs (replication logs) $db/mydb1/_local/replogs1?db_seq=true;

Works only for normal docs, but for local docs (replication logs) it won't return anything.

Right, local seq is the update seq of the document, so it won't be usable even if it worked for local documents. But since we have an API to return a sequence in the document I wanted to see if it would make sense having  the new API rather than invent something completely new, like returning data in the headers. So far I am 50/50 on the approaches...
With the main branch where sequences are global, it might be interesting to return both _local_seq and _db_seq so in one response we can tell when the document was updated relative to other documents, even in different databases on the same FDB cluster, and also, find out what the current db sequence is when this document was read.
Regarding the replication problem, I was wondering if connections are re-used if 1 vs 2 or 3 connection would have that much of an impact. Some time is spent during the SSL negotiation when setting up a new connection, but after that if the connection stays open there may not be a huge difference between having one or a few more requests.

we have an API to return a sequence in the document I wanted to see if it would make sense having the same kind of API rather than invent something completely new, like returning data in the headers...

Yeah any approach would be ok, as long as we get the DB update seq, along with the local docs. I suggested headers/flag based solution so that none of the old client has any kind API problems, for backward compatibility in mind.
Specifically the one which has some kind of checks in place for the data they receive rom couchdb.

With the main branch where sequences are global, it might be interesting to return both _local_seq and _db_seq so in one response we can tell when the document was updated relative to other documents, even in different databases on the same FDB cluster, and also, find out what the current db sequence is when this document was read.

Absolutely!, will have many usefull cases.

Regarding the replication problem, I was wondering if connections are re-used if 1 vs 2 or 3 connection would have that much of an impact. Some time is spent during the SSL negotiation when setting up a new connection, but after that if the connection stays open there may not be a huge difference between having one or a few more requests.

I am not able to understand this, not an expert on this matter. Are you suggesting that few steps of replication can be combined onto a single connections ?

At some point we might have another API or implement HTTP/2, HTTP/3, websockets, etc, but it probably won't be in the contexts of this issue. There are already issues on the roadmap with those ideas.
For now connection pooling as described in https://github.com/apache/couchdb/issues/3308#issuecomment-860778557 will avoid making new TCP/TLS connection.


Stop option does not work

Describe the bug

The stop option does not work.

To Reproduce
Steps to reproduce the behavior:

Create a new EV3 program
Open the simulation
Start the simulation
Open the background navigation dropdown.
Click on the stop option
See error

Expected behavior

The simulation should stop.

Screenshots

Device information

Type: Desktop
OS: MacOS
Browser: Chrome

Additional context

Additional information for the dropdown options should be given through a tooltip.

Sorry again, the whole menu should be removed, see #745.


Testing renter API functions

This PR expands TestIntegrationHostAndRent in order to attain complete coverage of renter API calls (excluding calls that rely on sharing, a feature that is currently disabled).
TestIntegrationHostAndRent is now unwieldy, but breaking the tests into separate functions wastes several seconds per function because of setup.
Renter testing is still incomplete, of course. We need to be doing tests that include multiple hosts and renters in realistic (out of sync) conditions.

Adding several seconds to testing time is not terrible if it makes the tests more approachable/readable. Especially if we start running nightly builds for the really long tests instead of running all tests every PR.

What's the status on this?

On its way, sorry

Latest changes:

clean up error checking
check that a file's renter nickname is a nonempty string
reorganize new testing into smaller tests rather than bloating up TestIntegration

Just needs some docstrings. Thanks for breaking down the functions more.

Getting 'too many open files' errors on my Mac, which we thought we fixed with the global http.DefaultClient in 1373. The failures occur even when the api package is run on its own.
=== RUN   TestRenterHostsAllHandler
--- FAIL: TestRenterHostsAllHandler (0.25s)
	renter_test.go:233: listen tcp: lookup localhost: no such host
=== RUN   TestRenterHandlerContracts
--- FAIL: TestRenterHandlerContracts (0.00s)
	renter_test.go:266: open /var/folders/ps/z2hlbr517dn0ps9b9wwpbnf00000gn/T/SiaTesting/api/TestRenterHandlerContracts/gateway/gateway.log: too many open files
=== RUN   TestRenterHandlerGetAndPost
--- FAIL: TestRenterHandlerGetAndPost (0.00s)
	renter_test.go:331: open /var/folders/ps/z2hlbr517dn0ps9b9wwpbnf00000gn/T/SiaTesting/api/TestRenterHandlerGetAndPost/gateway/gateway.log: too many open files
=== RUN   TestRenterLoadNonexistent
--- FAIL: TestRenterLoadNonexistent (0.00s)
	renter_test.go:428: open /var/folders/ps/z2hlbr517dn0ps9b9wwpbnf00000gn/T/SiaTesting/api/TestRenterLoadNonexistent/gateway/gateway.log: too many open files
=== RUN   TestRenterHandlerRename
--- FAIL: TestRenterHandlerRename (0.00s)
	renter_test.go:486: open /var/folders/ps/z2hlbr517dn0ps9b9wwpbnf00000gn/T/SiaTesting/api/TestRenterHandlerRename/gateway/gateway.log: too many open files
=== RUN   TestRenterHandlerDelete
--- FAIL: TestRenterHandlerDelete (0.00s)
	renter_test.go:569: open /var/folders/ps/z2hlbr517dn0ps9b9wwpbnf00000gn/T/SiaTesting/api/TestRenterHandlerDelete/gateway/gateway.log: too many open files
=== RUN   TestIntegrationHostAndRent
--- FAIL: TestIntegrationHostAndRent (0.00s)
	renterhost_test.go:29: open /var/folders/ps/z2hlbr517dn0ps9b9wwpbnf00000gn/T/SiaTesting/api/TestIntegrationHostAndRent/gateway/gateway.log: too many open files
=== RUN   TestExplorerPreset
--- FAIL: TestExplorerPreset (0.00s)
	server_test.go:16: open /var/folders/ps/z2hlbr517dn0ps9b9wwpbnf00000gn/T/SiaTesting/api/TestExplorerPreset/gateway/gateway.log: too many open files
=== RUN   TestReloading
--- FAIL: TestReloading (0.00s)
	server_test.go:40: open /var/folders/ps/z2hlbr517dn0ps9b9wwpbnf00000gn/T/SiaTesting/api/TestReloading/gateway/gateway.log: too many open files
=== RUN   TestAuthentication
--- FAIL: TestAuthentication (0.00s)
	server_test.go:71: open /var/folders/ps/z2hlbr517dn0ps9b9wwpbnf00000gn/T/SiaTesting/authenticated-api/TestAuthentication/gateway/gateway.log: too many open files
=== RUN   TestIntegrationWalletGETEncrypted
--- FAIL: TestIntegrationWalletGETEncrypted (0.00s)
	wallet_test.go:29: Failed to create gateway: open /var/folders/ps/z2hlbr517dn0ps9b9wwpbnf00000gn/T/SiaTesting/api/TestIntegrationWalletGETEncrypted/gateway/gateway.log: too many open files
=== RUN   TestIntegrationWalletBlankEncrypt
--- FAIL: TestIntegrationWalletBlankEncrypt (0.00s)
	wallet_test.go:94: open /var/folders/ps/z2hlbr517dn0ps9b9wwpbnf00000gn/T/SiaTesting/api/TestIntegrationWalletBlankEncrypt/gateway/gateway.log: too many open files
=== RUN   TestIntegrationWalletGETSiacoins
--- FAIL: TestIntegrationWalletGETSiacoins (0.00s)
	wallet_test.go:156: open /var/folders/ps/z2hlbr517dn0ps9b9wwpbnf00000gn/T/SiaTesting/api/TestIntegrationWalletGETSiacoins/gateway/gateway.log: too many open files
=== RUN   TestIntegrationWalletTransactionGETid
--- FAIL: TestIntegrationWalletTransactionGETid (0.00s)
	wallet_test.go:250: open /var/folders/ps/z2hlbr517dn0ps9b9wwpbnf00000gn/T/SiaTesting/api/TestIntegrationWalletTransactionGETid/gateway/gateway.log: too many open files
FAIL
FAIL	github.com/NebulousLabs/Sia/api	13.688s

rebase to master, we pulled the mac build. I'm not sure what the file limit is, but I think it's very low. The ecosystem test I was running was triggering the error regardless of defer Close calls. There might be a file leak somewhere, but it seems more likely that the max files on the mac build is under 1000, maybe under 100. 6 nodes simultaneously seems to overwhelm it.

I had rebased; Travis passed, just not my Mac.  Increased the open files limit on my Mac from 256 (the default) to 1024 (ulimit -n 1024) and tests started passing again.  The limit currently needs to be more than 367.

Should be ready to merge.


Relationship to Phone Model

With the new Phone (https://github.com/tipoff/addresses/issues/58) model, all strings of phone numbers in this package should be replaced with a relationship to that model.

The Phone model will have a custom nova field (https://github.com/tipoff/addresses/issues/108) and it will need to be added to the nova resources that have the Phone relationship.


Very minor spelling fix in README

Makes a minor change in the README file from "Used interally by" -->  "Used internally by"
Changes
Minor change from "interally" to "internally".
Testing
N/A
Notes
Checklist

[x] PR has an informative and human-readable title (this will be pulled into the release notes)
[x] Changes are limited to a single goal (no scope creep)
[x] Code can be automatically merged (no conflicts)
[ ] Code passed the pre-commit check & code is left cleaner/nicer than when first encountered.
[ ] Passes all existing automated tests
[ ] Any change in functionality is tested
[x] New functions are documented (with a description, list of inputs, and expected output)
[ ] Placeholder code is flagged / future TODOs are captured in comments
[ ] Project documentation has been updated if adding/changing functionality.
[ ] Reviewers requested with the Reviewers tool :arrow_right:

Testing checklist
Python - local testing

[ ] python 3.6
[ ] python 3.7

Thanks for the 👁 👁 !! Always appreciate attention to detail.

Sure thing! Happy to help (albeit in a very tiny way!). Thank you for your work on this project.


findMany unexpectedly deduplicates

Laravel Version
v11.8.0-57-g83e28d065b
PHP Version
PHP 8.3.8 (cli)
Database Driver & Version
No response
Description
findMany([1, 1]) returns one object instead of two. This is not obvious from the documentation:

Find multiple models by their primary keys.

findMany([1, 1]) creates a query with 'WHERE id IN (1, 1)`, and that returns one row. I think the best solution is to change the documentation. Either by explicitly describing that it deduplicates, or implicitly, e.g. "Finds all models whose primary keys are contained in the given array."
Steps To Reproduce
Use findMany to retrieve the same object multiple times:
var_dump(count(findMany([1, 1])));

I'm not sure what you expect here? I feel like all of this is very obvious? You're of course free to attempt a PR to the docs 👍


Correct Mod Logo

Changed logo and else depended textures so they match mod's name

not planned. :face_exhaling: