Skip to content

fixes #560#563

Merged
wyli merged 3 commits into
Project-MONAI:masterfrom
wyli:560
Feb 19, 2022
Merged

fixes #560#563
wyli merged 3 commits into
Project-MONAI:masterfrom
wyli:560

Conversation

@wyli

@wyli wyli commented Feb 19, 2022

Copy link
Copy Markdown
Contributor

Signed-off-by: Wenqi Li wenqil@nvidia.com

Fixes #560

Description

follow up of #546

Status

Ready

Checks

  • Notebook runs automatically ./runner [-p <regex_pattern>]

@review-notebook-app

Copy link
Copy Markdown

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

Signed-off-by: Wenqi Li <wenqil@nvidia.com>

@Nic-Ma Nic-Ma left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the quick fix, seems there are much change in this PR, may I know which is the key change to fix the error? Or what's the root cause?

Thanks.

Comment thread modules/3d_image_transforms.ipynb
Comment thread modules/3d_image_transforms.ipynb
Comment thread modules/3d_image_transforms.ipynb
wyli added 2 commits February 19, 2022 16:16
Signed-off-by: Wenqi Li <wenqil@nvidia.com>
@wyli wyli enabled auto-merge (squash) February 19, 2022 16:22
@wyli wyli merged commit b66f3a5 into Project-MONAI:master Feb 19, 2022
@wyli wyli deleted the 560 branch August 8, 2022 09:31
boneseva pushed a commit to boneseva/MONAI-tutorials that referenced this pull request Apr 21, 2024
* fixes Project-MONAI#560

Signed-off-by: Wenqi Li <wenqil@nvidia.com>

* update based on comments

Signed-off-by: Wenqi Li <wenqil@nvidia.com>
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.

testing error: modules/3d_image_transforms.ipynb

2 participants