Skip to content

Correct the configuration of LLaVA-CoT#705

Merged
kennymckormick merged 1 commit intoopen-compass:mainfrom
XuGW-Kevin:main
Dec 31, 2024
Merged

Correct the configuration of LLaVA-CoT#705
kennymckormick merged 1 commit intoopen-compass:mainfrom
XuGW-Kevin:main

Conversation

@XuGW-Kevin
Copy link
Copy Markdown
Contributor

@XuGW-Kevin XuGW-Kevin commented Dec 31, 2024

First of all, we sincerely appreciate the work of VLMEvalKit and the tremendous contributions it has made to the entire VLM community! However, the current configuration of LLaVA-CoT (e.g., max_new_tokens) is incorrect, leading to significant deviations in the benchmark test results. This PR aims to correct the configuration of LLaVA-CoT.

@kennymckormick kennymckormick merged commit 6e1a59a into open-compass:main Dec 31, 2024
kennymckormick pushed a commit to TobiasLee/VLMEvalKit that referenced this pull request Jan 1, 2025
kennymckormick added a commit that referenced this pull request Jan 1, 2025
* update vlrewardbench

* pre-commit fix

* formatter

* [Improvement] Better `AUTO_SPLIT` and model split for InternVL2

* [Minor] Improve CC-OCR Import

* [Model] Support QVQ

* [Model] Update Molmo Eval to Match Official Implementation (#648)

* add molmo prompts

* fix lint format

* [Fix] Refine Qwen-VL2 device assignment

* [Fix] Fix RealWorldQA md5

* update MMMU_DEV_VAL tsv

* [Fix] Fix confusing image width&height (#704)

Co-authored-by: Yuan Ye <yuany2@chinatelecom.cn>

* Update llama_vision.py (#705)

* [Fix] Fix Lint

* Fix Lint

* Fix Lint

---------

Co-authored-by: kennymckormick <dhd.efz@gmail.com>
Co-authored-by: jamespark3922 <jspark96@cs.washington.edu>
Co-authored-by: CMeteor <CMeteor@users.noreply.github.com>
Co-authored-by: Yuan Ye <yuany2@chinatelecom.cn>
Co-authored-by: Guowei Xu <113534787+XuGW-Kevin@users.noreply.github.com>
Mercury7353 pushed a commit to Mercury7353/VLMEvalKit that referenced this pull request Apr 28, 2025
Mercury7353 pushed a commit to Mercury7353/VLMEvalKit that referenced this pull request Apr 28, 2025
* update vlrewardbench

* pre-commit fix

* formatter

* [Improvement] Better `AUTO_SPLIT` and model split for InternVL2

* [Minor] Improve CC-OCR Import

* [Model] Support QVQ

* [Model] Update Molmo Eval to Match Official Implementation (open-compass#648)

* add molmo prompts

* fix lint format

* [Fix] Refine Qwen-VL2 device assignment

* [Fix] Fix RealWorldQA md5

* update MMMU_DEV_VAL tsv

* [Fix] Fix confusing image width&height (open-compass#704)

Co-authored-by: Yuan Ye <yuany2@chinatelecom.cn>

* Update llama_vision.py (open-compass#705)

* [Fix] Fix Lint

* Fix Lint

* Fix Lint

---------

Co-authored-by: kennymckormick <dhd.efz@gmail.com>
Co-authored-by: jamespark3922 <jspark96@cs.washington.edu>
Co-authored-by: CMeteor <CMeteor@users.noreply.github.com>
Co-authored-by: Yuan Ye <yuany2@chinatelecom.cn>
Co-authored-by: Guowei Xu <113534787+XuGW-Kevin@users.noreply.github.com>
Koii2k3 pushed a commit to wjnwjn59/VLMEvalKit that referenced this pull request Nov 13, 2025
Koii2k3 pushed a commit to wjnwjn59/VLMEvalKit that referenced this pull request Nov 13, 2025
* update vlrewardbench

* pre-commit fix

* formatter

* [Improvement] Better `AUTO_SPLIT` and model split for InternVL2

* [Minor] Improve CC-OCR Import

* [Model] Support QVQ

* [Model] Update Molmo Eval to Match Official Implementation (open-compass#648)

* add molmo prompts

* fix lint format

* [Fix] Refine Qwen-VL2 device assignment

* [Fix] Fix RealWorldQA md5

* update MMMU_DEV_VAL tsv

* [Fix] Fix confusing image width&height (open-compass#704)

Co-authored-by: Yuan Ye <yuany2@chinatelecom.cn>

* Update llama_vision.py (open-compass#705)

* [Fix] Fix Lint

* Fix Lint

* Fix Lint

---------

Co-authored-by: kennymckormick <dhd.efz@gmail.com>
Co-authored-by: jamespark3922 <jspark96@cs.washington.edu>
Co-authored-by: CMeteor <CMeteor@users.noreply.github.com>
Co-authored-by: Yuan Ye <yuany2@chinatelecom.cn>
Co-authored-by: Guowei Xu <113534787+XuGW-Kevin@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants