Test + docs: classical registration toy examples for integration testing purposes about deepreg HOT 12 CLOSED

Still on track to refactor for R1 @YipengHu !

from deepreg.

@tvercaut see: https://github.com/ucl-candi/DeepReg/tree/28-classical-registration/test/sit
It looks like working fine. ;)

from deepreg.

@zacbaum can you help with converting the two examples to sit please? let me know if you want them to be merged first, though the branch has just been rebased - so should be no big diff.

from deepreg.

@zacbaum can you help with converting the two examples to sit please? let me know if you want them to be merged first, though the branch has just been rebased - so should be no big diff.

I've PR anyway ;) please review/approve and you can start a new branch for it.

from deepreg.

@zacbaum @NMontanaBrown if adding SITs is too much for R1, I can convert this to a "special demo". What do you think?

from deepreg.

@YipengHu Let me review the test and get back to you

from deepreg.

@YipengHu Let me review the test and get back to you

these two are just examples, there are a few others you can add to cover other functions/modules, such as affine registration, registration on labels/features.

from deepreg.

@YipengHu This is cool, but the test works by looking at the plotted output? I think this is easily modifiable. We can incorporate this after unit tests, I will add to timeline if that makes sense?

from deepreg.

Good idea. We have to streamline, if not required by joss, not required for R1.
And it is a cool idea for sit!

from deepreg.

@NMontanaBrown @zacbaum what's our plan for this? if not before R1, please add post-r1 label.

from deepreg.

@zacbaum @NMontanaBrown
see #193 we will move some of the code for purpose of one or two demos. we will discuss it again how to make this sit again

• potentially post-r1

from deepreg.

closing this as they have been moved to demos.

from deepreg.