Skip to content

Add a field for a list of GPU models, and update versions to v10 - #27

Merged
ianmcorvidae merged 2 commits into
cyverse-de:mainfrom
ianmcorvidae:gpu-models
Feb 18, 2026
Merged

Add a field for a list of GPU models, and update versions to v10#27
ianmcorvidae merged 2 commits into
cyverse-de:mainfrom
ianmcorvidae:gpu-models

Conversation

@ianmcorvidae

Copy link
Copy Markdown
Member

(v10 won't exist until merging, but made sense to update it now, I thought)

Basically just plumbing. Jobs that want to limit the acceptable models provide a list, ones that don't care provide an empty list or null.

@slr71 slr71 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks good to me! 👍

- This module is versioned as v9 in the module path. If releasing a new major version, update module path and coordinate with downstream consumers.

## Where to look next / follow-ups
- If you need runtime examples of how these models are consumed, search in the `cyverse-de` organization for imports of `github.com/cyverse-de/model/v9`.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

👍

@ianmcorvidae

Copy link
Copy Markdown
Member Author

Cool, thanks for the review. I'll get it merged in so it's easier for me to start using it!

@ianmcorvidae
ianmcorvidae merged commit 82dfb78 into cyverse-de:main Feb 18, 2026
3 checks passed
@ianmcorvidae
ianmcorvidae deleted the gpu-models branch February 18, 2026 18:12
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