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

Parse ingredient processing specified with a comma, after the ingredient #3905

Open
Tracked by #9096
aleene opened this issue Jul 20, 2020 · 3 comments
Open
Tracked by #9096
Assignees
Labels
🐛 bug This is a bug, not a feature request. 🥗🔍 Ingredients analysis https://wiki.openfoodfacts.org/Ingredients_Extraction_and_Analysis ⏰ Stale This issue hasn't seen activity in a while. You can try documenting more to unblock it.

Comments

@aleene
Copy link
Contributor

aleene commented Jul 20, 2020

What

In 100% jus de pomme biologique de fruits de hautes tiges, pasteurisé, pasteurisé is recognized as separate ingredient.

Steps to Reproduce

See https://be.openfoodfacts.org/ingrediënt/fr:pasteurise.

Expected behaviour

It would suffice the have the comma(,) removed before pasteurisé. Then pasteurisé can be handles by the processing taxonomy.

Probably there are more words like this, also in other languages, so a generic solution would be nice.

Number of products impacted

  • 283 french products

Part of

Tasks

No tasks being tracked yet.
@aleene aleene added the 🐛 bug This is a bug, not a feature request. label Jul 20, 2020
@stephanegigandet stephanegigandet changed the title pasteurisé as ingredient Parse ingredient processing specified with a comma, after the ingredient Jul 20, 2020
@stephanegigandet stephanegigandet added the 🥗🔍 Ingredients analysis https://wiki.openfoodfacts.org/Ingredients_Extraction_and_Analysis label Jul 20, 2020
@stephanegigandet stephanegigandet self-assigned this Jul 20, 2020
@svensven
Copy link
Contributor

svensven commented Sep 23, 2020

at the same time, the fix would have to handle

oeuf entier issu de poule élevée en plein air (liquide, pasteurisé)

correctly, and not turn it into

oeuf entier issu de poule élevée en plein air (liquide pasteurisé)

https://world.openfoodfacts.org/product/3270160729241/pasteis-de-bacalhau-picard

@svensven
Copy link
Contributor

and if the pasteurisé is the last ingredient, it presumably applies to the whole product, not the preceding ingredient.

92% pur jus de pomme, 8% pur jus de framboise, pasteurisé

https://world.openfoodfacts.org/product/7640143660247/iris-jus-de-pomme-framboise

@github-actions github-actions bot added the ⏰ Stale This issue hasn't seen activity in a while. You can try documenting more to unblock it. label Jan 14, 2021
@teolemon teolemon removed the ⏰ Stale This issue hasn't seen activity in a while. You can try documenting more to unblock it. label Jul 19, 2021
Copy link
Contributor

This issue has been open 90 days with no activity. Can you give it a little love by linking it to a parent issue, adding relevant labels and projets, creating a mockup if applicable, adding code pointers from https://github.com/openfoodfacts/openfoodfacts-server/blob/main/.github/labeler.yml, giving it a priority, editing the original issue to have a more comprehensive description… Thank you very much for your contribution to 🍊 Open Food Facts

@github-actions github-actions bot added the ⏰ Stale This issue hasn't seen activity in a while. You can try documenting more to unblock it. label Feb 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛 bug This is a bug, not a feature request. 🥗🔍 Ingredients analysis https://wiki.openfoodfacts.org/Ingredients_Extraction_and_Analysis ⏰ Stale This issue hasn't seen activity in a while. You can try documenting more to unblock it.
Projects
Status: To do
Status: To discuss and validate
Development

No branches or pull requests

4 participants