Skip to content

Add SplitFunctions to split domains in hypercubes - #1521

Open
phcerdan wants to merge 3 commits into
DGtal-team:mainfrom
phcerdan:split_functions0
Open

Add SplitFunctions to split domains in hypercubes#1521
phcerdan wants to merge 3 commits into
DGtal-team:mainfrom
phcerdan:split_functions0

Conversation

@phcerdan

@phcerdan phcerdan commented Oct 7, 2020

Copy link
Copy Markdown
Member

Given a domain defined by lower and upper bound points and
requested_number_of_splits implements two functions:

  • computeSplits returns the number of splits that the
    region can hold (lesser or equal than the requested)
  • getSplit returns the ith split and the outputLowerBound and
    outputUpperBound of the split region.

Implementation acquired from ITK: itkImageRegionSplitterMultidimensional.cxx
https://github.com/InsightSoftwareConsortium/ITK/blob/f971477cdacff22a861dfcbaccd89c0cf2755af7/Modules/Core/Common/src/itkImageRegionSplitterMultidimensional.cxx

  • Add splitComplex to SplitFunctions
  • Add test and change testSplitFunctions to use Catch

Checklist

  • Unit-test of your feature with Catch.
  • Doxygen documentation of the code completed (classes, methods, types, members...)
  • [NA] Documentation module page added or updated.
  • New entry in the ChangeLog.md added.
  • No warning raised in Debug cmake mode (otherwise, Travis C.I. will fail).
  • All continuous integration tests pass (Travis & appveyor)

@phcerdan
phcerdan force-pushed the split_functions0 branch 2 times, most recently from 68c948a to 2b397e4 Compare October 7, 2020 14:16
@phcerdan

phcerdan commented Oct 7, 2020

Copy link
Copy Markdown
Member Author

This is a rebase from #1448. The thinning algorithm with splits was discarded.

Given a domain defined by lower and upper bound points and
requested_number_of_splits implements two functions:
- computeSplits returns the number of splits that the
region can hold (lesser or equal than the requested)
- getSplit returns the ith split and the outputLowerBound and
outputUpperBound of the split region.

Implementation acquired from ITK: itkImageRegionSplitterMultidimensional.cxx
https://github.com/InsightSoftwareConsortium/ITK/blob/f971477cdacff22a861dfcbaccd89c0cf2755af7/Modules/Core/Common/src/itkImageRegionSplitterMultidimensional.cxx

- Add splitComplex to SplitFunctions
- Add test and change testSplitFunctions to use Catch
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants