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.

Sign up for GitHub

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

Jump to bottom

Fix samtools piping #325

Merged
merged 3 commits into from
Sep 7, 2023
Merged

Fix samtools piping #325

merged 3 commits into from
Sep 7, 2023

Commits on Aug 11, 2023

  1. Remove samtools pipe stderr to stdout 

    Sometimes some of the libraries that samtools depends on don't have the correct
version information, and as such samtools will report this to stderr when run.
Because we pipe the output of samtools, we expect it to be valid SAM format, but
when these library version messages are reported, it breaks CRISPRessoWGS.
    @Colelyman
    Colelyman committed Aug 11, 2023
    Configuration menu
    Copy the full SHA
    93ac742 View commit details
    Browse the repository at this point in the history

  2. Remove extra spacing at end of lines and add missing comma in WGS

    @Colelyman
    Colelyman committed Aug 11, 2023
    Configuration menu
    Copy the full SHA
    c6f8277 View commit details
    Browse the repository at this point in the history

Commits on Sep 7, 2023

  1. Log stderr from samtools in CRISPRessoWGS

    @Colelyman
    Colelyman committed Sep 7, 2023
    Configuration menu
    Copy the full SHA
    caa8531 View commit details
    Browse the repository at this point in the history