Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Requirements.txt is out of date #14

Open
robertklee opened this issue Jul 20, 2020 · 3 comments
Open

Requirements.txt is out of date #14

robertklee opened this issue Jul 20, 2020 · 3 comments

Comments

@robertklee
Copy link

Tensorflow and keras download the wrong version. I will open a PR to fix this issue.

@XyzzyX44
Copy link

XyzzyX44 commented Aug 2, 2020

You can update the requirements text to include for example:
tensorflow==1.12.0
to get the versions you want to run.

@robertklee
Copy link
Author

I opened a PR #15 that fixes this. If you can merge that would be great

@pcazzola
Copy link

pcazzola commented Dec 6, 2020

It seems to run with tensorflow 2.3 if you make the following changes:

  1. move tensorflow up in the requirements.txt to be 3rd (after pandas) so that it installs a compatible version of numpy
  2. Update the called for setting the random seed

Change this:
from tensorflow import set_random_seed
set_random_seed(4)

To this:
tf.random.set_seed(4)

  1. Add this fix
    ValueError: Data cardinality is ambiguous: #18

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants