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

Erreur de merge avec une map vide (nonapplicable) #19

Closed
bamthomas opened this issue Nov 28, 2023 · 3 comments
Closed

Erreur de merge avec une map vide (nonapplicable) #19

bamthomas opened this issue Nov 28, 2023 · 3 comments

Comments

@bamthomas
Copy link

J'ai la même erreur que celle du commentaire :

#14 (comment)

pourtant dans notre map, il semble qu'il y ait les 3 types de valeurs (nc, c, na).

cf iroco-co/audits-frago#5

@bamthomas bamthomas changed the title erreur de merge avec une map vide (nonapplicable) Erreur de merge avec une map vide (nonapplicable) Nov 28, 2023
@bamthomas
Copy link
Author

hmm ça a l'air d'avoir été corrigé 51e0067

@bertrandkeller
Copy link
Contributor

Il ne merge pas si une map est vide. Faut vérifier tous les cas possibles ou bien mettre le if qu'il faut.

@bamthomas
Copy link
Author

bamthomas commented Nov 28, 2023

en fait ça échoue plus tôt. Pour le moment j'ai pu contourner en changeant la ligne 176 de aggregate-score.html par :

  {{ if ($data.Get (printf "average-%s-nonapplicable" $type)) }}
    {{ $dict1 = merge $dict1 (dict "all" ((merge (merge ($data.Get (printf "average-%s-conforme" $type)) ($data.Get (printf "average-%s-nonconforme" $type))) ($data.Get (printf "average-%s-nonapplicable" $type)))))}}
  {{ else }}
      {{ warnf "no nonapplicable data for %s" $type}}
      {{ $data.SetInMap (printf "average-%s-nonapplicable-t" $type) ("what?") (dict) }}
  {{ end }}

Parceque le soucis c'est que le test d'après, il s'applique à conforme/non-conforme, du coup on n'est pas sûr que la map nonapplicable ne soit pas nil.

Mais je ne sais pas quoi mettre comme valeur "thematique".

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