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

Allow Inference Graphs to mix Protocols with the Executor #2299

Closed
mwm5945 opened this issue Aug 19, 2020 · 0 comments · Fixed by #2574
Closed

Allow Inference Graphs to mix Protocols with the Executor #2299

mwm5945 opened this issue Aug 19, 2020 · 0 comments · Fixed by #2574
Assignees
Projects
Milestone

Comments

@mwm5945
Copy link
Contributor

mwm5945 commented Aug 19, 2020

Stems from #2189

Requests to the Executor should be allowed to be either REST or GRPC, independent of what the graph is using.

That is, if a graph uses REST, requests to the Executor can be in GRPC, and the executor will convert the request to REST internally for calling graph nodes.

@mwm5945 mwm5945 added the triage Needs to be triaged and prioritised accordingly label Aug 19, 2020
@ukclivecox ukclivecox added priority/p1 and removed triage Needs to be triaged and prioritised accordingly labels Aug 20, 2020
@ukclivecox ukclivecox added this to To do in 1.4 via automation Aug 20, 2020
@ukclivecox ukclivecox added this to the 1.4 milestone Aug 20, 2020
@ukclivecox ukclivecox self-assigned this Oct 1, 2020
@ukclivecox ukclivecox moved this from To do to In progress in 1.4 Oct 12, 2020
@axsaucedo axsaucedo added this to To do in 1.5 via automation Oct 15, 2020
@axsaucedo axsaucedo removed this from In progress in 1.4 Oct 15, 2020
@axsaucedo axsaucedo modified the milestones: 1.4, 1.5 Oct 15, 2020
@ukclivecox ukclivecox moved this from To do to In progress in 1.5 Oct 22, 2020
@ukclivecox ukclivecox moved this from In progress to In Review in 1.5 Nov 5, 2020
1.5 automation moved this from In Review to Done Nov 16, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
No open projects
1.5
Done
Development

Successfully merging a pull request may close this issue.

3 participants