stan-dev / stan-dev/stan

Dense Adaptation Fragile

Open
#1,853 4 comments 0 reactions 1 assignee View on GitHub

@betanalpha is already working on this.

Since Mar 30, 2017.

algorithm
Dominant language
C++
Stars
2.8k
Forks
388
Avg merge
2d 17h
Merged PRs (30d)
15

Description

Summary:

The covariance estimation currently used to construct a dense mass matrix is yielding sufficiently poor results that the resulting chains go crazy.

Description:

It looks like the regularization in covar_adaptation.hpp is insufficient. The initial estimates of the covariance are so noisy that they cause the subsequent adaptation windows to fail miserably. Adding stronger regularization helps, but it's not clear if this is a sufficiently robust solution.

Reproducible Steps:

Try to fit the simple model

parameters {
  real mu[10];
}
model {
  mu ~ normal(0, 1);
}

with the CmdStan configuration

./gauss sample num_samples=10000 algorithm=hmc metric=dense_e
Current Output:

Completely biased results. The diagnostics do indicate that the fit results should not be trusted.

Expected Output:

Accurate results. We should be able to fit an IID Gaussian.

Current Version:

develop

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.