NVIDIA-NeMo / NVIDIA-NeMo/RL

Chore: More config cleanup

Open
#1,201 0 comments 0 reactions 0 assignees View on GitHub
enhancement testing UX
Dominant language
Python
Stars
2k
Forks
561
Avg merge
4d 5h
Merged PRs (30d)
145

Description

After https://github.com/NVIDIA-NeMo/RL/pull/1024 is merged, we should also move other top level configs ([ex](https://github.com/NVIDIA-NeMo/RL/tree/main/examples/configs)) matching `examples/configs/*.yaml` under `examples/configs/recipes/**/*.yaml` since we should only have a few exemplar configs at the top, and everything should be a configuration of these.

This will improve test coverage of these yamls since unit tests enforce that each recipe config also has an associated driver script.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.