explicitly check marker value expressions to be greater than zero bef…

[ptajvar]

Small fix of Jaccard index to work also with the rate-diff. It works with boolean values and any non-zero value, including negative values is mapped to True. So adding an explicit check that the value is greater than zero fixes the issue for rate-diff and other potential transformations that may yield negative values.

Type of change

• Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?

Multiple samples have been run after the fix. The Jaccard index works as before with log1p and now also works with rate-diff.

PR checklist:

• This comment contains a description of changes (with reason).

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 81.76%. Comparing base (00be3d6) to head (9a8a42b).
Report is 4 commits behind head on dev.

Additional details and impacted files

@@            Coverage Diff             @@
##              dev     #140      +/-   ##
==========================================
- Coverage   81.81%   81.76%   -0.05%     
==========================================
  Files         118      118              
  Lines        6560     6560              
==========================================
- Hits         5367     5364       -3     
- Misses       1193     1196       +3     

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