-
Notifications
You must be signed in to change notification settings - Fork 4
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.
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
Infer shape of source space from target space #12
Conversation
Hi @kclamar, thanks for the PR! |
Hi @vigji, Thanks for the response! In my workflow I save the coordinates of points in the "sal" space and sometimes I need to convert the coordinates to the space of the mpin_zfish_1um atlas. When I initialize the source space for the conversion I need to calculate the shape of the atlas stack after transforming to the source space. I think it'd be convenient to have this new feature since the atlas space already contains the shape information. I also see why it could be dangerous, so I think that having a keyword argument for this feature and making it not the default behavior would be a good option. |
… coordinates transformation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Awesome!
This is good to go for me, would you mind just adding a little test for your addition in the tests module so that coverage is not decreased? It should be easy by looking at the rest of the tests in the module, but I can give more pointers if needed!
I've added a test with axes flipping and reordering. |
Awwwwwesome! Thank you very much @kclamar ! |
will release a new version with this |
Hi,
I'm wondering if it's possible to allow transformations of coordinates when the target space has a shape but the source space doesn't?