Possible addition to video/Movie class

tezzac73

Possible addition to video/Movie class

in Library and Tool Development • 1 year ago

Hi all,

Loving the work so far on 2.0. I have a need for the Movie class to be able to handle RTSP streams (and probably later, either RTP or raw UDP) so in the spirit of Processing I have changed my local copy. I have tested with a local Darwin Streaming instance and everything seems good (except the occasional exception about unknown Enum mappings). My diff is pretty simple (against SVN 9709):

--- old/Movie.java 2012-05-31 13:23:04.843192312 +0100

+++ new/Movie.java 2012-06-02 06:22:05.972646443 +0100

@@ -683,7 +683,7 @@

}

// Network read...

- if (gplayer == null && filename.startsWith("http://")) {

+ if (gplayer == null && (filename.startsWith("http://") || filename.startsWith("rtp://") || filename.startsWith("rtsp://") || filename.startsWith("udp://"))) {

try {

PApplet.println("network read");

gplayer = new PlayBin2("Movie Player");

It probably needs more work before inclusion. My tiny changes so far are of course licensed under LGPL.

Replies(4)

andres

Re: Possible addition to video/Movie class

1 year ago

Thanks for your suggestion, I actually never tested rtp streams with PlayBin, good to know it works out of the box!

However, for the time being is better if we document it as a hack in the wiki. The problem with including stream reading support directly in the core video library is that streams often come with DRM protection, so this would lead to people reporting that a command like new Movie("rtp://hulu.com/...") is broken.

This could be part of a future update of GSVideo though, since we'd still like to have advanced functionality such as RTP support, but as a separate library.

tezzac73

Re: Possible addition to video/Movie class

1 year ago

Hi andres,

That's fine. I'll probably be working on some other hacks/"features" in the same library soon. Are the classes under video considered stable under CVS at the moment or am I likely to have to merge often?

All the best, Terry

andres

Re: Possible addition to video/Movie class

1 year ago

Hello, I did a minor rewrite of the network read code in the Movie class so it is easier to hack in support for rtsp and other types of streams. These changes are checked in r9717. There is a new static array listing the formats that the Movie class will attempt after local read fails. Recreating this array with the additional formats should work:

Movie.supportedProtocols = new String[] {"http", "rtsp"};

or redefining it in a subclass of Movie.

Are the classes under video considered stable under CVS at the moment or am I likely to have to merge often?

The classes are fairly stable at this point, there will be a couple of API changes in the next days, but nothing major.

tezzac73

Re: Possible addition to video/Movie class

1 year ago

Awesome! Thanks andres.

I'll probably be putting my new work into a separate library, especially at the moment, as it'll be dealing with GStreamer at the pipeline level.