Re: [Plplot-devel] [PATCH] Fix plfimage function

2010-03-05 Thread Alan W. Irwin
On 2010-03-04 16:25-0800 David MacMahon wrote: > I get the feeling that I shouldn't have any numbers to the right of the > colons. It also seems like this build some of the tests, so I think I should > run it several times, discard the times from the first run and average the > rest. P.S. to

Re: [Plplot-devel] [PATCH] Fix plfimage function

2010-03-05 Thread David MacMahon
On Mar 4, 2010, at 21:46 , David MacMahon wrote: > I'm looking into some possible conflicts between system tcl/tk and > macports tcl/tk... Sure enough. I was inadvertently using a mix of system (Mac OS X 10.4) tcl (8.4) and MacPorts tcl (8.5). I've remedied the situation by using only the

Re: [Plplot-devel] [PATCH] Fix plfimage function

2010-03-05 Thread David MacMahon
On Mar 5, 2010, at 8:03 , Alan W. Irwin wrote: > Thanks for that suggestion for avoiding strcat whose effect depends on > octave version. (Ugh!) To be fair, the 3.0.1 behavior was arguably a bug since it was not Matlab-compatible behavior and I think Matlab compatibility is one of Octave's g

Re: [Plplot-devel] [PATCH] Fix plfimage function

2010-03-05 Thread Alan W. Irwin
On 2010-03-04 20:43-0800 David MacMahon wrote: > I think we can change [in octave example 29]... > > title = strcat("@frPLplot Example 29 - TAI-UTC ", title_suffix); > > ...to... > > title = ["@frPLplot Example 29 - TAI-UTC ", title_suffix]; > > ...if it works in 3.0.1 (it does in 3.2.3). Thanks

Re: [Plplot-devel] [PATCH] Fix plfimage function

2010-03-04 Thread David MacMahon
On Mar 4, 2010, at 17:30 , Alan W. Irwin wrote: > On 2010-03-04 16:25-0800 David MacMahon wrote: > >> octave >> Missing examples: 19 >> Differing postscript output : 28 29 >> Missing stdout : >> Differing stdout: 14 >> tcl >> Missing examples: >

Re: [Plplot-devel] [PATCH] Fix plfimage function

2010-03-04 Thread David MacMahon
Hi, Alan, On Mar 4, 2010, at 17:30 , Alan W. Irwin wrote: > On my own Linux system I confirm the above except I get good > results for > octave example 29. Note, I have octave-3.0.1 installed I think this difference is due to the use of Octave's strcat function in examples/octave/x29c.m. I

Re: [Plplot-devel] [PATCH] Fix plfimage function

2010-03-04 Thread Alan W. Irwin
On 2010-03-04 16:25-0800 David MacMahon wrote: > Hi, Alan, > > On Mar 3, 2010, at 12:05 , Alan W. Irwin wrote: > >> time make -j4 -k test_noninteractive >& make_testnoninteractive.out > > I'm trying to run this test now, but I'm not sure what to expect nor how to > interpret the results I get. I

Re: [Plplot-devel] [PATCH] Fix plfimage function

2010-03-04 Thread David MacMahon
Hi, Alan, On Mar 3, 2010, at 12:05 , Alan W. Irwin wrote: > time make -j4 -k test_noninteractive >& make_testnoninteractive.out I'm trying to run this test now, but I'm not sure what to expect nor how to interpret the results I get. It ends with an error message for me using either svn/trun

Re: [Plplot-devel] [PATCH] Fix plfimage function

2010-03-03 Thread David MacMahon
Hi, Alan, On Mar 3, 2010, at 18:35 , Alan W. Irwin wrote: > On 2010-03-03 13:13-0800 David MacMahon wrote: > >> [...] Tonight I will rolls all of these patches into one patch >> that can be applied to svn/trunk and send it out. > > Thanks. > >> >>> Would you be also willing to do the above test

Re: [Plplot-devel] [PATCH] Fix plfimage function

2010-03-03 Thread Alan W. Irwin
On 2010-03-03 13:13-0800 David MacMahon wrote: > Hi, Alan, > > On Mar 3, 2010, at 12:05 , Alan W. Irwin wrote: > >> For source trees with and without the patch >> >> time make -j4 -k test_noninteractive >& make_testnoninteractive.out >> time make -j4 -k test_interactive >& make_testinteractive.ou

Re: [Plplot-devel] [PATCH] Fix plfimage function

2010-03-03 Thread Hazen Babcock
Alan W. Irwin wrote: > On 2010-03-01 12:20-0800 David MacMahon wrote: > >> This fixes a parameter ordering problem in plfimage's call to the minmax >> operator function. >> --- >> src/plimage.c |2 +- >> 1 files changed, 1 insertions(+), 1 deletions(-) > > To Hazen and David: > > David, I hav

Re: [Plplot-devel] [PATCH] Fix plfimage function

2010-03-03 Thread David MacMahon
Hi, Alan, On Mar 3, 2010, at 12:05 , Alan W. Irwin wrote: > For source trees with and without the patch > > time make -j4 -k test_noninteractive >& make_testnoninteractive.out > time make -j4 -k test_interactive >& make_testinteractive.out I assume you would like, for example,... make && time m

Re: [Plplot-devel] [PATCH] Fix plfimage function

2010-03-03 Thread Alan W. Irwin
On 2010-03-01 12:20-0800 David MacMahon wrote: > > This fixes a parameter ordering problem in plfimage's call to the minmax > operator function. > --- > src/plimage.c |2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) To Hazen and David: David, I have some questions for you below. Haz

[Plplot-devel] [PATCH] Fix plfimage function

2010-03-01 Thread David MacMahon
This fixes a parameter ordering problem in plfimage's call to the minmax operator function. --- src/plimage.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/src/plimage.c b/src/plimage.c index bfa5ef7..d18b5cf 100644 --- a/src/plimage.c +++ b/src/plimage.c @@ -363,7 +36