Hi all, The FAST and ORB pull request<https://github.com/scikit-image/scikit-image/pull/834>needs a final round of review. Johannes is done with some crucial design changes on the original PR<https://github.com/scikit-image/scikit-image/pull/699>. Please review it if you have some time. This might sound pretty lame, but merging this PR will motivate me to complete the work on FREAK descriptor<https://github.com/scikit-image/scikit-image/pull/729>. Thank you. Regards, Ankit.
Hi Ankit, Sorry, that this PR takes so long, but it is a rather large change. I made some (in my opinion final) changes, to also follow the estimator convention as discussed here: https://github.com/scikit-image/scikit-image/pull/840 This was basically the one thing I was waiting for and from my side we are good to merge now. Johannes On Jan 19, 2014, at 8:05, Ankit Agrawal <aaaagrawal@gmail.com> wrote:
Hi all,
The FAST and ORB pull request needs a final round of review. Johannes is done with some crucial design changes on the original PR. Please review it if you have some time. This might sound pretty lame, but merging this PR will motivate me to complete the work on FREAK descriptor. Thank you.
Regards, Ankit.
-- You received this message because you are subscribed to the Google Groups "scikit-image" group. To unsubscribe from this group and stop receiving emails from it, send an email to scikit-image+unsubscribe@googlegroups.com. For more options, visit https://groups.google.com/groups/opt_out.
participants (2)
-
Ankit Agrawal -
Johannes Schönberger