Separators in purity ploidy are inconsistent with the

[kevinmhadi]

Describe the bug
@xanthopoulakis - We need to correct the formula for the red dotted separators in the purity ploidy histogram (Segment Width Distribution). The meaning of the slope and intercept for these are slightly different.

I believe the issue is this expression:

    let separators = d3
      .range(0, maxSeparatorsCount + 1)
      .map((i) => slope * i + intercept);

Below is the pseudocode of the right parameters to use and the final expression:

a = 2 * (1-purity) / 2
b = 1 / beta
ppfit_intercept = a / b
ppfit_slope = beta
ppfit_slope * i + ppfit_intercept

[xanthopoulakis]

@kevinmhadi , I have updated the code as expected, the patch is now applied on the NYU server gOS instance

This is the code I am applying

[kevinmhadi]

Thanks for updating - I realized made an error with a.. apologies. It should actually be:

a = 2 * (1-purity) / purity
b = 1 / beta
ppfit_intercept = a / b
ppfit_slope = beta
ppfit_slope * i + ppfit_intercept

[kevinmhadi]

So because of the above change..
The below line should be changed to:

    let maxSeparatorsCount = Math.ceil((extent[1] - intercept) / slope);

    let maxSeparatorsCount = Math.ceil((extent[1] - ppfit_intercept) / ppfit_slope);

[xanthopoulakis]

@kevinmhadi , I have pushed my latest changes, let me know if this solves the issue