Skip to content

Ignore error frames in can.player by default #690

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # 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
# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Problems with player.py
3 participants