pyro-ppl / brmp

Bayesian Regression Models in Pyro
Apache License 2.0
69 stars 8 forks source link

Rename some things #72

Closed null-a closed 4 years ago

null-a commented 4 years ago

This PR (only) renames a few things.

The motivation is to have the new define_model method (#68) return a Model instance, rather than a DefineModelResult. To make this possible I've renamed the existing Model class (in backend.py) to Assets. (Assets is just a tuple of Python code/functions generated by a backend, given a model description.)

I've also renamed GenResult to AssetWrapper. This is much less important than the above, but it seemed a bit nicer to me.