已合并
修复 qwen25vl_7B mbridge 特性报错的问题 #2906
bonjour_rzq1998创建于 20 天前
修复 qwen25vl_7B mbridge 特性报错的问题 #2906
已合并
bonjour_rzq1998成员
20 天前
20 天前
What this PR does / why we need it?
1、修改bridge/models/qwen_vl/__init__.py中对于异常情况的容忍范围。
Does this PR introduce any user-facing change?
不涉及
How was this patch tested?
不涉及
Pull Request已成功合入, 合并人@ascend-robot
（感谢 bonjour_rzq1998 的贡献）20 天前 创建了 pull request，commit 4eae3a6c
atomgit-bot
20 天前 评论：
20 天前 评论：
变更摘要
本 PR 修复了 qwen25vl_7B 模型在使用 mbridge 特性时因 fla_npu 依赖缺失导致的错误。通过在 bridge/models/qwen_vl/__init__.py 中将硬编码的单一可选依赖前缀 "mindspeed.te" 扩展为一个可配置的元组 known_optional_deps，并同时纳入 "fla_npu"，使得缺少这些可选依赖时不再抛出异常，而是被正确识别为可容忍的导入故障。
主要改动
- 新增可选依赖名常量元组: 在
bridge/models/qwen_vl/__init__.py中定义known_optional_deps = ("mindspeed.te", "fla_npu")，将原先硬编码的单个依赖前缀抽离为可扩展的元组，便于后续维护。 - 扩展
startswith匹配范围: 将exc_name.startswith("mindspeed.te")改为exc_name.startswith(known_optional_deps)，使得exc_name以"mindspeed.te"或"fla_npu"开头时均被识别为可选依赖缺失，从而避免因缺少fla_npu而导致的误报错误。
atomgit-bot
20 天前 评论：
20 天前 评论：
20 天前 添加了label：ascend-cla/yes
ascend-robot
20 天前 评论：
20 天前 评论：
ascend-robot
20 天前 评论：
20 天前 评论：
Thanks for your pull-request.
The full list of commands accepted by me can be found at here。
You can get sig-info at here
PR Approval Progress
✅ Congratulations! All modules have met the lgtm and approve requirements.
Module Approval Details
| module | lgtm status | approve status |
|---|---|---|
| repo-Ascend/MindSpeed-MM | ✅ 王海涛, gaojie_, young256, ye_qm (4/2) | ✅ 王海涛 (1/1) |
💡 Tip:
- Committer can comment
/approveor/lgtm- Commenting
/approveimplies both code review (lgtm) and intent to merge (approve)
ascend-robot
20 天前 评论：
20 天前 评论：
ascend docs pipeline is running...
20 天前 添加了label：docs-ci-pipeline-running
ascend-robot
20 天前 评论：
20 天前 评论：
✅ 跳过 docs ci 检查，没有需要检查的文档文件
20 天前 删除了label：docs-ci-pipeline-running
20 天前 添加了label：docs-ci-pipeline-success
ascend-robot
20 天前 评论：
20 天前 评论：
ascend docs pipeline is running...
20 天前 删除了label：docs-ci-pipeline-success
20 天前 添加了label：docs-ci-pipeline-running
ascend-robot
20 天前 评论：
20 天前 评论：
✅ 跳过 docs ci 检查，没有需要检查的文档文件
20 天前 删除了label：docs-ci-pipeline-running
20 天前 添加了label：docs-ci-pipeline-success
20 天前 修改了pull request 的描述
bonjour_rzq1998
20 天前 评论：
20 天前 评论：
compile
20 天前 添加了label：ci-pipeline-running
ascend-robot
20 天前 评论：
20 天前 评论：
ascend docs pipeline is running...
20 天前 删除了label：docs-ci-pipeline-success
20 天前 添加了label：docs-ci-pipeline-running
ascend-robot
20 天前 评论：
20 天前 评论：
✅ 跳过 docs ci 检查，没有需要检查的文档文件
20 天前 删除了label：docs-ci-pipeline-running
20 天前 添加了label：docs-ci-pipeline-success
20 天前 删除了label：ci-pipeline-running
20 天前 添加了label：ci-pipeline-passed
ascend-robot
20 天前 评论：
20 天前 评论：
流水线 PR-pipeline_MindSpeed-MM#4338 [ commitID：bd3821dc ] 已完成
>>>代码风格自动修复执行成功（无修复内容）
| 阶段 | 任务名 | 状态 | 详情 |
|---|---|---|---|
| 编译构建 | Build | ✅ | >>> |
| 恶意代码检查 | Antipoison | ✅ | >>> |
| 编码安全与规范检查 | Only_doc_commit | ✅ | >>> |
| codeCheck_pre-commit | ✅ | >>> | |
| pre-commit | ✅ | >>> | |
| Compatibility_Interface | ✅ | >>> | |
| 开源片段检查 | SCA | ✅ | >>> |
| 开发者测试 | UT | 🛑 | >>> |
| ST | 🛑 | >>> | |
| UT-pool | ✅ | >>> | |
| ST_pool | ✅ | >>> | |
| 流水线 | PR-pipeline_MindSpeed-MM | ✅ | >>> |
- compile : 运行流水线
- retry : 重试流水线所有失败子任务
- retry <任务名> : 仅重试指定失败子任务
- stop : 停止流水线
gaojie
20 天前 评论：
20 天前 评论：
验证无问题
gaojie
20 天前 评论：
20 天前 评论：
/lgtm
20 天前 添加了label：lgtm
young256
20 天前 评论：
20 天前 评论：
/lgtm
20 天前 修改了pull request 的描述
20 天前 修改了pull request 的描述
20 天前 添加了label：approved
20 天前 合入了pull request