irfanICMLL / structure_knowledge_distillation

The official code for the paper 'Structured Knowledge Distillation for Semantic Segmentation'. (CVPR 2019 ORAL) and extension to other tasks.
BSD 2-Clause "Simplified" License
702 stars 104 forks source link

Hello, I also cannot reproduce the results #38

Closed DanceInDark closed 3 years ago

DanceInDark commented 4 years ago

I try to reproduce the results with the code of PAMI version, but I got similar results with https://github.com/irfanICMLL/structure_knowledge_distillation/issues/22#issue-576083600. It seems that there is some problem. Could you please tell me how to fix it and get the right results? And could you offer me a train file so I can train with the code of cvpr version. Many thanks.

irfanICMLL commented 4 years ago

Use this branch: ce208e1. Following the instructions step by step. You will get the same results with GAN at about 74~75 mIoU. You can get the training log from this link.

https://cloudstor.aarnet.edu.au/plus/s/H2mh8G0xyreZLni

happycoding1996 commented 4 years ago

@irfanICMLL The PAMI paper writes that the lambda values for pi and pa are both 10.0, but the training log and the default parameters in this repo show that the lambda_pa is set to 0.5.

Could you clarify which parameter set we should use to reproduce the results?

irfanICMLL commented 4 years ago

If you want to reproduce the results of the PAMI version, please use the hyperparameters in this repo.

yiia commented 3 years ago

@irfanICMLL I have the same confusion about lambda values. In your answer above, you mentioned "use the hyperparameters in this repo." and, the lambda_pa is set to 1 in this repo. At the same time, I also noticed that setting different lambdas will get different miou. However, the description in the paper is "where λ1 and λ2 are set as 10 and 0.1, making these loss value ranges comparable." Q1: I don't understand this description very well. Could you please tell me in detail? Q2: I want to know how to set the value of lambda according to the actual situation in the experiment. Hope to get your reply soon, thank you very much!

yiia commented 3 years ago

@irfanICMLL The PAMI paper writes that the lambda values for pi and pa are both 10.0, but the training log and the default parameters in this repo show that the lambda_pa is set to 0.5.

Could you clarify which parameter set we should use to reproduce the results? @happycoding1996 Have you figured out the problem of this lambda setting?

irfanICMLL commented 3 years ago

The current implementation is a little sensitive to the loss weight. If you want to re-implement the experiments. Please follow the default setting in this repo to set the lambda_pa to 0.5.