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.

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

Motion estimator class (estimator.py) could be made more generic to accommodate other data types (e.g. PET) #195

Open
mnoergaard opened this issue May 16, 2024 · 1 comment
Labels
enhancement New feature or request

Comments

@mnoergaard
Copy link
Collaborator

What would you like to see added in this software?

Currently, the EddyMotionEstimator is targeted DWI data, however, several of the code can be re-used to other data types e.g. PET. Alternatively, we can make a PETMotionEstimator (branch enh/pet_model) that can estimate the motion.

Do you have any interest in helping implement the feature?

Yes

Additional information / screenshots

No response

@mnoergaard mnoergaard added the enhancement New feature or request label May 16, 2024
@jhlegarreta
Copy link
Collaborator

👍 There is maybe a way to propose a base class with properties that both share. This is also related:
#176

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants