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

Flaky test: test_distributions_random.py::TestNestedRandom::test_TruncatedNormal #4346

Closed
michaelosthege opened this issue Dec 16, 2020 · 7 comments

Comments

@michaelosthege
Copy link
Member

A wild, flaky test has appeared.

On master it's currently failing here: https://github.com/pymc-devs/pymc3/runs/1562295314#step:8:1434

I already observed the flaky tests in a recent PR but it went away after re-running the CI.

@Sayam753
Copy link
Member

Sayam753 commented Jan 3, 2021

Same flaky test appeared in https://github.com/pymc-devs/pymc3/runs/1638797078

@twiecki
Copy link
Member

twiecki commented Jan 3, 2021

Probably just needs a seed.

@Sayam753
Copy link
Member

@michaelosthege
Copy link
Member Author

Amazing how often they appear. And always in the same test, but sometimes with different parametrizations?!

But I just added the "wontfix" label, because all the random stuff will be replaced as we switch to RandomVariable. Let's hope it goes away then.

@Sayam753
Copy link
Member

Sounds great 🤩. Is there a doc listing all the features that RandomVariable Op will add?

@michaelosthege
Copy link
Member Author

Sounds great 🤩. Is there a doc listing all the features that RandomVariable Op will add?

Not for the RandomVariable Op that is already implemented in Theano.
However there is Brandon's blogpost from three years ago when he birthed the idea: https://brandonwillard.github.io/random-variables-in-theano.html#orged59f09

My understanding is that essentially the .eval() of the TensorVariable will result in drawing random samples - also on dependent variables that are created from a RandomVariable.
This way we don't need to walk up the graph in PyMC3 as we currently do in draw_samples and the like.

@ricardoV94
Copy link
Member

Test seems to be stable for a while now

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants