Tobias-Fischer / rt_gene

RT-GENE: Real-Time Eye Gaze and Blink Estimation in Natural Environments
http://www.imperial.ac.uk/personal-robotics
Other
361 stars 67 forks source link

About multiple persons in one image #87

Closed lindayuanyuan closed 3 years ago

lindayuanyuan commented 3 years ago

Hi, When I processed the pictures that have multiple persons in one image, It seemed that it processed multiple persons. However, only the results of the last person processed was saved. How to set up saving all the results of the same image? thanks.

Tobias-Fischer commented 3 years ago

Hello, could you please let us know which exact scripts you used? This will help to narrow down the problem.

lindayuanyuan commented 3 years ago

Hello, could you please let us know which exact scripts you used? This will help to narrow down the problem.

Hi, I am using the standalone version. Thanks for the reply.

lindayuanyuan commented 3 years ago

Hello, could you please let us know which exact scripts you used? This will help to narrow down the problem.

estimate_gaze_standalone.py

Tobias-Fischer commented 3 years ago

Hi, simply add the idx to the filename in https://github.com/Tobias-Fischer/rt_gene/blob/8aabdd34d9dccef69a9044a61494efb0695b333d/rt_gene_standalone/estimate_gaze_standalone.py#L93 and similarly the subject_id to the filename in https://github.com/Tobias-Fischer/rt_gene/blob/8aabdd34d9dccef69a9044a61494efb0695b333d/rt_gene_standalone/estimate_gaze_standalone.py#L120

It would be great if you contribute this in a pull request, the change should be straightforward.

lindayuanyuan commented 3 years ago

Hi, simply add the idx to the filename in

https://github.com/Tobias-Fischer/rt_gene/blob/8aabdd34d9dccef69a9044a61494efb0695b333d/rt_gene_standalone/estimate_gaze_standalone.py#L93

and similarly the subject_id to the filename in https://github.com/Tobias-Fischer/rt_gene/blob/8aabdd34d9dccef69a9044a61494efb0695b333d/rt_gene_standalone/estimate_gaze_standalone.py#L120

It would be great if you contribute this in a pull request, the change should be straightforward.

Thanks for the reply. I would love to contribute. I have made changes accordingly. Not sure if the pull request works properly. Yuanyuan

Tobias-Fischer commented 3 years ago

Fixed by #88. Many thanks @lindayuanyuan!