Discussion:
[Bug 768248] New: vaapiencode: Add Encoding region-of-interest (ROI) support
Add Reply
"GStreamer" (GNOME Bugzilla)
2016-06-30 14:42:14 UTC
Reply
Permalink
Raw Message
https://bugzilla.gnome.org/show_bug.cgi?id=768248

Bug ID: 768248
Summary: vaapiencode: Add Encoding region-of-interest (ROI)
support
Classification: Platform
Product: GStreamer
Version: unspecified
OS: Linux
Status: NEW
Severity: enhancement
Priority: Normal
Component: gstreamer-vaapi
Assignee: gstreamer-***@lists.freedesktop.org
Reporter: ***@gmail.com
QA Contact: gstreamer-***@lists.freedesktop.org
CC: ***@gmail.com, ***@igalia.com
GNOME version: ---

Add support for QP adjustments in specific(user provided) areas of the frames
when encoding.

ROIs can be set through the VAENCMiscParameterBufferROIs. The ROI set through
this structure is applicable only to the current frame/field, so must be sent
every frame/field to be applied. The number of supported ROIs can be queried
through the VAConfigAttribEncROI. The encoder will use the ROI information to
adjust the QP values of the MB's that fall within the ROIs.

This feature is for giving more control to the user in order to tune the encode
quality.
--
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.
"GStreamer" (GNOME Bugzilla)
2016-10-11 02:15:26 UTC
Reply
Permalink
Raw Message
https://bugzilla.gnome.org/show_bug.cgi?id=768248

Lim Siew Hoon <***@intel.com> changed:

What |Removed |Added
----------------------------------------------------------------------------
CC| |***@intel.com
--
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.
"GStreamer" (GNOME Bugzilla)
2017-01-12 08:07:12 UTC
Reply
Permalink
Raw Message
https://bugzilla.gnome.org/show_bug.cgi?id=768248

Hyunjun Ko <***@igalia.com> changed:

What |Removed |Added
----------------------------------------------------------------------------
CC| |***@igalia.com
--
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.
"GStreamer" (GNOME Bugzilla)
2017-01-17 07:15:14 UTC
Reply
Permalink
Raw Message
https://bugzilla.gnome.org/show_bug.cgi?id=768248

--- Comment #1 from Hyunjun Ko <***@igalia.com> ---
I've been working for this issue and I found some issues to discuss.

1. Handling region of interest meta?
There's already video meta for this, called "GstVideoRegionOfInterestMeta"
But, afaik, there's no element to use this except for opencv element.(face
detection)
Does vaapi encoder need to handle this or just create property like roi-x,
roi-y, roi-w, roi-h? Or both?

2. Other property to be created in vaapi encoder.
Maybe these 2 properties are necessary to provide.
- enable-roi : Enable/Disable roi encoding.
- roi-value : a kind of level of roi region
http://01org.github.io/libva_staging_doxygen/struct__VAEncMiscParameterBufferRoi_1_1VAEncROI.html#ab37eca45463295659565a46430b0b925

And not sure if these attributes are necessary or not as a property.
- max/min delta qp : works only if CQP mode
- num_of_roi : if we supports this, we should support for setting of the number
of {x,y,w,h}, corresponding to num_of_roi. but how?

Refer to
http://01org.github.io/libva_staging_doxygen/struct__VAEncMiscParameterBufferRoi.html


I think I'm going to start implementation of kinda version 0.1, but need
opinions about this :)
--
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.
"GStreamer" (GNOME Bugzilla)
2017-01-17 17:11:16 UTC
Reply
Permalink
Raw Message
https://bugzilla.gnome.org/show_bug.cgi?id=768248

--- Comment #2 from Víctor Manuel Jáquez Leal <***@igalia.com> ---
From my point of view, the "gst" way of implementing this, is creating a new
GST_EVENT_CUSTOM_DOWNSTREAM (perhaps also upstream) that the application could
inject into the pipeline with gst_element_send_event().

The event will have an GstStructure named, for example
GstVaapiEncoderRegionOfInterest, with roi-x, roi-y, roi-width, roi-height, etc.

The vaapi encoders will catch this event and will configure themselves
accordingly.

That's it, from the top of my head.
--
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.
"GStreamer" (GNOME Bugzilla)
2017-01-17 21:46:03 UTC
Reply
Permalink
Raw Message
https://bugzilla.gnome.org/show_bug.cgi?id=768248

Julien Isorce <***@gmail.com> changed:

What |Removed |Added
----------------------------------------------------------------------------
CC| |***@gmail.com
--
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.
"GStreamer" (GNOME Bugzilla)
2017-01-31 08:02:43 UTC
Reply
Permalink
Raw Message
https://bugzilla.gnome.org/show_bug.cgi?id=768248

--- Comment #3 from Hyunjun Ko <***@igalia.com> ---
Created attachment 344625
--> https://bugzilla.gnome.org/attachment.cgi?id=344625&action=edit
libs: encoder/context: query region of interest support

Query Region of Interest support during config creation.
--
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.
"GStreamer" (GNOME Bugzilla)
2017-01-31 08:03:25 UTC
Reply
Permalink
Raw Message
https://bugzilla.gnome.org/show_bug.cgi?id=768248

--- Comment #4 from Hyunjun Ko <***@igalia.com> ---
Created attachment 344626
--> https://bugzilla.gnome.org/attachment.cgi?id=344626&action=edit
libs: encoder: add api gst_vaapi_encoder_set_roi

Implements and exposes new api gst_vaapi_encoder_set_roi to set ROI regions.
--
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.
"GStreamer" (GNOME Bugzilla)
2017-01-31 08:04:01 UTC
Reply
Permalink
Raw Message
https://bugzilla.gnome.org/show_bug.cgi?id=768248

--- Comment #5 from Hyunjun Ko <***@igalia.com> ---
Created attachment 344627
--> https://bugzilla.gnome.org/attachment.cgi?id=344627&action=edit
libs: encoder: h264: set ROI params during encoding

Set ROI params during encoding each frame,
which is set via gst_vaapi_encoder_set_roi
--
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.
"GStreamer" (GNOME Bugzilla)
2017-01-31 08:04:42 UTC
Reply
Permalink
Raw Message
https://bugzilla.gnome.org/show_bug.cgi?id=768248

--- Comment #6 from Hyunjun Ko <***@igalia.com> ---
Created attachment 344628
--> https://bugzilla.gnome.org/attachment.cgi?id=344628&action=edit
tests: simple-encoder: add an option to set ROI

Adds an option "roi" to test roi support of encoder.
--
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.
"GStreamer" (GNOME Bugzilla)
2017-01-31 08:09:23 UTC
Reply
Permalink
Raw Message
https://bugzilla.gnome.org/show_bug.cgi?id=768248

--- Comment #7 from Hyunjun Ko <***@igalia.com> ---
Internally, query/set ROI params are implemented in these patches.
Once these can be acceptable, then we can discuss how to provide the way to set
ROI params to users. (GstEvent, or anytihng else)

And we can go further to support for other codecs such as vp8.
--
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.
"GStreamer" (GNOME Bugzilla)
2017-02-17 01:47:51 UTC
Reply
Permalink
Raw Message
https://bugzilla.gnome.org/show_bug.cgi?id=768248

--- Comment #8 from Hyunjun Ko <***@igalia.com> ---
Ping.
Victor, Sree? :)
--
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.
"GStreamer" (GNOME Bugzilla)
2017-02-17 15:41:20 UTC
Reply
Permalink
Raw Message
https://bugzilla.gnome.org/show_bug.cgi?id=768248

Víctor Manuel Jáquez Leal <***@igalia.com> changed:

What |Removed |Added
----------------------------------------------------------------------------
Attachment #344625|none |needs-work
status| |

--- Comment #9 from Víctor Manuel Jáquez Leal <***@igalia.com> ---
Review of attachment 344625:
--> (https://bugzilla.gnome.org/review?bug=768248&attachment=344625)

::: gst-libs/gst/vaapi/gstvaapicontext.c
@@ +314,3 @@
+ GST_ERROR ("ROI unsupported - number of regions supported: %d"
+ " ROI delta QP: %d", roi_config->bits.num_roi_regions,
+ roi_config->bits.roi_rc_qp_delat_support);

I guess you should improve the error message, imo it is misleading because ROI
can be supported but not the number of requested regions. Also, I would keep
this as a warning and do not break the encoding.

::: gst-libs/gst/vaapi/gstvaapiencoder.c
@@ +573,3 @@
+ GST_INFO ("ROI unsupported - number of regions supported: %d"
+ " ROI delta QP: %d", roi_config->bits.num_roi_regions,
+ roi_config->bits.roi_rc_qp_delat_support);

If ROI is unsupported, there is no reason to print the number of regions or if
delta QP is supported, since any of they are zero.

By the way, I guess we should open a bug in libva because of the typo delat ==
delta, but that will break the API.
--
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.
"GStreamer" (GNOME Bugzilla)
2017-02-17 15:48:25 UTC
Reply
Permalink
Raw Message
https://bugzilla.gnome.org/show_bug.cgi?id=768248

Víctor Manuel Jáquez Leal <***@igalia.com> changed:

What |Removed |Added
----------------------------------------------------------------------------
Attachment #344626|none |needs-work
status| |

--- Comment #10 from Víctor Manuel Jáquez Leal <***@igalia.com> ---
Review of attachment 344626:
--> (https://bugzilla.gnome.org/review?bug=768248&attachment=344626)

::: gst-libs/gst/vaapi/gstvaapiencoder.c
@@ +1230,3 @@
+
+ if (!config->roi_capability)
+ return;

Turn the function into boolean to report back if it's possible to execute the
operation.

@@ +1240,3 @@
+ g_array_free (encoder->roi_regions, FALSE);
+
+ encoder->roi_regions = g_array_new (FALSE, FALSE, sizeof (GstVaapiROI));

As we know the size, we should use g_array_sized_new ()

::: gst-libs/gst/vaapi/gstvaapiencoder.h
@@ +129,3 @@
+ guint w;
+ guint h;
+} GstVaapiROI;

I would reuse the GstVaapiRectangle nested in GstVaapiROI

@@ +187,3 @@

+void
+gst_vaapi_encoder_set_roi (GstVaapiEncoder * encoder, GArray * roi_regions);

What about an API to add and delete single ROIs??

gboolean gst_vaapi_encoder_add_roi (GstVaapiEncoder * encoder, GstVaapiROI *
roi);
gboolean gst_vaapi_encoder_del_roi (GstVaapiEncoder * encoder, GstVaapiROI *
roi);
--
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.
"GStreamer" (GNOME Bugzilla)
2017-02-17 15:53:20 UTC
Reply
Permalink
Raw Message
https://bugzilla.gnome.org/show_bug.cgi?id=768248

Víctor Manuel Jáquez Leal <***@igalia.com> changed:

What |Removed |Added
----------------------------------------------------------------------------
Attachment #344627|none |reviewed
status| |

--- Comment #11 from Víctor Manuel Jáquez Leal <***@igalia.com> ---
Review of attachment 344627:
--> (https://bugzilla.gnome.org/review?bug=768248&attachment=344627)

lgtm
--
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.
"GStreamer" (GNOME Bugzilla)
2017-02-17 16:02:49 UTC
Reply
Permalink
Raw Message
https://bugzilla.gnome.org/show_bug.cgi?id=768248

Víctor Manuel Jáquez Leal <***@igalia.com> changed:

What |Removed |Added
----------------------------------------------------------------------------
Attachment #344628|none |reviewed
status| |

--- Comment #12 from Víctor Manuel Jáquez Leal <***@igalia.com> ---
Review of attachment 344628:
--> (https://bugzilla.gnome.org/review?bug=768248&attachment=344628)

lgtm

::: tests/simple-encoder.c
@@ +197,3 @@
+ }
+
+ gst_vaapi_encoder_set_roi (encoder, roi_regions);

Looking at this, I guess that the approach of {add,del}_roi is the way to go.
--
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.
"GStreamer" (GNOME Bugzilla)
2017-02-17 16:05:01 UTC
Reply
Permalink
Raw Message
https://bugzilla.gnome.org/show_bug.cgi?id=768248

--- Comment #13 from Víctor Manuel Jáquez Leal <***@igalia.com> ---
Review of attachment 344626:
--> (https://bugzilla.gnome.org/review?bug=768248&attachment=344626)

::: gst-libs/gst/vaapi/gstvaapiencoder.c
@@ +1227,3 @@
+ guint roi_num, i;
+
+ g_return_if_fail (roi_regions != NULL);

Is is possible to add/delete ROIs when the encoder is already processing a
stream?

If not, we should also rely on the queued_codedbuf_num
--
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.
"GStreamer" (GNOME Bugzilla)
2017-02-21 04:45:46 UTC
Reply
Permalink
Raw Message
https://bugzilla.gnome.org/show_bug.cgi?id=768248

--- Comment #14 from Hyunjun Ko <***@igalia.com> ---
(In reply to Víctor Manuel Jáquez Leal from comment #9)
Post by "GStreamer" (GNOME Bugzilla)
::: gst-libs/gst/vaapi/gstvaapicontext.c
@@ +314,3 @@
+ GST_ERROR ("ROI unsupported - number of regions supported: %d"
+ " ROI delta QP: %d", roi_config->bits.num_roi_regions,
+ roi_config->bits.roi_rc_qp_delat_support);
I guess you should improve the error message, imo it is misleading because
ROI can be supported but not the number of requested regions. Also, I would
keep this as a warning and do not break the encoding.
I think this should be error since the supported roi thing is already queried
and set to config(cip->config.encoder). That's why roi supported_num should be
same. If we want to update, we should turn off "static" for
cip->config.encoder, but current logic isn't aimed at it, I guess.
Post by "GStreamer" (GNOME Bugzilla)
::: gst-libs/gst/vaapi/gstvaapiencoder.c
@@ +573,3 @@
+ GST_INFO ("ROI unsupported - number of regions supported: %d"
+ " ROI delta QP: %d", roi_config->bits.num_roi_regions,
+ roi_config->bits.roi_rc_qp_delat_support);
If ROI is unsupported, there is no reason to print the number of regions or
if delta QP is supported, since any of they are zero.
Agree, I'll change it.
Post by "GStreamer" (GNOME Bugzilla)
By the way, I guess we should open a bug in libva because of the typo delat
== delta, but that will break the API.
Wow, nice catch. I'm going to raise this issue.
--
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.
"GStreamer" (GNOME Bugzilla)
2017-02-21 04:48:20 UTC
Reply
Permalink
Raw Message
https://bugzilla.gnome.org/show_bug.cgi?id=768248

--- Comment #15 from Hyunjun Ko <***@igalia.com> ---
(In reply to Víctor Manuel Jáquez Leal from comment #10)
Post by "GStreamer" (GNOME Bugzilla)
::: gst-libs/gst/vaapi/gstvaapiencoder.c
@@ +1230,3 @@
+
+ if (!config->roi_capability)
+ return;
Turn the function into boolean to report back if it's possible to execute
the operation.
@@ +1240,3 @@
+ g_array_free (encoder->roi_regions, FALSE);
+
+ encoder->roi_regions = g_array_new (FALSE, FALSE, sizeof (GstVaapiROI));
As we know the size, we should use g_array_sized_new ()
::: gst-libs/gst/vaapi/gstvaapiencoder.h
@@ +129,3 @@
+ guint w;
+ guint h;
+} GstVaapiROI;
I would reuse the GstVaapiRectangle nested in GstVaapiROI
@@ +187,3 @@
+void
+gst_vaapi_encoder_set_roi (GstVaapiEncoder * encoder, GArray * roi_regions);
What about an API to add and delete single ROIs??
gboolean gst_vaapi_encoder_add_roi (GstVaapiEncoder * encoder, GstVaapiROI *
roi);
gboolean gst_vaapi_encoder_del_roi (GstVaapiEncoder * encoder, GstVaapiROI *
roi);
Well, I like your approach better than mine.
Let me finish this work according to your suggestion.
--
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.
"GStreamer" (GNOME Bugzilla)
2017-02-21 04:50:14 UTC
Reply
Permalink
Raw Message
https://bugzilla.gnome.org/show_bug.cgi?id=768248

--- Comment #16 from Hyunjun Ko <***@igalia.com> ---
(In reply to Víctor Manuel Jáquez Leal from comment #13)
Post by "GStreamer" (GNOME Bugzilla)
::: gst-libs/gst/vaapi/gstvaapiencoder.c
@@ +1227,3 @@
+ guint roi_num, i;
+
+ g_return_if_fail (roi_regions != NULL);
Is is possible to add/delete ROIs when the encoder is already processing a
stream?
If not, we should also rely on the queued_codedbuf_num
Yes, it's possible theoretically though I haven't tested it.
--
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.
"GStreamer" (GNOME Bugzilla)
2017-02-23 09:57:24 UTC
Reply
Permalink
Raw Message
https://bugzilla.gnome.org/show_bug.cgi?id=768248

Hyunjun Ko <***@igalia.com> changed:

What |Removed |Added
----------------------------------------------------------------------------
Attachment #344625|needs-work |none
status| |
Attachment #344625|0 |1
is obsolete| |
Attachment #344626|needs-work |none
status| |
Attachment #344626|0 |1
is obsolete| |
Attachment #344627|reviewed |none
status| |
Attachment #344627|0 |1
is obsolete| |
Attachment #344628|reviewed |none
status| |
Attachment #344628|0 |1
is obsolete| |

--- Comment #17 from Hyunjun Ko <***@igalia.com> ---
Created attachment 346550
--> https://bugzilla.gnome.org/attachment.cgi?id=346550&action=edit
libs: encoder/context: query region of interest support

Query Region of Interest support during config creation.
--
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.
"GStreamer" (GNOME Bugzilla)
2017-02-23 09:57:59 UTC
Reply
Permalink
Raw Message
https://bugzilla.gnome.org/show_bug.cgi?id=768248

--- Comment #18 from Hyunjun Ko <***@igalia.com> ---
Created attachment 346551
--> https://bugzilla.gnome.org/attachment.cgi?id=346551&action=edit
libs: encoder: add api gst_vaapi_encoder_add/del_roi

Implements and exposes new api gst_vaapi_encoder_add/del_roi to set ROI
regions.
--
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.
"GStreamer" (GNOME Bugzilla)
2017-02-23 09:58:41 UTC
Reply
Permalink
Raw Message
https://bugzilla.gnome.org/show_bug.cgi?id=768248

--- Comment #19 from Hyunjun Ko <***@igalia.com> ---
Created attachment 346552
--> https://bugzilla.gnome.org/attachment.cgi?id=346552&action=edit
libs: encoder: h264: set ROI params during encoding

Set ROI params during encoding each frame, which is set via
gst_vaapi_encoder_add_roi
--
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.
"GStreamer" (GNOME Bugzilla)
2017-02-23 09:59:10 UTC
Reply
Permalink
Raw Message
https://bugzilla.gnome.org/show_bug.cgi?id=768248

--- Comment #20 from Hyunjun Ko <***@igalia.com> ---
Created attachment 346553
--> https://bugzilla.gnome.org/attachment.cgi?id=346553&action=edit
tests: simple-encoder: add an option to set ROI
--
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.
"GStreamer" (GNOME Bugzilla)
2017-03-16 12:48:50 UTC
Reply
Permalink
Raw Message
https://bugzilla.gnome.org/show_bug.cgi?id=768248

Víctor Manuel Jáquez Leal <***@igalia.com> changed:

What |Removed |Added
----------------------------------------------------------------------------
Attachment #346551|none |needs-work
status| |

--- Comment #21 from Víctor Manuel Jáquez Leal <***@igalia.com> ---
Review of attachment 346551:
--> (https://bugzilla.gnome.org/review?bug=768248&attachment=346551)

::: gst-libs/gst/vaapi/gstvaapiencoder.c
@@ +1211,3 @@
}

+#if 0

Why is this code disabled? Should it be removed?

@@ +1243,3 @@
+#else
+gboolean
+gst_vaapi_encoder_add_roi (GstVaapiEncoder * encoder, GstVaapiROI * roi)

Document this function. Especially if @roi transfers its ownership.

@@ +1263,3 @@
+
+gboolean
+gst_vaapi_encoder_del_roi (GstVaapiEncoder * encoder, GstVaapiROI * roi)

ditto.
--
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.
"GStreamer" (GNOME Bugzilla)
2017-03-16 12:49:31 UTC
Reply
Permalink
Raw Message
https://bugzilla.gnome.org/show_bug.cgi?id=768248

Víctor Manuel Jáquez Leal <***@igalia.com> changed:

What |Removed |Added
----------------------------------------------------------------------------
Attachment #346550|none |reviewed
status| |

--- Comment #22 from Víctor Manuel Jáquez Leal <***@igalia.com> ---
Review of attachment 346550:
--> (https://bugzilla.gnome.org/review?bug=768248&attachment=346550)

lgtm
--
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.
"GStreamer" (GNOME Bugzilla)
2017-03-16 12:51:19 UTC
Reply
Permalink
Raw Message
https://bugzilla.gnome.org/show_bug.cgi?id=768248

Víctor Manuel Jáquez Leal <***@igalia.com> changed:

What |Removed |Added
----------------------------------------------------------------------------
Attachment #346552|none |reviewed
status| |

--- Comment #23 from Víctor Manuel Jáquez Leal <***@igalia.com> ---
Review of attachment 346552:
--> (https://bugzilla.gnome.org/review?bug=768248&attachment=346552)

lgtm
--
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.
"GStreamer" (GNOME Bugzilla)
2017-03-16 12:51:58 UTC
Reply
Permalink
Raw Message
https://bugzilla.gnome.org/show_bug.cgi?id=768248

Víctor Manuel Jáquez Leal <***@igalia.com> changed:

What |Removed |Added
----------------------------------------------------------------------------
Attachment #346553|none |reviewed
status| |

--- Comment #24 from Víctor Manuel Jáquez Leal <***@igalia.com> ---
Review of attachment 346553:
--> (https://bugzilla.gnome.org/review?bug=768248&attachment=346553)

lgtm
--
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.
"GStreamer" (GNOME Bugzilla)
2017-03-16 12:54:13 UTC
Reply
Permalink
Raw Message
https://bugzilla.gnome.org/show_bug.cgi?id=768248

--- Comment #25 from Víctor Manuel Jáquez Leal <***@igalia.com> ---
Almost there. Though I would not push this code until we have a way to use it
in a pipeline.
--
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.
"GStreamer" (GNOME Bugzilla)
2017-03-17 02:09:54 UTC
Reply
Permalink
Raw Message
https://bugzilla.gnome.org/show_bug.cgi?id=768248

Hyunjun Ko <***@igalia.com> changed:

What |Removed |Added
----------------------------------------------------------------------------
Attachment #346551|needs-work |none
status| |
Attachment #346551|0 |1
is obsolete| |

--- Comment #26 from Hyunjun Ko <***@igalia.com> ---
Created attachment 348142
--> https://bugzilla.gnome.org/attachment.cgi?id=348142&action=edit
libs: encoder: add api gst_vaapi_encoder_add/del_roi

Implements and exposes new api gst_vaapi_encoder_add/del_roi to set ROI
regions.
--
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.
"GStreamer" (GNOME Bugzilla)
2017-03-17 02:21:12 UTC
Reply
Permalink
Raw Message
https://bugzilla.gnome.org/show_bug.cgi?id=768248

--- Comment #27 from Hyunjun Ko <***@igalia.com> ---
(In reply to Víctor Manuel Jáquez Leal from comment #25)
Post by "GStreamer" (GNOME Bugzilla)
Almost there. Though I would not push this code until we have a way to use
it in a pipeline.
I'm going to start working on GstEvent to support this.
--
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.
"GStreamer" (GNOME Bugzilla)
2017-03-29 08:34:08 UTC
Reply
Permalink
Raw Message
https://bugzilla.gnome.org/show_bug.cgi?id=768248

--- Comment #28 from Hyunjun Ko <***@igalia.com> ---
Recently, seems ROI is broken in the driver. I filed up this issue to
https://github.com/01org/intel-vaapi-driver/issues/108
--
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.
"GStreamer" (GNOME Bugzilla)
2017-03-30 09:25:45 UTC
Reply
Permalink
Raw Message
https://bugzilla.gnome.org/show_bug.cgi?id=768248

Hyunjun Ko <***@igalia.com> changed:

What |Removed |Added
----------------------------------------------------------------------------
Attachment #348142|0 |1
is obsolete| |

--- Comment #29 from Hyunjun Ko <***@igalia.com> ---
Created attachment 348978
--> https://bugzilla.gnome.org/attachment.cgi?id=348978&action=edit
libs: encoder: add api gst_vaapi_encoder_add/del_roi

Implements and exposes new api gst_vaapi_encoder_add/del_roi to set ROI
regions.


-----
I revised this patch, especially about memory allocation happens inside api.
Please review this again.
--
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.
"GStreamer" (GNOME Bugzilla)
2017-03-30 09:26:24 UTC
Reply
Permalink
Raw Message
https://bugzilla.gnome.org/show_bug.cgi?id=768248

--- Comment #30 from Hyunjun Ko <***@igalia.com> ---
Created attachment 348979
--> https://bugzilla.gnome.org/attachment.cgi?id=348979&action=edit
vaapiencode: handle custom event GstVaapiEncoderRegionOfInterest

Handles new custom event GstVaapiEncoderRegionOfInterest
to enable/disable a ROI region.

Writes a way to use new event to document.
--
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.
"GStreamer" (GNOME Bugzilla)
2017-03-30 09:27:03 UTC
Reply
Permalink
Raw Message
https://bugzilla.gnome.org/show_bug.cgi?id=768248

--- Comment #31 from Hyunjun Ko <***@igalia.com> ---
Created attachment 348980
--> https://bugzilla.gnome.org/attachment.cgi?id=348980&action=edit
tests: elements: add an example for roi

This implements a pipleint to recognize difference between ROI and non-ROI.
See comments in this code in detail.
--
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.
"GStreamer" (GNOME Bugzilla)
2017-03-30 09:28:23 UTC
Reply
Permalink
Raw Message
https://bugzilla.gnome.org/show_bug.cgi?id=768248

--- Comment #32 from Hyunjun Ko <***@igalia.com> ---
Even though roi is broken on the current driver,
I would request review for a way to enable/disable roi.
--
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.
"GStreamer" (GNOME Bugzilla)
2017-03-30 09:29:51 UTC
Reply
Permalink
Raw Message
https://bugzilla.gnome.org/show_bug.cgi?id=768248

--- Comment #33 from Hyunjun Ko <***@igalia.com> ---
(In reply to Hyunjun Ko from comment #31)
Created attachment 348980 [details] [review]
tests: elements: add an example for roi
This implements a pipleint to recognize difference between ROI and non-ROI.
See comments in this code in detail.
I've been thinking about how to test and verify roi for a while.
This is an example, but if there's a better way, please let me know.
--
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.
"GStreamer" (GNOME Bugzilla)
2017-04-07 15:45:07 UTC
Reply
Permalink
Raw Message
https://bugzilla.gnome.org/show_bug.cgi?id=768248

Víctor Manuel Jáquez Leal <***@igalia.com> changed:

What |Removed |Added
----------------------------------------------------------------------------
Attachment #348978|none |reviewed
status| |

--- Comment #34 from Víctor Manuel Jáquez Leal <***@igalia.com> ---
Review of attachment 348978:
--> (https://bugzilla.gnome.org/review?bug=768248&attachment=348978)

this patch needs too be rebased because gstvaapiencoder.h has changed. But
still lgtm
--
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.
"GStreamer" (GNOME Bugzilla)
2017-04-07 15:56:14 UTC
Reply
Permalink
Raw Message
https://bugzilla.gnome.org/show_bug.cgi?id=768248

Víctor Manuel Jáquez Leal <***@igalia.com> changed:

What |Removed |Added
----------------------------------------------------------------------------
Attachment #348979|none |needs-work
status| |

--- Comment #35 from Víctor Manuel Jáquez Leal <***@igalia.com> ---
Review of attachment 348979:
--> (https://bugzilla.gnome.org/review?bug=768248&attachment=348979)

::: gst/vaapi/gstvaapiencode.c
@@ +738,3 @@
(GstTaskFunction) gst_vaapiencode_buffer_loop, encode, NULL);
break;
+ case GST_EVENT_CUSTOM_DOWNSTREAM:

I wonder if this event should be GST_EVENT_CUSTOM_DOWNSTREAM_OOB (out of band)

@@ +741,3 @@
+ {
+ const GstStructure *s = gst_event_get_structure (event);
+ if (gst_structure_has_name (s, "GstVaapiEncoderRegionOfInterest")) {

I don't know if GstVaapiEncoderRegionOfInterest is a good name. Starting with
the namespace. I am not sure if other encoders could enable ROI.

We need to discuss it with the rest of the community. IMO it should be
something more like "region-of-interest"
--
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.
"GStreamer" (GNOME Bugzilla)
2017-04-07 15:56:39 UTC
Reply
Permalink
Raw Message
https://bugzilla.gnome.org/show_bug.cgi?id=768248

Víctor Manuel Jáquez Leal <***@igalia.com> changed:

What |Removed |Added
----------------------------------------------------------------------------
Attachment #348980|none |reviewed
status| |

--- Comment #36 from Víctor Manuel Jáquez Leal <***@igalia.com> ---
Review of attachment 348980:
--> (https://bugzilla.gnome.org/review?bug=768248&attachment=348980)

This is beautiful! Great!
--
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.
"GStreamer" (GNOME Bugzilla)
2017-04-25 09:45:38 UTC
Reply
Permalink
Raw Message
https://bugzilla.gnome.org/show_bug.cgi?id=768248

--- Comment #37 from Víctor Manuel Jáquez Leal <***@igalia.com> ---
I don't know if this is a good timing to merge these patches, since we are in
the code freeze window, and the current version of Intel's driver (the only one
supporting this feature) has a regression on it (bug #780881).

Perhaps we might delay it until release 1.13/1.14 :S
--
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.
"GStreamer" (GNOME Bugzilla)
2017-05-10 14:43:14 UTC
Reply
Permalink
Raw Message
https://bugzilla.gnome.org/show_bug.cgi?id=768248

--- Comment #38 from Víctor Manuel Jáquez Leal <***@igalia.com> ---
I'm going to push these patches now.

Sree has some doubts about the usage of the GstEvent API for this. He thinks we
should find a way to emit it at a specific frame.
--
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.
"GStreamer" (GNOME Bugzilla)
2017-05-11 16:55:44 UTC
Reply
Permalink
Raw Message
https://bugzilla.gnome.org/show_bug.cgi?id=768248

--- Comment #39 from Víctor Manuel Jáquez Leal <***@igalia.com> ---
Review of attachment 346550:
--> (https://bugzilla.gnome.org/review?bug=768248&attachment=346550)

::: gst-libs/gst/vaapi/gstvaapiencoder_priv.h
@@ +231,3 @@
+ /* Region of Interest */
+ gboolean got_roi_capability;
+ gboolean roi_capability;

I feel these two variables are not required at all. The roi_capability is
repeated in the GstVaapiConfigInfoEncoder, and got_roi_capability doesn't
change much, since the query is fast, there's no need of cache it.
--
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.
"GStreamer" (GNOME Bugzilla)
2017-05-11 16:57:27 UTC
Reply
Permalink
Raw Message
https://bugzilla.gnome.org/show_bug.cgi?id=768248

--- Comment #40 from Víctor Manuel Jáquez Leal <***@igalia.com> ---
Review of attachment 346552:
--> (https://bugzilla.gnome.org/review?bug=768248&attachment=346552)

::: gst-libs/gst/vaapi/gstvaapiencoder_h264.c
@@ +2270,3 @@
+ VAEncMiscParameterBufferROI *roi_param;
+ VAEncROI *region_roi, *region_ptr;
+ gint roi_num = g_list_length (base_encoder->roi_regions);

we are repeting twice the call g_list_length(), it would be better if we just
call it once, avoiding traversing the list for the same size.
--
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.
"GStreamer" (GNOME Bugzilla)
2017-05-11 16:59:14 UTC
Reply
Permalink
Raw Message
https://bugzilla.gnome.org/show_bug.cgi?id=768248

--- Comment #41 from Víctor Manuel Jáquez Leal <***@igalia.com> ---
Review of attachment 346553:
--> (https://bugzilla.gnome.org/review?bug=768248&attachment=346553)

::: tests/simple-encoder.c
@@ +34,3 @@
static char **g_input_files = NULL;
+static guint g_roi_enable = 0;
+static GstVaapiROI *g_roi_region[2] = { NULL, };

There's no need to keep this global, it could bie in the App structure.

@@ +191,3 @@
+
+ for (i = 0; i < 2; i++) {
+ g_roi_region[i] = g_malloc0 (sizeof (GstVaapiROI));

we can save this malloc&free using normal array declared in App structure.
--
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.
"GStreamer" (GNOME Bugzilla)
2017-05-11 17:00:51 UTC
Reply
Permalink
Raw Message
https://bugzilla.gnome.org/show_bug.cgi?id=768248

--- Comment #42 from Víctor Manuel Jáquez Leal <***@igalia.com> ---
Review of attachment 348978:
--> (https://bugzilla.gnome.org/review?bug=768248&attachment=348978)

::: gst-libs/gst/vaapi/gstvaapiencoder.c
@@ +1138,3 @@

+ if (encoder->roi_regions)
+ g_list_free (encoder->roi_regions);

here's a memory leak, since the allocated data are not freed.

@@ +1307,3 @@
+ region->rect.width == roi->rect.width &&
+ region->rect.height == roi->rect.height) {
+ encoder->roi_regions = g_list_remove (encoder->roi_regions, region);

here's a memory leak: we are not freeing the memory allocated for the
GstVaapiROO in _add_roi()
--
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.
"GStreamer" (GNOME Bugzilla)
2017-05-11 17:03:02 UTC
Reply
Permalink
Raw Message
https://bugzilla.gnome.org/show_bug.cgi?id=768248

--- Comment #43 from Víctor Manuel Jáquez Leal <***@igalia.com> ---
Review of attachment 348980:
--> (https://bugzilla.gnome.org/review?bug=768248&attachment=348980)

::: tests/elements/test-roi.c
@@ +147,3 @@
+ " t. ! queue name=second_q ! " CODEC
+ " ! " VIDEO_ELEMENTS "!" TEXTOVERLAY "ROI "
+ " ! videobox border-alpha=0 left=-640 ! mix.", NULL);

this pipeline can be simplified a lot, avoiding many of those videoconverts,
and since it is only taking the snow part of videotestsrc, we can ask it to
generate only the snow pattern: pattern=snow
Thus we avoid the extract of that area.
--
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.
"GStreamer" (GNOME Bugzilla)
2017-05-11 17:06:26 UTC
Reply
Permalink
Raw Message
https://bugzilla.gnome.org/show_bug.cgi?id=768248

--- Comment #44 from Víctor Manuel Jáquez Leal <***@igalia.com> ---
Review of attachment 348979:
--> (https://bugzilla.gnome.org/review?bug=768248&attachment=348979)

::: gst/vaapi/gstvaapiencode.c
@@ +767,3 @@
+ }
+ }
+ break;

I would move this above the call to the parent, since this call only is handled
by us.
--
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.
"GStreamer" (GNOME Bugzilla)
2017-05-11 17:24:28 UTC
Reply
Permalink
Raw Message
https://bugzilla.gnome.org/show_bug.cgi?id=768248

Víctor Manuel Jáquez Leal <***@igalia.com> changed:

What |Removed |Added
----------------------------------------------------------------------------
Attachment #346550|reviewed |none
status| |
Attachment #346550|0 |1
is obsolete| |

--- Comment #45 from Víctor Manuel Jáquez Leal <***@igalia.com> ---
Created attachment 351651
--> https://bugzilla.gnome.org/attachment.cgi?id=351651&action=edit
libs: encoder/context: query region of interest support

Queries if the driver supports "Region of Interest" (ROI) during the config
creation.

This attribute conveys whether the driver supports region-of-interest (ROI)
encoding, based on user provided ROI rectangles. The attribute value is
partitioned into fields as defined in the VAConfigAttribValEncROI union.

If ROI encoding is supported, the ROI information is passed to the driver
using VAEncMiscParameterTypeROI.

https://bugzilla.gnome.org/show_bug.cgi?id=768248

Signed-off-by: Víctor Manuel Jáquez Leal <***@igalia.com>
--
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.
"GStreamer" (GNOME Bugzilla)
2017-05-11 17:24:41 UTC
Reply
Permalink
Raw Message
https://bugzilla.gnome.org/show_bug.cgi?id=768248

Víctor Manuel Jáquez Leal <***@igalia.com> changed:

What |Removed |Added
----------------------------------------------------------------------------
Attachment #346552|reviewed |none
status| |
Attachment #346552|0 |1
is obsolete| |

--- Comment #47 from Víctor Manuel Jáquez Leal <***@igalia.com> ---
Created attachment 351653
--> https://bugzilla.gnome.org/attachment.cgi?id=351653&action=edit
libs: encoder: h264: set ROI params during encoding

Set ROI params during encoding each frame, which are set via
gst_vaapi_encoder_add_roi ()

https://bugzilla.gnome.org/show_bug.cgi?id=768248

Signed-off-by: Víctor Manuel Jáquez Leal <***@igalia.com>
--
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.
"GStreamer" (GNOME Bugzilla)
2017-05-11 17:24:34 UTC
Reply
Permalink
Raw Message
https://bugzilla.gnome.org/show_bug.cgi?id=768248

Víctor Manuel Jáquez Leal <***@igalia.com> changed:

What |Removed |Added
----------------------------------------------------------------------------
Attachment #348978|reviewed |none
status| |
Attachment #348978|0 |1
is obsolete| |

--- Comment #46 from Víctor Manuel Jáquez Leal <***@igalia.com> ---
Created attachment 351652
--> https://bugzilla.gnome.org/attachment.cgi?id=351652&action=edit
libs: encoder: add api gst_vaapi_encoder_add/del_roi

Implements and exposes new api gst_vaapi_encoder_add/del_roi to set ROI
regions.
--
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.
"GStreamer" (GNOME Bugzilla)
2017-05-11 17:24:54 UTC
Reply
Permalink
Raw Message
https://bugzilla.gnome.org/show_bug.cgi?id=768248

--- Comment #49 from Víctor Manuel Jáquez Leal <***@igalia.com> ---
Created attachment 351655
--> https://bugzilla.gnome.org/attachment.cgi?id=351655&action=edit
vaapiencode: handle custom event GstVaapiEncoderRegionOfInterest

Handles new custom event GstVaapiEncoderRegionOfInterest
to enable/disable a ROI region.

Writes a way to use new event to document.

https://bugzilla.gnome.org/show_bug.cgi?id=768248

Signed-off-by: Víctor Manuel Jáquez Leal <***@igalia.com>
--
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.
"GStreamer" (GNOME Bugzilla)
2017-05-11 17:24:47 UTC
Reply
Permalink
Raw Message
https://bugzilla.gnome.org/show_bug.cgi?id=768248

Víctor Manuel Jáquez Leal <***@igalia.com> changed:

What |Removed |Added
----------------------------------------------------------------------------
Attachment #346553|reviewed |none
status| |
Attachment #346553|0 |1
is obsolete| |

--- Comment #48 from Víctor Manuel Jáquez Leal <***@igalia.com> ---
Created attachment 351654
--> https://bugzilla.gnome.org/attachment.cgi?id=351654&action=edit
tests: simple-encoder: add an option to set ROI

$ simple-encoder -r inputfile.y4m

And you'll got an output file in H264 with two regions of interest.

https://bugzilla.gnome.org/show_bug.cgi?id=768248

Signed-off-by: Víctor Manuel Jáquez Leal <***@igalia.com>
--
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.
"GStreamer" (GNOME Bugzilla)
2017-05-11 17:25:01 UTC
Reply
Permalink
Raw Message
https://bugzilla.gnome.org/show_bug.cgi?id=768248

--- Comment #50 from Víctor Manuel Jáquez Leal <***@igalia.com> ---
Created attachment 351656
--> https://bugzilla.gnome.org/attachment.cgi?id=351656&action=edit
tests: elements: add an example for ROI

This implements a pipleint to recognize difference between ROI and non-ROI.
See comments in this code in detail.

https://bugzilla.gnome.org/show_bug.cgi?id=768248

Signed-off-by: Víctor Manuel Jáquez Leal <***@igalia.com>
--
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.
"GStreamer" (GNOME Bugzilla)
2017-05-11 17:25:34 UTC
Reply
Permalink
Raw Message
https://bugzilla.gnome.org/show_bug.cgi?id=768248

Víctor Manuel Jáquez Leal <***@igalia.com> changed:

What |Removed |Added
----------------------------------------------------------------------------
Attachment #348980|0 |1
is obsolete| |
--
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.
"GStreamer" (GNOME Bugzilla)
2017-05-11 17:25:48 UTC
Reply
Permalink
Raw Message
https://bugzilla.gnome.org/show_bug.cgi?id=768248

Víctor Manuel Jáquez Leal <***@igalia.com> changed:

What |Removed |Added
----------------------------------------------------------------------------
Attachment #348979|0 |1
is obsolete| |
--
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.
"GStreamer" (GNOME Bugzilla)
2017-05-11 17:30:20 UTC
Reply
Permalink
Raw Message
https://bugzilla.gnome.org/show_bug.cgi?id=768248

--- Comment #51 from Víctor Manuel Jáquez Leal <***@igalia.com> ---
@Hyunjun: I'll push these patches tomorrow. Please check the modifications I
did.
--
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.
"GStreamer" (GNOME Bugzilla)
2017-05-12 06:03:43 UTC
Reply
Permalink
Raw Message
https://bugzilla.gnome.org/show_bug.cgi?id=768248

--- Comment #52 from Hyunjun Ko <***@igalia.com> ---
Review of attachment 351652:
--> (https://bugzilla.gnome.org/review?bug=768248&attachment=351652)

::: gst-libs/gst/vaapi/gstvaapiencoder.c
@@ +1427,3 @@
+ /* Duplicated region */
+ goto end;
+ GstVaapiROI *region = NULL;

This break can be removed. Sorry for this stupid thing.
--
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.
"GStreamer" (GNOME Bugzilla)
2017-05-12 06:04:33 UTC
Reply
Permalink
Raw Message
https://bugzilla.gnome.org/show_bug.cgi?id=768248

--- Comment #53 from Hyunjun Ko <***@igalia.com> ---
Review of attachment 351655:
--> (https://bugzilla.gnome.org/review?bug=768248&attachment=351655)

::: gst/vaapi/gstvaapiencode.c
@@ +741,3 @@

+ switch (GST_EVENT_TYPE (event)) {
+ case GST_EVENT_CUSTOM_DOWNSTREAM:{

Maybe you want GST_EVENT_CUSTOM_DOWNSTREAM_OOB :)
--
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.
"GStreamer" (GNOME Bugzilla)
2017-05-12 06:25:57 UTC
Reply
Permalink
Raw Message
https://bugzilla.gnome.org/show_bug.cgi?id=768248

--- Comment #54 from Hyunjun Ko <***@igalia.com> ---
(In reply to Víctor Manuel Jáquez Leal from comment #51)
Post by "GStreamer" (GNOME Bugzilla)
@Hyunjun: I'll push these patches tomorrow. Please check the modifications I
did.
Thanks for supporting this work!
Please see my comments.
--
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.
"GStreamer" (GNOME Bugzilla)
2017-05-12 09:18:18 UTC
Reply
Permalink
Raw Message
https://bugzilla.gnome.org/show_bug.cgi?id=768248

Víctor Manuel Jáquez Leal <***@igalia.com> changed:

What |Removed |Added
----------------------------------------------------------------------------
Status|NEW |RESOLVED
Resolution|--- |FIXED

--- Comment #55 from Víctor Manuel Jáquez Leal <***@igalia.com> ---
Attachment 351651 pushed as b41d72b - libs: encoder/context: query region of
interest support
Attachment 351652 pushed as 7a6f690 - libs: encoder: add api
gst_vaapi_encoder_add/del_roi
Attachment 351653 pushed as f3302a0 - libs: encoder: h264: set ROI params
during encoding
Attachment 351654 pushed as c21345c - tests: simple-encoder: add an option to
set ROI
Attachment 351655 pushed as 8f1b88d - vaapiencode: handle custom event
GstVaapiEncoderRegionOfInterest
Attachment 351656 pushed as eb17b71 - tests: elements: add an example for ROI
--
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.
"GStreamer" (GNOME Bugzilla)
2017-05-12 09:18:24 UTC
Reply
Permalink
Raw Message
https://bugzilla.gnome.org/show_bug.cgi?id=768248

Víctor Manuel Jáquez Leal <***@igalia.com> changed:

What |Removed |Added
----------------------------------------------------------------------------
Attachment #351651|none |committed
status| |
--
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.
"GStreamer" (GNOME Bugzilla)
2017-05-12 09:18:29 UTC
Reply
Permalink
Raw Message
https://bugzilla.gnome.org/show_bug.cgi?id=768248

Víctor Manuel Jáquez Leal <***@igalia.com> changed:

What |Removed |Added
----------------------------------------------------------------------------
Attachment #351652|none |committed
status| |
--
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.
"GStreamer" (GNOME Bugzilla)
2017-05-12 09:18:34 UTC
Reply
Permalink
Raw Message
https://bugzilla.gnome.org/show_bug.cgi?id=768248

Víctor Manuel Jáquez Leal <***@igalia.com> changed:

What |Removed |Added
----------------------------------------------------------------------------
Attachment #351653|none |committed
status| |
--
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.
"GStreamer" (GNOME Bugzilla)
2017-05-12 09:18:39 UTC
Reply
Permalink
Raw Message
https://bugzilla.gnome.org/show_bug.cgi?id=768248

Víctor Manuel Jáquez Leal <***@igalia.com> changed:

What |Removed |Added
----------------------------------------------------------------------------
Attachment #351654|none |committed
status| |
--
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.
"GStreamer" (GNOME Bugzilla)
2017-05-12 09:18:47 UTC
Reply
Permalink
Raw Message
https://bugzilla.gnome.org/show_bug.cgi?id=768248

Víctor Manuel Jáquez Leal <***@igalia.com> changed:

What |Removed |Added
----------------------------------------------------------------------------
Attachment #351656|none |committed
status| |
--
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.
"GStreamer" (GNOME Bugzilla)
2017-05-12 09:18:43 UTC
Reply
Permalink
Raw Message
https://bugzilla.gnome.org/show_bug.cgi?id=768248

Víctor Manuel Jáquez Leal <***@igalia.com> changed:

What |Removed |Added
----------------------------------------------------------------------------
Attachment #351655|none |committed
status| |
--
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.
"GStreamer" (GNOME Bugzilla)
2017-05-12 09:21:40 UTC
Reply
Permalink
Raw Message
https://bugzilla.gnome.org/show_bug.cgi?id=768248

Víctor Manuel Jáquez Leal <***@igalia.com> changed:

What |Removed |Added
----------------------------------------------------------------------------
Target Milestone|git master |1.13.1
--
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.
Loading...