Skip to content

Inverse transformation#1663

Closed
rijobro wants to merge 99 commits intoProject-MONAI:masterfrom
rijobro:inverse_transformation
Closed

Inverse transformation#1663
rijobro wants to merge 99 commits intoProject-MONAI:masterfrom
rijobro:inverse_transformation

Conversation

@rijobro
Copy link
Copy Markdown
Contributor

@rijobro rijobro commented Feb 26, 2021

Main PR for inverse transformations.

Try to keep changes to __call__ minimal. Just append the current transformation to a list of transformations. And then add an inverse method to spatial transformations.

I also needed to return some extra information from certain __call__'s, for example affine matrices. backwards compatibility should be ensured by defaulting the option to return this to False.

(After this there is just batch inversion and test time augmentation!)

Signed-off-by: Richard Brown <[email protected]>
@rijobro rijobro force-pushed the inverse_transformation branch from 14f1670 to d2a040b Compare March 1, 2021 13:20
rijobro added 3 commits March 1, 2021 14:20
Signed-off-by: Richard Brown <[email protected]>
Signed-off-by: Richard Brown <[email protected]>
Signed-off-by: Richard Brown <[email protected]>
@wyli wyli mentioned this pull request Mar 1, 2021
7 tasks
@rijobro rijobro force-pushed the inverse_transformation branch from 361e9a3 to 088f626 Compare March 2, 2021 18:03
rijobro added 27 commits March 16, 2021 10:28
Signed-off-by: Richard Brown <[email protected]>
Signed-off-by: Richard Brown <[email protected]>
Signed-off-by: Richard Brown <[email protected]>
Signed-off-by: Richard Brown <[email protected]>
Signed-off-by: Richard Brown <[email protected]>
Signed-off-by: Richard Brown <[email protected]>
Signed-off-by: Richard Brown <[email protected]>
Signed-off-by: Richard Brown <[email protected]>
Signed-off-by: Richard Brown <[email protected]>
Signed-off-by: Richard Brown <[email protected]>
Signed-off-by: Richard Brown <[email protected]>
Signed-off-by: Richard Brown <[email protected]>
Signed-off-by: Richard Brown <[email protected]>
Signed-off-by: Richard Brown <[email protected]>
Signed-off-by: Richard Brown <[email protected]>
Signed-off-by: Richard Brown <[email protected]>
Signed-off-by: Richard Brown <[email protected]>
Signed-off-by: Richard Brown <[email protected]>
Signed-off-by: Richard Brown <[email protected]>
Signed-off-by: Richard Brown <[email protected]>
Signed-off-by: Richard Brown <[email protected]>
Signed-off-by: Richard Brown <[email protected]>
Signed-off-by: Richard Brown <[email protected]>
@wyli
Copy link
Copy Markdown
Contributor

wyli commented Mar 30, 2021

major feature merged, the remaining tasks are covered by other tickets or PRs, I'm closing this

@wyli wyli closed this Mar 30, 2021
@rijobro rijobro deleted the inverse_transformation branch May 4, 2021 10:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants