Merging CISLAM Experimental model (no Test Suite available) in JGrassTools

The wrapper of the main module is working correctly. All Oms sub-modules are fine. Wrappers for sub-modules might not be fully functional. 
I forgot this mail: now OmsCislam should be free from dead/commented code.


On 3 November 2014 03:50, Andrea Antonello <notifications@github.com> wrote:

> In
> hortonmachine/src/main/java/org/jgrasstools/hortonmachine/modules/hydrogeomorphology/cislam/OmsCislam.java:
>
> > +    */
> > +
> > +    // #############################################
> > +    // van Genuchten parameters
> > +    // #############################################
> > +    @Description(OMSCISLAM_inAlfaVanGen_DESCRIPTION)
> > +    @Unit("1/m")
> > +    @UI(JGTConstants.FILEIN_UI_HINT)
> > +    @In
> > +    public GridCoverage2D inAlfaVanGen = null;
> > +    /*
> > +    @Description(OMSCISLAM_pAlfaVanGen_DESCRIPTION)
> > +    @In
> > +    public double pAlfaVanGen = 6.5;
> > +    */
> > +    @Description(OMSCISLAM_inNVanGen_DESCRIPTION)
>
> Hi, can I ask you to remove commented variables? If they are not needed,
> please just remove them.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/moovida/jgrasstools/pull/8/files#r19717772>.
>