Hi Jim (back on list) Is this fix committed to the online repo or just in your local repo?
Phil On 8 March 2015 at 03:47, Jim Dishaw <j...@dishaw.org> wrote: > I fixed the bug and it was a lot simpler than I thought. I was concerned > that the xwin driver was changing the colormap, which would cause the pen > color to change. Fortunately the xwin driver allocates pen colors > independently of the plplot colormaps. > > I fixed the state saving that plbuf_bop does so that rdbuf_bop restores the > correct colors. > > On Mar 5, 2015, at 3:58 AM, Phil Rosenberg <p.d.rosenb...@gmail.com> wrote: > > Hi Jim. > It is perhaps both more and less straightforward than you consider in your > email. Plplot honours changes in colour maps midway through a plot. So for > example I could plot one dataset, change the colour map and then plot a > different dataset. This would be not only possible, but very likely for > colour map 1. > > Currently in this situation a replot would result in both datasets using the > second colourmap. > > The fix, however, is not so complex. Either pl_bop, must call plcol0, > plwidth, set the colour maps etc, using the current values which will result > in them ending up in the buffer, or a bop event in the buffer must store all > these values, then on replot it must reinstate them and call the appropriate > functions so the state gets passed to the driver. > > The only alternative is to reset all state parameters to their defaults on > bop. This would break most of our examples and most user code I imagine (it > would certainly break all mine). > > So in summary, this bug has even more complex and likely repercussions than > you mention in your message, but the fix is pretty straightforward. > > Phil > ________________________________ > From: Jim Dishaw > Sent: 05/03/2015 04:20 > To: Phil Rosenberg > Subject: Re: [Plplot-devel] buffer state parameters > > I think I tracked down the bug and it might be tricky. I noticed that when > I put a pladv(0) before changing the colormap (my first guess at the > problem) the plot turned all yellow (the last color used) on a resize. So I > think there are two issues here. > > If it is just the wrong pen color is selected, then restoring the correct > state is a simple fix. If it is a colormap changing issue, then the bug is > more difficult to fix. > > The EOP does not get generated until the next map call, thus all the > colormap setup happens on the end of page #4. When the plot buffer is > executed on a resize, the colormap is changed at the end and that causes the > problem. > > The only two solutions I see are: > > 1) Require a pladv() prior to a colormap change that is for a new page. > 2) Have plbuf ignore colormap changes that occur at the end of the page when > replaying the plot buffer and then executing them at the next BOP. > > I will work on this tomorrow, but I wanted to give you an update. > > On Mar 4, 2015, at 4:03 PM, Phil Rosenberg <p.d.rosenb...@gmail.com> wrote: > >> Okay >> I will leave that with you then >> >> Thanks >> Phil >> >> On 4 March 2015 at 15:07, Jim Dishaw <j...@dishaw.org> wrote: >>> >>>> On Mar 4, 2015, at 9:13 AM, Phil Rosenberg <p.d.rosenb...@gmail.com> >>>> wrote: >>>> >>>> Hi Jim >>>> There is still a bug in the buffer with respect to the correct state >>>> parameters. I think we discussed this in the past regarding plplot >>>> initialisation, but this particular bug occurs at page change. >>>> >>>> Basically at the beginning of a render, plplot inherits it's state >>>> from the previous render. Mostly when we simply move from one page to >>>> the next this behaviour gives expected results. However if we rerender >>>> a page twice, then the second render inherits the state from the first >>>> render which is wrong. >>>> >>>> A good example is x19.4. At the end of this page the colour table is >>>> changed (including the background colour) in preparation for page 5. >>>> If you resize page 4 under the X driver you will see that the >>>> background and lines change colour because this new colour table is >>>> being incorrectly used. >>>> >>>> I have seen a similar thing happen with the line width and presumably >>>> other state variables behave similarly too. >>>> >>>> Basically I wanted to check if this is something that is on your >>>> radar? I am happy to fix the problem, but I don't want to create a >>>> clash with your current work. >>>> >>> >>> I will fix and patch. I will also make a test case. >>> >>>> Phil >>>> >>>> >>>> ------------------------------------------------------------------------------ >>>> Dive into the World of Parallel Programming The Go Parallel Website, >>>> sponsored >>>> by Intel and developed in partnership with Slashdot Media, is your hub >>>> for all >>>> things parallel software development, from weekly thought leadership >>>> blogs to >>>> news, videos, case studies, tutorials and more. Take a look and join the >>>> conversation now. http://goparallel.sourceforge.net/ >>>> _______________________________________________ >>>> Plplot-devel mailing list >>>> Plplot-devel@lists.sourceforge.net >>>> https://lists.sourceforge.net/lists/listinfo/plplot-devel > > ------------------------------------------------------------------------------ Dive into the World of Parallel Programming The Go Parallel Website, sponsored by Intel and developed in partnership with Slashdot Media, is your hub for all things parallel software development, from weekly thought leadership blogs to news, videos, case studies, tutorials and more. Take a look and join the conversation now. http://goparallel.sourceforge.net/ _______________________________________________ Plplot-devel mailing list Plplot-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/plplot-devel
