[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

Sebastian Dröge <slomo> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |slomo at circular-chaos.org

--- Comment #6 from Sebastian Dröge <slomo at circular-chaos.org> 2013-07-10 07:14:46 UTC ---
(In reply to comment #4)

> > @@ +276,3 @@
> > +  grabcut->facepos.y = 77;
> > +  grabcut->facepos.width = 60;
> > +  grabcut->facepos.height = 70;
> > 
> > What are these magic number? :)
> 
> They are the magic number of a person in a video conference environment looking
> at a laptop's front camera. Is just a failsafe option to avoid empty FG masks,
> see next comment.

Can't you just check later if you received (valid) facepos before cutting?

> > @@ +334,3 @@
> > +        if (abs (w) > 2)
> > +          grabcut->facepos.width = (int) w;
> > +        if (abs (h) > 2)
> > 
> > Why these checks for >2?
> > 
> 
> In case of misconfigured face detectors, some faces are spurious and would have
> widths/heights of 1. This is to filter those out, GrabCut doesn't like too
> small seeds... (there's an ASSERT somewhere).

Same here :)

-- 
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.