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

]]> Wed, 08 Jul 2026 14:02:43 -0700 Dara Navaei https://devapps.diality.us/cru/LEAHI-DIALIN-LDT-4979-6 https://devapps.diality.us/cru/LEAHI-DIALIN-LDT-4979-6 Dara Navaei started review LEAHI-DIALIN-LDT-4979-6 ]]> Wed, 08 Jul 2026 13:46:52 -0700 https://devapps.diality.us/cru/LEAHI-DIALIN-LDT-4733-1 https://devapps.diality.us/cru/LEAHI-DIALIN-LDT-4733-1 Vinayakam Mani , Sean Nash and Zoltan Miskolci finished reviewing LEAHI-DIALIN-LDT-4733-1 LEAHI-DIALIN-LDT-4733_(Request) Add variables to capture AC/DC heater electrical data ]]> Wed, 08 Jul 2026 10:45:48 -0700 https://devapps.diality.us/cru/LEAHI-DIALIN-LDT-4734-1 https://devapps.diality.us/cru/LEAHI-DIALIN-LDT-4734-1 Sean Nash finished reviewing LEAHI-DIALIN-LDT-4734-1 LEAHI-DIALIN-LDT-4734_Water Treatment and Degassing - FW Iteration - 5/4: ]]> Mon, 06 Jul 2026 15:08:42 -0700 Vendor - TEL - Santhoshkumar Reddy Alasandagutti https://devapps.diality.us/cru/LEAHI-DIALIN-LDT-4160-1 https://devapps.diality.us/cru/LEAHI-DIALIN-LDT-4160-1 Vendor - TEL - Santhoshkumar Reddy Alasandagutti Closed LEAHI-DIALIN-LDT-4160-1 ]]> Mon, 06 Jul 2026 15:08:20 -0700 Vendor - TEL - Santhoshkumar Reddy Alasandagutti https://devapps.diality.us/cru/LEAHI-DIALIN-LDT-4539-1 https://devapps.diality.us/cru/LEAHI-DIALIN-LDT-4539-1 Vendor - TEL - Santhoshkumar Reddy Alasandagutti Closed LEAHI-DIALIN-LDT-4539-1 ]]> Mon, 06 Jul 2026 15:02:57 -0700 https://devapps.diality.us/cru/LEAHI-DIALIN-LDT-4873-1 https://devapps.diality.us/cru/LEAHI-DIALIN-LDT-4873-1 Tiffany Mejia finished reviewing LEAHI-DIALIN-LDT-4873-1 LEAHI-DIALIN-LDT-4873_(Request) Add config to disable conductive level sensor fault ]]> Mon, 06 Jul 2026 12:55:22 -0700 https://devapps.diality.us/cru/LEAHI-DIALIN-LDT-4072-1 https://devapps.diality.us/cru/LEAHI-DIALIN-LDT-4072-1 Sean Nash finished reviewing LEAHI-DIALIN-LDT-4072-1 LEAHI-DIALIN-LDT-4072_Power Loss - FW Implementation - 2/3: ]]> Thu, 02 Jul 2026 15:39:35 -0700 Nicholas Ramirez https://devapps.diality.us/cru/LEAHI-DIALIN-LDT-4787-1 https://devapps.diality.us/cru/LEAHI-DIALIN-LDT-4787-1 Nicholas Ramirez Closed LEAHI-DIALIN-LDT-4787-1 ]]> Thu, 02 Jul 2026 15:38:23 -0700 https://devapps.diality.us/cru/LEAHI-DIALIN-LDT-4787-1 https://devapps.diality.us/cru/LEAHI-DIALIN-LDT-4787-1 Behrouz NematiPour and Stephen Quong finished reviewing LEAHI-DIALIN-LDT-4787-1 LEAHI-DIALIN-LDT-4787_NON-FEATURE: SW Misc & Enhc - SW - 01 - Power Control - Shutdown - R&I ]]> Thu, 02 Jul 2026 15:36:14 -0700 Nicholas Ramirez https://devapps.diality.us/cru/LEAHI-DIALIN-LDT-4787-1/#c30362 https://devapps.diality.us/cru/LEAHI-DIALIN-LDT-4787-1/#c30362 Nicholas Ramirez replied to Nicholas Ramirez in LEAHI-DIALIN-LDT-4787-1

bn approves

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

fixed. thanks!

fixed. thanks

Add uncompensated conductivity to the init too