Aurora Cramer

I think it might still be a decent idea, but as of now there's no rush towards an OOP model. The main benefit I suppose would be so that you...

That sounds right! You should be able to construct a new model stopping at the appropriate level and selectively load the model weights up to that layer from the existing...

If this is something we want for our v1.0.0 release, then help would be much appreciated @bmcfee. I have no experience with this; if you would be able to take...

SGTM!

Another question that comes up if we support multi-GPU support is what the default number of GPUs used is. Should it be just 1? Or should it be the maximum...

Fair enough, let's not prioritize this for the MVP. Supporting multiple GPUs wouldn't require saving additional model files, we can make each model supported by multi-GPU inference after loading it

Good catch! We'll fix the documentation in the next release. Regarding the slowdown, my guess is that it could be an issue with memory if TensorFlow isn't properly garbage collecting...

The documentation should be correct now, addressed in #72 Feel free to let us know if the slowdown was resolved!

Yeah, this is pretty annoying. You can use `--branch --single-branch` when cloning to only clone master. We can add it to the README for now. Longer term, I'm not sure...

Ahh actually the Zenodo page where the data is hosted indicates that you have to request access so this probably can't be added, right?