Hi Hez:

Thanks for your contribution to PLplot.

On 2008-03-09 17:45-0400 Hezekiah M. Carty wrote:

> With this new patch, the updated plimage function is called plimager
> ("r" for "range").  plimage is, as you suggested Alan, a wrapper
> around plimager using the min and max values from idata as valuemin
> and valuemax.  It should act in the same way though - please let me
> know if it does not.

I have changed the plimager name to plimagefr (fixed range).  I changed the
name because "imager" has a definite meaning that would give the wrong idea
about the capabilities of this function, but if there is a ground swell for
"plimager" or yet another name, I can make the appropriate changes.  I have
also changed this part of your patch so that if follows our "c_" convention
(don't ask) for our public API names.

Everything works for me on Debian testing (including ctest of example 20 with
gv of those plimage results to make sure they look fine).  Therefore, I have
committed this part of your patch.  Please try out our latest svn version to
make sure plimagefr works for you as intended.

>
> Finally, I updated api.xml with these two functions.  I am unfamiliar
> with DocBook so I copied and pasted another entry and used that as a
> template.

I am working on that part of your patch to get rid of a few issues found by
"make validate". However, that was an excellent first try.  Thank you very
much.  You are now a DocBook expert!  :-)

Alan






>
> I do not think that any bindings will have to be updated unless they
> want to add plimager support since the plimage function signature has
> not changed.
>

My new check script now shows plimagefr is missing everywhere in the various
bindings.  That is fine because it doesn't break anything, and we will
propagate plimagefr to all the bindings as time permits.

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
__________________________

-------------------------------------------------------------------------
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2008.
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/
_______________________________________________
Plplot-devel mailing list
Plplot-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/plplot-devel

Reply via email to