exadel-inc / CompreFace

Leading free and open-source face recognition system
https://exadel.com/accelerator-showcase/compreface/
Apache License 2.0
5.71k stars 775 forks source link

Update docker image, fix GPU vision & Deps to python 3.11 #1050

Closed jack60612 closed 1 year ago

github-actions[bot] commented 1 year ago

CLA Assistant Lite bot:
Thank you for your submission, we really appreciate it. Like many open-source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution. You can sign the CLA by just posting a Pull Request Comment same as the below format.


I have read the CLA Document and I hereby sign the CLA


You can retrigger this bot by commenting recheckcla in this Pull Request

jack60612 commented 1 year ago

I have read the CLA Document and I hereby sign the CLA

jack60612 commented 1 year ago

recheckcla

pospielov commented 1 year ago

Hi, thanks for your contribution. It doesn't make sense to use tensorflow image and then install mxnet. So this fix makes sense only for the default FaceNet build. As for mxnet it looks like it doesn't support cuda 11.8, so it won't work at all. This is why we are researching how to migrate to another library from mxnet.

jack60612 commented 1 year ago

Mxnet works for me(cuda 11.8), I was using arcface gpu + this image and kube

jack60612 commented 1 year ago

Hi, thanks for your contribution. It doesn't make sense to use tensorflow image and then install mxnet. So this fix makes sense only for the default FaceNet build. As for mxnet it looks like it doesn't support cuda 11.8, so it won't work at all. This is why we are researching how to migrate to another library from mxnet.

I think i can change it back to the nvidia image tbh, it just makes it install faster.

pospielov commented 1 year ago

I passed this pull request to our developer who researches this problem, but he is on vacation now

jack60612 commented 1 year ago

I passed this pull request to our developer who researches this problem, but he is on vacation now

ok, sounds good.

jack60612 commented 1 year ago

CLA Assistant Lite bot: Thank you for your submission, we really appreciate it. Like many open-source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution. You can sign the CLA by just posting a Pull Request Comment same as the below format.

I have read the CLA Document and I hereby sign the CLA

You can retrigger this bot by commenting recheckcla in this Pull Request

I have read the CLA Document and I hereby sign the CLA

Anatolii-R commented 1 year ago

Mxnet works for me(cuda 11.8), I was using arcface gpu + this image and kube

Hi Jack I can see that the CPU build is working, but I'm curious about how you managed to incorporate CUDA 11.8 into the GPU build. Would you be able to share some insights on that?

Anatolii-R commented 1 year ago

I'm encountering an error while attempting to build an image for use with a GPU.

Screenshot 2023-04-24 at 17 12 11
jack60612 commented 1 year ago

That should be solved by the changes in dependency.py