-
Notifications
You must be signed in to change notification settings - Fork 813
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Refactorised tabularisation + Jupyter notebook w/ experiments. (#1399)
* Refactorised tabularisation + Jupyter notebook w/ experiments. * Added 'moving window' method + refactored 'time intersection' method. * Refactoring/code simplification + bug fixes. * Added `is_training` flag. * Added tests + bug fixes. * More tests + debugging. * Fixed zero lag value not allowed bug + other debugging. * RegressionModel now calls `create_lagged_training_data` + passing tests. * ShapExplainer now uses `create_lagged_prediction_data` + minor test modifications. * Added further documentation, esp to tests. * Moved `_add_static_covariates` from `tabularization.py` to `regression_model.py`. * Static covariates refactorings. * typo fix in test_regression_models.py Co-authored-by: eliane-maalouf <112691612+eliane-maalouf@users.noreply.github.com> * Removed old `_create_lagged_data` and tests notebook. * Clarification about `check_inputs` in docstring. * Allow `lags_future_covariates` to be `> 0`, and enforce `lags_past_covariates < 0`. * Made `get_feature_times` private, now `_get_feature_times`. * Placed `for` loop back inside `create_lagged_data`; more info in docstrings; test `Sequence[TimeSeries]` inputs and stochasic inputs. * Fixed `bootstrap=True` in `test_regression_models.py`. * Added note about `np.split` in `regression_model.py`. * Fixed repeated static covariates width calculation. * Fixed `shap_explainer` bug. * Amended static covariates test so that values of appended static cov columns are explicitly checked. * Updated docstring error. Co-authored-by: Matthew Bilton <mabilton@pop-os.localdomain> Co-authored-by: eliane-maalouf <112691612+eliane-maalouf@users.noreply.github.com> Co-authored-by: Julien Herzen <julien@unit8.co>
- Loading branch information
1 parent
5483e2f
commit 864d190
Showing
12 changed files
with
7,008 additions
and
467 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.