PolyChord / PolyChordLite

Public version of PolyChord: See polychord.co.uk for PolyChordPro
https://polychord.io/
Other
84 stars 26 forks source link

typo in README? #77

Closed lukashergt closed 2 years ago

lukashergt commented 2 years ago

In the README.rst it says:

[root]phys_live-birth.txt & [root]dead-birth.txt

These can be used to reconstruct a full nested sampling run, as well as simulate dynamic nested sampling. The format & contents of these two files are as follows: They have has ndims+nderived+2 columns. The first ndims+nderived columns are the ndim parameter values along with the nderived additional parameters that are being passed by the likelihood routine for PolyChord to save along with the ndims parameters. The ndims+nderived+2 column is the log-likelihood value. The ndims+nderived+1 column is the log-likelihood value that the point was born at. They are is identical to the [root]_phys_live.txt and [root]_dead.txt file, except for an additional column including the birth contours

The way I understand this the order of the columns for n sampling parameters and m derived parameters is supposedly:

param_1   param_2   ...   param_n   derived_1    derived_2   ...   derived_m   logL_birth   logL

However, I have the impression that the output actually is ordered as:

param_1   param_2   ...   param_n   derived_1    derived_2   ...   derived_m   logL   logL_birth

Could you confirm/refute that suspicion, @williamjameshandley ?

williamjameshandley commented 2 years ago

That's certainly what the anesthetic reader assumes, and (fortunately!) that's also what the polychord code does. I'll update the readme now. Many thanks @lukashergt

williamjameshandley commented 2 years ago

Closed by this direct commit.