Hello,
Something weird just appears in the MR :
Not sure of how the changes have been done, but something went wrong. Are you more than one working on it ?
That said … i m not following you 
You said that you still want to go with the separated cw_beacon and cw_type fields (so cw_type_i, cw_type_g, cw_type_h and cw_beacon_i, cw_beacon_g, cw_beacon_h), which again is totally fine
but in the same time you just updated the decoder in the MR with the unified approach (so with just one cw_beacon and cw_type fields).
So to be clear :
if you want to go with the separated fields, keep using :
This fields declaration :
:field cw_beacon_g: ax25_frame.cw_beacon_g
:field cw_type_g: ax25_frame.cw_type_g
:field cw_beacon_h: ax25_frame.cw_beacon_h
:field cw_type_h: ax25_frame.cw_type_h
:field cw_beacon_i: ax25_frame.cw_beacon_i
:field cw_type_i: ax25_frame.cw_type_i
and properties declaration in the kaitai structure :
in cw_g:
...
cw_type_g:
value: no001_message_identifier
cw_beacon_g:
value: no001_message_identifier+no002_telemetry_type_hex+no003_cobc_boot_count_hex+no004_cobc_uptime_seconds_hex+no005_cobc_temperature_degc_hex+no006_satellite_operation_mode_hex+no007_antenna_deployment_status_hex+no008_uplink_reception_count_hex+no009_battery_1_voltage_mv_hex+no010_battery_1_charging_current_first_half_ma_hex
in cw_h :
...
cw_type_h:
value: no001_message_identifier
cw_beacon_h:
value: no001_message_identifier+no002_battery_1_charging_current_second_half_ma_hex+no003_battery_1_discharging_current_ma_hex+no004_battery_1_temperature_degc_hex+no005_battery_2_voltage_mv_hex+no006_battery_2_charging_current_ma_hex+no007_battery_2_discharging_current_ma_hex+no008_battery_2_temperature_degc_hex+no009_subsystem_power_fault_status_hex+no010_subsystem_power_onoff_status_hex+no011_tobc_main_boot_count_hex
in cw_i :
...
cw_type_i:
value: no001_message_identifier
cw_beacon_i:
value: no001_message_identifier+no002_main_tobc_operating_time_hour_hex+no003_main_tobc_reception_count_hex+no004_sub_tobc_boot_count_hex+no005_sub_tobc_operating_time_hour_hex+no006_sub_tobc_reception_count_hex+no007_aobc_operation_mode_hex+no008_attctrl_power_status_hex+no009_x_axis_angular_velocity_mdeg_s_hex+no010_y_axis_angular_velocity_mdeg_s_hex+no011_z_axis_angular_velocity_mdeg_s_hex+no012_mobc_operation_mode_hex
OR if you want to go with the unified approach :
This fields declaration :
:field cw_beacon: ax25_frame.cw_beacon
:field cw_type: ax25_frame.cw_type
and properties declaration in the kaitai structure like this :
in cw_g:
...
cw_type:
value: no001_message_identifier
cw_beacon:
value: no001_message_identifier+no002_telemetry_type_hex+no003_cobc_boot_count_hex+no004_cobc_uptime_seconds_hex+no005_cobc_temperature_degc_hex+no006_satellite_operation_mode_hex+no007_antenna_deployment_status_hex+no008_uplink_reception_count_hex+no009_battery_1_voltage_mv_hex+no010_battery_1_charging_current_first_half_ma_hex
in cw_h :
...
cw_type:
value: no001_message_identifier
cw_beacon:
value: no001_message_identifier+no002_battery_1_charging_current_second_half_ma_hex+no003_battery_1_discharging_current_ma_hex+no004_battery_1_temperature_degc_hex+no005_battery_2_voltage_mv_hex+no006_battery_2_charging_current_ma_hex+no007_battery_2_discharging_current_ma_hex+no008_battery_2_temperature_degc_hex+no009_subsystem_power_fault_status_hex+no010_subsystem_power_onoff_status_hex+no011_tobc_main_boot_count_hex
in cw_i :
...
cw_type:
value: no001_message_identifier
cw_beacon:
value: no001_message_identifier+no002_main_tobc_operating_time_hour_hex+no003_main_tobc_reception_count_hex+no004_sub_tobc_boot_count_hex+no005_sub_tobc_operating_time_hour_hex+no006_sub_tobc_reception_count_hex+no007_aobc_operation_mode_hex+no008_attctrl_power_status_hex+no009_x_axis_angular_velocity_mdeg_s_hex+no010_y_axis_angular_velocity_mdeg_s_hex+no011_z_axis_angular_velocity_mdeg_s_hex+no012_mobc_operation_mode_hex
So the message here is, if you wanted to stay with the separated approach you didn’ t had to update it with my suggestion in the txt file. The decoder in the txt file was to show you how to apply the unified approach.
So whatever you want to chose, update the decoder accordingly as you which and i will review it tomorrow 