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.

Sign up for GitHub

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

Jump to bottom

taxonomy: Test populating a category using ChatGPT #8084

Merged
merged 5 commits into from
May 12, 2023
Merged

taxonomy: Test populating a category using ChatGPT #8084

merged 5 commits into from
May 12, 2023

Conversation

teolemon
Copy link
Member

@teolemon teolemon commented Feb 8, 2023 •
edited
Loading

What

Prompt

I would like you to output all your answers using the following format:<2 letter language code:name of the parent in English
2 letter language code:name of the item in said language code, synonym one in the same language, synonym two in the same language, …
description:2 letter language code:a three line description of the item in said language code
wikidata:en:Qid of the item on Wikidata if you know it, otherwise nothing

@github-actions github-actions bot added categories 🧬 Taxonomies https://wiki.openfoodfacts.org/Global_taxonomies labels Feb 8, 2023
@teolemon teolemon changed the title Test populating a category using ChatGPT taxonomy: Test populating a category using ChatGPT Feb 8, 2023
@sonarqubecloud
Copy link

sonarqubecloud bot commented Feb 8, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@codecov-commenter
Copy link

codecov-commenter commented Feb 8, 2023 •
edited
Loading

Codecov Report

Merging #8084 (80e85d7) into main (a0cfac9) will increase coverage by 0.00%.
The diff coverage is 66.66%.

@@           Coverage Diff           @@
##             main    #8084   +/-   ##
=======================================
  Coverage   48.46%   48.47%           
=======================================
  Files         114      114           
  Lines       21268    21277    +9     
  Branches     4768     4770    +2     
=======================================
+ Hits        10308    10314    +6     
- Misses       9677     9679    +2     
- Partials     1283     1284    +1
Impacted Files Coverage Δ
lib/ProductOpener/Import.pm 30.73% <0.00%> (-0.04%) ⬇️
lib/ProductOpener/Packaging.pm 75.00% <75.00%> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

stephanegigandet
stephanegigandet reviewed Feb 9, 2023
View reviewed changes
taxonomies/categories.txt Outdated
@@ -69098,6 +69098,19 @@ wikidata:en:Q531184

<en:Meals with chicken
en:Chicken Tikka
hi:चिकन टिक्का, मसालेदार मुर्गा, तिक्के वाला मुर्गा
fr:Poulet Tikka, Poulet mariné aux épices
Copy link
Contributor

Choose a reason for hiding this comment

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

Synonyms must be specific to the taxonomy entry, here they are too generic and would match all other chicken with spices.

@stephanegigandet
Copy link
Contributor

Nice branch name!

teolemon
teolemon commented Feb 10, 2023
View reviewed changes
taxonomies/categories.txt Outdated
@@ -69098,6 +69098,19 @@ wikidata:en:Q531184

<en:Meals with chicken
en:Chicken Tikka
hi:चिकन टिक्का, मसालेदार मुर्गा, तिक्के वाला मुर्गा
fr:Poulet Tikka, Poulet mariné aux épices
Copy link
Member Author

Choose a reason for hiding this comment

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

Suggested change
fr:Poulet Tikka, Poulet mariné aux épices
fr:Poulet Tikka

teolemon
teolemon commented Feb 10, 2023
View reviewed changes
taxonomies/categories.txt Outdated
Comment on lines 69103 to 69106
es:Pollo Tikka, Pollo marinado en especias
de:Hähnchen Tikka, Gewürzmariniertes Hähnchen
ru:Чикен Тикка, Маринованный в специях куриный филе
zh:香料鸡块, 调味鸡块
Copy link
Member Author

Choose a reason for hiding this comment

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

Suggested change
es:Pollo Tikka, Pollo marinado en especias
de:Hähnchen Tikka, Gewürzmariniertes Hähnchen
ru:Чикен Тикка, Маринованный в специях куриный филе
zh:香料鸡块, 调味鸡块
es:Pollo Tikka
de:Hähnchen Tikka
ru:Чикен Тикка
zh:香料鸡块

@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@benbenben2
Copy link
Collaborator

@teolemon @stephanegigandet
I apply changes for both chaat and chicken tikka based on wikidata

stephanegigandet
stephanegigandet approved these changes May 12, 2023
View reviewed changes
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.

Thanks!

@benbenben2 benbenben2 merged commit 3c945d8 into main May 12, 2023
@benbenben2 benbenben2 deleted the chaaat-gpt branch May 12, 2023 17:19
@openfoodfacts-bot openfoodfacts-bot mentioned this pull request May 12, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@stephanegigandet stephanegigandet stephanegigandet approved these changes

Assignees

@teolemon teolemon

Labels
categories 🧬 Taxonomies https://wiki.openfoodfacts.org/Global_taxonomies
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@teolemon @codecov-commenter @stephanegigandet @benbenben2