Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix loading single safetensor file error #1427

Merged
merged 1 commit into from
Apr 12, 2024
Merged

Conversation

AllentDan
Copy link
Collaborator

@AllentDan AllentDan commented Apr 12, 2024

@lvhan028 lvhan028 requested a review from irexyc April 12, 2024 03:42
@lvhan028
Copy link
Collaborator

@zhulinJulia24 请问 pr test 失败的原因是?

@zhulinJulia24
Copy link
Collaborator

@zhulinJulia24 请问 pr test 失败的原因是?记忆性的assert还是不够健壮,我想想怎么优化下

AssertionError: 成都 shouldn't exist in:当然可以!以下是一些美食的介绍:\n\n1. 麻婆豆腐:这是四川省成都市的传统名菜,主要原料是豆腐和牛肉末,口感麻辣,是四川菜系中的代表之一。\n\n2. 小笼包:这是上海的特色美食,外皮薄而有弹性,内馅是鲜美的猪肉,汁水丰富,味道十分鲜美。\n\n3. 北京烤鸭:这是北京的传统名菜,以鸭子为原料,经过独特的烤制工艺,使得烤鸭皮脆肉嫩,香味四溢。\n\n4. 叉烧:这是广东省的传统名菜,主要原料是猪肉,经过腌制和烤制,口感香甜,肉质鲜嫩。\n\n5. 担担面:这是四川省成都市的传统小吃,主要原料是面条和肉末,口感麻辣,是四川菜系中的代表之一。\n\n6. 水饺:这是中国传统的小吃,主要原料是面粉和馅料,口感鲜美,营养丰富,是中国人喜爱的传统美食之一。\n\n以上这些美食都有着各自独特的风味和口感,如果你有机会品尝,一定不要错过哦!

@AllentDan AllentDan mentioned this pull request Apr 12, 2024
@lvhan028
Copy link
Collaborator

@zhulinJulia24 请问 pr test 失败的原因是?记忆性的assert还是不够健壮,我想想怎么优化下

AssertionError: 成都 shouldn't exist in:当然可以!以下是一些美食的介绍:\n\n1. 麻婆豆腐:这是四川省成都市的传统名菜,主要原料是豆腐和牛肉末,口感麻辣,是四川菜系中的代表之一。\n\n2. 小笼包:这是上海的特色美食,外皮薄而有弹性,内馅是鲜美的猪肉,汁水丰富,味道十分鲜美。\n\n3. 北京烤鸭:这是北京的传统名菜,以鸭子为原料,经过独特的烤制工艺,使得烤鸭皮脆肉嫩,香味四溢。\n\n4. 叉烧:这是广东省的传统名菜,主要原料是猪肉,经过腌制和烤制,口感香甜,肉质鲜嫩。\n\n5. 担担面:这是四川省成都市的传统小吃,主要原料是面条和肉末,口感麻辣,是四川菜系中的代表之一。\n\n6. 水饺:这是中国传统的小吃,主要原料是面粉和馅料,口感鲜美,营养丰富,是中国人喜爱的传统美食之一。\n\n以上这些美食都有着各自独特的风味和口感,如果你有机会品尝,一定不要错过哦!

句子里不是有 “四川省成都市” 么?

@lvhan028 lvhan028 changed the title Fix load single safetensor file Fix loading single safetensor file error Apr 12, 2024
@lvhan028 lvhan028 merged commit acdefab into InternLM:main Apr 12, 2024
4 of 5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants