search

Donders-Institute / PRESTUS

PREprocessing & Simulations for Transcranial Ultrasound Stimulation package
GNU General Public License v3.0
12 stars 10 forks source link

Results involving a skull mask are not saved in the CSV #18

Closed KTZ228 closed 1 year ago

KTZ228 commented 1 year ago

In the CSV, values such as the max_Isppa_skull, max_pressure_skull and maxT_skull are calculated based on a skull mask. In the current iteration, the CSV contains NaN values where the results should be.

KTZ228 commented 1 year ago

It appears that the medium_masks in the single_subject_pipeline do not contain the updated layer labels produced by charm, but instead use the old format of layer labels that were previously produced by headreco. Since the new skull_ids are assigned to all_skull_ids in lines 284-286 (single_subject_pipeline), none of these values are found in medium_masks resulting in a skull_mask containing only 0's.

KTZ228 commented 1 year ago

Issue is fixed (around line 285 of the single_subject_pipeline, depending on the version), will add to the list of things to push to the master:)