combust / mleap-docs

Documentation for MLeap
14 stars 23 forks source link

Fix TensorFlow Tutorial #25

Closed voganrc closed 3 years ago

voganrc commented 3 years ago

In MLeap's master branch, the TensorFlowModel graph argument has type Option, and the constructor requires a new graphBytes argument. So I updated the docs to reflect this:

https://github.com/combust/mleap/blob/2e42c0116702d494251433821958df0bfdc1ab35/mleap-tensorflow/src/main/scala/ml/combust/mleap/tensorflow/TensorflowModel.scala#L16

I also noticed that the example bundle under assets/bundles/tensorflow-model.zip cannot be deserialized with the latest release. So I re-ran the tutorial and re-zipped the resulting bundle.