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 patchGAN #33

Closed
SJTUzhanglj opened this issue Dec 16, 2016 · 10 comments
Closed

Question about patchGAN #33

SJTUzhanglj opened this issue Dec 16, 2016 · 10 comments

Comments

@SJTUzhanglj
Copy link

I read your paper and the implementation. The methods about patchGAN described in your paper seems promising, but how it is used in your code i wonder. You preprocess the data into patches before loading or some other way?

@ppwwyyxx
Copy link

patchGAN is still a convnet..
It's implemented in the code here.

@junyanz
Copy link
Collaborator

junyanz commented Dec 16, 2016

Yes, we use fully convolutional networks (FCN) as our disciminator, which is more efficient than patch-wise training/test. See here for more details about FCN.

@SJTUzhanglj
Copy link
Author

Thanks for your quick reply.

@danmic
Copy link

danmic commented Mar 15, 2017

@junyanz I was wondering how you get 70x70 patches using the configuration from the paper.

@phillipi
Copy link
Owner

As mentioned in #25, you can calculate the receptive field size of a layer using https://github.com/rbgirshick/rcnn/blob/master/utils/receptive_field_sizes.m.

@danmic
Copy link

danmic commented Mar 16, 2017

@phillipi Thank you, it is clear now.

@programath
Copy link

@danmic Can you tell me how to get 70x70 result? I find the output size of discriminator is 30x30 and by using the equation mentioned above input_size=(output_size - 1)*stride+ksize, I just cannot get the correct result.

@phillipi
Copy link
Owner

@programath I modified the rcnn script to apply to our architecture:
https://github.com/phillipi/pix2pix/blob/master/scripts/receptive_field_sizes.m

You can see exactly how we calculate the receptive field sizes in that file. Please let me know if you see any issue with the calculation.

@phillipi
Copy link
Owner

phillipi commented Apr 22, 2017 via email

@xmyqsh
Copy link

xmyqsh commented Apr 22, 2017 via email

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

7 participants