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

Reverse polarity of current through the TEC #82

Closed
atse wants to merge 1 commits from atse:reverse-tec-polarity into master
pull from: atse:reverse-tec-polarity
merge into: M-Labs:master
Conversation 3 Commits 1 Files Changed 2 +4 -4
atse commented 2023-08-10 17:47:51 +08:00
Contributor
Copy Link

Matches the i_set polarity with the front panel of the Thermostat.

Matches the i_set polarity with the front panel of the Thermostat.
atse added 1 commit 2023-08-10 17:47:53 +08:00 
Matches the i_set polarity with the front panel of the Thermostat.
sb10q commented 2023-08-10 17:49:39 +08:00
Owner
Copy Link

I think other things also need changing? For example setting the current and also the PID autotune script.

I think other things also need changing? For example *setting* the current and also the PID autotune script.
atse commented 2023-08-11 10:17:46 +08:00
Author
Contributor
Copy Link

I think other things also need changing? For example setting the current and also the PID autotune script.

Setting the current was done in set_i, but I admit I haven't checked the PID autotune script, will do.

> I think other things also need changing? For example *setting* the current and also the PID autotune script. Setting the current was done in `set_i`, but I admit I haven't checked the PID autotune script, will do.
atse commented 2023-08-15 11:46:00 +08:00
Author
Contributor
Copy Link

After looking at hardware schematics, it turns out that the front panel on my unit is wrong, and had the TEC polarities reversed.

my unit

Thermostat v2.2.1 schematic

After looking at hardware schematics, it turns out that the front panel on my unit is wrong, and had the TEC polarities reversed. ![my unit](/attachments/c2ec0936-1862-4790-9886-2f3760916aef) ![Thermostat v2.2.1 schematic](/attachments/8ab86e5d-69be-4170-b9df-0cd356eba05a)
rn_image_picker_lib_temp_fee33f0a-8913-4d43-aa69-ea223e4297e7.jpg
494 KiB
image.png
42 KiB
👀 1
atse closed this pull request 2023-08-15 11:50:16 +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
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#82
No description provided.
Delete Branch "atse:reverse-tec-polarity"

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?