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

SSD examples based on MindCV #704

Merged
merged 78 commits into from
Aug 10, 2023
Merged

SSD examples based on MindCV #704

merged 78 commits into from
Aug 10, 2023

Conversation

DexterJZ
Copy link
Contributor

@DexterJZ DexterJZ commented Jul 13, 2023

Implement SSD object detector based on MindCV backbones (MobileNetV2, ResNet50, MobileNetV3)

Thank you for your contribution to the MindCV repo.
Before submitting this PR, please make sure:

Motivation

(Write your motivation for proposed changes here.)

Test Plan

(How should this PR be tested? Do you require special setup to run the test or repro the fixed bug?)

Related Issues and PRs

Issue #4

(Is this PR part of a group of changes? Link the other relevant PRs and Issues here. Use https://help.github.com/en/articles/closing-issues-using-keywords for help on GitHub syntax)

DexterJZ and others added 30 commits May 5, 2023 15:10
@DexterJZ DexterJZ changed the title SSD examples based on MindCV (in development) SSD examples based on MindCV (in progress) Jul 13, 2023
@DexterJZ DexterJZ self-assigned this Jul 13, 2023
@DexterJZ DexterJZ changed the title SSD examples based on MindCV (in progress) SSD examples based on MindCV Aug 1, 2023
@geniuspatrick
Copy link
Collaborator

  1. Callback可以复用仓里已有的吗?
  2. 数据集是和模型绑定的还是和数据集绑定的?如果之和coco相关,是不是可以抽象到包内,以便后续检测模型同样能够使用。
  3. 文件不必以ssd作为前缀
  4. 考虑讲一部分逻辑进行解耦,只将与模型强相关的,放到example里,其他的可以放在包里,以作后续复用

@DexterJZ
Copy link
Contributor Author

DexterJZ commented Aug 2, 2023

Files renamed. Other tasks are suggested to be marked and left for the future.

@geniuspatrick geniuspatrick merged commit 1fb2453 into mindspore-lab:main Aug 10, 2023
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants