Conversation
Includes checking out a new test data commit that moves existing test data and adds new test data.
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
This comment was marked as outdated.
This comment was marked as outdated.
Sorry, something went wrong.
Member
Author
|
Was potentially a little too idealistic about this metric being useful as a scalar surrogate for multi-contrast registration (as multiple SH images can be provided as input to the entropy calculation). Kind of interesting, and maybe useful in specific use cases such as the unusual template above, but not a game-changer. Single-subject HCP data: |
Lestropie
force-pushed
the
sh2metric
branch
2 times, most recently
from
b2c9fdc to
fe604e9
Compare
Conflicts: cpp/cmd/sh2metric.cpp cpp/cmd/sh2power.cpp cpp/core/math/entropy.h testing/binaries/data
This comment was marked as outdated.
This comment was marked as outdated.
Sorry, something went wrong.
This comment was marked as outdated.
This comment was marked as outdated.
Sorry, something went wrong.
Conflicts: cpp/cmd/sh2power.cpp cpp/core/dwi/tractography/SIFT2/tckfactor.cpp testing/CMakeLists.txt
Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>
Affects the set of directions used to sample SH amplitudes in computation of entropy.
Generated-by: Claude Sonnet 4.6 <noreply@anthropic.com>
Generated-by: Claude Sonnet 4.6 <noreply@anthropic.com>
This comment was marked as outdated.
This comment was marked as outdated.
Sorry, something went wrong.
Prompt: "In file cpp/cmd/sh2metric.cpp, add capability to compute the complement of the SH entropy normalised to the [0.0, 1.0] range. Add new command-line option "-invnorm". The type of scaling to apply---normalisation as currently implemented, the complement of the normalised value, or no normalisation---should be represented as an enumeration.". Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Had previously thought about this from the perspective of registration, but have also been looking at a collaborator dataset where doing satisfactory masking for FBA has been difficult, and using that to revised FBA documentation (#2685).
Here is the WM FOD l=0 image, which you can see doesn't do a very good job of separating what one would like to analyse vs. not:
Then this is the entropy of the WM FOD template (normalised & inverted):
Implement the Spherical Harmonic Entropy (SHE) metric as per:
https://arxiv.org/pdf/1805.08084
This may yield the same or similar result much faster
Add command-line option to modify the direction set that's used for amplitude sampling
Implement entropy operations for:
Revisit templated entropy calculation code
For some reason had trouble specifying the logarithm function as a template. The
switchstatement should get optimised out by the compiler, but maybe I'll have another go before merging.