After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 736058 - [PATCH] adding one example to use gnlsource and gnlcomposition in gstreamer pipeline
[PATCH] adding one example to use gnlsource and gnlcomposition in gstreamer p...
Status: RESOLVED OBSOLETE
Product: GStreamer
Classification: Platform
Component: gnonlin
unspecified
Other Linux
: Normal enhancement
: git master
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2014-09-04 14:19 UTC by Mohan R
Modified: 2018-05-04 11:58 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
gstextractor.c (8.91 KB, patch)
2014-09-04 14:21 UTC, Mohan R
reviewed Details | Review
gstextractor.c updated (13.04 KB, patch)
2014-09-06 03:26 UTC, Mohan R
none Details | Review
testgdatetime.c (1.22 KB, text/x-csrc)
2014-09-06 03:32 UTC, Mohan R
  Details
testgstdatetime.c (1.09 KB, text/x-csrc)
2014-09-06 03:47 UTC, Mohan R
  Details

Description Mohan R 2014-09-04 14:19:29 UTC
This patch includes one example to use gnlsource and gnlcomposition in gstreamer pipeline.
Comment 1 Mohan R 2014-09-04 14:21:02 UTC
Created attachment 285383 [details] [review]
gstextractor.c
Comment 2 Nicolas Dufresne (ndufresne) 2014-09-04 15:02:23 UTC
Review of attachment 285383 [details] [review]:

This code look good in general. As an example it should probably be commented, what do you think ?

::: tests/examples/gstextractor.c
@@ +1,1 @@
+/*

Normally there should be copyright and licensing here. For an example you can make it as-is, it's your choice.

@@ +54,3 @@
+  g_strfreev (timev);
+  return (guint64) (time * GST_SECOND);
+}

Just wondering, but can't gst_date_time_new_from_iso8601_string() can be used here ? We do this kind of conversion in GST in general, I would be suprised if there is no utility for it.
Comment 3 Mohan R 2014-09-06 03:26:47 UTC
Created attachment 285547 [details] [review]
gstextractor.c updated

updated gstextractor.c with comments and license information.
Comment 4 Mohan R 2014-09-06 03:32:40 UTC
Created attachment 285548 [details]
testgdatetime.c
Comment 5 Mohan R 2014-09-06 03:47:08 UTC
I haved tested gst_date_time_new_from_iso8601_string(), Here is some observations. Test code attached(testgstdatetime.c)

1. gst_date_time_new_from_iso8601_string() expects date must be present (like YYYY-MM-DD).
2. There is no way to get combined seconds from gst_date_time_* api (HH * 60 * 60 + MM * 60 + SS). We have to take hours, minutes, seconds seperately and then do the calculation manually.

It will be helpful to developers if we have a function in gst_date_time_* api to directly get nanoseconds instead of developers doing the nanoseconds calculation.
Comment 6 Mohan R 2014-09-06 03:47:48 UTC
Created attachment 285549 [details]
testgstdatetime.c
Comment 7 Mohan R 2014-09-15 10:19:35 UTC
Any chance of getting this new patch reviewed?
Comment 8 Thibault Saunier 2014-12-15 16:22:29 UTC
GNL is now dead and the non linear video edition feature is now implement as part of NLE currently hidden inside the GStreamer editing services. The goal though is to stabilize the NLE implementation and API (which has an API pretty similar to what GNL used to have) and move it all in gst-plugins-base.

I keep that bug open as this example would be should be ported to NLE and added in -base when the move of NLE in there happens.
Comment 9 Thibault Saunier 2018-05-04 11:58:07 UTC
Thanks for the patch but no activity happened in the last 4 years and noone seem to care enough to port the example. Closing.