[Minor] Update utils.py

[YangYangGirl]

correct video path in videomme_dic_to_text+subtitle

[kcz358]

Hi @YangYangGirl , may I ask why would this additional hardcoded data path need to be added in the doc to visual function? I think current main branch should works well. If that is your default path in your own settings, maybe this change should be add into your own forked repo? If that is the case, I will close the PR.

[YangYangGirl]

The original video path in the main repo does not exist, resulting in total_frame = 0 and subtitle loading as none. It will run smoothly but result in inaccurate results.

[kcz358]

I think this problem has been fixed in commit Fix videomme evaluation? Current main branch seems already contains the path to data folder

lmms-eval/lmms_eval/tasks/videomme/utils.py

Lines 176 to 188 in bfff5ab

	def videomme_doc_to_visual(doc):
	cache_dir = os.path.join(base_cache_dir, cache_name)
	video_path = doc["videoID"] + ".mp4"
	video_path = os.path.join(cache_dir, "data", video_path)
	if os.path.exists(video_path):
	video_path = video_path
	elif os.path.exists(video_path.replace("mp4", "MP4")):
	video_path = video_path.replace("mp4", "MP4")
	elif os.path.exists(video_path.replace("mp4", "mkv")):
	video_path = video_path.replace("mp4", "mkv")
	else:
	sys.exit(f"video path:{video_path} does not exist, please check")
	return [video_path]

Seems like a duplicate fix. If this is the case, I will close this PR.

[YangYangGirl]

Thanks for the kind reply. But I checked again, in

lmms-eval/lmms_eval/tasks/videomme/utils.py

Line 21 in bfff5ab

"Humanity & History",

, the video path is still wrong.

[kcz358]

Yeah @YangYangGirl , I see your point. You are correct. I think this is some version problem as we change the logic and default folder for unzipping files. I revise a little bit to make it not absolute path. I think current version should be correct?

[kcz358]

Have done a quick check here and subtitles should have loaded correctly.

merging this PR now and thank you for spotting the issue.