Wrong center locations in BandPassActivityProfile
open | Created 2019-03-06 | Last updated 2019-03-06| Posted by pcostanza | See in Github


bogus bug learn GATK


Bug Report

Affected tool(s) or class(es)

org.broadinstitute.hellbender.utils.activityprofile.BandPassActivityProfile

Affected version(s)

Latest master branch as of March 6, 2019

Description

Line 147 should read:

final SimpleInterval loc = getLocForOffset(superState.getLoc(), i)

instead of

final SimpleInterval loc = getLocForOffset(justAddedState.getLoc(), i)

Likewise, line 154 should read:

states.add(superState)

instead of

states.add(justAddedState)

Otherwise, the location changes in org.broadinstitute.hellbender.utils.activityprofile.ActivityProfile, line 241 doesn't make a lot of sense, and neither does the extension of getMaxProbPropagationDistance() in org.broadinstitute.hellbender.utils.activityprofile.BandPassActivityProfile, lines 100-102.


Return to top