Extraneous grid types?

[ChrisBarker-NOAA]

Currently in the grids, we have:

fvcom_grid.py
regular_grid.py
regular_grid_2d.py
selfe_grid.py
sgrid.py 
ugrid.py

However, I can't see what regular_grid_2d.py does -- why make the distiction beetween 2D and 3D here? and if so, they should share more code.

and fvcom and selfe are both unstructured grids, so should be handled by ugrid, yes? (at least fvcom is now...)

@mpiannucci : do you remember what this is about? experiments when getting started? CAn we simply remove those?

[ChrisBarker-NOAA]

Ahh -- I see, in FVCOMGrid:

        # For now, the only reason for this subclass is to add support
        # for subsetting FVCOMs non standard vertical levels

so that makes sense. I do wonder if that can be done some other way, but this seems to work.

[ChrisBarker-NOAA]

I haven't looked at selfe yet, but I'm guessing it's similar.

[ChrisBarker-NOAA]

Still to determine what the story is with the regular_grid_2d ...

-CHB