Updating README #49
Open
Updating README #49
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Open
|
@ortengren can you take a look at some of these minor changes here and do one more quick readthrough? |
README.md
Outdated
|
|
||
| Before installing anisoap, please make sure you have at least the | ||
| following packages installed: | ||
| * python (3.6 or higher) |
There was a problem hiding this comment.
Version Numbers should be 3.9 or 3.10
There was a problem hiding this comment.
3.9 and higher or strictly 3.9 and 3.10?
There was a problem hiding this comment.
check with different docker file instances
There was a problem hiding this comment.
Have we checked that this works with 3.9 and 3.10, either through docker or different conda instances?
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
I adjusted the dependencies section of the README to reflect more accurately the required package versions. I also added the new Rust dependency and included a link to the rustup tool. Lastly, I reordered the document sections so that "Dependencies" comes before "Installation".