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

HTSP protocol extensions #350

Open
perexg opened this issue Jan 15, 2018 · 3 comments
Open

HTSP protocol extensions #350

perexg opened this issue Jan 15, 2018 · 3 comments

Comments

@perexg
Copy link
Contributor

perexg commented Jan 15, 2018

I'd like to discuss some extensions to the HTSP protocol:

  • move to "large" UUIDs which tvh uses internally - 128-bits (channels, tags, DVR entries)
  • EPG events might have 64-bit id numbers instead 32-bit id's in future to avoid 'reuses'
  • direct MPEG-TS data transfers
  • use htsmsg binary2 format (smaller length) encoding - see current tvh's master - src/htsmsg_binary2.c
  • add gzip compression for binary htsmsg - this will help to reduce the MPEG-TS overhead and it's ideal for EPG data transfers
@Jalle19
Copy link
Contributor

Jalle19 commented Jan 15, 2018

Sounds great! What does direct MPEG-TS transfer mean in practice?

@ksooo
Copy link
Member

ksooo commented Jan 15, 2018

If it is possible to keep master pvr.hts work with both tvh 4.2 (stable) and tvh master, without cluttering pvr.hts code to much, +1 from my end.

@perexg
Copy link
Contributor Author

perexg commented Jan 16, 2018

The major incompatibilities are 128-bit UUIDs and 64-bit EPG IDs. Currently, TVH manages the 32-bit IDs only for the HTSP protocol. Unfortunately, it means that if I remove support for 32-bit IDs in tvh, the older clients won't work. I probably have to maintain some translation tables for older clients - not a big fun. Anyway, I'd like to put this change ASAP to the code (with upgraded API number) so you can add those changes to next pvr.hts release.

Passing MPEG-TS means that tvh will not change the input data at all (except for transcoding).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants