Index: leahi_dialin/dd/modules/heaters.py =================================================================== diff -u -rda47fb6cd59dfe360546408c65e1ce152848d7a0 -r6d104d3185ac3ed7c18c97ecdc13fd59bf53a8d1 --- leahi_dialin/dd/modules/heaters.py (.../heaters.py) (revision da47fb6cd59dfe360546408c65e1ce152848d7a0) +++ leahi_dialin/dd/modules/heaters.py (.../heaters.py) (revision 6d104d3185ac3ed7c18c97ecdc13fd59bf53a8d1) @@ -8,21 +8,24 @@ # @file heaters.py # # @author (last) Zoltan Miskolci -# @date (last) 07-Jan-2026 +# @date (last) 04-May-2026 # @author (original) Dara Navaei # @date (original) 29-May-2020 # ############################################################################ -import struct +# Module imports from logging import Logger +# Project imports from leahi_dialin.common.constants import NO_RESET from leahi_dialin.common import dd_enum_repository -from leahi_dialin.common.msg_defs import MsgIds, MsgFieldPositions +from leahi_dialin.common.generic_defs import DataTypes +from leahi_dialin.common.msg_defs import MsgIds from leahi_dialin.common.override_templates import cmd_generic_broadcast_interval_override, cmd_generic_override -from leahi_dialin.protocols.CAN import DenaliChannels -from leahi_dialin.utils.base import AbstractSubSystem, publish +from leahi_dialin.protocols.CAN import DenaliCanMessenger, DenaliChannels +from leahi_dialin.utils.abstract_classes import AbstractSubSystem +from leahi_dialin.utils.base import publish from leahi_dialin.utils.conversions import integer_to_bytearray, float_to_bytearray @@ -31,7 +34,7 @@ Dialysate Delivery (DD) Dialin API sub-class for heaters related commands. """ - def __init__(self, can_interface, logger: Logger): + def __init__(self, can_interface: DenaliCanMessenger, logger: Logger): """ @param can_interface: Denali CAN Messenger object """ @@ -41,9 +44,9 @@ self.logger = logger if self.can_interface is not None: - channel_id = DenaliChannels.dd_sync_broadcast_ch_id - self.msg_id_dd_heaters_data = MsgIds.MSG_ID_DD_HEATERS_DATA.value - self.can_interface.register_receiving_publication_function(channel_id, self.msg_id_dd_heaters_data, self._handler_heaters_sync) + self.can_interface.register_receiving_publication_function(channel_id = DenaliChannels.dd_sync_broadcast_ch_id, + message_id = MsgIds.MSG_ID_DD_HEATERS_DATA.value, + function = self._handler_heaters_sync) self.dd_heaters_timestamp = 0.0 #: The timestamp of the latest message @@ -91,36 +94,34 @@ @param message: published DD heaters data message @returns none """ - sensor_list =[] - sensor_list.append(['self.dd_heaters[dd_enum_repository.DDHeaterNames.D5_HEAT.name][dd_enum_repository.DDHeaterAttributes.HEAT_DUTY_CYCLE.name]', 'f']) - sensor_list.append(['self.dd_heaters[dd_enum_repository.DDHeaterNames.D45_HEAT.name][dd_enum_repository.DDHeaterAttributes.HEAT_DUTY_CYCLE.name]', 'f']) - sensor_list.append(['self.dd_heaters[dd_enum_repository.DDHeaterNames.D5_HEAT.name][dd_enum_repository.DDHeaterAttributes.HEAT_TARGET_TEMP.name]', 'f']) - sensor_list.append(['self.dd_heaters[dd_enum_repository.DDHeaterNames.D45_HEAT.name][dd_enum_repository.DDHeaterAttributes.HEAT_TARGET_TEMP.name]', 'f']) - sensor_list.append(['self.dd_heaters[dd_enum_repository.DDHeaterNames.D5_HEAT.name][dd_enum_repository.DDHeaterAttributes.HEAT_STATE.name]', 'i']) - sensor_list.append(['self.dd_heaters[dd_enum_repository.DDHeaterNames.D45_HEAT.name][dd_enum_repository.DDHeaterAttributes.HEAT_STATE.name]', 'i']) - sensor_list.append(['self.dd_heaters[dd_enum_repository.DDHeaterNames.D5_HEAT.name][dd_enum_repository.DDHeaterAttributes.FEED_FORWARD.name]', 'f']) - sensor_list.append(['self.dd_heaters[dd_enum_repository.DDHeaterNames.D5_HEAT.name][dd_enum_repository.DDHeaterAttributes.PWM_PERIOD.name]', 'f']) - sensor_list.append(['self.dd_heaters[dd_enum_repository.DDHeaterNames.D5_HEAT.name][dd_enum_repository.DDHeaterAttributes.ADJUSTED_TARGET_TEMP.name]', 'f']) - sensor_list.append(['self.dd_heaters[dd_enum_repository.DDHeaterNames.D5_HEAT.name][dd_enum_repository.DDHeaterAttributes.TARGET_TEMP_TD.name]', 'f']) - sensor_list.append(['self.dbg1', 'f']) - sensor_list.append(['self.dbg2', 'f']) - sensor_list.append(['self.dbg3', 'f']) - sensor_list.append(['self.dbg4', 'f']) - sensor_list.append(['self.dbg5', 'f']) - sensor_list.append(['self.dbg6', 'f']) - sensor_list.append(['self.dbg7', 'f']) - sensor_list.append(['self.dbg8', 'f']) - sensor_list.append(['self.dbg9', 'f']) + sensor_list = [] + sensor_list.append((dd_enum_repository.DDHeaterNames.D5_HEAT, dd_enum_repository.DDHeaterAttributes.HEAT_DUTY_CYCLE, DataTypes.F32)) + sensor_list.append((dd_enum_repository.DDHeaterNames.D45_HEAT, dd_enum_repository.DDHeaterAttributes.HEAT_DUTY_CYCLE, DataTypes.F32)) + sensor_list.append((dd_enum_repository.DDHeaterNames.D5_HEAT, dd_enum_repository.DDHeaterAttributes.HEAT_TARGET_TEMP, DataTypes.F32)) + sensor_list.append((dd_enum_repository.DDHeaterNames.D45_HEAT, dd_enum_repository.DDHeaterAttributes.HEAT_TARGET_TEMP, DataTypes.F32)) + sensor_list.append((dd_enum_repository.DDHeaterNames.D5_HEAT, dd_enum_repository.DDHeaterAttributes.HEAT_STATE, DataTypes.U32)) + sensor_list.append((dd_enum_repository.DDHeaterNames.D45_HEAT, dd_enum_repository.DDHeaterAttributes.HEAT_STATE, DataTypes.U32)) + sensor_list.append((dd_enum_repository.DDHeaterNames.D5_HEAT, dd_enum_repository.DDHeaterAttributes.FEED_FORWARD, DataTypes.F32)) + sensor_list.append((dd_enum_repository.DDHeaterNames.D5_HEAT, dd_enum_repository.DDHeaterAttributes.PWM_PERIOD, DataTypes.F32)) + sensor_list.append((dd_enum_repository.DDHeaterNames.D5_HEAT, dd_enum_repository.DDHeaterAttributes.ADJUSTED_TARGET_TEMP, DataTypes.F32)) + sensor_list.append((dd_enum_repository.DDHeaterNames.D5_HEAT, dd_enum_repository.DDHeaterAttributes.TARGET_TEMP_TD, DataTypes.F32)) + msg_list =[] + msg_list.append(('self.dbg1', DataTypes.F32)) + msg_list.append(('self.dbg2', DataTypes.F32)) + msg_list.append(('self.dbg3', DataTypes.F32)) + msg_list.append(('self.dbg4', DataTypes.F32)) + msg_list.append(('self.dbg5', DataTypes.F32)) + msg_list.append(('self.dbg6', DataTypes.F32)) + msg_list.append(('self.dbg7', DataTypes.F32)) + msg_list.append(('self.dbg8', DataTypes.F32)) + msg_list.append(('self.dbg9', DataTypes.F32)) - i = 1 - for sensor in sensor_list: - start_pos = eval(f'MsgFieldPositions.START_POS_FIELD_{i}') - end_pos = eval(f'MsgFieldPositions.END_POS_FIELD_{i}') - value = struct.unpack(sensor[1],bytearray(message['message'][start_pos:end_pos]))[0] - if 'nan' in str(value).lower(): - value = 0.0 - exec(f'{sensor[0]} = {value}') - i += 1 + self.process_into_dict(dict_to_update = self.dd_heaters, + decoder_list = sensor_list, + message = message) + self.process_into_vars(decoder_list = msg_list, + message = message, + start_from_byte = len(sensor_list) * DataTypes.U32.size()) self.dd_heaters_timestamp = timestamp