Skip to content

Added the ability to add command line arguments when submitting a job.#112

Merged
cjh1 merged 4 commits into
NERSC:mainfrom
lwashington3:main
Jul 14, 2026
Merged

Added the ability to add command line arguments when submitting a job.#112
cjh1 merged 4 commits into
NERSC:mainfrom
lwashington3:main

Conversation

@lwashington3

Copy link
Copy Markdown
Contributor

Added the optional argument args for the /compute/jobs/{machine} endpoint.

@cjh1 cjh1 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Thanks @lwashington3, looks good, one inline suggestion. I also wrote a small test case to test this, do you mine if I push it to your PR?

Comment thread src/sfapi_client/_async/compute.py

@cjh1 cjh1 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

@lwashington3 thank you for your contribution!

@cjh1
cjh1 merged commit af5c03d into NERSC:main Jul 14, 2026
11 of 12 checks passed
@cjh1

cjh1 commented Jul 14, 2026

Copy link
Copy Markdown
Collaborator

@lwashington3 This change has been include in 0.6.0

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.

2 participants