feature: Implement Continuous<Vec<f64>> for MultivariateNormal#199
Conversation
|
Thanks for identifying a good solution here. I'm good to merge this, @henryjac thoughts? |
|
Hmm, I believe I merged this in a rush. I'd yet to get input here on #209 which reintroduces the coupling to internal and at API usage of |
|
As far as I could see, the problem of #194 (requiring the user to use possibly more than one version of |
|
I think you're right that both will work for now as it (and similar for later However, I think the traits that return |
|
Ah! You are right. I have not tested it but the second might not be much of a problem. |
|
Ah yes, it's on the other side. I'll write it as a separate issue, thanks for your input! |
Solves #194
The problem was the public dependency on
nalgebra::Dvector:Although statrs does not update its dependencies, users should easily sample from MultivariateNormal.
The users were using an updated version of nalgebra and the types had clashes.
Therefore, this implements the same trait for a type with guaranteed backward compatibility:
Vec.Maintaining or not the
nalgebra::DVectorin the public API is an open question that we did not touch.