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

[BUG] co-registration doesn't work properly in newer versions #24

Open
peter-mri opened this issue Mar 24, 2023 · 3 comments
Open

[BUG] co-registration doesn't work properly in newer versions #24

peter-mri opened this issue Mar 24, 2023 · 3 comments
Labels
bug Something isn't working

Comments

@peter-mri
Copy link

Contact details

peter.seres@ualberta.ca

Describe the bug

We have used Gannet 3.0 to co-register .rda spectra (PRESS in muscle, not MEGA-PRESS) on anatomical images and it works fine. However it doesn't work properly in 3.3.x. Not sure f relevant, but noticed that newer version reports voxel dimensions flipped compared to old version.

Steps to reproduce the bug

disp('Loading spectra');
MRS_Struct=GannetLoad({rda_file});
MRS_Struct.ii=1; % trick Gannet to use any spectra, not just MEGAPRESS
MRS_Struct.p.numScans=1; % trick Gannet 3.3 to use any spectra
disp('Co-registering');
MRS_Struct=GannetCoRegister(MRS_Struct,{anat_file});

Screenshots

3p0
3p3p1

Version

33.1

Operating system

Windows

MATLAB version

R2022b

Additional context

No response

@peter-mri peter-mri added the bug Something isn't working label Mar 24, 2023
@markmikkelsen
Copy link
Owner

Hi @peter-mri,

Firstly, .rda is not recommended. It's much better to export your data in TWIX (.dat) format.

Secondly, if you are co-registering MRS voxels that are not from edited acquisitions, you should use CoRegStandAlone.m. See these instructions for details.

Note that for muscle data, the segmentation will not be appropriate as the SPM template is for the adult brain. Happy to help you adjust this if you want to segment your images/MRS voxels for muscle (if there's an appropriate template available).

@peter-mri
Copy link
Author

Appreciate the super-quick response Mark. CoRegStandAlone returns the same (erroneous) output. I understand that .dat files are preferred, but there are sometimes limitations saving those (they were not saved for this study). Since the software offers support for DICOM and .rda files, we were hoping to use those. I found it curious that .rda files worked better in previous versions, but not in current one. We have an acceptable work-around for our data (using 3.0), I just figured you might be interested in the changed behaviour.

@markmikkelsen
Copy link
Owner

Are you able to share your data so I can explore the problem in the newer version? My email is mam4041@med.cornell.edu

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants