Use Gym’s exported server-type keys
Maintainers usually reply within 2 days
Nobody has claimed this yet.
Assessment
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Newbie friendliness
- 72/100
Research direction
Start with nemo_rl/environments/nemo_gym.py and inspect where GYM_SERVER_TYPE_KEYS is defined and used, keeping the Gym import inside the Gym actor runtime. Check the pinned Gym version for the exported nemo_gym.global_config.SERVER_TYPE_KEYS constant and whether it includes #4768. Done means RL uses that constant and server discovery plus the existing routing tests pass.
Written by the indexing model from the issue text.
Description
Follow-up to review of #4429.
Gym #4768 exports nemo_gym.global_config.SERVER_TYPE_KEYS. RL should use it so server-category discovery stays aligned with the pinned Gym version.
- Ensure RL’s Gym pin includes #4768 once it is merged.
- Replace the duplicated
GYM_SERVER_TYPE_KEYSinnemo_rl/environments/nemo_gym.pywith Gym’s exported constant, keeping the import inside the Gym actor runtime. - Verify server discovery and the existing routing tests pass.
- Dominant language
- Python
- Stars
- 2.1k
- Forks
- 599
- Avg merge
- 3d 12h
- Merged PRs (30d)
- 245
Getting set up
- No Dockerfile or Docker Compose file
- Has a pull request template
- Read the contributing guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from NVIDIA-NeMo/RL
-
r0.8.0: pyrefly rejects captured optional config in make_policy_factory, blocking CIPossibly taken A pull request linked to this issue is open or already merged. Openbug
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
NVIDIA-NeMo/RL#4599 ·
Maintainers usually reply within 2 days
-
Feature
Difficulty 2/5 Half a day Newbie friendliness 78/100
NVIDIA-NeMo/RL#4016 ·
Maintainers usually reply within 2 days
-
bug
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
NVIDIA-NeMo/RL#2948 ·
Maintainers usually reply within 2 days
-
accuracy bug
Difficulty 5/5 Over a week Newbie friendliness 28/100
NVIDIA-NeMo/RL#4777 ·
Maintainers usually reply within 2 days
-
PPO and xtoken off-policy distillation ignore checkpointing.pretrained_checkpointPossibly taken @dijor0310 claimed this 2 days ago. Openbug
Difficulty 3/5 1-2 days Newbie friendliness 68/100
NVIDIA-NeMo/RL#4768 · 1 comment ·
Maintainers usually reply within 2 days
Similar issues
-
Difficulty 1/5 Under an hour Newbie friendliness 80/100
521xueweihan/HelloGitHub#4063 ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
-
Difficulty 2/5 1-3 hours Newbie friendliness 62/100
Vanilla-OS/apx-gui#112 ·
-
Devstral and Voxtral: section 5 fails in us-east-1 with 400 "Model does not support audio modality"Open
Difficulty 2/5 1-3 hours Newbie friendliness 76/100
aws-samples/sample-per-model-bedrock#409 ·
Maintainers usually reply within 1 day
-
DuckDB 2.0 breaks to_pyarrow_batchesPossibly taken @DG47 claimed this today. Open
Difficulty 2/5 1-3 hours Newbie friendliness 76/100
ibis-project/ibis#12196 ·
Maintainers usually reply within 1 day