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

chore: adding beverage preparations #9615

Merged
merged 30 commits into from
Mar 6, 2024

Conversation

aleene
Copy link
Contributor

@aleene aleene commented Jan 3, 2024

What

Introduction of a new super category, which helps to distinguish between beverages (ready to drink) and products that form the basis of beverages (after adding a fluid (water/milk/etc) to dissolve, dilute, infuse, etc).

Additional subcategories could be added if necessary.

After the categories are available some products need clean up (esp teas).

@aleene aleene self-assigned this Jan 3, 2024
@github-actions github-actions bot added 🧬 Taxonomies https://wiki.openfoodfacts.org/Global_taxonomies categories labels Jan 3, 2024
@aleene aleene changed the title chor: adding beverage preparations chore: adding beverage preparations Jan 3, 2024
@codecov-commenter
Copy link

codecov-commenter commented Jan 3, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (b1923fb) 49.45% compared to head (63b9498) 49.45%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #9615   +/-   ##
=======================================
  Coverage   49.45%   49.45%           
=======================================
  Files          67       67           
  Lines       20645    20645           
  Branches     4978     4978           
=======================================
  Hits        10209    10209           
  Misses       9139     9139           
  Partials     1297     1297           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Comment on lines 1034 to 1040
en:Beverage preparations
es:Preparaciones de bebidas
de:Getränkezubereitungen
fr:Préparations pour boissons
it:Preparati per bevande
nl:Drankbereidingen
description:en:Any product which purpose it is to create a beverage which uses water (hor or cold) in some way (diluting, infusing, dissolving, etc).
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I see it's not in Beverage category, But can't it change Nutri-Score computation ?

See is_beverage_for_nutrition_score_2023 and is_beverage_for_nutrition_score_2021 in Food.pm, we rely on the category.

cc: @stephanegigandet

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The Nutriscore was under the subcategories.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@aleene: I think it makes sense to put "beverage preparations" under "beverages". Tea bags etc. have nutrition facts for the prepared beverage, and are considered beverages for the Nutri-Score. It's also where users are likely to look for them.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can do, but then we miss out on the category ready to drink beverages.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could we add a category under beverages to indicate ready to drink beverages?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Well do we really need one? It's a bit strange to add a category that should be applied to 99% of beverages.

Or maybe another possibility could be to have a parent "Beverages and beverages preparations"?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thats why I wanted to set the preparations apart, but we can create a super as you suggested.

@aleene aleene requested a review from a team as a code owner January 12, 2024 08:52
Copy link

sonarqubecloud bot commented Mar 4, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

Copy link
Contributor

@stephanegigandet stephanegigandet left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you @aleene !

@stephanegigandet stephanegigandet merged commit 8594c2e into main Mar 6, 2024
12 checks passed
@stephanegigandet stephanegigandet deleted the aleene-beverage-preparations branch March 6, 2024 10:09
john-gom pushed a commit that referenced this pull request May 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants