pymc icon indicating copy to clipboard operation
pymc copied to clipboard

Better document the 'mr' parameter to 'metrop_select'.

Open bangerth opened this issue 1 year ago • 1 comments

I found myself confused why metrop_select() has this comparison:

    if np.isfinite(mr) and np.log(uniform()) < mr:

If mr was just a ratio of probabilities, as one would typically see in theoretical descriptions, then it should be compared to uniform(), not the log. But mr is the logarithm of the ratio -- something not clear from the documentation, but that can be added :-)


📚 Documentation preview 📚: https://pymc--7160.org.readthedocs.build/en/7160/

bangerth avatar Feb 17 '24 04:02 bangerth

Thank You Banner] :sparkling_heart: Thanks for opening this pull request! :sparkling_heart: The PyMC community really appreciates your time and effort to contribute to the project. Please make sure you have read our Contributing Guidelines and filled in our pull request template to the best of your ability.

welcome[bot] avatar Feb 17 '24 04:02 welcome[bot]