Test tutorials on travis#678
Conversation
|
@neerajprad looks like build is failing due to wheel issues |
|
@fritzo - I had updated the wheel files for Multinomial, but forgot to overrride the permissions to public. Restarted the tests, it should work now. |
| "from pyro.distributions import categorical, normal\n", | ||
| "softmax = Softmax(dim=-1)" | ||
| "softmax = Softmax(dim=-1)\n", | ||
| "smoke_test = ('CI' in os.environ)" |
There was a problem hiding this comment.
Were the other changes (results and plots) in this file due to just running the nbval command?
|
I think this is a great solution for now! Thanks for fixing this. Could we add the |
|
This seems reasonable to me. Once #680 is merged the AIR tutorial should be working again and could be added to these smoke tests. I could do this as a follow-up PR if you like. |
|
@neerajprad sorry it took me so long to fix this PR. We'll need this before The Big Refactoring next week. |
This adds support for testing some of our tutorials on travis.
Why?
We're changing Pyro and we want to avoid bit rot of tutorials.
How?
This uses a kludgey mechanism of defining a boolean
smoke_test = 'CI' in os.environin each tutorial. This is a little ugly, so we might something cleaner before Pyro 0.2 release, e.g. replacing all strings of the formnum_iters = [0-9]*with the stringnum_iters = 1.