Incremental template metric

[ebellm]

No description provided.

[rhiannonlynne]

I realize this is a bit of a pain but can I request a couple of changes?

Since the time this metric was prototyped in the various notebooks, we did move from camelCase to snake_case .. this is a truly minor thing, but would be nice if the variables in the metric were consistent (dataSlice -> data_slice, etc).
Please run isort to fix the import ordering.
can you please add a unit test, just with some example inputs and then ensuring the metric returns the values expected?

[rhiannonlynne]

Edit to say never mind about the merge problem, I was misinterpreting the unit test failures.
Basically, instead - run black.

[codecov]

Codecov Report

Attention: 57 lines in your changes are missing coverage. Please review.

Comparison is base (87dd9cc) 55.33% compared to head (78851ce) 55.25%.
Report is 3 commits behind head on main.

❗ Current head 78851ce differs from pull request most recent head e5781f8. Consider uploading reports for the commit e5781f8 to get more accurate results

Files	Patch %	Lines
...bin_sim/maf/metrics/incremental_template_metric.py	19.71%	57 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #389      +/-   ##
==========================================
- Coverage   55.33%   55.25%   -0.09%     
==========================================
  Files         304      305       +1     
  Lines       29609    29681      +72     
  Branches     4282     4285       +3     
==========================================
+ Hits        16385    16400      +15     
- Misses      12281    12338      +57     
  Partials      943      943              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[rhiannonlynne]

@ebellm you can merge this is you'd like, or I can go ahead. If you have additional changes you'd like to add, let me know and we can wait.