Open
Conversation
e902a21 to
9257bf5
Compare
Owner
|
This is great, I'll try to make time to review this in detail today. |
Author
|
Awesome, thanks a lot :) I made a PR to TensorFlow |
Update bower dependencies. Also force urls to lowercase. Change: 119584968
Goals: - Have enough of each summary type that tag grouping is useful. (Wound up recording e.g. mean and stddev and min/max for each variable) - Use every summary type (adds images) - Write to multiple directories so there are several "runs" Change: 119585022
Author
|
@tmc may I have your approval to use your bindings so the TensorFlow guys can start reviewing the code? |
…n.*`. Change: 119589456
Change: 119591021
Merge internal changes
Usage example: ./remote_test.sh --num-workers 3 --sync-replicas
Also changed:
1) In local and remote tests, let different workers contact separate GRPC
sessions.
2) In local and remote tests, adding the capacity to specify the number of
workers. Before it was hard-coded at 2.
Usage example:
./remote_test.sh --num-workers 2 --sync-replicas
3) Using device setter in mnist_replica.py
Change: 119599547
Merge internal changes
…nsorflow#1864) * Moved mnist reading from input_data in tutorial to learn/datasets
…w#1866) * Update TF Mechanics 101 Tutorial to match recent code In the section of Inputs and Placeholders, the IMAGE_PIXELS should be mnist.IMAGE_PIXELS
Change: 119605636
Change: 119643238
5bace3b to
29b9392
Compare
29b9392 to
adbd4c9
Compare
Adding server/Dockerfile.test to versioning roll
Change: 120184745
Change: 120185825
Change: 120186451
Change: 120186712
Change: 120192253
Change: 120192816
- Oneof values should be printed even when equal to the default. - Fields should be printed in tag number order, not declaration order. Change: 120223509
Change: 120231603
* Removed TensorInt interface * Remove unsupported data types
1889fd8 to
9907cee
Compare
* Fixed some typos
…w#1987) During initialization, we check whether the loaded version of the cudnn library matches the version of the library that the binary was compiled against. When there is a mismatch, throw an error. Example: I built with cudnnv5 but I am loading V4, and I get: E tensorflow/stream_executor/cuda/cuda_dnn.cc:298] Loaded cudnn library: 4007 but source was compiled against 5004. If using a binary install, upgrade your cudnn library to match. If building from sources, make sure the library loaded matches the version you specified during compile configuration. I then copy the v5 version of libcudnn.so into /usr/local/cuda/lib64 and when I re-run the program, the error goes away. Thoughts, @zheng-xq, @leary-google ?
ee330cc to
a9d2ce3
Compare
…#2027) CUDNN_TENSOR_NHWC the only other format is still not supported since kInputYXOutput would correspond to CUDNN_TENSOR_CHWN which doesn’t exist for issue tensorflow#1786
Upstream changes from internal
The keys in the _word2id are bytes literals.
The keys in the _word2id are bytes literals.
* Fixed some typos * Relocated errors after types
2cacc6f to
da22433
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Hi Travis,
I have been working today on some helpers to access to the data contained in the tensors, as also adding some more tests. This is still a work in progress I just doing this PR just to show what I'm working on and to know what do you think, are you working on something similar? Please feel you free to take a look :)
I had been trying to do the binding with Swig and Bazel time ago, but since I have no experience with both of this tools and I had some personal problems I finally gave up. Thanks a lot for taking the time to prepare this.
I think that I could contribute with finishing the helpers to access the data as also to create tensors from Go and so.
By the moment, from the next data, types the ones with the V are supported: