Fisheye on leahi-dialin/leahi_dialin/td

leahi-dialin (develop): Merge branch 'LDT-4874' into develop

Michael Garthwaite — Mon, 13 Jul 2026 15:29:00 -0700

Michael Garthwaite committed 5c1132f on branch develop in leahi-dialin

Merge branch 'LDT-4874' into develop

· /leahi_dialin/dd/modules/alarms.py

(-1, +1)

| History

| Source

| Diff

· /leahi_dialin/dd/modules/balancing_chamber.py

(-1, +1)

| History

| Source

| Diff

· /leahi_dialin/dd/modules/blood_leak.py

(-1, +1)

| History

| Source

| Diff

· /leahi_dialin/dd/modules/concentrate_pump.py

(-1, +1)

| History

| Source

| Diff

· /leahi_dialin/dd/modules/conductivity_sensors.py

(-1, +1)

| History

| Source

| Diff

· /leahi_dialin/dd/modules/dialysate_pump.py

(-1, +1)

| History

| Source

| Diff

· /leahi_dialin/dd/modules/events.py

(-1, +1)

| History

| Source

| Diff

· /leahi_dialin/dd/modules/gen_dialysate.py

(-1, +1)

| History

| Source

| Diff

· /leahi_dialin/dd/modules/heaters.py

(-1, +1)

| History

| Source

| Diff

· /leahi_dialin/dd/modules/levels.py

(-1, +1)

| History

| Source

| Diff

· /leahi_dialin/dd/modules/post_gen_dialysate.py

(-1, +1)

| History

| Source

| Diff

· /leahi_dialin/dd/modules/pre_gen_dialysate.py

(-1, +1)

| History

| Source

| Diff

· /leahi_dialin/dd/modules/pressure_sensors.py

(-1, +1)

| History

| Source

| Diff

· /leahi_dialin/dd/modules/rinse_pump.py

(-1, +1)

| History

| Source

| Diff

· /leahi_dialin/dd/modules/spent_chamber_fill.py

(-1, +1)

| History

| Source

| Diff

… 31 more files in changeset.

leahi-dialin (LDT-4874): Changed handler debug to contain raw message and timestamp of recv

Michael Garthwaite — Mon, 13 Jul 2026 15:28:47 -0700

Michael Garthwaite committed 034ef98 on branch LDT-4874, develop in leahi-dialin

Changed handler debug to contain raw message and timestamp of recv

· /leahi_dialin/dd/modules/alarms.py

(-1, +1)

| History

| Source

| Diff

· /leahi_dialin/dd/modules/balancing_chamber.py

(-1, +1)

| History

| Source

| Diff

· /leahi_dialin/dd/modules/blood_leak.py

(-1, +1)

| History

| Source

| Diff

· /leahi_dialin/dd/modules/concentrate_pump.py

(-1, +1)

| History

| Source

| Diff

· /leahi_dialin/dd/modules/conductivity_sensors.py

(-1, +1)

| History

| Source

| Diff

· /leahi_dialin/dd/modules/dialysate_pump.py

(-1, +1)

| History

| Source

| Diff

· /leahi_dialin/dd/modules/drybicart.py

(-1, +1)

| History

| Source

| Diff

· /leahi_dialin/dd/modules/events.py

(-1, +1)

| History

| Source

| Diff

· /leahi_dialin/dd/modules/gen_dialysate.py

(-1, +1)

| History

| Source

| Diff

· /leahi_dialin/dd/modules/heaters.py

(-1, +1)

| History

| Source

| Diff

· /leahi_dialin/dd/modules/levels.py

(-1, +1)

| History

| Source

| Diff

· /leahi_dialin/dd/modules/mixing_cntrl.py

(-1, +1)

| History

| Source

| Diff

· /leahi_dialin/dd/modules/post_gen_dialysate.py

(-1, +1)

| History

| Source

| Diff

· /leahi_dialin/dd/modules/pre_gen_dialysate.py

(-1, +1)

| History

| Source

| Diff

· /leahi_dialin/dd/modules/pressure_sensors.py

(-1, +1)

| History

| Source

| Diff

… 34 more files in changeset.

LEAHI-DIALIN-LDT-3735-1: reviewers finished

Mon, 13 Jul 2026 14:46:50 -0700

Tiffany Mejia and Michael Garthwaite finished reviewing LEAHI-DIALIN-LDT-3735-1

LEAHI-DIALIN-LDT-3735_System Data and Logs - DD - Non-Volatile Memory - 04: DEV - Feature Implementation

LEAHI-DIALIN-LDT-4872-1: closed by Zoltan Miskolci

Zoltan Miskolci — Mon, 13 Jul 2026 11:51:27 -0700

Zoltan Miskolci Closed LEAHI-DIALIN-LDT-4872-1

leahi-dialin (develop): Merge branch 'LDT-4874' into develop

Michael Garthwaite — Mon, 13 Jul 2026 10:34:58 -0700

Michael Garthwaite committed 820c945 on branch develop in leahi-dialin

Merge branch 'LDT-4874' into develop

· /leahi_dialin/dd/modules/alarms.py

(-1, +5)

| History

| Source

| Diff

· /leahi_dialin/dd/modules/balancing_chamber.py

(-3, +4)

| History

| Source

| Diff

· /leahi_dialin/dd/modules/blood_leak.py

(-1, +4)

| History

| Source

| Diff

· /leahi_dialin/dd/modules/concentrate_pump.py

(-0, +6)

| History

| Source

| Diff

· /leahi_dialin/dd/modules/conductivity_sensors.py

(-1, +7)

| History

| Source

| Diff

· /leahi_dialin/dd/modules/dialysate_pump.py

(-0, +4)

| History

| Source

| Diff

· /leahi_dialin/dd/modules/events.py

(-0, +5)

| History

| Source

| Diff

· /leahi_dialin/dd/modules/gen_dialysate.py

(-0, +4)

| History

| Source

| Diff

· /leahi_dialin/dd/modules/heaters.py

(-0, +4)

| History

| Source

| Diff

· /leahi_dialin/dd/modules/levels.py

(-0, +4)

| History

| Source

| Diff

· /leahi_dialin/dd/modules/post_gen_dialysate.py

(-0, +4)

| History

| Source

| Diff

· /leahi_dialin/dd/modules/pre_gen_dialysate.py

(-0, +4)

| History

| Source

| Diff

· /leahi_dialin/dd/modules/pressure_sensors.py

(-1, +4)

| History

| Source

| Diff

· /leahi_dialin/dd/modules/rinse_pump.py

(-0, +4)

| History

| Source

| Diff

· /leahi_dialin/dd/modules/spent_chamber_fill.py

(-0, +4)

| History

| Source

| Diff

… 31 more files in changeset.

leahi-dialin (LDT-4874): LDT-4874: updates to td handlers for timing

Michael Garthwaite — Mon, 13 Jul 2026 10:32:42 -0700

Michael Garthwaite committed 8ad61b1 on branch LDT-4874, develop in leahi-dialin

LDT-4874: updates to td handlers for timing

· /leahi_dialin/dd/modules/alarms.py

(-1, +2)

| History

| Source

| Diff

· /leahi_dialin/dd/modules/conductivity_sensors.py

(-2, +5)

| History

| Source

| Diff

· /leahi_dialin/fp/modules/conductivity_sensors.py

(-1, +3)

| History

| Source

| Diff

· /leahi_dialin/fp/modules/events.py

(-1, +4)

| History

| Source

| Diff

· /leahi_dialin/td/modules/air_pump.py

(-1, +4)

| History

| Source

| Diff

· /leahi_dialin/td/modules/air_trap.py

(-1, +4)

| History

| Source

| Diff

· /leahi_dialin/td/modules/alarms.py

(-0, +19)

| History

| Source

| Diff

· /leahi_dialin/td/modules/blood_flow.py

(-0, +4)

| History

| Source

| Diff

· /leahi_dialin/td/modules/bubble_detector.py

(-0, +4)

| History

| Source

| Diff

· /leahi_dialin/td/modules/buttons.py

(-0, +4)

| History

| Source

| Diff

· /leahi_dialin/td/modules/ejector.py

(-0, +4)

| History

| Source

| Diff

· /leahi_dialin/td/modules/events.py

(-0, +3)

| History

| Source

| Diff

· /leahi_dialin/td/modules/pressure_sensors.py

(-2, +4)

| History

| Source

| Diff

· /leahi_dialin/td/modules/switches.py

(-0, +4)

| History

| Source

| Diff

· /leahi_dialin/td/modules/syringe_pump.py

(-1, +4)

| History

| Source

| Diff

… 4 more files in changeset.

LEAHI-DIALIN-LDT-4484-1: comment by Behrouz NematiPour "Zoltan and I had a meeting, and the counter sol..."

Behrouz NematiPour — Sun, 12 Jul 2026 23:43:02 -0700

Behrouz NematiPour replied to Zoltan Miskolci in LEAHI-DIALIN-LDT-4484-1

Zoltan and I had a meeting, and the counter solution he provided works better than mine.
He explained to me that my proposed solution has flaws.
RESOLVED.

LEAHI-DIALIN-LDT-4872-1: reviewers finished

Fri, 10 Jul 2026 11:45:54 -0700

Tiffany Mejia and Michael Garthwaite finished reviewing LEAHI-DIALIN-LDT-4872-1

LEAHI-DIALIN-LDT-4872_[Request] Dialin module for Patient Vitals

LEAHI-DIALIN-LDT-4874-1: started by Dara Navaei

Dara Navaei — Thu, 09 Jul 2026 10:48:25 -0700

Dara Navaei started review LEAHI-DIALIN-LDT-4874-1

leahi-dialin (handler_refactor): demo

Zoltan Miskolci — Wed, 08 Jul 2026 14:54:19 -0700

Zoltan Miskolci committed b167367 on branch handler_refactor in leahi-dialin

demo

· /leahi_dialin/common/generic_defs.py	(-0, +100)	\| History	\| Source	\| Diff
· /leahi_dialin/common/td_defs_proxy.py	(-0, +6)	\| History	\| Source	\| Diff
· /leahi_dialin/td/modules/alarms.py	(-168, +343)	\| History	\| Source	\| Diff
· /leahi_dialin/utils/abstract_classes.py	(-0, +128)	\| History	\| Source	\| Diff

leahi-dialin (develop): Merge branch 'LDT-4872-patient_vitals' into develop

Zoltan Miskolci — Tue, 07 Jul 2026 14:03:45 -0700

Zoltan Miskolci committed 25ee6be on branch develop in leahi-dialin

Merge branch 'LDT-4872-patient_vitals' into develop

· /leahi_dialin/common/msg_ids.py	(-2, +2)	\| History	\| Source	\| Diff
· /leahi_dialin/td/proxies/ui_proxy.py	(-0, +16)	\| History	\| Source	\| Diff
· /leahi_dialin/td/treatment_delivery.py	(-0, +2)	\| History	\| Source	\| Diff

leahi-dialin (LDT-4874): patient vitals module

Zoltan Miskolci — Mon, 06 Jul 2026 16:20:30 -0700

Zoltan Miskolci committed 3e8c157 on branch LDT-4872-patient_vitals, LDT-4874 and 2 more in leahi-dialin

patient vitals module

· /leahi_dialin/common/msg_ids.py	(-2, +2)	\| History	\| Source	\| Diff
· /leahi_dialin/td/modules/blood_pressure.py	(-0, +102)	\| History	\| Source
· /leahi_dialin/td/proxies/ui_proxy.py	(-0, +16)	\| History	\| Source	\| Diff
· /leahi_dialin/td/treatment_delivery.py	(-0, +2)	\| History	\| Source	\| Diff

LEAHI-DIALIN-LDT-4160-1: closed by Vendor - TEL - Santhoshkumar Reddy Alasandagutti

Vendor - TEL - Santhoshkumar Reddy Alasandagutti — Mon, 06 Jul 2026 15:08:42 -0700

Vendor - TEL - Santhoshkumar Reddy Alasandagutti Closed LEAHI-DIALIN-LDT-4160-1

LEAHI-DIALIN-LDT-4072-1: reviewers finished

Mon, 06 Jul 2026 12:55:22 -0700

Sean Nash finished reviewing LEAHI-DIALIN-LDT-4072-1

LEAHI-DIALIN-LDT-4072_Power Loss - FW Implementation - 2/3:

LEAHI-DIALIN-LDT-4160-1: reviewers finished

Thu, 02 Jul 2026 15:11:45 -0700

Vinayakam Mani finished reviewing LEAHI-DIALIN-LDT-4160-1

LEAHI-DIALIN-LDT-4160_[Request] Get function for full detailed Alarm Table

LEAHI-DIALIN-LDT-4505-1: closed by Vendor - TEL - Varshini Nagabooshanam

Vendor - TEL - Varshini Nagabooshanam — Thu, 02 Jul 2026 13:19:41 -0700

Vendor - TEL - Varshini Nagabooshanam Closed LEAHI-DIALIN-LDT-4505-1

LEAHI-DIALIN-LDT-4484-1: comment by Zoltan Miskolci "The approach of what this code does and the con..."

Zoltan Miskolci — Wed, 01 Jul 2026 14:28:52 -0700

Zoltan Miskolci replied to Behrouz NematiPour in LEAHI-DIALIN-LDT-4484-1

The approach of what this code does and the conversion module are similar in function but a differ on maintenance and flexibility.
1. The conversion module's bytearray to value methods work, but they require manual logistic around the indexing. It's the user's responsibility to handle where the reading starts and you need to code the handler for that (previous approach with MsgFieldPositions), It's inflating the method. I worked hard to remove the need for indexing where the value starts. Instead of the previous struct.unpack approach, where you needed to maintain the type, you will be needing to maintain the conversion methods that are being called, so it would be the same as before, you still need to mind the indexing, which was the main reason of changing from this approach.

2. In the current approach, you declare the type and pass it to a processor that dynamically handling the unpack type (based on the set datatype) and the indexing for that. This works much better and easier to maintain, as you can just insert a new value when it's added to the FW, and don't need to adjust all your indexes for the reading.

Example #1, changing build type from U16 to U08:

with the conversion approach, you need to change the function used, and update the starting index for all the following methods.
with the current approach you change the Datatype.U16 to Datatype.U08 and that's all you need to do.

Example #2: adding a new variable fpga_micro after fpga_minor, with U08 type:

with the conversion approach, you add the bytearray_to_byte, update the indexes for all the following reads.
with the current approach you insert an element after the fpga_minor with a value ('fpga_micro', Datatype.U08), and nothing more needed

LEAHI-DIALIN-LDT-4484-1: comment by Behrouz NematiPour "RESOLVED for now."

Behrouz NematiPour — Wed, 01 Jul 2026 12:33:22 -0700

Behrouz NematiPour replied to Behrouz NematiPour in LEAHI-DIALIN-LDT-4484-1

RESOLVED for now.

LEAHI-DIALIN-LDT-4484-1: comment by Behrouz NematiPour "I would not do that because the function names ..."

Behrouz NematiPour — Wed, 01 Jul 2026 12:32:17 -0700

Behrouz NematiPour replied to Zoltan Miskolci in LEAHI-DIALIN-LDT-4484-1

I would not do that because the function names and messages for each project should be different to avoid confusion.
I think that is fine for now.

RESOLVED.

LEAHI-DIALIN-LDT-4160-1: comment by Vendor - TEL - Santhoshkumar Reddy Alasandagutti "even though we are sending U08 at Firmware. Due..."

Vendor - TEL - Santhoshkumar Reddy Alasandagutti — Wed, 01 Jul 2026 12:12:10 -0700

Vendor - TEL - Santhoshkumar Reddy Alasandagutti replied to Zoltan Miskolci in LEAHI-DIALIN-LDT-4160-1

even though we are sending U08 at Firmware. Due to structure padding Raw message for priority is converted as 16 bits. While receiving we need to read them as U16. Here is my debug values [CortexR4] ALARM_PROP_T size = 22 bytes
byte[0] = 0x03
byte[1] = 0x00
byte[2] = 0x6F
byte[3] = 0x00
byte[4] = 0x01
byte[5] = 0x00
byte[6] = 0x00
byte[7] = 0x01
byte[8] = 0x00
byte[9] = 0x00
byte[10] = 0x01
byte[11] = 0x00
byte[12] = 0x00
byte[13] = 0x01
byte[14] = 0x00
byte[15] = 0x00
byte[16] = 0x01
byte[17] = 0x00
byte[18] = 0x00
byte[19] = 0x01
byte[20] = 0x0A
byte[21] = 0x00

This is structure defined at firmware side
struct AlarmTableRequestPayload

Unknown macro: { U08 alarmPriority; U16 alarmSubRank; U08 alarmSource; U08 alarmIsFault; U08 alarmIsDDFault; U08 alarmStops; U08 alarmConditionClearImmed; ///< Alarm condition is cleared immediately (does not persist after alarm triggered) U08 alarmNoClear; ///< Alarm cannot be cleared (unrecoverable)? U08 alarmNoResume; ///< Alarm prevents treatment resume U08 alarmNoRinseback; ///< Alarm prevents rinseback U08 alarmNoEndTreatment; ///< Alarm prevents ending treatment U08 alarmBlockRinseback; ///< Alarm should not trigger while in rinseback states U08 alarmBlockEndTx; ///< Alarm should not trigger while in post-treatment mode U08 alarmNoBloodRecirc; ///< Alarm prevents blood re-circulation U08 alarmNoDialysateRecirc; ///< Alarm prevents dialysate re-circulation U08 alarmAutoResume; ///< Alarm should resume automatically once the condition is cleared U08 alarmClearOnly; ///< Clear only this alarm on user acknowledgment U08 alarmTreatmentLog; ///< Log alarm to treatment log if active during treatment U16 alarmID; ///< Alarm ID }

;
typedef struct AlarmTableRequestPayload ALARM_PROP_T;

LEAHI-DIALIN-LDT-4484-1: comment by Behrouz NematiPour "So the same channels are used for both the Leah..."

Behrouz NematiPour — Wed, 01 Jul 2026 12:07:36 -0700

Behrouz NematiPour replied to Zoltan Miskolci in LEAHI-DIALIN-LDT-4484-1

So the same channels are used for both the Leahi and Denali?
I am not talking about how collisions are handled; that is handled by the CAN driver, not the FW or SW.
My point is that the CAN Channels by themselves do not have the meaning we want; they are only a priority.
As we defined them, we gave them a meaning as a category; therefore, it is better to name them the Denali/Leahi Channels rather than the CAN Channels.

LEAHI-DIALIN-LDT-4484-1: comment by Behrouz NematiPour "RESOLVED."

Behrouz NematiPour — Wed, 01 Jul 2026 12:06:39 -0700

Behrouz NematiPour replied to Zoltan Miskolci in LEAHI-DIALIN-LDT-4484-1

RESOLVED.

LEAHI-DIALIN-LDT-4484-1: comment by Behrouz NematiPour "{code} def fp_versions(self, major: int, minor..."

Behrouz NematiPour — Wed, 01 Jul 2026 12:05:06 -0700

Behrouz NematiPour replied to Zoltan Miskolci in LEAHI-DIALIN-LDT-4484-1

def fp_versions(self, major: int, minor: int, micro: int, build: int,
                     fpga_id: int, fpga_major: int, fpga_minor: int, fpga_lab: int,
                     compatibility_rev: int):
     """
     Broadcasts the current dd Version Data (Msg ID: 0x0F, 15)
     Args:
         @param major: (uint) - Major version number
         @param minor: (uint) - Minor version number
         @param micro: (uint) - Micro version number
         @param build: (uint) - Build version number
         @param fpga_id: (int) - FPGA id version number
         @param fpga_major: (int) - FPGA Major version number
         @param fpga_minor: (int) - FPGA Minor version number
         @param fpga_lab: (int) - FPGA Lab version number
         @param compatibility_rev: (uint) - The FWs/UI compatibility revision
     @return: None
     """
     if not self.can_enabled:
         raise ValueError("CAN Interface is not enabled")

     payload =  conversions.unsigned_byte_to_bytearray   (major            )
     payload += conversions.unsigned_byte_to_bytearray   (minor            )
     payload += conversions.unsigned_byte_to_bytearray   (micro            )
     payload += conversions.unsigned_short_to_bytearray  (build            )
     payload += conversions.unsigned_byte_to_bytearray   (fpga_id          )
     payload += conversions.unsigned_byte_to_bytearray   (fpga_major       )
     payload += conversions.unsigned_byte_to_bytearray   (fpga_minor       )
     payload += conversions.unsigned_byte_to_bytearray   (fpga_lab         )
     payload += conversions.unsigned_integer_to_bytearray(compatibility_rev)

     message = CAN.CanMessage.build_message(
                                         channel_id=CAN.CanChannels.fp_sync_broadcast_ch_id,
                                         message_id=msg_ids.MsgIds.MSG_ID_FP_VERSION_RESPONSE.value,
                                         payload=payload)

     self.can_interface.send(message, 0)

leahi-dialin (develop): Merge branch 'LDT-4505' into develop

Zoltan Miskolci — Wed, 01 Jul 2026 11:08:37 -0700

Zoltan Miskolci committed ad4c490 on branch develop in leahi-dialin

Merge branch 'LDT-4505' into develop

· /leahi_dialin/td/modules/blood_flow.py

(-1, +1)

| History

| Source

| Diff

leahi-dialin (LDT-4733-request-add-variables): Type change

Zoltan Miskolci — Wed, 01 Jul 2026 11:08:03 -0700

Zoltan Miskolci committed d4692c9 on branch LDT-4505, LDT-4614-dialysate-composition-mixing-algo-integration-with-td-proxy and 11 more in leahi-dialin

Type change

· /leahi_dialin/td/modules/blood_flow.py

(-1, +1)

| History

| Source

| Diff

LEAHI-DIALIN-LDT-4505-1: started by Dara Navaei

Dara Navaei — Wed, 01 Jul 2026 11:03:31 -0700

Dara Navaei started review LEAHI-DIALIN-LDT-4505-1

LEAHI-DIALIN-LDT-4160-1: comment by Zoltan Miskolci "FW sends U16 for now, after it's changed to U08..."

Zoltan Miskolci — Wed, 01 Jul 2026 10:06:19 -0700

Zoltan Miskolci replied to Vinayakam Mani in LEAHI-DIALIN-LDT-4160-1

FW sends U16 for now, after it's changed to U08, this will be updated

LEAHI-DIALIN-LDT-4160-1: comment by Vinayakam Mani "Priority data type is U08?"

Vinayakam Mani — Tue, 30 Jun 2026 14:57:25 -0700

Vinayakam Mani commented on LEAHI-DIALIN-LDT-4160-1

Priority data type is U08?

leahi-dialin (LDT-3735-non-volatile-memory): Merge branch 'staging' into LDT-3735-non-volatile-memory

Zoltan Miskolci — Mon, 29 Jun 2026 12:30:31 -0700

Zoltan Miskolci committed 5994ade on branch LDT-3735-non-volatile-memory in leahi-dialin

Merge branch 'staging' into LDT-3735-non-volatile-memory

· /leahi_dialin/common/dd_defs.py	(-2, +27)	\| History	\| Source	\| Diff
· /leahi_dialin/common/dd_defs_proxy.py	(-0, +1)	\| History	\| Source	\| Diff
· /leahi_dialin/common/msg_ids.py	(-8, +35)	\| History	\| Source	\| Diff
· /leahi_dialin/dd/dialysate_delivery.py	(-30, +2)	\| History	\| Source	\| Diff
· /leahi_dialin/td/treatment_delivery.py	(-10, +14)	\| History	\| Source	\| Diff

LEAHI-DIALIN-LDT-4160-1: reviewers finished

Mon, 29 Jun 2026 12:28:46 -0700

Zoltan Miskolci , Tiffany Mejia and Michael Garthwaite finished reviewing LEAHI-DIALIN-LDT-4160-1

LEAHI-DIALIN-LDT-4160_[Request] Get function for full detailed Alarm Table