Skip to content

Comments

proj transform trait#47

Closed
michaelkirk wants to merge 2 commits intofrewsxcv:masterfrom
michaelkirk:mkirk/proj-transform-trait
Closed

proj transform trait#47
michaelkirk wants to merge 2 commits intofrewsxcv:masterfrom
michaelkirk:mkirk/proj-transform-trait

Conversation

@michaelkirk
Copy link
Contributor

@michaelkirk michaelkirk commented Feb 15, 2022

bors bot added a commit to georust/proj that referenced this pull request Feb 20, 2022
109: Port transform trait from geo and add a mutable flavor r=frewsxcv a=michaelkirk

Fixes #101, #108

An alternative to #106, but as proposed in #108, I've done so by porting the [Transform trait from geo](georust/geo#718 ) to proj so that we won't need similar-but-different code in two places.

I chose to double down on a trait-based approach since it seemed idiomatic to our other georust code. However, like proposed in #106, I leveraged proj_array in more places.

/cc `@x4d3` 

corresponding PRs:
- georust/geo#730
- frewsxcv/rgis#47

Co-authored-by: Michael Kirk <michael.code@endoftheworl.de>
@frewsxcv
Copy link
Owner

done in 111929c

@frewsxcv frewsxcv closed this Feb 21, 2022
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