[kalman] update RV notation#900
Conversation
Update the two "Constant value of state x_t" comments to X_t so the code comments match the uppercase random-variable convention used in the surrounding prose. The realized initial value x_0 comment stays lowercase by design. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
|
Reviewed — the uppercase/lowercase RV convention is applied correctly and consistently, and the new I pushed a small follow-up commit (dc7ca23) updating the two |
📖 Netlify Preview Ready!Preview URL: https://pr-900--sunny-cactus-210e3e.netlify.app Commit: 📚 Changed LecturesBuild Info
|
There was a problem hiding this comment.
Pull request overview
This PR updates the Kalman filter lecture (lectures/kalman.md) to consistently apply the “uppercase = random variable / lowercase = realization” convention across the Implementation section and Exercises, aligning with the notation introduced earlier in the lecture and requested in #898.
Changes:
- Updated the
LinearStateSpacemodel equations in the Implementation section to use$X_t$ /$Y_t$ for random state and signal variables. - Revised Exercises 1–4 text and code comments to use
$X_t$ for random states while keeping lowercase for realizations/estimates/code variables. - Clarified in Exercise 3 that the plotted squared errors are based on simulated realizations (and added an explicit note tying
x[:, t]to the realized$X_t$ ).
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
|
Many thanks @mmcky for the update and I've addressed the comments from Copilot as well. Best, |
Hi @mmcky
This PR updates the RV notation following #898 .
In particular, this extends the new uppercase/lowercase convention to the Implementation section and Exercises:
Best,
Longye