fiuneuro / cis-processing

Automated transfer, conversion, and processing of data acquired at the FIU Center for Imaging Science.
Apache License 2.0
4 stars 4 forks source link

Rename modules #59

Closed tsalo closed 4 years ago

tsalo commented 4 years ago

cis.py and cis_proc.py should be renamed.

Each module has both a main workflow function and associated CLI. A brief summary of each workflow:

NOTE: Neither workflow uses mriqc.mriqc_group() at the moment.

Based on what each workflow does, what do you think we should rename them to?

akimbler commented 4 years ago

@tsalo maybe cis_proc -> build_bids and cis -> pull_dicoms

tsalo commented 4 years ago

I like pull_dicoms, but instead of build_bids what about conversion_workflow? I think including workflow in the filenames will make it easier for people to distinguish between what they should run and what it called internally.