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

feat(face-landmarks) merge face expressions and face centering #11283

Merged
merged 4 commits into from
Apr 4, 2022

Conversation

quitrk
Copy link
Contributor

@quitrk quitrk commented Mar 31, 2022

No description provided.

@quitrk quitrk force-pushed the tavram/workers branch 3 times, most recently from 88172f7 to 832b6a4 Compare April 1, 2022 12:18
@quitrk quitrk changed the title [wip] merge face recognition workers feat(face-landmarks) merge face expressions and face centering Apr 1, 2022
@saghul
Copy link
Member

saghul commented Apr 1, 2022

@gabiborlea can you pl review too?

@gabiborlea
Copy link
Contributor

@saghul seems to be fine

Copy link
Member

@saghul saghul left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM with a small question.

react/features/facial-recognition/actions.js Show resolved Hide resolved
react/features/facial-recognition/actions.js Show resolved Hide resolved
react/features/facial-recognition/actions.js Show resolved Hide resolved
@quitrk
Copy link
Contributor Author

quitrk commented Apr 4, 2022

Jenkins please test this please

@quitrk quitrk merged commit 2c165d4 into master Apr 4, 2022
@quitrk quitrk deleted the tavram/workers branch April 4, 2022 13:09
pull bot pushed a commit to e4basil/jitsi-meet that referenced this pull request Apr 4, 2022
…#11283)

* feat(face-landmarks) merge face expressions and face centering

* code review

* code review

* code review
ankit-programmer pushed a commit to ankit-programmer/jitsi-meet that referenced this pull request May 7, 2022
…#11283)

* feat(face-landmarks) merge face expressions and face centering

* code review

* code review

* code review
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants