Update factor analysis nb to v5

[erik-werner]

Fix so that the nb doesn't crash at pm.Minibatch

• Notebook follows style guide https://docs.pymc.io/en/latest/contributing/jupyter_style.html
• PR description contains a link to the relevant issue:
  • a tracker one for existing notebooks (tracker issues have the "tracker id" label)
  • or a proposal one for new notebooks
• Check the notebook is not excluded from any pre-commit check: https://github.com/pymc-devs/pymc-examples/blob/main/.pre-commit-config.yaml

---

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

[review-notebook-app]

Check out this pull request on 

See visual diffs & provide feedback on Jupyter Notebooks.

---

Powered by ReviewNB

[erik-werner]

Weird error in the CI pre-commit check:

Error: reserveCache failed: Cache already exists. Scope: refs/pull/586/merge, Key: pre-commit-2-3062ff7e79208312406649393529767a0ddbbadd6828c544993cabef7ae8f110-b70d2bc17f390a7fc1fd9e3e69c215cb25e6331d7218c479f779bcced1326092, Version: 25d34710bdd63bed70a22b8204c3e1d0942dc237e098600e3896b4334c6d784f

[erik-werner]

Reversed to draft PR as I think the sampling of F might not match the updated equation in the cell above it. I'll try to figure this out before marking it as Ready for review.

[erik-werner]

This turned into a bit of a rabbit hole which I unfortunately don't have time to dive into right now. I propose to just merge in the update to v5 that fixes the crash, and then I'll write an issue for the rest of the work that I think should be done.

[erik-werner]

I've added an issue here: #587.

[twiecki]

This is great - thanks @erik-werner!