Count only for the first hit for subject or tissue within filename

[yarikoptic]

Also added assertion so we do not count incorrectly. But may be should be just a warning?

Closes #172

[satra]

i think this also requires line 302 replaced with the sanitize function from dandi-cli.

[codecov]

Codecov Report

Patch coverage: 100.00% and no project coverage change.

Comparison is base (b310e3e) 97.71% compared to head (d22ac9c) 97.72%.

Additional details and impacted files

@@           Coverage Diff           @@
##           master     #173   +/-   ##
=======================================
  Coverage   97.71%   97.72%           
=======================================
  Files          17       17           
  Lines        1751     1755    +4     
=======================================
+ Hits         1711     1715    +4     
  Misses         40       40           

Flag	Coverage Δ
unittests	97.72% <100.00%> (+<0.01%)	⬆️

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

Impacted Files	Coverage Δ
dandischema/metadata.py	96.69% <100.00%> (+0.06%)	⬆️

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

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

[yarikoptic]

i think this also requires line 302 replaced with the sanitize function from dandi-cli.

did you try that and it resolved #172? would better be done in a separate PR then since this one does resolve issue too

[yarikoptic]

i think this also requires line 302 replaced with the sanitize function from dandi-cli.

did you try that and it resolved #172? would better be done in a separate PR then since this one does resolve issue too

I tested, it does. Will submit a complimentary PR.

[yarikoptic]

@satra so what do you think about this one? It is complimentary to #175