ci2cv / face-analysis-sdk

Facial detection, landmark tracking and expression transfer library for Windows, Linux and Mac
face.ci2cv.net
Other
416 stars 209 forks source link

several branchs containing fixes have not been merged. #3

Open linas opened 10 years ago

linas commented 10 years ago

I'm looking over the code, and I see that there are three branches containing fixes for build breaks, and two adding new features. None of the authors of these branches have submitted pull requests (yet), so it makes sense that they haven't been merged. However ...

Given that all of these are at least a month old, and some almost a year old ... isn't it time to spur activity? Seems like the fixes should be picked up. Specifically, I see:

https://github.com/tetard/face-analysis-sdk/ --fixes for apple & LLVM compiler https://github.com/hansonrobotics/face-analysis-sdk/ -- fixes for compiling on debian wheezy https://github.com/blackibiza/face-analysis-sdk/ -- fixes for Qt5

Next: the new features: https://github.com/supermartian/face-analysis-sdk/ -- stuff for recognizing emotions. Hard to tell how complete/incomplete/buggy this is.

https://github.com/iCogLabs/face-analysis-sdk/ -- beats me. Some kind of hacky stuff .. ??

madduci commented 10 years ago

You can find my pull request already, including fixes for building with Qt5 and OpenCV 3.0.0, at least on ubuntu machine :)

linas commented 8 years ago

I have merged many/most branches into https://github.com/hansonrobotics/face-analysis-sdk

I did not merge https://github.com/supermartian/face-analysis-sdk/ https://github.com/iCogLabs/face-analysis-sdk/

These looked incomplete.