Found a few bugs in 1.5.3 beta. Note the bold text:
class IS_NLP:
...
self.Size, self.Type, self.ReqI, self.NumP = struct.unpack('BBBB', data[:4])
offset = 4
while offset < self.Size:
nl = NodeLap()
nl.unpack(data[offset:offset + [b]4][/b])
self.NodeLaps.append(nl)
offset += [b]4[/b]
A NodeLap struct is 6 bytes long... change the 4s to 6s and it's fixed.
class IS_RST:
...
def __init__(self, ReqI=0, RaceLaps=0, QualMins=0, NumP=0, Track='', Weather=0, Wind=0, Flags=0, NumNodes=0, Finish=0, Split1=0, Split2=0, Split3=0):
...
self.Zero = [b]Zero[/b]
class IS_MSX:
...
def __init__(self, ReqI=0, Msg=''):
...
self.Zero = [b]Zero[/b]
class IS_AXI:
...
def __init__(self, ReqI=0, AXStart=0, NumCP=0, NumO=0, LName=''):
...
self.Zero = [b]Zero[/b]
Zero is never defined so these assignments all throw exceptions.
Also, you might consider wrapping the code in onPacketReceived with a try/except block and printing the stack trace on exceptions. I don't know if it's something weird I'm doing and not realizing it, but my Pyinsim apps always fail silently when my Pyinsim-related code raises an exception. I figure it's either me being dumb or the threading module suppressing output to stderr.
Example of what I did:
import traceback
...
def onPacketReceived(self, data):
"""Virtual method called when a packet is received. Handles the keep
alive pulse and raises packet events.
@type data: string
@param data: Packet data as a binary formatted string.
"""
try:
packetType = _packetType(data)
# Keep alive.
if packetType == ISP_TINY:
if _tinyType(data) == TINY_NONE:
self.sendB(data)
# Raise packet event.
if packetType in self.__callbacks:
for callback, _customPacket in self.__callbacks[packetType]:
if _customPacket:
packet = _customPacket()
else:
packet = _PACKET_DEFS[packetType]()
packet.unpack(data)
callback(self, packet)
except:
traceback.print_exc()
