[CI] Fix failed CI step "PyTorch Compilation Unit Tests"#41959
[CI] Fix failed CI step "PyTorch Compilation Unit Tests"#41959SoluMilken wants to merge 3 commits intovllm-project:mainfrom
Conversation
Signed-off-by: SoluMilken <ypiheyn.imm02g@g2.nctu.edu.tw>
Signed-off-by: SoluMilken <ypiheyn.imm02g@g2.nctu.edu.tw>
6a6990e to
50299c6
Compare
There was a problem hiding this comment.
Code Review
This pull request improves error reporting for test subprocesses by including both stdout and stderr in the failure message and using robust decoding to handle potential encoding issues. Additionally, it initializes a new dictionary, var_to_hint_override, in the AlwaysHitShapeEnv class. Feedback was provided to eliminate redundant output in the error message when a traceback file is missing by using a placeholder instead of duplicating the stderr content.
Signed-off-by: SoluMilken <ypiheyn.imm02g@g2.nctu.edu.tw>
|
Hi @ProExpertProg, I saw you just reviewed/approved another CI fix PR. Could you help me take a quick look at this one as well? I'm trying to fix the "PyTorch Compilation Unit Tests" CI failure and would appreciate your guidance to get it passing. Thanks! |
Purpose
Fix 41958.
Test Plan
Test Result
Essential Elements of an Effective PR Description Checklist
supported_models.mdandexamplesfor a new model.