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: add lower triangular scalars from DTI (could be useful for ML)? #863

Merged
merged 2 commits into from
Aug 3, 2022

Conversation

36000
Copy link
Collaborator

@36000 36000 commented Aug 3, 2022

These can already be calculated from our derivatives, but now they are formally added to the task system.
Also adds a change for tract profile calculation that should significantly speed it up.

@36000
Copy link
Collaborator Author

36000 commented Aug 3, 2022

@arokem this might be useful for the final NKI run.

Copy link
Collaborator

@arokem arokem left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Definitely an improvement! I have some questions about how we might incorporate DKI in here as well.

AFQ/tasks/segmentation.py Show resolved Hide resolved
@@ -280,6 +280,24 @@ def dti_fa(dti_tf):
return dti_tf.fa


@pimms.calc("dti_lt0", "dti_lt1", "dti_lt2", "dti_lt3", "dti_lt4", "dti_lt5")
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do we want dki_lt elements as well? I believe there are 15 elements in DKI.

And relatedly, we often calculate FA/MD from DKI, when that's doable (i.e., when there are multiple non-zero b-values). Do we want to be able to extract the six DTI elements from DKI?

@arokem
Copy link
Collaborator

arokem commented Aug 3, 2022

@arokem this might be useful for the final NKI run.

Agreed! And the DKI questions do not pertain to that, so we could merge this as-is for that, and then come back to deal with those questions later on.

@36000
Copy link
Collaborator Author

36000 commented Aug 3, 2022

@arokem can we merge this and do DKI in a separate PR?

@arokem arokem merged commit 0ad4dcb into yeatmanlab:master Aug 3, 2022
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.

2 participants