-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
Movement on Python API? #428
Comments
The interface of OpenFace has changed quite a bit since the original PRs. I want to add the binding eventually, but other additions to OpenFace came first. The OpenFace you link to is quite a different project, it does face recognition rather than landmark detection and facial behavior analysis. |
Oh interesting, so to pick up emotions like angry or sad, then this lib is more appropriate ?
If so, then I could still volunteer to get the bindings done, assuming my work will have use for it.
…Sent from my iPhone
On May 8, 2018, at 11:22 AM, Tadas Baltrusaitis ***@***.***> wrote:
The interface of OpenFace has changed quite a bit since the original PRs. I want to add the binding eventually, but other additions to OpenFace came first.
The OpenFace you link to is quite a different project, it does face recognition rather than landmark detection and facial behavior analysis.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub, or mute the thread.
|
Yes, this OpenFace is much more suitable for emotion recognition. If you were to create python bindings I would include them into the main repo! However, I would wait for a week or two as I will be releasing OpenFace 2.0 very shortly and that will change the interface a bit. |
@TadasBaltrusaitis Okay great, then I'll wait a bit till you reorganize it as you want. Appreciate your quick replies. |
I have updated the OpenFace version and the interface should be reasonably stable now. |
The python interface about Aus recognition can also be updated? Thansk for your reply! |
@fxfactorial @TadasBaltrusaitis dear gentlemen, any progress on openface 2.0 python APIs? |
Hi, for instructions on how to install on Ubuntu, please have a look here: https://github.com/TadasBaltrusaitis/OpenFace/wiki/Unix-Installation Currently I'm not working on the python API. |
@TadasBaltrusaitis Minor misunderstanding. The installation was not a (big) problem. I don’t know how to use it (the files are windows cpp files with vcx project files. How shall we compile/test/run openface on Ubuntu AFTER installation? There are no python files for testing, correct? |
Sorry, I'm still not exactly sure what the question is. What do you mean how to use it? The Compilation on Ubuntu is described in the wiki page I linked to (https://github.com/TadasBaltrusaitis/OpenFace/wiki/Unix-Installation), it also lists some example tests you can run to make sure building worked (https://github.com/TadasBaltrusaitis/OpenFace/wiki/Unix-Installation#actual-openface-installation). Testing and running OpenFace after the compilation is done can be done through a command line interface described here - https://github.com/TadasBaltrusaitis/OpenFace/wiki/Command-line-arguments |
@ewagner70 look in the build dir's bin directory for the executables to run. I might return to doing a Python API if my current project continues (since I accomplished my other desired task directly in C++.) |
@TadasBaltrusaitis @fxfactorial Thx for the hints, but what I get when trying to compile the cpp files is
any ideas? I'm running on Ubuntu 18.04 ... |
Maybe you don't have libstdc++ installed. You should follow the pre-req directions carefully, wiki says what prereqs needed on system. |
well, from my perspective, I've installed the respective packages see the following libstdc++ list
|
Please keep this thread only for Python API issues, not debugging your issues. |
ok ... opened new issue #460 |
Hi, Any news about the Python API? |
Hi, any python API news? |
Looks like there are two open PRs
#217
#207
for adding a python API.
@TadasBaltrusaitis is there a reason for delay? I can also do the python bindings as a fresh PR if need be.
The text was updated successfully, but these errors were encountered: