Open
Conversation
Signed-off-by: Christian Munley <cmunley@nvidia.com>
Signed-off-by: Christian Munley <cmunley@nvidia.com>
- Moved the article to concepts (it's mostly conceptual in its current form) - Reformatted the content - Added crosslinks to/from doc please double check that i did not distort any meaning as part of this change --------- Signed-off-by: Lawrence Lane <llane@nvidia.com> Co-authored-by: cmunley1 <cmunley@nvidia.com>
Signed-off-by: Lawrence Lane <llane@nvidia.com>
Contributor
Author
|
draft for disable on-policy assertion in on-policy training NVIDIA-NeMo/RL#1840 |
bxyu-nvidia
requested changes
Jan 29, 2026
Signed-off-by: Christian Munley <cmunley@nvidia.com>
hwolff99
reviewed
Feb 12, 2026
Contributor
|
lgtm |
Signed-off-by: cmunley1 <cmunley@nvidia.com>
Signed-off-by: cmunley1 <cmunley@nvidia.com>
cmunley1
commented
Feb 16, 2026
|
|
||
| For models with a chat template that drops previous reasoning traces: modify the chat template to retain all thinking, or use the non-thinking model. | ||
|
|
||
| For agents with non-monotonic trajectoires, the asserts may need to be disabled. This is not currently supported, but can be experimented with. |
Contributor
Author
There was a problem hiding this comment.
@bxyu-nvidia not sure i like to say 'This is not currently supported, but can be experimented with.' but not sure what else
Contributor
Author
|
i trimmed this down to just a FAQ on monotonic trajectory |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
add faq on how gym and rl enforce monotonic trajectories and token id correction for on policy training