feat(tools): add ESRI Wayback high-resolution timelapse imagery type#84
Merged
Conversation
Adds a new ESRI Wayback imagery type that renders high-resolution timelapses from Esri World Imagery Wayback tiles without requiring Earth Engine. Updates the factory guidance, QGIS workflow prompt, and adds dispatch tests for the external-source path.
|
🚀 Deployed on https://69fd6fcd700c0c214c735356--opengeos.netlify.app |
Contributor
There was a problem hiding this comment.
Pull request overview
Adds support for generating high-resolution timelapses from Esri World Imagery Wayback tiles via the QGIS Timelapse plugin’s external-source path (avoiding Earth Engine), and wires the new option into guidance/prompting and tests.
Changes:
- Add an ESRI Wayback imagery type plus dispatch in
create_timelapse()viatimelapse.core.external_sources, with newexternal_max_framesandloopcontrols. - Update QGIS UI workflow prompts and the Timelapse system prompt guidance to recommend ESRI Wayback for high-resolution timelapses.
- Add tests covering imagery type listing and ensuring ESRI Wayback requests do not load the Earth Engine core dispatch path.
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
tests/test_timelapse_tools.py |
Adds tests for ESRI Wayback imagery type advertisement and dispatch via external sources (EE core must not load). |
qgis_geoagent/open_geoagent/dialogs/chat_dock.py |
Updates the Timelapse sample prompt to steer users toward ESRI Wayback high-resolution timelapses. |
geoagent/tools/timelapse.py |
Introduces ESRI Wayback type metadata, adds lazy external-sources loader, and routes ESRI Wayback timelapse creation through external rendering with new parameters. |
geoagent/core/factory.py |
Updates Timelapse workflow guidance to recommend ESRI Wayback and clarify EE is not required for it. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Comment on lines
+177
to
+186
| deps = reload_deps() if callable(reload_deps) else core.check_dependencies() | ||
| except Exception as exc: | ||
| raise RuntimeError( | ||
| f"Could not check Timelapse external-source dependencies: {exc}" | ||
| ) from exc | ||
| if not deps.get("Pillow", True): | ||
| raise RuntimeError( | ||
| "Timelapse external-source dependencies are not importable: " | ||
| "Pillow. Open the Timelapse settings panel and install dependencies." | ||
| ) |
| "from historical XYZ/WMTS tiles without Earth Engine." | ||
| ), | ||
| "default_start_year": 2014, | ||
| "default_date_window": ["01-01", "12-31"], |
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.
Summary
external_max_framesandloopcontrols.Test plan