Add GeoTIFF dataset health report and auto-rebuild stale summary index#99
Open
chourmovs wants to merge 21 commits into
Open
Add GeoTIFF dataset health report and auto-rebuild stale summary index#99chourmovs wants to merge 21 commits into
chourmovs wants to merge 21 commits into
Conversation
…t-updates Fix Dockerfile, move extracted SRTM TIFFs, and update Python requirements
…t-updates-hnhz1z Fix GDAL Docker base tag, add GDAL deps & TZ, update requirements, and move extracted SRTM TIFFs
…docker-image Pin GDAL base image to osgeo/gdal:ubuntu-small-3.6.3
…ocker Remove GDAL from `requirements.txt` to avoid pip build failure
…script-if-/data-is-empty Add Docker entrypoint to auto-initialize dataset when /code/data is empty
…/data-is-empty-hyd8t3
…script-if-/data-is-empty-hyd8t3 Add Docker entrypoint with `INITIALIZE_DATASET` control and update docs
…script-if-/data-is-empty-98scan Add Docker entrypoint for dataset initialization; improve logging and spatial index lookup
…-tiff-files Add GeoTIFF dataset health checks and startup logging
…-tiff-files-cgatmx Add dataset health report and auto-rebuild stale summary on startup
There was a problem hiding this comment.
3 issues found across 8 files
Prompt for AI agents (unresolved issues)
Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.
<file name="server.py">
<violation number="1" location="server.py:10">
P1: Unvalidated `LOG_LEVEL` environment variable can crash the server at startup with a `ValueError` for unknown logging levels.</violation>
</file>
<file name="docker/Dockerfile">
<violation number="1" location="docker/Dockerfile:9">
P1: Installing Ubuntu GDAL packages on top of the OSGeo GDAL image mixes different GDAL versions (3.6.3 vs 3.4.1), which can cause ABI/runtime mismatches for the Python bindings and CLI tools.</violation>
</file>
<file name="gdal_interfaces.py">
<violation number="1" location="gdal_interfaces.py:184">
P1: Changing lookup to strict intersection can now error on coordinates outside tile coverage instead of returning the documented sea-level fallback, and overlap/border matches are selected arbitrarily.</violation>
</file>
Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review.
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.
Tested as is on coolify 4.0.0
add 2 env variable
[https://github.com/chourmovs/open-elevation]
Summary by cubic
Add a GeoTIFF dataset health check and automatically rebuild a stale
summary.jsonindex on startup. Also add an opt-in dataset initialization flow for Docker and make host port configurable.New Features
GDALTileInterface.dataset_health_report()reports.tifcount, total size (bytes/MB), and a sample file list..tiffiles are found..tiffiles exist but the index has 0 tiles, force-rebuildsummary.json./code/create-dataset.shwhenINITIALIZE_DATASET=yes; docs updated.docker-composesupportsHOST_PORTto override the host port.Dependencies
osgeo/gdal:ubuntu-small-3.6.3; installlibgdal-devandgdal-bin.GDAL; addednumpy,Pillow,Rtree(keptbottle,gunicorn,lazy).Written for commit 8f2051d. Summary will update on new commits.