Add Google Earth Engine accessor for satellite imagery (#53) - #76
Open
KarnakOza wants to merge 2 commits into
Open
Add Google Earth Engine accessor for satellite imagery (#53)#76KarnakOza wants to merge 2 commits into
KarnakOza wants to merge 2 commits into
Conversation
- Implement GoogleEarthEngineAccessor with NDVI, NDWI, NDBI, and NDBI-change methods covering the use cases in fccoelho#53 (vegetation coverage/breeding sites, urban density, informal settlement growth) - Register earthengine-api as an optional extra and add entry point so the accessor is discoverable via the standard source registry - Add tests covering initialization, list_countries, and indicator sign/range sanity checks against real-world reference points - Raise a clear ValueError when no cloud-free Landsat imagery is available for a given area/date range, instead of a cryptic band-not-found error Closes fccoelho#53
Owner
|
👋 PR Review Reminder This pull request has been open for over a week without review. Maintainers will review soon! If you need immediate feedback, feel free to tag us. Thanks for your contribution! 🙏 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
📋 Description
Adds a Google Earth Engine (GEE) accessor providing satellite-imagery-derived
indicators for epidemiological modeling, as requested in #53.
GoogleEarthEngineAccessorwithget_ndvi,get_ndwi,get_built_up_index, andget_built_up_changemethods, covering theuse cases in 🛰️ Add Google Earth Engine accessor (Satellite imagery) #53 (vegetation coverage/breeding sites, urban density,
informal settlement growth)
earthengine-apias an optional extra and add the entry pointso the accessor is discoverable via the standard source registry
list_countries, and indicatorsign/range sanity checks against real-world reference points
ValueErrorwhen no cloud-free Landsat imagery isavailable for a given area/date range, instead of a cryptic
band-not-found error
🎯 Type of Change
🔗 Related Issues
Closes #53
🧪 Testing
pytest)Test commands:
📊 Data Source Details
registration at https://signup.earthengine.google.com/, plus a linked
GEE Cloud Project (set via
EE_PROJECTenv var orproject=argument)Example usage:
📚 Documentation
✅ Checklist
Code Quality
Functionality
🌍 Impact Assessment
Who will benefit:
urban-density proxies
without a separate GEE integration
Breaking changes:
🔄 Testing Evidence
Validated NDBI sign convention against known land cover: positive for
dense urban core (São Paulo), negative for Amazon rainforest — confirming
the built-up index behaves correctly across contrasting terrain.