zooming of inline data

[Petr Mikulik]

I thought that Ethan has already committed his patch
    [ 1723715 ] Refresh plot or zoom without re-reading data
and that it was released in 4.2.1, but it is not the case.
What was the reason of not putting it into cvs (see 2007-07-01 04:36)?


There is also this patch
    [ 1745865 ] store raw data to refresh plot without rereading files
which is somehow similar in results (but caches all datafiles).


So, how to continue? Zooming of inline data is really important (and not
only for Octave users).

---
PM

-------------------------------------------------------------------------
This SF.net email is sponsored by: Splunk Inc.
Still grepping through log files to find problems?  Stop.
Now Search log events and configuration files using AJAX and a browser.
Download your FREE copy of Splunk now >>  http://get.splunk.com/
_______________________________________________
gnuplot-beta mailing list
gnuplot-beta@...
https://lists.sourceforge.net/lists/listinfo/gnuplot-beta

[Ethan Merritt]

On Thursday 30 August 2007 14:23, Petr Mikulik wrote:

> I thought that Ethan has already committed his patch
>     [ 1723715 ] Refresh plot or zoom without re-reading data
> and that it was released in 4.2.1, but it is not the case.
> What was the reason of not putting it into cvs (see 2007-07-01 04:36)?
>
>
> There is also this patch
>     [ 1745865 ] store raw data to refresh plot without rereading files
> which is somehow similar in results (but caches all datafiles).
>
>
> So, how to continue? Zooming of inline data is really important (and not
> only for Octave users).

Good question.  I don't know the correct answer.

My patch 1723715 is imperfect, as Daniel Sebald pointed out.
One imperfection is that is does not allow toggling log scale axes.
One positive feature is that it requires no additional internal
storage or data structures.

Daniel's alternative patch  174865 creates a new set of internal storage
buffers and saves all the input data for later re-use.  I do not
like the overhead, but I concede the point that it provides
identical parsing and plotting behaviour for fresh or stored data.
I have not had time to look at the code quality, nor have I tested
it on real data.

We also discussed the option of re-writing the log scale code so that
the original data is stored rather than rescaled data.
That is, the log scaling would be applied at the time of plotting
rather than at the time the data is read in. This would remove the major
limitation of patch 1723715, but at the cost of rewriting all the
code that references log scaling.  I.e., high risk of breakage.
I think this is an attractive long-term alternative, but it doesn't
address the immediate needs of Octave users.


--
Ethan A Merritt

-------------------------------------------------------------------------
This SF.net email is sponsored by: Splunk Inc.
Still grepping through log files to find problems?  Stop.
Now Search log events and configuration files using AJAX and a browser.
Download your FREE copy of Splunk now >>  http://get.splunk.com/
_______________________________________________
gnuplot-beta mailing list
gnuplot-beta@...
https://lists.sourceforge.net/lists/listinfo/gnuplot-beta

[Petr Mikulik]

> > I thought that Ethan has already committed his patch
> >     [ 1723715 ] Refresh plot or zoom without re-reading data
> > and that it was released in 4.2.1, but it is not the case.
> > What was the reason of not putting it into cvs (see 2007-07-01 04:36)?
> >
> >
> > There is also this patch
> >     [ 1745865 ] store raw data to refresh plot without rereading files
> > which is somehow similar in results (but caches all datafiles).
> >
> >
> > So, how to continue? Zooming of inline data is really important (and not
> > only for Octave users).
>
> Good question.  I don't know the correct answer.
>
> My patch 1723715 is imperfect, as Daniel Sebald pointed out.
> One imperfection is that is does not allow toggling log scale axes.
> One positive feature is that it requires no additional internal
> storage or data structures.

I propose to commit your patch into cvs for both 4.2 and 4.3. This will
allow us to use the mousing functionality immediately. (I thought his has
already happen.) It would be a good reason for releasing gnuplot 4.2.2 (also
with a bugfix to xylabels as reported today).

> Daniel's alternative patch  174865 creates a new set of internal storage
> buffers and saves all the input data for later re-use.
>
> We also discussed the option of re-writing the log scale code so that
> the original data is stored rather than rescaled data.

Both are more serious changes, so they can be developed for a later
inclusion into 4.3.

---
PM

-------------------------------------------------------------------------
This SF.net email is sponsored by: Splunk Inc.
Still grepping through log files to find problems?  Stop.
Now Search log events and configuration files using AJAX and a browser.
Download your FREE copy of Splunk now >>  http://get.splunk.com/
_______________________________________________
gnuplot-beta mailing list
gnuplot-beta@...
https://lists.sourceforge.net/lists/listinfo/gnuplot-beta

[Ethan Merritt]

On Friday 31 August 2007 01:44, Petr Mikulik wrote:
> I propose to commit your patch into cvs for both 4.2 and 4.3. This will
> allow us to use the mousing functionality immediately. (I thought his has
> already happen.)

I can apply the patch to CVS over the weekend. But I will be travelling
for the next 3 weeks, and will not be able to deal with any fallout or breakage.

Unfortunately, neither my patch nor Daniel's will apply directly against 4.2.
Both are large enough changes that I would worry about seriously breaking
something in 4.2 if there is a rush to back-port the new code.

> It would be a good reason for releasing gnuplot 4.2.2 (also
> with a bugfix to xylabels as reported today).

I think I will have to release a quick 4.2.2 anyhow, because the lack of
axis labels is indeed serious.  But that is a two-line fix.  To replace
the old zooming code and then back-port the "refresh" implementation
is way more than I could do quickly.  Let's get it into CVS so that it
sees some testing over the next few months.  At that point we can decide
whether it's solid enough and worth another 4.2.x release.
At least a few of the Octave users should be willing to build gnuplot
from cvs, and that will hopefully provide more immediate testing than
new features often get.

Or if you take the lead in backporting to 4.2 and thorough testing,
maybe it could be done more quickly?

        Ethan

> > > I thought that Ethan has already committed his patch
> > >     [ 1723715 ] Refresh plot or zoom without re-reading data
> > > and that it was released in 4.2.1, but it is not the case.
> > > What was the reason of not putting it into cvs (see 2007-07-01 04:36)?
> > >
> > >
> > > There is also this patch
> > >     [ 1745865 ] store raw data to refresh plot without rereading files
> > > which is somehow similar in results (but caches all datafiles).
> > >
> > >
> > > So, how to continue? Zooming of inline data is really important (and not
> > > only for Octave users).
> >
> > Good question.  I don't know the correct answer.
> >
> > My patch 1723715 is imperfect, as Daniel Sebald pointed out.
> > One imperfection is that is does not allow toggling log scale axes.
> > One positive feature is that it requires no additional internal
> > storage or data structures.
>
> I propose to commit your patch into cvs for both 4.2 and 4.3. This will
> allow us to use the mousing functionality immediately. (I thought his has
> already happen.) It would be a good reason for releasing gnuplot 4.2.2 (also
> with a bugfix to xylabels as reported today).
>
> > Daniel's alternative patch  174865 creates a new set of internal storage
> > buffers and saves all the input data for later re-use.
> >
> > We also discussed the option of re-writing the log scale code so that
> > the original data is stored rather than rescaled data.
>
> Both are more serious changes, so they can be developed for a later
> inclusion into 4.3.
>
> ---
> PM
>

--
Ethan A Merritt            Courier Deliveries: 1959 NE Pacific
Dept of Biochemistry
Health Sciences Building
University of Washington - Seattle WA 98195-7742

-------------------------------------------------------------------------
This SF.net email is sponsored by: Splunk Inc.
Still grepping through log files to find problems?  Stop.
Now Search log events and configuration files using AJAX and a browser.
Download your FREE copy of Splunk now >>  http://get.splunk.com/
_______________________________________________
gnuplot-beta mailing list
gnuplot-beta@...
https://lists.sourceforge.net/lists/listinfo/gnuplot-beta

[Petr Mikulik]

> > I propose to commit your patch into cvs for both 4.2 and 4.3. This will
> > allow us to use the mousing functionality immediately. (I thought his has
> > already happen.)
>
> I can apply the patch to CVS over the weekend. But I will be travelling
> for the next 3 weeks, and will not be able to deal with any fallout or breakage.

I see you have applied it already. I confirm it work with Octave 2.9.x.

> Unfortunately, neither my patch nor Daniel's will apply directly against 4.2.
> Both are large enough changes that I would worry about seriously breaking
> something in 4.2 if there is a rush to back-port the new code.
>
> To replace the old zooming code and then back-port the "refresh"
> implementation is way more than I could do quickly.  Let's get it into CVS
> so that it sees some testing over the next few months.  At that point we
> can decide whether it's solid enough and worth another 4.2.x release. At
> least a few of the Octave users should be willing to build gnuplot from
> cvs, and that will hopefully provide more immediate testing than new
> features often get.
>
> Or if you take the lead in backporting to 4.2 and thorough testing,
> maybe it could be done more quickly?

Sorry, I cannot do anything till the end of September.

So let us propose 4.3 (cvs) for Octave users. Then, you would either make
the patch ready for 4.2.n, or it would force an early release of 4.4.

---
PM

-------------------------------------------------------------------------
This SF.net email is sponsored by: Splunk Inc.
Still grepping through log files to find problems?  Stop.
Now Search log events and configuration files using AJAX and a browser.
Download your FREE copy of Splunk now >>  http://get.splunk.com/
_______________________________________________
gnuplot-beta mailing list
gnuplot-beta@...
https://lists.sourceforge.net/lists/listinfo/gnuplot-beta