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

Zola - relecture #6

Merged
merged 7 commits into from
Mar 19, 2022
Merged

Zola - relecture #6

merged 7 commits into from
Mar 19, 2022

Conversation

alix-tz
Copy link
Member

@alix-tz alix-tz commented Mar 4, 2022

@rayondemiel, je voulais vous proposer une relecture avant que vous ne mergiez la PR #5 car j'avais repéré des erreurs de transcription.
Je me suis permis de faire une relecture/correction et de renommer le dossier Zola. Pouvez-vous merger si cela vous convient ?

alix-tz added 2 commits March 3, 2022 18:56
pour simplifier gestion des caractères spéciaux et des points dans le nom du dossier
Il restait des erreurs de transcription
@alix-tz alix-tz changed the title Zole - relecture Zola - relecture Mar 4, 2022
@alix-tz
Copy link
Member Author

alix-tz commented Mar 4, 2022

Je crois aussi que vous avez oublié d'ajouter votre nom et votre rôle dans htr-united.yml

@PonteIneptique
Copy link
Member

Je crois aussi que vous avez oublié d'ajouter votre nom et votre rôle dans htr-united.yml

Je pensais ajouter les noms à la fin, mais à voir.

@rayondemiel
Copy link
Contributor

Je vous remercie pour l'ensemble de vos corrections qui me semblent tout à fait justes. J'ai juste un doute sur deux d'entre elles :

  • <String CONTENT="bièe, va de Donai à Conde ! La" | - <String CONTENT="lisée, va de Douai à Coude ! La -> Conde peut faire référence à un village de haut-de-France ce qui correspondrait à la phrase.
  • <String CONTENT="marchè." | <String CONTENT="marché." -> Sur ses notes, marché semble bien contenir un accent "è"
  • <String CONTENT="où mouis, suivant l'inclinaison de" | <String CONTENT="où moins, suivant l'inclinaison de" -> le i semble bien situé comme avant dernier caractère du mot.
  • <String CONTENT="Sucrerie Fauvelle" | <String CONTENT="(Sucrerie Fauvelle" -> Est-il nécessaire d'ajouter une parenthèse alors qu'il s'agit d'un trait ?

Concernant le htr-united.yml, que dois-je faire ? J'attends la fin ?

@alix-tz
Copy link
Member Author

alix-tz commented Mar 19, 2022

Bonjour,

En effet, "Conde" (notamment pour Condé-sur-Escaut) semble plus plausible, ça correspond à une ville traversée par la Scarpe, dont il est question dans la phrase de Zola. 👍


Effectivement l'accent sur "marché" ressemble à un accent grave, mais si on regardé les autres accents, on dirait que Zola fait souvent des accents graves là où on a des accents aigus aujourd'hui. Par exemple, sur la même page, on a :

image
"canalisée", qui a été transcrit avec un accent aigu alors qu'il est tracé comme un accent grave.

Même phénomène sur les autres pages. Ici, il nous faut donc définir une règle :

  • soit on dit que quelque soit l'accent tracé, on le transcrit tel qu'on l'écrirait aujourd'hui ;
  • soit on dit qu'on respect systématiquement les "erreurs" d'accentuation.

Il faudrait pour cela se coordonner avec les autres participant-es aux transcriptions de MSS-19 pour voir s'ils/elles ont rencontré le même problème.


Pour "mouis" au lieu de "moins" : je crois que se fier à l'emplacement du point du i n'est pas suffisant (si le point est mis après que le mot est écrit, il n'est pas forcément parfaitement au-dessus). Ici, "mouis" n'est pas un mot, alors que "moins" oui, et on a le contexte et le sens de la phrase qui lèvent toute ambiguïté. EDIT : je suis allée voir la page précédente (484) qui finit sur "plus", pour "plus ou moins".


Ok pour le trait, je retire la parenthèse ! 👍

@alix-tz
Copy link
Member Author

alix-tz commented Mar 19, 2022

Je pense que c'est tout bon pour merger, le test HTRVX échoue mais c'est à cause des données Jules Verne.
Je vais ouvrir une issue à par pour traiter la question des accents graves/aigus.

Bravo pour ces transcriptions !

@alix-tz
Copy link
Member Author

alix-tz commented Mar 19, 2022

Cf. issue #8

@rayondemiel rayondemiel merged commit c22f16b into main Mar 19, 2022
@rayondemiel
Copy link
Contributor

Je valide donc la merge. Merci pour votre aide.
Il n'est donc pas nécessaire de remplir le .yml ?

@PonteIneptique
Copy link
Member

PonteIneptique commented Mar 20, 2022 via email

@alix-tz alix-tz deleted the alix-tz-review branch May 9, 2022 15:29
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

Successfully merging this pull request may close these issues.

3 participants