libmir / dcv

Computer Vision Library for D Programming Language
http://dcv.dlang.io/
Boost Software License 1.0
90 stars 18 forks source link

ranged should be deprecated (again) #44

Open 9il opened 8 years ago

9il commented 8 years ago

As soon as mir.image be implemented, we will have good conversions between all formats we use. ranged is functions that stretch color domain for each channel. If you apply this for not very contrast image, it will make it more contrast. This implicit filter functionality is bad practice.

ljubobratovicrelja commented 8 years ago

If you apply this for not very contrast image, it will make it more contrast

Well, use should use histogram equalization to do that. ranged is not the right tool for contrast enhancement. There will be some more advanced methods for histogram equalization in DCV, but that's the basics.

This implicit filter functionality is bad practice

Primary use of ranged is not to adjust contrast in images. It is a simple tool, used often in signal (image) processing, in various occasions to linearly stretch the histogram. This is analogue to OpenCV's cv::normalize with norm type of NORM_MINMAX. Sorry, but ranged will not be deprecated, except mir.image can offer exactly the same functionality with as simple API.

9il commented 8 years ago

OK, but I am worrying that it is frequently used in DCV. BTW, I will add minmax and scale to Mir. So ranged will be optimized with this two functions.

ljubobratovicrelja commented 8 years ago

BTW, I will add minmax to Mir and scale

This is a whole another story. If those functions are present in low-level API manor, I'm sold. We'll deprecate ranged.

Reopening, and idling until minmax and scale are present in mir. Good?

9il commented 8 years ago

Yep, thanks!

ljubobratovicrelja commented 8 years ago

Np, thank you!