Open
Conversation
Add `print_script: bool = False` field to `SlurmExecutor`. When `True`, the generated sbatch script is printed to stdout after materialization in `_submit_dryrun()`, enabling users to copy-paste or pipe the script directly to `sbatch` without the NeMo Run dependency. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com> Signed-off-by: oliver könig <okoenig@nvidia.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com> Signed-off-by: oliver könig <okoenig@nvidia.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
print_script: bool = Falsefield toSlurmExecutorTrue, the generated sbatch script is printed tostdoutafter materialization in_submit_dryrun(), for both dryrun and real submissionssbatchwithout the NeMo Run dependencyMotivation
Currently
dryrun=Truelogs the script via the Rich console, but there is no clean way to get plain text to stdout for scripting. This option makes it easy to reproduce a job standalone:or pipe directly:
python my_job.py | sbatchTest plan
test_submit_dryrun_print_script— verifies#!/bin/bashappears in stdout whenprint_script=Truetest_submit_dryrun_no_print_by_default— verifies stdout is empty whenprint_script=False(default)uv run -- pytest test/🤖 Generated with Claude Code