Re: Plot failure: "Visible pixel grid has a scan line longer than previous scan lines"

[Dima K. <gn...@di...>]

Ethan A Merritt <me...@uw...> writes:

> In the end the fix was trivial, but I wouldn't call it "obvious".
>
> The clipping in proces_image() at step 5 correctly accounts for the pixel
> extent on x and y regardless of what flag is in pixel->type.  So nothing is
> gained by  setting it to OUTRANGE in step 2.  If instead we force all
> pixels to be flagged INRANGE at that stage, the incorrect zrange
> determination at step 3 doesn't happen and all pixels are kept.
> It wasn't obvious to me that marking all pixels INRANGE would preserve
> both autoscaling and clipping, but it does.
>
> fix commited to tip, queued for 5.4

I tested it out, and it appears to work. Thanks a lot! I would've had a
hard time not breaking stuff if I tried to fix this myself.