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

Gamma change fix #35

Merged
merged 3 commits into from
Sep 24, 2023
Merged

Gamma change fix #35

merged 3 commits into from
Sep 24, 2023

Conversation

trollLemon
Copy link
Owner

Fixed bug where low gamma values weren't applied due to the battery state of charge not included to check if the battery status changed.

Now, the code uses the battery state of charge to determine status changed, but only changes the screen to low gamma once. We use a bool to keep track if the screen was set to low.

@codecov
Copy link

codecov bot commented Sep 23, 2023

Codecov Report

Merging #35 (5de24e1) into main (63cb5c8) will increase coverage by 1.54%.
The diff coverage is 46.66%.

@@            Coverage Diff             @@
##             main      #35      +/-   ##
==========================================
+ Coverage   36.63%   38.18%   +1.54%     
==========================================
  Files           4        4              
  Lines         101      110       +9     
==========================================
+ Hits           37       42       +5     
- Misses         64       68       +4     
Files Changed Coverage Δ
src/daemon/mod.rs 36.26% <46.66%> (+2.11%) ⬆️

@trollLemon trollLemon merged commit 213faef into main Sep 24, 2023
5 of 6 checks passed
@trollLemon trollLemon deleted the gamma_change_fix branch September 24, 2023 04:07
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.

None yet

1 participant