M-Labs
/
thermostat
9
0
Fork 4
Code Issues 8 Pull Requests 7 Releases Wiki Activity

wrong current reports #99

New Issue
Closed
opened 2024-03-07 12:03:14 +08:00 by esavkin · 1 comment
esavkin commented 2024-03-07 12:03:14 +08:00
Owner

image

![image](/attachments/7a6c903e-b34d-4dfa-a906-320e050c83a6)
image.png
13 KiB
atse commented 2024-03-18 10:20:49 +08:00

Ahh, this is a bug from #84, where the firmware just saves whatever the user inputs and displays it, even if the input was invalid and the current doesn't actually gets set. Need to implement input checking for currents instead of relying on the readback of the dac code.

Ahh, this is a bug from #84, where the firmware just saves whatever the user inputs and displays it, even if the input was invalid and the current doesn't actually gets set. Need to implement input checking for currents instead of relying on the readback of the dac code.
sb10q closed this issue 2024-04-24 18:05:20 +08:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
master
GUI
softspi
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
2 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#99
There is no content yet.
Delete Branch "%!s(<nil>)"

Deleting a branch is permanent. Although the deleted branch may exist for a short time before cleaning up, in most cases it CANNOT be undone. Continue?