-
Notifications
You must be signed in to change notification settings - Fork 92
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
Mcmc unitary tests #2584
Mcmc unitary tests #2584
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for this significant improvement! Only minor changes will greatly improve these unit tests.
Co-authored-by: Michael Baudin <31351465+mbaudin47@users.noreply.github.com>
I don't understand the error which occurs:
|
3a97b32
to
8f04336
Compare
8f04336
to
9b5d7d2
Compare
@jschueller : this PR can now be merged into the 1.23 branch imo ! After the reviewer's approval... |
@jschueller : This PR is ready for merging. Can you please review or merge? |
I added three unitary tests to the MCMC classes to the existing Python test files:
t_Gibbs_std.py
t_IndependentMetropolisHastings_std.py
t_RandomVectorMetropolisHastings_std.py