Skip to content

Conversation

@Kontinuation
Copy link
Member

This is the final step of the forked dependency elimination plan: https://github.com/apache/sedona-db/pull/165/files. sedona-geo-generic-alg will go live and replace the original wherobots/geo dependency. We also replace forked wkb and wkt dependencies with the latest release versions.

This patch depends on wherobots/geo-index#7 to resolve a geo version conflict.

Remaining tasks:

  • geo-index will be the only forked dependency, and we are submitting patches to put the nearest neighbour search APIs needed by sedona-db to the upstream.
  • We need a released version of georust/wkb before releasing 0.2.0.

Delete more unused code in geo-generic-alg, add more inlines to geo-traits-ext

Fix doc tests

Fix clippy

Implemented geo-traits-ext for wkb

Passing pre-commit and adding licence to readme

Implemented fast wkb_to_geos

Now everything compiles, but some tests are failing

Clean up

Add Apache license headers to Rust source files

- Applied Apache license headers to 63 Rust source files in geo-generic-alg and geo-traits-ext crates
- Added license_ignore.txt to exclude .wkt test fixture files from license checks
- Fixes RAT (Release Audit Tool) violations for packaging workflow

Fix geo overlay tests

Switch git repo of forked dependencies

Switch from geo to geo-types for dev dependency sedona-geos

Add sedona- prefix to geo-traits-ext, geo-generic-alg and geo-test-fixtures

Add assert_scalar_equal_wkb_geometry_topologically to compare wkbs topologically

Add more geometry fixtures for testing sedona-geo-generic-alg (will be added later)

Removing sedona-geo-test-fixtures and move fixtures to sedona-testing

Update sedona-geo-traits-ext and sedona-geo-generic-alg
@Kontinuation Kontinuation marked this pull request as ready for review October 9, 2025 22:42
Copy link
Member

@paleolimbot paleolimbot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

@Kontinuation Kontinuation merged commit 95c156d into apache:main Oct 10, 2025
12 checks passed
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.

2 participants