search

ChristianGaser / cat12

Computational Anatomy Toolbox for SPM12
https://neuro-jena.github.io/cat
GNU General Public License v2.0
33 stars 5 forks source link

Removed kamap #10

Closed robdahn closed 11 months ago

robdahn commented 11 months ago

Removed kamap segmentation that was introduced in 2019 as alternative to the SPM unified segmentation but never really used. Removed the cat_main_kamap functions and its calls from cat_main[1639], cat_main_updateSPM[1639], cat_main_reportstring, and the definition of the extopts.segmentation.spm_kamap parameter from cat_conf_extopts and standalone scripts. Keep the enhanced cleanup option in cat_main_clean_gwc[1639] as it is also used for T2/PD images.

ChristianGaser commented 11 months ago

MRF für Amap würde ich trotzdem behalten und wird eben einfach immer auf 0 gesetzt...

Am 21.11.2023 um 13:07 schrieb Robert @.***>:

Removed kamap segmentation that was introduced in 2019 as alternative to the SPM unified segmentation but never really used. Removed the cat_main_kamap functions and its calls from cat_main[1639], cat_main_updateSPM[1639], cat_main_reportstring, and the definition of the extopts.segmentation.spm_kamap parameter from cat_conf_extopts and standalone scripts. Keep the enhanced cleanup option in cat_main_clean_gwc[1639] as it is also used for T2/PD images. You can view, comment on, or merge this pull request online at: https://github.com/ChristianGaser/cat12/pull/10 Commit Summary • ddd64e1 Removed kamap File Changes (14 files) • M batches/cat12_postmortem.m (1) • M cat_conf_extopts.m (50) • M cat_defaults.m (1) • M cat_main.m (22) • M cat_main1639.m (22) • M cat_main_clean_gwc.m (6) • M cat_main_clean_gwc1639.m (7) • D cat_main_kamap.m (284) • M cat_main_reportstr.m (28) • M cat_main_updateSPM.m (15) • M cat_main_updateSPM1639.m (15) • M standalone/cat_standalone_segment.m (3) • M standalone/cat_standalone_segment_enigma.m (3) • M standalone/cat_standalone_segment_long.m (1) Patch Links: • https://github.com/ChristianGaser/cat12/pull/10.patch • https://github.com/ChristianGaser/cat12/pull/10.diff — Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you are subscribed to this thread.Message ID: @.***>

robdahn commented 11 months ago

The MRF AMAP parameter is not related to these parameter/function.

I also ran additional testes SPM and CAT restarts (default+expert+developer) and cross-sectional segmented a 2 mm image to test changes in the CAT report.