jiayuasu opened a new pull request, #2876:
URL: https://github.com/apache/sedona/pull/2876
## Summary
Issue: #2700. Milestone: 1.9.1.
The notebooks bundled in the Sedona docker image are several releases
behind. This is the first of a planned series of PRs that refresh them. It's
intentionally scoped to scaffolding so the rest of the series (vector / raster
/ STAC notebooks) can land independently on top of it.
- **Move legacy notebooks to `docs/usecases/legacy/`.** All five existing
notebooks (`ApacheSedonaCore`, `ApacheSedonaSQL`,
`ApacheSedonaSQL_SpatialJoin_AirportsPerCountry`, `ApacheSedonaRaster`,
`Sedona_OvertureMaps_GeoParquet`) now live under `docs/usecases/legacy/`.
Existing GitHub URLs continue to work; the docker image stops bundling them
because the Dockerfile's `COPY docs/usecases/*.ipynb` is non-recursive.
- **Add `00-quickstart.ipynb`.** A nine-cell, ~30-second walkthrough — read
two shapefiles, spatial join, aggregate, write GeoParquet 1.1, render a
SedonaKepler choropleth. Uses the Natural Earth data already shipped under
`docs/usecases/data/`, so no new bytes and no network are required.
- **`docker/test-notebooks.sh`:**
- Bump per-notebook timeout 600s → 900s to absorb network variance for
upcoming notebooks that hit STAC and remote object stores.
- Honour `SEDONA_NOTEBOOK_OFFLINE=1`: notebooks tagged `requires-network:
true` are skipped, so the harness still passes in sandboxed CI environments
without outbound network access. The skip count is reported in the summary.
- **`docs/tutorial/sql.md`:** redirect the two existing links from the moved
AirportsPerCountry notebook to its new `legacy/` path.
## Test plan
- [ ] `docker build -f docker/sedona-docker.dockerfile -t sedona:dev .`
succeeds.
- [ ] `docker run --rm sedona:dev /opt/sedona/docker/test-notebooks.sh`
exits 0 and reports `00-quickstart` passing.
- [ ] `docker run --rm -e SEDONA_NOTEBOOK_OFFLINE=1 sedona:dev
/opt/sedona/docker/test-notebooks.sh` still exits 0 (no network-tagged
notebooks exist yet, so the offline path is a no-op until the next PR).
- [ ] Manual smoke: `docker run -p 8888:8888 -p 8080:8080 sedona:dev`, open
`00-quickstart.ipynb` in JupyterLab, run all cells, eyeball the Kepler map.
- [ ] Existing tutorial links from `docs/tutorial/sql.md` resolve to
`docs/usecases/legacy/ApacheSedonaSQL_SpatialJoin_AirportsPerCountry.ipynb`.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
