Comment Search

Please add a comment about this condition check.

Move this part : ( fabs( request.ufRate - validatedUFRateLhr ) < NEARLY_ZERO ) ) to the next line.

REmoved

Move to driver.

Yes

RESOLVED

RI: It's defined in two places because cs.py and watchdog.py live on opposite sides of a process boundary and don't share a common import. watchdog.py (SENTINEL_PATH) is the producer - it runs inside the cloud_sync.py app process and writes the sentinel file when a thread can't be recovered. cs.py (SENTINEL_FILE) is the consumer - it's the lightweight launcher that spawns cloud_sync.py and polls for that file to restart it. cs.py is deliberately kept minimal and doesn't import the cloudsync package, so the path is duplicated as a local literal rather than shared.

Add fluid bolus state

Done.

Done

Not a float anymore. Use u32Override function and no need to cast the pointer.

makes sense. removed it.

added TODO for all

Will fix on Beta branch to avoid changes to Dialin

Can we make these low priority and 999 rank?

We need to call this from ModeTreatment transition function.

These alarms are in the wrong place. Their enums are 187..199. So they should be at the very end of this table, not here near the beginning.

fixed. thank you.

Why are we bypassing driver here?

Add space between } and type name.

If both monitor and driver need visibility to something, consider moving it to the driver's header file since monitor needs to include driver anyway. Driver should not have to know about the monitor stuff as it is higher level.

Fix indent.

I thought this was a TEST_CONFIG thing. We aren't doing s/w config status, so maybe we should convert it to a TEST_CONFIG.

SW fault here?

Updated

Where are these get functions defined? I can't find them.

These are located in the integrity.c