Daniel Hrisca
Daniel Hrisca
With the latest code (v 4.1) the line is this https://github.com/ratal/mdfreader/blob/master/mdfreader/mdfreader.py#L834 ```python f = open(file_name, "wt", encoding=encoding, newline="") ``` This works on my setup
@Nimi42 Hi, can you send the file (you can find my email on my profile page).
@ratal I think _mdfreader_ reads 228 bytes for CNBLOCK. In .dat version 2.14 the CNBLOCK only has 222 bytes (there is no display name and additional byte offset).
2.14 file  3.10 file 
@Nimi42 I was able to parse the file that you have send over email with the code in the development branch. Do you have problems with other files?
@Nimi42 I've added initial support for MDF version 2.14 to the development branch https://github.com/danielhrisca/asammdf/tree/development/asammdf . I think issues not related to mdfreader should continue on the asammdf issue tracker.
> But it should be: > > [ 4.246 5.238 6.238 7.238 8.23799 9.23799 ... ] > > Similarly with mdfreader What application did you use to confirm those values?
@Nimi42 You're right. CANape shows correct values. ~~It must be something strange with the data layout in version 2 (compared to versions 3 and 4)~~
@raffsa this is obviously an error in the [_asammdf_](https://github.com/danielhrisca/asammdf) package, not in _mdfreader_. Please open an issue there.
Yes it is mostly the transport layer that needs adjusting. In this package it is needed to remove this hard assert if other transports are to be supported https://github.com/pylessard/python-udsoncan/blob/master/udsoncan/connections.py#L448 with...