Jeff King <peff@xxxxxxxx> writes: > Hmm. Looking at scale_linear, the formula is: > > return ((it - 1) * (width - 1) + max_change - 1) / (max_change - 1); > > I don't see how that can be accurate, since the magnitude of the "-1" > tweak will vary based on the value of "it". This code is due to > 3ed74e6, but I don't quite follow the logic in the commit message. Doesn't it need +1 at the end, I wonder? We want to map: - zero to zero - any number to at least 1 so scaling a non-zero "it" so that maximum maps to (width-1) and then adding 1 would be the right way for the latter case. Of course, an easy way out without worrying about the correct math is to scale the total and the smaller one and then declare that the scaled larger one is the difference between the two. That way, both of these two files have 109 in total so the length of the entire graph would be the same ;-). -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html