Closed rmndrs89 closed 6 months ago
+1 in favor of this!
REQUIRED_COLUMNS has been moved to utils/ngmt_dataclasses.py, the validate_channels
method has been update accordingly, and REQUIRED_COLUMNS
is now loaded with an import in the datasets/keepcontrol.py
script.
https://github.com/neurogeriatricskiel/NGMT/blob/d6cbe3793e82d24902247d2426b4eae174f0909c/ngmt/datasets/keepcontrol.py#L8
Should we move this (global) variable to the ngmt_dataclass.py? It should hold across all datasets, and is not specific to keepcontrol in my view?