nipy / nitransforms

a standalone fork of nipy/nibabel#656
https://nitransforms.readthedocs.io
MIT License
28 stars 15 forks source link

ENH: Outsource ``apply()`` from transform objects #195

Closed jmarabotto closed 2 months ago

jmarabotto commented 3 months ago

I have joined CHUV (Lausanne, Switzerland) for a 5 month placement and am working with Oscar Esteban on this project. We worked through this problem together as an example of how I can contribute and resolve issues, as I am somewhat new to GitHub. Eventually, this Resolves #192 - but for now, this acts as an example and further changes are needed :)

effigies commented 3 months ago

I assume this is intended to close https://github.com/nipy/nitransforms/issues/192? If so, please add "Resolves #192" or similar to your top post.

oesteban commented 2 months ago

Okay, with #197 out of the way we can focus on finalizing this. @effigies -- I'm working with Julien on this one, I will walk him through the process of preparing the PR we will let you know when this is ready for you to eyeball it.