Skip to content

fix(tests): support ROCm in AIME GPU memory checks - #763

Merged
jason-fxz merged 1 commit into
FlashML-org:mainfrom
LZ-QWQ:fix/rocm-aime-memory-query
Oct 21, 2026
Merged

jason-fxz merged 1 commit into
FlashML-org:mainfrom
LZ-QWQ:fix/rocm-aime-memory-query

Conversation

@LZ-QWQ

@LZ-QWQ LZ-QWQ commented Oct 20, 2026

Copy link
Copy Markdown
Contributor

The AIME memory check calls nvidia-smi, failing on ROCm before model loading. Query torch.cuda.mem_get_info(0) in a subprocess on ROCm. The NVIDIA path stays unchanged.

Validation on base 4d2e2e2: Qwen/Qwen3.5-35B-A3B-FP8, offload with auto cache, AIME25 question 1, pass@3, up to 16384 tokens per sample. Original sampling and answer assertions are unchanged.

FREETOKEN_TEST_MODEL=/path/to/Qwen3.5-35B-A3B-FP8 \
FREETOKEN_AIME25_JSONL=/path/to/aime25.jsonl \
FREETOKEN_TEST_MOE_CACHE_AUTO=1 \
FREETOKEN_AIME_MIN_FREE_GIB=40 \
python -m pytest tests/e2e/test_aime.py
GPU VRAM per GPU Before After
Radeon Pro W7900D 48 GiB Failed: missing nvidia-smi 1 passed
Radeon AI PRO R9700 32 GiB Failed: missing nvidia-smi 1 passed
RTX4090 48 GiB 1 passed 1 passed

On the 32 GiB R9700, set FREETOKEN_AIME_MIN_FREE_GIB=30 to match available VRAM and FREETOKEN_TEST_KV_RESERVE=18432 to preserve the 16384-token budget.

AMD validation: ROCm 10.1.0, Triton 3.8, PyTorch 2.14. Related to #535.

@github-actions github-actions Bot added the amd label Oct 20, 2026
KarrAcaRn pushed a commit to KarrAcaRn/FreeToken-ByAI that referenced this pull request Oct 20, 2026
FlashML-org#764 FlashML-org#763 FlashML-org#708 FlashML-org#687 deferred, FlashML-org#688 kept as a feature; FlashML-org#169 kept after its upstream close,
FlashML-org#494/FlashML-org#154/FlashML-org#315 replaced upstream by FlashML-org#690/FlashML-org#698; FlashML-org#278 dropped with upstream's auto->hybrid pick (FlashML-org#734).

Assisted-by: Claude Opus 5.5
@jason-fxz
jason-fxz merged commit fe5a5ec into FlashML-org:main Oct 21, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants