Skip to content

Remove meta_dict and fix affine to spacing conversion#5367

Merged
wyli merged 4 commits intoProject-MONAI:devfrom
mingxin-zheng:fix-5201-5332
Oct 20, 2022
Merged

Remove meta_dict and fix affine to spacing conversion#5367
wyli merged 4 commits intoProject-MONAI:devfrom
mingxin-zheng:fix-5201-5332

Conversation

@mingxin-zheng
Copy link
Copy Markdown
Contributor

@mingxin-zheng mingxin-zheng commented Oct 20, 2022

Signed-off-by: Mingxin Zheng [email protected]

Fixes #5201
Fixes #5332

Description

  • Remove deprecated meta_dict usage from Auto3DSeg.
  • Fix affine -> spacing conversion
  • Update docstring
  • Change "pixel_percentage" to "foreground_percentage" to unify foreground "pixel"/"voxel" naming

Status

Ready for review

Types of changes

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

@wyli
Copy link
Copy Markdown
Contributor

wyli commented Oct 20, 2022

/black

@mingxin-zheng mingxin-zheng changed the title remove meta_dict and fix affine to spacing conversion [WIP]remove meta_dict and fix affine to spacing conversion Oct 20, 2022
Signed-off-by: Mingxin Zheng <[email protected]>
@mingxin-zheng mingxin-zheng changed the title [WIP]remove meta_dict and fix affine to spacing conversion Remove meta_dict and fix affine to spacing conversion Oct 20, 2022
@wyli
Copy link
Copy Markdown
Contributor

wyli commented Oct 20, 2022

/build

@wyli wyli enabled auto-merge (squash) October 20, 2022 15:37
@wyli wyli merged commit 8aafd88 into Project-MONAI:dev Oct 20, 2022
wyli pushed a commit that referenced this pull request Oct 20, 2022
Signed-off-by: Mingxin Zheng
<[email protected]>

Fixes #5201 
Fixes #5332 

### Description

- Remove deprecated meta_dict usage from Auto3DSeg. 
- Fix affine -> spacing conversion
- Update docstring 
- Change "pixel_percentage" to "foreground_percentage" to unify
foreground "pixel"/"voxel" naming

### Types of changes
<!--- Put an `x` in all the boxes that apply, and remove the not
applicable items -->
- [x] Non-breaking change (fix or new feature that would not break
existing functionality).
- [x] In-line docstrings updated.

Signed-off-by: Mingxin Zheng <[email protected]>
@mingxin-zheng mingxin-zheng deleted the fix-5201-5332 branch October 21, 2022 02:36
bhashemian pushed a commit to JHancox/MONAI that referenced this pull request Oct 21, 2022
…5367)

Signed-off-by: Mingxin Zheng
<[email protected]>

Fixes Project-MONAI#5201 
Fixes Project-MONAI#5332 

### Description

- Remove deprecated meta_dict usage from Auto3DSeg. 
- Fix affine -> spacing conversion
- Update docstring 
- Change "pixel_percentage" to "foreground_percentage" to unify
foreground "pixel"/"voxel" naming

### Types of changes
<!--- Put an `x` in all the boxes that apply, and remove the not
applicable items -->
- [x] Non-breaking change (fix or new feature that would not break
existing functionality).
- [x] In-line docstrings updated.

Signed-off-by: Mingxin Zheng <[email protected]>
Signed-off-by: Behrooz <[email protected]>
KumoLiu pushed a commit that referenced this pull request Nov 2, 2022
Signed-off-by: Mingxin Zheng
<[email protected]>

Fixes #5201 
Fixes #5332 

### Description

- Remove deprecated meta_dict usage from Auto3DSeg. 
- Fix affine -> spacing conversion
- Update docstring 
- Change "pixel_percentage" to "foreground_percentage" to unify
foreground "pixel"/"voxel" naming

### Types of changes
<!--- Put an `x` in all the boxes that apply, and remove the not
applicable items -->
- [x] Non-breaking change (fix or new feature that would not break
existing functionality).
- [x] In-line docstrings updated.

Signed-off-by: Mingxin Zheng <[email protected]>
Signed-off-by: KumoLiu <[email protected]>
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.

DataAnalyzer in auto3dseg will record negative numbers of spacing deprecate meta_dict usage in monai.auto3dseg

2 participants