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

[GeodesicSegmentation] Always demands .nii.gz or .nii output extension, even if it already is #1405

Closed
AlexanderGetka-cbica opened this issue Mar 23, 2021 · 0 comments · Fixed by #1406
Assignees

Comments

@AlexanderGetka-cbica
Copy link
Contributor

Describe the bug
No matter what you put for the output filename, GeodesicSegmentation complains that the output isn't in a NIfTI compatible format. You also can't just let it default -- -o is required.

To Reproduce
Steps to reproduce the behavior:

Run GeodesicSegmentation with any input (-i), any mask (-m), and specify any output (-o).
Ex: GeodesicSegmentation -i image.nii.gz -m mask.nii.gz -o output.nii.gz
See error message:
Output file extension needs to be compatible with NIfTI format. Only '.nii.gz' or '.nii' allowed

Try it again with anything as the output, it doesn't work.

Expected behavior
Run the algorithm if I have .nii or .nii.gz extension on my output.

CaPTk Version
1.8.1 release

Additional context
Seems to be coming from src/applications/GeodesicSegmentation.cxx line 293.
Check is:
if ((extension IS NOT .nii) OR (extension IS NOT .nii.gz)) -- this can never be false.

@AlexanderGetka-cbica AlexanderGetka-cbica self-assigned this Mar 23, 2021
AlexanderGetka-cbica added a commit to AlexanderGetka-cbica/CaPTk that referenced this issue Mar 23, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant