Skip to content

Conversation

@LooperXX
Copy link
Contributor

  • Configuration rollback has been added to avoid errors on evaluation benchmarks with a mix of single and multiple images such as MMMU, SEEDBench, etc.
    • Original code will override the original image_aspect_ratio by this line. This behaviour appears in loglikelihood, generate_until and generate_until_multi_round functions.
    • Such overriding behaviour will result in the image_aspect_ratio of all single-image samples encountered after multiple-image samples being overridden by e.g. pad on benchmarks with both single-image and multiple-image samples.

Configuration rollback has been added to avoid errors on evaluation benchmarks with a mix of single and multiple images such as MMMU.
…h both single- and multi-image samples.

Configuration rollback has been added to avoid errors on evaluation benchmarks with a mix of single and multiple images such as MMMU.
…h both single- and multi-image samples.

Black this file.
@Luodian Luodian merged commit 720457c into EvolvingLMMs-Lab:main Oct 22, 2024
1 check passed
KairuiHu pushed a commit that referenced this pull request Oct 24, 2024
…h both single- and multi-image samples. (#338)

* Update llava_onevision.py

Configuration rollback has been added to avoid errors on evaluation benchmarks with a mix of single and multiple images such as MMMU.

* Update llava_onevision.py to avoid erros on evaluation benchmarks with both single- and multi-image samples.

Configuration rollback has been added to avoid errors on evaluation benchmarks with a mix of single and multiple images such as MMMU.

* Update llava_onevision.py to avoid erros on evaluation benchmarks with both single- and multi-image samples.

Black this file.
ZhaoCinyu pushed a commit to ZhaoCinyu/lmms-eval that referenced this pull request Dec 9, 2024
…h both single- and multi-image samples. (EvolvingLMMs-Lab#338)

* Update llava_onevision.py

Configuration rollback has been added to avoid errors on evaluation benchmarks with a mix of single and multiple images such as MMMU.

* Update llava_onevision.py to avoid erros on evaluation benchmarks with both single- and multi-image samples.

Configuration rollback has been added to avoid errors on evaluation benchmarks with a mix of single and multiple images such as MMMU.

* Update llava_onevision.py to avoid erros on evaluation benchmarks with both single- and multi-image samples.

Black this file.
MichalCiesiolka pushed a commit to MichalCiesiolka/lmms-eval-llmzszl that referenced this pull request Apr 3, 2025
…h both single- and multi-image samples. (EvolvingLMMs-Lab#338)

* Update llava_onevision.py

Configuration rollback has been added to avoid errors on evaluation benchmarks with a mix of single and multiple images such as MMMU.

* Update llava_onevision.py to avoid erros on evaluation benchmarks with both single- and multi-image samples.

Configuration rollback has been added to avoid errors on evaluation benchmarks with a mix of single and multiple images such as MMMU.

* Update llava_onevision.py to avoid erros on evaluation benchmarks with both single- and multi-image samples.

Black this file.
dadwadw233 pushed a commit to dadwadw233/lmms-eval that referenced this pull request Apr 28, 2025
…h both single- and multi-image samples. (EvolvingLMMs-Lab#338)

* Update llava_onevision.py

Configuration rollback has been added to avoid errors on evaluation benchmarks with a mix of single and multiple images such as MMMU.

* Update llava_onevision.py to avoid erros on evaluation benchmarks with both single- and multi-image samples.

Configuration rollback has been added to avoid errors on evaluation benchmarks with a mix of single and multiple images such as MMMU.

* Update llava_onevision.py to avoid erros on evaluation benchmarks with both single- and multi-image samples.

Black this file.
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