-
Notifications
You must be signed in to change notification settings - Fork 473
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
[refactor] Encapsulate _convert_raw_predictions_to_raw_df #1291
Conversation
Model Benchmark
|
Codecov Report
📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more @@ Coverage Diff @@
## main #1291 +/- ##
=======================================
Coverage 89.85% 89.86%
=======================================
Files 38 38
Lines 5128 5131 +3
=======================================
+ Hits 4608 4611 +3
Misses 520 520
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
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
🔬 Background
Related to #1133
This PR is part of a series of PRs belonging to step 2 of restructuring the
forecaster.py
. Step 1 was to move identified functions out of the forecaster.py into dedicated files. Step 2 is to encapsulate the input arguments of the moved functions. Step2 will be implemented step-by-step within multiple PRs🔮 Key changes
_convert_raw_predictions_to_raw_df
including docstring and typing.📋 Review Checklist
Please make sure to follow our best practices in the Contributing guidelines.