Skip to content

Conversation

@minsuking
Copy link
Contributor

Summary

Fix a minor typo: "UNet_meatdata" → "UNet_metadata".

Why

Correct spelling improves clarity and avoids confusion.

Changes

  • Updated spleen_segmentation_3d_visualization_basic.ipynb to use correct key UNet_metadata.

Validation

  • Searched repository for meatdata to confirm no other occurrences.

Issue

Fixes tutorials#2015

Notes

  • Documentation only, no code logic changes.

@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

Copy link
Member

@ericspod ericspod 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, thanks for spotting this!

@ericspod
Copy link
Member

Hi @minsuking please look at the output from the DCO check to see how to address this with a remediation commit, otherwise we're good to merge.

minsuking and others added 2 commits September 29, 2025 15:38
minsu <[email protected]>

DCO Remediation Commit for minsu <[email protected]>

I, minsu <[email protected]>, hereby add my Signed-off-by to this commit: 43f0943

Signed-off-by: minsu <[email protected]>
@minsuking
Copy link
Contributor Author

Hi @ericspod I’ve added the DCO remediation commit and the DCO check has passed.
All required checks have passed except for the premerge-notebooks build, which failed due to a “No space left on device” error on the GitHub runner (infra-related, not PR-related).

I will update the branch to bring it up to date with main so the checks can rerun.

@ericspod ericspod merged commit d96190e into Project-MONAI:main Sep 29, 2025
5 of 6 checks passed
@ericspod
Copy link
Member

Hi @minsuking thanks for the DCO fix, the other CICD issue is an independent issue we're aware of and are in the process of fixing.

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.

2 participants