On 2010-04-29 14:46-0700 Alan W. Irwin wrote: > On 2010-04-29 14:22-0700 Alan W. Irwin wrote: > >> python >> Missing examples : >> Differing postscript output : 06 07 15 19 21 >> Missing stdout : >> Differing stdout : >> tcl >> Missing examples : >> Differing postscript output : 06 07 15 16 19 21 28 >> Missing stdout : >> Differing stdout : 21 > > I have looked further at the 15 and 16 issues. Interestingly, the errors > for example 15 (empty first page, mostly empty second) are identical for > python and tcl. For tcl example 16, the issue is a mostly empty first page. > My guess is something got badly clobbered with python and tcl with the > recent extensive API additions for either arbitrary organization of 2D data > or arbitrary transformation of world coordinates. Dave and Hez, will > either/both of you have a look at this?
My apologies to Dave and Hez who may have been looking in the wrong direction due to the above speculation. I should have used svn-bisect right away rather than speculating. That showed the problem was caused by Hez's revision 10895 which just changed 3 lines of plshade.c to remove tests on whether pltr_data was NULL. It turns out those tests are important for both our python and tcl bindings. I reinstated the tests for revision 10959 and the result was substantially improved python and tcl differences (results for examples 15 and 21 became consistent with C for python while results for examples 15 and 16 became consistent with C for tcl). Here are the current python and tcl results in full. python Missing examples : Differing postscript output : 06 07 19 Missing stdout : Differing stdout : tcl Missing examples : Differing postscript output : 06 07 19 21 28 Missing stdout : Differing stdout : 21 The differences for other bindings were unaffected by revision 10959. For platforms with Tcl 8.5, it is possible that example 21 for Tcl might be consistent with C for revision 10959 because of the good python result for that example. But I don't know for sure because I didn't actually test example 21 for Tcl on my platform; the stdout message for that example was "This example require Tcl 8.5 or later: use of NaNs". Hez, from your revision 10895 commit message you likely just wanted to make it a little easier to use your OCaml bindings. However, this change to ignore the pltr_data tests for NULL obviously affected the python and tcl example results (for reasons I haven't looked into), and our C users may also depend on these tests being made. So if you would like to pursue this idea further, you should try to figure out how to deal with the python and tcl issues and also consider the significance of this change for our C users. 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); PLplot scientific plotting software package (plplot.org); 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 __________________________ ------------------------------------------------------------------------------ _______________________________________________ Plplot-devel mailing list Plplot-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/plplot-devel
