Skip to content

Conversation

@tomasjakubik
Copy link
Contributor

Add hooks for errors

  • CAN controller errors via socketcan
  • when dropping a Tx frame

@tomasjakubik

This comment was marked as outdated.

@coveralls
Copy link

coveralls commented Jan 20, 2026

Coverage Status

coverage: 93.408% (-0.4%) from 93.785%
when pulling 9f0b95a on tomasjakubik:report-errors
into 9c97bd1 on OpenCyphal:master.

@tomasjakubik tomasjakubik force-pushed the report-errors branch 2 times, most recently from d53ad72 to d30391d Compare January 20, 2026 08:13
@tomasjakubik tomasjakubik marked this pull request as ready for review January 20, 2026 08:45
@tomasjakubik
Copy link
Contributor Author

tomasjakubik commented Jan 20, 2026

@pavel-kirienko Please have a look when you have time.
I am not sure that the hook/callback architecture is the way to go, so feel free to suggest something better.
I don't like the coverage decrease, but I am not sure if we can test this code reasonably without hardware 🤷.

Those two fixes for libpcap and SOL_CAN_RAW might probably go to a minor patch version if needed.
edit: Libcap was fixed and the commit dropped.

Updated pydsdl does not like it. It was breaking many tests.
@tomasjakubik
Copy link
Contributor Author

I think the missing setuptools came from the updated libpcap which changed the dependency.

As for the missing bpf_program it is probably also linked with changes in libpcap. I am looking into it.

Sign up for free to 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