Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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
Redesign of MCSE #63
Redesign of MCSE #63
Changes from 20 commits
2d82222
e4b067b
34e0221
6839cd7
7dbda2d
b93ce5b
790a99b
9a1d3d5
c0d5a94
cf908af
93d121e
fe99356
d12648b
465afac
7dac85e
9407369
f95a066
88b6c41
899711e
01b8dbc
60d6441
2441bcb
8e3b06a
e7ca85a
2af67e9
da4ed63
b7cd495
4d55716
d9f6734
1c48266
f072b9e
bb47887
7f61907
a03cc2a
bac8a3c
652b86f
8dbae84
d9aff61
cced4be
ce9d427
787a05f
34f3771
d10740a
cf09e4e
39d74a9
a575fc8
b4eea8d
3738347
69fe0a1
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
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.
I haven't seen in the literature a study of how best to combine MCMC chains when estimating MCSE with SBM. I benchmarked a few alternatives and found this one underestimated the MCSE the least: arviz-devs/arviz#1974 (comment)
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.
For
mean
,std
, andquantile
, these tests are somewhat redundant with those foress_rhat
, since the only difference is that foress_rhat
we compute the exact asymptotic variance of the estimator since we know the stationary distribution, whereas formcse
we estimate the asymptotic variance of the estimator. If we want to speed up our test suite, it may be better to remove the correspondingess_rhat
tests and use these as integration tests.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.
I've opted to speed up our test suite by decreasing the number of replicates (params) for each suite to 100 instead of 1,000.