Skip to content

Need to run Realm e2e test on 2 GPUs to match mapping#1656

Open
elliottslaughter wants to merge 1 commit into
flexflow:masterfrom
elliottslaughter:fix-realm-gpu-test
Open

Need to run Realm e2e test on 2 GPUs to match mapping#1656
elliottslaughter wants to merge 1 commit into
flexflow:masterfrom
elliottslaughter:fix-realm-gpu-test

Conversation

@elliottslaughter

@elliottslaughter elliottslaughter commented Jun 18, 2026

Copy link
Copy Markdown
Collaborator

#1651 unified some code to generate the MPCG for Realm e2e tests. That's fine, but results in the CUDA test assuming 2 GPUs available.

There are two ways to fix this:

  1. Actually run on 2 GPUs
  2. Make the MPCG configurable so we don't assume 2 devices

For now option (1) seems easier and if we're moving GPU CI to Sapling eventually anyway, having 2 GPUs is a trivial requirement to satisfy.

Makes all tests pass on master.


This change is Reviewable

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.

1 participant