Add the core tpcpoint type port

.github/PULL_REQUEST_TEMPLATE.md

@@ -0,0 +1,14 @@
+<!--
+👀 Reviewers: this project's tier ranking, dependency chains and the
+standards checklist live in:
+  doc/contributing/reviewer-guide.md
+PR authors: update that file in the same commit as any PR queue change.
+-->
+
+## Summary
+
+<!-- What this PR does, in 1–3 bullets. Focus on the "why". -->
+
+## Test plan
+
+<!-- - [ ] Bullet checklist of TODOs for verifying this PR -->

.github/workflows/MainDistributionPipeline.yml

@@ -36,12 +36,14 @@ jobs:
       extension_name: mobilityduck
       ci_tools_version: v1.4.4
       vcpkg_commit: c27eeddba73f608f10605d80bc0144c1166f8fb7
-      # Windows is excluded because the MEOS vcpkg port does not currently
-      # build under MSVC or MinGW: it pulls in PostgreSQL-derived sources
-      # that depend on POSIX-only headers (e.g. <dirent.h>) and GCC-only
-      # attribute syntax (`__attribute__((unused))`). Re-enable once the
-      # MEOS port grows Windows support.
-      exclude_archs: windows_amd64;windows_amd64_mingw;linux_amd64_musl
+      # Windows / linux_amd64_musl / wasm_* are excluded because the
+      # MEOS vcpkg port does not currently build there: Windows pulls in
+      # POSIX-only headers (e.g. <dirent.h>) and GCC-only attribute syntax,
+      # and the Wasm/emscripten targets fail to compile PostgreSQL's port
+      # code (`pg_bitutils.h` cannot pick an integer type matching
+      # `uint64_t` under the emscripten ABI).  Re-enable once the MEOS
+      # port grows targets for those toolchains.
+      exclude_archs: windows_amd64;windows_amd64_mingw;linux_amd64_musl;osx_arm64;wasm_mvp;wasm_eh;wasm_threads
 
   duckdb-latest-deploy:
     needs: duckdb-latest-build
@@ -52,9 +54,11 @@ jobs:
       ci_tools_version: v1.4.4
       extension_name: mobilityduck
       deploy_latest: ${{ startsWith(github.ref, 'refs/heads/v') || github.ref == 'refs/heads/main' }}
-      # Windows is excluded because the MEOS vcpkg port does not currently
-      # build under MSVC or MinGW: it pulls in PostgreSQL-derived sources
-      # that depend on POSIX-only headers (e.g. <dirent.h>) and GCC-only
-      # attribute syntax (`__attribute__((unused))`). Re-enable once the
-      # MEOS port grows Windows support.
-      exclude_archs: windows_amd64;windows_amd64_mingw;linux_amd64_musl
+      # Windows / linux_amd64_musl / wasm_* are excluded because the
+      # MEOS vcpkg port does not currently build there: Windows pulls in
+      # POSIX-only headers (e.g. <dirent.h>) and GCC-only attribute syntax,
+      # and the Wasm/emscripten targets fail to compile PostgreSQL's port
+      # code (`pg_bitutils.h` cannot pick an integer type matching
+      # `uint64_t` under the emscripten ABI).  Re-enable once the MEOS
+      # port grows targets for those toolchains.
+      exclude_archs: windows_amd64;windows_amd64_mingw;linux_amd64_musl;osx_arm64;wasm_mvp;wasm_eh;wasm_threads

.gitignore

@@ -7,4 +7,5 @@ testext
 test/python/__pycache__/
 .Rhistory
 vcpkg/
-*.log
+*.log
+examples/quickstart/*.parquet

CMakeLists.txt

@@ -37,6 +37,25 @@ endif()
 # MEOS from overlay port
 find_package(MEOS CONFIG REQUIRED)
 
+# h3 — transitively needed because `meos_h3.h` includes `<h3api.h>`.
+# vcpkg's h3 port installs the header at `include/h3/h3api.h` (under
+# a subdirectory).  `find_package(h3 CONFIG)` finds the import target
+# but its `INTERFACE_INCLUDE_DIRECTORIES` only contains `include/`,
+# not `include/h3/`.  Probe the header directly with `find_path` and
+# add the resolved directory to the global include path so the
+# `#include <h3api.h>` in `meos_h3.h` resolves.
+find_package(h3 CONFIG)
+if(h3_FOUND)
+    message(STATUS "Found h3 (CONFIG): version ${h3_VERSION}")
+endif()
+find_path(H3_INCLUDE_DIR NAMES h3api.h PATH_SUFFIXES h3)
+if(H3_INCLUDE_DIR)
+    include_directories(${H3_INCLUDE_DIR})
+    message(STATUS "Found h3 include dir: ${H3_INCLUDE_DIR}")
+else()
+    message(WARNING "h3api.h not found; the th3index extension surface will fail to compile")
+endif()
+
 if(TARGET GEOS::geos_c)
   set(GEOS_TGT GEOS::geos_c)
 elseif(TARGET GEOS::geos)
@@ -63,12 +82,22 @@ set(EXTENSION_SOURCES
     src/temporal/temporal_aggregates.cpp
     src/temporal/tbox.cpp
     src/temporal/tbox_functions.cpp
+    src/geo/geography.cpp
+    src/geo/geography_functions.cpp
     src/geo/stbox.cpp
     src/geo/stbox_functions.cpp
     src/geo/tgeompoint.cpp
     src/geo/tgeompoint_functions.cpp
     src/geo/tgeogpoint.cpp
     src/geo/tgeogpoint_functions.cpp
+    src/geo/tcbuffer.cpp
+    src/geo/tcbuffer_in_out.cpp
+    src/geo/tnpoint.cpp
+    src/geo/tnpoint_in_out.cpp
+    src/geo/tpose.cpp
+    src/geo/tpose_in_out.cpp
+    src/rgeo/trgeometry.cpp
+    src/rgeo/trgeometry_in_out.cpp
     src/temporal/set.cpp
     src/temporal/set_functions.cpp
     src/temporal/span.cpp
@@ -86,14 +115,18 @@ set(EXTENSION_SOURCES
     src/geo/tgeography.cpp
     src/geo/tgeography_in_out.cpp
     src/geo/tgeography_ops.cpp
+    src/pointcloud/tpcpoint.cpp
+    src/pointcloud/tpcpoint_in_out.cpp
     src/geo/tgeogpoint.cpp
     src/geo/tgeogpoint_in_out.cpp
     src/geo/tgeogpoint_ops.cpp
+    src/h3/th3index.cpp
     src/index/rtree_module.cpp
     src/single_tile_getters.cpp
     src/index/rtree_index_create_physical.cpp
     src/index/rtree_index_scan.cpp
     src/index/rtree_optimize_scan.cpp
+    src/temporal/temporal_parquet.cpp
 )
 
 build_static_extension(${TARGET_NAME} ${EXTENSION_SOURCES})
@@ -113,13 +146,18 @@ endif()
 # -----------------------------
 # Link libraries 
 # -----------------------------
+if(TARGET h3::h3)
+    set(H3_TGT h3::h3)
+endif()
+
 target_link_libraries(${EXTENSION_NAME}
   MEOS::meos
   ${GEOS_TGT}
   PROJ::proj
   GSL::gsl
   GSL::gslcblas
   ${JSONC_TGT}
+  ${H3_TGT}
   OpenSSL::SSL
   OpenSSL::Crypto
 )
@@ -131,6 +169,7 @@ target_link_libraries(${LOADABLE_EXTENSION_NAME}
   GSL::gsl
   GSL::gslcblas
   ${JSONC_TGT}
+  ${H3_TGT}
   OpenSSL::SSL
   OpenSSL::Crypto
 )

Makefile

@@ -11,9 +11,52 @@ include extension-ci-tools/makefiles/duckdb_extension.Makefile
 # both MEOS (meos_initialize_timezone) and DuckDB (DBConfig::SetOptionByName
 # "TimeZone") to Europe/Brussels.  Tests pass on any OS timezone — the
 # extension is the single source of truth, no TZ env var needed.
+#
+# LoadInternal also calls ExtensionHelper::AutoLoadExtension(db, "icu") so
+# the timezone option is honoured. Autoload looks for the extension on disk
+# at $HOME/.duckdb/extensions/<duckdb_version>/<platform>/icu.duckdb_extension
+# and falls back to a hub download. That fails both inside the linux_amd64
+# test docker container (empty path, no network egress) and on the macOS
+# osx_arm64 test runner (hub icu not reliably resolvable). We copy the
+# icu.duckdb_extension that was built locally as part of this extension's
+# build (declared in extension_config.cmake) into the expected path.
+#
+# Target DuckDB is the v1.4.x LTS line, with later versions (v1.5.x) supported
+# in a multi-version matrix (PRs #166/#167) the same way MobilityDB supports
+# PostgreSQL 13-18 — so the staging path must NOT hardcode the version or the
+# platform. We derive both from the freshly-built duckdb binary (authoritative
+# for whatever version/platform is actually being tested); DUCKDB_VERSION_TAG
+# and the uname map below are kept only as fallbacks if that query is
+# unavailable.
+DUCKDB_VERSION_TAG := v1.4.4
+
+define stage_icu
+	@if [ -f ./build/$(1)/extension/icu/icu.duckdb_extension ]; then \
+	  duckdb_bin=./build/$(1)/duckdb; \
+	  version_tag=$$( [ -x "$$duckdb_bin" ] && "$$duckdb_bin" --version 2>/dev/null | grep -oE 'v[0-9]+\.[0-9]+\.[0-9]+' | head -1 ); \
+	  platform=$$( [ -x "$$duckdb_bin" ] && echo 'PRAGMA platform;' | "$$duckdb_bin" -noheader -list 2>/dev/null | tr -d '[:space:]' ); \
+	  [ -n "$$version_tag" ] || version_tag=$(DUCKDB_VERSION_TAG); \
+	  if [ -z "$$platform" ]; then \
+	    case "$$(uname -s)-$$(uname -m)" in \
+	      Linux-x86_64)  platform=linux_amd64 ;; \
+	      Linux-aarch64) platform=linux_arm64 ;; \
+	      Darwin-arm64)  platform=osx_arm64 ;; \
+	      Darwin-x86_64) platform=osx_amd64 ;; \
+	      *)             platform=$$(uname -m) ;; \
+	    esac; \
+	  fi; \
+	  target=$$HOME/.duckdb/extensions/$$version_tag/$$platform; \
+	  mkdir -p "$$target" && cp -f ./build/$(1)/extension/icu/icu.duckdb_extension "$$target/" && \
+	  echo "Staged icu.duckdb_extension at $$target/ (duckdb $$version_tag / $$platform)"; \
+	fi
+endef
+
 test_release_internal:
+	$(call stage_icu,release)
 	./build/release/$(TEST_PATH) "$(PROJ_DIR)test/*"
 test_debug_internal:
+	$(call stage_icu,debug)
 	./build/debug/$(TEST_PATH) "$(PROJ_DIR)test/*"
 test_reldebug_internal:
-	./build/reldebug/$(TEST_PATH) "$(PROJ_DIR)test/*"
+	$(call stage_icu,reldebug)
+	./build/reldebug/$(TEST_PATH) "$(PROJ_DIR)test/*"

README.md

@@ -21,6 +21,12 @@ MobilityDuck because of properties of DuckDB's parser, type system, or
 extension model. These cases — and the named-function workarounds where one
 exists — are documented in [`docs/DuckDB-Parity-Gaps.md`](docs/DuckDB-Parity-Gaps.md).
 
+Geodetic geography values cross the DuckDB columnar boundary via a dedicated
+`GEOGRAPHY` LogicalType that MobilityDuck registers in addition to the bundled
+DuckDB Spatial `GEOMETRY`. The design — closed-algebra inside MEOS, thin
+boundary layer in the binding, and TemporalParquet round-trip preservation of
+the geodetic flag — is documented in [`doc/geography-boundary.md`](doc/geography-boundary.md).
+
 ---
 ## 1. Requirements
 MobilityDuck needs some dependencies(including MEOS) which can be installed through VCPKG. Run the following to enable it: 

doc/contributing/reviewer-guide.md

@@ -0,0 +1,119 @@
+<!--
+Copyright(c) MobilityDB Contributors
+
+This documentation is licensed under a
+Creative Commons Attribution-Share Alike 3.0 License
+https://creativecommons.org/licenses/by-sa/3.0/
+-->
+
+# MobilityDuck PR Reviewer Guide
+
+Quick reference for anyone reviewing open pull requests.
+Updated in the same commit as any PR that changes PR state or adds new branches.
+**Last updated: 2026-05-10 — 22 open PRs (net consolidation today): folds #115+#119 → #120, #116+#118 → #121, #122+#123+#124+#125 → #126.  Squashed in place: #117, #111.  PR #112 TZ-neutral test fix landed (1 commit).**
+
+---
+
+## How to find this guide
+
+- **In the repo:** `doc/contributing/reviewer-guide.md`
+- **Rule:** every commit that opens, closes, or restructures a PR must update this file in the same commit. A one-liner status change is enough; a fuller rewrite is needed when the dependency graph changes.
+
+---
+
+## CI legend
+
+| Symbol | Meaning |
+|--------|---------|
+| ✅ | All checks green |
+| ❌ | Real failure — needs investigation before review |
+| ⏳ | CI running |
+| ❓ | No CI result yet (doc-only, draft, or external PR) |
+| ⚠️ | Non-blocking failure (e.g. macOS/Windows `continue-on-error`, Codacy ACTION_REQUIRED — maintainer overrides in UI) |
+
+---
+
+## Dependency chains — land in this order
+
+```
+#121  consolidate/main-hygiene-batch  (revert single-tz + sync MEOS API drift; subsumes #116+#118)
+  └─► #111  fix/per-thread-meos-init  (thread-safety foundation)
+        └─► #58   fix/splitnspans-spanset-result + aggregates batch (12 commits)
+        └─► #61   fix/geomset-srid-parameter
+        └─► #106  fix/span-arithmetic-correctness
+        └─► #107  fix/span-distance-return-types
+              └─► consolidate/ batch (#97–#105)     (parity surface — independent of each other)
+                    └─► #108  feat/parity-math-similarity-tbox
+                    └─► #109  feat/parity-elevation-restrict
+                    └─► #110  feat/parity-split-complement
+                    └─► #112  feat/wkb-roundtrip-all-types  (TZ-neutral test fix landed)
+                    └─► #113  feat/edge-to-cloud-quickstart
+                    └─► #114  feat/berlinmod-geo-functions2
+                    └─► #126  feat/parity-additions-batch    (bearing + covers + stbox-dim + seqSetGaps; subsumes #122+#123+#124+#125)
+                    └─► #120  consolidate/pr-coordination-and-tz-lint (subsumes #115+#119)
+                    └─► #117  doc/reviewer-guide   (visibility wiring; cross-repo with MobilityDB #931 / MobilitySpark #8)
+```
+
+**#121 should land first** — it reverts single-timezone state on `main` and syncs MEOS API drift, unblocking every other branch from local-build failures.  Then **#111** (per-thread MEOS init) is the thread-safety foundation all parity work builds on.
+
+The **consolidate/** PRs (#97–#105) are independent parity drops covering different function families; they can land in any order once the four correctness fixes (#58, #61, #106, #107) and #111 are in.
+
+---
+
+## Tier 1 — Merge immediately (bug fixes + visibility, trivially reviewable)
+
+| PR | Branch | Description | CI |
+|----|--------|-------------|----|
+| #117 | `doc/reviewer-guide` | **This guide** — PR review ordering, tiers, dependency chains; visibility wiring (PR template + README banner) | ✅ |
+| #121 | `consolidate/main-hygiene-batch` | Revert single-tz on main + sync MEOS API drift (subsumes #116+#118); unblocks every other branch | ✅ |
+| #120 | `consolidate/pr-coordination-and-tz-lint` | docs/PR-COORDINATION.md + scripts/lint-tz-pinned-tests.py (subsumes #115+#119) | ✅ |
+| #111 | `fix/per-thread-meos-init` | Replace global MEOS mutex with per-thread initialization | ✅ |
+| #107 | `fix/span-distance-return-types` | Fix distance return types; add `+` / `shift` alias for tstzset/tstzspan+interval | ✅ |
+| #106 | `fix/span-arithmetic-correctness` | Fix SpanSet serialization size and floatspan distance datum | ✅ |
+| #61 | `fix/geomset-srid-parameter` | `set(LIST(GEOMETRY), INTEGER)` overload — explicit SRID | ✅ |
+| #58 | `fix/splitnspans-spanset-result` | `splitNspans` fix on spanset + 10 aggregate-additions (12-commit rolling-topic, scope-creep tolerated) | ✅ |
+
+---
+
+## Tier 2 — Parity surface — consolidate/ batch (independent, all CI green)
+
+These cover different function families and can land in any order.
+
+| PR | Branch | Description | CI |
+|----|--------|-------------|----|
+| #105 | `consolidate/docs` | CONTRIBUTING.md + PARITY.md user guide + PARITY-INVENTORY.md | ✅ |
+| #104 | `consolidate/geo-types-parity` | tgeometry + tgeography + tgeogpoint — full parity surface | ✅ |
+| #100 | `consolidate/analytics-parity` | Temporal analytics parity — simplify / similarity / tnumber math | ✅ |
+| #98 | `consolidate/spatial-predicates-parity` | tspatial predicates parity — topological / comparison / position | ✅ |
+| #97 | `consolidate/temporal-ops-parity` | Temporal ops parity — boxops / comparison / position / precision / same | ✅ |
+| #103 | `consolidate/aggregates-parity` | Aggregate functions parity — extent / SkipList aggregates / tCentroid | ❌ |
+| #102 | `consolidate/tiles-bins-parity` | Tile and bin functions parity — emitters / table functions / getters | ❌ |
+| #99 | `consolidate/tgeompoint-ops-parity` | tgeompoint operations parity — distance / affine / transforms / geoMeasure | ❌ |
+
+---
+
+## Tier 3 — Recent feature additions (land after consolidate/ batch)
+
+| PR | Branch | Description | CI | Notes |
+|----|--------|-------------|----|----|
+| #126 | `feat/parity-additions-batch` | bearing + eCovers/tCovers + stbox dim + seqSetGaps (subsumes #122+#123+#124+#125) | ✅ | |
+| #114 | `feat/berlinmod-geo-functions2` | nearestApproachDistance, expandSpace, `&&` for TGEOMPOINT | ✅ | |
+| #113 | `feat/edge-to-cloud-quickstart` | Edge-to-cloud quickstart, temporalFooter(), SRID/geodetic fix, tgeogpoint tests | ✅ | |
+| #110 | `feat/parity-split-complement` | timeSplit / valueSplit / quadSplit emitters | ✅ | |
+| #109 | `feat/parity-elevation-restrict` | atElevation / minusElevation via public MEOS primitives | ✅ | |
+| #108 | `feat/parity-math-similarity-tbox` | Unskip tnumber math, tbox, and similarity parity tests | ✅ | |
+| #112 | `feat/wkb-roundtrip-all-types` | Complete binary + hex-WKB round-trip I/O for all types (TZ-neutral test fix landed) | ✅ | |
+
+---
+
+## Review checklist
+
+For every MobilityDuck PR, verify:
+
+- [ ] PostgreSQL License header on every new `.cpp` / `.hpp` file
+- [ ] New function registered in the correct `RegisterXxx()` function
+- [ ] SQL name matches MobilityDB alias (RFC #861 portable SQL contract)
+- [ ] NULL input handled (returns NULL or appropriate default)
+- [ ] DBL\_MAX sentinel from MEOS mapped to NULL for distance functions
+- [ ] New parity test added in `test/` with `nosort` tag where result order is non-deterministic
+- [ ] CI green before requesting merge (fix ❌ PRs in-branch, not in a follow-up)