Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add gpu_env_fallback platform attribute #1050

Merged
merged 4 commits into from
Aug 3, 2023
Merged

Add gpu_env_fallback platform attribute #1050

merged 4 commits into from
Aug 3, 2023

Conversation

shuds13
Copy link
Member

@shuds13 shuds13 commented Aug 1, 2023 •
edited

For when setting GPUs to slots in a sim function, without using the MPI executor. Find correct setting for platform.

  • Add gpu_env_fallback attribute (used if no env var given to set_env_to_gpus function.
  • Add unit tests
  • Check shows up in docs

@shuds13 shuds13 marked this pull request as ready for review August 2, 2023 04:37
@shuds13 shuds13 self-assigned this Aug 2, 2023
@shuds13 shuds13 requested review from jmlarson1 and jlnav August 2, 2023 04:38
@shuds13 shuds13 merged commit 682fa29 into develop Aug 3, 2023
15 checks passed
@jmlarson1 jmlarson1 deleted the feature/add_gpu_env_fallback branch August 3, 2023 16:29
@shuds13 shuds13 mentioned this pull request Aug 31, 2023
Closed
34 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jmlarson1 jmlarson1 jmlarson1 approved these changes

@jlnav jlnav jlnav approved these changes

Assignees

@shuds13 shuds13

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@shuds13 @jmlarson1 @jlnav