search

octoenergy / timeserio

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

Update to tf 2.8 and python 3.9 and drop s3fs dependency #41

Closed ali-tny closed 2 years ago

ali-tny commented 2 years ago

and generally relax some dependencies so we can actually use this downstream.

We remove the s3fs dependency in favour of using tentaclio, since the s3fs dependency was causing a lot of pain (strict requirements meant we couldn't update other packages)

ali-tny commented 2 years ago

@capitancambio changed to tanh since with the new random seed setting process it looked like things were getting randomly stuck with relu (ie in the test test_fit_default which checks that the metric after fitting 1 epoch of 13 random examples is lower than the starting values) - so moving it to tanh where it doesn't have the big zero section helped. nicked from igor's commit here: https://github.com/octoenergy/timeserio/pull/36/commits/bbc9ec739ff17af0e545e2a8866fb4e15f589db8