-
Notifications
You must be signed in to change notification settings - Fork 148
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
SavvyCAN compatible Session Files #20
Comments
I've pushed the modifications to this branch. Please try it out: |
Leave with me and I will check this weekend. I suspect further code is required to remove your "human" labels after the ID number for the export. |
Can you update the Python like this ? I have just created many logs for SavvyCan with this code.
|
@bionicbone I think it'd be much easier if you create a PR referencing this issue |
@Kwieeciol it would if I had got GitHub set up correctly |
This feature is really specific for SavvyCan, and would mess up other parts of my code: when the program reads back the saved session. This should be either a separate feature from the GUI, or the read back functionality of the GUI should also be aligned with the SavvyCan format. I have very limited availability nowadays, it would be certainly be faster if you can create the modifications and create a PR from it. |
Hi Adam,
I probably should create a pull request, but I've not got everything set up.
I made a change that you may want to consider, to the save file process that converts to time stamp to ms
Its not pritty, I've not used Python before so its just a logical flow (for me at least)
However, now the file can be loading directly into SavvyCAN for analysis, the program is free download
Once saved load into SavvyCAN using the "GVRET LOGS" format
Download SavvyCAN
https://github.com/collin80/SavvyCAN/releases?page=1
The text was updated successfully, but these errors were encountered: