Team2890/PhotonVision
4
0
Fork 0
mirror of https://github.com/PhotonVision/photonvision synced 2026-06-25 01:41:40 +00:00
Code Issues Packages Projects Releases Wiki Activity

Fix contour grouping (#298)

Browse Source 
Fixes bug where n+1 contours were needed to match a target of size n
This commit is contained in:
Matt
2021-10-05 12:16:50 -04:00
committed by GitHub
parent 243f06da2d
commit f922466d41
1 changed files with 1 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
photon-core/src/main/java/org/photonvision/vision/pipe/impl/GroupContoursPipe.java
View File

@@ -46,7 +46,7 @@ public class GroupContoursPipe
} else {
int groupingCount = params.getGroup().count;
if (input.size() > groupingCount) {
if (input.size() >= groupingCount) {
input.sort(Contour.SortByMomentsX);
// also why reverse? shouldn't the sort comparator just get reversed?
// TODO: Matt, see this