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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Refactor darknet backbone #41

Closed
4 tasks done
zhiqwang opened this issue Jan 26, 2021 · 2 comments 路 Fixed by #33
Closed
4 tasks done

Refactor darknet backbone #41

zhiqwang opened this issue Jan 26, 2021 · 2 comments 路 Fixed by #33
Labels
API Library use interface enhancement New feature or request

Comments

@zhiqwang
Copy link
Owner

zhiqwang commented Jan 26, 2021

馃殌 Feature

Refactor darknet backbone as separate modules

Motivation

To make the model structure more controllable, I decided to abandon the way of loading model with a yaml config files as Ultralytics and refactor the darknet backbone as separate modules.

Especially, the parse_model() as below containing the DarkNet, BackboneWithPAN, and PathAggregationNetwork modules, so first I will make this modules more visible and clear.

https://github.com/zhiqwang/yolov5-rt-stack/blob/9104887c4cf10061585eb5b8982270790437013d/models/backbone.py#L79

Then, this modifications would change the keys of checkpoint in state_dict, it's also necessary to revise the checkpiont update scripts.

Pitch

  • Refactor DarkNet
  • Refactor BackboneWithPAN
  • Refactor PathAggregationNetwork
  • Modify checkpoint update scripts

Alternatives

I found several works on this, like:

Additional context

Also follow the philosophy of torchvision especially for:

@zhiqwang zhiqwang added the enhancement New feature or request label Jan 26, 2021
@zhiqwang
Copy link
Owner Author

zhiqwang commented Jan 26, 2021

The model structure can be viewed as following:

yolov5

Copyright statement 漏: This article is the original article of the blogger and follows the CC 4.0 BY-SA copyright agreement. Please attach the original source link and this statement for reprinting.
Link to this article: https://blog.csdn.net/Q1u1NG/article/details/107511465

@zhiqwang
Copy link
Owner Author

zhiqwang commented Mar 28, 2022

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
API Library use interface enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant