Skip to content

Conversation

filaPro
Copy link
Contributor

@filaPro filaPro commented Jun 14, 2022

Motivation

Revert #1548 as it didn't fix any bugs.

Modification

Remove DepthInstance3DBoxes.overlaps.

BC-breaking (Optional)

Yes but it is a bug fix only in dev branch.

@codecov
Copy link

codecov bot commented Jun 14, 2022

Codecov Report

Merging #1556 (e68949b) into dev (7d0011d) will decrease coverage by 0.02%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##              dev    #1556      +/-   ##
==========================================
- Coverage   50.19%   50.16%   -0.03%     
==========================================
  Files         223      223              
  Lines       19113    19104       -9     
  Branches     3136     3134       -2     
==========================================
- Hits         9594     9584      -10     
- Misses       8940     8941       +1     
  Partials      579      579              
Flag Coverage Δ
unittests 50.16% <ø> (-0.03%) ⬇️

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

Impacted Files Coverage Δ
mmdet3d/core/bbox/structures/depth_box3d.py 88.67% <ø> (+5.20%) ⬆️
mmdet3d/datasets/pipelines/transforms_3d.py 87.39% <0.00%> (-1.15%) ⬇️
mmdet3d/datasets/pipelines/test_time_aug.py 51.04% <0.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7d0011d...e68949b. Read the comment docs.

@ZwwWayne ZwwWayne merged commit dea954e into open-mmlab:dev Jun 14, 2022
@filaPro filaPro deleted the revert_depth_boxes_overlaps branch June 14, 2022 13:41
@filaPro filaPro mentioned this pull request Jul 28, 2022
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.

2 participants