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

Default to ETI instead of HDI? #521

Closed
bwiernik opened this issue Apr 4, 2022 · 12 comments
Closed

Default to ETI instead of HDI? #521

bwiernik opened this issue Apr 4, 2022 · 12 comments
Labels
What's your opinion 🙉 Collectively discuss something

Comments

@bwiernik
Copy link
Contributor

bwiernik commented Apr 4, 2022

The more that I'm reading about HDI, the more I'm skeptical about the additional assumptions they require and their performance (especially for >80% intervals). I'm wondering if we should switch to using ETI as the default interval method for Bayesian samples?

the paper illustrates the challenge of reducing the variance without additional assumptions about the target distribution. After the paper had been published, Andrew started to think that HDI is not that useful, and thus hasn't advocated the paper and the package. In that paper the focus was on 95% HDI, and since then Andrew has advocated, e.g. 20% and 80% quantiles as they have less variance.

Originally posted by @avehtari in stan-dev/posterior#216 (comment)

Also: https://twitter.com/betanalpha/status/1479106549519831040?s=21&t=-bBVfRH2pajMzY_ZkqY1Ng

Highest density intervals are not defined by expectation values and so they cannot be constructed from samples alone. Like density estimators they require additional assumptions that can significantly affect the outcome of the intervals, especially if the credibility is large.

Because of this sensitivity to additional assumptions, in particular assumptions that are often only implicit with in a particular HDI estimation method, I strongly recommend that they be avoided entirely.

@bwiernik bwiernik added the What's your opinion 🙉 Collectively discuss something label Apr 4, 2022
@strengejacke
Copy link
Member

btw, @DominiqueMakowski, here's an argument for 89% instead of 95% interval range ;-)

@mattansb
Copy link
Member

mattansb commented Apr 4, 2022

image

@bwiernik
Copy link
Contributor Author

bwiernik commented Apr 4, 2022

No, those will be about the same. Seems You need to get down to about 60% before the variance is meaningfully smaller.

@strengejacke
Copy link
Member

That's a strong opinion about HDI:

I strongly recommend that they be avoided entirely.

(https://twitter.com/betanalpha/status/1479107186030624771)

@strengejacke
Copy link
Member

#522 implements SPI (shortest probability intervals)

@strengejacke
Copy link
Member

Any reason for closing this?

@bwiernik
Copy link
Contributor Author

It was done wasn't it?

@bwiernik bwiernik reopened this Apr 14, 2022
@strengejacke
Copy link
Member

In parameters, not yet bayestestR I think

@bwiernik
Copy link
Contributor Author

Oops

strengejacke added a commit that referenced this issue Apr 19, 2022
@strengejacke
Copy link
Member

@DominiqueMakowski any opinions? And: have you submitted bayestestR the last time, and it got suck somewhere? If so, we probably resolve this issue and then submit (again) to CRAN?

@DominiqueMakowski
Copy link
Member

I am okay to change it, but my knowledge of the underlying algorithms is a bit limited to improve the documentation :/

@bwiernik
Copy link
Contributor Author

Just stating the name of the algorithm used should be sufficient

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
What's your opinion 🙉 Collectively discuss something
Projects
None yet
Development

No branches or pull requests

4 participants