Better document the 'mr' parameter to 'metrop_select'. #7160
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
I found myself confused why
metrop_select()
has this comparison:If
mr
was just a ratio of probabilities, as one would typically see in theoretical descriptions, then it should be compared touniform()
, not the log. Butmr
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/