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

Fix some bugs and add upset plots #117

Merged
merged 12 commits into from
May 10, 2024
Merged

Fix some bugs and add upset plots #117

merged 12 commits into from
May 10, 2024

Conversation

nictru
Copy link
Contributor

@nictru nictru commented May 2, 2024

Hey, this PR addresses 4 things:

  1. Exclude overlong circRNAs from quantification as psirc-quant throws an error otherwise
  2. Use only the circRNAs which pass the detection majority vote in quantification
  3. Add upset plot functionality for comparing detection tools and add to multiqc
  4. Add TPM to the RDS output of the quantification subworkflow

Copy link

github-actions bot commented May 2, 2024

nf-core lint overall result: Passed ✅ ⚠️

Posted for pipeline commit 86af7b2

+| ✅ 260 tests passed       |+
!| ❗   1 tests had warnings |!

❗ Test warnings:

  • readme - README contains the placeholder zenodo.XXXXXXX. This should be replaced with the zenodo doi (after the first release).

✅ Tests passed:

Run details

  • nf-core/tools version 2.14.1
  • Run at 2024-05-10 16:47:02

@nictru nictru force-pushed the exclude-overlong-transcripts branch from b92d6c3 to bf06573 Compare May 10, 2024 10:29
Copy link

@erinyoung erinyoung left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have no changes to recommend.

@nictru nictru merged commit 5ddc061 into dev May 10, 2024
4 checks passed
@nictru nictru deleted the exclude-overlong-transcripts branch May 10, 2024 17:24
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.

None yet

2 participants