Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add pct_change function for Series #8304

Closed
wants to merge 39 commits into from

Conversation

TravisHester
Copy link
Contributor

closes #6133
unit test in test_stats.py: test_series_pct_change

Added pct_change function for Series. The limit and freq arguments could not be fully implemented due to lack of implementation in Series.fillna() and Series.shift() functions respectively.

raydouglass and others added 28 commits March 30, 2020 11:03
Merge pull request rapidsai#5690 from ajschmidt8/phase2
[skip ci] Update master references for main branch
[RELEASE] Re-release v0.15 cudf [skip-ci]
[RELEASE] v0.18.2 `cudf` release [skip-ci]
@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@github-actions github-actions bot added the Python Affects Python cuDF API. label May 20, 2021
@beckernick beckernick added the 2 - In Progress Currently a work in progress label May 25, 2021
@TravisHester TravisHester added 3 - Ready for Review Ready for review by team and removed 2 - In Progress Currently a work in progress labels May 25, 2021
@TravisHester TravisHester marked this pull request as ready for review May 25, 2021 21:12
@TravisHester TravisHester requested a review from a team as a code owner May 25, 2021 21:12
python/cudf/cudf/core/series.py Outdated Show resolved Hide resolved
python/cudf/cudf/tests/test_stats.py Outdated Show resolved Hide resolved
python/cudf/cudf/tests/test_stats.py Show resolved Hide resolved
python/cudf/cudf/tests/test_stats.py Show resolved Hide resolved
@beckernick
Copy link
Member

@TravisHester this appears to be failing style checks

@harrism harrism changed the title Added pct_change function for Series Add pct_change function for Series Jun 22, 2021
@TravisHester TravisHester requested a review from a team as a code owner June 25, 2021 15:23
@beckernick
Copy link
Member

Looks like some style changes + merge conflicts are all that remains

@TravisHester TravisHester requested review from a team as code owners July 2, 2021 18:52
@TravisHester TravisHester requested a review from ttnghia July 2, 2021 18:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3 - Ready for Review Ready for review by team feature request New feature or request non-breaking Non-breaking change Python Affects Python cuDF API.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[FEA] pct_change
7 participants