shimming-toolbox / shimming-toolbox-matlab

Code for performing real-time shimming using external MRI shim coils
GNU General Public License v3.0
16 stars 5 forks source link

Add MATLAB function to return current toolbox software version #130

Open rtopfer opened 4 years ago

rtopfer commented 4 years ago

To enable legacy support etc. there should be version tags included in saved files such as

Presumably .json should work in all cases? This would be analogous to version tag saved in the .json sidecar output by dcm2niix.

@kousu @jcohenadad perhaps you already know how to do this? Presumably git tags would be involved.

jcohenadad commented 4 years ago

So, there are various levels that needs to be considered here:

now, regarding how we fetch the version info:

gab-berest commented 4 years ago

So, for a one time use I think that logging in a single file should be a good idea since it will be less troublesome to find the information on the specific session (all the information in one place). However, for multiple sessions, it should be in different log files to keep track of the timming of the usage and to not have a single file with 10 000+ lines.

In the end, it all depends if there is a lot of log or not. Longer logs are more difficult to get through and understand, even if all the information is there.

For the version property, I don't think there is one in matlab. It could be a simple function that checks the config file for the version and outputs it.