-
Notifications
You must be signed in to change notification settings - Fork 13
Description
Description
-
x.optscontains general parameters for initialization or running ExploreASL, not specific to import or processing -
x.settingscontains settings specific to the processing (should not be used in import) Does it help to rename this tox.Processingor should these go tox.modules.processing.settings? -
x.S(statistics) should be initialized when starting the processing (begin ofExploreASL_ProcessMaster) -
x.Q(quantification) should be initialized when starting the processing (begin ofExploreASL_ProcessMaster) -
x.P(processing paths of the dataset) should be initialized when starting the processing (begin ofExploreASL_ProcessMaster) -
x.datasetshould not be initialized when only initialising ExploreASL? Also, this should probably go underx.opts. -
x.dirshould be a subtract ofx.optsto clarify that these directories are general (non-import/processing specific) -
x.MyPathshould go underx.optsas this is a general option -
x.subject_regexp,x.Quality`` andx.DELETETEMPare specific to processing, so should go tox.module.process` and should not exist before data loading & processing -
x.namethis is the same asx.datasetand can be removed I think -
x.nSubjectsx.nSessionsetc should go underx.module.process -
x.SESSIONScan go to x.module.process.SESSIONS, same forx.SUBJECTSbut perhaps better to wait with this as this would be a huge change
Release notes
The ExploreASL x struct and with that some of the ExploreASL settings were moved to dedicated fields. For backwards compatibility a table was created and automated workflows were implemented.