Catch-up-TV-and-More / plugin.video.catchuptvandmore

Replay, Live TV and websites videos addon for Kodi
https://catch-up-tv-and-more.github.io
GNU General Public License v2.0
284 stars 89 forks source link

Ajouter une option pour désactiver les extraits #62

Closed funtravel1208 closed 5 years ago

funtravel1208 commented 6 years ago

Bonjour,

Je viens de constater qu'il y a beaucoup de replays de chaines francaises qui affichent des replays de pas plus de 1-2 min, et meme des replays affichant la durée correcte de l'émission mais qui ne durent que 2 min... Exemples d'émissions: France 2 - Les pouvoirs extraordinaires du corps humain; Infrarouge et plein d'autres documentaires... Y a t'il un problème??? Merci d'avance pour votre aide...

wwark commented 6 years ago

Hello, Le plugin récupère les informations proposés sur le site du replay. Sur les replays France 2 nous avons des extraits et des émissions complètes. La durée n'est pas correcte dans tous les cas pour France TV. Nous allons regarder ce qui possible de faire sur la durée.

Voici le lien des vidéos pour Les pouvoirs extraordinaires du corps. https://www.france.tv/france-2/les-pouvoirs-extraordinaires-du-corps-humain/replay-videos/ J'ai regardé on a que des extraits et pas de nouvelles émissions depuis le 06/03/2018 dont la vidéo n'est plus disponible

Merci, Wwark

wwark commented 6 years ago

Hello, Je viens de commiter sur la branche de développement. L'ajout de l'information si c'est un extrait ou non et supprimer la durée pour les extraits.

J'espère que ce changement permettra de repondre au point remonté dans ce ticket. Merci, Wwark

funtravel1208 commented 6 years ago

Bonjour,

OK c’est une bonne idée. Je pense que les extraits n’apparaissaient pas avant. Pourriez-vous ajouter un dossier séparé pour les extraits? Personnellement je ne suis pas intéressée par les extraits et souhaiterait avoir accès facilement et directement aux émissions.

Merci beaucoup!!!!

Christine Sent from my amazing iPhone X !!!!

On May 6, 2018, at 3:37 AM, Marc Roy notifications@github.com wrote:

Hello, Je viens de commiter sur la branche de développement. L'ajout de l'information si c'est un extrait ou non et supprimer la durée pour les extraits.

J'espère que ce changement permettra de repondre au point remonté dans ce ticket. Merci, Wwark

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub, or mute the thread.

wwark commented 6 years ago

Bonjour, Oui c'est la nouvelle API de France TV (l'autre ne permettaient plus de mettre à jour les émisions) J'ai changé le titre et propose d'ajouter une option dans le plugin pour activer ou non les extraits.

Nous regardons ce point et fermerons le ticket quand ce sera implémenter dans le plugin.

Merci, Wwark

funtravel1208 commented 6 years ago

Bonjour Marc,

Je viens de tomber sur un nouveau problème de Catch Up TV. Tous les journaux télévisés de 2018 ont disparu... Merci de voir quel est le problème!

Christine Sent from my amazing iPhone X !!!!

On May 7, 2018, at 6:25 PM, Marc Roy notifications@github.com wrote:

Bonjour, Oui c'est la nouvelle API de France TV (l'autre ne permettaient plus de mettre à jour les émisions) J'ai changé le titre et propose d'ajouter une option dans le plugin pour activer ou non les extrais.

Nous regardons ce point et fermerons le ticket quand ce sera implémenter dans le plugin.

Merci, Wwark

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub, or mute the thread.

wwark commented 6 years ago

Bonjour Christine,

Sur le site de France.TV, ils semblent avoir des soucis https://www.france.tv/france-2/roland-garros/ Les vidéos datent de juin 2017. Certains menu n'existent plus dont les lives. Ils ont surement eu un problème. On va voir si ça se stabilise. Les replays pour les JT semblent ok par exemple (JT 20h30 dimanche 27 Mai) Je regarderai demain soir.

Bonne soirée,

funtravel1208 commented 6 years ago

Ok, merci pour ces infos!!!! En effet, ils doivent avoir des soucis techniques en ce moment! En tout cas, merci beaucoup pour cette app!!! Je suis une fan! :)

Christine Sent from my amazing iPhone X !!!!

On May 28, 2018, at 6:14 PM, Marc Roy notifications@github.com wrote:

Bonjour Christine,

Sur le site de France.TV, ils semblent avoir des soucis https://www.france.tv/france-2/roland-garros/ Les vidéos datent de juin 2017. Certains menu n'existent plus dont les lives. Ils ont surement eu un problème. On va voir pour que ça se stabilisent. Les replays pour les JT semblent ok par exemple (JT 20h30 dimanche 27 Mai) Je regarderai demain soir.

Bonne soirée,

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub, or mute the thread.

wwark commented 6 years ago

Merci c'est top d'avoir des fans :) On va pouvoir produire des teeshirts ^^ Sinon je pense que la majeur partie de la partie FranceTV fonctionnent correctement. Je checkerai ce weekend pour vérifier si on a encore des soucis.

funtravel1208 commented 5 years ago

Bonjour Marc,

Depuis ce matin, je n’ai plus accès au Live des chaînes françaises. J’ai un message d’erreur pour le Live France “UnknownTimeZoneError: cannot find any time zone configuration”

Merci de voir si vous pouvez faire quelquechose pour régler ce problème.

Bien cordialement,

Christine Sent from my amazing iPhone X !!!!

On May 29, 2018, at 9:44 AM, Marc Roy notifications@github.com wrote:

Merci c'est top d'avoir des fans :) On va pouvoir produire des teeshirts ^^ Sinon je pense que la majeur partie de la partie FranceTV fonctionnent correctement. Je checkerai ce weekend pour vérifier si on a encore des soucis.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub, or mute the thread.

Forage commented 5 years ago

Pourquoi inclure les extraits de toute façon ? Nous avons le moyen de les détecter, à mon avis ça sert à rien de les monter. Est-ce que quelqu'un est vraiment intéresser de les voire ?

wwark commented 5 years ago

Hello @Forage,

Dans l'API utilisé on remonte les replays et les extraits dans le même appel. Sur France TV ont a beaucoup d'extrait et certains programmes peuvent avoir les replay dans la deuxième page et que des extrais sur la première page. Si on masque les extraits l'utilisateurs pensera qu'il n'y a pas de replay et n'ira surement pas voir la deuxième page .... Il faudrait qu'on puisse trouver un appel pour remonter que les replays ...

La façon mytf1 est mieux organisé je trouve mais dépend du site .......

De plus sur FranceTV ils gardent aussi des vieux replays qui ne sont plus lisibles ... mais remonté par l'appel...

Il faudrait creuser un peu plus l'API pour voir si on pourrait limiter les éléments remontés sans perdre de contenus ...

Merci, wwark

Psychoses commented 5 years ago

https://github.com/Psychoses/plugin.video.catchuptvandmore/commit/0c26e06aad62d2f974f86053d327119d72759440

Bonjour wwark, tu en pense quoi ? c'est pushable en l'état ?

wwark commented 5 years ago

Bonjour @Psychoses, Merci ! Il faudrait faire un test de ce code avec RPI2 ou RPI3b+ pour voir si ce changement n'a pas impact trop important sur le temps de chargement. Ton test est fait sur quel environnement, quel est l'impact sur ton environnement ?

Encore Merci, Wwark

Psychoses commented 5 years ago

Je n'ai pas de rpi à ma dispo.

J'ai push la modif dans mon environnment sur windows, mibox3ab et tv tcl (armv7) et snapdragon625 (aarch64) depuis quelques semaines.

Difficile à dire, je n'utilise pas assez la fonctionalité et j'ai beaucoup d'autres addons gourmands, je pense pas que ca impacte bcp les perfs mais je peux me tromper. J'ai rien remarquer pour le moment.

delai entre clic et affichage du contenu (envoyespecial, cashinvestigation, fortboyard, etc...) : mido snapdragon625 : 3s mibox3ab tvtcl (S905 A53/MT5891) : 5s sur windows (core2duo) : quasi instantannée

wwark commented 5 years ago

Je pense avoir trouver la modif de l'API 😁 il faut rajouter ",only-replay" dans tous les params d'appel

Psychoses commented 5 years ago

Je viens de tester ",only-replay" a priori ca fonctionne, mais quand on regarde bien cela squizz quelques vidéos. par exemple pour cash_investigation toutes les vidéos ou il y a par exemple "S5-E3" disparaisse.

listes des vidéos qui disparraissent avec la méthode ",only-replay":

01:23.804 T:3344 NOTICE: Coton : l'envers de nos tee-shirts 01:23.874 T:3344 NOTICE: Travail, ton univers impitoyable 01:23.877 T:3344 NOTICE: Pédophilie dans l'Eglise : le poids du silence 01:23.880 T:3344 NOTICE: Razzia sur le bois : les promesses en kit des géants du meuble 01:23.897 T:3344 NOTICE: Climat : le grand bluff des multinationales 29:54.832 T:4776 NOTICE: Le business de la peur 29:54.849 T:4776 NOTICE: Santé : la loi du marché 29:54.853 T:4776 NOTICE: Mon président est en voyage d'affaires

J'ai essayé d'investiger sans grand succés

wwark commented 5 years ago

oui on a trois types de videos (episode, extrait et integrale) only-replay remonte juste les videos de type integrale mais pas de types episode ... only-extract remonte juste les extraits.

@SylvainCecchetto est-ce que tu aurais la liste de tous les filters possibles? http://api-front.yatta.francetv.fr/standard/publish/taxonomies/france-2_cash-investigation/contents/?size=200&page=0&sort=begin_date:desc&filter=with-no-vod,only-visible,only-replay

wwark commented 5 years ago

@Psychoses, j'ai réfléchis. On pourrait ajouter deux contextes menus (dans liste cf capture d'écran) pour lister les extraits (only-extrait) et les videos de types episode (ton code adapté pour récupérer toutes les vidéos (5000 size) et afficher que les videos de type episodes) et mettre les vidéos de type Integral par défaut (only-replay). Screenshot from 2019-09-13 19-31-57

Qu'est ce que tu penses ?

wwark commented 5 years ago

Proposition: Menu contexte Screenshot from 2019-09-13 20-14-53 Sans passer par les contextes Screenshot from 2019-09-13 20-15-08 Choix du context list extrait Screenshot from 2019-09-13 20-15-24 choix list episodes Screenshot from 2019-09-13 20-15-39

wwark commented 5 years ago

Commité e2a34c1129e69e41bb596f32902299e1c616e4b4

Forage commented 5 years ago

Great that you found ways to filter out the nonsense!

What is the difference between "episode" and "integrale" though? This might need a better translation to English (like "extrait" for that matter).

I must say that the way to filter it now is not very intuitive. People won't find it by accident and are likely to asks questions about not getting to see all types of videos now (not that anymore would miss the extraits though). I suppose there still isn't a way to add the filters to the left hand menu that slides out in the episodes list?

Note that if a show does not have a certain type, like episides, but you use this filter anyway, you get an error.

Also, if you stick to the current way of filtering, maybe display the options in a less technical way. Simply "List extrait videos" and "List episode videos" for example.

I don't know if the following is related to the changes you've been making for this issue, but at TF1 I now get "Exclu", "Replay" and "Extrait" sub-list items. Or at least I've never noticed it before. The same error occurs here when certain types don't exist and you try to use that list item.

wwark commented 5 years ago

Thank you for your propositions (it is just a first draft need some work like you point !)

For the moment the video with the type episode it seems to be very old catch-up's tv and I found them just for cash Investigation for the moment (very rare). So I think it is previous type for catch-up TV. In the case of video type of episode most of the time you will have the notification "No Items Found".

TF1 propose already to class contents so we follow that (France TV it is not the case). It is better to follow how the channel propose contents because it means less code so less issue.

We can also change France TV to follow TF1 with two choice "Extrait" and "Replay" and let episode in context menu just to keep content from cash investigation and maybe remove the context when no more video type episode is available.

Psychoses commented 5 years ago

fuck ! why are u speak english this time ? ^^ so let me try in english for fun...

part one about ftv :

1 i think context menu usage are bad here. because how can i add for sample "cash investigation episode list" on my fav ? shorcut-prob

2 there is more. not just episode (like you, i have only see that on "cash investigation"), integrale, extract. i have spot other stuff. i have use that :

    # What about "teaser" and "resume"?
    # E.g. http://api-front.yatta.francetv.fr/standard/publish/taxonomies/france-3_plus-belle-la-vie/contents/?size=20&page=0&sort=begin_date:desc&filter=with-no-vod,only-visible
    if video['type'] == "extrait":
        item.label += "[" + video['type'] + "] "
    elif video['type']:
        item.label += "[" + video['type'] + "] "
    else:
        item.label += "[No Type] "

and i have spotted "film" or "bande-annonce" categories on "doc/Infrarouge" and many "[No Type] " infrarouge cash

3 we cant delete extract for item with only extract like "societe\dans les yeux d'olivier". content is only extract.

a good approch will be :

for france tv, do like tf1, we copy how tf1 works :

on tf1 with have a step where py presume cat :

VIDEO_TYPES = {
    'Replay': 'replay',
    'Extrait': 'extract',
    'Exclu': 'bonus'
}

on ftv we can presume 3 cat : -"all vids cat" (with all content extract or not) (can be used for slow devices) => use standart function -"extract" => use only_extract api -"Other ?" (with all content without extract) ==> use 5000 way (never use only_replay for dont lost "film", "episode", "bande-annonce" or "No Type")

with that approch :

=================================

part two about empty subcat :

But if we go on that, the tf1 problem with empty item will appear to on ftv.

I don't know if the following is related to the changes you've been making for this issue, but at TF1 I now get "Exclu", "Replay" and "Extrait" sub-list items. Or at least I've never noticed it before. The same error occurs here when certain types don't exist and you try to use that list item.

We can maybe add a main settings addon like "i don't use a slow device and i want try to skip empty item if possible":

if slow device show empty cat : for sample "tf1\seriesfiction\4400\exclu" => "show error item found" like current version

if new settings used : use 5000 way => build item "Exclu\Replay\Extrait" only if 1 videos with this type is found. if only 1 subcat is found go directly on it. for sample : "tf1\seriesfiction\4400" => 5000 way => py files found only replay vid => so we can display directly all replay item. 4400

i use already this approch on a custom version of mytf1.py

if-fast-device

I would like to have your opinion now ^^

wwark commented 5 years ago

Thank you for all your comment. I pushed something taking in account some part your proposes. I put 500 (other videos) because just get around 150 videos take some times on high end device so it can be painfull in RPI (example Dans quelle etagères actualité France2) and I prefer not pull to many the TF1 channel and France TV because we might be blacklisted.

I prefer to let all videos category to limit the code. (so less issue :))

Thanks again for your help and propositions. I closed this long standing ticket !