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 plot_2d including run statistics #332

Open
yallup opened this issue Aug 21, 2023 · 1 comment
Open

Default plot_2d including run statistics #332

yallup opened this issue Aug 21, 2023 · 1 comment

Comments

@yallup
Copy link
Collaborator

yallup commented Aug 21, 2023

Describe the bug
Calling plot_2d without chosing which params to plot generically seems to error as it includes logL, nlive and logLbirth, which typically don't play nicely with 2d KDEs

To Reproduce
Using twin_gaussian example from PCLite repo chains (using anesthetic 2.3.0 installed via pip)

import anesthetic as ns
from matplotlib import pyplot as plt
samples=ns.read_chains("chains/twin_gaussian")

#fine because you force it to be scatters
samples.plot_2d(kinds=dict(upper="scatter_2d",lower='scatter_2d'))
#errors due to triangulation errors
samples.plot_2d()

twin_gaussian

Expected behavior
Default behavior should be all params but not nlive, logl and logl_birth.

Additional context
Might be related to the previous change wrt integer axis names

@williamjameshandley
Copy link
Collaborator

Very much related to #324. Can we move the discussion to there, in particular whether we think that default behaviour should be to avoid the nlive logl and logl_birth columns.

@yallup yallup mentioned this issue Aug 21, 2023
6 tasks
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