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

  • ...

Signed-off-by: abukharin-nv <abukharin@nvidia.com>
@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 <abukharin@nvidia.com>
Signed-off-by: abukharin-nv <abukharin@nvidia.com>
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 <48468750+SahilJain314@users.noreply.github.com>
Signed-off-by: abukharin-nv <abukharin@nvidia.com>
@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 <abukharin@nvidia.com>
Co-authored-by: Sahil Jain <48468750+SahilJain314@users.noreply.github.com>
Co-authored-by: Parth Chadha <pchadha@nvidia.com>
jialei777 pushed a commit to jialei777/nemo-rl that referenced this pull request Jul 23, 2025
…o#505)

Signed-off-by: abukharin-nv <abukharin@nvidia.com>
Co-authored-by: Sahil Jain <48468750+SahilJain314@users.noreply.github.com>
Co-authored-by: Parth Chadha <pchadha@nvidia.com>
Signed-off-by: Jialei Chen <jialeic@google.com>
KiddoZhu pushed a commit that referenced this pull request Jul 28, 2025
Signed-off-by: abukharin-nv <abukharin@nvidia.com>
Co-authored-by: Sahil Jain <48468750+SahilJain314@users.noreply.github.com>
Co-authored-by: Parth Chadha <pchadha@nvidia.com>
FannYYW pushed a commit to xxman-google/NeMo-RL that referenced this pull request Aug 5, 2025
…o#505)

Signed-off-by: abukharin-nv <abukharin@nvidia.com>
Co-authored-by: Sahil Jain <48468750+SahilJain314@users.noreply.github.com>
Co-authored-by: Parth Chadha <pchadha@nvidia.com>
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