-
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 all 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 @@ | ||
| theme: jekyll-theme-cayman |
| 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