· /firmware/App/Modes/ModePreTreat.c	(-69, +59)	| History	| Source	| Diff
· /firmware/App/Modes/StatePreTxHeparinSetup.c	(-0, +1)	| History	| Source	| Diff
· /firmware/App/Services/AlarmMgmtTD.c	(-2, +2)	| History	| Source	| Diff

]]> Wed, 15 Jul 2026 15:03:58 -0700 Dara Navaei https://devapps.diality.us/cru/LEAHI-TD-FIRMWARE-LDT-4611-4 https://devapps.diality.us/cru/LEAHI-TD-FIRMWARE-LDT-4611-4 Dara Navaei started review LEAHI-TD-FIRMWARE-LDT-4611-4 ]]> Wed, 15 Jul 2026 11:59:05 -0700 Dara Navaei https://devapps.diality.us/changelog/td-firmware?cs=83310420ae862de4724f8cfbbaeda9936c47801b https://devapps.diality.us/changelog/td-firmware?cs=83310420ae862de4724f8cfbbaeda9936c47801b Dara Navaei committed 8331042 on branch LDT-3992-disposable-tube-set-auto-priming, LDT-4009-isolated-uf-td and 6 more in td-firmware Bamboo Commit: Updated the Copyright section and replaced tabs with 4 spaces

· /firmware/App/Modes/ModePreTreat.c	(-1, +1)	| History	| Source	| Diff
· /firmware/App/Modes/ModeStandby.c	(-2, +2)	| History	| Source	| Diff
· /firmware/App/Modes/ModeTreatment.c	(-2, +2)	| History	| Source	| Diff
· /firmware/App/Monitors/Pressures.c	(-1, +1)	| History	| Source	| Diff
· /firmware/App/Services/Messaging.c	(-2, +2)	| History	| Source	| Diff
· /firmware/App/Services/StateServices/TubeSetInstall.c	(-2, +2)	| History	| Source	| Diff
· /firmware/App/Services/StateServices/TubeSetInstall.h	(-2, +2)	| History	| Source	| Diff
· /firmware/App/Services/TxParams.c	(-2, +2)	| History	| Source	| Diff
· /firmware/App/Services/TxParams.h	(-1, +1)	| History	| Source	| Diff

]]> Wed, 15 Jul 2026 09:15:19 -0700 https://devapps.diality.us/cru/LEAHI-TD-FIRMWARE-LDT-4028-2 https://devapps.diality.us/cru/LEAHI-TD-FIRMWARE-LDT-4028-2 Vendor - TEL - Arpita Srivastava finished reviewing LEAHI-TD-FIRMWARE-LDT-4028-2 LEAHI-TD-FIRMWARE-LDT-4028_Pre-Treatment - FW Implementation - 3/6: ]]> Tue, 14 Jul 2026 23:15:34 -0700 Sean Nash https://devapps.diality.us/cru/LEAHI-TD-FIRMWARE-LDT-4029-1/#c30500 https://devapps.diality.us/cru/LEAHI-TD-FIRMWARE-LDT-4029-1/#c30500 Sean Nash replied to Sean Nash in LEAHI-TD-FIRMWARE-LDT-4029-1

I don't see any change here. How does this function get called? I don't see the function being used anywhere - nobody is calling it. I think it should be called by Messaging.c when a UI confirmation msg is received.

I thought we'd removed the first 3 states. Isn't install state the first state? Maybe Praneeth did that and it will show up when you merge staging into your branch.

· /firmware/App/Modes/ModePreTreat.c	(-7, +21)	| History	| Source	| Diff
· /firmware/App/Modes/StatePreTxHeparinSetup.c	(-28, +42)	| History	| Source	| Diff
· /firmware/App/Modes/StatePreTxHeparinSetup.h	(-2, +3)	| History	| Source	| Diff
· /firmware/App/Services/AlarmMgmtSWFaults.h	(-1, +1)	| History	| Source	| Diff

]]> Tue, 14 Jul 2026 17:55:43 -0700 https://devapps.diality.us/cru/LEAHI-TD-FIRMWARE-LDT-4028-2 https://devapps.diality.us/cru/LEAHI-TD-FIRMWARE-LDT-4028-2 Sean Nash finished reviewing LEAHI-TD-FIRMWARE-LDT-4028-2 LEAHI-TD-FIRMWARE-LDT-4028_Pre-Treatment - FW Implementation - 3/6: ]]> Tue, 14 Jul 2026 17:53:17 -0700 Sean Nash https://devapps.diality.us/cru/LEAHI-TD-FIRMWARE-LDT-4029-1/#c30498 https://devapps.diality.us/cru/LEAHI-TD-FIRMWARE-LDT-4029-1/#c30498 Sean Nash commented on LEAHI-TD-FIRMWARE-LDT-4029-1

171

This reverts commit 069dee3a2428d3c6bb281db1844a372ae6e2063a.

· /firmware/App/Controllers/AirPump.c	(-35, +17)	| History	| Source	| Diff
· /firmware/App/Controllers/Valves.c	(-27, +17)	| History	| Source	| Diff
· /firmware/App/Controllers/Valves.h	(-18, +2)	| History	| Source	| Diff
· /firmware/App/Modes/ModePreTreat.c	(-3, +2)	| History	| Source	| Diff
· /firmware/App/Modes/ModeStandby.c	(-98, +24)	| History	| Source	| Diff
· /firmware/App/Modes/ModeStandby.h	(-13, +2)	| History	| Source	| Diff
· /firmware/App/Modes/StatePreTxHeparinSetup.c	(-29, +28)	| History	| Source	| Diff
· /firmware/App/Modes/StateTxBloodPrime.c	(-3, +3)	| History	| Source	| Diff
· /firmware/App/Modes/StateTxPaused.c	(-30, +17)	| History	| Source	| Diff
· /firmware/App/Monitors/Switches.c	(-1, +1)	| History	| Source	| Diff
· /firmware/App/Services/AlarmMgmtSWFaults.h	(-5, +3)	| History	| Source	| Diff
· /firmware/App/Services/AlarmMgmtTD.c	(-397, +342)	| History	| Source	| Diff
· /firmware/App/Services/AlarmMgmtTD.h	(-3, +2)	| History	| Source	| Diff
· /firmware/App/Services/Messaging.c	(-4, +2)	| History	| Source	| Diff
· /firmware/App/Services/StateServices/FluidBolus.c	(-71, +179)	| History	| Source	| Diff
… 8 more files in changeset.

]]> Tue, 14 Jul 2026 12:10:13 -0700 Vendor - TEL - Vijay Pamula https://devapps.diality.us/changelog/td-firmware?cs=069dee3a2428d3c6bb281db1844a372ae6e2063a https://devapps.diality.us/changelog/td-firmware?cs=069dee3a2428d3c6bb281db1844a372ae6e2063a Vendor - TEL - Vijay Pamula committed 069dee3 on branch LDT-4029-pre-treatment---fw-implementation in td-firmware LDT-4029: Merge latest staging and addressed comments

· /firmware/App/Controllers/AirPump.c	(-17, +35)	| History	| Source	| Diff
· /firmware/App/Controllers/Valves.c	(-17, +27)	| History	| Source	| Diff
· /firmware/App/Controllers/Valves.h	(-2, +18)	| History	| Source	| Diff
· /firmware/App/Modes/ModePreTreat.c	(-2, +3)	| History	| Source	| Diff
· /firmware/App/Modes/ModeStandby.c	(-24, +98)	| History	| Source	| Diff
· /firmware/App/Modes/ModeStandby.h	(-2, +13)	| History	| Source	| Diff
· /firmware/App/Modes/StatePreTxHeparinSetup.c	(-28, +29)	| History	| Source	| Diff
· /firmware/App/Modes/StateTxBloodPrime.c	(-3, +3)	| History	| Source	| Diff
· /firmware/App/Modes/StateTxPaused.c	(-17, +30)	| History	| Source	| Diff
· /firmware/App/Monitors/Switches.c	(-1, +1)	| History	| Source	| Diff
· /firmware/App/Services/AlarmMgmtSWFaults.h	(-3, +5)	| History	| Source	| Diff
· /firmware/App/Services/AlarmMgmtTD.c	(-342, +397)	| History	| Source	| Diff
· /firmware/App/Services/AlarmMgmtTD.h	(-2, +3)	| History	| Source	| Diff
· /firmware/App/Services/Messaging.c	(-2, +4)	| History	| Source	| Diff
· /firmware/App/Services/StateServices/FluidBolus.c	(-179, +71)	| History	| Source	| Diff
… 8 more files in changeset.

]]> Tue, 14 Jul 2026 10:18:21 -0700 Vendor - TEL - Vijay Pamula https://devapps.diality.us/changelog/td-firmware?cs=75650c25b8a04b96c46daf60dcffd3982e07f8af https://devapps.diality.us/changelog/td-firmware?cs=75650c25b8a04b96c46daf60dcffd3982e07f8af Vendor - TEL - Vijay Pamula committed 75650c2 on branch LDT-3992-disposable-tube-set-auto-priming, LDT-4009-isolated-uf-td and 7 more in td-firmware LDT-4028: Addressed Review Commentsfor Install state

· /firmware/App/Modes/ModePreTreat.c	(-1, +1)	| History	| Source	| Diff
· /firmware/App/Services/StateServices/TubeSetInstall.c	(-7, +7)	| History	| Source	| Diff

]]> Tue, 14 Jul 2026 09:25:06 -0700 Vendor - TEL - Arpita Srivastava https://devapps.diality.us/cru/LEAHI-TD-FIRMWARE-LDT-4028-2/#c30443 https://devapps.diality.us/cru/LEAHI-TD-FIRMWARE-LDT-4028-2/#c30443 Vendor - TEL - Arpita Srivastava commented on LEAHI-TD-FIRMWARE-LDT-4028-2

currentInstallState is an input.

Do we need doxygen comments for static function declarations?
We will have a header in the definition.

AirPump.h should be above AirTrap.h

Who is calling this function? Should be coming from UI, right? So should be a message handler (returns BOOL, takes a message as param, function is in table at top of Messaging.c).

Need to set state to preload state I think. Otherwise you will go to interrupted state.

Why did you remote call to init function here? Is it initialized somewhere else?

Use ternary operation to assign TRUE or FALSE explicitly.

This is at end of enum list above. So it should be at end of table here.

· /firmware/App/Modes/ModePreTreat.c	(-7, +17)	| History	| Source	| Diff
· /firmware/App/Modes/StatePreTxHeparinSetup.c	(-124, +69)	| History	| Source	| Diff
· /firmware/App/Modes/StatePreTxHeparinSetup.h	(-12, +0)	| History	| Source	| Diff

]]> Thu, 09 Jul 2026 14:07:20 -0700 Sean Nash https://devapps.diality.us/cru/LEAHI-TD-FIRMWARE-LDT-4029-1/#c30410 https://devapps.diality.us/cru/LEAHI-TD-FIRMWARE-LDT-4029-1/#c30410 Sean Nash commented on LEAHI-TD-FIRMWARE-LDT-4029-1

What alarm is this? It doesn't look like you've added a new alarm. This is one of the reserved (unused) alarms that UI s/w might use.

Do we really need these setup functions for each state? They all look very simple - just 1 or 2 lines of code.

I believe the SyringePump controller will trigger this alarm at end of seek operation.

Syringe pump should already be stopped if preload is done. Do we need to call this here?

This is called from preload state. Not needed here I think.

Don't put comments on function prototypes. Doxygen gets details of functions from the function header.

Why (void)?

Where is this being called? Should be called from pre-treatment init I think.

Why are we signaling syringe load confirmed here?