-
Notifications
You must be signed in to change notification settings - Fork 33
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
Issue#149 differential evolution mcmc #158
Conversation
Need to add more recent DE variants.
Need to (ideall) allow for burn-in selection of crossover distribution as per the paper.
Including a comment to ignore flake8 on a few lines as it is stopping me write perfectly fine code due to alignment.
"pints.plot.pairwise(stacked, kde=False)\n", | ||
"\n", | ||
"plt.show()" | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We probably want the examples to be more consistent. We can either split the two examples or we can also do one that use the same data and compare the performance of different MCMC algorithms?
I will split the two first.
self._num_chains = num_chains | ||
|
||
|
||
def R_draw(i, num_chains): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should we make this to a hidden method? e.g. _R_draw
and add some comments?
|
||
1. Select proposal: | ||
|
||
x_proposed = x[i,r] + (1 + e) * gamma(delta, d, p_g) * |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we possibly use LaTex format? Probably hard to read these equations in the documentation. (see above -- line 217)
|
||
|
||
class DreamMCMC(pints.MCMC): | ||
""" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we do something like r""" ... $equation$ ... """
to make things in LaTex format? @MichaelClerx probably have more ideas on this.
No description provided.