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

Crop Production Percentile model: update number of supported crops from 175 to 172 #154

Open
emilyanndavis opened this issue Jan 13, 2025 · 0 comments
Labels
good first issue Good for newcomers

Comments

@emilyanndavis
Copy link
Member

While working on InVEST #614, I noticed the number of crops in the supported crop list for the Crop Production Percentile model is 172, not 175 as advertised in the User Guide.

Compared to the full list of 175 crops in EarthStat's Global Harvested Area and Yield for 175 Crops Metadata and Technical Documentation PDF, our list is missing coir, gums, and popcorn.

It seems appropriate to continue excluding these 3 crops from the model since, for whatever reason, their associated datasets are not available from EarthStat's Harvested Area and Yield for 175 Crops data download page .

Still, for the sake of precision, we should probably update 175 to 172 in the User Guide, and it might also be worth adding a short note about the exclusion of coir, gums, and popcorn due to unavailable datasets.

@emilyanndavis emilyanndavis added the good first issue Good for newcomers label Jan 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

1 participant