search

ZettaAI / zetta_utils

MIT License
11 stars 0 forks source link

feat(training/ROIMaskProcessor): add upstream_pad #756

Closed trivoldus28 closed 3 months ago

trivoldus28 commented 3 months ago

Also change output mask type to uint8 by default

trivoldus28 commented 3 months ago

But I think maybe there should be a more long term solution. In the current setup I don't think we can have more than 2 augments that dynamically modify the index. Maybe save the original index in the current index that the processors can compare?

trivoldus28 commented 3 months ago

Some errors about docs this time.. I'll wait for that to be resolved first?

supersergiy commented 3 months ago

Yup I'm on it

supersergiy commented 3 months ago

But I think maybe there should be a more long term solution. In the current setup I don't think we can have more than 2 augments that dynamically modify the index. Maybe save the original index in the current index that the processors can compare?

Yeah we should think about what to do...

codecov[bot] commented 3 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 100.00%. Comparing base (55f9a72) to head (4906bc6). Report is 7 commits behind head on main.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #756 +/- ## =========================================== + Coverage 99.98% 100.00% +0.01% =========================================== Files 140 140 Lines 5448 5465 +17 =========================================== + Hits 5447 5465 +18 + Misses 1 0 -1 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.