State Machine and stack size ?

[antonius]

Dear Members,

I have a state machine but it seems that it stuck in one state,

Is it related with stack size ?

It's stuck on "case PULSE_IDLE" no matter the value of sensor_value,

The code :

switch(currentPulseDetectorState)
 {
 case PULSE_IDLE:
 if(sensor_value >= PULSE_MIN_THRESHOLD) {
				//printf("Sensor_value PULSE_IDLE %f \r\n",sensor_value);
 currentPulseDetectorState = PULSE_TRACE_UP;
				
 values_went_down = 0;
				printf("PULSE IDLE\r\n");
				return true;
 }
 break;
 
 case PULSE_TRACE_UP:
			printf("Sensor value PULSE_TRACE_UP %f\r\n",sensor_value);
 if(sensor_value > prev_sensor_value)
 {
				
 printf("PULSE TRACE_UP\r\n");
				currentBeat =HAL_GetTick();
 lastBeatThreshold = sensor_value;
				
				
 }
 else
 {
 
 printf("Peak reached: ");
 printf("%f \r\n",sensor_value);
 printf(" ");
 printf("%f \r\n",prev_sensor_value); 
 
 uint32_t beatDuration = currentBeat - lastBeat;
 lastBeat = currentBeat;
 
 float rawBPM=0;
 if(beatDuration > 0)
 rawBPM = 60000.0 / (float)beatDuration;
 printf("RawBPM = %f\r\n",rawBPM);
 
 //This method sometimes glitches, it's better to go through whole moving average everytime
 //IT's a neat idea to optimize the amount of work for moving avg. but while placing, removing finger it can screw up
 //valuesBPMSum -= valuesBPM[bpmIndex];
 //valuesBPM[bpmIndex] = rawBPM;
 //valuesBPMSum += valuesBPM[bpmIndex];
 
 valuesBPM[bpmIndex] = rawBPM;
 valuesBPMSum = 0;
 for(int i=0; i<PULSE_BPM_SAMPLE_SIZE; i++)
 {
 valuesBPMSum += valuesBPM[i];
 }
 
 
 
 bpmIndex++;
 bpmIndex = bpmIndex % PULSE_BPM_SAMPLE_SIZE;
 
 if(valuesBPMCount < PULSE_BPM_SAMPLE_SIZE)
 valuesBPMCount++;
 
 currentBPM = valuesBPMSum / valuesBPMCount;
 
 printf("AVg. BPM: ");
 printf("%f",currentBPM);
 
 
 
 currentPulseDetectorState = PULSE_TRACE_DOWN;
 
 return true;
 }
 break;
 
 case PULSE_TRACE_DOWN:
 if(sensor_value < prev_sensor_value)
 {
 values_went_down++;
 }
 
 
 if(sensor_value < PULSE_MIN_THRESHOLD)
 {
 currentPulseDetectorState = PULSE_IDLE;
 }
 break;
 }
 
 prev_sensor_value = sensor_value;

Any clues on solving it ? thanks

[Ozone]

Almost certrainly not.

With a stack problem, you would end up in the hardfault handler.

Perhaps your "sensor_value" is always less then PULSE_MIN_THRESHOLD ?

[antonius]

Data 7 Output MAX30100,0x09 register (FIFO DATA) IR : 1536 
 
DC filterIR = -1147.117188
 
MeanDiff filterIR = 52967.000000
 
LowPass filterIR = 12989.482422
 
Timestamp After IRRED data= 25885 
 
sensor_value > PULSE_MAX_THRESHOLD
 
Sensor_value PULSE_MAX_TRESHOLD 12989.482422 
 
Sensor_value at PULSE_IDLE 12989.482422 
 
PULSE IDLE
 
 

code :

switch(currentPulseDetectorState)
 {
 case PULSE_IDLE:
 if(sensor_value >= PULSE_MIN_THRESHOLD) {
				printf("Sensor_value at PULSE_IDLE %f \r\n",sensor_value);
 
				
 values_went_down = 0;
				printf("PULSE IDLE\r\n");
				 currentPulseDetectorState = PULSE_TRACE_UP;
				//return currentPulseDetectorState = PULSE_TRACE_UP;
 }
 break;

#define PULSE_MIN_THRESHOLD        300

I don't understand why ?