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

Provide checkpoints of re-trained DETR3D model #2279

Merged
merged 6 commits into from
Feb 23, 2023

Conversation

ZLTJohn
Copy link
Contributor

@ZLTJohn ZLTJohn commented Feb 15, 2023

All three configs have been trained and the correctness between old mmdet3d and new mmdet3d is tolerable.
Logs files and model checkpoints is available at Baidu Netdisk:

Link: https://pan.baidu.com/s/13XlTxPSeRK21ls9EH28sow
Password:1ydk

@codecov
Copy link

codecov bot commented Feb 15, 2023

Codecov Report

Base: 47.09% // Head: 47.09% // No change to project coverage 👍

Coverage data is based on head (85aadad) compared to base (92b24d9).
Patch has no changes to coverable lines.

❗ Current head 85aadad differs from pull request most recent head c08f5d3. Consider uploading reports for the commit c08f5d3 to get more accurate results

Additional details and impacted files
@@           Coverage Diff            @@
##           dev-1.x    #2279   +/-   ##
========================================
  Coverage    47.09%   47.09%           
========================================
  Files          261      261           
  Lines        21679    21679           
  Branches      3425     3425           
========================================
  Hits         10210    10210           
  Misses       10816    10816           
  Partials       653      653           
Flag Coverage Δ
unittests 47.09% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

Copy link
Contributor

@JingweiZhang12 JingweiZhang12 left a comment

Choose a reason for hiding this comment

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

Note that the PR of refactoring DETR-series has been merged in the latest MMDet and there are some BC breaking about DETRHead. open-mmlab/mmdetection#8763
Please restrict the version of mmdet <= V3.0.0rc5 in README.md and describe this situation.
Or, we'll very appreciate it if you can refactor the DETR3D to be compatible with the latest MMDet.

projects/DETR3D/README.md Show resolved Hide resolved
projects/DETR3D/configs/detr3d_r101_gridmask.py Outdated Show resolved Hide resolved
@ZLTJohn ZLTJohn changed the title Verified training-time correctness Verified DETR3D training-time correctness Feb 15, 2023
@JingweiZhang12
Copy link
Contributor

Please pull the latest MMDet3D.

@JingweiZhang12
Copy link
Contributor

This PR will be merged after the link of models and logs have been updated.

@JingweiZhang12 JingweiZhang12 changed the title Verified DETR3D training-time correctness Provide re-trained checkpoints of DETR3D Feb 15, 2023
@JingweiZhang12 JingweiZhang12 changed the title Provide re-trained checkpoints of DETR3D Provide checkpoints of re-trained DETR3D model Feb 15, 2023
@JingweiZhang12
Copy link
Contributor

The checkpoints and logs have been uploaded to the cloud. The link format in the readme:
https://download.openmmlab.com/mmdetection3d/v1.1.0_models/detr3d/${FILE_NAME}

projects/DETR3D/README.md Outdated Show resolved Hide resolved
projects/DETR3D/README.md Outdated Show resolved Hide resolved
projects/DETR3D/README.md Outdated Show resolved Hide resolved
projects/DETR3D/README.md Outdated Show resolved Hide resolved
@JingweiZhang12
Copy link
Contributor

@ZLTJohn Please pull the latest code and resolve the conflicts.

@ZLTJohn
Copy link
Contributor Author

ZLTJohn commented Feb 20, 2023

Already pulled the latest code and resolved the conflicts.

@ZwwWayne ZwwWayne merged commit 21de1af into open-mmlab:dev-1.x Feb 23, 2023
@ZLTJohn ZLTJohn deleted the detr3d-train branch February 24, 2023 06:49
@ZLTJohn ZLTJohn restored the detr3d-train branch February 24, 2023 06:50
xizaoqu pushed a commit to xizaoqu/mmdetection3d that referenced this pull request Mar 15, 2023
* fix 'custom_imports' in config files

* fix lint

* verified training-time correctness

* Fix detr3d_vovnet submission and update README

* fix README
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