manny405 / sapai

Super auto pets engine built with reinforment learning training in mind
MIT License
66 stars 21 forks source link

Remove redundant keras dependency #97

Closed andreped closed 1 year ago

andreped commented 1 year ago

As far as I can tell, keras is not used in this project. PyTorch seems to be used instead.

I would also avoid having both in the same project as in terms of versioning their dependencies often tend to collide.

Hence, I would remove keras as a requirement to avoid bloatware.

andreped commented 1 year ago

As Ubuntu 18.04 is deprecated in GitHub Actions, I also updated the Ubuntu version for the Build CI to Ubuntu 20.04.

This enabled the CIs to run successfully.