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

Fixed ignored -init flag and minor improvements #2198

Merged
merged 5 commits into from Mar 17, 2019
Merged

Conversation

jcohenadad
Copy link
Member

@jcohenadad jcohenadad commented Mar 16, 2019

Due to a regression bug, v4.0.0-beta.1 was not accounting for the flag -init anymore. This PR addresses that issue, as well as other minor improvements:

Because (i) it was saved as centerline.nii.gz and could interact with user's other data and (ii) there is another centerline output, called input_centerline.nii.gz and derived from the segmentation directly, which is more precise, so there is no point of having two output centerlines which are different.
@jcohenadad jcohenadad added bug category: fixes an error in the code sct_propseg context: labels Mar 16, 2019
@jcohenadad jcohenadad modified the milestone: v4.0.0 Mar 16, 2019
@jcohenadad jcohenadad changed the title Fixed ignored -init flag (regression bug) Fixed ignored -init flag and minor improvements Mar 16, 2019
Copy link
Member

@charleygros charleygros left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems good to me!

@charleygros charleygros merged commit a91890e into master Mar 17, 2019
@jcohenadad jcohenadad added this to the v4.0.0 milestone Apr 5, 2019
@gmotzespina gmotzespina deleted the jca/2190-propseg branch July 26, 2019 16:48
jcohenadad pushed a commit that referenced this pull request Dec 18, 2019
Fixed ignored -init flag and minor improvements

Former-commit-id: a91890e
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug category: fixes an error in the code sct_propseg context:
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants