Add density option for histograms

iaraota commented 11 months ago

This PR introduces the density matplotlib property to GWSumm.

codecov[bot] commented 11 months ago

Codecov Report

Merging #380 (28ea0ed) into master (f658c15) will not change coverage. The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #380   +/-   ##
=======================================
  Coverage   50.25%   50.25%           
=======================================
  Files          60       60           
  Lines        8670     8670           
=======================================
  Hits         4357     4357           
  Misses       4313     4313

Flag	Coverage Δ
Linux	`50.25% <ø> (ø)`
python3.10	`50.25% <ø> (ø)`
python3.7	`49.45% <ø> (ø)`
python3.8	`50.25% <ø> (ø)`
python3.9	`50.25% <ø> (ø)`

Flags with carried forward coverage won't be shown. Click here to find out more.

Files	Coverage Δ
gwsumm/plot/utils.py	`80.77% <ø> (ø)`

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

eagoetz commented 10 months ago

@iaraota Do we still need the density option if we're going to make a new type of plot in gwsumm to support the feature request?

gwpy / gwsumm

Add density option for histograms #380

Codecov Report