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
README overhaul #15
Comments
Any chance for a quick update that just shows what the index creation call should be with v7? For those of us who aren't seasoned ElasticSearch users it is totally not clear how we need to change it. |
For v7 just omit the
Otherwise you simply have to insert a
The WIP version of py-image-dedup is able to create such an index automatically. I currently just do not have the time to get to it :( |
Awesome. That worked. Very helpful for those of us who have never used ElasticSearch. |
Actually, while it seemed to create it, after running the script processing thousands of images I still seem to have an empty index. It looks like it is fetching against it but not inserting into it. Any best way to debug why it might not be adding images to the index?
|
I suspect it is because there are 400 errors on insert although it isn't clear why - |
py-image-dedup probably doesnt use the correct request format for your version of elasticsearch. v1.0.0 can not work around this without changing the code. You can try with the latest version from master which should detect your EL version automatically. There is no release for that version yet, its on my TODO list. |
@jasontitus I have invested a couple hours, updated dependencies and fixed related stuff. I have not yet released a new version since it doesn't feel polished enough yet, but you can try the latest version from master or dockerhub if you want to give it a try. |
Some things about how py-image-dedup works changed since v1.0.0 and the README needs some guidance on how to use the docker-compose file. A big overhaul of the README is necessary.
Specify
image-match
supporting elasticsearch v6 as well as v7 is now used the cumbersome package dependency section for it can be removeddocker-compose
The text was updated successfully, but these errors were encountered: