shandley / hecatomb

hecatomb is a virome analysis pipeline for analysis of Illumina sequence data
MIT License
53 stars 12 forks source link

hecatomb combine not working #107

Open gregorykrice opened 3 months ago

gregorykrice commented 3 months ago

Running "hecatomb combine --comb SRR4295172/hecatomb.out --comb SRR4295173/hecatomb.out --comb SRR4295174/hecatomb.out --comb SRR4295175/hecatomb.out --output combined_output"

[2024:03:07 10:57:15] Config file combined_output/hecatomb.config.yaml already exists. Using existing config file. [2024:03:07 10:57:15] Updating config file with new values [2024:03:07 10:57:15] Writing config file to combined_output/hecatomb.config.yaml [2024:03:07 10:57:15] ------------------ [2024:03:07 10:57:15] | Runtime config | [2024:03:07 10:57:15] ------------------

hecatomb: addHost: minViralAlnLen: 1000 args: comb:

[2024:03:07 10:57:15] Config file combined_output/hecatomb.profile/config.yaml already exists. Using existing config file. [2024:03:07 10:57:15] --------------------- [2024:03:07 10:57:15] | Snakemake command | [2024:03:07 10:57:15] ---------------------

snakemake -s /opt/mambaforge/envs/hecatomb/lib/python3.12/site-packages/hecatomb/snakemake/workflow/combineOutputs.smk --configfile combined_output/hecatomb.config.yaml --cores 12 --use-conda --conda-prefix /opt/mambaforge/envs/hecatomb/lib/python3.12/site-packages/hecatomb/snakemake/conda --workflow-profile combined_output/hecatomb.profile Using profile combined_output/hecatomb.profile and workflow specific profile combined_output/hecatomb.profile for setting default command line arguments. Config file /opt/mambaforge/envs/hecatomb/lib/python3.12/site-packages/hecatomb/snakemake/workflow/../config/config.yaml is extended by additional config specified via the command line. Config file /opt/mambaforge/envs/hecatomb/lib/python3.12/site-packages/hecatomb/snakemake/workflow/../config/dbFiles.yaml is extended by additional config specified via the command line. Config file /opt/mambaforge/envs/hecatomb/lib/python3.12/site-packages/hecatomb/snakemake/workflow/../config/immutable.yaml is extended by additional config specified via the command line. No/missing assembly files for SRR4295172/hecatomb.out, skipping assembly files. No/missing assembly files for SRR4295173/hecatomb.out, skipping assembly files. No/missing assembly files for SRR4295174/hecatomb.out, skipping assembly files. No/missing assembly files for SRR4295175/hecatomb.out, skipping assembly files. TypeError in file /opt/mambaforge/envs/hecatomb/lib/python3.12/site-packages/hecatomb/snakemake/workflow/rules/preflight/contig_mapping.smk, line 8: unsupported operand type(s) for +: 'NoneType' and 'str' File "/opt/mambaforge/envs/hecatomb/lib/python3.12/site-packages/hecatomb/snakemake/workflow/combineOutputs.smk", line 52, in File "/opt/mambaforge/envs/hecatomb/lib/python3.12/site-packages/hecatomb/snakemake/workflow/rules/preflight/contig_mapping.smk", line 8, in [2024:03:07 10:57:15] ERROR: Snakemake failed

The comb directories have results/merged_assembly.fasta files, but it looks like combineOutputs.smk is expecting assembly.fasta files.

beardymcjohnface commented 3 months ago

This might be a simple fix. Can you please try and update the combined_output/hecatomb.config.yaml file? I think you just need to add a config setting:

hecatomb:
    args:
+        assembly: merged

If that works I'll push a fix for the next release. I also need to add build tests for this workflow at some point.

gregorykrice commented 3 months ago

[2024:03:13 11:51:59] Config file combined_output/hecatomb.config.yaml already exists. Using existing config file. [2024:03:13 11:51:59] Updating config file with new values [2024:03:13 11:51:59] Writing config file to combined_output/hecatomb.config.yaml [2024:03:13 11:51:59] ------------------ [2024:03:13 11:51:59] | Runtime config | [2024:03:13 11:51:59] ------------------

hecatomb: addHost: minViralAlnLen: 1000 args: assembly: merged comb:

[2024:03:13 11:51:59] Config file combined_output/hecatomb.profile/config.yaml already exists. Using existing config file. [2024:03:13 11:51:59] --------------------- [2024:03:13 11:51:59] | Snakemake command | [2024:03:13 11:51:59] ---------------------

snakemake -s /opt/mambaforge/envs/hecatomb/lib/python3.12/site-packages/hecatomb/snakemake/workflow/combineOutputs.smk --configfile combined_output/hecatomb.config.yaml --cores 12 --use-conda --conda-prefix /opt/mambaforge/envs/hecatomb/lib/python3.12/site-packages/hecatomb/snakemake/conda --workflow-profile combined_output/hecatomb.profile Using profile combined_output/hecatomb.profile and workflow specific profile combined_output/hecatomb.profile for setting default command line arguments. Config file /opt/mambaforge/envs/hecatomb/lib/python3.12/site-packages/hecatomb/snakemake/workflow/../config/config.yaml is extended by additional config specified via the command line. Config file /opt/mambaforge/envs/hecatomb/lib/python3.12/site-packages/hecatomb/snakemake/workflow/../config/dbFiles.yaml is extended by additional config specified via the command line. Config file /opt/mambaforge/envs/hecatomb/lib/python3.12/site-packages/hecatomb/snakemake/workflow/../config/immutable.yaml is extended by additional config specified via the command line. No/missing assembly files for SRR4295172/hecatomb.out, skipping assembly files. No/missing assembly files for SRR4295173/hecatomb.out, skipping assembly files. No/missing assembly files for SRR4295174/hecatomb.out, skipping assembly files. No/missing assembly files for SRR4295175/hecatomb.out, skipping assembly files. KeyError in file /opt/mambaforge/envs/hecatomb/lib/python3.12/site-packages/hecatomb/snakemake/workflow/combineOutputs.smk, line 162: 'resources' File "/opt/mambaforge/envs/hecatomb/lib/python3.12/site-packages/hecatomb/snakemake/workflow/combineOutputs.smk", line 162, in [2024:03:13 11:51:59] ERROR: Snakemake failed

awk 'NR>=161 && NR<=163 {print}' /opt/mambaforge/envs/hecatomb/lib/python3.12/site-packages/hecatomb/snakemake/workflow/combineOutputs.smk threads: config["resources"]["med"]["cpu"] resources:

beardymcjohnface commented 3 months ago

I think there are a couple of things I need to fix. I'll see if I can get it done this week.