CV icon indicating copy to clipboard operation
CV copied to clipboard

ImageMath types

Open amnipar opened this issue 12 years ago • 1 comments

Some ImageMath functions have type issues that are revealed by haddock documents. Like, findMinMaxMask uses BareImage. It is also dubious which images can (or should) be called with D8 images. In my documentation commit, I have already changed some functions (like sqrt) to accept D32 image only. Then addS is now for D32 only while it might be possible to use it for D8 - but there could be overflow problems. Suggest deciding this matter for every function.

As a side note, should we have a BinaryImage type (perhaps newtype of Image GrayScale D8) that would be the only image type accepted for masks and such, and could be produced by comparison operators?

amnipar avatar Mar 28 '12 09:03 amnipar

Just an update: Could you be more specific on what needs to be changed?

aleator avatar Jul 25 '12 15:07 aleator