Skip to content

Remove to_be_fixed from inference tests on SpecialDaysTransform #1283

Merged
merged 2 commits into from Jun 8, 2023

Conversation

Mr-Geekman
Copy link
Contributor

@Mr-Geekman Mr-Geekman commented Jun 7, 2023

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

Remove to_be_fixed from inference tests on SpecialDaysTransform.

Closing issues

#1113 --- it should be closed as not planned.

@Mr-Geekman Mr-Geekman self-assigned this Jun 7, 2023
@codecov-commenter
Copy link

codecov-commenter commented Jun 7, 2023

Codecov Report

Merging #1283 (fac9bf4) into master (106b989) will decrease coverage by 0.01%.
The diff coverage is n/a.

❗ Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.

@@            Coverage Diff             @@
##           master    #1283      +/-   ##
==========================================
- Coverage   87.86%   87.86%   -0.01%     
==========================================
  Files         176      176              
  Lines       10510    10506       -4     
==========================================
- Hits         9235     9231       -4     
  Misses       1275     1275              

see 4 files with indirect coverage changes

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

@github-actions
Copy link

github-actions bot commented Jun 7, 2023

@github-actions github-actions bot temporarily deployed to pull request June 7, 2023 09:14 Inactive
@@ -19,6 +19,9 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
- Update requirements for `holidays` and `scipy`, change saving library from `pickle` to `dill` in `SaveMixin` ([#1268](https://github.com/tinkoff-ai/etna/pull/1268))
- Update requirement for `ruptures`, add requirement for `sqlalchemy` ([#1276](https://github.com/tinkoff-ai/etna/pull/1276))
- Optimize `make_samples` of `RNNNet` and `MLPNet` ([#1281](https://github.com/tinkoff-ai/etna/pull/1281))
- Remove `to_be_fixed` from inference tests on `SpecialDaysTransform` ([#1283](https://github.com/tinkoff-ai/etna/pull/1283))
Copy link
Collaborator

Choose a reason for hiding this comment

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

Not really sure that wee need to ad it in the Changeling

@Mr-Geekman Mr-Geekman merged commit 06cc015 into master Jun 8, 2023
13 checks passed
@Mr-Geekman Mr-Geekman deleted the fix-special-days-inference-tests branch June 8, 2023 07:11
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

None yet

3 participants