Replace experimental Distribution Strategies with non-experimental API
dustinvtran opened this issue · comments
Dustin Tran commented
For consistency, we should probably do across baselines.
- s/tf.distribute.experimental.TPUStrategy/tf.distribute.TPUStrategy
- s/strategy.experimental_distribute_datasets_from_function/strategy.distribute_datasets_from_function
Dustin Tran commented
Also related is to replace distribute_datasets_from_function with distribute_dataset. This removes the need for the input_fn
wrapper which adds some needless complexity. Using the function wrapper had slight performance improvements IIRC in 2019/early 2020. We should re-verify if that's the case. I think @ywen666 had checked 2-3 months ago that it no longer mattered.
Dustin Tran commented