M-Labs/thermostat
9
0
Fork 5
Code Issues 9 Pull Requests 5 Releases Wiki Activity

pwm: Show minimum in pwm fields summary #122

Closed
atse wants to merge 1 commits from atse:pwm-summary_minimum into master
pull from: atse:pwm-summary_minimum
merge into: M-Labs:master
Conversation 1 Commits 1 Files Changed 2 +14 -13
atse commented 2024-08-08 18:09:52 +08:00
Contributor
Copy Link

So that the full ranges of PWM values are shown.

So that the full ranges of PWM values are shown.
atse added 1 commit 2024-08-08 18:09:52 +08:00
atse commented 2024-08-09 10:10:44 +08:00
Author
Contributor
Copy Link

Actually, after thinking about this now, there's no need to show the range if it doesn't change, it just needs to be documented somewhere, especially because it's now clamped anyway. Closing.

Actually, after thinking about this now, there's no need to show the range if it doesn't change, it just needs to be documented somewhere, especially because it's now clamped anyway. Closing.
atse closed this pull request 2024-08-09 10:10:49 +08:00

Pull request closed

This pull request cannot be reopened because the branch was deleted.
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
1 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: M-Labs/thermostat#122
No description provided.
Delete Branch "atse:pwm-summary_minimum"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?