Skip to content

Incoherence validation #1

@HoustonLFB

Description

@HoustonLFB

Bonjour,

Ça faisait longtemps qu'un projet comme le vôtre trottait dans ma tête. En faisant des recherches, j'ai découvert hier votre repo et j'en suis extrêmement heureux.

Dans mon fork, j'ai ajouté l'ordo de Toulouse et je pensais avoir corrigé celui de France et le sanctoral avant de remarquer une incohérence entre la doc liturgical-scheme.md et error.ts. Une fois que tout sera réglé, je ferais des pull requests.

Dans la doc, les mémoires ont comme valeurs de precedence dans le YAML : 12 pour les mémoires obligatoires et 13 pour les mémoires facultatives.

Je prends l'exemple du premier saint qui fait planté : Ste Angèle Merici, le 27/01, c'est une mémoire facultative. La valeur de precedence dans le YAML est 12, je le passe à 13 et j'obtiens cette erreur :

[kal-forge] erreur : Parse(InvalidMemoriaPrecedence { slug: "angelae_merici", from: 1969, found_precedence: 12 })

Extrait de error.ts :

/// V-Natura-Memoria — nature=memoria + precedence ∉ {11,12}
InvalidMemoriaPrecedence { slug: String, from: u16, found_precedence: u8 },

Extrait de liturgical-scheme.md :

 V-Natura-Memoria — nature = memoria ⟹ precedence_yaml ∈ {12, 13}. → ParseError::InvalidMemoriaPrecedence

Je vois bien dans la doc la colonne pour les valeurs YAML et les valeurs internes. Y a t-il donc quelque chose que je ne comprends pas ? C'est comme si le code n'est pas à jour avec la doc.

Même chose pour certaines fêtes qui n'ont pas le non rang.
2 autres exemple :

  • Ascension dans le Temporal. Precedence 2, cohérent avec la doc
  • IIe Dimanche TO. Precedence 6, alors que c'est sensé être 8 dans le doc.

EDIT: je viens de voir dans cette doc (docs/resources/Table-des-preseances-des-jours-liturgiques.md), les valeurs qui semblent cohérente avec le code.
END EDIT

Voilà j'espère que cet issue pourra aider, je suis très reconnaissant de votre travail. Je l'utiliserai dans le futur et je souhaite également un peu y contribuer (seulement au niveau des données).

Note: Tout a été fait à la main, aucune IA. C'est peut-être de là que vient l'erreur.

Pax Christi
+ Wilfried

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions