[prototype] Extend detection result customization

[felixdittrich92]

This PR:

• Make box_thresh also accessible for end users (same as bin_thresh)
• Provide a way to add a hook / multiple hooks in the middle of the pipeline before cropping and recognition
• Extend the docs

Any feedback is welcome

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (f5445ef) 95.81% compared to head (70cac27) 95.83%.
Report is 5 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #1449      +/-   ##
==========================================
+ Coverage   95.81%   95.83%   +0.02%     
==========================================
  Files         162      162              
  Lines        7101     7112      +11     
==========================================
+ Hits         6804     6816      +12     
+ Misses        297      296       -1     

Flag	Coverage Δ
unittests	95.83% <100.00%> (+0.02%)	⬆️

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

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

[odulcy-mindee]

We need to do the same on HF Space

[felixT2K]

yep this would be good afterwards :)

[felixdittrich92]

@odulcy-mindee I added two points in the docs - as an advanced functionality for manipulation - i think it's better to mention it in the docs instead of producing some overhead to check after each hook that the coords are still relative in the range between 0 and 1 😅

[felixdittrich92]

@odulcy-mindee I added two points in the docs - as an advanced functionality for manipulation - i think it's better to mention it in the docs instead of producing some overhead to check after each hook that the coords are still relative in the range between 0 and 1 😅

@dchaplinsky What do you think from a user perspective ? Is it clear enough from the docs that input/output structure needs to be the same and the coords are relative so it needs also to be returned as relative (0-1) or should we add a check for the value range and shape (which would produce a small overhead)

[odulcy-mindee]

@felixdittrich92 I'm fine with it !

[odulcy-mindee]

@felixdittrich92 Thnak you Felix, nice to have these hooks !

[dchaplinsky]

@felixdittrich92 well documented interface is enough. And the output format is dictated by the input format, so it will be strange if someone will receive relative coordinates and return absolute instead.

[felixdittrich92]

@felixdittrich92 well documented interface is enough. And the output format is dictated by the input format, so it will be strange if someone will receive relative coordinates and return absolute instead.

Oh believe me i saw a lot 😂