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

merge bwamem_index and remove unneeded multiqc options #102

Merged
merged 11 commits into from
Aug 12, 2024

Conversation

tkchafin
Copy link
Contributor

@tkchafin tkchafin commented Aug 2, 2024

PR checklist

  • This comment contains a description of changes (with reason).
  • If you've fixed a bug or added code that should be tested, add tests!
  • If you've added a new tool - have you followed the pipeline conventions in the contribution docs
  • Make sure your code lints (nf-core lint).
  • Ensure the test suite passes (nextflow run . -profile test,docker --outdir <OUTDIR>).
  • Usage Documentation in docs/usage.md is updated.
  • Output Documentation in docs/output.md is updated.
  • CHANGELOG.md is updated.
  • README.md is updated (including new tool citations and authors/contributors).

@tkchafin tkchafin requested a review from reichan1998 August 2, 2024 07:37
@tkchafin
Copy link
Contributor Author

tkchafin commented Aug 7, 2024

@reichan1998 Good catch on the -resume issue! I think I have figured it out and fixed the issue. The problem was caused by special characters in the file names for the hic and illumina samples, and the fact that I was loading meta.file as a file-path object (nextflow.file.http.XPath) and not a string. These special characters must be dealt with in some hidden way during caching which was causing the join operation to fail (i.e., the file object created from the original file and the one created from the cached file were not identical). So... after a few hours of banging my head against this, the problem was fixed by simply changing the type of meta.file defines in assets/schema_input.json:

            "datafile": {
                "format": "string",
                "pattern": "^\\S+$",
                "errorMessage": "Data file for reads cannot contain spaces and must have extension 'cram', 'bam', '.fq.gz' or '.fastq.gz'",
                "meta": ["datafile"]
            },

Can you give it a try and see if the latest commits fix the problem?

@reichan1998
Copy link
Contributor

Thank you for letting me know. It's great to hear that you have solved this problem @tkchafin ^^ I will try and get back to you later.

@reichan1998
Copy link
Contributor

Hi @tkchafin, I have checked and your solution can fix the -resume issue. Happy weekend!

@tkchafin
Copy link
Contributor Author

tkchafin commented Aug 9, 2024

Thanks Weiwen! Can you click the "approve" button in the review so I can merge?

@reichan1998
Copy link
Contributor

I have already approved. Thank you, Tyler!

@tkchafin
Copy link
Contributor Author

Thanks for your review!

@tkchafin tkchafin merged commit eb95288 into sanger-tol:dev Aug 12, 2024
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants