cryptocurrency-price-prediction icon indicating copy to clipboard operation
cryptocurrency-price-prediction copied to clipboard

The block comparing prediction to test data is wrong

Open tastypenguinbacon opened this issue 4 years ago • 4 comments

The following line is plain wrong: preds = test[target_col].values[:-window_len] * (preds + 1) You're basically taking the result you want to see (test[target_col].values[:-window_len]) and multiply it by (1 + noise) where te amplitude of the noise was in my case always smaller than 0.3.

The other plot is given as: targets = test[target_col][window_len:] which is the same as preds, but shifted by five. (1 + preds) is near to 1, so basically the stuff you see is the stuff you wanted to see in the first place.

The correct way is to: preds = test[target_col].values[0] * (preds + 1) which does show no correlation at all - the predicted value diverges from the test data.

tastypenguinbacon avatar Dec 03 '19 23:12 tastypenguinbacon

I changed this value and now it looks not that accurate. @tastypenguinbacon is this the only modification needed? The results look way more reasonable that way.

danipolo avatar Feb 11 '20 22:02 danipolo

This code was featured on some major crypto news outlet clickbate article which explains why the "bug" was never fixed. In this case the bug is intentional to generate internet "clout".

geojitsu avatar Jun 30 '20 19:06 geojitsu

FAKE RESULTS :-(

frash83 avatar Dec 24 '20 18:12 frash83

Good grief. If this is fake science, how do we really do this then?

FullstackJack avatar Dec 29 '20 21:12 FullstackJack