diff options
author | 2018-01-08 20:53:48 -0800 | |
---|---|---|
committer | 2018-01-08 20:57:20 -0800 | |
commit | 11a19750bcb5802d44d51bb4baf74b3fc9ac52bc (patch) | |
tree | a123cc42184f7710e809a94661307657dd1d29db /tensorflow/contrib/batching | |
parent | ff2ba6bd17f71048c2c4ae44f5297a1e03644d09 (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.h | 1 | ||||
-rw-r--r-- | tensorflow/contrib/batching/batch_scheduler_test.cc | 2 |
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()); } |