Skip to content

SITL: In SIM::Battery, add "_amps" suffix on current variable names#32810

Open
hunt0r wants to merge 2 commits intoArduPilot:masterfrom
hunt0r:issue10050-add-amps-suffix-on-current-variables
Open

SITL: In SIM::Battery, add "_amps" suffix on current variable names#32810
hunt0r wants to merge 2 commits intoArduPilot:masterfrom
hunt0r:issue10050-add-amps-suffix-on-current-variables

Conversation

@hunt0r
Copy link
Copy Markdown
Contributor

@hunt0r hunt0r commented Apr 17, 2026

Summary

Simply rename battery current variables current to current_amps.
It is trivially "PR-chained" behind #32808 . (I will rebase for clarity when that one lands.)

Classification & Testing (check all that apply and add your own)

  • Checked by a human programmer
  • Non-functional change
  • No-binary change
  • Infrastructure change (e.g. unit tests, helper scripts)
  • Automated test(s) verify changes (e.g. unit test, autotest)
  • Tested manually, description below (e.g. SITL)
  • Tested on hardware
  • Logs attached
  • Logs available on request

This is confirmed correct by successfully compiling and also SIM::Battery's unit tests.

@hunt0r hunt0r marked this pull request as draft April 17, 2026 13:08
@hunt0r hunt0r force-pushed the issue10050-add-amps-suffix-on-current-variables branch from ab9639f to 5090b6d Compare April 17, 2026 13:21
@hunt0r hunt0r marked this pull request as ready for review April 18, 2026 12:03
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.

2 participants