· /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 10:16:24 -0700 Vendor - TEL - Vijay Pamula https://devapps.diality.us/changelog/td-firmware?cs=253f1ca9860b2d13f8b4d7e82ba440b35c4f2cb8 https://devapps.diality.us/changelog/td-firmware?cs=253f1ca9860b2d13f8b4d7e82ba440b35c4f2cb8 Vendor - TEL - Vijay Pamula committed 253f1ca on branch LDT-2240-disposable-tube-set-self-test, LDT-4029-pre-treatment---fw-implementation and 14 more in td-firmware LDT-4029:Addressed review comments

· /firmware/App/Modes/StatePreTxHeparinSetup.c

(-2, +4)

| History

| Source

| Diff

]]> 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.h	(-2, +2)	| History	| Source	| Diff
· /firmware/App/Modes/ModePreTreat.c	(-21, +182)	| History	| Source	| Diff
· /firmware/App/Modes/ModePreTreat.h	(-8, +12)	| History	| Source	| Diff
· /firmware/App/Modes/StatePreTxHeparinSetup.c	(-1, +1)	| History	| Source	| Diff
· /firmware/App/Services/AlarmMgmtSWFaults.h	(-2, +8)	| History	| Source	| Diff
· /firmware/App/Services/AlarmMgmtTD.c	(-4, +49)	| History	| Source	| Diff
· /firmware/App/Services/TxParams.c	(-2, +3)	| History	| Source	| Diff

]]> Wed, 12 Aug 2026 14:31:39 -0700 Vendor - TEL - Vijay Pamula https://devapps.diality.us/changelog/td-firmware?cs=8a3f16888f564619ab04fe904e4acbe9e8c24117 https://devapps.diality.us/changelog/td-firmware?cs=8a3f16888f564619ab04fe904e4acbe9e8c24117 Vendor - TEL - Vijay Pamula committed 8a3f168 on branch LDT-2240-disposable-tube-set-self-test, LDT-4029-pre-treatment---fw-implementation and 14 more in td-firmware LDT-4029:Updated heparin setup state

· /firmware/App/Modes/StatePreTxHeparinSetup.c

(-2, +2)

| History

| Source

| Diff

]]> Wed, 12 Aug 2026 14:25:51 -0700 Vendor - TEL - Vijay Pamula https://devapps.diality.us/changelog/td-firmware?cs=1233ccac080bb953acde0a9bde41607a7d0ff170 https://devapps.diality.us/changelog/td-firmware?cs=1233ccac080bb953acde0a9bde41607a7d0ff170 Vendor - TEL - Vijay Pamula committed 1233cca on branch LDT-2240-disposable-tube-set-self-test, LDT-4029-pre-treatment---fw-implementation and 14 more in td-firmware LDT-4029:Updated Heparin setup state

· /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?

· /firmware/App/Modes/StatePreTxHeparinSetup.c

(-4, +5)

| History

| Source

| Diff

]]> Fri, 31 Jul 2026 19:06:44 -0700 Vendor - TEL - Vijay Pamula https://devapps.diality.us/changelog/td-firmware?cs=26697e83bb707f1627e0707036d1bbc5b876fd6e https://devapps.diality.us/changelog/td-firmware?cs=26697e83bb707f1627e0707036d1bbc5b876fd6e Vendor - TEL - Vijay Pamula committed 26697e8 on branch LDT-2240-disposable-tube-set-self-test, LDT-4029-pre-treatment---fw-implementation and 14 more in td-firmware LDT-4029:Addressed review comments

· /firmware/App/Modes/StatePreTxHeparinSetup.c

(-3, +3)

| History

| Source

| Diff

]]> Fri, 31 Jul 2026 16:24:45 -0700 Vendor - TEL - Vijay Pamula https://devapps.diality.us/changelog/td-firmware?cs=7ad43a0dc494ad7d92fa874052467f48948636a3 https://devapps.diality.us/changelog/td-firmware?cs=7ad43a0dc494ad7d92fa874052467f48948636a3 Vendor - TEL - Vijay Pamula committed 7ad43a0 on branch LDT-2240-disposable-tube-set-self-test, LDT-4029-pre-treatment---fw-implementation and 14 more in td-firmware LDT-4029:Addressed review comments

· /firmware/App/Modes/StatePreTxHeparinSetup.c

(-2, +3)

| History

| Source

| Diff

]]> Thu, 30 Jul 2026 15:37:50 -0700 Vendor - TEL - Vijay Pamula https://devapps.diality.us/changelog/td-firmware?cs=9b005194e074d3f8df704ab57ac70bb32ade1d90 https://devapps.diality.us/changelog/td-firmware?cs=9b005194e074d3f8df704ab57ac70bb32ade1d90 Vendor - TEL - Vijay Pamula committed 9b00519 on branch LDT-2240-disposable-tube-set-self-test, LDT-4029-pre-treatment---fw-implementation and 14 more in td-firmware LDT-4029:Updated heparin setup state

· /firmware/App/Modes/StatePreTxHeparinSetup.c

(-7, +10)

| History

| Source

| Diff

]]> Thu, 30 Jul 2026 10:31:38 -0700 Sean Nash https://devapps.diality.us/cru/LEAHI-TD-FIRMWARE-LDT-4029-1/#c30874 https://devapps.diality.us/cru/LEAHI-TD-FIRMWARE-LDT-4029-1/#c30874 Sean Nash commented on LEAHI-TD-FIRMWARE-LDT-4029-1

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

· /firmware/App/Modes/StatePreTxHeparinSetup.c	(-6, +30)	| History	| Source	| Diff
· /firmware/App/Modes/StatePreTxHeparinSetup.h	(-4, +4)	| History	| Source	| Diff

]]> Tue, 28 Jul 2026 14:47:09 -0700 Vendor - TEL - Arpita Srivastava https://devapps.diality.us/cru/LEAHI-TD-FIRMWARE-LDT-4029-1/#c30793 https://devapps.diality.us/cru/LEAHI-TD-FIRMWARE-LDT-4029-1/#c30793 Vendor - TEL - Arpita Srivastava commented on LEAHI-TD-FIRMWARE-LDT-4029-1

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.

· /firmware/App/Modes/StatePreTxHeparinSetup.c

(-2, +2)

| History

| Source

| Diff

]]> Tue, 21 Jul 2026 14:12:46 -0700 Sean Nash https://devapps.diality.us/cru/LEAHI-TD-FIRMWARE-LDT-4029-1/#c30588 https://devapps.diality.us/cru/LEAHI-TD-FIRMWARE-LDT-4029-1/#c30588 Sean Nash commented on LEAHI-TD-FIRMWARE-LDT-4029-1

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

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