Skip to content
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

Color-code stealth mode in the TMC2130 indicator #306

Merged
merged 4 commits into from
Apr 18, 2021

Conversation

wavexx
Copy link
Collaborator

@wavexx wavexx commented Apr 18, 2021

Description

Use a different color ("acid green") for stealth mode, so that we can distinguish the driver state.
Stuff you'd normally do "by ear" ;)

Behaviour/ Breaking changes

No change in code, just in visuals:

2021-04-18T223827

Have you tested the changes?

Lookin' good

Other

Linked issues:

Going to use this to debug the nasty "Delta_PSU" behavior...

Use a different color ("acid green") for stealth mode, so that we can
distinguish the driver state.
parts/components/TMC2130.cpp Outdated Show resolved Hide resolved
@codecov-commenter
Copy link

codecov-commenter commented Apr 18, 2021

Codecov Report

Merging #306 (51539fb) into master (07579a3) will increase coverage by 0.00%.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #306   +/-   ##
=======================================
  Coverage   89.16%   89.17%           
=======================================
  Files         140      140           
  Lines        6619     6622    +3     
=======================================
+ Hits         5902     5905    +3     
  Misses        717      717           
Impacted Files Coverage Δ
parts/components/TMC2130.h 100.00% <ø> (ø)
parts/components/TMC2130.cpp 93.75% <100.00%> (+0.09%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 07579a3...51539fb. Read the comment docs.

@vintagepc vintagepc merged commit 616b76d into vintagepc:master Apr 18, 2021
@wavexx wavexx deleted the highlight_stealth_mode branch September 11, 2021 14:12
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.

3 participants