BlendingToolKit icon indicating copy to clipboard operation
BlendingToolKit copied to clipboard

Added efficiency matrix notebook for SEP and basic peak finding algorithm comparison.

Open MatthewDowicz opened this issue 2 years ago • 2 comments

What?

Added a notebook under the new directory paper_figs that produces efficiency matrix plots comparing SEP and Basic Peak Finding algorithm for the BTK paper.

Why?

To present detection algorithms and metrics available in BTK.

How?

The included notebook uses modified btk.plot_utils.plot_metrics_summary and btk.plot_utils.plot_efficiency_matrix functions to allow for a cleaner figure to be used in the BTK paper.

Testing?

I ran the new notebook with a batchsize of 3000 to check font sizes and styles.

MatthewDowicz avatar Oct 27 '22 00:10 MatthewDowicz

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

Codecov Report

Base: 87.89% // Head: 87.89% // No change to project coverage :thumbsup:

Coverage data is based on head (cf00d3f) compared to base (2574495). Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files
@@           Coverage Diff            @@
##           develop     #390   +/-   ##
========================================
  Coverage    87.89%   87.89%           
========================================
  Files           11       11           
  Lines         1578     1578           
========================================
  Hits          1387     1387           
  Misses         191      191           
Flag Coverage Δ
unittests 87.89% <100.00%> (ø)

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

Impacted Files Coverage Δ
btk/plot_utils.py 82.68% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

codecov[bot] avatar Oct 27 '22 00:10 codecov[bot]

Let's close this for now and revisit after #348 is merged

ismael-mendoza avatar Feb 22 '23 20:02 ismael-mendoza