Skip to content

Test load_medical_images.ipynb based on comment - #1221

Closed
wyli wants to merge 1 commit into
mainfrom
wyli-patch-1
Closed

Test load_medical_images.ipynb based on comment#1221
wyli wants to merge 1 commit into
mainfrom
wyli-patch-1

Conversation

@wyli

@wyli wyli commented Feb 17, 2023

Copy link
Copy Markdown
Contributor

Fixes #1219 (comment)

Checks

  • Avoid including large-size files in the PR.
  • Clean up long text outputs from code cells in the notebook.
  • For security purposes, please check the contents and remove any sensitive info such as user names and private key.
  • Ensure (1) hyperlinks and markdown anchors are working (2) use relative paths for tutorial repo files (3) put figure and graphs in the ./figure folder
  • Notebook runs automatically ./runner.sh -t <path to .ipynb file>

@review-notebook-app

review-notebook-app Bot commented Feb 17, 2023

Copy link
Copy Markdown

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.

Powered by ReviewNB

@wyli
wyli requested a review from mingxin-zheng February 17, 2023 06:26
@wyli

wyli commented Feb 17, 2023

Copy link
Copy Markdown
Contributor Author

It doesn't pass the ci #1221 so indeed the noqa line is currently in use @mingxin-zheng

@wyli wyli closed this Feb 17, 2023
@wyli
wyli deleted the wyli-patch-1 branch February 17, 2023 12:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@mingxin-zheng mingxin-zheng mingxin-zheng approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@wyli @mingxin-zheng