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

Impossibilité de publier une instance Lodex lorsque les données jsonl contiennent lodex_published #2207

Open
eonm-pro opened this issue Nov 8, 2024 · 2 comments
Labels
⚠️ Minor defect Le problème ne permet pas d'optimiser le processus standard ou concerne un cas d'usage spécifique

Comments

@eonm-pro
Copy link
Collaborator

eonm-pro commented Nov 8, 2024

Lorsqu'on importe un jeu de données jsonl contenant un clé lodex_published, il n'est pas possible de le publier sur lodex. Le bouton "publier" du back end n'a aucun effet.

{"uri":"ark:/67375/P5F-21PDK04C-7","Nom du corpus":"Mémoire-Neurosciences","Lien vers instance Lodex":"https://memoire-neurosciences.corpus.istex.fr/","lodex_published":true}
{"uri":"ark:/67375/P5F-8H1D4JN0-B ","Nom du corpus":"Mémoire-Neurosciences-V2","Lien vers instance Lodex":"https://memoire-neurosciencesv2.corpus.istex.fr/","lodex_published":true}

Ce bug a été observé lors de la migration d'anciennes instances Lodex (inistcnrs/lodex-theme-istex:12.23.33) vers de nouvelles instances (inistcnrs/lodex-theme-istex:12.55.4). Le bug affecte également la version 14.0.63 de Lodex.

Reproduire

Depuis le backend d'une instance Lodex :

  1. Importer le jeux de données jsonl avec l'importer JSON - import Lodex.
  2. Ajouter des éléments à l'affichage/ou importer un modèle.
  3. Cliquer sur publier.
  4. Les données ne sont pas publiées sur l'instance, l'icône de l’œil qui permet de se rendre sur l'affichage public de l'instance n'apparait pas non plus.

image

Environnement :

  • Google Chrome Version 130.0.6723.70 (Build officiel) (64 bits)
  • également testé sur Firefox Nightly (134.0a1)
  • cnrsinist/lodex:14.0.63
@eonm-pro eonm-pro added the Triage Nouveau problème ou besoin à étudier et à classer label Nov 8, 2024
@touv touv added ⚠️ Minor defect Le problème ne permet pas d'optimiser le processus standard ou concerne un cas d'usage spécifique and removed Triage Nouveau problème ou besoin à étudier et à classer labels Nov 18, 2024
@touv
Copy link
Contributor

touv commented Nov 18, 2024

une solution possible serait d'ajouter une règle dans les loaders qui supprime cette colonne

@yannguillemin
Copy link
Collaborator

Dans le cas présent le problème semble venir du caractère "underscore" (cad que tout fonctionne si on l'enlève), mais je viens de faire un test avec des données dont les colonnes sont nommées "dataset_A_1"..., et là tout fonctionne en l'état. Donc je retire ce que j'ai pu dire précédemment.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
⚠️ Minor defect Le problème ne permet pas d'optimiser le processus standard ou concerne un cas d'usage spécifique
Projects
None yet
Development

No branches or pull requests

3 participants