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
Feat/fit predict encodings #1925
Conversation
Codecov ReportPatch coverage:
❗ Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more. Additional details and impacted files@@ Coverage Diff @@
## master #1925 +/- ##
==========================================
- Coverage 93.84% 93.81% -0.03%
==========================================
Files 126 126
Lines 12174 12200 +26
==========================================
+ Hits 11425 11446 +21
- Misses 749 754 +5
☔ View full report in Codecov by Sentry. |
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.
Cool addition, I really like the fact that a different terminology is used for encoders compare to models (train/inference versus fit/predict), makes things easier to understand.
Spotted some typos in the tests
darts/tests/dataprocessing/encoders/test_covariate_index_generators.py
Outdated
Show resolved
Hide resolved
darts/tests/dataprocessing/encoders/test_covariate_index_generators.py
Outdated
Show resolved
Hide resolved
darts/tests/dataprocessing/encoders/test_covariate_index_generators.py
Outdated
Show resolved
Hide resolved
darts/tests/dataprocessing/encoders/test_covariate_index_generators.py
Outdated
Show resolved
Hide resolved
darts/tests/dataprocessing/encoders/test_covariate_index_generators.py
Outdated
Show resolved
Hide resolved
darts/tests/dataprocessing/encoders/test_covariate_index_generators.py
Outdated
Show resolved
Hide resolved
darts/tests/dataprocessing/encoders/test_covariate_index_generators.py
Outdated
Show resolved
Hide resolved
darts/tests/dataprocessing/encoders/test_covariate_index_generators.py
Outdated
Show resolved
Hide resolved
Co-authored-by: madtoinou <32447896+madtoinou@users.noreply.github.com>
* added encode_train_inference to encoders * added generate_fit_predict_encodings to ForecastingModel * simplify TransferrableFut..Model.generatice_predict_encodings * update changelog * Apply suggestions from code review Co-authored-by: madtoinou <32447896+madtoinou@users.noreply.github.com> * apply suggestions from PR review part 2 --------- Co-authored-by: madtoinou <32447896+madtoinou@users.noreply.github.com>
* udpate dtw example and add dtw to rendered documentation * make all windows inherit from Window * clean up windows * improve dtw documentation * improved forecasting model module documentation * update models and add model links in covariates user guide * add model links to README * update changelog * fix typo in dtw example notebook * remove outdated lines from tide model from before probabilistic support * apply suggestions from PR review * update readme model table * Feat/fit predict encodings (#1925) * added encode_train_inference to encoders * added generate_fit_predict_encodings to ForecastingModel * simplify TransferrableFut..Model.generatice_predict_encodings * update changelog * Apply suggestions from code review Co-authored-by: madtoinou <32447896+madtoinou@users.noreply.github.com> * apply suggestions from PR review part 2 --------- Co-authored-by: madtoinou <32447896+madtoinou@users.noreply.github.com> * update readme * apply suggestions from code review and improve README.md * Update README.md Co-authored-by: madtoinou <32447896+madtoinou@users.noreply.github.com> --------- Co-authored-by: madtoinou <32447896+madtoinou@users.noreply.github.com>
Summary
encode_train_inference()
to encoders. This allows to directly generate the encodings for training and predictiongenerate_fit_predict_encodings()
toForecastingModel
. Encodings can now be generated from the the model for training and prediction together.