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

ENH: Output precision type is added to BRAINSTransformConvert #58

Merged
merged 1 commit into from
Sep 18, 2013

Conversation

aghayoor
Copy link
Collaborator

@aghayoor aghayoor commented Aug 8, 2013

A new flags are added to BRAINSTransformConvert tool:
--outputPrecisionType

This flag can be set as "float" or "double", and it can work well with
other options like outputTransformType.

This patch is tested and works properly based on the last version of ITK
that includes updated transform file reader/writer filters.
The updated filters can convert their input transforms to the requested
output precision type automatically.

Testsuite is changed to add float precision tests. Number of tests are
increased from 63 to 125 and all of them are passed successfully.
"100% tests passed, 0 tests failed out of 125".

This patch can be merged safely.

@aghayoor
Copy link
Collaborator Author

aghayoor commented Aug 8, 2013

@hjmjohnson : Please review. I tested this patch based on the new version of transform file reader/writer filters, and all tests are passed successfully.

A new flags are added to BRAINSTransformConvert tool:
--outputPrecisionType

This flag can be set as "float" or "double", and it can work well with
other options like outputTransformType.

This patch is tested and works properly based on the last version of ITK
that includes updated transform file reader/writer filters.
The updated filters can convert their input transforms to the requested
output precision type automatically.

Testsuite is changed to add float precision tests. Number of tests are
increased from 63 to 125 and all of them are passed successfully.
"100% tests passed, 0 tests failed out of 125"
@ghost ghost assigned hjmjohnson Sep 18, 2013
hjmjohnson added a commit that referenced this pull request Sep 18, 2013
ENH: Output precision type is added to BRAINSTransformConvert
@hjmjohnson hjmjohnson merged commit d282999 into BRAINSia:master Sep 18, 2013
@aghayoor aghayoor deleted the BrainsTransformConverter_final branch December 22, 2016 16:49
@aghayoor aghayoor restored the BrainsTransformConverter_final branch December 22, 2016 16:49
@aghayoor aghayoor deleted the BrainsTransformConverter_final branch December 22, 2016 16:49
@Hui-Xie Hui-Xie mentioned this pull request Feb 12, 2017
@hjmjohnson hjmjohnson mentioned this pull request Jun 5, 2024
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.

None yet

2 participants