4587 mri transforms

[mersad95zd]

Fixes # .

Description

MRI transforms as a new feature:

• mri_transforms.py under monai.apps.reconstruction
• its unittest under tests
• its docs under transforms.rst/VanillaTransforms/MRITransforms

Status

Work in progress

Types of changes

• Non-breaking change (fix or new feature that would not break existing functionality).
• Breaking change (fix or new feature that would cause existing functionality to change).
• New tests added to cover the changes.
• Integration tests passed locally by running ./runtests.sh -f -u --net --coverage.
• Quick tests passed locally by running ./runtests.sh --quick --unittests --disttests.
• In-line docstrings updated.
• Documentation updated, tested make html command in the docs/ folder.

[mersad95zd]

I'm working on the failed tests, you may review the next commit, thanks!

[rijobro]

RandomKspaceMaskd and EquispacedKspaceMaskd are identical except for self.masker. You could have a parent class that stores all of the functionality and then RandomKspaceMaskd and EquispacedKspaceMaskd only need to create self.masker in the __init__.

[mersad95zd]

RandomKspaceMaskd and EquispacedKspaceMaskd are identical except for self.masker. You could have a parent class that stores all of the functionality and then RandomKspaceMaskd and EquispacedKspaceMaskd only need to create self.masker in the __init__.

Would it be ok to just create RandomKspaceMaskd, then inherit from it in EquispacedKspaceMaskd?
By doing so, we won't need to define KspaceMaskd.
Another issue with creating KspaceMaskd would be instantiating self.masker in its __init__ because the abstract class KspaceMask can't be instantiated as a self.masker.
So directly defining RandomKspaceMaskd solves this as well.

[mersad95zd]

Since this PR already has about 700 lines of code, I'll submit dictionary unit tests in my next PR.

[wyli]

/build