Skip to content

TSDataset 2.0 #956

Merged
merged 13 commits into from Sep 26, 2022
Merged

TSDataset 2.0 #956

merged 13 commits into from Sep 26, 2022

Conversation

alex-hse-repository
Copy link
Collaborator

Before submitting (must do checklist)

  • Did you read the contribution guide?
  • Did you update the docs? We use Numpy format for all the methods and classes.
  • Did you write any new necessary tests?
  • Did you update the CHANGELOG?

Proposed Changes

Closing issues

closes #876

@alex-hse-repository alex-hse-repository self-assigned this Sep 26, 2022
@alex-hse-repository alex-hse-repository changed the title Issue 876 TSDataset 2.0 Sep 26, 2022
@codecov-commenter
Copy link

codecov-commenter commented Sep 26, 2022

Codecov Report

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

@@              Coverage Diff               @@
##             tsdataset_2     #956   +/-   ##
==============================================
  Coverage               ?   83.75%           
==============================================
  Files                  ?      132           
  Lines                  ?     7792           
  Branches               ?        0           
==============================================
  Hits                   ?     6526           
  Misses                 ?     1266           
  Partials               ?        0           

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@github-actions
Copy link

github-actions bot commented Sep 26, 2022

🚀 Deployed on https://deploy-preview-956--etna-docs.netlify.app

@github-actions github-actions bot temporarily deployed to pull request September 26, 2022 05:44 Inactive
@github-actions github-actions bot temporarily deployed to pull request September 26, 2022 07:02 Inactive
for transform in self.transforms:
tslogger.log(f"Transform {repr(transform)} is applied to dataset")
df = transform.transform(df)
ts = TSDataset(df=df, freq=self.freq)
Copy link
Contributor

Choose a reason for hiding this comment

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

Is it working? What about regressors and known future? It seems like we are losing this information here.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Added comment

Copy link
Contributor

@Mr-Geekman Mr-Geekman left a comment

Choose a reason for hiding this comment

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

Answer questions above.

@github-actions github-actions bot temporarily deployed to pull request September 26, 2022 13:59 Inactive
@alex-hse-repository alex-hse-repository merged commit c2edede into tsdataset_2 Sep 26, 2022
@github-actions github-actions bot temporarily deployed to pull request September 26, 2022 15:18 Inactive
alex-hse-repository added a commit that referenced this pull request Sep 27, 2022
alex-hse-repository added a commit that referenced this pull request Feb 2, 2023
alex-hse-repository added a commit that referenced this pull request Feb 22, 2023
alex-hse-repository added a commit that referenced this pull request Mar 2, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants