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

simbatt: Clear BATTERY_CAPACITY_RELATIVE field for per-battery charge display #1063

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

forderud
Copy link
Contributor

@forderud forderud commented Dec 7, 2023

Clear BATTERY_INFORMATION::BATTERY_CAPACITY_RELATIVE field to enable per-battery charge display in Windows Settings when simulating multi-battery setups.

Motivating example

Example of simulated multi-battery setup prior to this PR:
image
As you can see, the individual batteries report 0% despite being fully charged. The aggregated 100% charge is however correct.

Corresponding example after clearing the BATTERY_CAPACITY_RELATIVE field:
image
As you can see, the individual batteries now correctly report 100%.

Testing instructions

Steps:

@forderud
Copy link
Contributor Author

forderud commented May 3, 2024

Gentle reminder.

@JakobL-MSFT JakobL-MSFT deleted the branch microsoft:develop May 22, 2024 16:08
@forderud
Copy link
Contributor Author

@JakobL-MSFT Would it be possible to get some kind of explanation for why this PR was closed instead of being considered for merging?

@JakobL-MSFT
Copy link
Contributor

I closed this PR unintentionally. I apologize. I am reopening the PR.

Clear BATTERY_INFORMATION::BATTERY_CAPACITY_RELATIVE field to enable per-battery charge display in Windows Settings.
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