Comment Search

nr means short form of bit 'number', unsigned long is portable, 32bit for 32-bit cpu and 64 bits for 64bit cpu whereas unsigned int is always 32 bits regardless of architecture.

Add blank line between declarations and rest of code.

you need to go to Treatment mode then change the treatmetn state

Remove and test to ensure model is not garbage collected in QML.

RESOLVED.

Updated

DO we need bicarb conversion factor as well?

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

Why increment the number when you didn't add a new sensor?

we are sending a message ack as response in the handler of this message request

What TD does with UF rate is mostly to tell DD what rate we want. DD wants mL/min. TD can keep L/hr and convert to mL/min every time it needs to send DD a message or it can keep mL/min and convert to L/hr every time it needs to send UI a message. Either way works for me.

recircPublishTimerCtr is an output

We don't have to align =, but if you're going to have all of these spaces before the =, you should align.

Consider separating the wait for door close to another state.

I prefer you create a local BOOL result so you can have a simpler return statement.

change to ///< Tubing set install state

Why are we removing this TODO comment? It looks like we should keep it.

Remove extra l at end of flag name (mis-spelled). And why are we sending this flag with scan event? Seems like the Received flag is what you want here.

Should be in "", not <>. And should be placed alphabetically down before TxParams.h.

Do we need TODO comments to restore these alarms later?

We ensured that simulator was closed while executing the script.

RESOLVED

fixed

I would move these navigations into a py file like navigation.py to ve used in other places.

Space between if and (.