Code

Circle Hough Transform improvements#1244

Code

Merged

fspindle merged 23 commits intolagadic:masterlagadic/visp:masterfrom rolalaro:vpCHTimprovementsrolalaro/visp:vpCHTimprovementsCopy head branch name to clipboardOct 4, 2023

+3,134-97

Commits

Commits on Sep 25, 2023

• [WIP] Added a method to compute the arc length of a circle that is contained in a rectangular RoI. Missing cases where the circle is bigger than the RoI and where the circle is near a corner and th…

  Show description for 476b489
  

  rlagneau

  committed
  476b489

• [WIP] Manage cases where a single axis is touched (additionally to cases where a single axis is crossed). WiP for two perpendicular axes crossing

  

  rlagneau

  committed
  8209a0d

Commits on Sep 27, 2023

• [WIP] Added cases of crossing top and left axes once, and crossing one of the two axes outside the RoI

  

  rlagneau

  committed
  e6a2865

• [CORPS] Validated last case of intersection with both left and top borders

  

  rlagneau

  committed
  6340a89

Commits on Sep 28, 2023

• [WIP] Added top right cases

  

  rlagneau

  committed
  e980cb7

• [WIP] Added cases of crossings with bottom and left axes

  

  rlagneau

  committed
  0b75cc8

• [WIP] Handle crossing bottom and right axes

  

  rlagneau

  committed
  8fa5445

Commits on Sep 29, 2023

• [CORPS] vpCHT radius candidate validation policy is now based on a probability scheme, which is the ratio of the number of visible pixels divided by the theoretical number of visible pixels

  

  rlagneau

  committed
  0ba0579

• [WIP] Added case top + bottom + X and tested case with two intersections on each axis for the left case

  

  rlagneau

  committed
  2273658

• [WIP] Successful tests: top/right/bottom with two intersections on each axis + top / bottom only

  

  rlagneau

  committed
  0262a6d

• [WIP] Added tests for top/bottom/left and top/bottom/right where only top and bottom are crossed in the RoI

  

  rlagneau

  committed
  52bec05

• [WIP] Added tests for top/bottom/left and top/bottom/right where only left (or right) axis is crossed

  

  rlagneau

  committed
  e595d5c

• [WIP] Added test for top/bottom/right and top/bottom/left where top and bottom are crossed and the third axis is touched

  

  rlagneau

  committed
  baf5b16

• [WIP] Added tests for top/bottom/left and top/bottom/right where the top + the lateral axes are crossed once and the bottom one is not crossed in the RoI

  

  rlagneau

  committed
  ecc391a

Commits on Oct 2, 2023

• [TEST] Added tests for bottom/top/right and bottom/top/left for cases where the circle intersects the only the bottom and lateral side, once each

  

  rlagneau

  committed
  201575c

• [TEST] Made homogeneity in the notation of the tests

  

  rlagneau

  committed
  07ea845

• Added case where all the axes are crossed

  

  rlagneau

  committed
  149a41e

• [FIX] Fixed typos when the left and right axes are the only axes that are crossed

  

  rlagneau

  committed
  fc7500a

• [FIX] Fixed some typos in the left/right/top case + tested all the cases corresponding to this case

  

  rlagneau

  committed
  19f36b7

Commits on Oct 3, 2023

• [CORPS] The method vpImageCircle::computeArcLengthInRoI has been successfully tested =)

  

  rlagneau

  committed
  2bbd72f

• [TUTO] Updated the vpCircleHoughTransform tutorial based on the recent changes on the circle candidate validation policy

  

  rlagneau

  committed
  db37b1f

• Merge branch 'master' into vpCHTimprovements

  

  rlagneau

  committed
  c494c16

• [FIX] Forgot to clean the votes vector

  

  rlagneau

  committed
  e5cce9c