Hi Alan, Phil

> Also, it was
> really good to hear that your tests show we finally have the
> definitive fix for this Linux issue

You're welcome, glad I could help.

I was going to send an email questioning the reasoning of both 
solutions assuming both were
working (Phil's  and mine), but I just tried on my CentOS with the 
latest git code, and what happens
is that the execution is always on

12:50:24: Debug: Plot() Yielding

About the solutions

1) Current solution


Right now, this has to be done in the client code

//We must wait for the Create event to be processed and the
        //wxPLplotstream to be prepared
        while (!frame->IsReady())
        {
                PLPLOT_wxLogDebug("Plot() Yielding");
                wxGetApp().Yield();
        }

So, the user has to do some extra step in order to wait for wxWidgets 
to
process some event.
Ideally he should not have to do this but rather all should be done 
inside wxWidgets.
This is error prone, because the usual way to use wxWidgets is

MyFrame *frame = new Frame()
frame->Show()


here we have

MyFrame *frame = new Frame()
frame->Create()
While ()
  {
   wait()
  }
frame->Show()
Plot(


so a user might not even be aware of this wait extra step.

that is assuming it's working, but on my CentOS it gets stuck on the 
Yield()


2) Solution of creating the fame in Show()

Client code is

MyFrame *frame = new Frame()
frame->Create()
frame->Show()
Plot()


there is no wait loop; there is also no need to create the stream in 
OnCreate()

-Pedro





On 2016-12-20 04:01, Alan W. Irwin wrote:
> On 2016-12-19 23:32-0500 Pedro Vicente wrote:
>
>> Hi Alan
>>
>> I subscribed to the feed, got the latest master, and all is working 
>> now on wxWidgetsdemo.
>
> Hi Pedro:
>
> That is good that you are subscribed to the git feed.  Also, it was
> really good to hear that your tests show we finally have the
> definitive fix for this Linux issue thanks to your original report of
> the issue, your implementation of an initial solution, your 
> subsequent
> test efforts on various Linux platforms, and your discussions of the
> issue with Phil.  And my thanks to Phil for ultimately finding the
> definitive fix.  That was good collaborative development work on an
> open source project you can both be proud of.
>
> 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
> __________________________

-- 
Pedro Vicente
pedro.vice...@space-research.org
http://www.space-research.org/

------------------------------------------------------------------------------
Developer Access Program for Intel Xeon Phi Processors
Access to Intel Xeon Phi processor-based developer platforms.
With one year of Intel Parallel Studio XE.
Training and support from Colfax.
Order your platform today.http://sdm.link/intel
_______________________________________________
Plplot-devel mailing list
Plplot-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/plplot-devel

Reply via email to