stock-trading-ml icon indicating copy to clipboard operation
stock-trading-ml copied to clipboard

Published 20 hours ago • verified publisher icon yacoubb

Requirements.txt is out of date

Open robertklee opened this issue 4 years ago • 3 comments

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

robertklee avatar Jul 20 '20 07:07 robertklee

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

XyzzyX44 avatar Aug 02 '20 06:08 XyzzyX44

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

robertklee avatar Aug 07 '20 03:08 robertklee

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 https://github.com/yacoubb/stock-trading-ml/issues/18

pcazzola avatar Dec 06 '20 20:12 pcazzola