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
fix ResampleWithDistributionTransform with holidays transform #82
Conversation
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## master #82 +/- ##
==========================================
+ Coverage 89.11% 89.23% +0.12%
==========================================
Files 194 205 +11
Lines 12491 12807 +316
==========================================
+ Hits 11131 11428 +297
- Misses 1360 1379 +19
☔ View full report in Codecov by Sentry. |
🚀 Deployed on https://deploy-preview-82--etna-docs.netlify.app |
@@ -132,3 +136,16 @@ def test_get_regressors_info_not_fitted(): | |||
def test_params_to_tune(): | |||
transform = ResampleWithDistributionTransform(in_column="regressor_exog", distribution_column="target") | |||
assert len(transform.params_to_tune()) == 0 | |||
|
|||
|
|||
def test_working_with_categorical_columns(example_tsds): |
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.
Not sure that we need pipeline here, we just want to check that fit_transform
works with categoricals. Try to be more minimalistic
Before submitting (must do checklist)
Proposed Changes
Closing issues
closes #34