Skip to content

Commit

Permalink
Correction on vertice overlap which use same variable for two things ...
Browse files Browse the repository at this point in the history
  • Loading branch information
AntSimi committed May 10, 2023
1 parent b93850a commit 6597187
Showing 1 changed file with 5 additions and 5 deletions.
10 changes: 5 additions & 5 deletions src/py_eddy_tracker/poly.py
Original file line number Diff line number Diff line change
Expand Up @@ -477,22 +477,22 @@ def vertice_overlap(
if intersection == 0:
cost[i] = 0
continue
p0_area, p1_area = p0.area(), p1.area()
p0_area_, p1_area_ = p0.area(), p1.area()
if minimal_area:
cost_ = intersection / min(p0_area, p1_area)
cost_ = intersection / min(p0_area_, p1_area_)
# we divide intersection with p1
elif p1_area:
cost_ = intersection / p1_area
cost_ = intersection / p1_area_
# we divide intersection with polygon merging result from 0 to 1
else:
cost_ = intersection / (p0_area + p1_area - intersection)
cost_ = intersection / (p0_area_ + p1_area_ - intersection)
if cost_ >= min_overlap:
cost[i] = cost_
else:
if (
hybrid_area
and cost_ != 0
and (intersection / min(p0_area, p1_area)) > 0.99
and (intersection / min(p0_area_, p1_area_)) > 0.99
):
cost[i] = cost_
else:
Expand Down

0 comments on commit 6597187

Please sign in to comment.