Skip to content

Rollout collection tutorial fixes#790

Merged
bxyu-nvidia merged 1 commit intomainfrom
cwing/rollout-collection-fixes
Mar 2, 2026
Merged

Rollout collection tutorial fixes#790
bxyu-nvidia merged 1 commit intomainfrom
cwing/rollout-collection-fixes

Conversation

@cwing-nvidia
Copy link
Contributor

Remove the "Where Do Reward Scores Come From?" note that implied custom verification logic is optional. Also fix tutorial goals to match actual content and correct the resource server name.

Fixes #776

…orial

Remove the "Where Do Reward Scores Come From?" note that incorrectly
stated the default reward is 1.0 and implied custom verification logic
is optional. Also fix tutorial goals to match actual content and
correct the resource server name.

Closes #776

Signed-off-by: Chris Wing <cwing@nvidia.com>
@bxyu-nvidia bxyu-nvidia merged commit 796f14b into main Mar 2, 2026
6 checks passed
@bxyu-nvidia bxyu-nvidia deleted the cwing/rollout-collection-fixes branch March 2, 2026 05:45
abubakaria56 pushed a commit to abubakaria56/Gym that referenced this pull request Mar 2, 2026
Remove the "Where Do Reward Scores Come From?" note that implied custom
verification logic is optional. Also fix tutorial goals to match actual
content and correct the resource server name.

Fixes NVIDIA-NeMo#776

Signed-off-by: Chris Wing <cwing@nvidia.com>
abubakaria56 pushed a commit to abubakaria56/Gym that referenced this pull request Mar 2, 2026
Remove the "Where Do Reward Scores Come From?" note that implied custom
verification logic is optional. Also fix tutorial goals to match actual
content and correct the resource server name.

Fixes NVIDIA-NeMo#776

Signed-off-by: Chris Wing <cwing@nvidia.com>
abubakaria56 pushed a commit to abubakaria56/Gym that referenced this pull request Mar 2, 2026
Remove the "Where Do Reward Scores Come From?" note that implied custom
verification logic is optional. Also fix tutorial goals to match actual
content and correct the resource server name.

Fixes NVIDIA-NeMo#776

Signed-off-by: Chris Wing <cwing@nvidia.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Reward scores note causing confusion in Rollout Collection tutorial

2 participants