Inist-CNRS / lodex

Linked Open Data EXperiment
https://www.lodex.fr/
Other
53 stars 12 forks source link

La routine cross-by (et sans doute d'autres) ne prend pas en compte 0 comme valeur #2109

Closed AnaelKremer closed 2 months ago

AnaelKremer commented 2 months ago

La routine cross-by ne prend pas toutes les données en compte si des valeurs numériques sont égales à 0.

si nous avons comme données des années et des coûts par exemple, comme [{id: "2017",value:0}, {id: "2017",value:1000}, {id: "2017",value:0}, {id: "2017",value:1000}]

{source:"2017",target:1000,weight:2}

alors que l'on devrait également avoir aussi

{source:"2017",target:0,weight:2}

On a donc perdu 2 notices dans cet exemple.

Après discussion avec @parmentf il semblerait que cela provienne de l'utilisation de LodexOutput # On génére le json de sortie [LodexOutput] indent = true extract = total extract = maxSize extract = maxValue extract = minValue

Annotation 2024-07-23 154245

le code LodexOutput écarte visiblement les 0 car falsy > https://github.com/Inist-CNRS/lodex/blob/d9209d231418ca7e914d894238101a75f2d34e39/packages/ezsLodex/src/formatOutput.js#L50