search

imglib / imglib2-algorithm

Image processing algorithms for ImgLib2
http://imglib2.net/
Other
22 stars 20 forks source link

Deprecate/Remove one of HyperSphere or HyperSphereNeighborhood #68

Open hanslovsky opened 6 years ago

hanslovsky commented 6 years ago

HyperSphere and HyperSphereNeighborhood do the same thing. Having both in imglib2-algorithm is confusing for the caller. I suggest to remove HyperSphere once #67 is fixed because HyperSphereNeighborhood implements Neighborhood as a common interface with other similar classes (HyperSphere is the lone class in its package).

acardona commented 5 years ago

To further add: HyperSphereNeighborhood is missing documentation, in addition to having bug #67. The deprecation of HyperSphere seems premature.

hanslovsky commented 5 years ago

That's why I suggested to remove it

once #67 is fixed