search

populse / mri_conv

3 stars 0 forks source link

bvacs /bvecs files by default exported in mia #11

Closed servoz closed 2 years ago

servoz commented 3 years ago

My feeling is that this should only be done as an option under the control of the user (check button, option to be defined somewhere, etc.)

montigno commented 3 years ago

Yes, it's possible. I will see how to add this option (in a menu probably).

servoz commented 2 years ago

As agreed, I have just tested the devt branch and we are finding the bvals/bvecs files for each image, even if it is not a diffusion MRI. Below rocmi_120618test1-IRMFonct_+perfusion-2018-06-12060428-02-T13DSENSE-T1TFE-000424_000 is an Anat and rocmi_120618test1-IRMFonct_+perfusion-2018-06-12060428-00-BOLD_CVR_7_53sl_ModeratePNS-FEEPI-001212_000 is a BOLD, for patient rocmi:

% ls
logExport_2021-09-02_12-42-09
logExport_2021-09-02_12-42-09.json
rocmi_120618test1-IRMFonct_+perfusion-2018-06-12060428-00-BOLD_CVR_7_53sl_ModeratePNS-FEEPI-001212_000-bvals-MRtrix.txt
rocmi_120618test1-IRMFonct_+perfusion-2018-06-12060428-00-BOLD_CVR_7_53sl_ModeratePNS-FEEPI-001212_000-bvecs-bvals-MRtrix.txt
rocmi_120618test1-IRMFonct_+perfusion-2018-06-12060428-00-BOLD_CVR_7_53sl_ModeratePNS-FEEPI-001212_000-bvecs-MRtrix.txt
rocmi_120618test1-IRMFonct_+perfusion-2018-06-12060428-00-BOLD_CVR_7_53sl_ModeratePNS-FEEPI-001212_000.json
rocmi_120618test1-IRMFonct_+perfusion-2018-06-12060428-00-BOLD_CVR_7_53sl_ModeratePNS-FEEPI-001212_000.nii
rocmi_120618test1-IRMFonct_+perfusion-2018-06-12060428-02-T13DSENSE-T1TFE-000424_000-bvals-MRtrix.txt
rocmi_120618test1-IRMFonct_+perfusion-2018-06-12060428-02-T13DSENSE-T1TFE-000424_000-bvecs-bvals-MRtrix.txt
rocmi_120618test1-IRMFonct_+perfusion-2018-06-12060428-02-T13DSENSE-T1TFE-000424_000-bvecs-MRtrix.txt
rocmi_120618test1-IRMFonct_+perfusion-2018-06-12060428-02-T13DSENSE-T1TFE-000424_000.json
rocmi_120618test1-IRMFonct_+perfusion-2018-06-12060428-02-T13DSENSE-T1TFE-000424_000.nii
% more rocmi_120618test1-IRMFonct_+perfusion-2018-06-12060428-02-T13DSENSE-T1TFE-000424_000-bvals-MRtrix.txt
0.00
more rocmi_120618test1-IRMFonct_+perfusion-2018-06-12060428-02-T13DSENSE-T1TFE-000424_000-bvecs-bvals-MRtrix.txt
-0.000 -0.000 0.000 0.00
% more rocmi_120618test1-IRMFonct_+perfusion-2018-06-12060428-02-T13DSENSE-T1TFE-000424_000-bvecs-MRtrix.txt
-0.000
-0.000
0.000

As mentioned in the first post and to complete it. Bvals/bvecs should only be created when it is a diffusion experience. If this is not possible (or if there is no certainty that it will work properly in all cases), users should be able to choose whether they want bvals/bvecs( e.g. with a checkbox) . The first solution is the best, but the second would be acceptable.

montigno commented 2 years ago

Effectively, this problem occurs with Philips data, not with Bruker or Dicom. Ok, I will look at this !

montigno commented 2 years ago

Now the problem is solved (I hope so !) .

The last version has been pushed in master and devpt branches.

servoz commented 2 years ago

Yes it's doing the job now !

montigno commented 2 years ago

nice !