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

feat(video): Adds ability to enable & retrieve HEVC profiles #174

Merged
merged 5 commits into from
Oct 10, 2017

Conversation

asciidisco
Copy link
Owner

I urge @trummerjo & @liberty-developer to have a look on this, as I have no experience with HEVC profiles & don't know if all of them should be enabled.

serialized_data = '[{},{"headers":{},"path":"/cbp/cadmium-13","payload":{"data":"' + serialized_data + '"},"query":""}]\n'
serialized_data = json.dumps(data)
serialized_data = serialized_data.replace('"', '\\"')
serialized_data = '[{},{"headers":{},"path":"/cbp/cadmium-13'"payload":{"data":"' + serialized_data + '"},"query":""}]\n'
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"path":"/cbp/cadmium-13", "payload"?

@asciidisco
Copy link
Owner Author

@trummerjo Silly me committed to fast. Issues have been resolved.

@trummerjo
Copy link
Collaborator

regarding the profiles: i dont know either which ones are needed. For now it should be finde to enable all of them

@im85288
Copy link

im85288 commented Oct 10, 2017

Just wanted to provide my feedback having just tested this on android shield tv.

Initially I thought it was not working correctly as the resolution was always either 1080p or 720p, I then went into InputStream settings and changed the stream selection to manual. With this activated playing an episode initially started in a really low resolution but I can go into the video settings and then select the highest quality 4k stream. Once that is selected it plays flawlessly. I do however have one question and that is with regards HDR. I used an episode of Glow as a trial and in the native Netflix it plays with HDR but in kodi it is missing. Is that something that may be supported in the future.

I think once auto select of the highest 4k stream can be done and HDR is added it's feature complete.

@trummerjo
Copy link
Collaborator

First of all: Great to hear that it is working with 4K streams on Android!
The problem that the HDR stream ist not displayed could have more reasons:

  1. We generate a wrong ESN for the netflix API and netflix does not provide the HDR Streams in the manifest
  2. We don't have the right profiles and the netflix API does not providehe HDR Streams in the manifest

I would bet it is reason 2).

The problem is: It is very hard to find out the correct profiles. The profiles now included are taken from Microsoft's Browser Edge. Android may have diffrent ones. For now I dont have an idea how to find them out. Decomiling the APK and digging in the code may be the best approach.

@asciidisco
Copy link
Owner Author

@im85288 Thanks for testing it (and being extremely quick in doing so).
I´ll try to investigate (or poke @peak3d to give it a shot) why the best possible profile isn't chosen. In the meantime, to make this feature available, we should add a note to the README about this, so that people know that they should use the manual stream selection.

@asciidisco
Copy link
Owner Author

Updated README & corresponding issue

@asciidisco asciidisco merged commit d31166d into master Oct 10, 2017
@im85288
Copy link

im85288 commented Oct 10, 2017

No problem at all thanks for adding it, very cool to have 4k streams working.

I did some googling and it looks like HDR is coming to Microsoft Edge with the October 17th release of windows 10 fall creators update. The only link I could find was in german (scroll down quite a bit of whitespace at the top): https://translate.google.co.uk/translate?hl=en&sl=de&u=https://www.heise.de/newsticker/meldung/Nvidia-Netflix-Streaming-in-4K-und-HDR-moeglich-ab-Windows-10-Fall-Creators-Update-3842435.html&prev=search
Hopefully when that comes out the new profiles can be taken from the edge browser which would save digging into the APK

@asciidisco asciidisco deleted the feat/enable-hevc-for-android branch October 10, 2017 18:25
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.

3 participants