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

[feat] Add an option to sort by sequence role in CollectAlternateContigNames #868

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

nh13
Copy link
Member

@nh13 nh13 commented Jul 31, 2022

This is so that I can have assembled molecules first, whereas in some assembly reports chrM (an assembled molecule) is listed at the end of the report.

@nh13 nh13 requested a review from tfenne July 31, 2022 18:48
@codecov-commenter
Copy link

codecov-commenter commented Jul 31, 2022

Codecov Report

Merging #868 (bbacea4) into main (da9ecbc) will decrease coverage by 0.09%.
The diff coverage is 22.22%.

@@            Coverage Diff             @@
##             main     #868      +/-   ##
==========================================
- Coverage   95.66%   95.56%   -0.10%     
==========================================
  Files         125      125              
  Lines        7239     7246       +7     
  Branches      507      514       +7     
==========================================
  Hits         6925     6925              
- Misses        314      321       +7     
Flag Coverage Δ
unittests 95.56% <22.22%> (-0.10%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
...umgenomics/fasta/CollectAlternateContigNames.scala 85.54% <22.22%> (-7.88%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update da9ecbc...bbacea4. Read the comment docs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants