probcomp / bayeslite

BayesDB on SQLite. A Bayesian database table for querying the probable implications of data as easily as SQL databases query the data itself.
http://probcomp.csail.mit.edu/software/bayesdb
Apache License 2.0
922 stars 64 forks source link

Fixing bug in code dumping models. #636

Closed Schaechtle closed 4 years ago

Schaechtle commented 4 years ago

itertools.groupby was not behaving as anticipated in 3 places (see 3 individual commits of this PR). Tested on Travis.

jar398 commented 4 years ago

Hmm... well as I remember this was difficult to test at the time. Thanks for figuring all this out!

Schaechtle commented 4 years ago

@jar398 yeah; testing was pain. I only found the bugs via end to-end ad hoc CI testing (including InferenceQL) while preparing to ship a demo.