Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

PR to fix deployment examples #28

Open
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

jmarine
Copy link

@jmarine jmarine commented Jun 15, 2020

Hi,
I had problems in docker image with examples showed in deployment menu, due to unexpected signature_name ("predict") and incorrect JSON syntax of POST data in "curl" command.
This PR fixes:

  • "signature_name" changed to "serving_default", that worked for me with tabular and image models.
  • JSON syntax in POST data (using "json.dumps" function, instead of str).

Thank you very much for publishing the source code of this project with Apache License, v2.0.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant