Skip to content

Commit

Permalink
Fix doc build by removing tensorflow dependency (#2610)
Browse files Browse the repository at this point in the history
Currently the doc builds are failing due to running out of memory when pip installing tensorflow.
I investigated fixing this problem by upgrading to the paid version of readthedocs, and found out that:

1. They do not have a way to just turn your free project into a paid project. You need to configure a new project.
2. Paid projects have *less* (**LESS!**) control over the domain name than the free projects (the URL would be prefixed with "quantum-lib-", also the top-level domain would differ; I strongly suspect we would not be able to use a custom domain that was not owned by us such as the existing docs location).
3. The support staff does not think that this is a situation that needs to be fixed.

So as a fix I'm killing off the TPU translation code from contrib and removing the tensorflow dependency.
I will also look into using other documentation services.

The tpu code is in contrib, so it does not need to be deprecated. The tpu code was still slower than non-tpu simulation due to the large compile times.

As a side benefit, this will likely speed up our builds.
  • Loading branch information
Strilanc authored and CirqBot committed Dec 3, 2019
1 parent 3a32a40 commit 31b5436
Show file tree
Hide file tree
Showing 6 changed files with 1 addition and 684 deletions.
1 change: 0 additions & 1 deletion cirq/contrib/contrib-requirements.txt
Original file line number Diff line number Diff line change
Expand Up @@ -2,4 +2,3 @@

ply>=3.4
pylatex~=1.3.0
tensorflow~=1.14
76 changes: 0 additions & 76 deletions cirq/contrib/tpu/__init__.py

This file was deleted.

0 comments on commit 31b5436

Please sign in to comment.