possible fix for issue #535

Author: ebroecker

src/canmatrix/formats/dbc.py

@@ -932,7 +932,7 @@ def add_frame_by_id(new_frame):  # type: (canmatrix.Frame) -> None
             ecu.name = ecu.attributes.get("SystemNodeLongSymbol")[1:-1]
             ecu.del_attribute("SystemNodeLongSymbol")
     for frame in db.frames:
-        frame.cycle_time = int(frame.attributes.get("GenMsgCycleTime", 0))
+        frame.cycle_time = int(float(frame.attributes.get("GenMsgCycleTime", 0)))
         if frame.attributes.get("SystemMessageLongSymbol", None) is not None:
             frame.name = frame.attributes.get("SystemMessageLongSymbol")[1:-1]
             frame.del_attribute("SystemMessageLongSymbol")

src/canmatrix/tests/test_dbc.py

@@ -489,3 +489,21 @@ def test_escaped_quotes():
     assert matrix.frames[0].signals[0].values[2] == r'string with "escaped" double quotes'
 
 
+def test_float_cycle_time():
+    dbc = io.BytesIO(textwrap.dedent(u'''\
+        BO_ 17 Frame_1: 8 Vector__XXX
+        SG_ sig2 : 8|8@1- (1,0) [0|0] "" Vector__XXX
+        SG_ sig1 : 0|8@1- (1,0) [0|0] "" Vector__XXX
+
+
+        BA_DEF_ BO_  "GenMsgCycleTime" INT 0 3600000;
+        BA_ "GenMsgCycleTime" BO_ 17 100.0;
+''').encode('utf-8'))
+
+
+    matrix = canmatrix.formats.dbc.load(dbc, dbcImportEncoding="utf8")
+
+    assert matrix.frames[0].cycle_time == 100
+    assert matrix.frames[0].signal_by_name("sig1").cycle_time == 10
+    assert matrix.frames[0].signal_by_name("sig2").cycle_time == 20
+