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

logprob values #1

Closed
grburgess opened this issue Apr 9, 2020 · 7 comments
Closed

logprob values #1

grburgess opened this issue Apr 9, 2020 · 7 comments
Labels
enhancement New feature or request

Comments

@grburgess
Copy link

Hello,

I cannot tell from the documentation if the log prob or log likelihood values are stored. Can you point me to this?

@minaskar
Copy link
Owner

minaskar commented Apr 9, 2020

Hi Michael,

The log prob values are not stored in the current (1.0.3) release of zeus, just the chain samples. However, this is something that will be added in a future release, probably in 1.0.4 next week.

@minaskar minaskar added the enhancement New feature or request label Apr 9, 2020
@grburgess
Copy link
Author

Thanks! We store them in our software and wanted to add zeus into our sampler suite, but right now it means revaluate the whole chain :)

looking forward to that.

@minaskar
Copy link
Owner

minaskar commented Apr 9, 2020

May ask if your software package is/will be public. I'm interested to see how people use zeus.

I'll update this thread when the feature is added to zeus (probably a matter of days).

@grburgess
Copy link
Author

Absolutely:
https://threeml.readthedocs.io/en/latest/notebooks/Bayesian_tutorial.html

https://threeml.readthedocs.io/en/latest/index.html

It is a multi-messenger spectral/spatial fitting software package for astrophysics. I'm currently rewriting the Bayes API to make it easier to switch between the samplers and add them on.

@minaskar
Copy link
Owner

Hi Michael, please have a look at the latest version (1.0.5).

Let me know if that solves your issue.

Minas

@grburgess
Copy link
Author

I think this got it. Thanks!

@grburgess
Copy link
Author

And we have implemented it here:
https://threeml.readthedocs.io/en/latest/notebooks/Bayesian_tutorial.html#The-Zeus-sampler

We cannot have it yet as a default sampler because it is not on conda which we use to build all of our dependencies due to having many, but when it is on conda, we will pop that right in. Thanks for this!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants