Changed QuTiP to pull from PyPI.

QInfer / python-qinfer

Library for Bayesian inference via sequential Monte Carlo for quantum parameter estimation.

BSD 3-Clause "New" or "Revised" License

92 stars 31 forks source link

Changed QuTiP to pull from PyPI. #109

Closed cgranade closed 7 years ago

cgranade commented 7 years ago

This PR should fix #107 and allow for proper testing of #106.

ihincks commented 7 years ago

Travis: There was an error while loading data. ??

cgranade commented 7 years ago

I've been getting this on all Travis builds recently. I'm not sure what's going on, but I'll investigate.

On Sat, Feb 4, 2017, 08:16 Ian Hincks notifications@github.com wrote:

Travis: There was an error while loading data. ??

— You are receiving this because you were assigned. Reply to this email directly, view it on GitHub https://github.com/QInfer/python-qinfer/pull/109#issuecomment-277364335, or mute the thread https://github.com/notifications/unsubscribe-auth/AAB7HHIB4-XHaU0tQMecxJwVodHwR1Lfks5rY5klgaJpZM4L0vMP .

cgranade commented 7 years ago

The Travis CI builds seem to be broken only for this project, so I'll go on and send them a message and let you know what they say.

cgranade commented 7 years ago

I've filed an issue with Travis CI, we'll see if that gets addressed soon. Looks like it's an internal server error on their end, though, that's propagating outward through the JSON API in a way that's not handled especially well by the client-side web app.

cgranade commented 7 years ago

It seems as though our use of encrypted environment variables to support automatic PyPI deployment may have triggered an internal bug within Travis. In lieu of a permanent fix, I've worked around this for now with 7fcf6b2, which moves encrypted environment variables from the repository settings to .travis.yml. This workaround seems to be working for now, hopefully?

coveralls commented 7 years ago

Coverage remained the same at 65.545% when pulling 7fcf6b2cc48fa4778525361992cc43bde8d7f865 on fix-qutip-doctest into a93f9d2a971258f6f7a0e9e645b28f7601711067 on master.

coveralls commented 7 years ago

Coverage remained the same at 65.545% when pulling a976285cbd40abc01a26fc7334004b41f32bd399 on fix-qutip-doctest into a93f9d2a971258f6f7a0e9e645b28f7601711067 on master.

cgranade commented 7 years ago

Now that the Travis builds are working, I can diagnose the underlying problem: it seems as though the very old version of GCC used by default on Travis is not compatible with recent QuTiP releases, such that we need to bump up to at least 4.8. There's some instructions to be found in other Travis CI issues, such that hopefully the latest build should work.

coveralls commented 7 years ago

Coverage decreased (-7.6%) to 57.957% when pulling c270349b652e357489a8f3a098dd4c92285858cb on fix-qutip-doctest into a93f9d2a971258f6f7a0e9e645b28f7601711067 on master.

cgranade commented 7 years ago

Since all checks now pass, I'd like to get this PR merged in ASAP to help test other pending PRs. Marking it as ready for review.