Skip to content

Conversation

@DEKHTIARJonathan
Copy link
Member

@DEKHTIARJonathan DEKHTIARJonathan commented Apr 29, 2018

This PR aims to a more elegant solution to the issue #532:

  • Do not create a specific requirements file for RTD.
  • Refactor all the requirements in a more readable way
  • Update the readme with the installation commands

Btw. my RTD configuration needs to be tested for real ;) I am quite confident that it should work.

@tensorlayer tensorlayer deleted a comment Apr 29, 2018
@tensorlayer tensorlayer deleted a comment Apr 29, 2018
@DEKHTIARJonathan DEKHTIARJonathan changed the title Installation refactored Installation refactored - Attempt for issue #532 Apr 29, 2018
@DEKHTIARJonathan DEKHTIARJonathan requested a review from luomai April 29, 2018 22:57
@DEKHTIARJonathan DEKHTIARJonathan added this to the 1.8.5 milestone Apr 29, 2018
@tensorlayer tensorlayer deleted a comment Apr 29, 2018
@tensorlayer tensorlayer deleted a comment Apr 30, 2018
@DEKHTIARJonathan DEKHTIARJonathan merged commit b10975a into master Apr 30, 2018
@DEKHTIARJonathan DEKHTIARJonathan deleted the Installation-Refactored branch April 30, 2018 04:12
luomai pushed a commit that referenced this pull request Nov 21, 2018
* Update requirements.txt

* Readme updated with the newest installation commands

* Installation Workflow Redesigned with ReadTheDocs Config Added

* Error fix in __init__

* Codacy Issues Fixed

* Error fix when TL installed from PyPI or Wheel file without Tensorflow

* Codacy Error Fix + YAPF Error Fix
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants