Bug dans les dépôts ayant plusieurs fichiers d'annotation

maxime-fily commented 9 months ago

Les dépôts ayant plusieurs fichiers d'annotation ne répondent plus bien : il est possible d'accéder une fois au premier fichier, une fois au second, et ensuite impossible de rechanger d'onglet d'annotation. Ceci est bloquant pour les fichiers déjà déposés. Etant à ma connaissance le seul déposant avec deux annotations pour un fichier audio, je propose qu'on voie ensemble quoi faire car cette 'issue' a des ramifications vis à vis de la convention OLAC, dont il faudrait discuter ensemble.

alexis-michaud commented 9 months ago

Effectivement, le passage d'une annotation à l'autre ne se fait qu'en sens unique ! Il y a un bug.

On peut passer de "annotation 1" (celle en XML, qui s'affiche en premier) à "annotation 2", en cliquant sur "annotation 2". Cool ! Mais on est ensuite "coincé" sur la 2e annotation, sans possibilité de revenir à la première. L'indication "annotation 1" est inerte (il ne se passe rien qd on clique dessus).

Voici un "minimal non-working example" : https://doi.org/10.24397/pangloss-0007681

Le bouton "actif" (sur lequel on peut cliquer) c'est "annotation 2" ... même si c'est déjà cette annotation-là qui s'affiche. Qd on clique sur le bouton, la page se recharge, à l'identique (puisqu'on est déjà sur cette annotation 2) : on ne peut pas revenir à "annotation 1".

Sans doute un problème de gestion des boutons "annotation 1" et "annotation 2" ? @EdouardSombie

sguillaume commented 8 months ago

Autre exemple : https://doi.org/10.24397/pangloss-0007530

m8nli9ht commented 8 months ago

Je constate le problème de gestion des boutons: une fois qu'on a cliqué sur le lien "annotations 2", le lien "annotations 1" n'est pas cliquable. Il faut donc regarder de ce côté là en premier lieu (pas géré dans le player)

alexis-michaud commented 8 months ago

Du coup, @EdouardSombie ça serait à toi de regarder ? ++

EdouardSombie commented 8 months ago

Oui, je regarde ce truc. C'est vraiment étrange. Je vous tiens au courant.

EdouardSombie commented 8 months ago

Re-salut, J'ai réglé le souci d'affichage sur le bouton et étrangement, les annotations se chargent bien :) https://doi.org/10.24397/pangloss-0007681

@m8nli9ht : Quand je charge l'exemple donné par Séverine( https://doi.org/10.24397/pangloss-0007530 ), le player ne se charge pas et me donne cette erreur dans la console : Uncaught (in promise) TypeError: Cannot read properties of undefined (reading 'transcriptions') Quand je passe à l'annotation 2 et que je reviens à la 1, ça marche...

@Alexis : tu peux faire un ou 2 tests sur des ressources avec plusieurs annotations ?

A bientôt Edouard

alexis-michaud commented 8 months ago

Yes, c'est parti ! Voici le retour un peu détaillé.

J'ai réussi à reproduire le bug "Séverine" :1st_place_medal: Avec son exemple, et aussi avec "C'est demain la fin du monde" [titre qu'on va changer en urgence pour éviter la GAV de toute la #teamPangloss pour écoterrorisme islamogauchiste, ça va de soi]. C'est ici : pangloss-0004326. Et aussi pour un texte na, ici. La situation commune à ces 3 ressources : un XML, un PDF. Là, ça rame : ça patine dans la semoule, rien ne s'affiche. Il semble que ça soit lié au fait que ça soit un PDF qui est déclaré comme annotation 1. En effet, quand je clique sur "annotation 2", elle s'affiche bien. Et quand, de là, je reviens à "annotation 1", tout va bien : le PDF est affiché. (C'est pareil pour toutes les ressources qui partagent cette typologie.) Une solution rapide (mais incomplète) au problème consisterait donc, tout simplement, à utiliser systématiquement un fichier XML comme annotation 1, quand il y a plusieurs annotations dont une au moins en XML. Ca aurait du sens, vu que le XML est du texte structuré logiquement : le format Pangloss (XML), c'est du Pangloss dans toute sa splendeur, comme on aime particulièrement à le diffuser. Néanmoins ça serait pas forcément une bonne idée de mettre ce réglage en dur dans l'interface : que, si on a du XML, ça soit toujours le XML qui s'affiche en annotation 1. Là c'est à toi @sguillaume de dire. Il y a vraisemblablement des situations dans lesquelles il pourrait paraître préférable de mettre un PDF en premier ? (par exemple quand c'est une partition de musique ou autre affichage très clair, parlant & pro, que le format XML Pangloss ne permet pas de mettre en forme ?) Ou ça ne peut en aucun cas se présenter à ton avis ? Une meilleure solution consisterait à être capable de résoudre le bug sans changer l'organisation des métadonnées : afficher un PDF chaque fois qu'il est appelé, même quand c'est en onglet "annotation 1".

Je n'ai pas sous la main d'exemple de document pour lequel on n'ait qu'un PDF et pas de XML. Il faudrait regarder : l'hypothèse serait que ça devrait... ne pas se charger correctement, pas plus que dans les exemples ci-dessus.

Un document qui n'a pas seulement une ni deux ni trois mais quatre annotations : pangloss-0004321. Deux XML, deux PDF. Là, tout marche bien. Ca s'affiche de suite dès l'arrivée sur la page. :100: La raison pour laquelle ça passe bien est sans doute que l'annotation 1 est du XML.

m8nli9ht commented 8 months ago

Re-salut, J'ai réglé le souci d'affichage sur le bouton et étrangement, les annotations se chargent bien :) https://doi.org/10.24397/pangloss-0007681

@m8nli9ht : Quand je charge l'exemple donné par Séverine( https://doi.org/10.24397/pangloss-0007530 ), le player ne se charge pas et me donne cette erreur dans la console : Uncaught (in promise) TypeError: Cannot read properties of undefined (reading 'transcriptions') Quand je passe à l'annotation 2 et que je reviens à la 1, ça marche...

@alexis : tu peux faire un ou 2 tests sur des ressources avec plusieurs annotations ?

A bientôt Edouard

Je corrige le bug.

Pour la petite explication : la différence se trouve dans les paramètres de l'URL selon si on renseigne les options d'affichage ou pas (optionTextTranscriptions=&optionTextTranslations=&optionWordTranscriptions=&optionWordTranslations=). C'est ce qui fait que dans un cas ça marche et pas dans l'autre.

m8nli9ht commented 8 months ago

Corrigé et livré, à valider

sguillaume commented 8 months ago

C'est bon pour moi ! Je vous laisse regarder aussi avant de fermer le ticket au cas où je n'aurais pas testé tous les cas.

alexis-michaud commented 8 months ago

Pour moi, ça ne marche pas mieux qu'avant... Ca rame et ça ne charge pas le PDF. Testé sous Firefox (après suppression du cache & rechargement).

L'URL est :

https://pangloss.cnrs.fr/corpus/show?corpus=Ubykh&lang=en&mode=pro&oai_primary=cocoon-f3cccaf0-cd8e-3353-b227-fc6e196577ce&oai_secondary=cocoon-3ea915d3-14f9-3f69-bfd4-a6e91e06f321

Comme avant, je dois cliquer sur "annotation 2" puis revenir sur "annotation 1" pour que ça se charge. L'URL devient alors :

https://pangloss.cnrs.fr/corpus/show?continuousPlay=false&corpus=Ubykh&lang=en&mode=pro&oai_primary=cocoon-f3cccaf0-cd8e-3353-b227-fc6e196577ce&oai_secondary=cocoon-3ea915d3-14f9-3f69-bfd4-a6e91e06f321&optionMorphemeTranscriptions=other&optionMorphemeTranslations=en&optionNotes=&optionSentenceTranscriptions=phono&optionSentenceTranslations=en&optionTextTranscriptions=&optionTextTranslations=&optionWordTranscriptions=other&optionWordTranslations=en

Même résultat sous Brave.

maxime-fily commented 8 months ago

Sur les annotations doubles à l'origine de la demande (na de Lataddi), tout va bien. Par contre, le cas mentionné par Alexis ne fonctionne toujours pas (sur l'ubykh). Mais on peut voir ça demain, c'est pas la fin du monde ;)

m8nli9ht commented 7 months ago

Re-Corrigé et livré, à valider!

alexis-michaud commented 7 months ago

Bilan du test : l'annotation 1 (la page PDF) se charge, comme souhaité :100: ... mais on y a perdu la page XML, qui est vide de contenu, parce que toutes les cases d'options sont décochées.

URL de la page qui charge "à vide" : https://pangloss.cnrs.fr/corpus/show?continuousPlay=false&corpus=Ubykh&lang=en&mode=pro&oai_primary=cocoon-f3cccaf0-cd8e-3353-b227-fc6e196577ce&oai_secondary=cocoon-81e5e086-e55c-3636-a835-586929d58316&optionMorphemeTranscriptions=&optionMorphemeTranslations=&optionNotes=&optionSentenceTranscriptions=&optionSentenceTranslations=&optionTextTranscriptions=&optionTextTranslations=&optionWordTranscriptions=&optionWordTranslations=

Ce qu'on voudrait : l'affichage ordinaire :

URL de la page qui se charge normalement (affichage par défaut sur "landing page") : https://pangloss.cnrs.fr/corpus/show?lang=en&mode=pro&oai_primary=cocoon-f3cccaf0-cd8e-3353-b227-fc6e196577ce&oai_secondary=cocoon-81e5e086-e55c-3636-a835-586929d58316&optionTextTranscriptions=&optionTextTranslations=&optionSentenceTranscriptions=phono&optionSentenceTranslations=en&optionWordTranscriptions=other&optionWordTranslations=en&optionMorphemeTranscriptions=other&optionMorphemeTranslations=en&optionNotes=&continuousPlay=false

m8nli9ht commented 7 months ago

Je n'ai rien modifié dans cette partie du lecteur, qui d'ailleurs est conforme à cette demande : https://github.com/CNRS-LACITO/eastlingplayer/issues/27

Il faudrait donc que l'URL vers l'Annotation XML ne contienne aucune des options, soit https://pangloss.cnrs.fr/corpus/show?lang=en&mode=pro&oai_primary=cocoon-f3cccaf0-cd8e-3353-b227-fc6e196577ce&oai_secondary=cocoon-81e5e086-e55c-3636-a835-586929d58316 J'ai vérifié et cela fonctionne de mon côté.

@EdouardSombie, tu as changé quelque chose entre temps concernant l'URL et ses options d'affichage?

CNRS-LACITO / Pangloss_website

Bug dans les dépôts ayant plusieurs fichiers d'annotation #221