Skip to content

Conversation

@k-artem
Copy link
Contributor

@k-artem k-artem commented Dec 23, 2025

TestAmp failures observed on ROCm due to:
53e91a0#diff-e6635a81d2c2bf0938b5f83b1c4945e0f344e0106191a6960df8ee4aa64cc55fR2305, but since amp is deprecated and removed from apex (NVIDIA/apex#1896), and these tests are already skipped on NVIDIA, we will skip them on ROCm as well. finally TestAmp is removed from UT, see comments.

Command for verification:
pytest -v tests/unit/runtime/half_precision/test_fp16.py -k 'TestAmp'

TestAmp failures observed on ROCm due to:
deepspeedai@53e91a0#diff-e6635a81d2c2bf0938b5f83b1c4945e0f344e0106191a6960df8ee4aa64cc55fR2305
Since amp is deprecated and removed from apex (NVIDIA/apex#1896),
and these tests are already skipped on NVIDIA, we will skip them on ROCm as well.

Signed-off-by: Artem Kuzmitckii <artem.kuzmitckii@amd.com>
@sfc-gh-truwase sfc-gh-truwase enabled auto-merge (squash) January 5, 2026 15:30
@sfc-gh-truwase sfc-gh-truwase merged commit d7bb6f3 into deepspeedai:master Jan 5, 2026
9 checks passed
@k-artem k-artem deleted the skipped_amp_testcase branch January 5, 2026 17:00
@k-artem k-artem changed the title [AMD] Skipped amp testcases for ROCm Removed amp testcases Jan 5, 2026
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