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

Changed the intermediate files written during SplitBamByCellBarcodes from BAM to SAM #79

Merged
merged 2 commits into from
Jul 28, 2020

Conversation

kishorikonwar
Copy link
Contributor

Purpose

To speed-up the SplitBamByCellBarcode step of the Optimus pipeline

Changes

Changed the format of the intermediate barcode-wise files from BAM to SAM

@codecov-commenter
Copy link

Codecov Report

Merging #79 into master will not change coverage.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master      #79   +/-   ##
=======================================
  Coverage   88.85%   88.85%           
=======================================
  Files          28       28           
  Lines        3006     3006           
=======================================
  Hits         2671     2671           
  Misses        335      335           
Impacted Files Coverage Δ
src/sctools/bam.py 93.11% <100.00%> (ø)

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 0a8c4df...200a032. Read the comment docs.

@kishorikonwar kishorikonwar merged commit f82f789 into master Jul 28, 2020
@khajoue2 khajoue2 deleted the kmk-split-bam-by-barcode branch March 9, 2022 19:44
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

4 participants