Expand the api of esp-who to fully fledged API (AIS-2051)

[mike88macedon]

Checklist

• Checked the issue tracker for similar issues to ensure this is not a duplicate.
• Described the feature in detail and justified the reason for the request.
• Provided specific use cases and examples.

Feature description

Taking into consideration the following code :

auto model = new PedestrianDetect();
auto detect = new WhoDetectAppLCD({{255, 0, 0}});
detect->set_cam(cam);
detect->set_lcd(lcd);
detect->set_model(model);
// detect->set_fps(5);
detect->run();

We can only set the cam,model,lcd and run the detect-->run task. But we are missing the functionallity of retrieving the detected results on the image.

Use cases

Expand the API so we can get the detected results on the image it can be callback function that can return the detected results on an image frame. With this we will be able to do further processing with that data like sending it to a server or something like that. With the current implementation we are only limited to display the detected results on the image.

Alternatives

No response

Additional context

No response

[100312dog]

@mike88macedon
https://github.com/espressif/esp-who/blob/master/components/who_detect/who_detect_base.hpp#L41
https://github.com/espressif/esp-who/blob/master/components/who_detect/who_detect_lcd.cpp#L26
https://github.com/espressif/esp-who/blob/master/components/who_detect/who_detect_term.cpp#L8

It's easy to achieve this by overriding the on_new_detect_result() function.
I can add something like set_new_detect_result_cb() in the next version.

[mike88macedon]

@mike88macedon https://github.com/espressif/esp-who/blob/master/components/who_detect/who_detect_base.hpp#L41 https://github.com/espressif/esp-who/blob/master/components/who_detect/who_detect_lcd.cpp#L26 https://github.com/espressif/esp-who/blob/master/components/who_detect/who_detect_term.cpp#L8

It's easy to achieve this by overriding the on_new_detect_result() function. I can add something like set_new_detect_result_cb() in the next version.

Thanks that would be more convenient if we can implement the callback then deriving my own class that can display the results on the lcd and getting the detected metadata results on the on_new_detect_result().

[100312dog]

@mike88macedon
The set_detect_result_cb callback function is added.
You can refer to WhoDetectAppLCD or WhoDetectAppTerm and call it in your own class.
If you want to keep the original functionality, just extend it. For example, display on lcd and sending to server. You can inherit the WhoDetectAppLCD and override the detect_result_cb, call the parent class's implementation then complete your own implementation.