[Klaud Cold] Update qwen3.5-fp8-mi355x-sglang (+mtp) SGLang ROCm image to v0.5.20-rocm720-mi35x-20260924 - #3423
Conversation
…e to v0.5.20-rocm720-mi35x-20260924 Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
ef8421f to
5b52e4a
Compare
|
Thanks for the contribution!
中文感谢你的贡献!
|
| - config-keys: | ||
| - qwen3.5-fp8-mi355x-sglang | ||
| - qwen3.5-fp8-mi355x-sglang-mtp | ||
| description: |
There was a problem hiding this comment.
🔴 The new entry's pr-link: PRLINK_PLACEHOLDER will make the merge-prep automation reject this PR instead of auto-filling the real link. infx/workflows/prepare_perf_changelog_merge.py (validate_perf_changelog.py) only accepts "XXX" or "https://github.com/SemiAnalysisAI/InferenceX/pull/XXX" as placeholders (PR_LINK_PLACEHOLDERS); any other value raises ChangelogValidationError(f"appended entry {index + 1} has unexpected pr-link {link!r}"). Fix: use the documented placeholder https://github.com/SemiAnalysisAI/InferenceX/pull/XXX (or XXX) so the merge automation can canonicalize it to the real PR link.
Why this was flagged
perf-changelog.yaml:8894 sets pr-link: PRLINK_PLACEHOLDER for the newly appended entry (config-keys qwen3.5-fp8-mi355x-sglang / -mtp). infx/workflows/prepare_perf_changelog_merge.py:96-109 computes expected_link and, for each appended entry whose link isn't already the expected link, requires it be in PR_LINK_PLACEHOLDERS = {"XXX", "https://github.com/SemiAnalysisAI/InferenceX/pull/XXX"} (validate_perf_changelog.py:21-24); otherwise it raises ChangelogValidationError. PRLINK_PLACEHOLDER matches neither, so the merge-prep step (and validate_perf_changelog.py:138's same check) fails, blocking this PR from merging via the normal automated path where the base branch would have a correctly formatted placeholder or real link.
Verification: normal. The appended entry sets pr-link: PRLINK_PLACEHOLDER (perf-changelog.yaml:8896). The merge-prep automation infx/workflows/prepare_perf_changelog_merge.py command canonicalize calls canonicalize_appended_links -> compare_entries(base, head, pr_number) (line 86). Inside compare_entries, every appended entry passes through validate_added_pr_link(link, pr_number)… | normal. The…
|
View unofficial run (performance): https://inferencex.semianalysis.com/inference?unofficialRun=36086631310 View unofficial run (accuracy): https://inferencex.semianalysis.com/evaluation?unofficialRun=36086631310 |
|
/reuse-sweep-run 36086631310 |
1 similar comment
|
/reuse-sweep-run 36086631310 |
# Conflicts: # perf-changelog.yaml
Summary
Update SGLang ROCm image from
v0.5.18-rocm720-mi35x-20260828tov0.5.20-rocm720-mi35x-20260924(latest nightly) for the Qwen3.5 FP8 MI355X 8k1k recipes.Recipes touched:
qwen3.5-fp8-mi355x-sglang,qwen3.5-fp8-mi355x-sglang-mtpChanges:
configs/amd-master.yaml: image bump for both keys.srt-slurm-recipes/qwen3.5/sglang/mi355x-fp8{,-mtp}/8k1k.yaml:containerbump;cuda-graph-max-bs→cuda-graph-max-bs-decode(SGLang nightlies ≥ 09-15 reject the old flag as ambiguous; matches the b200 siblings).perf-changelog.yaml: new entry.Test plan
🤖 Generated with Claude Code