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_statistics_FEWSNET review #170

Open
meronmi opened this issue May 22, 2024 · 4 comments
Open

crop_statistics_FEWSNET review #170

meronmi opened this issue May 22, 2024 · 4 comments
Assignees

Comments

@meronmi
Copy link

meronmi commented May 22, 2024

As requested at the AgML meering on 21/5/2024, here the result of a quick check of the github entry “crop_statistics_FEWSNET”.
Sentences “Method and citation to be added” and “A formal citation for FEWS NET data will be provided in due course” to be updated.
Please specify the source of the administrative layer (it seems similar to FAO Gaul2 but with important differences). Side note: Admin NE-A2-0109 looks bit strange, it is a perfect circle.
Additional information on season name may be handy for the user. Possible seasons are: Main, Annual, Summer, Meher. Differences are not fully clear (e.g. between Main (there is no secondary) and Annual.

@krsnapaudel
Copy link
Collaborator

@gnodnooh : Please address comments from @meronmi. Thanks.

@gnodnooh
Copy link
Collaborator

Thank you, @meronmi, for checking our data.

  • FEWS NET does not have a formal citation yet. A citation for our data paper will be added once we submit it in June 2024.
  • FEWS NET uses a variety of sources for administrative boundaries to best reflect agricultural statistics at the given time (see: https://fews.net/data/geographic-boundaries). It is possible to have slight differences with other sources. We’ve recognized those tiny circles but do not want to change them, as they may serve specific purposes.
  • I can double-check with the data maintainers. I believe the “Annual” season is reserved for when the country collectively reports multi-season records.

I will update our data card accordingly shortly. Hi @krsnapaudel, if possible, could you add a link to our data card in the draft so that people can use that citation once updated?

@krsnapaudel
Copy link
Collaborator

@gnodnooh: If you mean paper draft, we will reference all data cards in the Appendix of paper. Also in the README of GitHub repo.

@gnodnooh
Copy link
Collaborator

@krsnapaudel That sounds good to me. Thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants