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

send_feedback() got an unexpected keyword argument 'routing' #1321

Closed
zjorgensenbits opened this issue Jan 7, 2020 · 3 comments
Closed

send_feedback() got an unexpected keyword argument 'routing' #1321

zjorgensenbits opened this issue Jan 7, 2020 · 3 comments

Comments

@zjorgensenbits
Copy link

zjorgensenbits commented Jan 7, 2020

When sending a request to the feedback endpoint of my deployed model, I get a Microservice Error and I see the following in the container log: send_feedback() got an unexpected keyword argument 'routing'.

This is the data payload I'm sending: '{"reward": 1.0, "truth": {"data":{"names":["a","b"],"tensor":{"shape":[2,1],"values":[0,1]}}}, "response": {"data":{"names":["a","b"],"tensor":{"shape":[2,1],"values":[0,1]}}},"request": {"data":{"names":["a","b"],"tensor":{"shape":[2,1],"values":[0,1]}}}}'

@ukclivecox
Copy link
Contributor

It looks like it is adding routing to the call to your microservice.

return user_model.send_feedback(
features, feature_names, reward, truth, routing=routing
)

Do you have this defined in your signature for the method?

Can you explain more how you created the example? Maybe one of our docs/examples is out of date?

@zjorgensenbits
Copy link
Author

Ahh, yes, I was using an image created from the example here: seldon-core/wrappers/s2i/python/test/model-template-app/MyModel.py
which is out of date. I fixed the send_feedback function signature and it seems to work now. I'm not getting an error, at least. Should I see any kind of response?

@ukclivecox
Copy link
Contributor

Send feedback does not give a payload response at present - so a 200 http code would be all.

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

No branches or pull requests

2 participants