Skip to content

upgrade min version of pandas and remove rolling primitives workaround - #2246

Merged
ozzieD merged 15 commits into
mainfrom
1771-remove-workaround-rolling
Aug 17, 2022
Merged

upgrade min version of pandas and remove rolling primitives workaround#2246
ozzieD merged 15 commits into
mainfrom
1771-remove-workaround-rolling

Conversation

@ozzieD

@ozzieD ozzieD commented Aug 16, 2022

Copy link
Copy Markdown
Contributor

resolves: #1771 and #2241

Comment thread docs/source/release_notes.rst Outdated
@codecov

codecov Bot commented Aug 16, 2022

Copy link
Copy Markdown

Codecov Report

Merging #2246 (7e80d83) into main (94b40ad) will decrease coverage by 0.00%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #2246      +/-   ##
==========================================
- Coverage   99.29%   99.29%   -0.01%     
==========================================
  Files         146      146              
  Lines       17590    17588       -2     
==========================================
- Hits        17466    17464       -2     
  Misses        124      124              
Impacted Files Coverage Δ
featuretools/primitives/rolling_primitive_utils.py 100.00% <ø> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@gsheni

gsheni commented Aug 16, 2022

Copy link
Copy Markdown
Contributor

You'll need to update the minimum_x_.txt files in here:
https://github.com/alteryx/featuretools/tree/main/featuretools/tests/requirement_files

@gsheni

gsheni commented Aug 16, 2022

Copy link
Copy Markdown
Contributor

@ozzieD I think there is a unit test error showing up due to the new min pandas:
https://github.com/alteryx/featuretools/runs/7865620018?check_suite_focus=true

@ozzieD ozzieD changed the title upgrade min version of pandas and remove rolling primitives workaround Draft: upgrade min version of pandas and remove rolling primitives workaround Aug 16, 2022
@ozzieD ozzieD changed the title Draft: upgrade min version of pandas and remove rolling primitives workaround upgrade min version of pandas and remove rolling primitives workaround Aug 16, 2022
@ozzieD
ozzieD requested a review from gsheni August 17, 2022 00:18

@gsheni gsheni left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Lgtm

@ozzieD
ozzieD merged commit 3578303 into main Aug 17, 2022
@ozzieD
ozzieD deleted the 1771-remove-workaround-rolling branch August 17, 2022 00:21
@ozzieD ozzieD mentioned this pull request Aug 18, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Remove pandas workaround in _roll_series_with_gap

2 participants