AlgebraicJulia / AlgebraicJulia/SyntacticModels.jl

AMR.load and AMR.amr_to_string changes data during read in test/amr_examples

Ouverte
#25 1 commentaire 0 réactions 1 personne assignée Réclamée par @quffaro Voir sur GitHub
Langage dominant
Julia
Étoiles
4
Forks
5
Métriques de merge des PR
Aucune PR mergée en 30 j

Description

amr_to_string appears to change data as its being read. This means that a test case `(AMR.amr_to_string(AMR.load(ASKEModel, model_expr))) == modelrep` will always fail. Here are two concrete observations of this issue:

1. "@" character is omitted on `AMR.load`
**Expected** Call the argument of `amr_to_string` in the test case `M`. A substring of `M.header` should read:
"...ASKE Model Representation: SIR Model@0.1..."
**Observed** The substring of `M.header` omits the "@" character:
"...ASKE Model Representation: SIR Model0.1..."
2. A match case in `amr_to_string` erroneously interprets `ODERecord` as `ODEList`
**Expected** The `ODERecord` is interpreted by the `ODERecord` match case and return "ODE_Record = ..."
**Observed** The relevant substring in the output of `amr_to_string` has "ODE Equations" instead.

Guide de contribution

Aucun guide de contribution indexé pour ce dépôt

Évaluation

Cette issue n'a pas encore été évaluée.

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.