ITK
ITK copied to clipboard
BUG: fix an infinite loop in itk::VoronoiDiagram2DGenerator
The loop exit condition was present in the code, but never wired and removed via https://github.com/InsightSoftwareConsortium/ITK/commit/cd97879e6bfec6cd9a33a3dd374d68cf4ff4303e
PR Checklist
- [x] No API changes were made (or the changes have been approved)
- [x] No major design changes were made (or the changes have been approved)
- [ ] Added test (or behavior not changed)
- [x] Updated API documentation (or API not changed)
- [x] Added license to new files (if any)
- [x] Added Python wrapping to new files (if any) as described in ITK Software Guide Section 9.5
- [x] Added ITK examples for all new major features (if any)
Refer to the ITK Software Guide for further development details if necessary.
Don't know if you want a regression test, and if your framework can handle "infinite looping" type failures.
A regression test is advantageous. I don't think we have an explicit support for infinite looping, but such a test would fail via timeout.
Added the original case by doing "monkey see monkey do", but it looks like I can't build locally due to IPFS + work firewall or something.
@bradking Is ghostflow acting up? Commit 62457de has a very short commit message.
Nah, the full "http://..." link is probably what triggers it, GitHub prettifies it on display.
@ILoveGoulash Thank you for your contribution. There are just a few final actions that need to be finalized so that we can incorporate this into the main tree.
Should be okay now.
@ILoveGoulash I rebased and removed the merge conflict, then updated the commit message for the regression test to be more descriptive.
Thanks!