Hi @longye-tian,
Thanks again for the excellent PR in #897 — the new $X$ (random vector) vs $x$ (realization) distinction reads really well and makes the exposition much clearer.
As a small follow-up suggestion: the new convention is currently applied to the main exposition (from The basic idea through Convergence), but a couple of later spots still use lowercase $x$ for the random state, so the notation isn't quite consistent end-to-end:
-
Implementation section — the
LinearStateSpace equations still read
$$
x_{t+1} = A x_t + C w_{t+1}, \qquad y_t = G x_t + H v_t
$$
which uses lowercase $x_t$ / $y_t$ for the random state and signal.
-
Exercises — these still refer to the hidden state as ${x_t}$, $x_t$, $x_0$, etc. (e.g. Exercise 1's "the hidden state ${x_t}$ is in fact constant", and the squared-error expressions in Exercise 3).
A reader who has just internalized "uppercase = random variable, lowercase = a realization" then meets lowercase random states in these sections.
There's a reasonable case either way:
- Extend the convention to the Implementation section and Exercises so it's consistent throughout, or
- Leave it scoped to the conceptual exposition — one could argue the
LinearStateSpace API genuinely uses lowercase, so matching it there is fine.
What do you think?
Follow-up to #893 / #897.
Hi @longye-tian,
Thanks again for the excellent PR in #897 — the new$X$ (random vector) vs $x$ (realization) distinction reads really well and makes the exposition much clearer.
As a small follow-up suggestion: the new convention is currently applied to the main exposition (from The basic idea through Convergence), but a couple of later spots still use lowercase$x$ for the random state, so the notation isn't quite consistent end-to-end:
LinearStateSpaceequations still read$$
x_{t+1} = A x_t + C w_{t+1}, \qquad y_t = G x_t + H v_t
$$
which uses lowercase
A reader who has just internalized "uppercase = random variable, lowercase = a realization" then meets lowercase random states in these sections.
There's a reasonable case either way:
LinearStateSpaceAPI genuinely uses lowercase, so matching it there is fine.What do you think?
Follow-up to #893 / #897.