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

Include/exclude MP(2)RAGE results? #84

Open
dpapp86 opened this issue Jan 25, 2024 · 3 comments
Open

Include/exclude MP(2)RAGE results? #84

dpapp86 opened this issue Jan 25, 2024 · 3 comments
Labels
wontfix This will not be worked on

Comments

@dpapp86
Copy link
Collaborator

dpapp86 commented Jan 25, 2024

I'm starting this discussion here as it will require some data upload/deletion/coding to sort out.

As far as I am concerned, the two main results are the B1+ maps and the B1+ distribution along the SC ("we said we can shim, and indeed, we can"), and the CSF/SC signal ration in the GRE scans ("we said we can shim and this is the effect").

Originally, we added the MP1RAGE scans to show the effect on a more "clinically relevant" scan type.

Then you had a good comment on how the MP2RAGE figure from Gaspard looks much nicer.

So I see three ways forward:

1, We only show the results on B1+ maps and the GRE

2, We show that plus the MPRAGE scans.
Pro: we can do stats on it, as we have five subjects
Con: the results are not as "popping"

3, We show the MP2RAGE scan
Pro: quite impressive results
Con: arrived at using a different workflow, might conflict with the paper's messaging

@jcohenadad what is your opinion?

@jcohenadad
Copy link
Member

I'd go with "1" for the first submission and keep the MP2RAGE in the fridge to feed reviewers with additional food if they are hungry.

@dpapp86
Copy link
Collaborator Author

dpapp86 commented Jan 25, 2024

Agreed.

@jcohenadad
Copy link
Member

For the record, we're talking about subject ACDC_026

@jcohenadad jcohenadad added the wontfix This will not be worked on label Jan 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
wontfix This will not be worked on
Projects
None yet
Development

No branches or pull requests

2 participants