Skip to content

Conversation

tiangolo
Copy link
Contributor

@tiangolo tiangolo commented Mar 5, 2018

Add optional dependencies' links to download pages in README.md.

The links go directly to the download / install pages to simplify their installation and make it more straightforward.

README.md Outdated
- cuDNN (recommended if you plan on running Keras on GPU).
- HDF5 and h5py (required if you plan on saving Keras models to disk).
- graphviz and pydot (used by [visualization utilities](https://keras.io/visualization/) to plot model graphs).
- [cuDNN](http://docs.nvidia.com/deeplearning/sdk/cudnn-install/index.html) (recommended if you plan on running Keras on GPU).
Copy link
Collaborator

Choose a reason for hiding this comment

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

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done ✔️

Copy link
Collaborator

@fchollet fchollet left a comment

Choose a reason for hiding this comment

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

LGTM

@fchollet fchollet merged commit 830c363 into keras-team:master Mar 6, 2018
@tiangolo tiangolo deleted the add-dependency-links-to-readme branch March 7, 2018 05:15
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