-
Notifications
You must be signed in to change notification settings - Fork 190
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
Upgraded for Python3.6+ and PyTorch 1.1.0+ #25
base: master
Are you sure you want to change the base?
Conversation
from PIL import Image, ImageFont, ImageDraw | ||
from copy import deepcopy | ||
|
||
from miscc.config import cfg | ||
from miscc.utils import mkdir_p | ||
from model import G_NET, D_NET64, D_NET128, D_NET256, D_NET512, D_NET1024, INCEPTION_V3 | ||
|
||
from tensorboardX import FileWriter, summary |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I had to install tensorboardX
dependency from https://github.com/lanpa/tensorboardX, this is not included in dependency section in the README.md
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Gotcha. I had tensorboard
in the README.md not tensorboardX
. Will update that. Thanks
What holds back this PR? |
from PIL import Image, ImageFont, ImageDraw | ||
from copy import deepcopy | ||
|
||
from miscc.config import cfg | ||
from miscc.utils import mkdir_p | ||
from model import G_NET, D_NET64, D_NET128, D_NET256, D_NET512, D_NET1024, INCEPTION_V3 | ||
|
||
from tensorboardX import FileWriter, summary |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would suggest there to be used:
from torch.utils.tensorboard import summary, FileWriter
Upgraded all the main files for Python3.6+ and PyTorch 1.1.0+ usage