Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Maisi readme #1743

Merged
merged 63 commits into from
Aug 1, 2024
Merged

Maisi readme #1743

merged 63 commits into from
Aug 1, 2024

Conversation

Can-Zhao
Copy link
Contributor

@Can-Zhao Can-Zhao commented Jun 27, 2024

Fixes # .

Description

Maisi readme.

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>

Can-Zhao and others added 4 commits June 27, 2024 20:07
Signed-off-by: Can Zhao <canz@nvidia.com>
Signed-off-by: Can Zhao <canz@nvidia.com>
Signed-off-by: Can Zhao <canz@nvidia.com>
@Can-Zhao Can-Zhao marked this pull request as ready for review June 27, 2024 20:12
@Can-Zhao Can-Zhao self-assigned this Jun 27, 2024
Can-Zhao added 2 commits June 27, 2024 20:26
Signed-off-by: Can Zhao <canz@nvidia.com>
@Can-Zhao Can-Zhao requested a review from KumoLiu June 27, 2024 20:35
Copy link
Contributor

@KumoLiu KumoLiu left a comment

Choose a reason for hiding this comment

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

Thanks for the PR, overall looks good to me.
Several minor comments inline.

generative/maisi/README.md Outdated Show resolved Hide resolved
generative/maisi/data/README.md Show resolved Hide resolved
generative/maisi/README.md Outdated Show resolved Hide resolved
Signed-off-by: Can Zhao <canz@nvidia.com>
@Can-Zhao Can-Zhao marked this pull request as draft June 28, 2024 17:36
generative/README.md Outdated Show resolved Hide resolved
@guopengf guopengf mentioned this pull request Jul 8, 2024
5 tasks
Signed-off-by: dongyang0122 <don.yang.mech@gmail.com>
Signed-off-by: dongyang0122 <don.yang.mech@gmail.com>
Signed-off-by: dongyang0122 <don.yang.mech@gmail.com>
Signed-off-by: dongyang0122 <don.yang.mech@gmail.com>
Signed-off-by: dongyang0122 <don.yang.mech@gmail.com>
@dongyang0122 dongyang0122 marked this pull request as ready for review July 29, 2024 04:13
generative/maisi/README.md Outdated Show resolved Hide resolved
@KumoLiu
Copy link
Contributor

KumoLiu commented Jul 29, 2024

Hi @Can-Zhao, please help renamed the generative dir to generation and modify the related "Setup environment" section. Thanks!

dongyang0122 and others added 5 commits July 29, 2024 07:48
Signed-off-by: dongyang0122 <don.yang.mech@gmail.com>
Signed-off-by: dongyang0122 <don.yang.mech@gmail.com>
Signed-off-by: dongyang0122 <don.yang.mech@gmail.com>
Signed-off-by: Pengfei Guo <pengfeig@nvidia.com>
Signed-off-by: Pengfei Guo <pengfeig@nvidia.com>
@Can-Zhao
Copy link
Contributor Author

Hi @Can-Zhao, please help renamed the generative dir to generation and modify the related "Setup environment" section. Thanks!

I will change it after all other comments are resolved. Or we can open another PR to do it to make it more tractable

guopengf and others added 2 commits July 31, 2024 15:14
Signed-off-by: Pengfei Guo <pengfeig@nvidia.com>
@KumoLiu KumoLiu merged commit f444983 into Project-MONAI:main Aug 1, 2024
6 checks passed
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.

5 participants