Skip to content

Conversation

HJZollner
Copy link
Collaborator

-Getting changes from develop into MRSI branch

HJZollner and others added 30 commits March 5, 2021 13:31
Versioning:
-Removed single module versioning
- versioning is defined by single number (Osprey Version)
-Osprey version is used for checking on older containers, which is implemented in the toolbox check function
- Changed logging to Matlab diary function which hopefully allows us logging of MRSI processing etc
- Bugfix of VE11c and VE11E  universal data
-Fixes error that was introduced during merge conflict
-Fixes introduction of multiple new lines in the console during module call
Add a file so GitHub Pages understands it can host the HTML files right away, without building through Jekyll.
-Added basis set for MM suppressed GABA-edited Philips MEGA-PRESS
-This commit fixes the bug reported in issue schorschinho#242. The title definition in the osp_plotModule were changed in v.1.0.0, as individual version numbers of the modules were removed.
-Resolves issues reported in schorschinho#244
Previously, we relied on MATLAB functions only available in the Curve Fitting Toolbox (spmak, fnval) to generate the cubic B-spline spline functions for baseline estimation.
These are now replaced with recursive iteration, eliminating the need for the toolbox altogether. In the process, the function has been cleaned up, removing the second derivatives that were never really used in the first place.
Adding a few lines to the gitignore path, mainly to prevent accidentally committing anything in our own "derivatives" folders in the exampledata folder, as well as overwriting the default SPM path.
Added some fancy shields.io badges to the readme file, like all the other great open source projects have.
Remove the YAML file for the previous Jekyll-style documentation system, which is now obsolete since we switched to bookdown.
Fix small bug in shields.io website-online? badge.
Fix an error that occurred when data with short-TE water, but without water reference, was displayed.
Remove a closed bracket that made the HERMES fitting routine throw an error upon calling.
Fix a bug in the function creating a job file from GUI inputs: Previously, only the first entry in a GUI list of multiple water reference datasets would be parsed correctly into the job file, while the rest of the entries would be erroneously replaced by the metabolite data. This is now corrected.
- Added new version number v.1.0.1 for next release
-Fixes version comparison for v1.0.1 . The integrated version comparison was not properly update, therefore the GUI did not show the content of an already processed MRS container.
HJZollner added 21 commits April 8, 2021 11:06
- updated PE-edited MP basis sets (added EA basis function)
- delete Philips 40 ms unedited basis sets which was not stored in the right folder
- added Philips 80 and 140 ms unedited basis sets
- Removed H2O from the default metabolite list, so that it is not included in the metabolite fit anymore. It can still be included if the user specifies H2O in his metabolite list manually.
- The water basis function is loaded from the specific basis set if it is needed for the fit of the water scan. This happens regardless of the metabolite list the user specifies.
Added a new Osprey module OspreyMinReport.m that creates a markdown table including the minimum parameters for reporting of MRS studies (defined in MRSinMRS). This includes the following changes to other functions:

- Extracting software versions during the loading process (io_loadspec_sdat, io_loadspec_twix, and io_loadspec_GE)
- Automatic generation of md report during OspreyOverview call
- Bug fix related to vendor-specific naming conventions in the voxel geometry
- Minor layout  changes
- Updated the calculation of the fit quality parameter to match the original paper Barros & Slotboom. The quality is defined as a variance ratio between residual in the fitted region and noise
Two minor bug fixes:
1) Possible issues in the metabolite list when new metabolites are added to a default Osprey basis set

2) Avoid crashes when hidden files are stored in the DCM folder
-Added a decision tree, that creates a metabolite list for the preliminary reduced fit according to the input list of the user (e.g. if the user performs a phantom scan with fCho instead of GPC or PCh in the list) this metabolite is picked instead of the default (NAA, Cr, PCh, Glu, and Ins.
Fixes reported Bug in the correlation plot within the GUI where it was not possible to switch between external variables
- OspreyQuantify has now relaxation times for 7T metabolites as well
- fieldstrength is automatically picked
Added 7T relaxation time lookup table
@HJZollner HJZollner merged commit ddb120c into schorschinho:MRSI May 7, 2021
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