· /firmware/App/Controllers/SyringePump.c	(-2, +2)	| History	| Source	| Diff
· /firmware/App/Controllers/SyringePump.h	(-2, +2)	| History	| Source	| Diff
· /firmware/App/Modes/ModePreTreat.c	(-1, +1)	| History	| Source	| Diff
· /firmware/App/Modes/ModePreTreat.h	(-1, +1)	| History	| Source	| Diff
· /firmware/App/Modes/StatePreTxHeparinSetup.c	(-2, +2)	| History	| Source	| Diff
· /firmware/App/Modes/StatePreTxHeparinSetup.h	(-2, +2)	| History	| Source	| Diff
· /firmware/App/Monitors/Switches.c	(-2, +2)	| History	| Source	| Diff
· /firmware/App/Services/AlarmMgmtSWFaults.h	(-2, +2)	| History	| Source	| Diff
· /firmware/App/Services/AlarmMgmtTD.c	(-2, +2)	| History	| Source	| Diff
· /firmware/App/Services/TxParams.c	(-2, +2)	| History	| Source	| Diff

]]> Thu, 13 Aug 2026 11:15:11 -0700 https://devapps.diality.us/cru/LEAHI-TD-FIRMWARE-LDT-4029-1 https://devapps.diality.us/cru/LEAHI-TD-FIRMWARE-LDT-4029-1 Sean Nash and Vendor - TEL - Arpita Srivastava finished reviewing LEAHI-TD-FIRMWARE-LDT-4029-1 LEAHI-TD-FIRMWARE-LDT-4029_Pre-Treatment - FW Implementation - 4/6: ]]> Thu, 13 Aug 2026 09:53:08 -0700 Vendor - TEL - Arpita Srivastava https://devapps.diality.us/cru/LEAHI-TD-FIRMWARE-LDT-4029-1/#c31286 https://devapps.diality.us/cru/LEAHI-TD-FIRMWARE-LDT-4029-1/#c31286 Vendor - TEL - Arpita Srivastava commented on LEAHI-TD-FIRMWARE-LDT-4029-1

Add to inputs : heparinSetupResumeRequested

Add @alarm

Add to inputs : interruptedPreTxHeparinSetupState

· /firmware/App/Controllers/SyringePump.c	(-0, +1)	| History	| Source	| Diff
· /firmware/App/Modes/StatePreTxHeparinSetup.c	(-5, +13)	| History	| Source	| Diff

]]> Wed, 12 Aug 2026 10:26:12 -0700 Sean Nash https://devapps.diality.us/cru/LEAHI-TD-FIRMWARE-LDT-4029-1/#c31246 https://devapps.diality.us/cru/LEAHI-TD-FIRMWARE-LDT-4029-1/#c31246 Sean Nash commented on LEAHI-TD-FIRMWARE-LDT-4029-1

Does default mean invalid state (s/w fault) or does it mean it's a state that we don't need to do anything for? If latter, add a comment here saying that.

Use if else or ternary operator to set BOOL to TRUE or FALSE explicitly.

Add a TODO comment to adjust this when new syringe pump assembly is ready.

· /firmware/App/Controllers/SyringePump.c	(-1, +1)	| History	| Source	| Diff
· /firmware/App/Modes/ModePreTreat.c	(-11, +11)	| History	| Source	| Diff
· /firmware/App/Modes/StatePreTxHeparinSetup.c	(-26, +12)	| History	| Source	| Diff
· /firmware/App/Services/TxParams.c	(-2, +2)	| History	| Source	| Diff

]]> Wed, 05 Aug 2026 12:44:35 -0700 Sean Nash https://devapps.diality.us/cru/LEAHI-TD-FIRMWARE-LDT-4029-1/#c31028 https://devapps.diality.us/cru/LEAHI-TD-FIRMWARE-LDT-4029-1/#c31028 Sean Nash commented on LEAHI-TD-FIRMWARE-LDT-4029-1

What if we're in paused state (because your alarm has stop property set)? Should we return TRUE here if in paused state and interrupted state was the await syringe load confirmation state?

Set "Stops" property back to TRUE like the other reserved UI alarms.

break this line

add an empty else block after the last else if block in these functions
handlePreTxHeparinPreloadState
handlePreTxHeparinAwaitSyringeLoadConfirmationState
handlePreTxHeparinSeekState
handlePreTxHeparinPrimeState
handlePreTxHeparinBolusState

break this line

Add initialization for heparinSetupResumeRequested and syringeLoadConfirmed

Align all the function comments

Delete this row.

Should this be the "Load Syringe" alarm that you just added?

· /firmware/App/Controllers/SyringePump.c	(-14, +27)	| History	| Source	| Diff
· /firmware/App/Controllers/SyringePump.h	(-0, +1)	| History	| Source	| Diff
· /firmware/App/Modes/ModePreTreat.c	(-4, +2)	| History	| Source	| Diff

]]> Tue, 21 Jul 2026 17:44:54 -0700 Sean Nash https://devapps.diality.us/cru/LEAHI-TD-FIRMWARE-LDT-4029-1/#c30599 https://devapps.diality.us/cru/LEAHI-TD-FIRMWARE-LDT-4029-1/#c30599 Sean Nash commented on LEAHI-TD-FIRMWARE-LDT-4029-1

I think ( setTxDur - preStop ) should be changed to ( preStop ).
Also, I think preStop should be renamed to hepDurMin.

Can we remote this TODO comment?

Remove blank line.

We should go to dry self test state. Why did this change to Rx state?

Comment - should be "install" not "instll".

· /firmware/App/Controllers/SyringePump.c	(-0, +13)	| History	| Source	| Diff
· /firmware/App/Controllers/SyringePump.h	(-0, +1)	| History	| Source	| Diff
· /firmware/App/Modes/ModePreTreat.c	(-0, +4)	| History	| Source	| Diff
· /firmware/App/Modes/StatePreTxHeparinSetup.c	(-56, +106)	| History	| Source	| Diff

]]> 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.

171

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).