stripe / rainier

Bayesian inference in Scala.
https://rainier.fit
Apache License 2.0
432 stars 51 forks source link

Get rid of Variable #441

Closed avibryant closed 4 years ago

avibryant commented 4 years ago

We previously had Variable as a common ancestor for Placeholder and Parameter. This reorganizes things a bit so that:

This is in anticipation of changing Column to be considered Constant (so that you have a Constant super trait over Column and Scalar). However that is a much bigger change that's hard to see to the end of, so for now Column is still NonConstant.

This also dumps rainier-cats and rainier-scalacheck.