[readme] Update readme#81
Conversation
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request updates the English and Chinese README files by adding a project introduction, a recommended runtime environment table, and reorganizing the supported model list into text-only and multimodal categories. The review feedback identifies several areas for improvement, including maintaining header consistency, addressing duplicate model entries for 'bailing_moe', and correcting the potential misclassification of 'qwen3_5' and 'qwen3_5_moe' models within the multimodal list.
| | Qwen | qwen2_vl, qwen2_5_vl, qwen2_5_omni<br />qwen3_vl, qwen3_vl_moe, qwen3_omni_moe, qwen3_asr<br />qwen3_5, qwen3_5_moe | | ||
| | GLM | glm4v, glm4v_moe | | ||
| | Kimi | kimi_vl | | ||
| | Bailing | bailing_moe | |
There was a problem hiding this comment.
The model type bailing_moe is duplicated in both the text-only and multimodal model lists. If this is intended to represent a multimodal variant, it should be renamed for clarity (e.g., to bailing_moe_vl); otherwise, the duplicate entry should be removed.
| | Qwen | qwen2_vl, qwen2_5_vl, qwen2_5_omni<br />qwen3_vl, qwen3_vl_moe, qwen3_omni_moe, qwen3_asr<br />qwen3_5, qwen3_5_moe | | ||
| | GLM | glm4v, glm4v_moe | | ||
| | Kimi | kimi_vl | | ||
| | Bailing | bailing_moe | |
There was a problem hiding this comment.
The model type bailing_moe is duplicated in both the text-only and multimodal model lists. If this is intended to represent a multimodal variant, it should be renamed for clarity (e.g., to bailing_moe_vl); otherwise, the duplicate entry should be removed.
| **multimodal large models** | ||
| | Series | model_type | | ||
| | -------- | ------------------------------------------------------------ | | ||
| | Qwen | qwen2_vl, qwen2_5_vl, qwen2_5_omni<br />qwen3_vl, qwen3_vl_moe, qwen3_omni_moe, qwen3_asr<br />qwen3_5, qwen3_5_moe | |
There was a problem hiding this comment.
The model types qwen3_5 and qwen3_5_moe appear to be text-only models and should likely be listed in the 'text-only large models' table instead of the multimodal one, especially since qwen3_next is correctly placed in the text list. If these refer to multimodal versions, consider renaming them to qwen3_5_vl and qwen3_5_moe_vl for consistency with other multimodal entries like qwen2_vl and qwen3_vl.
|
|
||
| | 系列 | model_type | | ||
| | -------- | ------------------------------------------------------------ | | ||
| | Qwen | qwen2_vl, qwen2_5_vl, qwen2_5_omni<br />qwen3_vl, qwen3_vl_moe, qwen3_omni_moe, qwen3_asr<br />qwen3_5, qwen3_5_moe | |
There was a problem hiding this comment.
The model types qwen3_5 and qwen3_5_moe appear to be text-only models and should likely be listed in the '纯文本模型' table instead of the multimodal one. If these refer to multimodal versions, consider renaming them to qwen3_5_vl and qwen3_5_moe_vl for consistency with other multimodal entries like qwen2_vl and qwen3_vl.
|
/gemini review |
There was a problem hiding this comment.
Code Review
This pull request updates the English and Chinese README files by adding an introduction to the mcore-bridge library, a recommended runtime environment table, and a restructured model list categorized into text-only and multimodal models. Feedback indicates that the Qwen 3.5 models are currently misclassified as multimodal and should be moved to the text-only section. Additionally, several models mentioned in the new introduction, such as GLM 5.1 and InternVL 3.5, are missing from the detailed model tables and need to be added for consistency.
| | Series | model_type | | ||
| | -------- | ------------------------------------------------------------ | | ||
| | Qwen | qwen2, qwen2_moe<br />qwen2_vl, qwen2_5_vl, qwen2_5_omni<br />qwen3, qwen3_moe<br />qwen3_vl, qwen3_vl_moe, qwen3_omni_moe, qwen3_asr<br />qwen3_next, qwen3_5, qwen3_5_moe | | ||
| | Qwen | qwen2, qwen2_moe<br />qwen3, qwen3_moe, qwen3_next | |
There was a problem hiding this comment.
The models qwen3_5 and qwen3_5_moe appear to be misclassified in the multimodal table. Based on the naming convention and their placement in the previous version of the README, they should likely be listed here under text-only models.
| | Qwen | qwen2, qwen2_moe<br />qwen3, qwen3_moe, qwen3_next | | |
| | Qwen | qwen2, qwen2_moe<br />qwen3, qwen3_moe, qwen3_next, qwen3_5, qwen3_5_moe | |
| **multimodal large models:** | ||
| | Series | model_type | | ||
| | -------- | ------------------------------------------------------------ | | ||
| | Qwen | qwen2_vl, qwen2_5_vl, qwen2_5_omni<br />qwen3_vl, qwen3_vl_moe, qwen3_omni_moe, qwen3_asr<br />qwen3_5, qwen3_5_moe | |
There was a problem hiding this comment.
As noted in the text-only table, qwen3_5 and qwen3_5_moe should be moved to the text-only section. Additionally, the introduction mentions Qwen3.5-VL, which is missing from this list if it differs from qwen3_5.
| | Qwen | qwen2_vl, qwen2_5_vl, qwen2_5_omni<br />qwen3_vl, qwen3_vl_moe, qwen3_omni_moe, qwen3_asr<br />qwen3_5, qwen3_5_moe | | |
| | Qwen | qwen2_vl, qwen2_5_vl, qwen2_5_omni<br />qwen3_vl, qwen3_vl_moe, qwen3_omni_moe, qwen3_asr | |
| | Series | model_type | | ||
| | -------- | ------------------------------------------------------------ | | ||
| | Qwen | qwen2, qwen2_moe<br />qwen2_vl, qwen2_5_vl, qwen2_5_omni<br />qwen3, qwen3_moe<br />qwen3_vl, qwen3_vl_moe, qwen3_omni_moe, qwen3_asr<br />qwen3_next, qwen3_5, qwen3_5_moe | | ||
| | Qwen | qwen2, qwen2_moe<br />qwen3, qwen3_moe, qwen3_next | | ||
| | DeepSeek | deepseek_v3, deepseek_v32 | | ||
| | GLM | glm4, glm4_moe, glm4_moe_lite<br />glm4v, glm4v_moe, <br />glm_moe_dsa | | ||
| | GLM | glm4, glm4_moe, glm4_moe_lite<br />glm_moe_dsa | | ||
| | MiniMax | minimax_m2 | | ||
| | Kimi | kimi_k2, kimi_vl, kimi_k25 | | ||
| | Kimi | kimi_k2, kimi_k25 | | ||
| | Bailing | bailing_moe | | ||
| | InternLM | internlm3, internvl_chat, internvl | | ||
| | Ovis | ovis2_5 | | ||
| | Llama | llama, llama4 | | ||
| | InternLM | internlm3 | | ||
| | Llama | llama | | ||
| | GPT-OSS | gpt_oss | | ||
| | Hunyuan | hy_v3 | | ||
| | ERNIE | ernie4_5, ernie4_5_moe | | ||
| | MiMo | mimo | | ||
| | Dots | dots1 | | ||
| | OLMoE | olmoe | | ||
|
|
||
| **multimodal large models:** | ||
| | Series | model_type | | ||
| | -------- | ------------------------------------------------------------ | | ||
| | Qwen | qwen2_vl, qwen2_5_vl, qwen2_5_omni<br />qwen3_vl, qwen3_vl_moe, qwen3_omni_moe, qwen3_asr<br />qwen3_5, qwen3_5_moe | | ||
| | GLM | glm4v, glm4v_moe | | ||
| | Kimi | kimi_vl | | ||
| | InternVL | internvl_chat, internvl | | ||
| | Ovis | ovis2_5 | | ||
| | Llama | llama4 | | ||
| | Llava | llava-onevision | |
There was a problem hiding this comment.
There are several inconsistencies between the models mentioned in the Introduction and those listed in the Model List tables: - Text-only: GLM5.1 and Minimax2.7 are mentioned in the intro but missing from the table. - Multimodal: GLM4.6-V and InternVL3.5 are mentioned in the intro but missing from the table (only glm4v and internvl are listed). Please update the tables to include these models with their respective model_type identifiers.
| | 系列 | model_type | | ||
| | -------- | ------------------------------------------------------------ | | ||
| | Qwen | qwen2, qwen2_moe<br />qwen2_vl, qwen2_5_vl, qwen2_5_omni<br />qwen3, qwen3_moe<br />qwen3_vl, qwen3_vl_moe, qwen3_omni_moe, qwen3_asr<br />qwen3_next, qwen3_5, qwen3_5_moe | | ||
| | Qwen | qwen2, qwen2_moe<br />qwen3, qwen3_moe, qwen3_next | |
There was a problem hiding this comment.
The models qwen3_5 and qwen3_5_moe should be moved here from the multimodal table to maintain consistency with the text-only classification of these base models.
| | Qwen | qwen2, qwen2_moe<br />qwen3, qwen3_moe, qwen3_next | | |
| | Qwen | qwen2, qwen2_moe<br />qwen3, qwen3_moe, qwen3_next, qwen3_5, qwen3_5_moe | |
|
|
||
| | 系列 | model_type | | ||
| | -------- | ------------------------------------------------------------ | | ||
| | Qwen | qwen2_vl, qwen2_5_vl, qwen2_5_omni<br />qwen3_vl, qwen3_vl_moe, qwen3_omni_moe, qwen3_asr<br />qwen3_5, qwen3_5_moe | |
There was a problem hiding this comment.
Remove qwen3_5 and qwen3_5_moe from the multimodal list as they are primarily text models. Also, consider adding qwen3_5_vl if that is the intended multimodal model mentioned in the introduction as Qwen3.5-VL.
| | Qwen | qwen2_vl, qwen2_5_vl, qwen2_5_omni<br />qwen3_vl, qwen3_vl_moe, qwen3_omni_moe, qwen3_asr<br />qwen3_5, qwen3_5_moe | | |
| | Qwen | qwen2_vl, qwen2_5_vl, qwen2_5_omni<br />qwen3_vl, qwen3_vl_moe, qwen3_omni_moe, qwen3_asr | |
| | 系列 | model_type | | ||
| | -------- | ------------------------------------------------------------ | | ||
| | Qwen | qwen2, qwen2_moe<br />qwen2_vl, qwen2_5_vl, qwen2_5_omni<br />qwen3, qwen3_moe<br />qwen3_vl, qwen3_vl_moe, qwen3_omni_moe, qwen3_asr<br />qwen3_next, qwen3_5, qwen3_5_moe | | ||
| | Qwen | qwen2, qwen2_moe<br />qwen3, qwen3_moe, qwen3_next | | ||
| | DeepSeek | deepseek_v3, deepseek_v32 | | ||
| | GLM | glm4, glm4_moe, glm4_moe_lite<br />glm4v, glm4v_moe, <br />glm_moe_dsa | | ||
| | GLM | glm4, glm4_moe, glm4_moe_lite<br />glm_moe_dsa | | ||
| | MiniMax | minimax_m2 | | ||
| | Kimi | kimi_k2, kimi_vl, kimi_k25 | | ||
| | Kimi | kimi_k2, kimi_k25 | | ||
| | Bailing | bailing_moe | | ||
| | InternLM | internlm3, internvl_chat, internvl | | ||
| | Ovis | ovis2_5 | | ||
| | Llama | llama, llama4 | | ||
| | InternLM | internlm3 | | ||
| | Llama | llama | | ||
| | GPT-OSS | gpt_oss | | ||
| | Hunyuan | hy_v3 | | ||
| | ERNIE | ernie4_5, ernie4_5_moe | | ||
| | MiMo | mimo | | ||
| | Dots | dots1 | | ||
| | OLMoE | olmoe | | ||
|
|
||
| **多模态模型：** | ||
|
|
||
| | 系列 | model_type | | ||
| | -------- | ------------------------------------------------------------ | | ||
| | Qwen | qwen2_vl, qwen2_5_vl, qwen2_5_omni<br />qwen3_vl, qwen3_vl_moe, qwen3_omni_moe, qwen3_asr<br />qwen3_5, qwen3_5_moe | | ||
| | GLM | glm4v, glm4v_moe | | ||
| | Kimi | kimi_vl | | ||
| | InternVL | internvl_chat, internvl | | ||
| | Ovis | ovis2_5 | | ||
| | Llama | llama4 | | ||
| | Llava | llava-onevision | |
There was a problem hiding this comment.
Several models mentioned in the Introduction are missing from the Model List tables in this file as well: - Text-only: Missing GLM5.1 and Minimax2.7. - Multimodal: Missing GLM4.6-V and InternVL3.5. Please ensure the Chinese documentation stays in sync with the introduction and the English version.
1 participant
No description provided.