cmdstan icon indicating copy to clipboard operation
cmdstan copied to clipboard

stansummary change label "N_Eff" to "ESS"

Open mitzimorris opened this issue 5 years ago • 1 comments

Summary:

Change N_Eff to ESS as N is misleading as it in manual refers to to the number iterations in each chain (or is often used for the number of observations). ESS would be more clear as it is acronym for effective sample size. posterior package used .e.g by CmdStanR has at least moved to use ESS.

Description:

From this comment: https://github.com/stan-dev/cmdstan/pull/907#issuecomment-663546361

Reproducible Steps:

run current bin/stansummary program

Current Output:

current output

Expected Output:

desired output: column labels "ESS" and "ESS/S"

Additional Information:

Provide any additional information here.

Current Version:

v2.23.0

mitzimorris avatar Jul 24 '20 13:07 mitzimorris

https://github.com/stan-dev/cmdstan/pull/907#issuecomment-663565319

Bob Carpenter comment:

+1 to that. The problem with "n_eff" is that it's not the number of effective samples. We consider the set of draws to be a single sample from the posterior. It's the effective sample size.

I don't like the n_ prefix for "number" elsewhere, either, but that's a different issue.

mitzimorris avatar Jul 24 '20 14:07 mitzimorris