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

Confusion over mi vs mi2 metric #1668

Closed
dzenanz opened this issue Jan 29, 2024 · 2 comments
Closed

Confusion over mi vs mi2 metric #1668

dzenanz opened this issue Jan 29, 2024 · 2 comments

Comments

@dzenanz
Copy link
Contributor

dzenanz commented Jan 29, 2024

There is a discrepancy between converting metric from enum to string:
https://github.com/ANTsX/ANTs/blob/v2.5.1/Examples/itkantsRegistrationHelper.h#L253-L256
and string to enum:
https://github.com/ANTsX/ANTs/blob/v2.5.1/Examples/itkantsRegistrationHelper.hxx#L145-L152

I am new to ANTs, so I don't know what is the best way to fix it.

@ntustison
Copy link
Member

Thanks. I'll take a look.

@ntustison
Copy link
Member

Okay, so these two functions serve separate purposes and aren't meant to be used inversely. StringToMetricType() is basically used for reading from the command line. Mattes MI has always been prioritized in ANTs over JointHistogram MI which is why we relegated the latter to "mi2." The other function, GetMetricAsString() is strictly used for printing to the screen. I added a bit of clarification in the latter here. If you feel that this ambiguity needs to be clarified further, just let me know if you have any ideas.

@dzenanz dzenanz closed this as completed Jan 30, 2024
dzenanz added a commit to dzenanz/ANTsWasm that referenced this issue Jan 30, 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

No branches or pull requests

2 participants