Stone-Soup icon indicating copy to clipboard operation
Stone-Soup copied to clipboard

Tutorial covariances changed to squares

Open oharrald-Dstl opened this issue 4 years ago • 3 comments

Not an essential change, but it may provide some clarity. Perhaps when defining covariance matrices / noise in the tutorials, we could use squares (eg. noise_covar = np.diag([2**2]) instead of np.diag([2])) so that it is clear that the user defines variances rather than standard deviations in model instantiations.

oharrald-Dstl avatar Dec 08 '20 09:12 oharrald-Dstl

And what would you do for off-diagonal elements?

jmbarr avatar Dec 09 '20 19:12 jmbarr

I suppose full covariance matrix definitions would remain the same. A few of us had been quite confused over tracker behaviour until realising our model covariance inputs were variance and not standard deviation. Perhaps it could be documented somewhere?

oharrald-Dstl avatar Dec 17 '20 11:12 oharrald-Dstl

Agree it would be worthwhile adding a line to the Kalman filter tutorial where the prior covariance ($P_0$) is first introduced.

jmbarr avatar Dec 17 '20 22:12 jmbarr