-
Notifications
You must be signed in to change notification settings - Fork 58
Add Windows Support to tensorboard #27
base: master
Are you sure you want to change the base?
Changes from 7 commits
b723a75
5e3b42c
a004cd5
01b5de4
6975f9b
3ac6d8a
8cd98d2
66fbab2
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,30 @@ | ||
| #!/usr/bin/env bash | ||
|
|
||
| # make protobufs for logging part first | ||
| make all | ||
|
|
||
| # get tensorflow | ||
| git clone https://github.com/tensorflow/tensorflow | ||
| cd tensorflow | ||
|
|
||
| # run configuration. | ||
| bash configure --prefix=/mingw | ||
|
|
||
| # build tensorboard | ||
| #bazel build tensorflow/tensorboard:tensorboard | ||
| bazel build -c opt --cpu=x64_windows_msvc --host_cpu=x64_windows_msvc --copt="/w" --verbose_failures tensorflow/tensorboard:tensorboard | ||
|
|
||
| # prepare pip installation package | ||
| cp -r ../tools/* bazel-bin/tensorflow/tools/ | ||
|
|
||
| # get .whl file in python/dist/ | ||
| bash bazel-bin/tensorflow/tools/pip_package/build_pip_package.sh ../python/dist/ | ||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. In |
||
|
|
||
| # install tensorboard package from .whl file | ||
| cd .. | ||
| pip install python/dist/*.whl | ||
|
|
||
| # clean up | ||
| echo 'Now you can remove tensorflow with rm -rf tensorflow' | ||
| #rm -rf tensorflow/ | ||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -183,7 +183,7 @@ def image(tag, tensor): | |
| def make_image(tensor, height, width, channel): | ||
| """Convert an numpy representation image to Image protobuf""" | ||
| image = Image.fromarray(tensor) | ||
| output = StringIO.StringIO() | ||
| output = StringIO() # This is working for python 2.7 | ||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Why change this? Any error in the origin code?
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I use python 2.7 and it's ok.
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I am using Anaconda python 2.7. I got errors with this error
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Let me test this in my env, thanks. |
||
| image.save(output, format='PNG') | ||
| image_string = output.getvalue() | ||
| output.close() | ||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1 @@ | ||
| Windows 10 tensorboard build output |
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.
Why prefix here?
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.
This the only thing worked with me. Without this, I got bazel build errors.
See Reference