Skip to content

Conversation

@occam25
Copy link
Contributor

Summary

Recently socketcand project merged a PR to enable error CAN frames ( linux-can/socketcand#67 ) so now it can send error messages through the TCP socket when an error frame is generated by the CAN driver.

This PR updates the python-can socketcand interface to generate an error CAN message when socketcand error message is received

@zariiii9003
Copy link
Collaborator

Could you add some test for this?

@occam25
Copy link
ContributorAuthor

Could you add some test for this?

Hi @zariiii9003 ,
I have added the test. Please, have a look
Thks

@occam25occam25force-pushed the process_socketcand_error_messages branch from fa53e32 to b30b834CompareJune 12, 2025 14:38
Copy link
Collaborator

@zariiii9003zariiii9003 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, thank you 👍

@zariiii9003zariiii9003 merged commit 8970fce into hardbyte:mainJun 12, 2025
30 checks passed
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@occam25@zariiii9003