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

Error in SeldonClient in debugger #1499

Closed
tomasz-sadowski opened this issue Mar 3, 2020 · 0 comments · Fixed by #1536
Closed

Error in SeldonClient in debugger #1499

tomasz-sadowski opened this issue Mar 3, 2020 · 0 comments · Fixed by #1536
Assignees
Labels
Milestone

Comments

@tomasz-sadowski
Copy link

tomasz-sadowski commented Mar 3, 2020

I receive strange error while using SeldonClient:

sc = SeldonClient(deployment_name="DEPLOYMENT_NAME",                    
                    namespace="NAMESPACE_NAME",
                    transport="rest",
                    gateway="istio",
                    gateway_endpoint="127.0.0.1:31380"
                )
r = sc.predict(transport="rest", data=data, payload_type="ndarray")```

Error message:

File "/home/tsadowsk/.local/lib/python3.7/site-packages/seldon_core/seldon_client.py", line 380, in predict
self._validate_args(**k)
TypeError: _validate_args() got multiple values for argument 'self'

Potential culprit:
https://github.com/SeldonIO/seldon-core/blob/master/python/seldon_core/seldon_client.py#L210
I run debugger and this statement does not delete "self" key from self.configLooks like line above the one should be replaced with:

self.config = locals().copy()

instead of:

self.config = locals()

python 3.7.5
seldon-core: 1.0.2
Fedora 30 (edited)
python/seldon_core/seldon_client.py:210

        del self.config["self"]

Conversation link:
https://seldondev.slack.com/archives/C8Y9A8G0Y/p1583237412118600

https://github.com/SeldonIO/seldon-core|SeldonIO/seldon-coreSeldonIO/seldon-core | Added by GitHub

@tomasz-sadowski tomasz-sadowski added bug triage Needs to be triaged and prioritised accordingly labels Mar 3, 2020
@ukclivecox ukclivecox removed the triage Needs to be triaged and prioritised accordingly label Mar 6, 2020
@ukclivecox ukclivecox added this to the 1.2 milestone Mar 6, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants