Skip to content
Snippets Groups Projects
Commit 0777c2b4 authored by Tiago de Freitas Pereira's avatar Tiago de Freitas Pereira
Browse files

Removed bob.ip.mtcnn, bob.bio.caffe_face and bob.ip.caffe_extractor [skip ci]

parent b81d9284
No related branches found
No related tags found
Loading
Checking pipeline status
Loading
  • Owner

    @tiago.pereira this commit has broken bob.pad.face

  • Amir MOHAMMADI @amohammadi

    mentioned in commit c685cbe3

    ·

    mentioned in commit c685cbe3

    Toggle commit list
  • Author Owner

    So, are we going to drop these dependencies with caffe once and for all?

  • Author Owner
  • Author Owner

    I even archived the projects bob.ip.mtcnn, bob.ip.caffe_extractor and bob.bio.caffe_face

  • mentioned in issue bob.pad.face#33 (closed)

    Toggle commit list
  • Maintainer

    As mentioned somewhere already, bob.ip.mtcnn is not a hard requirement for bob.pad.face, so that should not be a problem to archive it. I'm currently updating bob.pad.face to see if this would be ok (bob.pad.face!100 (merged))

    @ageorge A little warning though: the way you are using FaceCropAlign.py won't be available anymore. This is a good opportunity to get rid of that and implement something cleaner ...

  • Maintainer

    @heusch I once tried to use the extractor from bob.ip.tensorflow_extractor, its here https://gitlab.idiap.ch/bob/bob.pad.face/blob/mtcnn/bob/pad/face/preprocessor/FaceCropAlign.py#L168

  • Maintainer

    Well, what I meant is that it would be a good occasion to get rid of FaceCropAlign.py altogether (for a lot of reasons, see bob.pad.face#30 (closed) for instance)

    Anyway, glad if you find a way not to use bob.ip.mtcnn !

0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment