made Memory property of Collections, not Butler

nextml / NEXT

NEXT is a machine learning system that runs in the cloud and makes it easy to develop, evaluate, and apply active learning in the real-world. Ask better questions. Get better results. Faster. Automated.

http://nextml.org

Apache License 2.0

160 stars 53 forks source link

made Memory property of Collections, not Butler #176

Closed dconathan closed 7 years ago

dconathan commented 7 years ago

This PR addresses #173. Switches over from butler.memory to butler.algorithms.memory and butler.experiment.memory, etc. Basically every collection gets its own "bucket" in the RedisDB (by automatically prepending the key with the collection name + exp_uid).

I wasn't sure if anyone else is using butler.memory at the moment, so I left it in there, but it prints a deprecation warning and this will be removed in the future.

dconathan commented 7 years ago

Note there's a one-liner conflict with #174 because I randomly changed an == None to an is None ... once that PR is merged I'll fix the conflict.