imageproc icon indicating copy to clipboard operation
imageproc copied to clipboard

Add bilateral filter

Open theotherphil opened this issue 7 years ago • 7 comments

Possibly the fast version here: http://www.shellandslate.com/download/fastmedian_5506.pdf

theotherphil avatar Jul 03 '17 21:07 theotherphil

Hi, may I presume no one is working on this currently? If not, might there be any new thoughts on an implementation?

csheaff avatar Jan 12 '21 14:01 csheaff

I’m not working on this, and I’m not aware of anyone else working on it either.

theotherphil avatar Jan 12 '21 15:01 theotherphil

Thanks, I'm giving it a shot. I've taken a cursory look at the above approach and am curious if this was singled out for a particular reason. As mentioned in A Concise Review of Fast Bilateral Filtering (Das et al., 2017), going with a fast method typically involves some kind of trade off with accuracy. I hesitate to choose for the user what kind of trade off is acceptable and am curious if some other kind of module should be reserved for "fast" algorithms, such as OpenCV's ximgproc. Thoughts?

csheaff avatar Jan 17 '21 14:01 csheaff

Great! Providing the option to the user seems ideal, but an implementation of either approach would be a valuable addition to this library.

theotherphil avatar Jan 17 '21 14:01 theotherphil

I believe I've successfully implemented a version for grayscale images, so feel free to assign this ticket to me. I figure the next step would be getting it to work for color images. If you have any broad suggestions for adjustments (where the function belongs, testing, benchmarking, etc), they would be most welcome.

csheaff avatar Jan 25 '21 14:01 csheaff

Getting ready to submit a pull request for a grayscale implementation. Question: would you prefer that I commit a truth image to the repo with test function? I have created a test function using compare_to_truth_with_tolerance with a truth image generated by skimage.restoration.denoise_bilateral and it passes using a tolerance of 1 pixel. However, I imagine that having a truth image for every function won't be sustainable without blowing up the size of the repo. Thoughts?

csheaff avatar Mar 27 '21 16:03 csheaff

I submitted a pull request with the truth image. Feel free to chime in there regarding the above question.

csheaff avatar Mar 27 '21 21:03 csheaff

Can this issue be closed since it was added in #462?

ripytide avatar May 12 '24 19:05 ripytide

I plan on leaving this open until we have a version that works on colour images

theotherphil avatar May 12 '24 19:05 theotherphil