Skip to content

Ignore error frames in can.player by default #690

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

Merged
merged 3 commits into from
Sep 4, 2019

Conversation

christiansandberg
Copy link
Collaborator

They are usually not of interest and may fill up the transmit buffer.

Fixes #683

@christiansandberg christiansandberg added this to the 4.0 Release milestone Aug 19, 2019
@christiansandberg christiansandberg self-assigned this Aug 19, 2019
Copy link
Collaborator

@felixdivo felixdivo left a comment

Choose a reason for hiding this comment

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

Sounds reasonable

@codecov
Copy link

codecov bot commented Sep 3, 2019

Codecov Report

Merging #690 into develop will increase coverage by 0.03%.
The diff coverage is 25%.

@@             Coverage Diff             @@
##           develop     #690      +/-   ##
===========================================
+ Coverage    68.67%   68.71%   +0.03%     
===========================================
  Files           69       69              
  Lines         6254     6249       -5     
===========================================
- Hits          4295     4294       -1     
+ Misses        1959     1955       -4

@codecov
Copy link

codecov bot commented Sep 3, 2019

Codecov Report

Merging #690 into develop will increase coverage by 3.01%.
The diff coverage is 25%.

@@             Coverage Diff             @@
##           develop     #690      +/-   ##
===========================================
+ Coverage    65.69%   68.71%   +3.01%     
===========================================
  Files           69       69              
  Lines         6134     6249     +115     
===========================================
+ Hits          4030     4294     +264     
+ Misses        2104     1955     -149

@felixdivo felixdivo requested a review from Lauszus September 4, 2019 07:44
Copy link
Collaborator

@Lauszus Lauszus 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 to me as well.

@felixdivo felixdivo merged commit 1ac5510 into develop Sep 4, 2019
@felixdivo felixdivo deleted the player-error-frame-fix branch September 4, 2019 09:18
@karlding karlding mentioned this pull request Oct 25, 2019
hardbyte pushed a commit that referenced this pull request Dec 28, 2019
Expose options to send or skip error frame messages
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Problems with player.py
3 participants