octoenergy / timeserio

Better `keras` models for time series and beyond
MIT License
61 stars 16 forks source link

Replace `keras` with `tensorflow.keras` and fix random states in tests #20

Closed ig248 closed 4 years ago

ig248 commented 4 years ago

Recent PRs have highlighted increasing issues with mixing keras and tensorflow - things like random seeds for reproducible testing, callbacks, and minor API changes and deprecation warning. While keras is still maintained, compatibility between versions is ill-documented and the general assumption appears to be that ppl should use tf.keras - in fact, keras.io now exclusively documents the latter.