Skip to content

SITL: Put SIM::Battery's temperature update into a function#32808

Open
hunt0r wants to merge 1 commit intoArduPilot:masterfrom
hunt0r:issue10050-update-battery-temperature-in-standalone-function
Open

SITL: Put SIM::Battery's temperature update into a function#32808
hunt0r wants to merge 1 commit intoArduPilot:masterfrom
hunt0r:issue10050-update-battery-temperature-in-standalone-function

Conversation

@hunt0r
Copy link
Copy Markdown
Contributor

@hunt0r hunt0r commented Apr 17, 2026

Summary

Simple change: Put a block of code into its own function.

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 covered by SIM::Battery's unit tests.
For fun, I added an incorrect change to the new code and verified locally that the test fails as expected.

Copy link
Copy Markdown
Contributor

@peterbarker peterbarker left a comment

Choose a reason for hiding this comment

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

LGTM

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