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

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

Open
maxime-fily opened this issue Oct 18, 2023 · 15 comments
Open

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

maxime-fily opened this issue Oct 18, 2023 · 15 comments
Assignees

Comments

@maxime-fily
Copy link

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
Copy link
Collaborator

alexis-michaud commented Oct 19, 2023

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

image

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
Copy link
Contributor

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

@m8nli9ht
Copy link
Collaborator

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
Copy link
Collaborator

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

@EdouardSombie
Copy link
Collaborator

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

@EdouardSombie
Copy link
Collaborator

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
Copy link
Collaborator

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

  • J'ai réussi à reproduire le bug "Séverine" 🥇
    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.
    image
    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. 💯 La raison pour laquelle ça passe bien est sans doute que l'annotation 1 est du XML.

@m8nli9ht
Copy link
Collaborator

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
Copy link
Collaborator

Corrigé et livré, à valider

@sguillaume
Copy link
Contributor

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.

@maxime-fily
Copy link
Author

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
Copy link
Collaborator

Re-Corrigé et livré, à valider!

@m8nli9ht
Copy link
Collaborator

m8nli9ht commented Nov 22, 2023

Je n'ai rien modifié dans cette partie du lecteur, qui d'ailleurs est conforme à cette demande :
CNRS-LACITO/eastlingplayer#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?

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

No branches or pull requests

5 participants