incubateur-ademe / agribalyse

Site vitrine agribalyse.ademe.fr et application d'exploration des données
https://agibalyse.ademe.fr
MIT License
1 stars 1 forks source link

Problème pour les données avec une variante saison ou avion #1

Closed martinregner closed 3 years ago

martinregner commented 4 years ago

Il y a deux particularités qui ne sont pas prises en compte dans l'app (et renvoie des valeurs fausses) :

Il y a donc 10 lignes avec un code AGB =/= code CIQUAL. Pour autant, il semblerait que la recherche se fasse sur le code CIQUAL. Ainsi, la recherche ne se fait pas sur toutes les lignes + les valeurs renvoyées ne sont pas les bonnes, ex. :

Est-ce que le problème peut être réglé en cherchant directement dans les codes AGB ?

(Pour autant, les codes CIQUAL à afficher sur les fiches reste les mêmes 😄)

JenniferStephan commented 4 years ago

hola :) fichier de données corrigé pour les transports (ex de la mangue; haricot) et saison (ex de la fraise; tomate)

martinregner commented 4 years ago

Top ! Il y a tout, merci @JenniferStephan 👍

Par contre les codes CIQUAL ne sont plus les bons : normalement, pour les variantes (ex. les 3 fraises, ou les 2 haricots) les codes CIQUAL sont les mêmes (nutritionnellement c'est la même fraise, enfin en théorie, ça pourrait se discuter 😜 ). Par contre c'est bien 2 codes différents dans AGB. Possible d'avoir le code CIQUAL sur les fiches (sans le dernier chiffre donc ?) ?

Pour les URL, est-ce que l'on pourrait garder le "_" pour séparer des variantes ? c'est juste que pour d'autres variantes qui pourrait arriver lors des prochaines màj, il pourrait y avoir recouvrement entre un code CIQUAL et une variante (ex. 1000, si on a une variante avec un 1 ça revoie vers 10001 ... c'est pas pareil).

martinregner commented 3 years ago

👍