Skip to content

MetaTensor non-breaking changes#4539

Merged
wyli merged 3 commits intoProject-MONAI:devfrom
rijobro:MetaTensorNonBreaking
Jun 20, 2022
Merged

MetaTensor non-breaking changes#4539
wyli merged 3 commits intoProject-MONAI:devfrom
rijobro:MetaTensorNonBreaking

Conversation

@rijobro
Copy link
Copy Markdown
Contributor

@rijobro rijobro commented Jun 20, 2022

Non-breaking changes from https://github.com/Project-MONAI/MONAI/tree/feature/MetaTensor

Status

Ready

Types of changes

  • Non-breaking change (fix or new feature that would not break existing functionality).

Signed-off-by: Richard Brown <[email protected]>
@rijobro rijobro requested review from Nic-Ma, ericspod and wyli June 20, 2022 10:21
@wyli
Copy link
Copy Markdown
Contributor

wyli commented Jun 20, 2022

(there are some related comments from @Nic-Ma #4536 (comment))

rijobro added 2 commits June 20, 2022 12:01
Signed-off-by: Richard Brown <[email protected]>
Signed-off-by: Richard Brown <[email protected]>
@Nic-Ma
Copy link
Copy Markdown
Contributor

Nic-Ma commented Jun 20, 2022

Hi @rijobro ,

I have 2 minor comments in: #4536 (comment).
There are related to your modification in this PR, could you please help take a look?

Thanks in advance.

Copy link
Copy Markdown
Contributor

@wyli wyli left a comment

Choose a reason for hiding this comment

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

looks good to me, with these code changes, some statements in the docstrings of metatensor/metaobj may not be true, we should review the docstrings.

@wyli
Copy link
Copy Markdown
Contributor

wyli commented Jun 20, 2022

/build

@wyli wyli enabled auto-merge (squash) June 20, 2022 20:48
@wyli wyli merged commit 8b5ddf5 into Project-MONAI:dev Jun 20, 2022
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.

3 participants