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

[Fix] The interface multiscale_output is defined but not used #830

Merged
merged 3 commits into from
Aug 27, 2021

Conversation

AronLin
Copy link
Contributor

@AronLin AronLin commented Aug 26, 2021

The interface multiscale_output is defined in _make_stage but never used.

HRNet should be allowed to output a one-level feature in stage 4.
So we should use multiscale_output while making stage 4 and add interface multiscale_output in the initialization function of HRNet.

Add unit test for HRNet in this PR, too.

@CLAassistant
Copy link

CLAassistant commented Aug 26, 2021

CLA assistant check
All committers have signed the CLA.

@AronLin
Copy link
Contributor Author

AronLin commented Aug 26, 2021

Kindly ping@Junjun2016

@MengzhangLI
Copy link
Contributor

Kindly ping@Junjun2016

Let me help you @Junjun2016

@Junjun2016
Copy link
Collaborator

Hi @AronLin
Thanks for your contribution.
We will review it ASAP!

@codecov
Copy link

codecov bot commented Aug 26, 2021

Codecov Report

Merging #830 (5944488) into master (7ccb9c3) will increase coverage by 0.04%.
The diff coverage is 80.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #830      +/-   ##
==========================================
+ Coverage   88.73%   88.77%   +0.04%     
==========================================
  Files         107      107              
  Lines        5805     5809       +4     
  Branches      932      933       +1     
==========================================
+ Hits         5151     5157       +6     
+ Misses        462      461       -1     
+ Partials      192      191       -1     
Flag Coverage Δ
unittests 88.75% <80.00%> (+0.04%) ⬆️

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

Impacted Files Coverage Δ
mmseg/models/backbones/hrnet.py 84.37% <80.00%> (+1.04%) ⬆️

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 7ccb9c3...5944488. Read the comment docs.

@Junjun2016
Copy link
Collaborator

Hi @AronLin
Please add more unitests to improve the coverage.

@Junjun2016 Junjun2016 merged commit aa438f5 into open-mmlab:master Aug 27, 2021
bowenroom pushed a commit to bowenroom/mmsegmentation that referenced this pull request Feb 25, 2022
…mlab#830)

* Add interface multiscale_output

* Add space between args and their types

* Fix default value
aravind-h-v pushed a commit to aravind-h-v/mmsegmentation that referenced this pull request Mar 27, 2023
wjkim81 pushed a commit to wjkim81/mmsegmentation that referenced this pull request Dec 3, 2023
* Add unified SMPL-like model interface and builder
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.

4 participants