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

Combined updates: get_canopyheight.sh, get_bareground.sh, modscag processing in dem_mask.py #51

Open
wants to merge 11 commits into
base: master
Choose a base branch
from

Conversation

jmichellehu
Copy link
Contributor

Ran into issues with fetching tile_list using get_modis_tile_list() so I hardcoded western conus tile_list. I didn't troubleshoot beyond confirming modis_dict contains 460 items, but something seems to be awry with locating intersecting geometries.

@dshean
Copy link
Owner

dshean commented Nov 3, 2022

Thanks @jmichellehu . Can you provide more infromation on the issues for the modis tile list? We just need an index of the modis tiles, which I downloaded and stored as a dictionary here: https://github.com/dshean/demcoreg/blob/master/demcoreg/modis_grid.py. We could likely find a geojson for the modis grid index and read on the fly.

I'm not thrilled about hardcoding to CONUS tiles, as this tool currently works globally.

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

Successfully merging this pull request may close these issues.

2 participants