-
Notifications
You must be signed in to change notification settings - Fork 687
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
Create extract-raster-values-for-points tutorial #243
Conversation
Fixed file/dir structure. I also modified some language around points vs. plots and polygons, to make it clear that this would work for polygon plot data as well.
Hi @jdbcode , wanted to make sure this worked since there is an error about the Travis CI build that blocked merging when I submitted it last week. I don't know what the error means - is it something I need to correct? |
RE Travis CI build failure: appears to be failing because there is no period at the end of the tutorial description sentence in the yaml header. It is fine for now. I'll add it to the review comments. I'm starting my review now. |
Hi @swinsem, I've emailed you about some ideas for making the tutorial a little more modular and including a couple more examples. |
This update includes changes recommended by Justin, including a couple new examples, and a section at the end on the pixels included in region reductions and notes on the CRS and scale that are necessary for that process.
Hi @swinsem, please see my changes and let me know if you are okay with them or if you'd like to discuss anything. |
@jdbcode it looks great, thanks for the excellent edits! |
Made a couple minor changes to clarify wording on weighted reducers.
Fixed file/dir structure. I also modified some language around points vs. plots and polygons, to make it clear that this would work for polygon plot data as well.
@jdbcode @gino-m