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

Modification of the file CdS-9858.xml in corpus #2

Open
FloChiff opened this issue Jul 3, 2024 · 0 comments
Open

Modification of the file CdS-9858.xml in corpus #2

FloChiff opened this issue Jul 3, 2024 · 0 comments
Assignees

Comments

@FloChiff
Copy link
Contributor

FloChiff commented Jul 3, 2024

J'ai fermé la PR et ajouté dans la branche main un dossier pour y mettre le corpus, dans lequel j'ai mis le fichier que tu as fait. je l'ai renommé en lui mettant l'ID que tu avais mis dans la balise TEI et en mettant l'extension .xml. À partir de maintenant, tu peux créer une nouvelle branche à partir de cela et travailler depuis le fichier tel qu'il est maintenant.

Parmi les éléments encore à modifier:

  • Tu pourras vérifier avec Marieke si elle est d'accord, mais je ne pense pas qu'il soit nécessaire de mettre un rs ou une autre balise pour des entités que tu ne connais pas. L'intérêt de la reconnaissance d'entités nommées, c'est de donner des informations sur ces entités et si tu ne connais rien sur eux, je ne pense pas qu'il y ait un intérêt à les baliser.
  • De même pour les dates, si tu n'y mets aucun @when ou @when-iso, cela n'a pas grand intérêt que tu balises
  • Pour les métadonnées, je pense qu'il serait mieux de mettre les informations en anglais plutôt qu'en français, mais après ça dépend de ce qui est prévu comme directive pour le projet
  • Le principal devrait plutôt être Marieke ou le DHI, car cela désigne la personne ou l'organisation en charge du projet au lieu de ceux qui s'occupent de l'établissement de l'édition
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

2 participants