END_PUBLIC --- Commit1e1b3d902
authored by Pete Warden<pete@petewarden.com> Committed by gunan<gunan@google.com>: Changed output directory for Pi CI build to fix permissions problem with nightlies (#13257) * Fix for RTLD_GLOBAL breakage of Pi builds, and removed Eigen version change for Pi that's no longer needed * Fixed Pi Zero OpenBLAS build problems and tidied up directories used * More robust checks in Pi build script * Changed output directory for Pi CI build to fix permissions problem --- Commitfe3a2e65c
authored by Yan Facai (???)<facai.yan@gmail.com> Committed by drpngx<drpngx@users.noreply.github.com>: check invalid string type for dest_nodes in extract_sub_graph (#13057) * BUG: check str type * TST: add unit test * CLN: remove list check * CLN: use warning * CLN: 2 indent * CLN: raise TypeError if not list * CLN: check string only --- Commit225ab7629
authored by Jean Wanka<jm.wanka@gmail.com> Committed by Jean Wanka<jm.wanka@gmail.com>: Fix polynomial decay with cycle for global step=0 For polynomial decay with cycle=True the learning rate at step 0 becomes NaN, because in the process of calculating it we devide by 0. This change should fix it, by setting the multiplier for the decay steps to one for global_step=0. --- Commit286f57061
authored by Bjarke Hammersholt Roune<broune@google.com> Committed by TensorFlower Gardener<gardener@tensorflow.org>: Make Service::TransferToClient not attempt to manipulate the literal when the transfer failed, preventing a crash and allowing the caller to see the reason for the failed transfer. PiperOrigin-RevId: 169770126 --- Commite0501bc4d
authored by Yong Tang<yong.tang.github@outlook.com> Committed by Shanqing Cai<cais@google.com>: Fix GRUBlockCell parameter naming inconsistency (#13153) * Fix GRUBlockCell parameter naming inconsistency This fix tries to fix the issue in 13137 where parameter `cell_size` is used instead of `num_units`. This is inconsistent with other RNN cells. This fix adds support of `num_units` while at the same time maintains backward compatiblility for `cell_size`. This fix fixes 13137. Signed-off-by: Yong Tang <yong.tang.github@outlook.com> * Add `@deprecated_args` for 'cell_size' in `GRUBlockCell` This commit adds `@deprecated_args` for 'cell_size' in `GRUBlockCell` Signed-off-by: Yong Tang <yong.tang.github@outlook.com> * Address review comment Signed-off-by: Yong Tang <yong.tang.github@outlook.com> --- Commit02a2eba05
authored by Pete Warden<pete@petewarden.com> Committed by gunan<gunan@google.com>: Fix for RTLD_GLOBAL breakage of Pi builds, and removed Eigen version change that's no longer needed (#13251) * Fix for RTLD_GLOBAL breakage of Pi builds, and removed Eigen version change for Pi that's no longer needed * Fixed Pi Zero OpenBLAS build problems and tidied up directories used * More robust checks in Pi build script --- Commit8ef722253
authored by Sanjoy Das<sanjoy@google.com> Committed by TensorFlower Gardener<gardener@tensorflow.org>: Remove a redundant setName. The EmitComputation should have emitted a function with the right name, so use a CHECK instead. PiperOrigin-RevId: 169764856 --- Commit1b94147dc
authored by Neal Wu<wun@google.com> Committed by TensorFlower Gardener<gardener@tensorflow.org>: Fix broken GitHub links in tensorflow and tensorflow_models resulting from The Great Models Move (a.k.a. the research subfolder) PiperOrigin-RevId: 169763373 --- Commitb1ada5f0c
authored by Justine Tunney<jart@google.com> Committed by TensorFlower Gardener<gardener@tensorflow.org>: Fix TensorBoard python -m invoke in docs PiperOrigin-RevId: 169758752 --- Commit2957cd894
authored by Mustafa Ispir<ispir@google.com> Committed by TensorFlower Gardener<gardener@tensorflow.org>: Local run option of estimator training. PiperOrigin-RevId: 169756384 --- Commit1dc2fe7ac
authored by Gunhan Gulsoy<gunan@google.com> Committed by TensorFlower Gardener<gardener@tensorflow.org>: BEGIN_PUBLIC Automated g4 rollback of changelist 166264198 PiperOrigin-RevId: 169998124
115 lines
4.0 KiB
Python
115 lines
4.0 KiB
Python
# Copyright 2017 The TensorFlow Authors. All Rights Reserved.
|
|
#
|
|
# Licensed under the Apache License, Version 2.0 (the "License");
|
|
# you may not use this file except in compliance with the License.
|
|
# You may obtain a copy of the License at
|
|
#
|
|
# http://www.apache.org/licenses/LICENSE-2.0
|
|
#
|
|
# Unless required by applicable law or agreed to in writing, software
|
|
# distributed under the License is distributed on an "AS IS" BASIS,
|
|
# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
|
|
# See the License for the specific language governing permissions and
|
|
# limitations under the License.
|
|
# ==============================================================================
|
|
|
|
|
|
from __future__ import absolute_import
|
|
from __future__ import division
|
|
from __future__ import print_function
|
|
|
|
import os
|
|
import numpy as np
|
|
import tensorflow as tf
|
|
import tensorflow.contrib.mpi_collectives as mpi
|
|
from tensorflow.python.platform import test
|
|
|
|
|
|
average_allgather = False
|
|
|
|
|
|
class AllgatherTest(test.TestCase):
|
|
def checkAllgather(self, num_ranks, all_gathered, local_gathered):
|
|
# Ensure that indices match.
|
|
all_gat_ind = np.sort(all_gathered.indices)
|
|
loc_gat_ind = np.sort(local_gathered.indices)
|
|
assert(len(loc_gat_ind) == len(all_gat_ind))
|
|
for i in range(len(loc_gat_ind)):
|
|
assert(loc_gat_ind[i] == all_gat_ind[i])
|
|
|
|
# For each index, verify same values.
|
|
local_checked = []
|
|
for i in range(len(local_gathered.indices)):
|
|
local_checked.append(False)
|
|
for i in range(len(all_gathered.indices)):
|
|
all_index = all_gathered.indices[i]
|
|
# TODO(jthestness): Make this lookup quicker using sorting.
|
|
loc_index = -1
|
|
for j in range(len(local_gathered.indices)):
|
|
if local_gathered.indices[j] == all_index and not local_checked[j]:
|
|
loc_index = j
|
|
local_checked[j] = True
|
|
break
|
|
assert(loc_index >= 0)
|
|
correct_output = local_gathered.values[loc_index][0]
|
|
if average_allgather:
|
|
correct_output = correct_output / float(num_ranks)
|
|
assert(all_gathered.values[i][0] == correct_output)
|
|
|
|
|
|
def test_mpi_allgather(self):
|
|
# Get MPI rank
|
|
my_rank = int(os.environ['PMI_RANK'])
|
|
num_ranks = int(os.environ['PMI_SIZE'])
|
|
|
|
indices_per_rank = 100
|
|
tensor_width = 10
|
|
|
|
# Create IndexedSlices for each rank, some with overlapping indices.
|
|
to_gather_indices = []
|
|
to_gather_values = []
|
|
to_gather = []
|
|
for rank_id in range(num_ranks):
|
|
indices = []
|
|
values = []
|
|
my_multiple = rank_id + 1
|
|
current_index = my_multiple
|
|
for i in range(indices_per_rank):
|
|
indices.append(current_index)
|
|
ones_tensor = tf.ones([tensor_width])
|
|
values.append(tf.multiply(ones_tensor,
|
|
tf.fill(ones_tensor.get_shape(),
|
|
float(current_index))))
|
|
current_index += my_multiple
|
|
concat_ind = tf.stack(indices)
|
|
concat_vals = tf.stack(values)
|
|
to_gather_indices.append(concat_ind)
|
|
to_gather_values.append(concat_vals)
|
|
to_gather.append(tf.IndexedSlices(concat_vals, concat_ind))
|
|
|
|
# Collect the local IndexedSlices (indices and values) to create
|
|
# correct IndexedSlices output.
|
|
correct_gather_indices = tf.concat(to_gather_indices, 0)
|
|
correct_gather_values = tf.concat(to_gather_values, 0)
|
|
correct_gather = tf.IndexedSlices(correct_gather_values,
|
|
correct_gather_indices)
|
|
|
|
all_gather = mpi.allreduce(to_gather[my_rank], average_allgather)
|
|
|
|
# NOTE: This assumes that device IDs are numbered the same as ranks.
|
|
gpu_options = tf.GPUOptions(visible_device_list=str(my_rank))
|
|
config = tf.ConfigProto(gpu_options=gpu_options)
|
|
|
|
# MPI Session to test allgather.
|
|
with mpi.Session(config=config) as sess:
|
|
sess.run(tf.global_variables_initializer())
|
|
|
|
all_gathered, local_gathered = sess.run([all_gather, correct_gather])
|
|
|
|
# Compare all_gathered with local_gathered.
|
|
self.checkAllgather(num_ranks, all_gathered, local_gathered)
|
|
|
|
|
|
if __name__ == '__main__':
|
|
test.main()
|