Add logging to TSDataset.make_future
#555
Conversation
Codecov Report
@@ Coverage Diff @@
## master #555 +/- ##
=======================================
Coverage 87.28% 87.28%
=======================================
Files 118 118
Lines 5692 5693 +1
=======================================
+ Hits 4968 4969 +1
Misses 724 724
Continue to review full report at Codecov.
|
etna/datasets/tsdataset.py
Outdated
@@ -288,6 +288,7 @@ def make_future(self, future_steps: int) -> "TSDataset": | |||
|
|||
if self.transforms is not None: | |||
for transform in self.transforms: | |||
tslogger.log(f"Transform {transform.__class__.__name__} is applied to dataset") |
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.
why not transform.__repr__()
?
if you have smth like transforms=[LagTransform(in_column="target", ...), LagTransform(in_column="exog", ...)]
there are two different transforms of the same class
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.
Here I just repeat code from TSDataset.transform
. If wee want this, I can change it in both places and in TSDataset.fit_transform
too.
IMPORTANT: Please do not create a Pull Request without creating an issue first.
Before submitting (must do checklist)
Type of Change
Proposed Changes
Look #553.
Related Issue
#553.
Closing issues
Closes #553.