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

v4 det cml configs #11258

Merged
merged 2 commits into from
Nov 17, 2023
Merged

v4 det cml configs #11258

merged 2 commits into from
Nov 17, 2023

Conversation

sylarwcy
Copy link
Contributor

@sylarwcy sylarwcy commented Nov 15, 2023

PR 类型 PR types

Bug fixes

PR 变化内容类型 PR changes

Others

描述 Description

  1. 修复了找不到模型的问题,原因是PPLCNetNew不在可选模型名称的列表内,将PPLCNetNew改为PPLCNetV3;
  2. 修复了db_fpn.py解析in_channels的报错问题,db_fpn.py中按序列解析in_channels,而lcnetv3.py中det为false时输出是数值,为true才返回序列。在两个student模型的backbone中添加"det: true";
  3. 修复了训练过程中eval时的显存溢出问题。通过限制过大的测试数据可解决该问题,具体调整是,在eval→DetResizeForTest的配置中增加"limit_side_len: 960,limit_type: max"。

提PR之前的检查 Check-list

  • 这个 PR 是提交到dygraph分支或者是一个cherry-pick,否则请先提交到dygarph分支。
    This PR is pushed to the dygraph branch or cherry-picked from the dygraph branch. Otherwise, please push your changes to the dygraph branch.
  • 这个PR清楚描述了功能,帮助评审能提升效率。This PR have fully described what it does such that reviewers can speedup.
  • 这个PR已经经过本地测试。This PR can be convered by current tests or already test locally by you.

1. 修复了找不到模型的问题,原因是PPLCNetNew不在可选模型名称的列表内,将PPLCNetNew改为PPLCNetV3;
2. 修复了db_fpn.py解析in_channels的报错问题,db_fpn.py中按序列解析in_channels,而lcnetv3.py中det为false时输出是数值,为true才返回序列。在两个student模型的backbone中添加"det: true";
3. 减少CPU占用,将cal_metric_during_train: true 改为 false;
4. 修复了训练过程中eval时的显存溢出问题。通过限制过大的测试数据可解决该问题,具体调整是,在eval→DetResizeForTest的配置中增加"limit_side_len: 960,limit_type: max"。
Copy link
Collaborator

@tink2123 tink2123 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@tink2123 tink2123 merged commit 68b3842 into PaddlePaddle:dygraph Nov 17, 2023
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants