Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

pvr: display timeshift information to user #3319

Closed
wants to merge 4 commits into from

Conversation

FernetMenta
Copy link
Contributor

  • add required functions to the api
  • actual playing time is used for progress bar
  • would be nice if this information could be added to skins, maybe as an additional progress bar plus the values
  • corresponding pr to addons repo will follow

@ghost ghost assigned opdenkamp Sep 23, 2013
@da-anda
Copy link
Member

da-anda commented Sep 23, 2013

do we need an additional progress bar for it as you wrote in the PR description? We already can indicate the buffer status e.g. on web-streams and we could use the same for the timeshift buffer I think.

if (GetPlayingClient(client))
{
return client->GetPlayingTime();
}

This comment was marked as spam.

@FernetMenta
Copy link
Contributor Author

there has been a discussion on the forum: http://forum.xbmc.org/showthread.php?tid=151970
maybe we can do it similar to mediaportal (see link in thread)
I would like the current position displayed as a value with seconds.
Also start/end times and title of EPG should be selected by current playing time.

@da-anda
Copy link
Member

da-anda commented Sep 23, 2013

I use MediaPortal every day for watching TV, so I know it very well (I won't/can't switch to XBMC for LiveTV until certain issues in XBMC and the addon I use are resolved).
But all I wanted to say above was that there should not be the need for a new GUI element (c++), if really needed extend/fix current progress bar.

@@ -576,6 +576,22 @@
void SetSpeed(int speed);

/*!
* Get actual playing time from addon. With timeshift enabled this is
* different to live.

This comment was marked as spam.

@piotrasd
Copy link

yeep this will be good, like also good without stop watching back to actual time of LiveTV, for now we must stop and play this channel again :/ or switch and back.

If we watch Timeshift - second presed Play button should back inmiditly to LiveTV

@janbar
Copy link
Contributor

janbar commented Oct 16, 2013

Hi, no way to implement these new functions with addons does'nt handle demuxing. Time or frame info are handle by internal demuxer (ffmpeg). Could be an other way to manage it in XBMC itself ?

@opdenkamp
Copy link
Member

that would have to be handled differently because those use file operations.

@FernetMenta
Copy link
Contributor Author

In any case this must be handled by the addons/backends. XBMC itself knows nothing about the timeshift buffers or the offset to live.

@janbar
Copy link
Contributor

janbar commented Oct 19, 2013

I thought it was possible using pts of stream and epg start time. Else to provide time info i need to ask from the backend the number of frames already written and keep in a map at regular time. It is ugly and i don't know if all backend could provide it. Percentage could easily be provided, but there is not really related to the duration.

@opdenkamp
Copy link
Member

feel free to create a PR with an implementation of it :)

@janbar
Copy link
Contributor

janbar commented Oct 31, 2013

@opdenkamp @FernetMenta , yes addon can provide "GetBufferTimeStart" and "GetBufferTimeEnd". But "GetPlayingTime" could be provided by player itself and so avoid ugly code for addon don't handle demuxing. I thought using the start time and first PTS of live playback and current PTS. But it is possible only if XBMC can handle the PTS.
I haven't good skill with xbmc code but if any body can help and know member function to retrieve it, i could get time to do a PR

@FernetMenta
Copy link
Contributor Author

player can,t handle this because it does not know the offset between pts and current time. If you want player to display actual time, this has to implemented by the InputStream classes (which call the methods implemented here)

@janbar
Copy link
Contributor

janbar commented Nov 2, 2013

@FernetMenta , Thanks. I will have a look

@opdenkamp opdenkamp mentioned this pull request Nov 9, 2013
@opdenkamp
Copy link
Member

replaced by #3604

@opdenkamp opdenkamp closed this Nov 9, 2013
@piotrasd
Copy link

piotrasd commented Nov 9, 2013

Please guys looks on my proposition of timeshift bar i skin - thanks
http://forum.xbmc.org/showthread.php?tid=177613

@t-nelson
Copy link
Contributor

t-nelson commented Nov 9, 2013

@piotrasd, if you don't have patches, keep it on the forum. Every github
comment spams a hundred or so developers. Thanks.

@FernetMenta FernetMenta deleted the timeshift branch March 23, 2014 08:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants