Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Data type in MQC report is always "Paired End" for TSV input, regardless of SeqType in TSV. #757

Closed
1 of 5 tasks
TCLamnidis opened this issue May 26, 2021 · 1 comment · Fixed by #758
Closed
1 of 5 tasks
Labels
bug Something isn't working

Comments

@TCLamnidis
Copy link
Collaborator

Check Documentation

I have checked the following places for your error:

Description of the bug

When TSV input is used, the parameter single_end is left as false. It seems that within MQC it is that parameter that is checked to report the "Data Type" in the "Workflow Summary" section, leading to mixed messages when TSV input is used on SE or mixed data.

Steps to reproduce

Steps to reproduce the behaviour:

  1. Command line: nextflow run --input <any TSV with SE data>
  2. See error: None. The pipeline does not fail, simply misreports a field in MQC.

Expected behaviour

For the "Data Type" field in the MQC report, it should ideally say "Single End" if a TSV of SE data is provided, and/or "Mixed/Unknown" if the TSV has a mix of PE and SE data.

Log files

Have you provided the following extra information/files:

  • The command used to run the pipeline
  • The .nextflow.log file
  • The exact error:

System

  • Hardware: HPC
  • Executor: slurm
  • OS: Linux Ubuntu
  • Version 18.04.5

Nextflow Installation

  • Version: 20.10.0

Container engine

  • Engine: Singularity
  • version: 3.5.1+124-g54a90cd63+dirty
  • Image tag: nfcore-eager-2.3.{3,4}.img

Additional context

@TCLamnidis TCLamnidis added the bug Something isn't working label May 26, 2021
@jfy133
Copy link
Member

jfy133 commented May 26, 2021

Ooof... This is a tricky one... I'm not sure how we can get around this one unfortunately given its a Boolean parameter... Can look into it though.

@jfy133 jfy133 added this to the 2.3.5 patch release milestone May 28, 2021
@jfy133 jfy133 mentioned this issue May 28, 2021
11 tasks
@jfy133 jfy133 linked a pull request May 28, 2021 that will close this issue
11 tasks
@jfy133 jfy133 closed this as completed May 28, 2021
@jfy133 jfy133 mentioned this issue Jun 2, 2021
11 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants