Skip to content

benchmarks: avoid use of deprecated pandas API#290

Closed
grlee77 wants to merge 1 commit intorapidsai:branch-22.06from
grlee77:benchmarks-pandas-api-update
Closed

benchmarks: avoid use of deprecated pandas API#290
grlee77 wants to merge 1 commit intorapidsai:branch-22.06from
grlee77:benchmarks-pandas-api-update

Conversation

@grlee77
Copy link
Copy Markdown
Contributor

@grlee77 grlee77 commented May 24, 2022

The append method of DataFrame was deprecated in pandas 1.4.0.

Here, we update to use pandas.concat instead (as recommended)

The append method of DataFrame is deprecated since pandas 1.4.0
The recommendation is to use pandas.concat instead:
https://pandas.pydata.org/docs/whatsnew/v1.4.0.html#whatsnew-140-deprecations-frame-series-append
@grlee77 grlee77 added improvement Improves an existing functionality non-breaking Introduces a non-breaking change labels May 24, 2022
Copy link
Copy Markdown
Contributor

@gigony gigony left a comment

Choose a reason for hiding this comment

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

Looks good to me!

@jakirkham jakirkham self-requested a review May 31, 2022 19:11
@jakirkham
Copy link
Copy Markdown
Member

Is it alright if we move this to 22.08?

@grlee77
Copy link
Copy Markdown
Contributor Author

grlee77 commented Jun 18, 2022

Yes

@grlee77
Copy link
Copy Markdown
Contributor Author

grlee77 commented Jul 26, 2022

replaced by #339

@grlee77 grlee77 closed this Jul 26, 2022
rapids-bot bot pushed a commit that referenced this pull request Jul 26, 2022
This is an update of the previously-approved #290 from branch-22.06 to branch-22.08

Authors:
  - Gregory Lee (https://github.com/grlee77)

Approvers:
  - https://github.com/gbae
  - Gigon Bae (https://github.com/gigony)
  - https://github.com/jakirkham

URL: #339
rapids-bot bot pushed a commit that referenced this pull request Jul 28, 2022
This PR expands the approach of #322 to the remaining `cucim.skimage` benchmarks as well

This PR is built on top of a few other open benchmarking PRs, so please review and merge those first: (#322, #323, #290)

Authors:
  - Gregory Lee (https://github.com/grlee77)
  - https://github.com/aasthajh

Approvers:
  - https://github.com/jakirkham

URL: #324
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

improvement Improves an existing functionality non-breaking Introduces a non-breaking change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants