-
Notifications
You must be signed in to change notification settings - Fork 487
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
Remove residuals in forecast dataframe returned by m.predict() #927
Conversation
*Disabled the plotting option for residuals as well *Not really needed as residuals are part of basic metrics such as MAE and RMSE *forecast df is very large, which may overwhelm the user. Now more lean
Codecov Report
@@ Coverage Diff @@
## main #927 +/- ##
==========================================
+ Coverage 87.43% 87.68% +0.24%
==========================================
Files 17 17
Lines 4466 4433 -33
==========================================
- Hits 3905 3887 -18
+ Misses 561 546 -15
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
Can you have it pass the flake8 and metrics test? Thanks. |
Can you pass the flake8 test by resolving this following issue?
|
In the future we may add a tutorial on how to analyze residuals and what one can learn from them. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
fixed merge commits. ready to merge when tests pass. |
Model Benchmark
Model TrainingPeytonManningYosemiteTempsAirPassengers |
*Disabled the plotting option for residuals as well
*Not really needed as residuals are part of basic metrics such as MAE and RMSE
*forecast df is very large, which may overwhelm the user. Now more lean