google / uncertainty-baselines

High-quality implementations of standard and SOTA methods on a variety of tasks.

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Replace experimental Distribution Strategies with non-experimental API

dustinvtran opened this issue · comments

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

04d4241

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.

Solved with #218, #221, #219.