Skip to content

The required dependency is tensorboard-pytorch and not tensorboard #5

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

The required dependency is tensorboard-pytorch and not tensorboard #5

wants to merge 1 commit into from

Conversation

JosephKJ
Copy link

@JosephKJ JosephKJ commented Apr 7, 2018 •
edited
Loading

The dependency while setting up StackGan++ is tensorboard-pytorch and not tensorboard.

There is no FileWriter name in the original tensorboard. Hence
from tensorboard import FileWriter in trainer.py will throw an error.

I got past the error by using tensorboard-pytorch
@JosephKJ
The required dependency is tensorboard-pytorch and not tensorboard
d1a74e8 
The dependency while setting up StackGan++ is tensorboard-pytorch and not tensorboard. There is no FileWriter name in the original tensorboard. Hence ```from tensorboard import FileWriter``` in trainer.py will throw an error.
@akanimax
Copy link

Damn! I spent around an hour trying to get around this. Searched the tensorflow/tensorboard repo for a module named FileWriter but all I found was tf.summary.FileWriter.
This pr should be merged 😅
@happytaoxiaoli happytaoxiaoli mentioned this pull request Jun 7, 2018
Open
hughkf referenced this pull request in mazzaAnt/StackGAN-v2 Nov 27, 2018
@hughkf
fix for this code: "imgs = encoder(self.fake_imgs[0])", need to load …
0af6faf 
…in all imgs with a loop #5
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
2 participants
@JosephKJ @akanimax