Skip to content

Add check that all segments end at the same timestamp #24

Merged
merged 4 commits into from
Sep 13, 2021
Merged

Conversation

Mr-Geekman
Copy link
Contributor

@Mr-Geekman Mr-Geekman commented Sep 10, 2021

The check was added to TSDataset.transform, TSDataset.fit_transform methods.

@Mr-Geekman Mr-Geekman added the enhancement New feature or request label Sep 10, 2021
@Mr-Geekman Mr-Geekman self-assigned this Sep 10, 2021
@codecov-commenter
Copy link

codecov-commenter commented Sep 10, 2021

Codecov Report

❗ No coverage uploaded for pull request base (master@e0ddddc). Click here to learn what that means.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master      #24   +/-   ##
=========================================
  Coverage          ?   89.05%           
=========================================
  Files             ?       41           
  Lines             ?     1763           
  Branches          ?        0           
=========================================
  Hits              ?     1570           
  Misses            ?      193           
  Partials          ?        0           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e0ddddc...b987a4e. Read the comment docs.

@martins0n martins0n removed the request for review from iKintosh September 13, 2021 10:55
Copy link
Contributor

@martins0n martins0n left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@martins0n martins0n merged commit 80c2028 into master Sep 13, 2021
@iKintosh iKintosh deleted the ETNA-778 branch September 17, 2021 07:34
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants