Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ST_Overlaps gives the error answer #45

Open
cuteDen-ECNU opened this issue Mar 4, 2024 · 1 comment
Open

ST_Overlaps gives the error answer #45

cuteDen-ECNU opened this issue Mar 4, 2024 · 1 comment

Comments

@cuteDen-ECNU
Copy link
Owner

cuteDen-ECNU commented Mar 4, 2024

Considering the following statement:

FORCE INSTALL spatial FROM 'http://nightly-extensions.duckdb.org';
LOAD spatial;
SELECT ST_Overlaps(g1, g2)
  FROM (VALUES
  ('MULTILINESTRING((14 71,92 96),(6 25,63 63,6 25),(71 55,25 87,7 18,34 26,15 55))'::geometry, 
   'LINESTRING(6 25,63 63,6 25)'::geometry)
  ) AS sdb(g1, g2);

--expected{false}; actual{true}

According to the definition of ST_Overlaps:

Two geometries overlap if they have the same dimension, their interiors intersect in that dimension. and each has at least one point inside the other (or equivalently, neither one covers the other).

g1 covers g2, which violates "neither one covers the other", thus, they do not overlap each other.

Besides, after updating the geos version, the result of the above statement is false, which makes me believe it is a solved logic bug in goes.

Version:
Spatial version:
FORCE INSTALL spatial FROM 'http://nightly-extensions.duckdb.org/'; and LOAD spatial;

DuckDB version:

┌─────────────────┬────────────┐
│ library_version │ source_id  │
│     varchar     │  varchar   │
├─────────────────┼────────────┤
│ v0.10.0         │ 20b1486d11 │
└─────────────────┴────────────┘
@cuteDen-ECNU
Copy link
Owner Author

bugtracker link: duckdb/duckdb-spatial#274

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant