Fix EXT-X-PROGRAM-DATE-TIME in stream#58036
Conversation
|
Hey there @hunterjm, @allenporter, mind taking a look at this pull request as it has been labeled with an integration ( |
allenporter
left a comment
There was a problem hiding this comment.
Apologies, i had a draft review that I forgot to send!
allenporter
left a comment
There was a problem hiding this comment.
Feel free to add tests as a follow up.
My streams currently are still displayed with relative zero based timestamps, and not an absolute time fwiw.
This actually led me down a small rabbit hole of revisiting the part boundary behavior and adjusting some minor tests. I will see if I can add them to this PR today. |
ab35feb to
c19a06e
Compare
|
@allenporter I made some updates and force pushed everything, so this needs another look. Edit: I just pushed a commit to reset the |
Proposed change
The extra
#EXT-X-PROGRAM-DATE-TIMEtags added to HLS playlists for LL-HLS were incorrect which may have caused issues with some players including Exoplayer. This PR fixes those tags.Type of change
Additional information
Checklist
black --fast homeassistant tests)If user exposed functionality or configuration variables are added/changed:
If the code communicates with devices, web services, or third-party tools:
Updated and included derived files by running:
python3 -m script.hassfest.requirements_all.txt.Updated by running
python3 -m script.gen_requirements_all..coveragerc.The integration reached or maintains the following Integration Quality Scale:
To help with the load of incoming pull requests: