Skip to content

Commit e507001

Browse files
Fold the core tnpoint type port (#150)
Brings the temporal network point (tnpoint) type into the accumulate: construction, text/EWKT/MFJSON I/O, casts, and accessors, registered in LoadInternal after the tcbuffer surface. The network model resolves route geometries from a ways CSV; MEOS hardcodes its path (/usr/local/share/ways1000.csv) with no setter, so the canonical ways1000.csv is embedded (ways_csv.inc) and materialized there on load (best-effort). Fixes applied on top of the #150 branch: - getValue/startValue/endValue value executors renamed to tnpoint-unique names (Tnpoint_{get,start,end}_value). Generic names (Tinstant_value / Temporal_start_value / Temporal_end_value) are also defined at namespace scope in the other geo .cpp — an ODR violation that would otherwise dispatch these to the surviving (geometry) definition and render the npoint as a geometry ("Unknown geometry type: 0"). - Refresh the asEWKT test expecteds with the ways-network SRID prefix (SRID=5676;), matching MobilityDB's asEWKT(npoint) reference; asText omits it. Full sqllogictest suite green: 1799 assertions across 78 test cases. (cherry picked from commit 94e01ad) (cherry picked from commit a0c2f9e)
1 parent 3ca2613 commit e507001

9 files changed

Lines changed: 12430 additions & 0 deletions

File tree

CMakeLists.txt

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -108,6 +108,14 @@ if(CBUFFER)
108108
src/geo/tcbuffer_in_out.cpp)
109109
endif()
110110

111+
option(NPOINT "Build the tnpoint temporal type" ON)
112+
if(NPOINT)
113+
add_definitions(-DNPOINT=1)
114+
list(APPEND EXTENSION_SOURCES
115+
src/geo/tnpoint.cpp
116+
src/geo/tnpoint_in_out.cpp)
117+
endif()
118+
111119
build_static_extension(${TARGET_NAME} ${EXTENSION_SOURCES})
112120
build_loadable_extension(${TARGET_NAME} "" ${EXTENSION_SOURCES})
113121

benchmark/BerlinMOD/results/output/query_1.csv

Whitespace-only changes.

benchmark/BerlinMOD/results/output/query_2.csv

Whitespace-only changes.

0 commit comments

Comments
 (0)