-
Notifications
You must be signed in to change notification settings - Fork 44
Prepare release v2.1.0 #399
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
Merged
Conversation
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
Updated io_writeniimrs to work with more general input data. This should now manage any 7D SVS data that uses Matlab MRS structure. Fixed error in main loop of NIFTI write routine, and added a temporary seq label for NFITI data. Fixed error in osp_processHERCULES concatenation of ref averages
This work has been supported by NIH grants R01 EB016089, P41 EB15909, P41 EB031771, R01 EB023963, and K99 AG062230.
Edit ReadMe.md
- Bug fix for single average HERMES/HERCULES Siemens data - Feature request: Update Twix loader to accept combined fMRI/fMRS sequence
Update io_loadspec_twix
- Adapted struct output according to new QM measures
Update OspreyMinReport
Some plot lines in osp_plotLoad and osp_plotProcess did not specify the real part of spectra to be plotted, so the obligatory "Warning: Imaginary parts ignored" would pop up. This commit fixes this behavior.
Added voxel coregistration for Nii MRS. Currently working for Philips data but not Siemens. Main changes: Added coreg_nifti.m which uses nii_tool to coregister voxel to structural. Added nii_xform to lib - function used by nii_tool updated OspreyCoreg to handle NIFTI Modified osp_fitinitialise to throw warning if sequence type not detected Modified osp_LoadNii to try and populate vendor/seq fields of MRSCont Fixed bug in osp_plotSegment
Intercept and skip read errors during Siemens DICOM header parsing. Add missing flag indicating whether multi-file IMA data have been averaged on-scanner. Fix an error resulting from an unknown `seq` variable. Fixes #395
Empty protocol name handling for NIfTI MRS, and bug fix in osp_combineCoils
Integrate NIfTI MRS read/write/coreg implementaion
Fix RDA writing error (data points were not ordered correctly).
Fixes TWIX coil combination procedure in case no water reference is provided.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Update master to current develop version to prepare release.