Skip to content

feat: Log code in wandb#175

Merged
SahilJain314 merged 6 commits intomainfrom
yifu/wandb_code
Jun 27, 2025
Merged

feat: Log code in wandb#175
SahilJain314 merged 6 commits intomainfrom
yifu/wandb_code

Conversation

@yfw
Copy link
Contributor

@yfw yfw commented Apr 14, 2025

What does this PR do ?

Logs code in wandb. This currently only logs code under the nemo_reinforcer subdirectory to prevent scanning all the files in the root reinforcer directory. Sample wandb run: https://wandb.ai/nvidia/reinforcer-test-wandbcode/runs/176v9pxz/code. Diffs can be viewed in the Artifacts tab (https://community.wandb.ai/t/code-comparer-can-only-show-the-difference-of-main-training-file/3020/5).

Issues

Closes #146

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:

  • Make sure you read and followed Contributor guidelines
  • Did you write any new necessary tests?
  • Did you run the unit tests and functional tests locally? Visit our Testing Guide for how to run tests
  • 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: Yi-Fu Wu <yifu.wu@gmail.com>
@yfw yfw added the Run CICD label Apr 14, 2025
yfw added 3 commits April 17, 2025 12:10
Signed-off-by: Yi-Fu Wu <yifu.wu@gmail.com>
Signed-off-by: Yi-Fu Wu <yifu.wu@gmail.com>
@SahilJain314 SahilJain314 enabled auto-merge June 26, 2025 21:48
@SahilJain314 SahilJain314 added this pull request to the merge queue Jun 26, 2025
Merged via the queue into main with commit 7448d69 Jun 27, 2025
13 of 14 checks passed
@SahilJain314 SahilJain314 deleted the yifu/wandb_code branch June 27, 2025 18:09
xxman-google pushed a commit to xxman-google/NeMo-RL that referenced this pull request Jun 27, 2025
Signed-off-by: Yi-Fu Wu <yifu.wu@gmail.com>
Signed-off-by: Parth Chadha <pchadha@nvidia.com>
Co-authored-by: Parth Chadha <pchadha@nvidia.com>
xxman-google pushed a commit to xxman-google/NeMo-RL that referenced this pull request Jun 30, 2025
Signed-off-by: Yi-Fu Wu <yifu.wu@gmail.com>
Signed-off-by: Parth Chadha <pchadha@nvidia.com>
Co-authored-by: Parth Chadha <pchadha@nvidia.com>
Signed-off-by: Xuehan <xxman@google.com>
xxman-google pushed a commit to xxman-google/NeMo-RL that referenced this pull request Jun 30, 2025
Signed-off-by: Yi-Fu Wu <yifu.wu@gmail.com>
Signed-off-by: Parth Chadha <pchadha@nvidia.com>
Co-authored-by: Parth Chadha <pchadha@nvidia.com>
Signed-off-by: Xuehan <xxman@google.com>
therealnaveenkamal pushed a commit to therealnaveenkamal/RL that referenced this pull request Jul 7, 2025
Signed-off-by: Yi-Fu Wu <yifu.wu@gmail.com>
Signed-off-by: Parth Chadha <pchadha@nvidia.com>
Co-authored-by: Parth Chadha <pchadha@nvidia.com>
YzjiaoNvd pushed a commit to YzjiaoNvd/NeMo-RL that referenced this pull request Jul 14, 2025
Signed-off-by: Yi-Fu Wu <yifu.wu@gmail.com>
Signed-off-by: Parth Chadha <pchadha@nvidia.com>
Co-authored-by: Parth Chadha <pchadha@nvidia.com>
KiddoZhu pushed a commit that referenced this pull request Jul 28, 2025
Signed-off-by: Yi-Fu Wu <yifu.wu@gmail.com>
Signed-off-by: Parth Chadha <pchadha@nvidia.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
Signed-off-by: Yi-Fu Wu <yifu.wu@gmail.com>
Signed-off-by: Parth Chadha <pchadha@nvidia.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

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

save diffs when running code automatically

3 participants