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

Peltierupdate #118

Open
wants to merge 10 commits into
base: DEV
Choose a base branch
from
Open

Peltierupdate #118

wants to merge 10 commits into from

Conversation

davidkim918
Copy link
Collaborator

No description provided.

@davidkim918 davidkim918 requested a review from mlj5j October 13, 2023 16:55
@ryansantos1174
Copy link
Collaborator

Currently, there are two LCD widgets displaying temperatures on top of one another making it difficult to read the temperature of thermistor 1.

@ryansantos1174
Copy link
Collaborator

We currently don't have multiple thermistors to test the halt signals. Otherwise, I believe the pull request is good. We just need to keep this branch up to date until we get the second thermistor.

@ryansantos1174 ryansantos1174 added the Awaiting Test Looks good but needs to be tested before merge label Jan 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Awaiting Test Looks good but needs to be tested before merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants