-
Notifications
You must be signed in to change notification settings - Fork 246
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
Clarify how to sample the centered parameter in LocScaleReparam #1598
Labels
Comments
You are running MCMC, rather than optimizing. As mentioned in the forum thread, you can use |
fehiepsi
changed the title
Bug in LocScaleReparam | Correction in Documentation
Clarify how to sample the centered parameter in LocScaleReparam
Jun 1, 2023
Please feel free to submit a PR to clarify the doc. :) |
Sure, I will do that. |
Madhav-Kanda
added a commit
to Madhav-Kanda/numpyro
that referenced
this issue
Jun 1, 2023
fehiepsi
pushed a commit
that referenced
this issue
Jun 1, 2023
* Update reparam.py Refer #1598 * Making suggested changes
OlaRonning
pushed a commit
to aleatory-science/numpyro
that referenced
this issue
Jun 2, 2023
* Update reparam.py Refer pyro-ppl#1598 * Making suggested changes
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
The documentation for LocScaleReparam in case of centeredness mentions that: " If None (default) learn a per-site per-element centering parameter in [0,1]", but upon looking through the implementation of the algorithm, I did not find anything relevant to finding the correct centeredness (as far as I understand the implementation by default it takes the centeredness to be 0.5 and does not optimize it).
Example:
In the case of the eight schools' examples, the ideal centeredness should be close to 0. Still, upon using LocScaleReparam with None as its parameter, we get the centeredness is 0.5, as seen below (which I believe is hard coded in the implementation of LocScaleReparam itself for the case of None).
In the above case both of them provide the same print_summary. Thus, I believe that either the code for the LocScaleReparam should be updated to include SVI for finding the best centeredness or the documentation should be updated accordingly.
The text was updated successfully, but these errors were encountered: