Thread Rating:
  • 0 Vote(s) - 0 Average
API - deprecated features v18
#31
Does this mean that the client function GetLiveStreamURL is also going away?

It sounds like the PVR is now required to handle the network IO for reading remote streams, as this could be previously specified by strStreamURL.

Assuming that this is correct, are there calls into Kodi that I can make to handle the networking? The HDHomeRun PVR is now using strStreamURl to specify the stream, which is opened by the *DVD* reader.
Reply
#32
Partially answering my own question, it looks like I can create an object of type CDVDInputStreamFFmpeg, as in the function CDVDFactoryInputStream::CreateInputStream. Once I figure out how to build the CFileItem that it needs...

Can I call these functions from within the PVR code?
Reply
#33
(2017-06-06, 06:10)MatthewLundberg Wrote: Partially answering my own question, it looks like I can create an object of type CDVDInputStreamFFmpeg, as in the function CDVDFactoryInputStream::CreateInputStream. Once I figure out how to build the CFileItem that it needs...

Can I call these functions from within the PVR code?


Sorry, no. These classes/functions are not part of the official Kodi API, thus can change incompatible at any (!) random point in time, breaking your add-on.

You may want to take a look at kodi-platform, which provides stable basic functionality that can be used safely in kodi binary add-ons. => https://github.com/Pulse-Eight/platform
Reply
#34
Thanks, ksooo, I am looking there. It appears to have everything that I will need.

I did notice though, that the P8PLATFORM:TonguereventCopy class does not allow moves. Is this code being actively maintained?
Reply
#35
As far as I know, yes.

@opdenkamp might have more insights.
Reply
#36
Confirmed, it is easy to support ReadLiveStream, and I have done so in yet-to-be-pushed code. No more strStreamURL or ThisIsAHack calling GetLiveStreamURL. And maybe it's my imagination, but channel changes seem to be faster and smoother.

I do have a question about this though. In this thread, picture-in-picture is mentioned. However, the function for ReadLiveStream does not indicate a channel or any other handle to allow the PVR to supply multiple streams. Is picture-in-picture only supported for two instances of the PVR, or for the PVR and another stream (say, from a file)?
Reply
#37
(2017-06-07, 05:20)MatthewLundberg Wrote: Confirmed, it is easy to support ReadLiveStream, and I have done so in yet-to-be-pushed code. No more strStreamURL or ThisIsAHack calling GetLiveStreamURL. And maybe it's my imagination, but channel changes seem to be faster and smoother.

I do have a question about this though. In this thread, picture-in-picture is mentioned. However, the function for ReadLiveStream does not indicate a channel or any other handle to allow the PVR to supply multiple streams. Is picture-in-picture only supported for two instances of the PVR, or for the PVR and another stream (say, from a file)?


PIP is currently not supported at all. Larger changes in Kodi core and add-ons (including API) are required for this. We're working on this, but still a long way to go, no release date scheduled so far.
Reply
#38
(2017-06-07, 05:20)MatthewLundberg Wrote: Confirmed, it is easy to support ReadLiveStream, and I have done so in yet-to-be-pushed code. No more strStreamURL or ThisIsAHack calling GetLiveStreamURL. And maybe it's my imagination, but channel changes seem to be faster and smoother.

Do you know if that would also work for streams which should be handled by an inputstream addon (e.g. a DASH stream)?
Reply
#39
I don't know what a DASH stream is. It would probably work for the simple IPTV addon, on which the first version of the HDHomeRun PVR is based.

See here: https://github.com/MatthewLundberg/pvr.h...e5bdb440c0
Reply
#40
(2017-06-07, 14:19)MatthewLundberg Wrote: I don't know what a DASH stream is. It would probably work for the simple IPTV addon, on which the first version of the HDHomeRun PVR is based.

See here: https://github.com/MatthewLundberg/pvr.h...e5bdb440c0

DASH = MPEG-DASH - a streaming standard that allows dynamic bitrate switching whilst streaming. It's increasingly used by Catch Up TV and OTT operators.
Reply
#41
post #1 updated. SwitchChannel is deprecated: https://github.com/xbmc/xbmc/blob/master...dll.h#L418
Reply
#42
Please reconsider this. Dropping this will break many PVR addons like the Mediaportal PVR addon and the ArgusTV addon and I cannot fix them since I don't know how...
Removing deprecated functions is ok, but I don't see a viable alternative.

I understand that changing the path is a hack. I came up with this since my other proposal to add a return value to the OpenLiveStream function which indicates that Kodi should open an URL instead was rejected.
The GetLiveStreamURL() url is used to let the Kodi player play MediaPortals rtsp stream or IPTV stream or web radio URLs. I do us the OpenLiveStream() and OpenRecordedStream() for normal DVB streams but I'm not planning in any way to re-invent the wheel by extending the addon with logic to play streams that the Kodi player already handles.

So my question is what the new way is of playing RTSP streams, mp3 streams, and other webstream from a PVR addon? I haven't seen a solution for this yet...

W.r.t to the removal of SwitchChannel:
What is the new calling sequence for this?
Is this just another OpenLiveStream() call without closing the existing stream or is this a CloseLiveStream() followed by a new OpenLiveStream(). The latter will break my fast channel switching logic for RTSP streams since closing followed by opening will add more that 3 seconds to a channel switch.

I hope to see some good suggestions for my questions and some time for me to implement them before you remove the deprecated functions. One week is waaay tooo short.
Developer of the MediaPortal PVR addon and retired developer of the Argus-TV PVR-addon.
http://www.scintilla.utwente.nl/~marcelg/xbmc
Reply
#43
What's too short? V18 release date has not been set yet so there enough time for some one to fix it. During alpha I simply don't care if some Addons are broken. If it doesn't happen we'll then sorry for those who use those addons. They can stick with v17
Read/follow the forum rules.
For troubleshooting and bug reporting, read this first
Interested in seeing some YouTube videos about Kodi? Go here and subscribe
Reply
#44
Well the PR mentions "merge in 1 week if there are no objections". That is a bit short to prepare if I don't know yet how to fix the addon in a proper way Wink
Developer of the MediaPortal PVR addon and retired developer of the Argus-TV PVR-addon.
http://www.scintilla.utwente.nl/~marcelg/xbmc
Reply
#45
Well then in one week the addon is broken for alpha builds. I can't see the issue in that. After that there's probably some months left for someone to pick up whatever needs to be done.
Read/follow the forum rules.
For troubleshooting and bug reporting, read this first
Interested in seeing some YouTube videos about Kodi? Go here and subscribe
Reply



API - deprecated features v1800