tensorflow / lucid

A collection of infrastructure and tools for research in neural network interpretability.
Apache License 2.0
4.65k stars 655 forks source link

Defensive Programming of Model.save() #188

Closed colah closed 4 years ago

colah commented 5 years ago

Model.save() is the new recommended way for people to import models into Lucid. While pairing with @Newmu the other day, we ran into a number of issues where we made silly mistakes, but it took a bit of time to debug.

Since Model.save() is such an essential user facing function, I think it warrants heavy defensive programming, good error messages, and documentation to protect against this.

Some checks that would have been nice: