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

[FR - MYCANAL] force 640p in DRM for devices not certified #1192

Closed

Conversation

darodi
Copy link
Contributor

@darodi darodi commented Aug 7, 2023

[FR - MYCANAL] force 640p in DRM for devices not certified

@darodi
Copy link
Contributor Author

darodi commented Aug 7, 2023

@joaopa00
Copy link
Contributor

joaopa00 commented Aug 8, 2023

Je ne sais pas si je suis le seul, mais j'utilise sur un appareil une version patchée de Linux qui me permet d'avoir la certif L1 et donc la HD ( Widevine L1 sur Linux )
Avec ce type de patch, je pense que je vais perdre la HD. Je ne sais pas si je suis le seul à patcher Linux et donc le seul à être pénaliser.

@darodi
Copy link
Contributor Author

darodi commented Aug 8, 2023

@joaopa00
Merci pour l'info, très intéressant! 😲
Donc on peut regarder >640p sur un Linux et alors que ça ne fonctionne pas sous windows! C'est fou! 😂
Je regarde si on peut repérer qu'on a la certif L1 d'un manière ou d'une autre.

@darodi
Copy link
Contributor Author

darodi commented Aug 8, 2023

sans ce PR et sans notifier les utilisateurs de la limitation, on risque d'avoir beaucoup d'issues inutiles.

@darodi darodi closed this Aug 8, 2023
@darodi
Copy link
Contributor Author

darodi commented Aug 8, 2023

@joaopa00

Sous linux, la qualité maximale est 960x540 (1499 kbs). La video et le son fonctionnent normalement pour cette qualité
Originally posted by @joaopa00 in #1185 (comment)

Pour ce test, c'était un appareil non patché?

@joaopa00
Copy link
Contributor

joaopa00 commented Aug 8, 2023

En effet, pour developper pour CUTVM, je n'utilise que les versions officiels des OS.
Et je suis d'accord avec ta remarque sur le nombre de rapports de bugs qui risque de fleurir s'il n'y a pas de notification pour les utilisateurs lambda. Peux-tu réouvrir cette PR? Je vais la merger. Tant pis pour moi 🤣

@darodi
Copy link
Contributor Author

darodi commented Aug 8, 2023

on peut ajouter un petit switch dans la config "my device is L1 certified!" en mode expert 😁

@darodi darodi reopened this Aug 8, 2023
@darodi darodi marked this pull request as draft August 8, 2023 21:46
@darodi
Copy link
Contributor Author

darodi commented Aug 8, 2023

je ferme, voir #1194

@darodi darodi closed this Aug 8, 2023
@darodi darodi deleted the mycanal-force_drm_quality branch August 18, 2023 10:14
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

Successfully merging this pull request may close these issues.

2 participants