-
Notifications
You must be signed in to change notification settings - Fork 447
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
Http Server #37
base: dev
Are you sure you want to change the base?
Http Server #37
Conversation
Add documentation to project.
Add Google Analytics to documentation and fix images.
Hi, thanks a lot for the PR! It's a great idea to add a Flask server to the repo - there are a few things that need to be addressed in your PR:
The model should be provided as an argument when starting the Flask server. This has the downside that not both models are available in the same server, but makes it a lot more flexible for other users to serve their own models.
Many thanks! |
Hi, thanks for considering it! Please have a look at the server.py changes and let me know if anything else needs to be done! |
Up :D |
It needed a few more bug fixes, but I'm using this now as well. Sorry not to push the fixes but I needed some personal customisations too. Anyway, thanks. Hope this gets merged soon! |
Hi guys, thank you for the great work and sharing it ! |
Added a dockerfile which exposed the model via rest api.
flask server will run on port 5005
URL - POST /query/{model} - model can be aesthetics or technical
Parameters - [
"<image_url_1>",
"<image_url_2>"
]
Response -