Skip to content

Remove requirement for app output_dirs needing to contain app ID #174

@jethror1

Description

@jethror1

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):

Metadata

Metadata

Assignees

No one assigned

    Labels

    enhancementNew feature or request

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions