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

Evaluate the return prediction, not the price prediction #3

Closed
sam31415 opened this issue Oct 18, 2018 · 2 comments
Closed

Evaluate the return prediction, not the price prediction #3

sam31415 opened this issue Oct 18, 2018 · 2 comments

Comments

@sam31415
Copy link

I only went through the readme. The aim stated there is to predict stock returns, so why evaluate the algorithm using stock prices?

Predicting prices with great accuracy is very easy: predict that tomorrow's price will be today's price. The prediction will be correct up to a few percents at most. But that's useless. The analysis in the readme file doesn't make it obvious that the model does better.

I would suggest computing the f1 score or the return-weighted accuracy of a classifier predicting the sign of the returns.

@VivekPa
Copy link
Owner

VivekPa commented Nov 9, 2018

added metrics for both returns and prices now.

@VivekPa VivekPa closed this as completed Nov 9, 2018
@sam31415
Copy link
Author

Thanks for the update. Still only reading the readme.md file, it is not very clear that the model learns something. This could be shown by computing a metric for the return predictions and compare it to the metric of the constant model mentioned above. Also, imho it is not very useful to have an actual vs predictions plot, it's hard to see anything there.

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

2 participants