Skip to content

Conversation

@ybgao-nvidia
Copy link
Contributor

This PR provides feature parity with this PR in RL.

@copy-pr-bot
Copy link

copy-pr-bot bot commented Aug 26, 2025

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

@ffrujeri ffrujeri requested a review from a team as a code owner August 26, 2025 16:38
@ffrujeri ffrujeri force-pushed the nemo-rl-submodule branch 3 times, most recently from d99eb90 to 58ea771 Compare August 28, 2025 16:22
@ffrujeri ffrujeri force-pushed the ybgao/fix-model-checkpointing branch from 6508e44 to 73ac7cc Compare August 28, 2025 17:16
ffrujeri
ffrujeri previously approved these changes Aug 28, 2025
akoumpa
akoumpa previously approved these changes Aug 28, 2025
@ffrujeri ffrujeri changed the base branch from nemo-rl-submodule to main August 28, 2025 17:21
@ffrujeri ffrujeri dismissed stale reviews from akoumpa and themself August 28, 2025 17:21

The base branch was changed.

akoumpa
akoumpa previously approved these changes Aug 28, 2025
@ffrujeri ffrujeri enabled auto-merge (squash) August 28, 2025 17:21
@akoumpa
Copy link
Contributor

akoumpa commented Aug 29, 2025

/ok to test 73ac7cc

Signed-off-by: Felipe Vieira Frujeri <[email protected]>
Signed-off-by: Felipe Vieira Frujeri <[email protected]>
@ffrujeri
Copy link
Contributor

ffrujeri commented Sep 4, 2025

/ok to test 8c34b71

Copy link
Contributor

@akoumpa akoumpa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks.

@ffrujeri ffrujeri merged commit e4780fc into main Sep 4, 2025
42 checks passed
@ffrujeri ffrujeri deleted the ybgao/fix-model-checkpointing branch September 4, 2025 20:17
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.

4 participants