Repository navigation
feat(spatial): certify GeoArrow producer portability - #823
Conversation
|
Warning Review limit reachedYou’ve reached a temporary PR review limit under our Fair Usage Limits Policy. Next review available in: 9 minutes Limit details: You’ve used all 2 included reviews currently available. Your 53 included PR review attempts over the past 7 days set your current allowance at 2 reviews per hour. Enable usage-based reviews in Billing to review now. Otherwise, wait until the next included review is available. How can I continue?After more reviews become available, a review can be triggered using the To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews. How do review limits work?CodeRabbit enforces per-developer PR review limits within each organization. For paid Pro and Pro+ reviews, CodeRabbit uses a developer's included PR review attempts over the past 7 days to set the current hourly allowance. At typical activity levels, the full plan allowance applies. Higher sustained activity can lower the allowance until earlier attempts leave the 7-day window. Please refer docs for additional details. Review details⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: ⛔ Files ignored due to path filters (5)
📒 Files selected for processing (21)
Comment |
Merging this PR will improve performance by 18.51%
Performance Changes
Tip Curious why this is faster? Comment Comparing |
Closes #802
Outcome
point()anddistance()computation limited to the certified CRS profile with explicit preserved-only failuresEvidence
CARGO_TARGET_DIR=/private/tmp/graphforge-802-target cargo test -p graphforge-core -p graphforge-plan -p graphforge-rel -p graphforge-storage -p graphforge-api --lib spatialCARGO_TARGET_DIR=/private/tmp/graphforge-802-target cargo test -p graphforge-rel --lib distance_rejects_preserved_only_spatial_literals_explicitlyCARGO_TARGET_DIR=/private/tmp/graphforge-802-target cargo test -p graphforge-api --test e2e_baseline certified_spatial_point_and_distance_execute_in_rust -- --exactCARGO_TARGET_DIR=/private/tmp/graphforge-802-target cargo test -p graphforge-cli --lib geoarrownode --test tests/spatial-interchange.test.mjspython crates/graphforge-bindings-py/tests/spatial_interchange.pytests/fixtures/geoarrow-v1/SHA256SUMSbazelisk test //:resource_inputsscripts/ci/cargo-bazel-drift-check.pygit diff --checkNeed help on this PR? Tag
@codesmith-botwith what you need. Autofix is disabled.