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

Sync Batch norm not used? #34

Open
dragen1860 opened this issue May 19, 2020 · 1 comment
Open

Sync Batch norm not used? #34

dragen1860 opened this issue May 19, 2020 · 1 comment

Comments

@dragen1860
Copy link

Dear author:
Thanks for your re-implementation, it's helpful!
When I tried to train with multi gpus, I noticed you have not convert the batch norm to synced batch norm. I wonder if you realized it or not? Thank you.

@daa233
Copy link
Owner

daa233 commented May 19, 2020

@dragen1860 Thanks for your attention!

The synced batch norm is not used in this repo. The original tensorflow implementation of deep fill v1 uses only 1 gpu to train the model. I extended this repo to support multiple gpus but not synced batch norm. You are welcomed to contribute the code!

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