Update hierarchical partial pooling notebook #2288
Conversation
|
CC @ColCarroll That was yours, right? |
|
A nitpick, but I would move the paragraph-long comment in cell 3 out into its own markdown cell. You can even break down the model specification into 2 |
|
Not mine -- I think this was @usptact. I did an API fix and some proofreading though. Looks fine to me! |
|
Yes, I created this notebook. Is there something I need to change? |
|
I don't think so, just wanted to let you know it is still useful!
…On Tue, Jun 13, 2017 at 10:59 AM Vladislavs Dovgalecs < ***@***.***> wrote:
Yes, I created this notebook. Is there something I need to change?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#2288 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/ACMHEJkUVUtKbLIrgl_HLImD7EIUOlB-ks5sDqO6gaJpZM4N1RCQ>
.
|
|
@ColCarroll Thanks! It would be much more difficult without help of @twiecki and @fonnesbeck ! Currently I work more with Infer.NET (due to the platform) but I check out regularly what are you guys doing with PyMC3! Cheers! |
|
Thanks again @usptact |
|
This is ready to merge right? @aseyboldt |
aseyboldt
force-pushed
the
fix-hier-part-pool
branch
from
202e6e5 to
a8c9481
Compare
|
Now it is. I moved the long comment into a paragraph. |
I fixed another notebook. Then I noticed that it hadn't been broken in the first place.
Still, I improved mixing by switching around the parametrization a bit and updated the api usage.