see no errors (other than ipynb files)
This looks good to me! Since we are doing doc changes, it might be a good idea to also have a screen shot of the effected pages just to make sure we aren't pushing up bad changes (It also makes it easier for folks to review)?
Why are we defining EnumValues when we are already specifying Enums above? It looks very verbose? (Also should be fixed in a different revision, that's just a question for me XD).
EnumValue is the wrapper class that allows you to specify other (optional) information in addition to just the entry in the enum. It behaves much like Field does for Dicts