Skip to content

Conversation

@maxy-shpfy
Copy link
Collaborator

@maxy-shpfy maxy-shpfy commented Jan 8, 2026

Description

Restricted GPU options in the launcher task annotation schema to only include NVIDIA H200, removing NVIDIA H100 and NVIDIA A100 options.

Related Issue and Pull requests

Type of Change

  • Bug fix
  • New feature
  • Improvement
  • Cleanup/Refactor
  • Breaking change
  • Documentation update

Checklist

  • I have tested this does not break current pipelines / runs functionality
  • I have tested the changes on staging

Screenshots (if applicable)

Test Instructions

image.png

Verify that only NVIDIA H200 appears as a GPU option in the launcher task interface.

Additional Comments

This change simplifies GPU selection by limiting options to only the H200 model.
See https://vault.shopify.io/docs/craft/138-Data/data_handbook/ml_at_shopify/ml_infrastructure#shopify-gpu-policy

Copy link
Collaborator Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

@maxy-shpfy maxy-shpfy marked this pull request as ready for review January 8, 2026 05:11
@maxy-shpfy maxy-shpfy requested a review from a team as a code owner January 8, 2026 05:11
@maxy-shpfy maxy-shpfy requested a review from Mbeaulne January 8, 2026 05:11
Copy link
Collaborator

@Mbeaulne Mbeaulne left a comment

Choose a reason for hiding this comment

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

Apparently Shopify is made of money. Those H200 can't be cheap!

Copy link
Collaborator Author

maxy-shpfy commented Jan 8, 2026

Merge activity

  • Jan 8, 3:45 PM UTC: A user started a stack merge that includes this pull request via Graphite.
  • Jan 8, 3:46 PM UTC: Graphite rebased this pull request as part of a merge.
  • Jan 8, 3:47 PM UTC: @maxy-shpfy merged this pull request with Graphite.

@maxy-shpfy maxy-shpfy force-pushed the 01-07-fix_remove_unsupported_gpus_from_configuration branch from 3c2c5b8 to 13d8c13 Compare January 8, 2026 15:45
@maxy-shpfy maxy-shpfy merged commit 25f31a1 into master Jan 8, 2026
8 checks passed
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.

3 participants