iso15765_example.py sending corrupt ID and data.

[jvanorsdal]

Hi all. Hoping someone can give me some guidance here. I'm trying to get my feet wet with ICS python and I'm running into problems right out of the gate.

When I used a stripped down TX-only version of iso15765_example.py, the first frame does not send correctly. The arb ID is wrong and the message content is mostly empty. For example, I try to send 64 bytes on 0x7E0 per the example. The message that comes out on the other end is 0x1AA 10 40 00 00 00 00 00 00.

I've played with this a bit and found that it's using the tx.dl value for the upper 3 bits of the arb ID, and the padding value for the lower 8 bits. EG if I change tx_dl to 2 the arb ID is 0x2AA. If I change the padding to 0xAC, the arb ID is 0x1AC (or whatever the value of tx_dl is at that moment).

Sending single CAN messages with tx_can_example.py works perfectly. The problem seems isolated to 15765 messages.

My setup is a NeoVI Plasma with a Fire VNet, transmitting on HS-CAN1. I've got a Ford control module on the bus, and a Peak PCAN listening and providing network keepalive messages. I've verified all the data on the Ford and Peak side, so I'm confident the problem is with the Plasma or the underlying code.

OS is Windows 10.
icsneo40.dll is 3.9.19.17 installed from RP1210KitInstall.zip
Python is 3.13.5
python_ics is 922.16
I do not have a Vehicle Spy license.

Firmware on my Plasma is:
MPIC: 7.56
Core: 8.21
LPIC: 2.12
HID: 6.0

Here is my code:

import ics
import ctypes

enable_print_message = True
enable_use_server = True


# Helper Functions ##########################################################

def print_message(msg):
    if not enable_print_message:
        if isinstance(msg, ics.SpyMessage):
            print("\tArbID: {}\tData: {}".format(hex(msg.ArbIDOrHeader), [hex(x) for x in msg.Data]))
        return
    print("\t" + str(type(msg)))
    for attribute in dir(msg):
        if attribute.startswith("_"):
            continue
        length = len(attribute)
        if attribute == "data":
            print("\t\t{}:{}{}".format(attribute, " " * (30 - length), msg.data[: msg.num_bytes]))
        else:
            value = getattr(msg, attribute)
            try:
                value = hex(value)
            except:
                pass
            print("\t\t{}:{}{}".format(attribute, " " * (30 - length), value))
    print()


def open_device(index=0):
    device = None
    if enable_use_server:
        # ics.open_device() won't open a device if we have handles open already
        # so we need to find them and specify which ones to connect to.
        devices = ics.find_devices()
        print(
            "Opening Device {} (Open Client handles: {})...".format(
                devices[index], devices[index].NumberOfClients
            )
        )
        ics.open_device(devices[index])
        device = devices[index]
    else:
        print("Opening Device...")
        device = ics.open_device()
    print(f"Opened Device {device}.")
    return device


# Iso15765 Fuctions #########################################################
def transmit_iso15765_msg(device, netid=ics.NETID_HSCAN, is_canfd=False):
    number_of_bytes = 64
    msg = ics.st_cm_iso157652_tx_message.st_cm_iso157652_tx_message()
    msg.id = 0x7E0
    msg.vs_netid = netid
    msg.num_bytes = number_of_bytes
    msg.padding = 0xAA    
    # Flow Control
    msg.fc_id = 0x7E8
    msg.fc_id_mask = 0xFFF
    msg.flowControlExtendedAddress = 0xFE
    msg.fs_timeout = 0x10  # ms
    msg.fs_wait = 0x3000  # ms
    msg.blockSize = 0
    msg.stMin = 0
    # paddingEnable
    msg.paddingEnable = 1
    # CANFD: Enable + BRS
    if is_canfd:
        msg.iscanFD = 1
        msg.isBRSEnabled = 1
    # tx_dl
    msg.tx_dl = 1
    # Data
    my_data = [x for x in range(number_of_bytes)]
    #my_data = [0x41, 0x41, 0x41, 0x41, 0x41, 0x41, 0x41, 0x41, 0x41, 0x41]
    msg.data = (ctypes.c_ubyte * len(msg.data))(*my_data)


    # Transmit the message
    print("Transmitting iso15765 message on {}...".format(device))
    ics.iso15765_transmit_message(device, netid, msg, 3000)
    # Wait for the messages to be transmitted, this can be calculated a lot better but works here.
    time.sleep((((number_of_bytes / 8) * msg.fs_timeout) / 1000.0) + 0.5)
    slept = ((((number_of_bytes / 8) * msg.fs_timeout) / 1000.0) + 0.5)
    print_message(msg)
    print("Slept {} seconds.".format(slept))
    print("Transmitted iso15765 message on {}.".format(device))



if __name__ == "__main__":
    import time

    netid = ics.NETID_HSCAN

    tx_device = open_device(0)

    ics.iso15765_enable_networks(tx_device, netid)
    print("Enabled network: {}".format(netid))
    transmit_iso15765_msg(tx_device)

    ics.iso15765_disable_networks(tx_device)

This is the output from the Python console:

python iso15765_tx_only.py
Opening Device neoVI PLASMA SN REDACTED (Open Client handles: 0)...
Opened Device neoVI PLASMA SN REDACTED.
Enabled network: 1
Transmitting iso15765 message on neoVI PLASMA SN REDACTED...
<class 'ics.structures.st_cm_iso157652_tx_message.st_cm_iso157652_tx_message'>
: 0x0
Nameless18511: <ics.structures.st_cm_iso157652_tx_message.Nameless18511 object at 0x000001F65FF8C150>
blockSize: 0x0
data: [0, 1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12, 13, 14, 15, 16, 17, 18, 19, 20, 21, 22, 23, 24, 25, 26, 27, 28, 29, 30, 31, 32, 33, 34, 35, 36, 37, 38, 39, 40, 41, 42, 43, 44, 45, 46, 47, 48, 49, 50, 51, 52, 53, 54, 55, 56, 57, 58, 59, 60, 61, 62, 63]
ext_address_enable: 0x0
extendedAddress: 0x0
fc_ext_address_enable: 0x0
fc_id: 0x7e8
fc_id_29_bit_enable: 0x0
fc_id_mask: 0xfff
flags: 0x1000040
flowControlExtendedAddress: 0xfe
fs_timeout: 0x10
fs_wait: 0x3000
id: 0x7e0
id_29_bit_enable: 0x0
isBRSEnabled: 0x0
iscanFD: 0x0
num_bytes: 0x40
overrideBlockSize: 0x0
overrideSTmin: 0x0
padding: 0xaa
paddingEnable: 0x1
stMin: 0x0
tx_dl: 0x1
tx_index: 0x0
vs_netid: 0x1
Slept 0.628 seconds.
Transmitted iso15765 message on neoVI PLASMA SN REDACTED.

And this is the resulting CAN message: ID 0x1AA, DLC 8, Data 10 40 00 00 00 00 00 00

I sincerely appreciate any advice or hints. Thanks!