RFC: Replace controllers with Redux slices

This represents a prototype implementation of a new controller API design. The motivations for this are written about in this document: https://docs.google.com/document/d/1DjtFfNB0JuziijGqs4h3vW8PL8zCI6zuaovqvSHbJ2o/edit#heading=h.mtvpfhpuzjof
The CurrencyRateController has been updated to use Redux.
Note that the CurrencyRateController no longer works correctly with the ComposableController - I didn't think that was worth the effort to get working.

We've decided to go with the other prototype for now.


DocsArchive-<IP_ADDRESS>Auto archive content

Auto archive content from https://github.com/OPS-E2E-PPE/docs-archive-test-source

Docs Build status updates of commit 32096cd:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error]

Cannot sync git repo to specified commit because branch Release_Archive_master_2020-02-22-14-41-03 has been deleted or has been force pushed: fatal: couldn't find remote ref refs/heads/Release_Archive_master_2020-02-22-14-41-03

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


--auto-publish return 400 error

When i try to run publish with autopublish like here
webstore upload --auto-publish --source extension.zip --extension-id $EXTENSION_ID --client-id $CLIENT_ID --client-secret $CLIENT_SECRET --refresh-token $REFRESH_TOKEN
it returns Response code 400 (Bad Request)

#10 and have a look at the guide. You might be using the wrong keys: https://github.com/bfred-it/chrome-webstore-upload/blob/aae21046d0ae214769993fc7294e34dd3bc23857/How to generate Google API keys.md

Try 1.1.1 to see the full error.

@bfred-it Could it be that we're experiencing the same issues with the Refined GitHub builds? The last four CRON builds failed because of this error resulting in the Chrome version not being published:
> @ release:cws /home/travis/build/sindresorhus/refined-github
> cd distribution && webstore upload --auto-publish
Response code 400 (Bad Request)
{
    "error": "invalid_request",
    "error_description": "Parameter not allowed for this message type: redirect_uri"
}

Quote from a StackOverflow answer on an issue with the same error:

It seems Google has recently changed the url for getting a new access token from a refresh token. The older url was https://accounts.google.com/o/oauth2/token and the new one is https://www.googleapis.com/oauth2/v4/token.

No, that's https://github.com/DrewML/chrome-webstore-upload/pull/14 soon to be published.

hello @bfred-it,
you can probably close this one since DrewML/chrome-webstore-upload#14 is merged


community: add tencent cloud as an adopter

What type of PR is this?

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

Release Notes: No

this doesnt seem like the right place @Xunzhuo, we need to update https://github.com/envoyproxy/gateway/blob/main/site/data/adopters.yaml


[SPARK-32844][SQL] Make DataFrameReader.table take the specified options for datasource v1

What changes were proposed in this pull request?
Make DataFrameReader.table take the specified options for datasource v1.
Why are the changes needed?
Keep the same behavior of v1/v2 datasource, the v2 fix has been done in SPARK-32592.
Does this PR introduce any user-facing change?
Yes. The DataFrameReader.table will take the specified options. Also, if there are the same key and value exists in specified options and table properties, an exception will be thrown.
How was this patch tested?
New UT added.

cc @cloud-fan

Merged build finished. Test PASSed.

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

Test build #128511 has started for PR 29712 at commit 33b8fca.

Merged build finished. Test PASSed.

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

Test build #128518 has started for PR 29712 at commit b4aa95e.

Test build #128511 has finished for PR 29712 at commit 33b8fca.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
case class UnresolvedCatalogRelation(

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/128511/
Test FAILed.

Test build #128518 has finished for PR 29712 at commit b4aa95e.

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

Merged build finished. Test PASSed.

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

Test build #128564 has started for PR 29712 at commit cc250b9.

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

Test build #128565 has started for PR 29712 at commit 05aa245.

Merged build finished. Test PASSed.

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

Test build #128564 has finished for PR 29712 at commit cc250b9.

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

Merged build finished. Test PASSed.

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

Test build #128565 has finished for PR 29712 at commit 05aa245.

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

Merged build finished. Test PASSed.

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

Test build #128579 has started for PR 29712 at commit f3f8c8d.

Merged build finished. Test PASSed.

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

Test build #128579 has finished for PR 29712 at commit f3f8c8d.

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

Merged build finished. Test PASSed.

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

thanks, merging to master!

Thanks for the review.


[AMNESIA] Add service tags as value objects?

Would it be a good idea to add default symfony service tags in a value object.
/**
 * Class ServiceTags
 */
final class ServiceTags
{

    const CONSOLE_COMMAND = 'console.command';

}

$services->set(MyConsoleCommand::class)->tag(ServiceTags::CONSOLE_COMMAND);

https://symfony.com/doc/current/reference/dic_tags.html

I'm surprised this is not in Symfony 😮
Go for it!

https://github.com/symplify/symplify/pull/3234

Closing as resolved, thank you :+1:
Btw, to avoid this manual closing, you can use "Closing" keyword in PR description:

It must be added before merging to work.


create github sponsors integration

solves #18320. In addition to the created event mentioned in the issue, I have added cases for tier changes and cancellations as well.
Testing plan:

Followed the tutorial on the hello world integration and created tests for each of the events (created, pending_tier_change, pending_cancellation)

Can you squash the commits and rebase this PR to not have merge commits?
Check out our GitHub guide and commit guidelines for advice on how to do rebases and more details.

Closing in favor of https://github.com/zulip/zulip/pull/24836, which seems the most developed of the PRs for this issue.


[Translation] plugins/module-concatenation-plugin.mdx

Summary

[x] Translate plugins/pmodule-concatenation-plugin.mdx

https://github.com/line/webpack.kr/pull/388

Summary

[x]  Translate plugins/pmodule-concatenation-plugin.mdx

https://github.com/line/webpack.kr/pull/388

#388


Usability: Reporting of errors in source code not friendly

rust-analyzer version: rust-analyzer version: 6634eaf13 2022-01-30 nightly
My rust source code has errors, and the way errors are presented (in vscode) is not optimal in helping me addressing these errors.
My (faulty) source code is:
fn returns_option() -> Option<i32> {
    return None;
}

pub fn wombat() {
    println!("hello

    ");

    let x = returns_option()?;
    println!("{x}");

    let y = returns_option()?;
    println!("{y}");
}

Obviously, returns_option()? on pub fn wombat() { is no good, so there's that.
The way this is presented to me in vscode is not really helpful, though:

There are three different "issues" that I expect to be raised
a) wombat does not return Option, although the ? operator is present inside the body: line 5
b) ? operator is present in two locations: lines 16, 20
Not friendly (I): While I get diagnostics for all three issues, the ? diagnostics get duplicated. I'd prefer to not see these diagnostics. Imagine pasting copy with a bunch of ? operators ... It seems as if the "label" and the "help" (rustc terminology) are each issued as distinct diagnostics, while they'd apply both to the one root problem.
Not friendly (II): The complete body of the function has red squiggles. Already in my light-on-text sample, the red squiggles dominate everything, make the function hard to read. And, of course, by marking the complete body of the function, any squiggles only applied to each of the ? operators gets masked, making it somewhat less convenient to visually locate those cases.

rustc presentation is much more approachable, IMHO:

When it comes to highlighting errors in functions, would it make sense to render the squiggles not over the complete span of the function at large, but leave out the body block (i.e. the { ... } implementation part) and only squiggle the function's sort-of-declaration part (in my case pub fn wombat())

Reducing the number of diagnostics by collapsing "hint" text and the "help:" component would help into the same diagnostic entry would probably reduce visual load, too.
It seems as if there are "error", "label", "hint" and "help" components for each diagnostic? If so, get them all into the same diagnostic, prefix consistently in text? (vscode doesn't support custom drawn visual cues in that panel, I believe)

What rustc version are you compiling on?(Since these are rustc diagnostics)
Interestingly enough, what I get is the following:
It seems for one the help span is not being collapsed into the first three characters of the function like it is for me, and secondly your diagnostic entries are duplicated in the problems pane. Now I am not sure why that is though.

❯ rustc -V
rustc 1.58.1 (db9d1b20b 2022-01-20)
And, for good measure: vscode 1.63.2

FWIW, I am on Fedora Linux 35, and I have a ... handful ... of extensions active in this workspace (none of which matters, I would seriously hope)

I just disabled every vscode extension for that workspace except for

codelldb
rust-analyzer
... was forced to "reload" the vscode <whatever - extension host?> ... and  ...

It works well.
This must have been some extension, because the box I restart often enough, vscode I restart often enough ... Blech.

I suspect that https://marketplace.visualstudio.com/items?itemName=panicbit.cargo (v0.2.3) is the culprit for "my mess".
Above extension

disabled - very usable error diagnostics from rust analyzer
enabled - see my mess

Anybody able to confirm?

That would explain the duplicate errors and different spans.

Checks your code using cargo check (can be disabled)

You probably want to disable this part of the extension, since r-a already does that for you.

I think this has been resolved then.


Fix syntax in ceph_integration.yml

There is an unneeded space is ceph_integration.yml that will throw msg: this module requires key=value arguments errors when deploying cinder on package installs.

Can one of the admins verify this patch?

"ok to test" to accept this pull request for testing
"test this please" for a one time test run
"add to whitelist" to add the author to the whitelist

LGTM

ok to test

retest


Bump fedora-coreos-config to latest next-devel

Some images are still missing from the origin/4.9 namespace since they're not yet being promoted there. Failures are thus currently expected on master here (which is still release-4.9).
/lgtm

/hold
Still something wrong with the package list, investigating

Closing in favor of #242


Unexpected Behavior in PyQt6

Occasionally, PyQt6's C/C++ objects are forcibly deleted.
This issue primarily occurs with QTimer or QNetwork-related objects.
TwitchLink_2023.10.30_09-00-48#cc9efbd6-a49b-4668-a66e-2118ec3a62ae.log

Hi there,
It has been quite a while since this issue was last discussed. During this time, numerous bug fixes have been implemented in Qt, and TwitchLink has also updated its base Qt version to the latest releases several times.
Given these updates, it is very likely that the issue caused by previous Qt bugs has been resolved. As a result, I am closing this issue.
However, if you encounter the same or a similar problem in the future, please feel free to open a new issue.
Closed with commit d5bf121291784c5643ade9c653ef822b5ea65b2b


[Captcha] Push patchnote 2

This PR consist of alerting user of the repository's change coming in Fixator10/Fixator10-Cogs#163, this needs to be pushed AFTER @Fixator10 has pushed its PR.
Some days after this PR get pushed, I will open a new PR to remove the captcha cog from this repository, more likely in the next 30 days.

@Kreusada Feel free to merge by now.


Try Mac integration tests via "swift test" without a test plan

Issue #

Description of changes

New/existing dependencies impact assessment, if applicable

Conventional Commits

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Opened for testing.  Closing & deleting.


make (e=2): The system cannot find the file specified.

Help
I run make and get this error:
PS C:\Users\George\Workspace\OSdev> .\make
mkdir -p build
nasm src/bootloader/boot.asm -f bin -o build/bootloader.bin
process_begin: CreateProcess(NULL, nasm src/bootloader/boot.asm -f bin -o build/bootloader.bin, ...) failed.
make (e=2): The system cannot find the file specified.
make: *** [Makefile:25: build/bootloader.bin] Error 2

The way the project is setup, you will need a unix-like environment. WSL is the easiest option. Alternatively, you can try msys2/cygwin or running a Linux distro in a VM.

i`m using wsl and the same is happening


Feature Request: CLI tool that talks with GitHub directly

Today I experience slowness of the website. GitHub loads fine, so I assume the server is under load for some reason? That got me thinking, it would be nice to have a gi tool that talks directly with GitHub to generate my .gitignore so there is no dependency on gitignore.io.

Hi @martin-braun, we haven't checked anything out of the ordinary in the error rate or response time in our servers:

Even in our spikes, the requests keep under 300ms (average of 10 minutes). Do you think that could be generated by a local issue?
Using raw files directly from Github could be an alternative for a fork, but I feel it contradicts the current architecture.


multi node training error:NCCL error in: ../torch/csrc/distributed/c10d/ProcessGroupNCCL.cpp:1269, internal error, NCCL version 2.14.3 ncclInternalError: Internal check failed.

Bug description
I'm using a kubernetes environment cluster(6 nodes), and I want to use multi-node for training. but it always encounters the error as described in the title.
I use the following scripts to start my program(it will be run on every node):
#!/bin/bash -l

SCRIPTPATH=$(dirname $(readlink -f "$0"))
PROJECT_DIR="${SCRIPTPATH}/../../"

export PYTHONPATH=$PROJECT_DIR:$PYTHONPATH
cd $PROJECT_DIR

# set visible GPUs
export CUDA_VISIBLE_DEVICES=0,1,2,3,4,5,6,7

# NCCL debug
export NCCL_DEBUG=INFO
export NCCL_DEBUG_SUBSYS=ALL
# NCCL_IB_DISABLE=1
echo "*************************************************"

n_nodes=6
n_gpus_per_node=8
torch_num_workers=16
accumulate_grad_batches=1
batch_size=3 # real batch size = batch_size * accumulate_grad_batches
pin_memory=true

python -u ./train.py \
    --accelerator="gpu" --devices=${n_gpus_per_node} --precision=32 --num_nodes=${n_nodes} \
    --batch_size=${batch_size} --num_workers=${torch_num_workers} --pin_memory=${pin_memory} \
    --check_val_every_n_epoch=1 \
    --log_every_n_steps=20 \
    --limit_val_batches=1. \
    --num_sanity_val_steps=10 \
    --benchmark=True \
    --max_epochs=50

When I use only one node, everything goes well.
I checked the doc and found that the following statement:
MASTER_ADDR=localhost MASTER_PORT=random() WORLD_SIZE=3 NODE_RANK=0 LOCAL_RANK=0 python my_file.py --accelerator 'gpu' --devices 3 --etc

My understanding is that the lightning will set the MASTER_ADDR of every node to localhost, but the kubernetes environment has set a default MASTER_ADDR when it starts and will be overwrite to localhost by lightning. I think it is not very reasonable as the MASTER_ADDR should be the same among all nodes rather than the localhost of themself. The same applies to MASTER_PORT.
The specific error information is as follows. Thinks!
cc@awaelchli
What version are you seeing the problem on?
v1.9
How to reproduce the bug
the trainer is:
trainer = pl.Trainer.from_argparse_args(
        args,
        strategy=DDPStrategy(find_unused_parameters=False),
        gradient_clip_val=config.TRAINER.GRADIENT_CLIPPING,
        callbacks=callbacks,
        logger=logger,
        sync_batchnorm=config.TRAINER.WORLD_SIZE > 0,
        replace_sampler_ddp=False,  # use custom sampler
        reload_dataloaders_every_n_epochs=0,  # avoid repeated samples!
        # weights_summary='full',
        profiler=profiler)

Error messages and logs
# Error messages and logs here please

on master
2023-10-09 14:25:43.857 | INFO     | __main__:main:130 - Trainer initialized!
2023-10-09 14:25:43.857 | INFO     | __main__:main:131 - Start training!
[rank: 0] Global seed set to 66
Initializing distributed: GLOBAL_RANK: 7, MEMBER: 8/48
2023-10-09 14:25:43.882 | INFO     | __main__:main:130 - Trainer initialized!
2023-10-09 14:25:43.883 | INFO     | __main__:main:131 - Start training!
[rank: 0] Global seed set to 66
Initializing distributed: GLOBAL_RANK: 4, MEMBER: 5/48
2023-10-09 14:25:43.920 | INFO     | __main__:main:130 - Trainer initialized!
2023-10-09 14:25:43.920 | INFO     | __main__:main:131 - Start training!
[rank: 0] Global seed set to 66
Initializing distributed: GLOBAL_RANK: 5, MEMBER: 6/48
2023-10-09 14:25:43.930 | INFO     | __main__:main:130 - Trainer initialized!
2023-10-09 14:25:43.931 | INFO     | __main__:main:131 - Start training!
[rank: 0] Global seed set to 66
Initializing distributed: GLOBAL_RANK: 1, MEMBER: 2/48
----------------------------------------------------------------------------------------------------
distributed_backend=nccl
All distributed processes registered. Starting with 48 processes
----------------------------------------------------------------------------------------------------
t65239c8810264be35dd36d461696832660195-master-0:122:122 [0] NCCL INFO Bootstrap : Using eth0:<IP_ADDRESS><0>
t65239c8810264be35dd36d461696832660195-master-0:122:122 [0] NCCL INFO NET/Plugin : No plugin found (libnccl-net.so), using internal implementation
t65239c8810264be35dd36d461696832660195-master-0:122:122 [0] NCCL INFO cudaDriverVersion 11060
NCCL version 2.14.3+cuda11.6
t65239c8810264be35dd36d461696832660195-master-0:122:122 [0] NCCL INFO init.cc:1147 Cuda Host Alloc Size 4 pointer 0x7fd3c2600000
t65239c8810264be35dd36d461696832660195-master-0:122:1638 [0] NCCL INFO NCCL_IB_HCA set to mlx5
t65239c8810264be35dd36d461696832660195-master-0:122:1638 [0] NCCL INFO NET/IB : Using [0]mlx5_1:1/RoCE [1]mlx5_2:1/RoCE [2]mlx5_3:1/RoCE [3]mlx5_4:1/RoCE [RO]; OOB eth0:<IP_ADDRESS><0>
t65239c8810264be35dd36d461696832660195-master-0:122:1638 [0] NCCL INFO Using network IB

on worker-0
...
[rank: 1] Global seed set to 66
Initializing distributed: GLOBAL_RANK: 5, MEMBER: 6/48
----------------------------------------------------------------------------------------------------
distributed_backend=nccl
All distributed processes registered. Starting with 48 processes
----------------------------------------------------------------------------------------------------
t65239c8810264be35dd36d461696832660195-worker-0:132:132 [0] NCCL INFO Bootstrap : Using eth0:<IP_ADDRESS><0>
t65239c8810264be35dd36d461696832660195-worker-0:132:132 [0] NCCL INFO NET/Plugin : No plugin found (libnccl-net.so), using internal implementation
t65239c8810264be35dd36d461696832660195-worker-0:132:132 [0] NCCL INFO cudaDriverVersion 11060
NCCL version 2.14.3+cuda11.6
t65239c8810264be35dd36d461696832660195-worker-0:132:132 [0] NCCL INFO init.cc:1147 Cuda Host Alloc Size 4 pointer 0x7f82ee600000
t65239c8810264be35dd36d461696832660195-worker-0:132:1712 [0] NCCL INFO NCCL_IB_HCA set to mlx5
t65239c8810264be35dd36d461696832660195-worker-0:132:1712 [0] NCCL INFO NET/IB : Using [0]mlx5_1:1/RoCE [1]mlx5_2:1/RoCE [2]mlx5_3:1/RoCE [3]mlx5_4:1/RoCE [RO]; OOB eth0:<IP_ADDRESS><0>
t65239c8810264be35dd36d461696832660195-worker-0:132:1712 [0] NCCL INFO Using network IB
-------------------------------------------------------------------------------
train.py 173 <module>
main()
train.py 171 main
trainer.fit(model, datamodule=data_module)
trainer.py 608 fit
call._call_and_handle_interrupt(
call.py 38 _call_and_handle_interrupt
return trainer_fn(*args, **kwargs)
trainer.py 650 _fit_impl
self._run(model, ckpt_path=self.ckpt_path)
trainer.py 1040 _run
self.__setup_profiler()
trainer.py 1500 __setup_profiler
self.profiler.setup(stage=self.state.fn, local_rank=local_rank, log_dir=self.log_dir)
trainer.py 1817 log_dir
dirpath = self.strategy.broadcast(dirpath)
ddp.py 314 broadcast
torch.distributed.broadcast_object_list(obj, src, group=_group.WORLD)
distributed_c10d.py 2084 broadcast_object_list
broadcast(object_sizes_tensor, src=src, group=group)
distributed_c10d.py 1400 broadcast
work = default_pg.broadcast([tensor], opts)
RuntimeError:
NCCL error in: ../torch/csrc/distributed/c10d/ProcessGroupNCCL.cpp:1269, internal error, NCCL version 2.14.3
ncclInternalError: Internal check failed.
Last error:
Net : Connect to <IP_ADDRESS><61785> failed : Connection refused
...

on worker-1
...
[rank: 2] Global seed set to 66
Initializing distributed: GLOBAL_RANK: 5, MEMBER: 6/48
[rank: 2] Global seed set to 66
Initializing distributed: GLOBAL_RANK: 2, MEMBER: 3/48
[rank: 2] Global seed set to 66
Initializing distributed: GLOBAL_RANK: 7, MEMBER: 8/48
[rank: 2] Global seed set to 66
Initializing distributed: GLOBAL_RANK: 4, MEMBER: 5/48
----------------------------------------------------------------------------------------------------
distributed_backend=nccl
All distributed processes registered. Starting with 48 processes
----------------------------------------------------------------------------------------------------
t65239c8810264be35dd36d461696832660195-worker-1:134:134 [0] NCCL INFO Bootstrap : Using eth0:<IP_ADDRESS><0>
t65239c8810264be35dd36d461696832660195-worker-1:134:134 [0] NCCL INFO NET/Plugin : No plugin found (libnccl-net.so), using internal implementation
t65239c8810264be35dd36d461696832660195-worker-1:134:134 [0] NCCL INFO cudaDriverVersion 11060
NCCL version 2.14.3+cuda11.6
t65239c8810264be35dd36d461696832660195-worker-1:134:134 [0] NCCL INFO init.cc:1147 Cuda Host Alloc Size 4 pointer 0x7f645a600000
t65239c8810264be35dd36d461696832660195-worker-1:134:1729 [0] NCCL INFO NCCL_IB_HCA set to mlx5
t65239c8810264be35dd36d461696832660195-worker-1:134:1729 [0] NCCL INFO NET/IB : Using [0]mlx5_1:1/RoCE [1]mlx5_2:1/RoCE [2]mlx5_3:1/RoCE [3]mlx5_4:1/RoCE [RO]; OOB eth0:<IP_ADDRESS><0>
t65239c8810264be35dd36d461696832660195-worker-1:134:1729 [0] NCCL INFO Using network IB
t65239c8810264be35dd36d461696832660195-worker-1:134:1726 [0] bootstrap.cc:125 NCCL WARN Bootstrap Root : rank 3 of 48 ranks has already checked in
-------------------------------------------------------------------------------
train.py 173 <module>
main()
train.py 171 main
trainer.fit(model, datamodule=data_module)
trainer.py 608 fit
call._call_and_handle_interrupt(
call.py 38 _call_and_handle_interrupt
return trainer_fn(*args, **kwargs)
trainer.py 650 _fit_impl
self._run(model, ckpt_path=self.ckpt_path)
trainer.py 1040 _run
self.__setup_profiler()
trainer.py 1500 __setup_profiler
self.profiler.setup(stage=self.state.fn, local_rank=local_rank, log_dir=self.log_dir)
trainer.py 1817 log_dir
dirpath = self.strategy.broadcast(dirpath)
ddp.py 314 broadcast
torch.distributed.broadcast_object_list(obj, src, group=_group.WORLD)
distributed_c10d.py 2084 broadcast_object_list
broadcast(object_sizes_tensor, src=src, group=group)
distributed_c10d.py 1400 broadcast
work = default_pg.broadcast([tensor], opts)
RuntimeError:
NCCL error in: ../torch/csrc/distributed/c10d/ProcessGroupNCCL.cpp:1269, internal error, NCCL version 2.14.3
ncclInternalError: Internal check failed.
Last error:
Net : Connect to <IP_ADDRESS><61785> failed : Connection refused
-------------------------------------------------------------------------------
train.py 173 <module>
...

on worker-2
...
[rank: 3] Global seed set to 66
Initializing distributed: GLOBAL_RANK: 7, MEMBER: 8/48
[rank: 3] Global seed set to 66
Initializing distributed: GLOBAL_RANK: 4, MEMBER: 5/48
[rank: 3] Global seed set to 66
Initializing distributed: GLOBAL_RANK: 6, MEMBER: 7/48
----------------------------------------------------------------------------------------------------
distributed_backend=nccl
All distributed processes registered. Starting with 48 processes
----------------------------------------------------------------------------------------------------
t65239c8810264be35dd36d461696832660195-worker-2:135:135 [0] NCCL INFO Bootstrap : Using eth0:<IP_ADDRESS><0>
t65239c8810264be35dd36d461696832660195-worker-2:135:135 [0] NCCL INFO NET/Plugin : No plugin found (libnccl-net.so), using internal implementation
t65239c8810264be35dd36d461696832660195-worker-2:135:135 [0] NCCL INFO cudaDriverVersion 11060
NCCL version 2.14.3+cuda11.6
t65239c8810264be35dd36d461696832660195-worker-2:135:135 [0] NCCL INFO init.cc:1147 Cuda Host Alloc Size 4 pointer 0x7f8480600000
t65239c8810264be35dd36d461696832660195-worker-2:135:1715 [0] NCCL INFO NCCL_IB_HCA set to mlx5
t65239c8810264be35dd36d461696832660195-worker-2:135:1715 [0] NCCL INFO NET/IB : Using [0]mlx5_1:1/RoCE [1]mlx5_2:1/RoCE [2]mlx5_3:1/RoCE [3]mlx5_4:1/RoCE [RO]; OOB eth0:<IP_ADDRESS><0>
t65239c8810264be35dd36d461696832660195-worker-2:135:1715 [0] NCCL INFO Using network IB
t65239c8810264be35dd36d461696832660195-worker-2:135:1714 [0] bootstrap.cc:125 NCCL WARN Bootstrap Root : rank 4 of 48 ranks has already checked in
-------------------------------------------------------------------------------
train.py 173 <module>
main()
train.py 171 main
trainer.fit(model, datamodule=data_module)
trainer.py 608 fit
call._call_and_handle_interrupt(
call.py 38 _call_and_handle_interrupt
return trainer_fn(*args, **kwargs)
trainer.py 650 _fit_impl
self._run(model, ckpt_path=self.ckpt_path)
trainer.py 1040 _run
self.__setup_profiler()
trainer.py 1500 __setup_profiler
self.profiler.setup(stage=self.state.fn, local_rank=local_rank, log_dir=self.log_dir)
trainer.py 1817 log_dir
dirpath = self.strategy.broadcast(dirpath)
ddp.py 314 broadcast
torch.distributed.broadcast_object_list(obj, src, group=_group.WORLD)
distributed_c10d.py 2084 broadcast_object_list
broadcast(object_sizes_tensor, src=src, group=group)
distributed_c10d.py 1400 broadcast
work = default_pg.broadcast([tensor], opts)
RuntimeError:
NCCL error in: ../torch/csrc/distributed/c10d/ProcessGroupNCCL.cpp:1269, internal error, NCCL version 2.14.3
ncclInternalError: Internal check failed.
Last error:
Net : Connect to <IP_ADDRESS><61785> failed : Connection refused
-------------------------------------------------------------------------------
train.py 173 <module>
...

on worker-3
...
[rank: 4] Global seed set to 66
Initializing distributed: GLOBAL_RANK: 2, MEMBER: 3/48
[rank: 4] Global seed set to 66
Initializing distributed: GLOBAL_RANK: 5, MEMBER: 6/48
Initializing distributed: GLOBAL_RANK: 4, MEMBER: 5/48
----------------------------------------------------------------------------------------------------
distributed_backend=nccl
All distributed processes registered. Starting with 48 processes
----------------------------------------------------------------------------------------------------
t65239c8810264be35dd36d461696832660195-worker-3:136:136 [0] NCCL INFO Bootstrap : Using eth0:<IP_ADDRESS><0>
t65239c8810264be35dd36d461696832660195-worker-3:136:136 [0] NCCL INFO NET/Plugin : No plugin found (libnccl-net.so), using internal implementation
t65239c8810264be35dd36d461696832660195-worker-3:136:136 [0] NCCL INFO cudaDriverVersion 11060
NCCL version 2.14.3+cuda11.6
t65239c8810264be35dd36d461696832660195-worker-3:136:136 [0] NCCL INFO init.cc:1147 Cuda Host Alloc Size 4 pointer 0x7fb0ee600000
t65239c8810264be35dd36d461696832660195-worker-3:136:1716 [0] NCCL INFO NCCL_IB_HCA set to mlx5
t65239c8810264be35dd36d461696832660195-worker-3:136:1716 [0] NCCL INFO NET/IB : Using [0]mlx5_1:1/RoCE [1]mlx5_2:1/RoCE [2]mlx5_3:1/RoCE [3]mlx5_4:1/RoCE [RO]; OOB eth0:<IP_ADDRESS><0>
t65239c8810264be35dd36d461696832660195-worker-3:136:1716 [0] NCCL INFO Using network IB
-------------------------------------------------------------------------------
train.py 173 <module>
main()
train.py 171 main
trainer.fit(model, datamodule=data_module)
trainer.py 608 fit
call._call_and_handle_interrupt(
call.py 38 _call_and_handle_interrupt
return trainer_fn(*args, **kwargs)
trainer.py 650 _fit_impl
self._run(model, ckpt_path=self.ckpt_path)
trainer.py 1040 _run
self.__setup_profiler()
trainer.py 1500 __setup_profiler
self.profiler.setup(stage=self.state.fn, local_rank=local_rank, log_dir=self.log_dir)
trainer.py 1817 log_dir
dirpath = self.strategy.broadcast(dirpath)
ddp.py 314 broadcast
torch.distributed.broadcast_object_list(obj, src, group=_group.WORLD)
distributed_c10d.py 2084 broadcast_object_list
broadcast(object_sizes_tensor, src=src, group=group)
distributed_c10d.py 1400 broadcast
work = default_pg.broadcast([tensor], opts)
RuntimeError:
NCCL error in: ../torch/csrc/distributed/c10d/ProcessGroupNCCL.cpp:1269, internal error, NCCL version 2.14.3
ncclInternalError: Internal check failed.
Last error:
Net : Connect to <IP_ADDRESS><40725> failed : Connection refused
...

on worker-4
...
[rank: 5] Global seed set to 66
Initializing distributed: GLOBAL_RANK: 2, MEMBER: 3/48
[rank: 5] Global seed set to 66
Initializing distributed: GLOBAL_RANK: 6, MEMBER: 7/48
[rank: 5] Global seed set to 66
Initializing distributed: GLOBAL_RANK: 7, MEMBER: 8/48
[rank: 5] Global seed set to 66
Initializing distributed: GLOBAL_RANK: 3, MEMBER: 4/48
[rank: 5] Global seed set to 66
Initializing distributed: GLOBAL_RANK: 5, MEMBER: 6/48
----------------------------------------------------------------------------------------------------
distributed_backend=nccl
All distributed processes registered. Starting with 48 processes
----------------------------------------------------------------------------------------------------
t65239c8810264be35dd36d461696832660195-worker-4:137:137 [0] NCCL INFO Bootstrap : Using eth0:<IP_ADDRESS><0>
t65239c8810264be35dd36d461696832660195-worker-4:137:137 [0] NCCL INFO NET/Plugin : No plugin found (libnccl-net.so), using internal implementation
t65239c8810264be35dd36d461696832660195-worker-4:137:137 [0] NCCL INFO cudaDriverVersion 11060
NCCL version 2.14.3+cuda11.6
t65239c8810264be35dd36d461696832660195-worker-4:137:137 [0] NCCL INFO init.cc:1147 Cuda Host Alloc Size 4 pointer 0x7ff6ac600000
t65239c8810264be35dd36d461696832660195-worker-4:137:1717 [0] NCCL INFO NCCL_IB_HCA set to mlx5
t65239c8810264be35dd36d461696832660195-worker-4:137:1717 [0] NCCL INFO NET/IB : Using [0]mlx5_1:1/RoCE [1]mlx5_2:1/RoCE [2]mlx5_3:1/RoCE [3]mlx5_4:1/RoCE [RO]; OOB eth0:<IP_ADDRESS><0>
t65239c8810264be35dd36d461696832660195-worker-4:137:1717 [0] NCCL INFO Using network IB

Environment
Current environment
Lightning Component (e.g. Trainer, LightningModule, LightningApp, LightningWork, LightningFlow):
pytorch-lightning         1.9.0
nvidia-cublas-cu11        <IP_ADDRESS>
nvidia-cuda-nvrtc-cu11    11.7.99
nvidia-cuda-runtime-cu11  11.7.99
nvidia-cudnn-cu11         <IP_ADDRESS>
torch                     1.13.1+cu116
Python 3.8.17
OS : based on docker image nvidia/cuda:11.6.2-base-ubuntu20.04
GPU : A100

ps: I modified two lines of lightning source code as mentioned in this issue https://github.com/Lightning-AI/lightning/issues/16236#issuecomment-1752249152
More info
No response

@Master-cai This doesn't look like an issue with Lightning. Here in the error message we can see that network access at the given address is not working:
RuntimeError:
NCCL error in: ../torch/csrc/distributed/c10d/ProcessGroupNCCL.cpp:1269, internal error, NCCL version 2.14.3
ncclInternalError: Internal check failed.
Last error:
Net : Connect to <IP_ADDRESS><61785> failed : Connection refused

It means that probably the port is not accessible between machines and needs to be opened.

My understanding is that the lightning will set the MASTER_ADDR of every node to localhost, but the kubernetes environment has set a default MASTER_ADDR when it starts and will be overwrite to localhost by lightning. I think it is not very reasonable as the MASTER_ADDR should be the same among all nodes rather than the localhost of themself. The same applies to MASTER_PORT.

Where did you read this? This is not correct. Lightning won't set the address if it is already set by the environment. You can see in your logs that the address is torch.distributed is trying to connec to is <IP_ADDRESS>:40725 on one machine, but <IP_ADDRESS>:61785` on another.
Please make sure that MASTER_ADDR AND MASTER_PORT correspond to the master node with an open TCP port, and all machines should have the same env variable set to the same address and port.


renderToNodeList not exported in react-fela/index.d.ts

Type: Bug / Feature / Infrastructure / Docs / Question

Packages
Version

react-fela
11.4.0

Description
When I do
import {renderToNodeList} from 'react-fela'

in my typescript project, typecheck fails and says renderToNodeList is not exported.
I think it's because the export is missing in the typescript interface:
https://github.com/robinweser/fela/blob/master/packages/react-fela/index.d.ts

Feel free to go ahead and quickly add it to the definitions :)
Should be a simple one.


Use emplacement operations to avoid unnecessary copies/moves

Description
This PR changes some calls to insert and push_back to the corresponding emplacement operations to avoid unnecessary copy-constructor/move-constructor calls when adding elements to certain containers. The behavior should be unchanged.
Tasks

[ ] Tested on Linux
[x] Tested on Windows
[ ] Tested on macOS
[ ] Tested on iOS
[ ] Tested on Android

How to test this PR?
CI.

Small request: is it possible to leave PRs open for a short while, to give people (not just me) the chance to review them? For non-overlapping PRs in particular, it doesn't matter if they're open for a few days. Not everyone has the option to check multiple times per day, especially during workdays 😉
On topic: are IDEs meanwhile powerful enough to resolve the full type inference when emplacing? I.e. can they show type errors directly for the emplace() arguments?

Small request: is it possible to leave PRs open for a short while, to give people (not just me) the chance to review them? For non-overlapping PRs in particular, it doesn't matter if they're open for a few days. Not everyone has the option to check multiple times per day, especially during workdays 😉

Sure, I am personally not in a hurry. You should add yourself as a reviewer if you want to go over this particular one.

On topic: are IDEs meanwhile powerful enough to resolve the full type inference when emplacing? I.e. can they show type errors directly for the emplace() arguments?

Not really, as far as I know. Maybe CLion could be clever enough, I've seen it do quite nice stuff with type inference.


xref pulls the first met keydef keyword

With this map
<map>
    <title>Training Courses</title>
    <topicref href="get-started.dita" keyscope="gadget" keys="get-started">
        <keydef keys="valid-thru">
            <topicmeta>
                <keywords>
                    <keyword>2020</keyword>
                </keywords>
            </topicmeta>
        </keydef>
        <keydef keys="module-name">
            <topicmeta>
                <keywords>
                    <keyword>Gadget</keyword>
                </keywords>
            </topicmeta>
        </keydef>
    </topicref>
    <topicref href="get-started.dita" keyscope="widget" keys="get-started">
        <keydef keys="valid-thru">
            <topicmeta>
                <keywords>
                    <keyword>2019</keyword>
                </keywords>
            </topicmeta>
        </keydef>
        <keydef keys="module-name">
            <topicmeta>
                <keywords>
                    <keyword>Widget</keyword>
                </keywords>
            </topicmeta>
        </keydef>
    </topicref>
    <topicref href="further-reading.dita"/>
</map>

and get-started.dita
<topic id="topic_kjb_wy2_jw">
    <title>Getting Started with <ph keyref="module-name"/></title>

Product is valid through <ph keyref="valid-thru"/>

</topic>

and further-reading.dita
<topic id="topic_wm5_rz2_jw">
    <title>Further Reading</title>

Before getting into details, see <xref keyref="gadget.get-started"/> and <xref
                keyref="widget.get-started"/>

</topic>

I expected the XHTML publishing to generate this text:
Before getting into details, see Getting Started with Gadget and Getting Started with Widget

But DITA-OT 2.2.3 generates this:
Before getting into details, see 2020 and 2019

Is that really working as intended?

The result does seem wrong.
Given this:
    <topicref href="get-started.dita" keyscope="widget" keys="get-started">
        <keydef keys="valid-thru">

I do wonder if we're' getting into a circular condition here:

To establish the (link)text associated with the key widget.get-started, we have to pull the title from get-started.dita
To figure out the title for get-started.dita, we have to resolve the key module-name in the current scope
To resolve the key module-name in the current scope, we need to figure out the (link)text associated with keys

Again, I think the current output is confusing / broken, just noting the difficulty of processing this to get something better.

Even more confusing than the 1st map is this 2nd map. It incorporates the 1st map.
<map>
 <title>Product Line</title>
    <topicref href="portfolio.dita" keys="portfolio">
        <mapref href="[path to the 1st map]"/>
    </topicref>
    <topicref href="price-list.dita"/>
</map>

portfolio.dita
<topic id="topic_chc_sv3_jw">
  <title>Portfolio</title>
</topic>

price-list.dita
<topic id="topic_bq4_yv3_jw">
  <title>Price List</title>

Our <xref keyref="portfolio"/> price list

</topic>

Publishing the 2nd map, I'd expect price-list.dita's body to be
Our Portfolio price list

but OT 2.2.3 emits
Our 2020 price list

Reproduced on the latest development version. (dita-ot-2.3.1+3142690)
Attaching the files: test.gz

Reproduced on the latest devlop clone as of this writing, using the test files attached previously.

Link text for links to topics cannot be fully determined until all key definitions have been established (that is, the key spaces constructed and any key-to-text bindings resolved). Once that has been done then the link text for links to topics should be unambiguous (because all key-related information is now fully known).

With the latest development version (dita-ot-3.6.0-SNAPSHOT@3b50023), the result is the same as https://github.com/dita-ot/dita-ot/issues/2421#issuecomment-453284481.

Same behavior occurs with DITA-OT 4.0.1.


unknown np2 server option

hi,

is "-f" supported? above code is in tests/np_test.c
btw, "-v3" is also not supported, the max is "-v2", right?

Yes, -f is supported, look at netopeer2-server -h. And -v3 seems to be unsupported verbosity but it is normalized to -v2 so it makes no difference.

ah, yes, "-f" supported in latest code.


portainer add-on with default script

Hi
Portainer is a free, open source toolset for managing containerized applications. It works with Kubernetes, Docker, Docker Swarm, Azure ACI in both data centres and at the edge.
Portainer removes the complexity associated with orchestrators so developers can deploy apps quickly, easily and accurately. In addition to managing apps, it can be used to observe the behavior of containers, manage the underlying platform and provide the security and governance necessary for organization to deploy containers widely.
This add-on service will be enabled on nodeport 30777 and internally runs on 9000 port.
As part installation this add-on should be installed by default. document about portainer https://docs.portainer.io/v/ce-2.9/
pl guide me adding this add-on

Portainer is a very nice application, but we feel the DevOps Stack already provides the necessary features for this. In particular, the ArgoCD interface allows to view all the resources managed by the stack. You can also use non-invasive, client-side tools such as Lens or K9s for the same purpose.
If you really want to use Portainer with the DevOps Stack, you can pass it as an extra application to the stack for ArgoCD to manage it.


SPM suppport

Description
Added Package.swift file
Motivation and Context
integrate this nice piece of code using SPM
How Has This Been Tested?
building it with xed . in repo folder to open the Package.swift "pseudo-project" with in Xcode
Screenshots (if appropriate):
Types of changes

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

Checklist:

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

related to https://github.com/maxep/MXParallaxHeader/issues/116

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

:/

One more step in order to have an Xcode project find MXParallaxHeader to be found : it needs a new tag, maybe 1.1.0 should be chosen

Any plan to tag/release this @maxep ?
We now have to target the master branch to use SPM, would be nicer using a release !

Spoke too fast, using the master branch results in missing headers for Swift projects
Umbrella directory '~/Library/Developer/Xcode/DerivedData/Template-fhwdlwrvaxparjdwvnikcfyarzff/SourcePackages/checkouts/MXParallaxHeader/Sources/include' not found

Just made a second PR to fix this : https://github.com/maxep/MXParallaxHeader/pull/122


add configurable action that is published on close

There should be a feature close.action to optionally configure the name of an action, that is published every time the layer was closed (i.e. after all animations are completed).

Implemented on release-1.3.x (v1.3.0) and master (v1.4.0)


feat(dashboard-page): apply dashboard-list type & create pagination

To Reviewers

[ ] Skip (style, chore ONLY)
[ ] Not that difficult

Type of Change

[x] New feature
[ ] Bug fixes
[ ] Feature improvement
[ ] Refactor
[ ] Others (performance improvement, refactoring, CI/CD, etc.)

Checklist

[ ] Error / Warning / Lint / Type

Description
Important

Add dashboard type in DashboardBoardList for branching Project and Domain.
Add pagination in DashboardBoardList.
Apply delete API.

Not that difficult

Add more mock data.
Other some changes.

Thank you VERY much for the detailed review.
All of your reviews have been reflected!
Thank you again!!!


The NodeInfoView is missing

It seems that this feature is only working in the old UI. If I understand it right you need ember to get this.
Hope that I find some time for the ReactUI stuff :)

Solved with #075143a6ba76a87b7a842a3a297baa60722194b6


Need to add a TaskName attribute to the ScheduleRequest

Currently when an app is launched by the impl the task will use the default task name (application-1).  The user may want to set this when scheduling a task.

I can not repeat this behavior.  When testing on an IMPL it works as designed.  I see the name associated with the task as it was passed via the properties.


Retrieval Augmented Generation (RAG) implementation

Identify and integrate a suitable data source (e.g., CSV file, text corpus) for RAG
Implement a basic RAG process using libraries like LangChain or similar tools
One option, provide a Jupyter notebook demonstrating the RAG process (easier option)
Ideally, integrate RAG directly into the application (e.g., via Streamlit)
Consider these repos: https://github.com/NirDiamant/RAG_Techniques/tree/main

Merged pull request #10.

Data source is synthetic data from Anthropic.
RAG developed using OpenAI and FAISS.
Streamlit integration will come in a future enchancement


Add configurable timeout and pollingInterval for Hardhat

Addresses https://github.com/OpenZeppelin/openzeppelin-upgrades/issues/55

Adds timeout and pollingInterval options for Hardhat. timeout value of 0 means poll indefinitely until the transaction confirms.

Not needed for Truffle since Truffle makes use of network options for timeout and polling

Looks good. Thanks!


Bug in version v-0.20.5 : Metric names having capital letter like "GMV" is not rendering and throwing error

Make sure these boxes are checked before submitting your issue - thank you!

[Y ] I have checked the superset logs for python stacktraces and included it here as text if any
[ Y] I have reproduced the issue with at least the latest released version of superset
[ Y] I have checked the issue tracker for the same issue and I haven't found one similar

Superset version
V-0.20.5
It is proving to be prod-blocker for us.
Expected results
Slices should render and plot graphs
Actual results
Following error is shown whenever we try to plot any graph (such as time series or pie chart) with metric name in capital letterl (such as "Take Rate", "GMV" etc.)
Traceback (most recent call last):
File "/home/ubuntu/sherlock/lib/python3.5/site-packages/superset/viz.py", line 277, in get_payload
data = self.get_data(df)
File "/home/ubuntu/sherlock/lib/python3.5/site-packages/superset/viz.py", line 1036, in get_data
df = self.process_data(df)
File "/home/ubuntu/sherlock/lib/python3.5/site-packages/superset/viz.py", line 972, in process_data
values=fd.get('metrics'))
File "/home/ubuntu/sherlock/lib/python3.5/site-packages/pandas/core/reshape/pivot.py", line 112, in pivot_table
raise KeyError(i)
KeyError: 'L2A'
Steps to reproduce
Just change name of metric in capital letter.

What database is this?

Underlying database where data is stored is redshift.
And backend of SuperSet is Postgersql.

@xrmx did I answer your question?
What could be issue here?

i have the same issue. The workaround Works correctly tho
db: Oracle 10g

@vmoranvargas what workaround you have applied here?


Holiday Mode

I'm using the app with default settings.
Right now I'm on holiday and my car is parked on long term parking. In a week the battery drained approx. 100km (almost 33%)
Should be nice to have a holiday mode to suspend the data collection.
Also an in app warning would be appropriate, since it really draines the battery and so limiting your Tesla's range.

Oh thats bad draining. Which model do you own?
You can change the polling settings on the device settings screen. Will that resolve your problem for now?

I'm driving a 2018 Model S 75D
I dissabled the polling yesterday to get the root cause of the drain (and to assure I can drive back home after our holiday)
Will think about a good polling interval the next couple of days, since I got plenty of time 😉😂
Thanks for the suggestion.

Have you give this some thoughts yet? Would flow cards to enable/disable polling help? With flow action cards you can make your own 'i'm on holiday' so disable draining settings'.


Final Frames!

Slid in a fix for Cheapen hitting X-costs into this.  It simply can't now, but if I remember there's some weird thing where "card is free" + "x cost" = "play for free at current X"?  Would someone who knows about that expect Cheapen hitting an X-cost to behave that way?  If so, might need some assistance knowing the right fields to modify.


Implement Fragment API

React added the concept of Fragments, which are elements with multiple children that render into their parent.
I think this would be useful to avoid having junk Roblox instances in the tree, and help flatten some of the gross patterns that have developed with regards to nesting.

Anyone is willing to take a stab at this, but it'll probably depend on #48.


refactor get reward info

Codecov Report
Patch coverage: 97.72% and project coverage change: -1.66% :warning:

Comparison is base (c73ec88) 87.13% compared to head (2f49df8) 85.48%.
Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #12      +/-   ##
==========================================
- Coverage   87.13%   85.48%   -1.66%
==========================================
  Files           4        4
  Lines         342      310      -32
==========================================
- Hits          298      265      -33
- Misses         44       45       +1

Files Changed
Coverage Δ

contracts/halo-farm/src/state.rs
100.00% <ø> (ø)

contracts/halo-farm/src/formulas.rs
90.00% <85.71%> (-10.00%)
:arrow_down:

contracts/halo-farm/src/contract.rs
88.20% <98.64%> (-1.53%)
:arrow_down:

contracts/halo-farm-factory/src/contract.rs
74.62% <100.00%> (ø)

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


No matching version found for<EMAIL_ADDRESS>

🐞 Bug report
Installing @angular/cli@latest leads to following error message
No matching version found for<EMAIL_ADDRESS>It was specified as a dependency of '@angular/cli'

It looks like devkit/core was not published to npm as npm view @angular-devkit/core  shows 9.1.1 as latest dist tag
Command (mark with an x)
npm i -g @angular/cli@latest (9.1.2)

It was an issue with our private registry - no issues on angular side :)


list() function changes

Hi @tpunt
I was trying out the examples for the list() function and I was getting different results for
$a = [1, 2];
list($a, $b) = $a;

// PHP <7 : $a = 1, $b = 2
// PHP 7: $a = 1, $b = null + "Undefined index 1"

$b = [1, 2];
list($a, $b) = $b;

// PHP <7 : $a = null + "Undefined index 0", $b = 2
// PHP 7: $a = 1, $b = 2

In my case I got
$a = [1, 2];
list($a, $b) = $a;

// PHP <7 : $a = 1, $b = 2
// PHP 7: $a = 1, $b = 2

$b = [1, 2];
list($a, $b) = $b;

// PHP <7: $a = null, $b = 2
// PHP 7: $a = 1, $b = 2

After checking the php versions and trying to understand why I got those different values, I found that in the documentation it's stated Modification of the array during list() execution (e.g. using list($a, $b) = $b) results in undefined behavior.
Given this discrepancies I've open a bug on php.net and they are looking into it. I've also done some modifications and I will open a PR as soon as the matter is cleared :)

Nice catch @p16. I'll wait for your PR to rectify this.


Remove version of dependencies from BIR

Description:
When two compatible versions for a dependency exist in the dependency graph, Project API resolves it to the latest version. During compilation, if the dependencies are resolved from the BIR from the initial run, this scenario fails at the BIR loading phase if the BIR is built with the old version of the respective dependency.
Affected Versions
SL Beta1

Steps to reproduce
The issue can be reproduced as follows with SL Beta1 distribution.

Create a newer compatible version of http (e.g.: 1.1.0-beta-1.1), build and publish it to local repo using bal push --repository=local. Now the distribution should contain http 1.1.0-beta.1 and the local repo should have 1.1.0-beta.1.1
Create a new package with bal new command and do the following changes:

main.bal
import ballerina/websub as _;
import ballerina/http as _;

Dependencies.toml
[[dependency]]
org = "ballerina"
name = "websub"
version = "1.2.0-beta.1"

[[dependency]]
org = "ballerina"
name = "http"
version = "1.1.0-beta.1.1"
repository = "local"

Now, build the project using bal build

Output:
Compiling source
	asmaj/testPkg:0.1.0
error: failed to load the module 'ballerina/websub:1.2.0-beta.1' from its BIR

This happens because Project API resolves to the latest http version 1.1.0-beta.1.1 whereas the BIR of the websub module is built with the http 1.1.0-beta.1 version and when loading dependencies, it looks for the symbols by the older version.


Syntax highlighting is wrong when putting SQL queries in PHP strings

Prerequisites

[ X] Put an X between the brackets on this line if you have done all of the following:

Reproduced the problem in Safe Mode: https://flight-manual.atom.io/hacking-atom/sections/debugging/#using-safe-mode
Followed all applicable steps in the debugging guide: https://flight-manual.atom.io/hacking-atom/sections/debugging/
Checked the FAQs on the message board for common solutions: https://discuss.atom.io/c/faq
Checked that your issue isn't already filed: https://github.com/issues?utf8=✓&q=is%3Aissue+user%3Aatom
Checked that there is not already an Atom package that provides the described functionality: https://atom.io/packages

Description
Syntax highlighting is wrong when putting SQL queries in PHP strings.
Steps to Reproduce

Write: $myvar = "AND (";
Write any code after that string
Enjoy wrong highlighting

Expected behavior: Right highlighting
Actual behavior: Everything is highlighted as a string
Reproduces how often: 100%
Versions
Tried on latest version: 1.35.1 x64
Additional Information
Any additional information, configuration or data that might be necessary to reproduce the issue.

Thanks for contributing!
We noticed that this looks like a duplicate of https://github.com/atom/language-php/issues/333 so you can subscribe there for updates if you'd like.

Because we treat our issues list as the Atom team's backlog, we close duplicates to focus our work and not have to touch the same chunk of code for the same reason multiple times. This is also why we may mark something as duplicate that isn't an exact duplicate but is closely related.
For information on how to use GitHub's search feature to find out if something is a duplicate before filing, see the How Can I Contribute? section of the Atom CONTRIBUTING guide.


Casting a web page

There has been few requests to cast a webpage.
This would probably need to build custom receiver which you can control to open web page

Would be great to have the possibility to send a web page.

This could be achieved rather simply
Someone could develop a single custom receiver that holds nothing more than a 100% height, 100% width iFrame. A call could set the SRC URL of the iFrame.
This application could be reused by anyone. it raises some questions though about who will host this and privacy concerns around a third party hosting it.
The alternative would be to build and host your own custom receiver application, it's dead simple for this scenario.

I think so too.
I started my receiver application here:
https://github.com/tapanila/SharpCaster-Receiver
I also published it with application id:
FE058D36
It is hosted at
https://sharpcaster.azurewebsites.net/
Currently having problems on transferring data to the website. I think the SDK does not support simple types (just string) and only supports json. So have to make the message format object based instead of string based.
Then need to add the logic to load a web page and send a scroll events. Also some kind of refresh option

you don't need to create custom application and iframe. there is already receiver application available called URL Cast Receiver and its application id is : 5CB45E5A

Thanks to @kalpesh2804 this issue is now solved.
Sample how to do this is here:
https://github.com/Tapanila/SharpCaster/blob/dev/SharpCaster.Test/WebPageCastingTester.cs


Make DF recreation retry when client error

When reading DF events fails with client error 400, the API call is doing a retry by recreating the DF, the DF recreation API call unfortunately does not have a retry on client error.
This commit fixed this bug.
Description
Closes #[ISSUE NUMBER]
Please put here the intent of your pull request.
Dependencies
List the other pull requests that should be merged before/along this one.
Checklist

[ ] Referenced an issue in the PR title or description
[ ] Filled properly the description and dependencies, if any
[ ] Unit/Integration tests updated or added
[ ] Javadoc added or updated
[ ] Updated the documentation in docs folder

LGTM, we need to support the same in Python BDK.


caseValueParam field is required for taskType: DECISION

After upgrading from 1.x to 2.7.1 we have discovered an issue with DECISION tasks that use a caseExpression instead of caseValueParam and getting the error caseValueParam field is required for taskType: DECISION when trying to create and or update a definition. There was some validation added in that has been really great but this is the last error it's giving and it doesn't seem to fit with how we've used this DECISION task in before.
It was my impression that you could use caseExpression instead of caseValueParam.
Example of very basic definition showing the issue
[
    {
        "name": "case-expression-usage",
        "description": "example of caseExpression being used in a decision",
        "version": 1,
        "schemaVersion": 2,
        "tasks": [
            {
                "name": "basic_task",
                "taskReferenceName": "checkExpression",
                "type": "DECISION",
                "inputParameters": {
                    "valueCheck": "some_value"
                },
                "caseExpression": "$.valueCheck == null ? 'true': 'false'",
                "decisionCases": {
                    "true": [
                        {
                            "type": "SIMPLE",
                            "name": "basic_task",
                            "taskReferenceName": "was_null"
                        }
                    ]
                },
                "defaultCase": [
                    {
                        "type": "SIMPLE",
                        "name": "basic_task",
                        "taskReferenceName": "was_not_null"
                    }
                ]
            }
        ]
    }
]

{"status":400,"message":"Validation failed, check below errors for detail.","retryable":false,"validationErrors":[{"path":"updateWorkflowDef.arg0[0].tasks[0]","message":"caseValueParam field is required for taskType: DECISION taskName: basic_task"}]}

@falu2010-netflix Would you mind taking a look please.

#997


add module version of rtcstats

adds a version of rtcstats which is exported as a module.
Most importantly this allows me to insert it after adapter.js
so that my Edge shim can be monitored.
Still haven't figured out a good way how to maintain both versions.

@fippo The tests have been failing for the longest time. I think you mentioned the old version of the adapter in the past. Do you have a way to fix it ?

I think this is close to the right way to do it. But we need a build process to generate rtcstats.js from this version.

updated. The only thing missing now is allowing a way to configure which getUserMedia objects to wrap.

so the same code that passed three days ago is now generating errors on travis... :-/

@aullman FYI looks like selenium-webdriver needs to be pinned to 3.3.0 as well (might be incompability between newer versions and the pinned geckodriver)
😕

squashing...


reference issues in getting started doc for installing envoy using ge…

Commit Message: Reference url issues in getting started doc
Additional Description: NA
Risk Level: LOW
Testing: Unit
Docs Changes: Yes
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Deprecated:]
[Optional API Considerations:]

/retest

i also checked the links for the repos themselves and it appears they have also been merged, could you fix those links too ?
@phlax Here, which links are you reffering here ?

also - im thinking that given the instructions are now the same for rhel and centos can we collapse these two sets of instructions to one
@phlax agree. i will do it

which links are you reffering here ?

the line like:
$ sudo yum-config-manager --add-repo https://getenvoy.io/linux/rhel/tetrate-getenvoy.repo

@phlax Ok. Thanks.. i will do it.
Just quick thing, i am thinking to do it in different PR, please suggest ?

Just quick thing, i am thinking to do it in different PR, please suggest ?

i think lets do it all here - im happy to review it and i think all of these issues stem from the same objective - ie updating the redhat install instructions to use the latest getenvoy

@phlax Sure

@phlax please suggest on current build failure.

/retest

rendered docs are here https://storage.googleapis.com/envoy-pr/16630/docs/index.html

@phlax please suggest on current build failure.

unrelated windows flake i think

rendered docs are here https://storage.googleapis.com/envoy-pr/16630/docs/index.html

@phlax please suggest on current build failure.

unrelated windows flake i think

Ok. Thanks for quick review @phlax

@phlax just thinking, anything more need to be done to get it merged in main stream ?

all good @ankatare - thanks - merged

@phlax thanks a lot :)


[ISSUE] Visa regex is incorrect.

The validation regex for Visa is incorrect. Visa card numbers start with 4, and that seems to be the only rule. There are no limitation on the second digit.
https://baymard.com/checkout-usability/credit-card-patterns
This is a showstopper for us, as even several of our own developers have Visa cards that do not pass the current validation regex.
PR here: https://github.com/verygoodsecurity/vgs-collect-ios/pull/153

@DagAgren thanks for your PR. The fix released in v1.5.3.


Share dialog teams integration 8.0

 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.Olga Rueckert 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.


UofT-DSI | Sampling - Assignment 3

What changes are you trying to make? (e.g. Adding or removing code, refactoring existing code, adding reports)

I am making refinements to the original survey design based on class content discussions on survey quality and potential errors in sampling.

What did you learn from the changes you have made?

From these changes, I learned the importance of precise wording in survey questions to minimize response bias and ensure the quality and relevance of collected data.

Was there another approach you were thinking about making? If so, what approach(es) were you thinking of?

Initially, I considered implementing more extensive demographic stratification in the survey design to better capture diverse voter demographics across regions. However, I prioritized clarity in question formulation as a foundational improvement before further stratification.

Were there any challenges? If so, what issue(s) did you face? How did you overcome it?

A challenge was balancing between specificity and brevity in question revision. Ensuring that questions were clear and concise while still capturing essential information required careful wording adjustments.

How were these changes tested?

The changes were tested by reviewing several existing surveys that focus on similar topics.

A reference to a related issue in your repository (if applicable)

N/A

Checklist

[x] I can confirm that my changes are working as intended

A3 complete. Well done, in particular I was impressed with your answer to Q1.

Thank you, Jarlathbr.


changed contentEncoding for messages to be standard compliant

as per
http://www.iana.org/assignments/character-sets/character-sets.xhtml
utf8 does not exist.
when publishing messages as JSON-RPC a proper implemented server rejects the message as invalid due the invalid content encoding
const hrTime = process.hrtime();
rabbot.publish( "service.rpc", {
    type: "ping",
    contentType: "application/json",
    body: { ping: hrTime[0] * 1000 + hrTime[1] / 1000000 },
    headers: {
      jsonrpc: "2.0",
    },
}

Coverage increased (+0.02%) to 88.977% when pulling 1d0b23d459aedcf33fb530c20b771eb54f4c9be8 on schubter:content-encoding into 80b63c08bdb32d04fd36502f24180e0c8e8f69ee on arobson:master.


Single user not found

[Enter feedback here]
Attempting to add/remove users (call answering users) using the PowerShell cmdlet Set-CsCallQueue
To add a new user, this works
$o = Get-CsCallQueue -Identity "d2187857-4011-41b2-b44a-2019ee167526" | Select Users
$o.Users.Add("A111DDA8-DCFC-4905-B26B-7E5EC44E9C7F")
Set-CsCallQueue -Identity "d2187857-4011-41b2-b44a-2019ee167526" -Users $o.Users
Subsequently, to remove a user, when I execute this script, I would expect it to work (if I do not cast my string as a GUID, it does not remove the item from the array)
$o = Get-CsCallQueue -Identity "d2187857-4011-41b2-b44a-2019ee167526" | Select Users
$o.Users.Remove([GUID]"8a67db3e-46e4-4c76-8b46-720854b6d5b8")
Set-CsCallQueue -Identity "d2187857-4011-41b2-b44a-2019ee167526" -Users $o.Users
I get the following error
Single user not found (9ff75dfd-dfd9-4a1d-b75b-205f26cc3303).
+ CategoryInfo          : NotSpecified: (:) [Set-CsCallQueue], OnlineCmdletException
+ FullyQualifiedErrorId : Unspecified,Microsoft.Rtc.Management.Hosted.CallQueue.SetCsCallQueueCmdlet
+ PSComputerName        : api.interfaces.records.teams.microsoft.com
9ff75dfd-dfd9-4a1d-b75b-205f26cc3303 is a user in $o.Users
$o.Users
Guid
b66e8544-a3c3-415f-acf3-dc5b29eacccc
a111dda8-dcfc-4905-b26b-7e5ec44e9c7f
9ff75dfd-dfd9-4a1d-b75b-205f26cc3303
281c1487-8a2c-4ed0-952b-09c0615da6ca
Each of the elements in the array is a GUID
$o.Users[0].GetType()
IsPublic IsSerial Name                                     BaseType

True     True     Guid                                     System.ValueType

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

ID: 86ade68d-5dc0-c598-a659-8b018cef3e26
Version Independent ID: 67efa086-edb0-2828-cc14-4c245290e4f3
Content: Set-CsCallQueue (SkypeForBusiness)
Content Source: skype/skype-ps/skype/Set-CsCallQueue.md
Service: skypeforbusiness-powershell
GitHub Login: @tomkau
Microsoft Alias: tomkau

The error is valid.  At some point, that user's Enterprise voice enabled status changed, thus, they are not eligible to be a call answer user

Hello @sarollins How are you? Before I try this, if I understood well based on your last message, you found out the cause of the error being the user is not EV enabled? Thank you for your clarification.

Correct.  Not an issue with the PowerShell cmdlet
Thank you
Sent from my iPhone
On Apr 27, 2021, at 8:10 AM, Andres Mariano Gorzelany @.***> wrote:

Hello @sarollinshttps://github.com/sarollins How are you? Before I try this, if I understood well based on your last message, you found out the cause of the error being the user is not EV enabled? Thank you for your clarification.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/office-docs-powershell/issues/7621#issuecomment-827594650, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AKYFAJN5DFBPHUBHCZ4EQC3TK2ZVTANCNFSM43AEMQYA.

Thanks for returning and clarifying that.
@yogkumgit please kindly close.

@get-itips Thank you very much for the contribution and sharing this explanation. @sarollins Hope this comment is helpful for you. If you see a documentation update is required, please feel free to open an issue for the same. We proceed here to close it. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


Initial GitHub Action to build all artifacts

Description
This PR adds a new GitHub Action for building all artifacts. This PR currently performs the release builds and keeps all artifacts locally. It is a starting point for further work.
This action requires a user manually start it. It is not triggered automatically.
Issues Resolved
This is only part of #977.
Check List

[x] New functionality includes testing.
[x] New functionality has been documented.

[x] New functionality has javadoc added

[x] Commits are signed with a real name per the DCO

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@dinujoh , To clarify on the documentation: This release process isn't yet ready. This is just the first commit which shows how the artifacts and Docker image are built locally. I plan to add additional steps to this action to upload these to the staging repositories. This commit sets the baseline job.
If you are curious, you can see the jobs on my fork: https://github.com/dlvenable/data-prepper/actions/workflows/release.yml
(The failed builds are from some experiments which I'm postponing.)


Implement pagination support for sqlalchemy queries exposed by the Flask server

Some queries may yield over 5000 entries which generates overhead when retrieving them from database and sending in the HTTP response. There should be support for requesting only a select range of the request results.

UI Bootsrap's pagination directive requires total number of items to work properly, so the paginated method(s) should be extended  to include it in payload.


[BetterTouchTool]...Error: Command was killed with null

Extension
https://raycast.com/dnnsmnstrr/bettertouchtool
Description
Error:
Error: Command was killed with null: osascript -l JavaScript -e function run(argv) {
    let BetterTouchTool = Application('BetterTouchTool');
    return BetterTouchTool.get_triggers({trigger_id: 643 });
  } run();
execution error: Error: Error: 信息无法识别。 (-1708)
    at s4 (/Users/ttym/.config/raycast/extensions/bb1e5e21-0827-4c47-88af-ab498770f9b0/trigger.js:12:57)
    at Object.o4 [as current] (/Users/ttym/.config/raycast/extensions/bb1e5e21-0827-4c47-88af-ab498770f9b0/trigger.js:12:310)
    at Object.current (/Users/ttym/.config/raycast/extensions/bb1e5e21-0827-4c47-88af-ab498770f9b0/trigger.js:12:1115)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)

Steps To Reproduce
No response
Current Behaviour
No response
Expected Behaviour
No response

I'll have a look at this, it might be related to the problems in https://github.com/dnnsmnstrr/alfred-btt/issues/2

Possible fix: https://github.com/raycast/extensions/pull/6458

Hopefully this problem is resolved, let's close the issue for now...
@raycastbot close this issue


Geonames update - file exists check should be dir exists check

It seems commit https://github.com/michaeldrennen/Geonames/commit/1caf30daedfece12f8b85467b4697f1e9a6e27b2#diff-4ff216f810727b4379e6b093f6d5c5a7 is breaking the geonames:update command.
The lines added here check whether the file exists and is writeable, before doing a write to this file.
IMO, this should be a directory check, not a check on the file.
Is this correct?
Error stack trace:
/www # php artisan geonames:update --connection=mysql
The database connection name was set to: mysql
Testing database connection...
Confirmed database connection set up correctly.
Starting geonames:update-geonames
    {--connection= : If you want to specify the name of the database connection you want used.}
Downloading the modifications file from geonames.org
Downloaded http://download.geonames.org/export/dump/modifications-2019-06-28.txt

   Exception  : You need to create a directory at: /www/storage/geonames/modifications-2019-06-28.txt

  at /www/vendor/michaeldrennen/geonames/src/Console/UpdateGeonames.php:317
    313|         $localFilePath = GeoSetting::getAbsoluteLocalStoragePath( $this->connectionName ) . DIRECTORY_SEPARATOR . $this->modificationsTxtFileName;
    314|
    315|         // Determine if the local storage path exists and is writable.
    316|         if ( FALSE === file_exists( $localFilePath ) ):
  > 317|             throw new \Exception( "You need to create a directory at: " . $localFilePath );
    318|         endif;
    319|
    320|         if ( FALSE === is_writable( $localFilePath ) ):
    321|             throw new \Exception( "This directory needs to be writable: " . $localFilePath );

  Exception trace:

  1   MichaelDrennen\Geonames\Console\UpdateGeonames::saveRemoteModificationsFile()
      /www/vendor/michaeldrennen/geonames/src/Console/UpdateGeonames.php:138

  2   MichaelDrennen\Geonames\Console\UpdateGeonames::handle()
      /www/vendor/laravel/framework/src/Illuminate/Container/BoundMethod.php:32

  Please use the argument -v to see more details.

https://www.php.net/manual/en/function.file-exists.php
file_exists
(PHP 4, PHP 5, PHP 7)
file_exists — Checks whether a file or directory exists
So that method is OK for this purpose.
You just need to create the directory ahead of time.
Should be good to go after that.


Jest issue

Here is how I implemented Recaptcha in my app
import React from 'react';
import RecaptchaV3 from 'react-native-recaptcha-that-works';
import {RECAPTCHA_SECRET, RECAPTCHA_DOMAIN} from '@env';

const Recaptcha = React.forwardRef(({onSuccess}, ref) => {
  const onVerifyRecaptcha = token => {
    onSuccess(token);
  };

  const onExpire = () => {
    console.warn('expired!');
  };
  return (
    <RecaptchaV3
      ref={ref}
      siteKey={RECAPTCHA_SECRET}
      baseUrl={RECAPTCHA_DOMAIN}
      onVerify={onVerifyRecaptcha}
      onExpire={onExpire}
      size="normal"
    />
  );
});

export default Recaptcha;

when I run npm test, I get the following error
React.createElement: type is invalid -- expected a string (for built-in components) or a class/function (for composite components) but got: object. You likely forgot to export your component from the file it's defined in, or you might have mixed up default and named imports.
Anyone was able to solve this issue or have a mock file for the package.

What version of React Native?0.66.1
What version of React Native WebView? 11.2.2
What version of the library?1.2.0
iOS/Android version?
Did the problem happen after updating React Native? no
Are you using the library for the first time? yes
It's a bug? Provide a link to a minimal reproduction case.

following up on this, I managed to pass the error by creating this mock file react-native-recaptcha-that-works.js inside mocks folder
import React from 'react';
import {View} from 'react-native';
const Recaptcha = props => <View {...props} />;
export default Recaptcha;

I added some tests to the Sample project, but I don't need any mock to make it work.


UI freezing for a second on every save

This is what I see in the console -
[09:30:04] margo: pre-save timedout on view 19: <file>.go
[09:35:45] margo: pre-save timedout on view 19: <file>.go
[09:35:51] margo: pre-save timedout on view 19: <file>.go
[09:36:35] margo: pre-save timedout on view 19: <file>.go
[09:39:06] margo: pre-save timedout on view 19: <file>.go

I am on 18.03.26-1

Are you using golang.Goimports? does it happen if you switch to golang.GoFmt?
goimports is known to be slow sometimes. If gofmt works fine, then maybe the timeout just needs to be increased from 2.000 seconds or made configurable again.

Hmm .. seems to be working fine after a restart. I will keep an eye for it and report if it happens again.


Opam-Nix not providing the latest library versions from opam-repository

Describe the bug
I can see in opam-repository that bwd-2.2 and yuujinchou-5.0.2 are available:
https://github.com/ocaml/opam-repository/blob/master/packages/bwd/bwd.2.2.0/opam
https://github.com/ocaml/opam-repository/blob/master/packages/yuujinchou/yuujinchou.5.0.2/opam
However Opam-nix is giving me bwd-2.1 and yuujinchou-4.0.2.
To Reproduce
➜ nix shell github:tweag/opam-nix#bwd.latest

➜ echo $PATH
/nix/store/gk1f1i3hba3cgbxmpg9i144qbmszw9lc-bwd-2.1.0/bin:/run/wrappers/bin:/home/solomon/.nix-profile/bin:/etc/profiles/per-user/solomon/bin:/nix/var/nix/profiles/default/bin:/run/current-system/sw/bin:/home/solomon/.local/scripts:/home/solomon/.local/bin:/home/solomon/.cabal/bin:/home/solomon/.zsh/plugins/zsh-nix-shell

➜ nix shell github:tweag/opam-nix#yuujinchou.latest

➜ echo $PATH
/nix/store/9di530vv5hwafixg94gan84xgqimiwvx-yuujinchou-4.0.0/bin:/nix/store/gk1f1i3hba3cgbxmpg9i144qbmszw9lc-bwd-2.1.0/bin:/run/wrappers/bin:/home/solomon/.nix-profile/bin:/etc/profiles/per-user/solomon/bin:/nix/var/nix/profiles/default/bin:/run/current-system/sw/bin:/home/solomon/.local/scripts:/home/solomon/.local/bin:/home/solomon/.cabal/bin:/home/solomon/.zsh/plugins/zsh-nix-shell

➜ nix shell github:tweag/opam-nix#yuujinchou.5.0.2
error: flake 'github:tweag/opam-nix' does not provide attribute 'packages.x86_64-linux.yuujinchou.5.0.2', 'legacyPackages.x86_64-linux.yuujinchou.5.0.2' or 'yuujinchou.5.0.2'

Expected behavior
I would expect the latest version from opam-repository to be provided.
Environment

Nixos 23.05

Looks like the issue is that the opam-repository input to the flake is locked to an old commit. Is there a way for me to tell opam-nix to use a newer version of repository?

I solved this in my project by making opam-repository an input my project's flake and forcing opam-nix to follow my pinned version:
   inputs = {
+    opam-repository = {
+      url = "github:ocaml/opam-repository";
+      flake = false;
+    };
     opam-nix.url = "github:tweag/opam-nix";
+    opam-nix.inputs.opam-repository.follows = "opam-repository";
     flake-utils.url = "github:numtide/flake-utils";
     nixpkgs.follows = "opam-nix/nixpkgs";
   };
-  outputs = { self, flake-utils, opam-nix, nixpkgs }@inputs:
+  outputs = { self, flake-utils, opam-nix, opam-repository, nixpkgs }@inputs: