-
Notifications
You must be signed in to change notification settings - Fork 240
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
Discussion #489: add CopyAffine transform #584
Conversation
merci alban |
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.
Thanks for your PR! I added a comment. Could you please update the corresponding rst file in the documentation?
Codecov Report
@@ Coverage Diff @@
## master #584 +/- ##
==========================================
- Coverage 96.83% 96.79% -0.05%
==========================================
Files 122 124 +2
Lines 6358 6401 +43
==========================================
+ Hits 6157 6196 +39
- Misses 201 205 +4
Continue to review full report at Codecov.
|
Hi, I am not sure to understand what the error is, |
I think the error is happening because of a Sphinx problem that was fixed in 6e61447, before this branch was created. Don't worry about it. I'll take a look at this PR tonight or tomorrow. |
Thanks for your contribution, @Albans98! @all-contributors please add @Albans98 for code |
I've put up a pull request to add @Albans98! 🎉 |
Thanks for your time and your reviews ! |
@Albans98 just find out, there is one line missing at the end of the init self.args_names = ['target'] |
That's my fault, I removed it thinking it wasn't necessary. It's needed to reproduce the transform, right? |
may be @Albans98 can confirm, I don't know what the exact problem was, but anyway It make sens to have it for all the transform (preprocessing and augmentation) |
Hello, yes the problem is when we try to save the history of all applied transforms, we get an error 'TypeError: init() missing 1 required positional argument: 'target''. Adding the target in the list of args makes it possible to trace back to the 'target' value. |
Ok, let me fix that. |
Done. Thanks for pointing that out! |
Thank you for fixing this ! |
Fixes #489 .
Description
Add a CopyAffine transform with a target_key parameter to copy the affine from this key to all LabelMap objects of a given torchio subject.
Checklist
CONTRIBUTING
docs and have a developer setup (especially important arepre-commit
andpytest
)pytest
make html
inside thedocs/
folder