Rethinking_2 Chp_03: add seed, fix typo

[joannadiong]

I added a seed to the Chp 3 notebook as per the PyMC Notebook style, and updated the ax.imshow() origin keyword to lower. Notebook has been re-compiled.

Another small PR. Cheers.

[review-notebook-app]

Check out this pull request on 

See visual diffs & provide feedback on Jupyter Notebooks.

---

Powered by ReviewNB

[MarcoGorelli]

cool, thanks! Looks good, though

updated the ax.imshow() origin keyword to lower

, where is this change? I can't see it

[AlexAndorra]

Any reason this was closed? I don't see any problem with it

[aloctavodia]

I think @joannadiong accidentally included in this PR a change already merged in a previous PR. @joannadiong let us know if you need help with git/github. We know it can be very tricky!

[joannadiong]

Hi @MarcoGorelli, I appear to have mistaken a Chp 4 change for Chp 3. So Chp 3 changes are only the seed and a typo. I will update the Issue title for clarity but leave intact the orignal comment message.

To @AlexAndorra and @aloctavodia, not sure, but I may have accidentally closed the Issue when the branch I was working on was deleted. Apologies. In future, should branches be deleted only after Issues are closed by a maintainer?

Yes please, Git/GitHub refs especially for contributor standard practice would be great. Thank you.

[MarcoGorelli]

Cool, thanks for explaining - I think this is fine then

Re-git: I think the first 3 chapters of this book do a really good job of teaching you the fundamentals

---

And yes, you probably will only want to delete a branch after it's been merged (unless you decide you no longer want the pull request to be considered!)

[joannadiong]

Thanks @MarcoGorelli for the ref -- it's a good book.

Happy for you to merge the small edits and close the PR. Then I will delete the branch. Cheers

[MarcoGorelli]