updated.

done.

updated.

updated.

removed and used default value as they are the same.

yes. updated.

updated and used as suggested.

noted.

We have a reset function above that does this. Should we call it?

Are we filtering at zero set rate? I guess we should, so then the duration would be 1 sec then?

I think minimum samples would be for max rate (500 mL/min), so 3 secs at 20 Hz = 60 samples.

PRODUCT should probably come before FILTER_SAMPLES above so that it can be used in that definition.
Show derivation again. i.e. (500 * 3) indicating we want 3 seconds of filter duration for 500 mL/min rate and duration grows as rate is reduced.

600 will take a lot of memory (600 x 4 bytes = 2.4K). Ok for now, but if memory gets scarce, we will have to consider sampling less frequently to reduce the size of the moving average array.

Also, show derivation of this number so it's clear what it means and how we got to 600.
i.e. ((PRODUCT / 50) * (MS_PER_SECOND / TASK_GENERAL_INTERVAL)) indicating how many samples we need to buffer for worst case which is lowest flow rate (50) requiring max duration (30 sec) at 20 Hz sample rate.

We typically refer to set blood flow rate as Qb.

Refactor this function to have only one return statement.

You could just do a memset() here.

Wrong place.

I'll drop by and we can set properties together.

Added

Updated

Sorry, Updated

Updated, if this is wrong then I need your help