search

stan-dev / stanc3

The Stan transpiler (from Stan to C++ and beyond).
BSD 3-Clause "New" or "Revised" License
140 stars 44 forks source link

rename cov_matrix to spd_matrix #472

Open bob-carpenter opened 11 years ago

bob-carpenter commented 11 years ago

"spd" is for symmetric positive definite. The problem with "cov_matrix" is that they can be used for precision matrices, too, making the declaration look confusing.

This will also match the functions Marcus has added (but aren't quite yet through the pipe into the Stan language).

We should just deprecate cov_matrix with a warning.

We should also rename the Cholesky factors to match.

(Thanks to Ben for noticing the problem and Marcus for suggesting a better name.)

jgabry commented 8 years ago

Is the plan still to deprecate cov_matrix in favor of spd_matrix? If so I'll add this to the v3 milestone (since it will break backwards compatibility) and keep it open. If not I'll close this.

bob-carpenter commented 8 years ago

Let me bring it up on stan-dev.

On Mar 4, 2016, at 1:52 PM, Jonah Gabry notifications@github.com wrote:

Is the plan still to deprecate cov_matrix in favor of spd_matrix? If so I'll add this to the v3 milestone (since it will break backwards compatibility) and keep it open. If not I'll close this.

— Reply to this email directly or view it on GitHub.

VMatthijs commented 5 years ago

Is this still desired, or should we close the issue?

bob-carpenter commented 5 years ago

Yes, it's still desired.

"cov_matrix" should be deprecated and replaced with "spd_matrix".

"cholesky_factor_cov" should be deprecated and replaced with either just plain "cholesky_factor" or "cholesky_factor_spd" (the latter clarifies it's full rank).

I don't like that "cov" ("spd") is a prefix in one type and a suffix in another, but we should probably stick with this now unless people feel strongly that it should be rationalized going forward.

On Dec 13, 2018, at 8:27 AM, Matthijs Vákár notifications@github.com wrote:

Is this still desired, or should we close the issue?

— You are receiving this because you were assigned. Reply to this email directly, view it on GitHub, or mute the thread.