Skip to content

Add target components handling in get_level_dataframe #1179

Merged
merged 25 commits into from
Mar 29, 2023

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 #1169

@alex-hse-repository alex-hse-repository self-assigned this Mar 23, 2023
@github-actions
Copy link

github-actions bot commented Mar 23, 2023

@github-actions github-actions bot temporarily deployed to pull request March 23, 2023 07:57 Inactive
@github-actions github-actions bot temporarily deployed to pull request March 23, 2023 09:47 Inactive
@codecov-commenter
Copy link

codecov-commenter commented Mar 23, 2023

Codecov Report

Merging #1179 (8e169ef) into master (2a2e96e) will increase coverage by 0.15%.
The diff coverage is 100.00%.

📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

@@            Coverage Diff             @@
##           master    #1179      +/-   ##
==========================================
+ Coverage   87.51%   87.67%   +0.15%     
==========================================
  Files         166      177      +11     
  Lines       10093    10409     +316     
==========================================
+ Hits         8833     9126     +293     
- Misses       1260     1283      +23     
Impacted Files Coverage Δ
etna/datasets/tsdataset.py 92.84% <100.00%> (+0.11%) ⬆️
etna/datasets/utils.py 96.70% <100.00%> (+0.95%) ⬆️
etna/reconciliation/base.py 94.44% <100.00%> (+1.11%) ⬆️
etna/transforms/base.py 92.71% <100.00%> (ø)

... and 13 files with indirect coverage changes

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

CHANGELOG.md Outdated Show resolved Hide resolved
etna/datasets/tsdataset.py Outdated Show resolved Hide resolved
etna/datasets/tsdataset.py Outdated Show resolved Hide resolved
etna/datasets/utils.py Show resolved Hide resolved
etna/datasets/utils.py Show resolved Hide resolved
tests/conftest.py Show resolved Hide resolved
tests/test_datasets/test_utils.py Outdated Show resolved Hide resolved
tests/test_reconciliation/test_base.py Show resolved Hide resolved
tests/test_reconciliation/test_top_down.py Show resolved Hide resolved
@github-actions github-actions bot temporarily deployed to pull request March 27, 2023 13:32 Inactive
@github-actions github-actions bot temporarily deployed to pull request March 27, 2023 14:11 Inactive
etna/datasets/tsdataset.py Outdated Show resolved Hide resolved
etna/datasets/tsdataset.py Outdated Show resolved Hide resolved
etna/datasets/tsdataset.py Show resolved Hide resolved
etna/datasets/tsdataset.py Show resolved Hide resolved
ts_reconciled = TSDataset(
df=df_reconciled,
freq=ts.freq,
df_exog=ts.df_exog,
known_future=ts.known_future,
hierarchical_structure=ts.hierarchical_structure,
)
if ts.target_components_names is not None:
ts_reconciled.add_target_components(target_components_df=target_components_df)
Copy link
Contributor

Choose a reason for hiding this comment

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

Same here about target_components_df.

tests/test_pipeline/test_pipeline.py Outdated Show resolved Hide resolved
tests/test_pipeline/test_pipeline.py Outdated Show resolved Hide resolved
tests/test_pipeline/test_pipeline.py Outdated Show resolved Hide resolved
tests/test_pipeline/test_pipeline.py Outdated Show resolved Hide resolved
@github-actions github-actions bot temporarily deployed to pull request March 29, 2023 06:27 Inactive
Mr-Geekman
Mr-Geekman previously approved these changes Mar 29, 2023
@alex-hse-repository alex-hse-repository enabled auto-merge (squash) March 29, 2023 11:24
@github-actions github-actions bot temporarily deployed to pull request March 29, 2023 11:28 Inactive
@github-actions github-actions bot temporarily deployed to pull request March 29, 2023 12:35 Inactive
@alex-hse-repository alex-hse-repository merged commit 5cc7c87 into master Mar 29, 2023
@Mr-Geekman Mr-Geekman deleted the issue-1169 branch March 29, 2023 14:36
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.

Add target components handling in get_level_dataframe
4 participants