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

Refine PIESNO #1231

Open
arokem opened this issue Apr 3, 2024 · 1 comment
Open

Refine PIESNO #1231

arokem opened this issue Apr 3, 2024 · 1 comment
Labels

Comments

@arokem
Copy link
Collaborator

arokem commented Apr 3, 2024

What would you like to see added in this software?

PIESNO calculation depends on the number of coils that were used for measurement and the recon algorithm. See dipy docstring. Realizing that many datasets would not have this information, I wonder whether there is any way to use it when available.

Do you have any interest in helping implement the feature?

Yes

Additional information / screenshots

No response

@oesteban
Copy link
Member

oesteban commented Apr 3, 2024

It boils down to whether there is enough metadata stored in the sidecar json. This node:

meta = pe.Node(ReadDWIMetadata(index_db=config.execution.bids_database_dir), name='metadata')

extracts the metadata, and has a dictionary at the output that can be checked (I would imagine for number of coils and type of acceleration?)

The problem is that different converters may name the metadata differently.

Once the heuristic is implemented, we just need to plug it in into the PIESNO node, which already has an n_channels input for it.

@oesteban oesteban added the dMRI label Apr 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants