LISA icon indicating copy to clipboard operation
LISA copied to clipboard

I always encounter assertion errors when trying to reproduce the results of llava_v1.5.

Open xuewenMei opened this issue 7 months ago • 10 comments

I always encounter assertion errors when trying to reproduce the results of llava_v1.5. In this section, may I ask if there is any solution to this problem? /dataset.py/ if cur_len < tokenizer.model_max_length: assert cur_len == total_len

xuewenMei avatar May 27 '25 07:05 xuewenMei

I always encounter assertion errors when trying to reproduce the results of llava_v1.5. In this section, may I ask if there is any solution to this problem? /dataset.py/ if cur_len < tokenizer.model_max_length: assert cur_len == total_len

Have you solved this problem?

fwbc avatar May 30 '25 12:05 fwbc

请问你解决了吗

xuancan1 avatar Jun 24 '25 14:06 xuancan1

这个好像和我之前问你的那个问题是一样的,你在llava中的converstion.py指定默认对话模板为llava_v1,这个是因为他的模板不对,导致他的seq错误,然后用seq分割就错误了

fwbc avatar Jun 25 '25 07:06 fwbc

那这里可能还需要时间去研究下模板的不同之处,我目前还没精力去解决,在着手其它实验上

xuewenMei avatar Jun 25 '25 07:06 xuewenMei

那这里可能还需要时间去研究下模板的不同之处,我目前还没精力去解决,在着手其它实验上

在llava中的conversation.py,不是untils中的,untils.converstaion.py仿佛没有任何作用

fwbc avatar Jun 25 '25 07:06 fwbc

对的,如果想要性能,你可以再去试试llava-llama2-7b的版本13b如果硬件不行的话,在作者的描述中,这个版本展现出比llava-v1-13b更优性能

xuewenMei avatar Jun 25 '25 07:06 xuewenMei

没有,可能需要修改模板那里的细节

xuewenMei avatar Jun 25 '25 07:06 xuewenMei

对的,如果想要性能,你可以再去试试llava-llama2-7b的版本13b如果硬件不行的话,在作者的描述中,这个版本展现出比llava-v1-13b更优性能

你是指lisa模型的lisa-7b吗?我好像并没有找到llava-llama2-7b这个版本

fwbc avatar Jun 25 '25 08:06 fwbc

liuhaotian/llava-llama-2-7b-chat-lightning-lora-preview

xuewenMei avatar Jun 25 '25 08:06 xuewenMei

The bug can be seen at https://github.com/dvlab-research/LISA/issues/193

wudongming97 avatar Jul 09 '25 03:07 wudongming97