-
Notifications
You must be signed in to change notification settings - Fork 107
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.
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
Add support to GPUs in TaskChain spec #10805
Conversation
Jenkins results:
|
validate top level GPU in TaskChain
TaskChain unit tests for GPU
Jenkins results:
|
Basic tests went fine in my VM. @todor-ivanov feel free to leave any questions and/or your review. I will tackle the StepChain use case tomorrow, so I can fix whatever is left from this one. Thanks! |
@@ -1217,6 +1217,9 @@ def getChainCreateArgs(firstTask=False, generator=False): | |||
'Memory': {'default': None, 'null': True, 'type': float, 'validate': lambda x: x > 0}, | |||
'Multicore': {'default': 0, 'type': int, 'validate': lambda x: x > 0}, | |||
'PrepID': {'default': None, 'null': True, 'optional': True, 'type': str}, | |||
"RequiresGPU": {"default": "forbidden", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I hope these few lines do not overlap with the some of the previous PR. Otherwise this may cause some conflicts.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me @amaltaro
Thanks!
Fixes #10400
Status
ready
Description
As a follow up of: #10799
this PR enabled GPU support in the TaskChain spec, thus, supporting it inside each Task definition as well.
Note that for
TaskChain
workflows, there is the same default values as those for top level, thus:and the top level arguments will have
Is it backward compatible (if not, which system it affects?)
YES (new arguments were made optional)
Related PRs
Complement to: #10799
External dependencies / deployment changes
None