search

Donders-Institute / PRESTUS

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

[bug] max. CEM43 = 0 #45

Closed jkosciessa closed 4 days ago

jkosciessa commented 3 weeks ago

This affects the currently unmerged updated branch in https://github.com/Donders-Institute/PRESTUS/pull/34.

The simulations run successfully, including calculation of CEM43 metrics, but the output table always codes CEM43 as 0. Can you take a look @sirmrmarty?

sub-001_layered_CEM_by_time_left_MD_duty_20_trials_3200 sub-001_layered_output_table_left_MD_duty_20_trials_3200.csv

[I also wonder why we obtain max. ISPPA of 45 W/cm2 with free-water output of max. 40 W/cm2]

sirmrmarty commented 6 days ago

I fix the bug and commit it into your branch

jkosciessa commented 4 days ago

Fixed with commit f3e569f2be4935d2da04e9639821bc788ac8a4d8

jkosciessa commented 4 days ago

Fixed another skin/brain indexing error in commit 59c8c1a200b9fa2dffa123d406a9de31091e35c0