- 
          
- 
                Notifications
    You must be signed in to change notification settings 
- Fork 10.9k
[VLM] Add Qwen3-VL generation test #25185
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Signed-off-by: Isotr0py <mozf@mail2.sysu.edu.cn>
Signed-off-by: Isotr0py <mozf@mail2.sysu.edu.cn>
| Should we wait until model release before actually merging this? | 
Signed-off-by: Isotr0py <mozf@mail2.sysu.edu.cn>
| 
 I'm fine to wait until model release and Transformers upgrade, because this test won't run on CI without them. The goal of PR is just to provide a test for generation correctness validation, so that we can run correctness test for other Qwen3-VL related PRs locally to avoid regression. So whether merge this PR or not won't be a big deal IMO. | 
Signed-off-by: Isotr0py <mozf@mail2.sysu.edu.cn>
Signed-off-by: Roger Wang <hey@rogerw.io>
| I'll work on this PR since @Isotr0py might be busy | 
Signed-off-by: Roger Wang <hey@rogerw.io>
| Test results from   | 
Signed-off-by: Roger Wang <hey@rogerw.io>
Signed-off-by: Roger Wang <hey@rogerw.io>
Signed-off-by: Roger Wang <hey@rogerw.io>
Signed-off-by: Isotr0py <mozf@mail2.sysu.edu.cn> Signed-off-by: Roger Wang <hey@rogerw.io> Co-authored-by: Roger Wang <hey@rogerw.io>
Purpose
Test Plan
Test Result
Test should pass
Essential Elements of an Effective PR Description Checklist
supported_models.mdandexamplesfor a new model.