Skip to content

Add nanosecond resolution time stamping to socketcan#1015

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.

Sign up for GitHub

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

Jump to bottom
Merged
mergify merged 9 commits into from
Apr 19, 2021
Merged

Add nanosecond resolution time stamping to socketcan #1015

mergify merged 9 commits into from
Apr 19, 2021

Conversation

@felixdivo
Copy link
Collaborator

@felixdivofelixdivo commented Apr 16, 2021•
edited
Loading

Implements and closes#938.

@felixdivofelixdivo added this to the 4.0.0 Release milestone Apr 16, 2021
@felixdivofelixdivo self-assigned this Apr 16, 2021
@mergifymergifybot requested a review from hardbyteApril 16, 2021 09:21
@codecov
Copy link

codecovbot commented Apr 16, 2021•
edited
Loading

Codecov Report

Merging #1015 (5794fd0) into develop (0b7e615) will increase coverage by 0.04%.
The diff coverage is 89.47%.

@@ Coverage Diff @@## develop #1015 +/- ## =========================================== + Coverage 70.72% 70.76% +0.04%  =========================================== Files 78 78 Lines 7542 7553 +11 =========================================== + Hits 5334 5345 +11  Misses 2208 2208

hardbyte
hardbyte approved these changes Apr 17, 2021
View reviewed changes
Copy link
Owner

@hardbytehardbyte left a comment

Choose a reason for hiding this comment

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

🕐

felixdivoand others added 2 commits April 18, 2021 12:04
@felixdivo@hardbyte
Update can/interfaces/socketcan/socketcan.py
f14f699
 fix typo Co-authored-by: Brian Thorne <[email protected]>
@felixdivo@hardbyte
Update doc/interfaces/socketcan.rst
2476dba
 fix typo Co-authored-by: Brian Thorne <[email protected]>
@felixdivo
Copy link
CollaboratorAuthor

felixdivo commented Apr 18, 2021

@Mergifyio refresh

@mergify
Copy link
Contributor

mergifybot commented Apr 18, 2021

Command refresh: success

Pull request refreshed

@hardbytehardbyte mentioned this pull request Apr 19, 2021
Merged
@mergifymergifybot merged commit 13c0e75 into developApr 19, 2021
@mergifymergifybot deleted the socketcan-timestamp-ns branch April 19, 2021 23:29
@felixdivofelixdivo mentioned this pull request Apr 22, 2021
Closed
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@hardbytehardbytehardbyte approved these changes

Assignees

@felixdivofelixdivo

Labels

backend:socketcanenhancementos:linux

Projects

None yet

Milestone

4.0.0 Release

Development

Successfully merging this pull request may close these issues.

Could use nansosecond resolution time stamping for Linux sockets

3 participants

@felixdivo@hardbyte