Skip to content

Commit b32757d

Browse files
tf-transform-teamtfx-copybara
authored andcommitted
nop
PiperOrigin-RevId: 450974902
1 parent b6d6301 commit b32757d

File tree

1 file changed

+0
-4
lines changed

1 file changed

+0
-4
lines changed

tensorflow_transform/analyzers.py

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -305,9 +305,6 @@ def add_input(self, accumulator, batch_values):
305305
]
306306

307307
def merge_accumulators(self, accumulators):
308-
# TODO(b/422923883): Operate in place on accumulators[0] or batch values
309-
# internally for vectorization benefits after AccumulateFn is in use.
310-
311308
# If the first subaccumulator is default, then the accumulator is default
312309
# and can be discarded.
313310
non_default_accumulators = [
@@ -2422,7 +2419,6 @@ def add_input(self, accumulator, batch_values):
24222419

24232420
def merge_accumulators(self, accumulators):
24242421
"""Sums values in each accumulator entry."""
2425-
# TODO(b/215378946): Consider updating accumulators[0] in place.
24262422
products, vectors, counts = zip(*accumulators)
24272423
return [
24282424
np.sum(products, axis=0),

0 commit comments

Comments
 (0)