search

radical-cybertools / radical.analytics

Analytics for RADICAL-Cybertools
Other
1 stars 1 forks source link

who needs pandas anyway? #197

Closed andre-merzky closed 2 weeks ago

andre-merzky commented 4 weeks ago

This reduces the memory footprint from >512GB to, well, nothing really. And runtime is now less than lifetime of the universe also, which is nice. No idea what pd.merge was trying to do here, probably creating an information black hole or something... ¯\_(ツ)_/¯

codecov[bot] commented 4 weeks ago

Codecov Report

Attention: Patch coverage is 17.64706% with 14 lines in your changes missing coverage. Please review.

Project coverage is 21.70%. Comparing base (00aaa00) to head (8adea0c).

Files Patch % Lines
src/radical/analytics/utils/plot.py 17.64% 14 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## devel #197 +/- ## ========================================== - Coverage 21.89% 21.70% -0.19% ========================================== Files 7 7 Lines 1160 1170 +10 ========================================== Hits 254 254 - Misses 906 916 +10 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.