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

Work-around for lumpy segfaults from lumpyexpress #277

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Commits on Oct 18, 2018

  1. The call to lumpy from lumpyexpress segfaults when the -pe flags are …

    …omitted when the insert mean and stdev are not calculated in lumpyexpress. Furthermore, the lumpy galaxy wrapper fails with an error that mean & stdev are missing.
    
    I kept the warning, but allowed mean and stdev to be output in all cases to avoid the segfaults from lumpyexpress and avoid the error & no output from the lumpy galaxy wrapper.
    
    This is admittedly a work-around. Lumpy should probably be edited to be able to not segfault when the -pe discordant options are missing and should gracefully output no results if mean and stdev do not have valid values.
    
    Files checked in: scripts/lumpyexpress scripts/pairend_distro.py
    hepcat72 committed Oct 18, 2018
    Configuration menu
    Copy the full SHA
    4836748 View commit details
    Browse the repository at this point in the history

Commits on Mar 26, 2019

  1. Changes to get around errors in some cases

    My modified version of this script I use locally encountered a case where some type exceptions were occurring due to unexpected nan values.  These changes work around that issue and allow lumpy to run.
    hepcat72 authored Mar 26, 2019
    Configuration menu
    Copy the full SHA
    6df60a0 View commit details
    Browse the repository at this point in the history