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

Facenet face recognition #217

Closed
wants to merge 8 commits into from
Closed

Conversation

iasonth95
Copy link
Contributor

Here is an initial implementation of facenet running from ros. The face recognition node is not fully cleaned up since i expect to have some changes upon discussion in this file.

Copy link
Member

@MatthijsBurgh MatthijsBurgh left a comment

Choose a reason for hiding this comment

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

I am not going to review this. As I suggest we work together to create a new clean pkg.

@@ -13,8 +13,8 @@ from std_srvs.srv import Empty
from image_recognition_msgs.msg import Recognition, Recognitions, CategoryProbability, CategoricalDistribution
from image_recognition_msgs.srv import Recognize, Annotate
from image_recognition_util import image_writer


sys.path.append('/home/iason/ros/noetic/repos/github.com/tue-robotics/image_recognition/image_recognition_openface/src/image_recognition_openface')
Copy link
Member

Choose a reason for hiding this comment

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

This is only needed, when you are doing something wrong with your workspace or your editor.

Copy link
Contributor Author

@iasonth95 iasonth95 Feb 20, 2024

Choose a reason for hiding this comment

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

Indeed we can work on this together so to see the whole procedure and become more independent for my next tasks (to create new clean pkg) !

@MatthijsBurgh MatthijsBurgh deleted the facenet_face_recognition branch May 14, 2024 17:27
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.

None yet

2 participants