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

Test de 5 fichiers xml complets sur CapiTainS Guideline Single File Validator #38

Open
saroulle opened this issue Feb 24, 2018 · 12 comments

Comments

@saroulle
Copy link
Contributor

J'ai réalisé des tests avec https://capitains-validator.herokuapp.com/ sur les 5 fichiers xml suivants :
pos2000.pos200001.xml
pos2000.pos200006.xml
pos2006.pos200604.xml
pos2010.pos201021.xml
pos2014.pos201410.xml
Choix Resource Type : CapiTainS CTS Guidelines - TEI Text

Les résultats ont toujours deux même erreurs :
test-travis-fichier-complet

J'ai tenté d'ajouter dans la balise text : xml:lang="fr" mais cela ne change rien.

Une erreur concernant la déclaration de la langue est signalée dans une issue de Capitains HookTest, mais je ne sais pas si c'est le même problème :
Capitains/HookTest#111

@PonteIneptique Pouvez-vous nous éclairer ?

@saroulle
Copy link
Contributor Author

pull request du dossier tests avec les 5 fichiers.

@vasaura
Copy link
Contributor

vasaura commented Feb 24, 2018

@saroulle Le problème peut être réglé de la manière suivante:

  • l'urn et la langue doivent être rajoutés dans la balise text
  • l'urn doit être fini par un label. Nous avons choisi avec @AnZadykhina "posENC" . ce label doit être spécifié dans le teiHeader dans xml=id
  • le nouvel urn pour edition est :"urn:cts:freLit:pos2000.pos200001.posENC"
    voici la nouvelle balise text
<text xml:lang="fr" xml:base="urn:cts:freLit:pos2000.pos200001.posENC">
  • Les metadonnées cts work nécessitent des modifications pour être valides: ajout de l'élément edition et le nouvel urn.
    On va reprendre ces éléments avec @AnZadykhina en changeant tous les templates et le xslt.
    Le schema aussi doit être modifié car oxygen invalide les fichiers.

@saroulle
Copy link
Contributor Author

Parfait, j'ai corrigé les fichiers tests et ça fonctionne avec.
Pouvez-vous me valider ma pull request avec le dossier tests que je le mette à jour ?

@vasaura
Copy link
Contributor

vasaura commented Feb 24, 2018

on est en train de faire le traitement automatique pour tous les fichiers cts et tei et on craint d'avoir des conflits et des répétitions. C'est la raison pour laquelle on n'a pas validé ce pull request. En plus le validator n'est pas une garantie absolue. Il faut passer par travis qui execute d'autres testes. Nous, on a eu d'autres erreurs en executant le test travis en local.

@SybilleClochet
Copy link
Contributor

On croise les doigts. Dites-nous quand on pourra vérifier les fichiers. Je modifie le fichier relaxNG avec les modifs indiquées par @vasaura.

@AnZadykhina
Copy link
Contributor

@SybilleClochet avec le schéma RelaxNG il y a beaucoup de problèmes. Oxygène trouve beaucoup d'erreurs quand on essaie de faire la validation pour fichier header+body (avant toutes les modifications d'aujourd'hui). Voila le print screen.
validation du fichier en oxygene

@saroulle
Copy link
Contributor Author

Ce sont essentiellement des erreurs liées aux balises html de mise en forme (hi, num, lb). Peut-on les intégrer au schéma RelaxNG ?

@SybilleClochet
Copy link
Contributor

oui je suis en train de regarder.

@SybilleClochet
Copy link
Contributor

c'est bon j'ai terminé. je pousse le nouveau fichier.

@vasaura
Copy link
Contributor

vasaura commented Feb 24, 2018

as-tu vérifié si le schéma valide un fichier complet?

@SybilleClochet
Copy link
Contributor

oui j'ai vérifié, c'est ok. En revanche j'ai problème pour git push de déconnexions intempestives (je suis dans une bibli). je sors en recommence chez moi.

@vasaura
Copy link
Contributor

vasaura commented Feb 25, 2018

J'apporte une précision à mon message concernant l'urn qui devait être fini par un label. Initialement, nous avions choisit "posENC", mais on s'est rendu compte qu'il nous est suggéré dans le fichier position.xml un autre modèle: "position-id(work)". Donc, voici l'exemple final "urn:cts:freLit:pos2000.pos200001.position-200001"
J'ai modifié les fichiers tei et cts en conséquence.

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

4 participants