Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add option to modify only input transform for shifts in apply_matrix()? #499

Open
rhugonnet opened this issue Mar 28, 2024 · 0 comments · May be fixed by #531
Open

Add option to modify only input transform for shifts in apply_matrix()? #499

rhugonnet opened this issue Mar 28, 2024 · 0 comments · May be fixed by #531
Labels
enhancement Feature improvement or request

Comments

@rhugonnet
Copy link
Contributor

This is mostly code architecture, it would allow to get rid of redundant apply_rst functions for Affine classes that only perform shifts (NuthKaab, VerticalShift, etc...). And ensure the apply operation is fully consistent across affine classes.
@rhugonnet rhugonnet added the enhancement Feature improvement or request label Mar 28, 2024
@rhugonnet rhugonnet mentioned this issue Mar 28, 2024
Merged
3 tasks
@rhugonnet rhugonnet linked a pull request May 24, 2024 that will close this issue
Open
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement Feature improvement or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Rework affine transformation to be more accurate rhugonnet/xdem
1 participant
@rhugonnet