Should CollectHsMetrics also emit PCT@5x?

[yfarjoun]

It currently emits 1,2,10,20,30,40,50,100

there seems to be at least one user who's interested, #851. But I'm not sure if thats good enough evidence for changing the summary metric.

Alternatively, we could include in the histogram PCT@ANYx which would be less changes to the API.

opinions?

@ktibbett @jacarey @tfenne @nh13

[tfenne]

I agree with your thought here and elsewhere that it would be much nicer to emit the histogram. We could even emit a pair of histograms that are either counts (or proportions) of bases @ nX and >= nX to make it so that users don't have to do too much math.

[nh13]

I think making the metrics extensible (i.e. the user can specify the coverage thresholds) has sailed, so I think the histogram would be nicer, since then we don't need to change the summary metrics.

[jacarey]

Agreed.. from a production standpoint I'd prefer to leave the summary metrics alone.

[yfarjoun]

sounds good! I'll tag this as a easy task for someone to do when they want to learn coding for picard....