Skip to content

Fix/don't transform geometry by default#132

Merged
robitalec merged 5 commits intomainfrom
fix/dont-transform-by-default
Dec 15, 2025
Merged

Fix/don't transform geometry by default#132
robitalec merged 5 commits intomainfrom
fix/dont-transform-by-default

Conversation

@robitalec
Copy link
Copy Markdown
Member

@robitalec robitalec commented Dec 15, 2025

Costs outweigh the benefits:

  • expected behaviour from users is processing in their data's original crs
  • measures (direction, distance, centroid) are slower in some cases
  • transform only required for direction, not distance and centroid

@robitalec
fix default output_crs
9b3e6bb 
breaks expectation of transformed geometry by default
@robitalec robitalec self-assigned this Dec 15, 2025
@robitalec robitalec added priority: high type: enhancement new features, improvements labels Dec 15, 2025
@robitalec robitalec merged commit f252932 into main Dec 15, 2025
5 of 6 checks passed
@robitalec robitalec deleted the fix/dont-transform-by-default branch December 15, 2025 15:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@robitalec robitalec

Labels

priority: high type: enhancement new features, improvements

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@robitalec