★ wanayoo — archive 1999 https://github.com/robmarkcole/deepstack-python/pull/25Nouvelle recherche | Portail wanayoo
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

Include image mime in request #25

Open
wants to merge 3 commits into
base: master
from

Conversation

@bestlibre
Copy link

@bestlibre bestlibre commented Jul 22, 2020

Fix for #24 . I have tested with a local install of tensorflow-lite-server, it's working. I don't have an instance of deepstack to test against it and I'm not familiar with the test suite to add test. I hope this PR is still usefull.

bestlibre added 3 commits Jul 22, 2020
@robmarkcole
Copy link
Owner

@robmarkcole robmarkcole commented Jul 22, 2020

@bestlibre how did you find that solution? Possibly here

@bestlibre
Copy link
Author

@bestlibre bestlibre commented Jul 22, 2020

I found the solution based on the curl command in the readme of tensorflow-lite-server and the documentation of requests. The first part of the solution was with a fixed mime type and then I search how to get the mime type from a byte array which lead me to filetype.

@robmarkcole
Copy link
Owner

@robmarkcole robmarkcole commented Jul 23, 2020

Unfortunately on deepstack official this approach raises error: {'success': False, 'error': 'No valid image file found'}

You can run deepstack official with:

docker run -e VISION-DETECTION=True -v localstorage:/datastore -p 5000:5000 -e API-KEY="" --name deepstack deepquestai/deepstack:cpu-x3-beta
@robmarkcole robmarkcole changed the title Tensorflow server compat Include image mime in request Jul 23, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.