[eig_circulant ] Some updates on text and code#952
Conversation
|
🤖 Status note for a future session — from a maintainer investigation on 2026-07-08 into why open-PR previews 404. Context only, not instructions. Netlify preview: https://pr-952--sunny-cactus-210e3e.netlify.app/ currently returns 404. Why previews are down (repo-wide findings)1. This branch is stale — 32 commits behind 2. The arviz failure was a red herring — do NOT pin arviz or rewrite plotting. A 2026-07-07 rebuild also failed in Recommended first step for this PRUpdate this branch to This PR touches: |
📖 Netlify Preview Ready!Preview URL: https://pr-952--sunny-cactus-210e3e.netlify.app Commit: 📚 Changed LecturesBuild Info
|
Hi @jstac @thomassargent30
This PR updates the
eig_circulantlecture following #946. I’d be grateful if you could take a look when you have a chance.In response to @jstac 's comments, I have incorporated a little more motivation in the overview and added a formal definition of a circulant matrix using the
sphinx-proofdefinition environment.This PR also makes a set of small text, notation, and code cleanups discussed in the issue, including removing unnecessary Numba use, clarifying the cyclic-shift convention, fixing the determinant formula, updating transpose notation, and correcting a few typos.
I will open a separate issue for adding exercises, since that seems like a natural follow-up and can be handled separately.
Best,
Longye