Alan, Thanks for spotting this and sorry for the slightly tardy response. I think all is now more or less fixed (see comments below).
On Wed, Jul 21, 2010 at 12:17:24PM -0700, Alan Irwin wrote: > Hi Andrew: > > I find that with Octave 3.2.4-6 from Debian testing that there are > errors in the p7, p16, p17, p20, and p21 examples. > > In particular, if you run > > make test_octave_xwin > > (which indirectly runs plplot_test/test_octave_interactive.sh(.in)) or > > make test_octave_psc > > (which indirectly runs plplot_test/test_octave.sh(.in)) you will find > the following error messages: > > p7 > warning: split is obsolete and will be removed from a future version > of Octave; please use strsplit instead strsplit was only introduced in octave3.2 so I do not (yet) want to switch to it. Octave 3.0 is still widely used. Everything else in plplot just requires 3.0. > error: dimension mismatch in argument a > error: called from: > error: plshade1 at line 4848, column 1 > error: > /home/software/plplot_svn/HEAD/plplot_cmake_qt/examples/../bindings/octave/PLplot/shade.m > at line 175, column 3 > error: > /home/software/plplot_svn/HEAD/plplot_cmake_qt/examples/octave/p7.m at > line 25, column 3 > > and similarly for p16, and p20. This was a bug introduced by me. I've now fixed this, and while I was on I improved shade to also support 2-d arrays for x and y values. Matlab contourf allows this and it was something I've missed for a while in the octave plplot bindings. > p17 has its own special error (note the first two warnings are for > previous examples): > > p17 > warning: split is obsolete and will be removed from a future version > of Octave; please use strsplit instead See above > warning: comet: FIXME: axis is 'auto', turning it to 'tight'. This is a very old comment from Joao or Rafael which I don't entirely understand. > warning: loadimage is obsolete and will be removed from a future > version of Octave; please use imread instead Fixed. Imread has been available for a while. Once I sort out the path issues imread will actually load lena.pgm as well so we don't need a separate lena.img file for octave. > warning: division by zero > warning: division by zero > warning: division by zero > warning: division by zero > > *** PLPLOT WARNING *** > plwind: Invalid window limits in x. > > *** PLPLOT WARNING *** > plwind: Invalid window limits in y. > > *** PLPLOT ERROR, ABORTING OPERATION *** > plimagefr: nx and ny must be positive, aborting operation This comes if your selection rectangle has zero width or height. plimage does not like this... > I presume the errors for p7, p16, and p20 are at least partially > generated by your revision 11089 to bindings/octave/PLplot/shade where > you replaced plshade by plshade1. I don't know what causes the > error(s) for p17. > > I assume you ran test_octave_xwin or equivalent with your version of > octave with no errors so there is probably a verification issue for these > errors. Note, the recommended standard version of octave according to > http://www.gnu.org/software/octave/download.html is 3.2.4, and I > presume the Debian testing version 3.2.4-6 that I have used has only > had bug fixes applied rather than substantial changes to that standard > version. Therefore, I believe it is likely you will be able to verify > my result if you can upgrade/downgrade your octave version to 3.2.4. > > I have temporarily (revision 11094) removed p7, p16, p17, p20, and p21 > (the combined example) from these tests so the above two specific test > targets and the corresponding test_interactive and test_noninteractive > targets will complete without errors. Of course, these examples > should be restored when these issues are fixed. I've now re-enabled all these tests. Please check. In testing I've discovered some other "issues". Strangely, calling the builtin octave pause function clears the contour plot, but not the axes or title. This is bizarre! I will look further. Andrew ------------------------------------------------------------------------------ The Palm PDK Hot Apps Program offers developers who use the Plug-In Development Kit to bring their C/C++ apps to Palm for a share of $1 Million in cash or HP Products. Visit us here for more details: http://ad.doubleclick.net/clk;226879339;13503038;l? http://clk.atdmt.com/CRS/go/247765532/direct/01/ _______________________________________________ Plplot-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/plplot-devel
