Open
Description
So I noticed this while working on #628.
I was getting a failing test here. An assertion in test_new_data_predict_method
raised an exception. It compares the mean of a fixture toy_y
with the 2.5% quantile of the posterior predictive mean, and it failed. I then went to check the fixture toy_y
and saw that it is a random integer sequence. Since it is random (with no seed) I just reran the test action and it passed.
That's probably not the behavior we want.
My suggestion is we just seed the RNG so it behaves deterministically, but just want @wd60622 s opinion on this before I make a PR for the change.
Activity