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

pseudo-priors and derived parameters #204

Merged
merged 2 commits into from
Oct 8, 2015
Merged

Conversation

ntessore
Copy link
Contributor

This PR allows priors to be declared pseudo-priors, which return a value not using some random distribution but a given function. For now, only the delta function prior is a pseudo-prior.

Additionally, parameters with a pseudo-prior are now treated as derived parameters for MultiNest (cf. #27), reducing the dimensionality of the parameter space that has to be explored.

There is a caveat: MultiNest expects the derived parameters to be at the end of the parameter list, and this is reflected in its generated files. To make sure the parameter order is correct, check out the generated .paramnames file.

@ntessore
Copy link
Contributor Author

I'm not sure there's anything we can do to keep MultiNest's output in line with the one from Lensed, except

  1. reorder the parameters in Lensed as well, or
  2. write the posterior table ourselves.

Going with the first option could mean displaying two parameter tables, one for real and one for derived parameters. This would split objects apart, but make it clearer which parameters are real and which are derived.

The second choice would be nicer if we could write all output files ourselves, but right now it is not possible to access MultiNest's information on multiple modes programmatically.

@fabiobg83 @rbmetcalf What do you think?

@ntessore ntessore added this to the v1.1 milestone Sep 29, 2015
@ntessore ntessore force-pushed the nt/derived-parameters branch 5 times, most recently from cd43496 to 0e43f2f Compare October 8, 2015 14:31
@ntessore
Copy link
Contributor Author

ntessore commented Oct 8, 2015

@rbmetcalf @fabiobg83 Once Travis turns green I will merge this. If the MultiNest output poses a problem in the future we can handle it then.

@ntessore ntessore force-pushed the nt/derived-parameters branch from 0e43f2f to 96f5b53 Compare October 8, 2015 14:41
@ntessore ntessore force-pushed the nt/derived-parameters branch from 96f5b53 to 56020c2 Compare October 8, 2015 14:41
ntessore pushed a commit that referenced this pull request Oct 8, 2015
pseudo-priors and derived parameters, closes #27
@ntessore ntessore merged commit 516bc4a into master Oct 8, 2015
@ntessore ntessore deleted the nt/derived-parameters branch October 8, 2015 14:53
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

Successfully merging this pull request may close these issues.

1 participant