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

Fix darkness calculation in web interface #333

Closed
wants to merge 1 commit into from

Conversation

proski
Copy link
Contributor

@proski proski commented Feb 6, 2024

Round stored percent values to the closest hardware darkness value.

Round stored percent values to the closest hardware darkness value.
Copy link
Owner

@michaelrsweet michaelrsweet left a comment

Choose a reason for hiding this comment

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

You can't just round the number in one place.

@michaelrsweet michaelrsweet self-assigned this Feb 6, 2024
@michaelrsweet michaelrsweet added the wontfix This will not be worked on label Feb 6, 2024
@proski
Copy link
Contributor Author

proski commented Feb 6, 2024

The original code would only preserve 0%, 20%, 40%, 60%, 80% and 100% for TSPL driver with 16 darkness levels. Other values would slide one step towards 0% on every save.

@michaelrsweet michaelrsweet reopened this Feb 6, 2024
@michaelrsweet
Copy link
Owner

OK, I've re-tested and agree.

@michaelrsweet michaelrsweet added bug Something isn't working priority-medium and removed wontfix This will not be worked on labels Feb 6, 2024
@michaelrsweet michaelrsweet added this to the Stable milestone Feb 6, 2024
michaelrsweet added a commit that referenced this pull request Feb 6, 2024
michaelrsweet added a commit that referenced this pull request Feb 6, 2024
@michaelrsweet
Copy link
Owner

[master 7a395b9] Round darkness value (Issue #333)

[v1.4.x 8f62251] Round darkness value (Issue #333)

@proski proski deleted the darkness-webif-master branch February 9, 2024 05:15
michaelrsweet added a commit that referenced this pull request Jun 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working priority-medium
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants