SLCP Deploy Option doesn't work with xg24 Demo

SethW · October 14, 2022, 9:13pm

Hey @mateusz, in the last two commits of the xg24 example code:

The label_detected member was added to ei_impulse_result_t.

edgeimpulse/firmware-silabs-xg24/blob/main/ei-model/edge-impulse-sdk/classifier/ei_classifier_types.h#L62

      
        
            
            
typedef struct {
            #if EI_CLASSIFIER_OBJECT_DETECTION == 1
                ei_impulse_result_bounding_box_t *bounding_boxes;
                uint32_t bounding_boxes_count;
            #else
                ei_impulse_result_classification_t classification[EI_CLASSIFIER_LABEL_COUNT];
            #endif
                float anomaly;
                ei_impulse_result_timing_t timing;
                int32_t label_detected;
            } ei_impulse_result_t;
            
            
#endif // _EDGE_IMPULSE_RUN_CLASSIFIER_TYPES_H_

Then in the display results function its used.

github.com

edgeimpulse/firmware-silabs-xg24/blob/main/src/inference/ei_run_audio_impulse.cpp#L67

      
        
            #if EI_CLASSIFIER_HAS_ANOMALY == 1
                    ei_printf("    anomaly score: ");
                    ei_printf_float(result->anomaly);
                    ei_printf("\r\n");
            #endif
            }
            
            
static void display_results(ei_impulse_result_t* result)
            {
                if(continuous_mode == true) {
                    if(result->label_detected >= 0) {
                        ei_printf("LABEL DETECTED : %s\r\n", result->classification[result->label_detected].label);
                        ble_send_classifier_output(result->classification[result->label_detected].label);
                        timing_and_classification(result);
                    }
                    else {
                        const char spinner[] = {'/', '-', '\\', '|'};
                        static int spin = 0;
                        ei_printf("Running inference %c\r", spinner[spin]);
            
            
            if(++spin >= sizeof(spinner)) {

But when deploying using the Simplicity Studio Component option. It doesn’t deploy with the label_detected member.

This means that when trying to recreate the demo with my own project I cant.

It seems like this issue is happening because of whatever source project the Simplicity Studio Component is generating from isn’t updated.

Do you think my assessment is accurate? Let me know if its something else instead.

Thanks

SethW · October 14, 2022, 9:15pm

I guess this isn’t technically a bug because in the demo you never specify how to run the demo with a different model, so this problem would never be run into.

But still seems worth pointing out.

mateusz · October 17, 2022, 1:19pm

Hi @SethW
It seems like your model in the project is not up to date. Could you invoke model re-train in our Studio?
The struct that you are pointing to has a missing label_detected field is a part of our SDK and model. They should be upgraded after model retrain.

Mateusz

SethW · October 17, 2022, 4:09pm

Hey @mateusz

I did a re-train and then deployed again and the label_detected field is still not there:

I used the Retrain model page and deployed as a Simplicity Studio Component.
Proj id#: 145138

Seth

SethW · October 24, 2022, 11:30pm

Hey @mateusz,

Any ideas on why this is still happening?

Thanks,
Seth

mateusz · November 3, 2022, 7:20am

Hi @SethW
The issue is not relevant anymore, as we’ve reverted the addition of the label_detected field. The GH repository has been upgraded.

BR,
Mateusz