|
From: Dima K. <gn...@di...> - 2018-11-08 03:29:16
|
sfeam <me...@uw...> writes: > Ah, I understand now. Yes there is a sanity check on the aspect ratio > that limits it to a ratio of 1:100. That code dates waayyy back and I > don't know what logic went into deciding this was the limit. You could > try changing it to, say, 1:1000. Hi Ethan. I just went to look at this, and it looks like you applied a patch already. Thanks! I don't quite understand why this limit is there at all. As long as the aspect ratio is > 0 and < infinity, I think we should accept it. I hit this through normal usage, and I can imagine hitting even the more extreme limit at some point later. My use case is this: - I'm plotting an image with annotations plotted on top (points, lines, etc). - The initial plot indeed has a crazy aspect ratio, and isn't very useful. But I can clearly see the annotations, and I can then interactively zoom into the area with the annotations, producing a useful plot. The square aspect ratio is preserved by the zoom operation Unless something predictable will break, maybe the limit should go away, or become much larger? |