已合并
Add epoch parameter support for BEVFusion training script. #1973
xiangyuming创建于 4月7日
Add epoch parameter support for BEVFusion training script. #1973
已合并
xiangyuming创建于 4月7日
xiangyuming
xiangyuming成员
4月7日

What this PR does / why we need it?

新增bevfusion训练脚本支持epoch输入

Does this PR introduce any user-facing change?

脚本中新增epoch输入

How was this patch tested?

已自测

likedislike
Pull Request已成功合入, 合并人@ascend-robot
(感谢 xiangyuming 的贡献)
xiangyumingxiangyuming成员
4月7日 创建了 pull request,commit 831d8bb6
xiangyumingxiangyuming成员
4月7日 关联了issue:[Bug]: epochs参数硬编码影响模型训练效果
ascend-robot
ascend-robot成员
4月7日 评论:

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/DrivingSDK ✅ 华郁秀, gitdzy (2/2) ✅ 华郁秀 (1/1)

💡 Tip:

  • Committer can comment /approve or /lgtm
  • Commenting /approve implies both code review (lgtm) and intent to merge (approve)

CLA Signature Pass

zjuxym, thanks for your pull request. All authors of the commits have signed the CLA. 👍

likedislike
ascend-robotascend-robot成员
4月7日 添加了label:ascend-cla/yes
ascend-robot
ascend-robot成员
4月7日 评论:

问题/功能描述

本次PR为BEVFusion模型训练脚本新增了--epochs命令行参数,旨在解决原有脚本中训练轮数固定、无法灵活调整的问题。通过此修改,用户可以在启动训练时自定义训练总轮数,从而增强了脚本的易用性和灵活性。

修改方案描述

修改方案主要包括两部分:一是更新了README文档,在相关章节补充了--epochs参数的说明及其默认值(6)。二是修改了FP16和FP32精度的全量训练Shell脚本,在参数解析部分接收epochs参数,并通过--cfg-options将其动态配置到训练配置(train_cfg.max_epochs)及相关数据增强流水线中,从而实现对训练轮数的控制。

likedislike
xiangyuming
xiangyuming成员
4月7日 评论:

compile

likedislike
ascend-robotascend-robot成员
4月7日 添加了label:ci-pipeline-running
ascend-robotascend-robot成员
4月7日 删除了label:ci-pipeline-running
ascend-robotascend-robot成员
4月7日 添加了label:ci-pipeline-passed
ascend-robot
ascend-robot成员
4月7日 评论:
流水线 PR-pipeline_DrivingSDK#905 已完成
阶段 任务名 状态 详情
编译构建 Build_x86 ✅ >>>
Build_ARM ✅ >>>
恶意代码检查 Antipoison ✅ >>>
编码安全与规范检查 CodeCheck ✅ >>>
开源片段检查 SCA ✅ >>>
开发者测试 UT_ARM ✅ >>>
流水线 PR-pipeline_DrivingSDK ✅ >>>
likedislike
xiangyumingxiangyuming成员
4月8日 预合并成功(commit_id: aaafa3ec6f0db5f724e604efa3517c29e81ca398)
xiangyumingxiangyuming成员
4月8日 强制推送  1 个提交:28c2b61b-bev
xiangyumingxiangyuming成员
4月8日 预合并成功(commit_id: ad0133b6839baba13af07b4c09dc0c84a4fdb682)
ascend-robotascend-robot成员
4月8日 删除了label:ci-pipeline-passed
ascend-robot
ascend-robot成员
4月8日 评论:

Notification

This pull request source branch has changed, so removes the following label(s): ci-pipeline-passed.

likedislike
AtlasAccountAtlasAccount成员
4月8日 添加了label:ci-pipeline-failed
xiangyuming
xiangyuming成员
4月8日 评论:

compile

likedislike
ascend-robot
ascend-robot成员
4月8日 评论:

Ascend docs pipeline is running...

likedislike
ascend-robotascend-robot成员
4月8日 添加了label:docs-ci-pipeline-running
ascend-robot
ascend-robot成员
4月8日 评论:

❌ 文档门禁未通过!

检查项 检查结果 详情
markdownlint ❌ 未通过 查看详情
link-validity-check ✅ 已通过 查看详情
resource-existence-check ✅ 已通过 查看详情
tag-closed-check ✅ 已通过 查看详情

💡 本次检查出 1 项错误,详情内容如下:

序号 错误详情
1 [文件路径]:model_examples/BEVFusion/README.md
[检查类型]:markdownlint
[错误位置]:第 72 行
[错误信息]:MD029:有序列表项前的数字请按顺序递增。期望:1;当前:2; Style: 1/2/3
[错误内容]:2. 源码安装`mmdetection3d v1.2.0`版本
likedislike
ascend-robotascend-robot成员
4月8日 删除了label:docs-ci-pipeline-running
ascend-robotascend-robot成员
4月8日 添加了label:docs-ci-pipeline-failed
ascend-robotascend-robot成员
4月8日 删除了label:ci-pipeline-failed
ascend-robotascend-robot成员
4月8日 添加了label:ci-pipeline-running
ascend-robotascend-robot成员
4月8日 删除了label:ci-pipeline-running
ascend-robotascend-robot成员
4月8日 添加了label:ci-pipeline-passed
ascend-robot
ascend-robot成员
4月8日 评论:
流水线 PR-pipeline_DrivingSDK#916 已完成
阶段 任务名 状态 详情
编译构建 Build_x86 ✅ >>>
Build_ARM ✅ >>>
恶意代码检查 Antipoison ✅ >>>
编码安全与规范检查 CodeCheck ✅ >>>
开源片段检查 SCA ✅ >>>
开发者测试 UT_ARM ✅ >>>
流水线 PR-pipeline_DrivingSDK ✅ >>>
likedislike
jayhua
jayhua成员
4月8日 评论:

/approve

likedislike
ascend-robotascend-robot成员
4月8日 添加了label:approved
xiangyuming
xiangyuming成员
4月9日 评论:

/check-pr

likedislike
ascend-robot
ascend-robot成员
4月9日 评论:

The following label is not ready.

lgtm: Please wait for reviewers to review the code.

likedislike
gitdzy
gitdzy成员
4月9日 评论:

/lgtm

likedislike
ascend-robotascend-robot成员
4月9日 添加了label:lgtm
ascend-robot
ascend-robot成员
4月9日 评论:

Review Guide

This pull-request passes review.
Committers who wrote a comment of /approve are: 华郁秀.
Reviewers who wrote a comment of /lgtm are: 华郁秀, gitdzy.

likedislike
ascend-robotascend-robot成员
4月9日 关闭了关联的issue
ascend-robotascend-robot成员
4月9日 合入了pull request
ascend-robot
ascend-robot成员
4月14日 评论:

✅ 跳过 docs ci 检查,没有需要检查的文档文件

likedislike
ascend-robotascend-robot成员
4月14日 删除了label:docs-ci-pipeline-failed
ascend-robotascend-robot成员
4月14日 添加了label:docs-ci-pipeline-success