Skip to content

Conversation

abukharin-nv
Copy link
Contributor

What does this PR do ?

Add a one line overview of what this PR aims to accomplish.

Adds compute requirement to deepscaler guide.

Issues

List issues that this PR closes (syntax):

Closes: #493
Closes: #456

Usage

  • You can potentially add a usage example below
# Add a code snippet demonstrating how to use this 

Before your PR is "Ready for review"

Pre checks:

  • [Y] Make sure you read and followed Contributor guidelines
  • [N] Did you write any new necessary tests?
  • [N] Did you run the unit tests and functional tests locally? Visit our Testing Guide for how to run tests
  • [Y] Did you add or update any necessary documentation? Visit our Document Development Guide for how to write, build and test the docs.

Additional Information

  • ...

@github-actions github-actions bot added the documentation Improvements or additions to documentation label Jun 11, 2025
@abukharin-nv abukharin-nv changed the title Update guide to include minimum compute requirement docs: Update guide to include minimum compute requirement Jun 11, 2025
parthchadha
parthchadha previously approved these changes Jun 11, 2025
Signed-off-by: abukharin-nv <[email protected]>
Copy link
Contributor

@SahilJain314 SahilJain314 left a comment

Choose a reason for hiding this comment

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

quick reformat

Co-authored-by: Sahil Jain <[email protected]>
Signed-off-by: abukharin-nv <[email protected]>
@parthchadha parthchadha enabled auto-merge July 3, 2025 20:56
@parthchadha parthchadha added this pull request to the merge queue Jul 3, 2025
Merged via the queue into main with commit 7527d14 Jul 3, 2025
13 of 14 checks passed
@parthchadha parthchadha deleted the abukharin/deepscaler-guide branch July 3, 2025 23:12
therealnaveenkamal pushed a commit to therealnaveenkamal/RL that referenced this pull request Jul 7, 2025
…o#505)

Signed-off-by: abukharin-nv <[email protected]>
Co-authored-by: Sahil Jain <[email protected]>
Co-authored-by: Parth Chadha <[email protected]>
jialei777 pushed a commit to jialei777/nemo-rl that referenced this pull request Jul 23, 2025
…o#505)

Signed-off-by: abukharin-nv <[email protected]>
Co-authored-by: Sahil Jain <[email protected]>
Co-authored-by: Parth Chadha <[email protected]>
Signed-off-by: Jialei Chen <[email protected]>
KiddoZhu pushed a commit that referenced this pull request Jul 28, 2025
Signed-off-by: abukharin-nv <[email protected]>
Co-authored-by: Sahil Jain <[email protected]>
Co-authored-by: Parth Chadha <[email protected]>
FannYYW pushed a commit to xxman-google/NeMo-RL that referenced this pull request Aug 5, 2025
…o#505)

Signed-off-by: abukharin-nv <[email protected]>
Co-authored-by: Sahil Jain <[email protected]>
Co-authored-by: Parth Chadha <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

CUDA OOM when running deepscaler tutorial on 1 A6000 OOM when trying to reproduce the grpo-deepscaler run
4 participants