search

SpatioTemporal / pystare

The Python interface for the SpatioTemporal Adaptive Resolution Encoding (STARE), a unified indexing for geolocated data.
https://pystare.readthedocs.io/en/latest/
12 stars 2 forks source link

cmpTemporal does not return True based on overlap. #114

Open michaelleerilee opened 1 year ago

michaelleerilee commented 1 year ago

cmpTemporal is based on STARE's inline TemporalIndex.h::bool cmp_JulianTAIDays(const TemporalIndex& a, const TemporalIndex& b) which tests whether two tivs are within one of their neighborhoods.

Thus the following yields False.

tiv1:  inf..............tiv.........................sup
tiv2:                                            inf..............tiv.........................sup

And the following yields True.

tiv1:  inf..............tiv.........................sup
tiv2:                             inf..........tiv.........................sup

Saying that cmpTemporal returns True based on overlap is incorrect.