I just quickly scanned through your source file, and it certainly looks OK to me in terms of (1) passes compilation, (2) is sufficiently documented, and (3) follows the vxl standards.
Maybe just add a (small) test which calls the compute() method and verifies the result.
Such a test should go in a new (separate) file in gel/mrc/vpgl/algo/tests/

(One minor remark: I would replace "or" by "||"; could be that some compilers would not understand the "or", I'm not sure.)

-- Peter.


--- Den mån 2011-04-25 skrev Andrew Hoelscher <>:

Från: Andrew Hoelscher <>
Ämne: Re: [Vxl-maintainers] Vpgl
Till: "Joe Mundy" <>
Datum: måndag 25 april 2011 21:47


Sounds good, attached is the file I would like to add.


On Mon, Apr 25, 2011 at 12:57 PM, Joe Mundy <> wrote:

Probably the easiest approach is for you to become a VXL developer. Why
don't you send and example of your code. If it looks reasonable to us,
then you just have to ask one of the administrators to add you to the list
of developers and you can update the repository directly going forward.

Joe Mundy

-----Infogad bilaga följer-----

WhatsUp Gold - Download Free Network Management Software
The most intuitive, comprehensive, and cost-effective network
management toolset available today.  Delivers lowest initial
acquisition cost and overall TCO of any competing solution.

-----Infogad bilaga följer-----

Vxl-maintainers mailing list