-
Notifications
You must be signed in to change notification settings - Fork 25
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
Update README.md #37
Update README.md #37
Conversation
@@ -40,7 +40,12 @@ To open DICAT, simply double click on the executable. | |||
|
|||
To install DICAT source code on a computer, download and save the content of the current Github repository into a workstation. | |||
|
|||
Before running DICAT, make sure your systems contains a [Python](https://www.python.org) compiler with the [TkInter](https://wiki.python.org/moin/TkInter) library (usually, TkInter comes by default with most Python installations. The [PyDICOM](http://www.pydicom.org) package is also required by DICAT. | |||
Before running DICAT, make sure your systems contains a [Python](https://www.python.org) compiler with the [TkInter](https://wiki.python.org/moin/TkInter) library (usually, TkInter comes by default with most Python installations. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
missing a closing parenthesis after installations.
This is great! Thank you!! :) I just had two tiny comments to take care but other than that (typo and some more precision). |
updated the documentation for clarity.
@@ -40,7 +40,12 @@ To open DICAT, simply double click on the executable. | |||
|
|||
To install DICAT source code on a computer, download and save the content of the current Github repository into a workstation. | |||
|
|||
Before running DICAT, make sure your systems contains a [Python](https://www.python.org) compiler with the [TkInter](https://wiki.python.org/moin/TkInter) library (usually, TkInter comes by default with most Python installations. The [PyDICOM](http://www.pydicom.org) package is also required by DICAT. | |||
Before running DICAT, make sure your systems contains a [Python](https://www.python.org) compiler with the [TkInter](https://wiki.python.org/moin/TkInter) library (usually, TkInter comes by default with most Python installations.) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry... I am being picky but the ")" should be between installations and the dot.
period now outside the parenthesis.
@gluneau Sorry. I just realized, could you send the pull request to the branch 2.1-dev instead of master? |
I needed to install this package for the requirements to be fulfilled.