Skip to content

Conversation

@zhshj0110
Copy link
Contributor

  1. Correct the path error in mvbench.
  2. Fix the bug of evaluation of llava_vid on mvbench.

Before you open a pull-request, please check if a similar issue already exists or has been closed before.

When you open a pull-request, please be sure to include the following

  • A descriptive title: [xxx] XXXX
  • A detailed description

If you meet the lint warnings, you can use following scripts to reformat code.

pip install pre-commit
pre-commit install
pre-commit run --all-files

Thank you for your contributions!

Modify DATA_LIST so that it can find the corresponding video
"mvbench_video/tvqa/frames_fps3_hq/castle_s07e04_seg02_clip_14" is the sampled video frame, not the original video file. And The current code logic cannot handle this subtask.
@Luodian Luodian merged commit 2508d42 into EvolvingLMMs-Lab:main Feb 19, 2025
1 check passed
MichalCiesiolka pushed a commit to MichalCiesiolka/lmms-eval-llmzszl that referenced this pull request Apr 3, 2025
* Update README.md

* Update README.md

* [Fix] of "mvbench missing videos"

Modify DATA_LIST so that it can find the corresponding video

* [Fix] of "evaluation of llava_vid on mvbench"

"mvbench_video/tvqa/frames_fps3_hq/castle_s07e04_seg02_clip_14" is the sampled video frame, not the original video file. And The current code logic cannot handle this subtask.

* Fixing lmms_eval/models/llava_vid.py
dadwadw233 pushed a commit to dadwadw233/lmms-eval that referenced this pull request Apr 28, 2025
* Update README.md

* Update README.md

* [Fix] of "mvbench missing videos"

Modify DATA_LIST so that it can find the corresponding video

* [Fix] of "evaluation of llava_vid on mvbench"

"mvbench_video/tvqa/frames_fps3_hq/castle_s07e04_seg02_clip_14" is the sampled video frame, not the original video file. And The current code logic cannot handle this subtask.

* Fixing lmms_eval/models/llava_vid.py
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