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

[Feature]: Update panda to copy-on-write and re-enable warnings #224

Open
paulf81 opened this issue Oct 18, 2024 · 1 comment
Open

[Feature]: Update panda to copy-on-write and re-enable warnings #224

paulf81 opened this issue Oct 18, 2024 · 1 comment
Assignees
Labels
enhancement An improvement of an existing feature

Comments

@paulf81
Copy link
Collaborator

paulf81 commented Oct 18, 2024

Description

FLASC makes extensive use of pandas. Currently many of the scripts in flasc use warning suppressions, and many of these are warning about changes made to copies to dataframes. Beginning as an option in pandas 2 and becoming default in pandas 3, is the option to enable a copy-on-write behavior. Enabling this throughout the flasc should:

  • Lower risk of bugs
  • Gain performance improvements
  • Remove the need to suppress warnings

Propose we should work this in soon

Related URLs

https://pandas.pydata.org/docs/user_guide/copy_on_write.html#copy-on-write-chained-assignment

@paulf81 paulf81 added the enhancement An improvement of an existing feature label Oct 18, 2024
@Bartdoekemeijer
Copy link
Collaborator

Sounds great! This always felt like a big challenge working with Pandas Dataframes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement An improvement of an existing feature
Projects
None yet
Development

No branches or pull requests

3 participants