Skip to content

Added motor and current saturation to the led status bar#82

Open
acheronfail wants to merge 1 commit into
lukash:mainfrom
acheronfail:acheronfail/status-current-colours
Open

Added motor and current saturation to the led status bar#82
acheronfail wants to merge 1 commit into
lukash:mainfrom
acheronfail:acheronfail/status-current-colours

Conversation

@acheronfail
Copy link
Copy Markdown
Contributor

@acheronfail acheronfail commented Apr 28, 2026

Following up on this suggestion here: #44 (comment)

A couple things to align on before this is ready to merge:

  • right now DUTY = orange, MOTOR I = magenta, BATTERY I = green - let me know what you'd prefer
  • currently when the battery is low the last LED pulses every second - do we want this faster/slower?
  • should I split out the battery blinking into a separate commit + feature: / leave it as is / update the current commit message + trailer? done in On low battery pulse a single red led #84
  • I renamed status_duty_blend -> status_blend, and also duty_threshold -> status_threshold, hope that's alright renamed to status.motor_utilization_threshold

@acheronfail acheronfail force-pushed the acheronfail/status-current-colours branch 2 times, most recently from fae1217 to 0d9be03 Compare May 2, 2026 01:17
Copy link
Copy Markdown
Owner

@lukash lukash left a comment

Choose a reason for hiding this comment

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

Thanks for this.

I was actually just recently thinking I should get to add the last led blinking when battery empty. Yes please definitely do that as a separate commit, it's independent from the motor utilization change. Could even go into a separate PR.

I'll have to find some time to do a full review, how determined are you to get this into 1.3? Should be possible... At first glance the code looks reasonable...

Comment thread src/conf/settings.xml Outdated
@acheronfail
Copy link
Copy Markdown
Contributor Author

I'll have to find some time to do a full review, how determined are you to get this into 1.3? Should be possible... At first glance the code looks reasonable...

I'm not fussed, if I want these features I can just put them on my board myself.

I was actually just recently thinking I should get to add the last led blinking when battery empty. Yes please definitely do that as a separate commit, it's independent from the motor utilization change. Could even go into a separate PR.

Sounds good, I'll remove from this PR and create a new one.

@acheronfail acheronfail force-pushed the acheronfail/status-current-colours branch from 0d9be03 to fae111a Compare May 8, 2026 12:40
@acheronfail acheronfail force-pushed the acheronfail/status-current-colours branch from fae111a to fae1f2e Compare May 8, 2026 12:45
Feature: Motor and Current saturation are shown in the led status bar
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