Skip to content
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

Fix sliceViewer #256

Merged
merged 3 commits into from
Mar 22, 2016
Merged

Fix sliceViewer #256

merged 3 commits into from
Mar 22, 2016

Conversation

rolanddenis
Copy link
Member

PR Description

Fixing sliceViewer bug that cause assert failure and exception raise when the imported image domain doesn't contain the (0,0,0) point (like the example file provided in #255).

Checklist

  • [N/A] Doxygen documentation of the code completed (classes, methods, types, members...)
  • [N/A] Check if it follows the tools structure described in CONTRIBUTING.md
  • New entry in the ChangeLog.md added.
  • [N/A] Update the readme with potentially a screenshot of the tools if it applies.
  • No warning raised in Debug cmake mode (otherwise, Travis C.I. will fail).

@rolanddenis rolanddenis changed the title Fix slice viewer Fix sliceViewer Mar 22, 2016
@kerautret
Copy link
Member

Nice, thanks a lot @rolanddenis , works fine now ;) !
Just waiting AppVeyor and merging

kerautret added a commit that referenced this pull request Mar 22, 2016
@kerautret kerautret merged commit 695b10a into DGtal-team:master Mar 22, 2016
@rolanddenis rolanddenis deleted the fix_sliceViewer branch March 23, 2016 10:10
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.

None yet

2 participants