-
Notifications
You must be signed in to change notification settings - Fork 38
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Vol add border interior #371
Conversation
ChangeLog.md
Outdated
@@ -11,6 +11,10 @@ | |||
(Roland Denis, [#359](https://github.com/DGtal-team/DGtalTools/pull/359/files)) | |||
- Using SourceForge to download doxygen sources during Travis CI jobs. | |||
(Roland Denis [#360](https://github.com/DGtal-team/DGtalTools/pull/360)) | |||
- volAddBorder: add an option to fill the border of interior volume (Bertrand |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't understand what it means..
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
in other words, it changes the image values of voxels located on the frontier of the domain.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is that :
Add an option that sets zero value to domain boundary voxels without changing the domain extent
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes exactly, thanks I add sorry for the not clear description
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thanks
volumetric/volAddBorder.cpp
Outdated
@@ -102,6 +102,7 @@ int main(int argc, char**argv) | |||
general_opt.add_options() | |||
( "help,h", "display this message." ) | |||
( "input,i", po::value<std::string>(), "Input vol file." ) | |||
( "inside", "Add border inside the volume (set 0 inside the border)." ) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Update the option doc as well ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes oops..
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I also update the doxygen description that I forget ;)
Thx !;) |
PR Description
Add a small option to add border inside the volume
Checklist
cmake
mode (otherwise, Travis C.I. will fail).