-
-
Notifications
You must be signed in to change notification settings - Fork 67
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
Missing nutrients (e.g. Polyols) #862
Comments
@CharlesNepote There are 47 "nutrients" found in Food.pm that are currently not in off-dart.
|
@monsieurtanuki, I have investigated a little bit and, yes, all your questions' answers are in the Perl code: https://github.com/openfoodfacts/openfoodfacts-server/blob/7d295d10fcc8939cccfd093810cb0edb8a3c9449/lib/ProductOpener/Food.pm#L400 We should ask ourselves if these data shouldn't be provided as data, and not as Perl code, to let any kind of apps reuse it more easily -- a YAML file as it allows comments? Should I open a bug @stephanegigandet ? |
@CharlesNepote Developers can already get a JSON version of it with https://world.openfoodfacts.org/cgi/nutrients.pl?lc=fr&cc=fr, easy to reuse, therefore I think we're already almost good here. |
@monsieurtanuki @CharlesNepote all the nutrients and their default unit are in the nutrients taxonomy: https://github.com/openfoodfacts/openfoodfacts-server/blob/main/taxonomies/nutrients.txt I can add the units to the output of https://world.openfoodfacts.org/cgi/nutrients.pl?lc=fr&cc=fr |
@stephanegigandet That would be nice, that's typically the |
@monsieurtanuki : FYI, @stephanegigandet has added the units into https://world.openfoodfacts.org/cgi/nutrients.pl?lc=fr&cc=fr |
Thank you @CharlesNepote @stephanegigandet! For the record, some stats about the
|
To be checked, added or edited in off-dart (data from
|
Description
cf. @CharlesNepote's openfoodfacts/smooth-app#4943
There are some nutrients in Food.pm and https://world.openfoodfacts.org/cgi/nutrients.pl?lc=fr&cc=fr that we don't deal with in off-dart: they are not in off-dart's
Nutrient
enum
.Expected behavior
We should be able to deal with all nutrients, at least most of them.
The text was updated successfully, but these errors were encountered: