search

turingmachine / omxplayer-sync

OMXPlayer-Sync facilitates synchronization of multiple OMXPlayer instances over the network in a master/slave fashion.
194 stars 70 forks source link

Stats/OMXMediaTime Issue #1

Closed Pro-Use closed 10 years ago

Pro-Use commented 11 years ago

Sorry if this isn't the place for this question but it's driving me crazy so I thought I'd ask:

Using OMXPlayer b853c39 (popcornmix) the 'v:' stats bear no relation to the video position and do not increase. Does this script require a particular fork of OMXPlayer or video encoding?

Thanks

turingmachine commented 11 years ago

At the moment the solution works best with my custom fork of omxplayer (https://github.com/turingmachine/omxplayer/tree/fixes). I tried to push the changes in there upstream. They were rejected because the things I fixed in there should apparently be fixed in the new_stall branch in upstream. I've tried this 'new_stall' branch, the stats output is completely broken and also seeking behaviour is much worse.

So i decied to give it some time and work with my own build of omxplayer for now (latest commit of fixes branch). You can find a build here: http://yokto.magdesign.ch.

turingmachine commented 11 years ago

That was the pull request: https://github.com/huceke/omxplayer/pull/169

Pro-Use commented 11 years ago

Hi Simon,

Thanks so much for sending this over, I'm very new to this kind of thing so didn't see the fork on your github - the compiled version is incredibly useful though.

I did manage to get something working using your code but modified it to use the the fast-forward/rewind actions and monitor the packets sent to the encoder (which was the only stat that seemed to make any sense). This works surprisingly well for a short video but I think it might have issues with something longer. I also think I'd need to adjust the tolerance for every different file played so it will be great to have working video position stats instead!

Thanks again, Rob

On Mon, Jul 8, 2013 at 7:48 PM, Simon Josi notifications@github.com wrote:

That was the pull request: huceke/omxplayer#169https://github.com/huceke/omxplayer/issues/169

— Reply to this email directly or view it on GitHubhttps://github.com/turingmachine/omxplayer-sync/issues/1#issuecomment-20627267 .

mxbrt commented 11 years ago

Hi Pro-Use,

Would you mind uploading your modified code? I am stuck with the same problems you had.

Thanks