Skip to content

Commit f3b493f

Browse files
committed
don't reference non-existent doc
1 parent 8844b1c commit f3b493f

File tree

1 file changed

+5
-9
lines changed

1 file changed

+5
-9
lines changed

mkdocs/docs/plans/Geospatial_PR_How_To_Review.md

Lines changed: 5 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -7,24 +7,20 @@ Recommended strategy: review in focused passes by concern, not file order.
77

88
## Recommended Review Order
99

10-
1. **Decisions and intended behavior**
11-
- `mkdocs/docs/dev/geospatial-types-decisions-v1.md`
12-
- Confirm the intended policy, especially Decision 9 (planar ambiguity boundary handling).
13-
14-
2. **Core geospatial utility correctness**
10+
1. **Core geospatial utility correctness**
1511
- `pyiceberg/utils/geospatial.py`
1612
- `tests/utils/test_geospatial.py`
1713
- Focus on envelope extraction, antimeridian behavior, and bound encoding formats.
1814

19-
3. **Metrics integration and write/import paths**
15+
1. **Metrics integration and write/import paths**
2016
- `pyiceberg/io/pyarrow.py`
2117
- `tests/io/test_pyarrow_stats.py`
2218
- Focus on:
2319
- geospatial bounds derived from row WKB values
2420
- skipping Parquet binary min/max for geospatial columns
2521
- partition inference not using geospatial envelope bounds
2622

27-
4. **GeoArrow compatibility and ambiguity boundary**
23+
1. **GeoArrow compatibility and ambiguity boundary**
2824
- `pyiceberg/schema.py`
2925
- `pyiceberg/io/pyarrow.py`
3026
- `tests/io/test_pyarrow.py`
@@ -33,7 +29,7 @@ Recommended strategy: review in focused passes by concern, not file order.
3329
- spherical mismatch still fails
3430
- fallback warning when GeoArrow dependency is absent
3531

36-
5. **Spatial expression surface area**
32+
1. **Spatial expression surface area**
3733
- `pyiceberg/expressions/__init__.py`
3834
- `pyiceberg/expressions/visitors.py`
3935
- `tests/expressions/test_spatial_predicates.py`
@@ -42,7 +38,7 @@ Recommended strategy: review in focused passes by concern, not file order.
4238
- visitor plumbing is complete
4339
- conservative evaluator behavior is explicit and documented
4440

45-
6. **User-facing docs**
41+
1. **User-facing docs**
4642
- `mkdocs/docs/geospatial.md`
4743
- Check limitations and behavior notes match implementation.
4844

0 commit comments

Comments
 (0)