sarek icon indicating copy to clipboard operation
sarek copied to clipboard

Error handle for sentieon-dedup with non-sentieon aligner

Open asp8200 opened this issue 1 year ago • 1 comments

Description of the bug

There is nothing preventing the user from selecting the tool sentieon_dedup with a non-sentieon aligner, that combination however causes the pipeline to crash.

https://nfcore.slack.com/archives/CGFUX04HZ/p1713879506993449

No user has requested that it should be possible to run such a combination, but the "bug" did pop up during the test done by a user.

Unless the combination is requested by a user, let's just have the pipeline stop with a suitable error message on such a combination of tool and aligner.

Command used and terminal output

No response

Relevant files

No response

System information

No response

asp8200 avatar Apr 23 '24 19:04 asp8200

With nf-shema, the problem can be solved by adding the following to nextflow.config :

            "if": {
                "properties": {
                  "tools": { "pattern": ".*sentieon_dedup.*" }
                }
            },
            "then": {
                "properties": {
                  "aligner": { "const": "sentieon-bwamem" }
                }
            }

https://nextflow.slack.com/archives/C056RQB10LU/p1713899757306899?thread_ts=1713799615.441729&cid=C056RQB10LU

asp8200 avatar Apr 23 '24 19:04 asp8200