pymc icon indicating copy to clipboard operation
pymc copied to clipboard

Raise informative error if dictionary is passed as observed of DensityDist

Open ricardoV94 opened this issue 3 years ago • 2 comments

This is a common source of confusion in people switching from V3 to V4. We could print an informative message when users try to do it.

ricardoV94 avatar Aug 06 '22 12:08 ricardoV94

It is still a confusion for me (I had to use pm.Potential) 🙈 , what is the correct type for the observed parameter? (I can add a fix accordingly)

juanitorduz avatar Aug 31 '22 07:08 juanitorduz

observed should be exactly the same as for other distributions (pd.Series, np.array, pm.Data). Dictionaries are no longer a thing.

ricardoV94 avatar Aug 31 '22 10:08 ricardoV94

Closed by #6292

michaelosthege avatar Nov 15 '22 17:11 michaelosthege