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

Inconsistent with paper presentation #3

Closed
zhanghengdev opened this issue Apr 19, 2019 · 5 comments
Closed

Inconsistent with paper presentation #3

zhanghengdev opened this issue Apr 19, 2019 · 5 comments

Comments

@zhanghengdev
Copy link

很棒!但是在看了源码以后, 发现好像代码和文章配图的网络结构不符?我画了一个简图,在生成一个分辨率更低的branch的时候只使用了上一个stage分辨率最接近的feature maps?
HRNet

期待您的回复

@wondervictor
Copy link
Member

Hi, in our implementation, we split the last fusion in each stage into two parts: fusion and transition. We append a fusion module after every HighResolutionModule. After fusing features from different resolutions, we then obtain the new branch by the last branch. Actually, the figure presented in our paper is consistent with our implementation.
Thanks for your attention : )

@zhanghengdev
Copy link
Author

zhanghengdev commented Apr 19, 2019

Thanks a lot for your explanation! ^^ What you mean is that even though the new branch comes from one branch, but this branch contains information about different resolutions from the previous stage, is that right? And could you take a little time and check if my plot is correct, please? I am a pytorch beginner.

@sunke123
Copy link
Member

@ZHANGHeng19931123
Hi Heng, your plot is more accurate.
In our paper, we emphasize that the output representations on different resolutions aggregate all the input representations. The two implementations are almost the same. You can check that.
fusion

Hope this help you. If you have any question, welcome to contact us.

@zhanghengdev
Copy link
Author

Thanks a lot! It's clear to me now 👍

@shui-tian-ju-shi
Copy link

Thanks a lot!

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

No branches or pull requests

4 participants