-
Notifications
You must be signed in to change notification settings - Fork 1
Description
When an app is updated it causes issues if the output_dirs is also not updated, see here: eastgenomics/eggd_conductor_configs#158
Originally designed to be a dict since for workflows there are multiple stages and this maps as here: https://github.com/eastgenomics/eggd_conductor_configs/blob/281b804a95bccadf18e107bf04a7afda385a75a8/assay_configs/PCAN/eggd_conductor_eunomia_PCAN_config_v1.5.0.json#L111
For apps when using dxpy to run an app it needs to be in structure {"app_id": "output_path"}
, maybe change the requirement to test if the output_dirs
is a dict or string, this would allow for apps just defining the output path and if it is a string then use the provided executable
param to set the output dir to be a dict with the given app ID.
Function for setting this is here:
def populate_output_dir_config(self, executable, sample=None): |