Skip to content
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

Failed: TensorLayer (65e40296) #532

Closed
fire-bot opened this issue Apr 29, 2018 · 5 comments
Closed

Failed: TensorLayer (65e40296) #532

fire-bot opened this issue Apr 29, 2018 · 5 comments

Comments

@fire-bot
Copy link
Collaborator

fire-bot commented Apr 29, 2018

Sent by Read the Docs (readthedocs@readthedocs.org). Created by fire.


| TensorLayer build #7115414

|

| Build Failed for TensorLayer (stable)

You can find out more about this failure here:

TensorLayer build #7115414 - failed

If you have questions, a good place to start is the FAQ:

https://docs.readthedocs.io/en/latest/faq.html

You can unsubscribe from these emails in your Notification Settings

Keep documenting,
Read the Docs
| Read the Docs
https://readthedocs.org


@lgarithm
Copy link
Member

It works!
(The requirements of docs was changed from requirements.txt to requirements-rtd.txt.)

@DEKHTIARJonathan
Copy link
Member

DEKHTIARJonathan commented Apr 29, 2018

There will be problems very soon with TF 1.5.0, many tf APIs are deprecated or not existing on this versions.

We need to fix this very soon. And BTW. It must be working because the official TF document is also using Sphinx.

@lgarithm
Copy link
Member

I currently created two files: requirements.txt and requirements-rtd.txt.
requirements-rtd.txt is for doc build only.
(Can't think of other solutions ATM, use this work around temporarily)

@DEKHTIARJonathan
Copy link
Member

DEKHTIARJonathan commented Apr 29, 2018

We are not the only ones with these issues:
readthedocs/readthedocs.org#3738
tensorflow/tensorflow#17411
Bihaqo/t3f#127

It seems to be linked with facts that the binaries are built with AVX instruction support. Some systems may lack this instruction

@DEKHTIARJonathan
Copy link
Member

@lgarithm I have looked quite extensively the documentation for read the docs. I think I may have a more elegant.
I will give it a shot ;) I assign you @lgarithm as reviewer.

DEKHTIARJonathan added a commit that referenced this issue Apr 30, 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
luomai pushed a commit that referenced this issue 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
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants