search

probcomp / bdbcontrib

BayesDB contributions, including plotting, helper methods, and examples
http://probcomp.csail.mit.edu/bayesdb
Apache License 2.0
9 stars 6 forks source link

Why is "name" a necessary argument of Population? #145

Closed leocasarsa closed 8 years ago

leocasarsa commented 8 years ago

If the bdb file already exists, it already has one and only one population table associated with it. Asking for the user to input a name is only sensible when the bdb file does not exist already.

gregory-marton commented 8 years ago

A bdb may have multiple populations stored in it. Right now none of that is explicit so we don't even have a place to start. Once it's explicit, we might reasonably have a default, but requiring a name doesn't seem onerous. We should have good messages in case there is an apparent conflict or duplication. In person you seemed to like where we were headed on that score with the recently updated message that does list_tables and list_generators on the bdb for you, so I'm closing this for now. We'll improve on that when we can.