[Bug 702722] opencv: add GrabCut segmentation element

[GStreamer (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=702722
  GStreamer | gst-plugins-bad | 1.x

--- Comment #11 from Sebastian Dröge <slomo at circular-chaos.org> 2013-07-11 10:54:52 UTC ---
(In reply to comment #10)
> (In reply to comment #9)
> > Review of attachment 248843 [details] [details]:
> > 
> > ::: ext/opencv/gstfacedetect.c
> > @@ +681,3 @@
> > +        ev = gst_event_new_custom (GST_EVENT_CUSTOM_DOWNSTREAM, str);
> > +        gst_pad_push_event (GST_BASE_TRANSFORM_SRC_PAD
> > (GST_BASE_TRANSFORM_CAST
> > +                (filter)), ev);
> > 
> > I still believe this should be a GstMeta, attached to any buffer to which it is
> > relevant... potentially many per buffer if there are many faces :)
> > 
> 
> Yes but facedetect selects the largest face detected and grabcut does not
> operate well on non-connected foreground patches.

But that's IMHO an implementation detail that shouldn't be part of the API.
grabcut can still just select the largest face from the list it gets, and
facedetect can only detect a single face. Other algorithms later could do
something more optimal :)

-- 
Configure bugmail: https://bugzilla.gnome.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.
You are the assignee for the bug.