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

Smoothing filter #26

Closed
wants to merge 2 commits into from
Closed

Conversation

iory
Copy link
Contributor

@iory iory commented Aug 30, 2016

related to
#25

smoothing

@k-okada
Copy link
Contributor

k-okada commented Aug 30, 2016

please write code on top of
#25 and also add lines
like // https://github.com/Itseez/opencv/blob/2.4/samples/cpp/lk_demo.cpp,
to stop updating code. because if we have "original sample code", if some
one look at code and want to fix them, we can say this code is based on
original code, so I understand you want to change this, but do not have to.
If not, we'll have never-ending code updating process.

◉ Kei Okada

On Tue, Aug 30, 2016 at 7:20 PM, iory [email protected] wrote:

related to
#25 #25

[image: smoothing]

https://cloud.githubusercontent.com/assets/4690682/18085593/be4ddb44-6ee6-11e6-95aa-f735d9cd8f6d.png

You can view, comment on, or merge this pull request online at:

#26
Commit Summary

  • [smoothing_filter] Add smoothing_filters
  • [smoothing_filter] Add sample and test

File Changes

Patch Links:


You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
#26, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AAeG3B-xlGSorKSNIjMOiPBiVfSr3sM6ks5qlAPUgaJpZM4JwXos
.

@iory iory force-pushed the smoothing-filter branch 2 times, most recently from 11efc5a to 503ef9f Compare August 30, 2016 12:02
@iory
Copy link
Contributor Author

iory commented Aug 30, 2016

OK, I fixed those points.

@iory iory force-pushed the smoothing-filter branch 8 times, most recently from 9b19fe5 to 5f32f2a Compare September 5, 2016 14:52
@k-okada
Copy link
Contributor

k-okada commented Sep 12, 2016

fixed in #32

@k-okada k-okada closed this Sep 12, 2016
@k-okada
Copy link
Contributor

k-okada commented Jun 5, 2017

@iory please add info to wiki page http://wiki.ros.org/opencv_apps

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 this pull request may close these issues.

2 participants