-
Notifications
You must be signed in to change notification settings - Fork 96
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
PrettyTensor compatibility #4
Comments
Great thanks! I've since rewritten a bunch of this to do a number of things like getting rid of prettytensor, implementing deepmind's up-convolutions tweaks, but need to clean up the code, and reapply it to the faces dataset. I will try and update the repo as soon as I get a little bit of time! |
Is there any chance of the updated code being uploaded soon? |
Hi all, sorry I've been a bit swamped with research the past few months. I don't quite have time to rebuild and debug, but here are some quick updates you can implement. The examples here are from a different network with a slightly different architecture, so a few changes would be needed. To get rid of pretty tensor in layer creation, replace the pt.wrap layers with tf.layers layers. For example:
To use the resize deconvolutions discussed in here, replace deconv with something like this
In inference, instead of
You can use arg scope:
If anyone implements this stuff we can pull in the new version! Sorry again for not updating sooner! |
The issues I was having were related to the Thanks for your response! |
Does anyone implement this |
sorry, have u solve this problem? |
Really nice repo but appears prettytensor support is somewhat broken due to fast changing TF api
Some issues similar to google/prettytensor#46
And the data is a bit harder to locate now due to downed links. The attributes labels can be found here at least: https://s3.amazonaws.com/cadl/celeb-align/list_attr_celeba.txt
The text was updated successfully, but these errors were encountered: