search

andreufont / LyaCosmoParams

Notes on cosmological parameters and Lyman alpha simulations
2 stars 2 forks source link

Clarify likelihood free parameters #72

Closed Chris-Pedersen closed 4 years ago

Chris-Pedersen commented 4 years ago

Related to issue #60 , removed like.free_parameters and like.free_parameter_limits, replaced with a method to return the list of free parameters, although I don't think this is currently used anywhere in the code. There might be a notebook where it is.

Chris-Pedersen commented 4 years ago

Tested that this is working with the sampler. If you approve of the changes please merge!

Chris-Pedersen commented 4 years ago

I'm going to merge this for now as I want to run some samplers with the two latest PRs merged into master - we can revisit these changes if you think there's anything that can be improved.