Re: replotting in-line data and autoscaling

[Ethan A M. <sf...@us...>]

On Thursday, April 26, 2018 11:50:25 PM PDT Dima Kogan wrote:
> sfeam <sf...@us...> writes:
> 
> > Heh.  And indeed I found a stupid error that failed to check for
> > logscaling at one point during the "refresh" command that is substituted
> > for "replot" for inline data.  I make no guarantee that it fixes
> > everything but it does work on your test script.
> >
> > The larger routine that this error is embedded in all looks suspect
> > to me, but changing the whole thing is more than I'm going to tackle
> > right now.
> >
> > Can you test the stupid-error fix using git tip for the development
> > version? I'll run some additional tests here also. If nothing bad
> > turns up I'll make a decision about including it in 5.2.3 also.
> 
> Thanks for doing this so quickly. I just tested it, and it indeed makes
> the test script work. Your commit to fix that problem only touches the x
> axis. I changed the test script to "set logscale y" instead of x to see
> if that's also broken, and indeed it is. Can you do whatever you did
> with the x axis to the y axis?

The stupid error I found was in code that only dealt with the x axis
of the current plot (so either x or x2).

> I can imagine that x2 and y2 axes are affected and maybe 3d axes. And
> maybe other stuff. Handling x,y,x2,y2 should hit most of the use cases,
> probably.

A higher-level fix is required so that all axes are correctly re-evaluated
on refresh.  The difficulty comes from trying to handle both pure
"replot/refresh" requests and zoom operations.  

	Ethan