Re: [Plplot-devel] Subscribing to our git feed

2016-12-19 Thread Pedro Vicente
Hi Alan I subscribed to the feed, got the latest master, and all is working now on wxWidgetsdemo. on my linux 14.04 it tries to go to Plot() one time, then OnCreate is called, then goes to Plot() again 23:26:43: Debug: wxPLplotwindow::wxPLplotwindow 23:26:44: Debug: frame->Create 23:26:44:

[Plplot-devel] Subscribing to our git feed

2016-12-19 Thread Alan W. Irwin
Hi Pedro: Phil found his own way of solving the two big Linux wxwidgets issues this evening with his latest series of commits. But from your recent posts it appears you are not yet aware of those commits. To take care of that issue, I strongly recommend you subscribe to our git feed. You do

Re: [Plplot-devel] New way to generate wxwidgets debug output

2016-12-19 Thread Pedro Vicente
you can use the function "getrandom" as explained here http://stackoverflow.com/questions/2572366/how-to-use-dev-random-or-urandom-in-c http://man7.org/linux/man-pages/man2/getrandom.2.html -Pedro - Original Message - From: "Pedro Vicente" To: "Alan

Re: [Plplot-devel] New way to generate wxwidgets debug output

2016-12-19 Thread Pedro Vicente
you could use "open" to try to open the stream as described here http://www.cplusplus.com/reference/fstream/fstream/open/ I'll do a small test to see -Pedro - Original Message - From: "Pedro Vicente" To: "Alan W. Irwin" ;

Re: [Plplot-devel] New way to generate wxwidgets debug output

2016-12-19 Thread Pedro Vicente
Hi Alan ok, I see now. > That is check if > > std::fstream fin( "/dev/urandom", std::ios::in ); this is the constructor, there is no return value, which is one of the criticisms made to C++. In this case probably you need to do a small I/O operation after that call and check for the result.

Re: [Plplot-devel] New way to generate wxwidgets debug output

2016-12-19 Thread Alan W. Irwin
On 2016-12-19 16:33-0500 Pedro Vicente wrote: > Hi Alan > > >> The only trouble with the above fix is not every Unix platform has >> /dev/urandom (although from the above URL most do). >> >> So I would like to change the above fix to check for /dev/urandom >> and use it if it exists, but

Re: [Plplot-devel] New way to generate wxwidgets debug output

2016-12-19 Thread Alan W. Irwin
On 2016-12-19 12:25-0800 Alan W. Irwin wrote: > On 2016-12-19 12:33-0500 Pedro Vicente wrote: > >> Hi Alan >> >> I just did a git pull of the master branch with these changes and I get >> compiling errors >> if I don't add >> >> -DPLPLOT_WX_NANOSEC=ON > > To Pedro and Phil: > > @Pedro: > > Thanks

Re: [Plplot-devel] New way to generate wxwidgets debug output

2016-12-19 Thread Pedro Vicente
Hi Alan > The only trouble with the above fix is not every Unix platform has > /dev/urandom (although from the above URL most do). > > So I would like to change the above fix to check for /dev/urandom > and use it if it exists, but otherwise fall back to using > /dev/random. > > How do I do

Re: [Plplot-devel] New way to generate wxwidgets debug output

2016-12-19 Thread Alan W. Irwin
On 2016-12-19 15:52-0500 Pedro Vicente wrote: > Hi Alan > >> Was that as a result of running (exactly) > > no, my previous output was just > make VERBOSE=1 test_wxPLplotDemo > > > this is the second output of > > time examples/c/x01c -dev wxwidgets -np; echo "done x01c"; time > examples/c/x01c

Re: [Plplot-devel] New way to generate wxwidgets debug output

2016-12-19 Thread Alan W. Irwin
On 2016-12-19 19:47- p.d.rosenb...@gmail.com wrote: > Hi Alan > I am on my commute home right now. But if you want to test if the random > number generator is the cause then find the Rand constructor – Rand::Rand() > and comment out everything, replacing it with a single line that sets the

Re: [Plplot-devel] New way to generate wxwidgets debug output

2016-12-19 Thread Pedro Vicente
Hi Alan >Was that as a result of running (exactly) no, my previous output was just make VERBOSE=1 test_wxPLplotDemo this is the second output of time examples/c/x01c -dev wxwidgets -np; echo "done x01c"; time examples/c/x01c -dev wxwidgets -np;echo "done x01c" PLplot library version:

Re: [Plplot-devel] New way to generate wxwidgets debug output

2016-12-19 Thread p.d.rosenberg
Hi Alan I am on my commute home right now. But if you want to test if the random number generator is the cause then find the Rand constructor – Rand::Rand() and comment out everything, replacing it with a single line that sets the seed (probably m_seed or something) to a fixed value, like 0.

Re: [Plplot-devel] New way to generate wxwidgets debug output

2016-12-19 Thread Alan W. Irwin
To Phil and Pedro: Just woke up, skimmed through your e-mails, and I promise to look at the build issue that Pedro found starting just when I finish this e-mail. Further thoughts below on what Phil said. On 2016-12-19 17:42- Phil Rosenberg wrote: > Hi Alan > Could you just confirm to me

Re: [Plplot-devel] wxPLplotDemo.cpp -- linux build error - PATCH working

2016-12-19 Thread Pedro Vicente
Hi Phil It seems you got it working I applied the patch, to 2 previous master commits that compile. results are: make VERBOSE=1 test_wxPLplotDemo 14:26:35: Debug: wxPLplotwindow::wxPLplotwindow 14:26:35: Debug: frame->Create 14:26:35: Debug: Plot() Yielding 14:26:35: Debug: Plot() Yielding

Re: [Plplot-devel] wxPLplotDemo.cpp -- linux build error

2016-12-19 Thread Phil Rosenberg
On 19 December 2016 at 17:47, Pedro Vicente wrote: > Hi Phil > >> Pedro can you please check if this works on your systems? > > > ok, I'll try it when the patch is pushed to the master . > > Alan > > I assume you are going to push the patch? > I get some

Re: [Plplot-devel] wxPLplotDemo.cpp -- linux build error

2016-12-19 Thread Pedro Vicente
Hi Phil > Pedro can you please check if this works on your systems? ok, I'll try it when the patch is pushed to the master . Alan I assume you are going to push the patch? I get some compiling errors on the current master, please see my last post a new idea: what about if we just override

Re: [Plplot-devel] New way to generate wxwidgets debug output

2016-12-19 Thread Phil Rosenberg
Hi Alan Could you just confirm to me what command you are using to test the timings? Phil On 19 December 2016 at 17:33, Pedro Vicente wrote: > Hi Alan > > I just did a git pull of the master branch with these changes and I get > compiling errors > if I don't

Re: [Plplot-devel] New way to generate wxwidgets debug output

2016-12-19 Thread Pedro Vicente
Hi Alan I just did a git pull of the master branch with these changes and I get compiling errors if I don't add -DPLPLOT_WX_NANOSEC=ON I did cmake .. -G "Unix Makefiles" -DBUILD_SHARED_LIBS:BOOL=OFF -DENABLE_f95:BOOL=OFF -DENABLE_tcl:BOOL=OFF -DENABLE_tk:BOOL=OFF -DBUILD_TEST:BOOL=ON make

Re: [Plplot-devel] wxPLplotDemo.cpp -- linux build error

2016-12-19 Thread Phil Rosenberg
No worries :-) Attached is a patch which I hope should fix things. It is the simplest possible fix as far as I can tell. It does have a while loop with no counter, but unless there is some rather fatal bug in wxWidgets it should never turn into an infinite loop. Alan if you prefer I will add a

Re: [Plplot-devel] wxPLplotDemo.cpp -- linux build error

2016-12-19 Thread Alan W. Irwin
On 2016-12-19 11:31- Phil Rosenberg wrote: > I'm just generating a test fix. I will send a patch round once it's > done. If it works then I'll tidy it up. However you appear to have > bumped up the minimum CMake version so I need to reinstall CMake - > turns out the latest Windows version

[Plplot-devel] I have now pinned down the long pause to a few lines of completely innocuous code.

2016-12-19 Thread Alan W. Irwin
Hi Phil and Pedro: Please look at the commit message for af41765, and the corresponding part of the source code (lines 1537-1548 of drivers/wxwidgets_dev.cpp). I cannot spot anything between those two uses of the PLPLOT_wxLogDebug macro that would cause the 13-second pause that shows up on the

Re: [Plplot-devel] wxPLplotDemo.cpp -- linux build error

2016-12-19 Thread Phil Rosenberg
I'm just generating a test fix. I will send a patch round once it's done. If it works then I'll tidy it up. However you appear to have bumped up the minimum CMake version so I need to reinstall CMake - turns out the latest Windows version requires a manual uninstall first. I'm sure you're not

Re: [Plplot-devel] Comprehensive testing

2016-12-19 Thread Alan W. Irwin
On 2016-12-19 09:39- Arjen Markus wrote: > Hi Alan, > > > >> -Original Message- >> From: Alan W. Irwin [mailto:ir...@beluga.phys.uvic.ca] >> The 5 new reports there are based on the Fortran subset of 4 recent >> comprehensive test reports (Cygwin and MinGW-w64/MSYS2 from you, and >>

Re: [Plplot-devel] Comprehensive testing

2016-12-19 Thread Arjen Markus
Hi Alan, > -Original Message- > From: Alan W. Irwin [mailto:ir...@beluga.phys.uvic.ca] > The 5 new reports there are based on the Fortran subset of 4 recent > comprehensive test reports (Cygwin and MinGW-w64/MSYS2 from you, and > Debian Jessie with CMake-3.0.2 and CMake-3.7.0 from me)

Re: [Plplot-devel] wxPLplotDemo.cpp -- linux build error

2016-12-19 Thread Alan W. Irwin
On 2016-12-19 00:30-0800 Alan W. Irwin wrote: > Anyhow, good luck with implementing this readiness check anyway you > like, and I look forward to your commit to that effect which can go > right on top of master tip (which should simplify your life). Then we can ask > Pedro to drop his (now

Re: [Plplot-devel] wxPLplotDemo.cpp -- linux build error

2016-12-19 Thread Alan W. Irwin
On 2016-12-19 07:56- p.d.rosenb...@gmail.com wrote: > Hi Alan, Pedro > Yes Alan, that is what I intended. Sorry, that doesn't compile. I was on my > way to bed when I sent the email and was doing it from memory. > > I will try to find a moment today to fix this. > > I'm not sure what your