aboutsummaryrefslogtreecommitdiffhomepage
path: root/tensorflow/contrib/batching
diff options
context:
space:
mode:
authorGravatar A. Unique TensorFlower <gardener@tensorflow.org>2018-01-08 20:53:48 -0800
committerGravatar TensorFlower Gardener <gardener@tensorflow.org>2018-01-08 20:57:20 -0800
commit11a19750bcb5802d44d51bb4baf74b3fc9ac52bc (patch)
treea123cc42184f7710e809a94661307657dd1d29db /tensorflow/contrib/batching
parentff2ba6bd17f71048c2c4ae44f5297a1e03644d09 (diff)
subtract the task's size from the batch's when removing it.
PiperOrigin-RevId: 181262266
Diffstat (limited to 'tensorflow/contrib/batching')
-rw-r--r--tensorflow/contrib/batching/batch_scheduler.h1
-rw-r--r--tensorflow/contrib/batching/batch_scheduler_test.cc2
2 files changed, 3 insertions, 0 deletions
diff --git a/tensorflow/contrib/batching/batch_scheduler.h b/tensorflow/contrib/batching/batch_scheduler.h
index e18cf6c350..aa8891ab4e 100644
--- a/tensorflow/contrib/batching/batch_scheduler.h
+++ b/tensorflow/contrib/batching/batch_scheduler.h
@@ -210,6 +210,7 @@ std::unique_ptr<TaskType> Batch<TaskType>::RemoveTask() {
return nullptr;
}
std::unique_ptr<TaskType> task = std::move(tasks_.back());
+ size_ -= task->size();
tasks_.pop_back();
return task;
}
diff --git a/tensorflow/contrib/batching/batch_scheduler_test.cc b/tensorflow/contrib/batching/batch_scheduler_test.cc
index f15d8cc8e5..b627fee972 100644
--- a/tensorflow/contrib/batching/batch_scheduler_test.cc
+++ b/tensorflow/contrib/batching/batch_scheduler_test.cc
@@ -74,7 +74,9 @@ TEST(BatchTest, Basic) {
EXPECT_EQ(task1->size(), batch.task(1).size());
EXPECT_EQ(7, batch.RemoveTask()->size());
+ EXPECT_EQ(3, batch.size());
EXPECT_EQ(3, batch.RemoveTask()->size());
+ EXPECT_EQ(0, batch.size());
EXPECT_TRUE(batch.empty());
}