Skip to content

Name every facade method by the class the catalog puts it in - #35

Merged
estebanzimanyi merged 1 commit into
MobilityDB:mainfrom
estebanzimanyi:fix/facade-references-follow-the-object-model
Sep 12, 2026
Merged

estebanzimanyi merged 1 commit into
MobilityDB:mainfrom
estebanzimanyi:fix/facade-references-follow-the-object-model

Conversation

@estebanzimanyi

Copy link
Copy Markdown
Member

The MeosOps* facades are grouped by the catalog's object model, which gives a
class to every base value MEOS passes by reference: geom_in belongs to
Geometry, the cbuffer, npoint and pose constructors and accessors to Cbuffer,
Npoint and Pose, geo_as_text to Geo. The smoke tests call those methods on the
free-function classes, MeosOpsFreeGeo, MeosOpsFreeCbuffer, MeosOpsFreeNpoint and
MeosOpsFreePose, which hold none of them, so the binding module's tests do not
compile against facades generated from the current catalog. Each call names
the class that holds the method.

The wiring javadoc names methods the same way: edwithin_tgeo_tgeo is on
MeosOpsFreeGeo, and the windowed-tier examples use tpoint_length and
tnumber_twavg, the MEOS functions for a trajectory's length and a temporal
number's time-weighted average.

The MeosOps* facades are grouped by the catalog's object model, which gives a
class to every base value MEOS passes by reference: geom_in belongs to
Geometry, the cbuffer, npoint and pose constructors and accessors to Cbuffer,
Npoint and Pose, geo_as_text to Geo. The smoke tests call those methods on the
free-function classes, MeosOpsFreeGeo, MeosOpsFreeCbuffer, MeosOpsFreeNpoint and
MeosOpsFreePose, which hold none of them, so the binding module's tests do not
compile against facades generated from the current catalog. Each call names
the class that holds the method.

The wiring javadoc names methods the same way: edwithin_tgeo_tgeo is on
MeosOpsFreeGeo, and the windowed-tier examples use tpoint_length and
tnumber_twavg, the MEOS functions for a trajectory's length and a temporal
number's time-weighted average.
@estebanzimanyi
estebanzimanyi merged commit 4565c97 into MobilityDB:main Sep 12, 2026
2 checks passed
estebanzimanyi added a commit that referenced this pull request Sep 12, 2026
Run every MEOS test in every build (stacked on #35)
@estebanzimanyi
estebanzimanyi deleted the fix/facade-references-follow-the-object-model branch September 12, 2026 14:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant