Move to TF 2 for good (!672) · Merge requests · Emanuele Roberto Nocera / nnpdf

Emanuele Roberto Nocera requested to merge n3fit_use_tf_two into master Mar 09, 2020

Created by: scarlehoff

In this PR I try to modify the code so that it doesn't use anything deprecated. This includes using "tf.keras" instead of "keras".

Several things are at this points hacks around what tensorflow removes and introduces with respect to Keras and with respect to version 1. Some of those are extremely ugly and I have to look around to see whether I can improve on them. Any suggestions on how to work around these issues are welcomed.

Benchmark the speed of the code
Understand whether it is possible to do a regression test with the same seed (no, see https://www.tensorflow.org/guide/versions#what_is_not_covered )
'-----> do a regression test
Fix the TODOs

Move to TF 2 for good

Merge request reports