-
Notifications
You must be signed in to change notification settings - Fork 20
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
Update NumCosmo walkers #436
Comments
The version number of NC seems to be 1 after NC gets imported. Not sure if it is possible to distinguish between different versions of NC.
Cell 26:
|
@vitenti In Example2, I tested cell 26 with different versions of NC and the results are: |
@hsinfan1996 In the lastest version (on github) you can try the old behavior of APES using:
I'll be releasing a new version of NumCosmo that includes this possibility in the next days. If you like, you could test using this constructor and 100 walkers to see if the previous results is recovered. The default method used by apes now is Ncm.FitESMCMCWalkerAPESMethod.VKDE which is more costly but much more powerful to probe complicated posteriors. |
Thanks. It is good that I do not have to switch between different versions now. I am able to install NC from github, and I will get back to you once I finish the tests. |
@vitenti @combet |
From @vitenti - API has slightly changed with the latest conda released of NC (v0.15.4). In NumCosmo examples, e.g.
example2...
:Ncm.FitESMCMCWalkerAPS.new (nwalkers, mset3.fparams_len ())
should be replaced by
Ncm.FitESMCMCWalkerAPES.new (nwalkers, mset3.fparams_len ())
Also, with this new version of the sampler, the number of walkers needs to be increase
nwalkers = 100
-->nwalkers=800
The text was updated successfully, but these errors were encountered: