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)


Affected version(s)

Latest master branch as of March 6, 2019


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:


instead of


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