NREL / flasc

A rich floris-driven suite for SCADA analysis
https://nrel.github.io/flasc/
BSD 3-Clause "New" or "Revised" License
31 stars 18 forks source link

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

Open paulf81 opened 1 day ago

paulf81 commented 1 day ago

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:

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

Bartdoekemeijer commented 1 day ago

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