diff options
| author |  Zongheng Yang <zongheng@google.com> | 2016-09-01 07:32:38 -0800 |
|---|---|---|
| committer |  TensorFlower Gardener <gardener@tensorflow.org> | 2016-09-01 08:46:38 -0700 |
| commit | e146dc63437d23acf4e221350d9a52a2599f2975 (patch) | |
| tree | 3fdd744d23b3b20f9a26dc3f8dc2c4b330612acf /tensorflow/core/framework/tensor_slice_test.cc | |
| parent | b36576e48db9c9c5448a1f15ded364f4f1f5a9d9 (diff) | |
TensorSlice: fix a subtle, long-standing bug with IsFullAt().
Discovered during the development of V2 format.
This has not been triggered since the TensorSlice class has seen little use
outside of the V1 codepath. That code path has made careful use of it to not
violate the implicit contract (that lengths_[d] == -1 iff starts_[d] == 0).
Change: 131950924
Diffstat (limited to 'tensorflow/core/framework/tensor_slice_test.cc')
| -rw-r--r-- | tensorflow/core/framework/tensor_slice_test.cc | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/tensorflow/core/framework/tensor_slice_test.cc b/tensorflow/core/framework/tensor_slice_test.cc index e26c840998..bb32fa0724 100644 --- a/tensorflow/core/framework/tensor_slice_test.cc +++ b/tensorflow/core/framework/tensor_slice_test.cc @@ -273,8 +273,8 @@ TEST(TensorSliceTest, Deserialization) { TensorSlice ts3(proto3); // Both serializations should be interpreted the same. - EXPECT_EQ("0,5:0,10:14,1:-:-", ts2.DebugString()); - EXPECT_EQ("0,5:0,10:14,1:-:-", ts3.DebugString()); + EXPECT_EQ("0,5:0,10:14,1:1,-1:-", ts2.DebugString()); + EXPECT_EQ("0,5:0,10:14,1:1,-1:-", ts3.DebugString()); } TEST(TensorSliceTest, UpdateToCover) { |