From: Pedro V. <ped...@sp...> - 2017-01-05 01:08:27
|
>> Yes that is the expected output, although the window titles should be different. But actually it may be nicer to at least use 2 different curves. An easy change. What do you think? I don't have an opinion on the plot itself, but regarding the test_c_wxwidgets test, it would be a good idea to put a simple sleep(1) betwwen each test so we can see that it's doing what it should do I tried on my linux 16.04, same results as CentOS (I don'have the 14.04 and debian anymore) -Pedro ----- Original Message ----- From: p.d...@gm... To: Alan W. Irwin ; Pedro Vicente ; PLplot development list Sent: Wednesday, January 04, 2017 7:54 PM Subject: RE: Linux OnCreate delay bug -- solution That looks promising 😊 To answer your questions Alan (sorry for top posting, but I'm on my phone) 1.. Yes that is the expected output, although the window titles should be different. But actually it may be nicer to at least use 2 different curves. An easy change. What do you think? 2.. Will do, plus this all really needs properly adding to the docs. I will try to do that asap. Sent from my Windows 10 phone From: Alan W. Irwin Sent: 04 January 2017 19:17 To: Phil Rosenberg; Pedro Vicente; PLplot development list Subject: Re: Linux OnCreate delay bug -- solution On 2017-01-04 13:04-0500 Pedro Vicente wrote: > Hi Phil > > I got a good plot on CentOS, below output. > I'll test other Linux later > >> So what I >> would propose is that for this release we attempt to not add or remove >> anything from the API and just stick with an example that works with >> what we've got. Then after this release we can add non-template >> classes for wxFrame, wxDialog and wxPanel which can be used really >> easily by our users and we have time to test them and make sure the >> API for their use is stable before the next release. >> >> Does that seem sensible? > To Phil and Pedro: @Phil: The test_c_wxwidgets and test_wxPLplotDemo targets worked without issues here. So we now have three good tests on Linux and one good test on Windows. So this indeed seems promising as a solution to this release critical bug, and I thank you for this timely release-critical bug fixing. @Both: To finish off this topic for the release I need some more from both of you. @Phil: 1. It sounded from your description that you were changing the example to try two different methods of generating those plots so it makes sense that test_wxPLplotDemo now produces two identical plots for me, but please confirm that the two plots I observed are the intended effect. 2. Please commit a change to README.release describing what you have done to deal with this bug equivalent to what Pedro wrote up for his solution. @Pedro: I think your current CentOS test is equivalent to just building the test_wxPLplotDemo target? Please extend that test to building the test_c_wxwidgets target as well. You were already planning to test Phil's solution on all the Linux platforms accessible to you, but please do the full test (build both the test_wxPLplotDemo and test_c_wxwidgets targets) for all Linux platforms. And similarly for your Windows platforms. There, the convenient test_* targets won't work so you will have to do the equivalent by hand (run wxPLplotDemo and run several C examples with -dev wxwidgets). Finally, please also test your own software projects that link with the plplotwxwidgets library on all platforms where you expect your own software projects to currently work. (I assume that is just Windows, but please test on at least one Linux platform as well if you expect your software to work on both Windows and Linux.) Every tested platform that you add gives us just that much further confidence that Phil's solution is robust for this release, and I thank you in advance for all this essential testing work for our release. Alan __________________________ Alan W. Irwin Astronomical research affiliation with Department of Physics and Astronomy, University of Victoria (astrowww.phys.uvic.ca). Programming affiliations with the FreeEOS equation-of-state implementation for stellar interiors (freeeos.sf.net); the Time Ephemerides project (timeephem.sf.net); PLplot scientific plotting software package (plplot.sf.net); the libLASi project (unifont.org/lasi); the Loads of Linux Links project (loll.sf.net); and the Linux Brochure Project (lbproject.sf.net). __________________________ Linux-powered Science __________________________ |