Skip to content

PiP fixed probe + polygon return, suppress KNN warning, remove 12-node configs - #345

Merged
jathavaan merged 4 commits into
mainfrom
feature/341-reduce-databricks-runs
May 24, 2026
Merged

jathavaan merged 4 commits into
mainfrom
feature/341-reduce-databricks-runs

Conversation

@jathavaan

Copy link
Copy Markdown
Collaborator

Summary

Closes #303
Closes #304
Closes #295

Test plan

  • Run point-in-polygon-lookup-duckdb-small — verify returns polygon rows, not counts
  • Run point-in-polygon-lookup-postgis-small — same
  • Run point-in-polygon-lookup-local-small — returns GeoDataFrame
  • Run knn-search-local-small — no UserWarning in logs
  • Verify result_cardinality in output reflects actual polygon count

Copilot AI review requested due to automatic review settings May 24, 2026 11:29
@jathavaan
jathavaan enabled auto-merge May 24, 2026 11:32
@jathavaan jathavaan self-assigned this May 24, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@jathavaan
jathavaan merged commit a6b20ae into main May 24, 2026
23 of 24 checks passed
@jathavaan
jathavaan deleted the feature/341-reduce-databricks-runs branch May 24, 2026 11:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

2 participants