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

Include additional statistics in info #5

Open
sethaxen opened this issue May 18, 2021 · 0 comments
Open

Include additional statistics in info #5

sethaxen opened this issue May 18, 2021 · 0 comments
Assignees

Comments

@sethaxen
Copy link
Collaborator

Currently DynamicHMCChain only contains in info what DynamicHMC calls tree_stats, but this is only a subset of the quantities that one might find useful for diagnosing sampling issues. In particular, DynamicHMC.MCMCSteps, which is stored in the meta field, also stores the current step size ϵ and the hamiltonian H. While it doesn't store the already-evaluated current value of the energy, this can be computed from H at minimal cost relative to that of generating a new trajectory.

Are warmup samples returned as well? If so, it would be good to have a boolean flag in the info so warmup samples can be distinguished from post-warmup samples.

So I propose that info be modified to a NamedTuple with entries step_size, energy, and tree_stats, where tree_stats points to an instance of TreeStatisticsNUTS. Or alternatively the structure is flattened to a single NamedTuple.

@cscherrer cscherrer self-assigned this May 18, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants