Skip to content
GitLab
Projects Groups Topics Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in
  • bob.bio.face bob.bio.face
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributor statistics
    • Graph
    • Compare revisions
  • Issues 24
    • Issues 24
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 6
    • Merge requests 6
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages and registries
    • Packages and registries
    • Package Registry
    • Infrastructure Registry
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • bobbob
  • bob.bio.facebob.bio.face
  • Merge requests
  • !179

Replacing bob.ip.base.FaceEyesNorm

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Tiago de Freitas Pereira requested to merge new-cropper into master Apr 05, 2022
  • Overview 11
  • Commits 9
  • Pipelines 17
  • Changes 31

This is next chapter on our C++ deprecation crusade (https://gitlab.idiap.ch/groups/bob/-/milestones/4#tab-issues).

In this MR I'm deprecating a VERY sensible part of our pipeline the C++ bob.ip.base.FaceEyesNorm.

I've implemented in python an EXACT implementation of our face alignment in pure python. Amazing, this was reduced to less than 100 lines of code (https://gitlab.idiap.ch/bob/bob.bio.face/-/blob/new-cropper/bob/bio/face/preprocessor/Base.py#L134).

Furthermore, I've addressed the aliasing issue reported in #74 by changing the bi-linear interpolation to a bi-cubic interpolation (thanks @lcolbois).

Follow an example of our old crop (bilinear) and the new crop (bicubic), side by side (final crop with 112x112, arcface coordinates).

new_interpolation

Follow also a pair of images from one of our test cases.

New test case Crop m_new

Old test case crop m_reference

I still need to run some FR experiments on it and change some other face cropper classes.

With that, we can start moving forward towards detaching bob.ip.base from this package.

This is still a WIP

ping @mguenther, @linghuy @lcolbois @amohammadi @ydayer @flavio.tarsetti

Edited Apr 13, 2022 by Tiago de Freitas Pereira
Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: new-cropper