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

ICNet Problem #1

Closed
erichhhhho opened this issue Dec 9, 2019 · 6 comments
Closed

ICNet Problem #1

erichhhhho opened this issue Dec 9, 2019 · 6 comments

Comments

@erichhhhho
Copy link

erichhhhho commented Dec 9, 2019

In the low-res branch, the input of the low-res feature map(x_sub4) in your code is from the medium and downsample by 0.2.

But from the original paper, it should be downsampled from the raw input by 0.25, is it?

@lxtGH
Copy link
Owner

lxtGH commented Dec 9, 2019

No, the original code is a litttle different from the figure in the paper. See this issue hszhao/ICNet#7.

@lxtGH lxtGH closed this as completed Dec 9, 2019
@erichhhhho
Copy link
Author

@lxtGH Yes. I also noticed that the original code use element-wise summation instead of the concatenation. Would you mind sharing your trained model performance?

@lxtGH lxtGH reopened this Dec 10, 2019
@lxtGH
Copy link
Owner

lxtGH commented Dec 10, 2019

I will report the results and trained model as soon as possible.

@lxtGH
Copy link
Owner

lxtGH commented Dec 11, 2019

Please use the lastest code for reproducing ICnet results.

@lxtGH lxtGH closed this as completed Dec 11, 2019
@erichhhhho
Copy link
Author

@lxtGH Thank you.
I think the ICNet loss in your code is inaccurate.

When you calculate the loss, according to the paper: the ground truth should be downsampled to the prediction size in 1/4, 1/8, 1/16 branch, instead of upsampling the sub prediction. There is a difference. And I was wondering the mIoU in your trained model is reported in the test set or val set?

@lxtGH
Copy link
Owner

lxtGH commented Dec 12, 2019

You can have a try by not upsampling. and I report the results on val set

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

2 participants