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

Brainsfit log file output - Metric value appears to be hard coded #352

Open
tenulate opened this issue Sep 15, 2017 · 2 comments
Open

Brainsfit log file output - Metric value appears to be hard coded #352

tenulate opened this issue Sep 15, 2017 · 2 comments
Assignees
Labels
S2:Ready for Discussion Group input needed to scope solutions (pilot testing)
Milestone

Comments

@tenulate
Copy link

When performing image registrations with Brainsfit the output log file always records the mutual information metric value as -123.457, regardless of the image data used.

I think it is related to the BRAINSFitHelper::RunRegistration() function BRAINSTools/BRAINSCommonLib/BRAINSFitHelper.h, where it appears the code to retrieve the metric value has been commented out and replaced with (line 500)
this->m_FinalMetricValue = -123.456789;

@hjmjohnson
Copy link
Member

It would be great if you had a fix for this. I'd be happy to incorporate.

@hjmjohnson
Copy link
Member

@tenulate This is a lower priority task. We would be happy to accept a pull request if you have a tested solution.

@hjmjohnson hjmjohnson added the S2:Ready for Discussion Group input needed to scope solutions (pilot testing) label Nov 18, 2017
@hjmjohnson hjmjohnson added this to the Spring2018 milestone Nov 18, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S2:Ready for Discussion Group input needed to scope solutions (pilot testing)
Projects
None yet
Development

No branches or pull requests

3 participants