Skip to content

Update information about Localization Distillation#7350

Merged
ZwwWayne merged 3 commits intoopen-mmlab:masterfrom
HikariTJU:ld2
Mar 9, 2022
Merged

Update information about Localization Distillation#7350
ZwwWayne merged 3 commits intoopen-mmlab:masterfrom
HikariTJU:ld2

Conversation

@HikariTJU
Copy link
Copy Markdown
Contributor

@HikariTJU HikariTJU commented Mar 9, 2022

Motivation

Our paper (Localization Distillation for Dense Object Detection) has been accepted, so some info about it might need to be updated.

Modification

Some readme file

Checklist

  1. Pre-commit or other linting tools are used to fix the potential lint issues.
  2. The modification is covered by complete unit tests. If not, please add more unit test to ensure the correctness.
  3. If the modification has potential influence on downstream projects, this PR should be tested with downstream projects, like MMDet or MMCls.
  4. The documentation has been modified accordingly, like docstring or example tutorials.

@mm-assistant mm-assistant bot added the size/XS label Mar 9, 2022
@codecov
Copy link
Copy Markdown

codecov bot commented Mar 9, 2022 •
edited
Loading

Codecov Report

Merging #7350 (004fd4d) into master (b9fe216) will increase coverage by 0.02%.
The diff coverage is n/a.

❗ Current head 004fd4d differs from pull request most recent head 67f9feb. Consider uploading reports for the commit 67f9feb to get more accurate results

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #7350      +/-   ##
==========================================
+ Coverage   63.40%   63.43%   +0.02%     
==========================================
  Files         340      340              
  Lines       27493    27493              
  Branches     4645     4645              
==========================================
+ Hits        17433    17439       +6     
+ Misses       9132     9129       -3     
+ Partials      928      925       -3
Flag Coverage Δ
unittests 63.40% <ø> (+0.01%) ⬆️

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

Impacted Files Coverage Δ
mmdet/core/bbox/assigners/max_iou_assigner.py 72.36% <0.00%> (-1.32%) ⬇️
mmdet/models/roi_heads/mask_heads/maskiou_head.py 89.65% <0.00%> (+2.29%) ⬆️
mmdet/core/post_processing/bbox_nms.py 79.10% <0.00%> (+4.47%) ⬆️
mmdet/core/bbox/samplers/random_sampler.py 80.55% <0.00%> (+5.55%) ⬆️

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 6ddbff8...67f9feb. Read the comment docs.

hhaAndroid
hhaAndroid reviewed Mar 9, 2022
View reviewed changes
@ZwwWayne ZwwWayne merged commit 70f6d9c into open-mmlab:master Mar 9, 2022
zytx121 added a commit to zytx121/mmdetection that referenced this pull request Mar 15, 2022
@zytx121
Revert "Update information about Localization Distillation (open-mmla…
e8a9dc2 
…b#7350)"

This reverts commit 70f6d9c.
zytx121 added a commit to zytx121/mmdetection that referenced this pull request Mar 16, 2022
@zytx121
Revert "Update information about Localization Distillation (open-mmla…
9668575 
…b#7350)"

This reverts commit 70f6d9c.
ZwwWayne pushed a commit that referenced this pull request Jul 18, 2022
@HikariTJU
Update information about Localization Distillation (#7350)
55d60e1 
* update LD info

* update ld info

* update readme_zhcn
ZwwWayne pushed a commit to ZwwWayne/mmdetection that referenced this pull request Jul 19, 2022
@HikariTJU @ZwwWayne
Update information about Localization Distillation (open-mmlab#7350)
d8fabb1 
* update LD info

* update ld info

* update readme_zhcn
SakiRinn pushed a commit to SakiRinn/mmdetection-locount that referenced this pull request Mar 17, 2023
@HikariTJU @SakiRinn
Update information about Localization Distillation (open-mmlab#7350)
596fa26 
* update LD info

* update ld info

* update readme_zhcn
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@hhaAndroid hhaAndroid hhaAndroid left review comments

@ZwwWayne ZwwWayne ZwwWayne approved these changes

Assignees

No one assigned

Labels

size/XS

Projects

Progress of MMDetection v2.23.0
Status: Done

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@HikariTJU @hhaAndroid @ZwwWayne