Repository navigation
Skip the batch sampler runtime test on ROCm - #24013
Conversation
Summary: The ROCm AOTI unittest job on main fails during pytest collection: the newly added batch sampler test imports the Python runtime at module scope, and the ROCm CI build only ships executor_runner, not the pybindings extension. One collection error aborts the whole backends/cuda/tests run, so every ROCm test stops running. Import the runtime inside the test and skip the class on ROCm, following the existing convention for tests that load through the Python runtime. Differential Revision: D126231386
🔗 Helpful Links🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/executorch/24013
Note: Links to docs will display an error until the docs builds have been completed. ⏳ 37 Pending, 2 Unrelated FailuresAs of commit 901fab9 with merge base b1beef3 ( BROKEN TRUNK - The following jobs failed but were present on the merge base:👉 Rebase onto the `viable/strict` branch to avoid these failures
This comment was automatically generated by Dr. CI and updates every 15 minutes. |
meta-cla
Bot
added
the
CLA Signed
|
@Gasoonjia has exported this pull request. If you are a Meta employee, you can view the originating Diff in D126231386. |
This PR needs a
|
This branch was successfully deployed
2 participants
Summary:
The ROCm AOTI unittest job on main fails during pytest collection: the newly added batch sampler test imports the Python runtime at module scope, and the ROCm CI build only ships executor_runner, not the pybindings extension. One collection error aborts the whole backends/cuda/tests run, so every ROCm test stops running.
Import the runtime inside the test and skip the class on ROCm, following the existing convention for tests that load through the Python runtime.
Differential Revision: D126231386