Skip to content

Conversation

@mhubii
Copy link
Contributor

@mhubii mhubii commented Mar 19, 2020

Addresses #188

Description

  • Introduces setuptools
  • Removes relative paths from examples
  • Updates readme

@mhubii mhubii mentioned this pull request Mar 19, 2020
Copy link
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.

thanks a lot! for some reason, the ci didn't start for PR from forks, I'll look into that. I put some inline comments based on my local tests.

Copy link
Contributor Author

@mhubii mhubii left a comment

Choose a reason for hiding this comment

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

changed contributers, mail, and added empty line to resolve warning

wyli
wyli previously approved these changes Mar 19, 2020
README.md Outdated

```bash
git clone https://github.com/Project-MONAI/MONAI.git
python setup.py install
Copy link
Contributor

Choose a reason for hiding this comment

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

forgot to check this just now, i think we need cd MONAI/ before python setup.py, or pip install -e MONAI/ works fine as well

Copy link
Contributor Author

Choose a reason for hiding this comment

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

oh yes you are right

@wyli wyli dismissed their stale review March 19, 2020 18:52

need to update the readme

@wyli wyli merged commit 9f10654 into Project-MONAI:master Mar 19, 2020
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