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 forderud force-pushed the simbatt-relative-rm branch from e285432 to 4149179 Compare January 31, 2024 11:28
@forderud forderud force-pushed the simbatt-relative-rm branch from 4149179 to 38797e5 Compare May 3, 2024 10:32
@forderud forderud requested a review from a team as a code owner May 3, 2024 10:32
@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.

@JakobL-MSFT JakobL-MSFT reopened this May 22, 2024
@forderud forderud force-pushed the simbatt-relative-rm branch from 38797e5 to fe85bdb Compare June 18, 2024 19:07
Clear BATTERY_INFORMATION::BATTERY_CAPACITY_RELATIVE field to enable per-battery charge display in Windows Settings.
@forderud forderud force-pushed the simbatt-relative-rm branch from fe85bdb to 97b9377 Compare August 21, 2024 10:17
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