COVESA / vss-tools

Software for working with VSS (https://github.com/COVESA/vehicle_signal_specification)
Mozilla Public License 2.0
55 stars 55 forks source link

Removed C dependency from binary exporter #395

Closed SebastianSchildt closed 1 month ago

SebastianSchildt commented 1 month ago

This removes the C dependency from the binary, making it pure Python.

That would make it also usable when installing the Pypi packages without compiling any additional dll

@UlfBj can you check this works for you? Only changes are in vss2binary.py

sschleemilch commented 1 month ago

Looks pretty good to me, would be great to remove that dependency 👍

erikbosch commented 1 month ago

@SebastianSchildt - now there are conflicts

SebastianSchildt commented 1 month ago

@SebastianSchildt - now there are conflicts

Rebased. Added some additional make-up to pass our new ruff bouncer

erikbosch commented 1 month ago

@UlfBj - could you check if output is compatible

erikbosch commented 1 month ago

MoM:

erikbosch commented 1 month ago

I did a test on my Debian AMD64 VM comparing the output generated by this PR and latest master and the files were identical.

@UlfBj - what type of errors did you experience. Was it in generating the binary file, or that the binary file seems to be faulty? If it is latter case how can one try to reproduce it?

erikbosch commented 1 month ago

To me it seems that the build problems that Ulf experience is unrelated to this PR itself as he experience them also for master branch. I tested with a virgin Debian VM and did not experience any build problems, and the output generated with/without this PR was identical. Suggest merge decision at the meeting tonight

erikbosch commented 1 month ago

MoM: