search

QInfer / python-qinfer

Library for Bayesian inference via sequential Monte Carlo for quantum parameter estimation.
BSD 3-Clause "New" or "Revised" License
93 stars 31 forks source link

Old Distributions #99

Closed ihincks closed 8 years ago

ihincks commented 8 years ago

There are a couple of distributions in qinfer/distributions.py that I think have been superceded by new ones in qinfer/tomography/distributions.py and should be deprecated:

ihincks commented 8 years ago

Also, I think MVUniformDistribution has a confusing name. Perhaps it should eventually just be replaced with a more general Dirichlet distribution?

cgranade commented 8 years ago

I agree, and will PR soon to mark those as deprecated. I also think it makes sense, in light of your point about MVUniformDistribution having a confusing name to deprecate that as well, pending a better named or more general class.

ihincks commented 8 years ago

Thanks. The PR could also include a link between the tomography.distributions and distributions in the API docs.

cgranade commented 8 years ago

That's a good suggestion, I'll add that as well.