Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Histogram equalization #109

Closed
ivan1993br opened this issue Oct 8, 2020 · 4 comments
Closed

Histogram equalization #109

ivan1993br opened this issue Oct 8, 2020 · 4 comments

Comments

@ivan1993br
Copy link
Contributor

ivan1993br commented Oct 8, 2020

Hi all,

I was just wondering if a Histogram equalization would be in the scope of this package, I didn't find any ROS Node that does that, so I was thinking about adding it here.

What do you think @k-okada ?

Thanks!

@k-okada
Copy link
Contributor

k-okada commented Oct 9, 2020

@ivan1993br thank you for proposal Yes, addiing histgram equalization would be nice. This package is basically ROS node of opencv sample program and there already exsits https://github.com/opencv/opencv/tree/master/samples/cpp/tutorial_code/Histograms_Matching.
I have added some information on how to contirbute. so please look at
https://github.com/ros-perception/opencv_apps/blob/contrib/CONTRIBUTING.md

@bochen87
Copy link

since this feature is now available, would be really cool if it could be released as well!

@k-okada
Copy link
Contributor

k-okada commented Feb 25, 2022

@bochen87 thanks,
I also added node related to histogram at #128 in addition to #126. Do you have any comment/review on that?

@bochen87
Copy link

Yes, I tested #126 with clahe to improve our image preprocessing pipeline for better april tag detection under different lighting conditions, works very well!

I can test out #128 next week.

Would be really awesome to release a new debian package version with all these new features!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants