Merged
Conversation
Collaborator
zhulinJulia24
commented
Sep 9, 2025
- add 5080 testcase
- update test environment preparation steps
- remove deprecated models
- refactor mllm testcase's assertion logic
Contributor
There was a problem hiding this comment.
Pull Request Overview
This PR updates the CI testing infrastructure to support RTX 5080 testcases and refactors existing test cases. It removes deprecated models from test configurations and improves code maintainability by using f-string formatting in assertion logic.
- Adds support for RTX 5080 GPU testing environment
- Removes deprecated models and refactors mllm test assertion logic
- Updates CI workflow configurations and docker image versions
Reviewed Changes
Copilot reviewed 22 out of 22 changed files in this pull request and generated 5 comments.
Show a summary per file
| File | Description |
|---|---|
| requirements/lite.txt | Adds timm dependency for test environment |
| autotest/utils/run_restful_chat.py | Extends 3090 configuration support to include 5080 GPUs |
| autotest/utils/run_client_chat.py | Removes deprecated CodeLlama-specific logic and extends 5080 support |
| autotest/utils/quantization_utils.py | Extends batch size configuration for 5080 environment |
| autotest/utils/pipeline_chat.py | Modernizes string formatting and extends 5080 support |
| autotest/tools/restful/test_restful_chat_hf_turbomind_mllm.py | Removes deprecated Qwen2.5-VL-32B test configurations |
| autotest/tools/restful/test_restful_chat_hf_turbomind_llm.py | Removes deprecated Qwen2.5-VL model references |
| autotest/tools/pipeline/test_pipeline_chat_turbomind_mllm.py | Removes Qwen2.5-VL-7B and 32B model test configurations |
| autotest/tools/pipeline/test_pipeline_chat_turbomind_llm.py | Removes Qwen2.5-VL-32B from test parameter lists |
| autotest/tools/chat/test_command_chat_hf_turbomind.py | Removes Qwen2.5-VL-32B from test configurations |
| autotest/config.yaml | Updates model lists and adds/removes deprecated models |
| autotest/config-pr.yaml | Removes entire PR configuration file |
| autotest/config-5080.yaml | Adds new 5080 GPU test configuration |
| autotest/config-3090.yaml | Updates 3090 configuration removing deprecated models |
| autotest/chat_prompt_case.yaml | Removes hash character from test case |
| .github/workflows/*.yml | Updates CI workflows with new Docker images and environment setups |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
lvhan028
reviewed
Sep 11, 2025
lvhan028
approved these changes
Sep 12, 2025
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.