ComStock icon indicating copy to clipboard operation
ComStock copied to clipboard

Modify postprocessing/comstock.py and cbecs.py to use polars instead of pandas except for plotting

Open asparke2 opened this issue 2 years ago • 1 comments

Currently, we use a mix of Pandas and Polars in comstock-postprocessing. Change this all over to Polars. It performs better, and having one library to manage changes in would be nice. The exception to this is converting to Pandas right before plotting, because Seaborn works natively with Pandas.

asparke2 avatar Jan 09 '24 04:01 asparke2

I suggest we wait until polars has a version 1 before doing this; polars is much more unstable, and dealing with break changes may be slower than the minor additional run time in existing code.

mdahlhausen avatar Jan 10 '24 18:01 mdahlhausen