Conversation
Signed-off-by: mersad95zd <[email protected]>
|
Thank you for submitting your first PR! Would you mind taking a look at the code format check flake8? |
Signed-off-by: mersad95zd <[email protected]>
|
After you finalize the code, please add docstring reference in docs/source/apps.rst and make sure the document looks correct. |
Signed-off-by: mersad95zd <[email protected]>
…odule Signed-off-by: mersad95zd <[email protected]>
Signed-off-by: mersad95zd <[email protected]>
Signed-off-by: mersad95zd <[email protected]>
|
Please move the unit test to |
|
Looks good to me |
|
Have you added rst for the script in monai.networks.blocks? |
Signed-off-by: mersad95zd <[email protected]>
Signed-off-by: mersad95zd <[email protected]>
Signed-off-by: mersad95zd <[email protected]>
|
Please solve this error: |
|
Sorry I somehow missed this comment yesterday. torch.fft.ifftshift is supported in the latest pytorch release: https://pytorch.org/docs/stable/generated/torch.fft.ifftshift.html |
Please see https://github.com/Project-MONAI/MONAI/runs/7056882942?check_suite_focus=true |
Signed-off-by: mersad95zd <[email protected]>
Signed-off-by: Wenqi Li <[email protected]>
Signed-off-by: Wenqi Li <[email protected]>
|
/black |
|
/build |
Signed-off-by: mersad95zd [email protected]
Fixes #4566.
(Fixes #4606 to unblock the merging)
Description
The file contains a few required transforms for MRI reconstruction.
Status
Work in progress
Types of changes
./runtests.sh -f -u --net --coverage../runtests.sh --quick --unittests --disttests.make htmlcommand in thedocs/folder.