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

[FIX] Replace DataFrame.append with pd.concat #540

Merged
merged 1 commit into from
Apr 6, 2023

Conversation

jgranley
Copy link
Member

@jgranley jgranley commented Apr 6, 2023

Description

pandas.DataFrame.append no longer exists in version 2.0, which causes our tests to fail. I changed it to use the recommended pd.concat instead.

https://pandas.pydata.org/pandas-docs/version/1.5/whatsnew/v1.4.0.html#whatsnew-140-deprecations-frame-series-append

Type of Change

  • Bug fix (non-breaking change which fixes an issue)

Checklist

  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes

@jgranley jgranley merged commit 4a3ea98 into pulse2percept:master Apr 6, 2023
19 checks passed
@jgranley jgranley deleted the pandas-append branch April 6, 2023 23:07
jgranley added a commit that referenced this pull request Apr 7, 2023
* [MNT] Update requirements.txt (#507)

* [DOC] Fix gallery thumbnail images (#510)

* [FIX] Add check for empty stimulus (#522)

* add check for empty stimulus in stim setter for implants

* add check for empty np.ndarray

* [FIX] Fix electrode numbering annotation in implant.plot() (#523)

* fix implant annotation

* used zorder

* [MNT][FIX] Remove outdated ubuntu workflows, add Jax version requirement (#529)

* Remove 18.04, add 20.04

* Add python version to display name

* remove python version

* add jax version

* Fix jax version

* test for wheels: remove jax version

* Add jax version back

* test for wheels: remove jax version

* Add jax version back

* test for wheels

* also skip win32 for python3.7

* [FIX] Use pandas concat instead of append (#540)

---------

Co-authored-by: isaac hoffman <trsileneh@gmail.com>
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.

None yet

1 participant