Open lmanan opened 2 months ago
Attention: Patch coverage is 12.90323%
with 27 lines
in your changes are missing coverage. Please review.
Project coverage is 77.13%. Comparing base (
4221b06
) to head (570c976
). Report is 1 commits behind head on main.
Files | Patch % | Lines |
---|---|---|
src/motile_toolbox/utils/flow.py | 0.00% | 27 Missing :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Proposed Change
This PR primarily includes calculation of flow by phase cross correlation and corrects the edge distance feature by incorporating the now available flow. It is designed to be used as follows:
By calculating the flow separately and after constructing the candidate graph, allows the flexibility to replace this step by a flow coming from an alternate source (for example, a DL model).
Checklist
Go through these things before merge. Actions should run automatically to test them, but for information on how to run locally, see CONTRIBUTING.md.
Further Comments
Other minor changes include:
3.10
onwards (f75e60d5e3ce829cf38b4f09a95d5b39f2748297) since the typing does not work for versions below.motile
from the github (6c27a49743524530421ec2092d1f1af5329bdf1d).BBOX
andFLOW
by default (210371fadeab6d25b66bf96347b03f323996cff0).