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

Add option for MINC2 support to superbuild #198

Merged
merged 1 commit into from
Sep 21, 2015
Merged

Conversation

gdevenyi
Copy link
Contributor

Added support for MINC2 in superbuild

@vfonov
Copy link
Contributor

vfonov commented Jul 3, 2015

I made changes to v2.1.0 to work properly with minc-style transformation files. Should I send a merge request?

@ntustison
Copy link
Member

Hi Vlad, I defer to Brian on this but he's currently traveling so he'll have to weigh in when he gets a chance. Thanks.

@gdevenyi
Copy link
Contributor Author

gdevenyi commented Jul 3, 2015

@vfonov Yes please, xfm/mnc support for the transforms would be very helpful in my existing toolchains.

@gdevenyi
Copy link
Contributor Author

gdevenyi commented Jul 3, 2015

Hrmm, travis build is failing because travis has old cmake.

@ntustison
Copy link
Member

travis-ci/travis-ci#2030

@gdevenyi
Copy link
Contributor Author

gdevenyi commented Jul 8, 2015

I think this PR needs to be merged for the MINC features merged from @vfonov to work properly, in fact, I think given the merged code MINC always has to be built now for ANTS to work...?

@gdevenyi gdevenyi changed the title Add option for MINC2 support Add option for MINC2 support to superbuild Jul 10, 2015
@gdevenyi
Copy link
Contributor Author

Still waiting on this patch to be merged. All it does is add MINC as a configuration option during the build...

ntustison added a commit that referenced this pull request Sep 21, 2015
Add option for MINC2 support to superbuild
@ntustison ntustison merged commit 6a8df44 into ANTsX:master Sep 21, 2015
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 this pull request may close these issues.

3 participants