Hallo Leute, Ich möchte ein Python Toolset schreiben, mit dem ich sehr einfach auf Signale in CAN-Botschaften zugreifen kann, die als CAN-Trace vorliegen. Das parsen des Traces läuft. Nun möchte ich aber die ganzen Infos zu den CAN-Signalen in einem "Include"-File möglichst einfach zugänglich machen. Daran scheitere ich.
Tatsächlich gibt es zu jeder CAN-Botschaft mehrere Eigenschaften. Ich hatte das erstmal so codiert:
class CAN_AC_Voltage: def __init__(self): self.ID = 16FC8B11 self.BitOffset = 48 self.BitLength = 8 self.Gain = 1 self.ValueOffset = 0 self.Unit = "V" self.Rate = 0.5
Dann könnte ich so zugreifen: CAN_ACV = CAN_AC_Voltage()
ID = CAN_ACV.ID Unit = CAN_ACV.Unit
Was mich dran nervt ist, dass jede einzelne CAN-Botschaft eine eigene Klasse ist und ich die im Code erstmal instanzieren muss, bevor ich sie nutzen kann. Das muss doch auch einfacher gehen.
Ideen?
Am 19.02.21 um 15:27 schrieb Michael S.:
Was mich dran nervt ist, dass jede einzelne CAN-Botschaft eine eigene Klasse ist und ich die im Code erstmal instanzieren muss, bevor ich sie nutzen kann. Das muss doch auch einfacher gehen
Mache Klassen-Variabblen draus:
class CAN_AC_Voltage: ID = 0x16FC8B11 BitOffset = 48 BitLength = 8 Gain = 1 ValueOffset = 0 Unit = "V" Rate = 0.5
On 19/02/2021 15:27, Michael S. wrote:
Hallo Leute, Ich möchte ein Python Toolset schreiben, mit dem ich sehr einfach auf Signale in CAN-Botschaften zugreifen kann, die als CAN-Trace vorliegen. Das parsen des Traces läuft. Nun möchte ich aber die ganzen Infos zu den CAN-Signalen in einem "Include"-File möglichst einfach zugänglich machen. Daran scheitere ich.
Tatsächlich gibt es zu jeder CAN-Botschaft mehrere Eigenschaften. Ich hatte das erstmal so codiert:
class CAN_AC_Voltage: def __init__(self): self.ID = 16FC8B11 self.BitOffset = 48 self.BitLength = 8 self.Gain = 1 self.ValueOffset = 0 self.Unit = "V" self.Rate = 0.5
Dann könnte ich so zugreifen: CAN_ACV = CAN_AC_Voltage()
ID = CAN_ACV.ID Unit = CAN_ACV.Unit
Was mich dran nervt ist, dass jede einzelne CAN-Botschaft eine eigene Klasse ist und ich die im Code erstmal instanzieren muss, bevor ich sie nutzen kann. Das muss doch auch einfacher gehen.
from types import SimpleNamespace
CAN_AC_Voltage = SimpleNamespace( ID=16FC8B11 BitOffset=48 BitLength=8, ... )
Am 19.02.2021 um 15:27 schrieb Michael S.:
Das muss doch auch einfacher gehen.
Ideen?
Ich habs jetzt so gemacht. Da muss ich in der Anwendung nur die CAN-Signals_Klasse instanzieren, dafür halt in der Lib jeweils jedes eigene Signal. Aber die Lib wird eh automatisch generiert.
class C_frequency: def __init__(self): self.ID = "10FC80FC" self.BitOffset = 48 self.BitLength = 8 self.Gain = 1 self.ValueOffset = 0 self.Unit = "Hz" self.Rate = 0.5
class C_OutputVoltage: def __init__(self): self.ID = "25FF1250" self.BitOffset = 32 self.BitLength = 16 self.Gain = 0.1 self.ValueOffset = 0 self.Unit = "V" self.Rate = 0.25
class C_Cable_temp: def __init__(self): self.ID = "12FA5AFC" self.BitOffset = 0 self.BitLength = 8 self.Gain = 1 self.ValueOffset = -40 self.Unit = "°C" self.Rate = 10 class CAN_Signals: def __init__(self): self.frequency = C_frequency() self.OutputVoltage = C_OutputVoltage() self.Cable_temp = C_Cable_temp()
On Mon, 22 Feb 2021 07:00:47 +0100, Michael S. wrote:
Ich habs jetzt so gemacht. Da muss ich in der Anwendung nur die CAN-Signals_Klasse instanzieren, dafür halt in der Lib jeweils jedes eigene Signal. Aber die Lib wird eh automatisch generiert.
class C_frequency: def __init__(self): self.ID = "10FC80FC" self.BitOffset = 48 self.BitLength = 8 self.Gain = 1
self.ValueOffset
= 0 self.Unit = "Hz" self.Rate = 0.5class C_OutputVoltage: def __init__(self): self.ID = "25FF1250" self.BitOffset = 32 self.BitLength = 16 self.Gain = 0.1 self.ValueOffset = 0 self.Unit = "V" self.Rate = 0.25
class C_Cable_temp: def __init__(self): self.ID = "12FA5AFC" self.BitOffset = 0 self.BitLength = 8 self.Gain = 1
self.ValueOffset =
-40 self.Unit = "°C" self.Rate = 10class CAN_Signals: def __init__(self): self.frequency = C_frequency() self.OutputVoltage =
C_OutputVoltage()
self.Cable_temp = C_Cable_temp()
Verschiedene Klassen mit genau den gleichen Attributen sieht ”falsch” aus. Da würde man eher *eine* Klasse schreiben, die die Attribute hat, und davon dann mehrere Exemplare erstellen.
Mit `collections.namedtuple()` und Namen die sich an PEP8 halten, könnte das dann so aussehen:
from collections import namedtuple
Signal = namedtuple( "Signal", "id bit_offset bit_length gain value_offset unit rate" )
FREQUENCY = Signal("10FC80FC", 48, 8, 1, 0, "Hz", 0.5) OUTPUT_VOLTAGE = Signal("25FF1250", 32, 16, 0.1, 0, "V", 0.25) CABLE_TEMPERATURE = Signal("12FA5AFC", 0, 8, 1, -40, "°C", 10)
Ciao, Marc 'BlackJack' Rintsch
participants (4)
-
Hartmut Goebel -
Marc 'BlackJack' Rintsch -
Michael S. -
Peter Otten