Repository navigation
Update docs; remove need for conda (library) in dev install - #847
Conversation
|
@sroet : Can you verify that this fixes for you? Since the change is in a secondary script in the repo, not the bash conda_ops_dev_install.sh openpathsampling docs_deployThat will check out the Also, could you review the changes here, since most of them are to address lack of clarity in the docs that you identified in #846? |
sroet
left a comment
There was a problem hiding this comment.
LGTM, the install works for 3.6 it doe (fails for 3.5 because of yaml versions, but the docs are updated to indicate 3.6 is the lowest py3 version to be tested, so that is fine by me).
Just curious: what do you mean by this? (Also, currently having some weird flaky test fail in test_pyemma notebook; I'll try to track that down before merging. I can't imagine it has anything to do with this PR, but there also don't seem to be any version changes in requirements). |
|
On which is probably due to the fact that |
|
Ah, yeah, the script to find reqs will now require newer versions of pyyaml, and I guess they never released those for Python 3.5. The explicit |
|
We're definitely getting flakey errors in the test_pyemma notebook. Something to do with storage, it appears. But I can't find any version differences that make the problem evident. Anyway, @sroet says that this fixes his problems, and it's mainly docs anyway, so I'm going to ahead and merge this. |
Fixes #846, plus a few other issues (in particular, I noticed that our docs were building on all Python versions, not only the deploy version — not a problem, but a waste of time on Travis.)