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

question about results #8

Closed
Ha0Tang opened this issue May 2, 2020 · 6 comments
Closed

question about results #8

Ha0Tang opened this issue May 2, 2020 · 6 comments

Comments

@Ha0Tang
Copy link

Ha0Tang commented May 2, 2020

Hi, why your results in Table 2 (cityspaces and ade) different from those from SPADE paper while you used the same dataset train/test splits? For instance, results of SPADE on cityscapes are 62.3 mIoU, 81.9 accu and 71.8 FID, but you reported 57.88 mIoU, 93.59 accu and 50.38 FID, respectively.

@Ha0Tang Ha0Tang changed the title question question about results May 2, 2020
@ZPdesu
Copy link
Owner

ZPdesu commented May 3, 2020

There are several reasons.

The results reported in SPADE only use segmentation masks as input.

The SPADE results reported in SEAN paper are tested on reconstruction images. Instance maps and style images(GT images) are used. The network are trained 50 epochs on cityscapes and ade.

Also, the author of SPADE did not provide the evaluation codes. They only mentioned where to download the model and pre-trained weights in their repo issues. We have retested these results and they are also different from what they reported in the paper. Therefore, we use the results of our fair comparison on these reconstruction task.

@Ha0Tang
Copy link
Author

Ha0Tang commented May 3, 2020

So, you need to input both 'Label' and 'Ground Truth' for generating your results in Fig. 6 of your paper, am I right?

@ZPdesu
Copy link
Owner

ZPdesu commented May 3, 2020

You are right.

@Ha0Tang
Copy link
Author

Ha0Tang commented May 3, 2020

The results of both Pix2pixHD and SPADE in Fig. 6 are also generated by using both 'Label' and 'Ground Truth'?

@ZPdesu
Copy link
Owner

ZPdesu commented May 3, 2020

Yes. Both of these frameworks have their own style encoders.

@Ha0Tang
Copy link
Author

Ha0Tang commented May 3, 2020

I see. Thanks.

@Ha0Tang Ha0Tang closed this as completed May 3, 2020
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