-
Notifications
You must be signed in to change notification settings - Fork 80
Conversation
Codecov Report
@@ Coverage Diff @@
## master #420 +/- ##
==========================================
+ Coverage 87.13% 87.28% +0.15%
==========================================
Files 99 111 +12
Lines 5067 5127 +60
==========================================
+ Hits 4415 4475 +60
Misses 652 652
Continue to review full report at Codecov.
|
from etna.transforms.timestamp import FourierTransform | ||
from etna.transforms.timestamp import HolidayTransform | ||
from etna.transforms.timestamp import SpecialDaysTransform | ||
from etna.transforms.timestamp import TimeFlagsTransform |
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.
There are 40 lines in a new file and 41 in the old version. It is strange.
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.
it's true
I didn't add import for BaseFeatureSelectionTransform
at the top level of transforms because this class seems to be not very useful for users
IMPORTANT: Please do not create a Pull Request without creating an issue first.
Before submitting (must do checklist)
Type of Change
Proposed Changes
Related Issue
Closing issues
#403