This repository has been archived by the owner on Aug 26, 2024. It is now read-only.
Setup ssh tunnel in example #29
Open
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Signed-off-by: Eduardo Apolinario <eapolinario@users.noreply.github.com>
Signed-off-by: Eduardo Apolinario <eapolinario@users.noreply.github.com>
Signed-off-by: Eduardo Apolinario <eapolinario@users.noreply.github.com>
kumare3
reviewed
Feb 15, 2022
| warnings.simplefilter("ignore") | ||
| # Setup ssh tunnel used to route connections from gradio.app to localhost on the specific | ||
| # local port | ||
| share_url = networking.setup_tunnel(local_server_port=local_port, endpoint=None) |
There was a problem hiding this comment.
experience is still not the best?
There was a problem hiding this comment.
what if we add, start_server, start_server_public_endpoint?
There was a problem hiding this comment.
oh, so a flag to control if we want to start a server, but what would be the use of start_server_public_endpoint? This is part of the gradio API, apparently they allow for proxy servers other than gradio.app (probably for testing or maybe segmenting by stage?). In our case we would only use the production one until we decide if we want to have this implemented on our side.
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR introduces an ssh tunnel (by way of remote port forwarding) such that connections coming from the gradio.app end up hitting the local port where we're running the uvicorn server.
The intent behind this is for us to experiment with this kind of experience and eventually decide if we should offer this as part of mulearn.
Notice how we changed the http verb for the predict method from a get to post as the gradio proxy does not allow for get requests. This also affects the aws lambda work, so we might as well get this change in.
TODO
/requests, including bash and python./requestsSigned-off-by: Eduardo Apolinario eapolinario@users.noreply.github.com