· /firmware/App/Controllers/AirTrap.c	(-5, +4)	| History	| Source	| Diff
· /firmware/App/Controllers/BloodLeak.c	(-1, +2)	| History	| Source	| Diff
· /firmware/App/Controllers/Bubble.c	(-5, +6)	| History	| Source	| Diff
· /firmware/App/Controllers/DGInterface.c	(-0, +4)	| History	| Source	| Diff
· /firmware/App/Controllers/PresOccl.c	(-11, +29)	| History	| Source	| Diff
· /firmware/App/Controllers/PresOccl.h	(-0, +1)	| History	| Source	| Diff
· /firmware/App/Controllers/SyringePump.c	(-1, +0)	| History	| Source	| Diff
· /firmware/App/Controllers/Valves.c	(-0, +3)	| History	| Source	| Diff
· /firmware/App/Modes/ModePreTreat.c	(-0, +3)	| History	| Source	| Diff
· /firmware/App/Modes/ModeStandby.c	(-0, +2)	| History	| Source	| Diff

]]> Wed, 29 Apr 2026 16:50:17 -0700 Dara Navaei https://devapps.diality.us/cru/HD-DENBUG-334-2 https://devapps.diality.us/cru/HD-DENBUG-334-2 Dara Navaei started review HD-DENBUG-334-2 ]]> Wed, 29 Jan 2025 18:09:02 -0800 Dara Navaei https://devapps.diality.us/changelog/hdfirmware?cs=fba69244d94307e50fefaa1e88bcbb979584461e https://devapps.diality.us/changelog/hdfirmware?cs=fba69244d94307e50fefaa1e88bcbb979584461e Dara Navaei committed fba6924 on branch DEN-17208-blood-leak-alarm-126-129-triggered-when-VDI-and-VDO-are-in-position-C-staging, DEN-17213-release_20_vectorcast-staging and 10 more in hdfirmware Bamboo Commit: Updated the Copyright section and replaced tabs with 4 spaces

· /firmware/App/Controllers/AirPump.c	(-1, +1)	| History	| Source	| Diff
· /firmware/App/Controllers/AirPump.h	(-1, +1)	| History	| Source	| Diff
· /firmware/App/Controllers/AirTrap.c	(-1, +1)	| History	| Source	| Diff
· /firmware/App/Controllers/AirTrap.h	(-1, +1)	| History	| Source	| Diff
· /firmware/App/Controllers/AlarmLamp.c	(-1, +1)	| History	| Source	| Diff
· /firmware/App/Controllers/AlarmLamp.h	(-1, +1)	| History	| Source	| Diff
· /firmware/App/Controllers/BloodFlow.c	(-1, +1)	| History	| Source	| Diff
· /firmware/App/Controllers/BloodFlow.h	(-1, +1)	| History	| Source	| Diff
· /firmware/App/Controllers/BloodLeak.c	(-2, +2)	| History	| Source	| Diff
· /firmware/App/Controllers/BloodLeak.h	(-1, +1)	| History	| Source	| Diff
· /firmware/App/Controllers/Bubble.c	(-1, +1)	| History	| Source	| Diff
· /firmware/App/Controllers/Bubble.h	(-1, +1)	| History	| Source	| Diff
· /firmware/App/Controllers/Buttons.c	(-1, +1)	| History	| Source	| Diff
· /firmware/App/Controllers/Buttons.h	(-1, +1)	| History	| Source	| Diff
· /firmware/App/Controllers/DGInterface.c	(-1, +1)	| History	| Source	| Diff
… 102 more files in changeset.

]]> Mon, 13 May 2024 13:23:09 -0700 Dara Navaei https://devapps.diality.us/cru/HD-DENBUG-249-1 https://devapps.diality.us/cru/HD-DENBUG-249-1 Dara Navaei Closed HD-DENBUG-249-1 ]]> Mon, 13 May 2024 10:13:27 -0700 https://devapps.diality.us/cru/HD-DENBUG-249-1 https://devapps.diality.us/cru/HD-DENBUG-249-1 Vinayakam Mani and Sean Nash finished reviewing HD-DENBUG-249-1 HD-DENBUG-249_Pressure Window Starting Values P1 Staging ]]> Sun, 12 May 2024 14:38:57 -0700 Dara Navaei https://devapps.diality.us/cru/HD-DENBUG-249-1 https://devapps.diality.us/cru/HD-DENBUG-249-1 Dara Navaei started review HD-DENBUG-249-1 ]]> Fri, 10 May 2024 13:06:54 -0700 Dara Navaei https://devapps.diality.us/changelog/hdfirmware?cs=8121bf62592934e01f365a973ac813c3d4748cda https://devapps.diality.us/changelog/hdfirmware?cs=8121bf62592934e01f365a973ac813c3d4748cda Dara Navaei committed 8121bf6 on branch master, DEN-16565-sw-updater-hd-dg-app-side-intake-from-sunrise and 34 more in hdfirmware Bamboo Commit: Updated the Copyright section and replaced tabs with 4 spaces

· /firmware/App/Controllers/PresOccl.c	(-1, +1)	| History	| Source	| Diff
· /firmware/App/Controllers/PresOccl.h	(-1, +1)	| History	| Source	| Diff
· /firmware/App/Modes/ModeTreatment.c	(-2, +2)	| History	| Source	| Diff
· /firmware/App/Services/SystemComm.c	(-2, +2)	| History	| Source	| Diff
· /firmware/App/Services/SystemCommMessages.c	(-2, +2)	| History	| Source	| Diff
· /firmware/App/Services/SystemCommMessages.h	(-2, +2)	| History	| Source	| Diff

]]> Fri, 10 May 2024 11:59:08 -0700 Vinayakam Mani https://devapps.diality.us/cru/HD-DEN-16851-1 https://devapps.diality.us/cru/HD-DEN-16851-1 Vinayakam Mani Closed HD-DEN-16851-1 ]]> Fri, 10 May 2024 11:35:14 -0700 https://devapps.diality.us/cru/HD-DEN-16851-1 https://devapps.diality.us/cru/HD-DEN-16851-1 Dara Navaei finished reviewing HD-DEN-16851-1 HD-DEN-16851_New Alarm For Increased Occlus ]]> Thu, 09 May 2024 12:43:35 -0700 https://devapps.diality.us/cru/HD-DEN-16851-1 https://devapps.diality.us/cru/HD-DEN-16851-1 Sean Nash finished reviewing HD-DEN-16851-1 HD-DEN-16851_New Alarm For Increased Occlus ]]> Thu, 02 May 2024 12:54:21 -0700 Sean Nash https://devapps.diality.us/cru/HD-DEN-16851-1/#c19752 https://devapps.diality.us/cru/HD-DEN-16851-1/#c19752 Sean Nash replied to Vinayakam Mani in HD-DEN-16851-1

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

Done.

Done

Done

Done

Yes, when we are not in treatment mode, the state machine sets the state to "pressure_limits_state_off" at start of this function ( line : 673). hence mode check is not necessary.

Done.

Done.

I think we just want to publish the baseline here (not +1000).

Yes, if you cast either value in division as a F32, the result will be F32. This is all we need as F32 (the .data). The samples and total can stay U32.
You might want to cast the total as F32 as well for consistency.

F32 not needed here.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

Casting not necessary.