-
Notifications
You must be signed in to change notification settings - Fork 43
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
high res version #10
Comments
Hi, Yes we did try one version for 256x256 resolution, and it was working decently well. You can simply add more upsampling layers towards the end of all the generator modules. |
@utkarshojha
But if generator outputs size become twice, discriminator inputs and real_imgs size also should be twice (otherwise it causes size mismatch error), right? Though I tried to do that, I couldn't get satisfactory results as below. fake_imgs[0] (background stage)fake_imgs[1] (parent stage)fake_imgs[2] (child stage)It seems that bounding box process doesn't work well and disentaglement of the background fails. |
One problem which could have been there in your implementation of the 256x256 version would be at your background stage. We use PatchGAN at the background stage, due to which we need to define the values of some hyperparameters, which are defined in lines 381-383 of trainer.py. These parameters are needed to accurately extract patches lying outside the bounding box. For 256x256 version, the updated values of those parameters would be: My version of 256x256 isn't cleaned up, so I don't think it will be helpful to you. You should try the correction I mentioned and let me know if it works. If not then I can further look into it. |
@utkarshojha fake_imgs[0] (background stage)fake_imgs[1] (parent stage)fake_imgs[2] (child stage)And this is my changes. Could you check this? |
Hi kidach1, Thanks |
@utkarshojha Are the channel sizes of G and D (this and this line) same as yours? |
Thank you for sharing.
Did you try high res ver (like 256x or 512x)?
If not, what difficulties can be considered?
The text was updated successfully, but these errors were encountered: