treflehq / trefle-api

🍀 Trefle is a botanical JSON REST API for plants species, allowing you to search and query over all the registered species, and build the next gardening apps and farming robots.
https://trefle.io
GNU Affero General Public License v3.0
488 stars 51 forks source link

The common pear is listed as not edible, with edible_part=null #68

Open JackieCalapristi opened 3 years ago

JackieCalapristi commented 3 years ago

Describe the bug The common pear is listed as not edible and edible_part=null under the species endpoint. I assume this is listed as not edible correctly because pear trees aren't edible, but pears are, which makes me think that edible_part should list "fruits" instead of null. If these are all false assumptions, maybe updating the documentation to explain this (i.e. edible refers to the entire plant, even if the common name refers to the fruit) would be helpful? I thought about submitting a correction in a POST request, but I wasn't sure if this was actually an accurate error or not because I'm definitely not a plant expert.

To Reproduce Steps to reproduce the behavior: Make a get request to https://trefle.io/api/v1/species/172991/?token= and note values for edible and edible_part for the common pear.

Expected behavior To see that edible_part listed "fruits" instead of null in the response body.

Screenshots Screen Shot 2020-10-10 at 5 17 29 PM

Additional context If my assumptions here are correct, I've noticed the same thing for other plants (corn, for example) and would be happy to submit POST requests to correct these as I see them. Thanks so much in advance!