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

update Compute() so it computes on the first call #109

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

tlatorre-uchicago
Copy link

This is a small change to update Comput() so that it produces a non-zero output on the first call. The issue I noticed was that the PID output after the first call was always zero. I guess for most setups this wouldn't be a big issue. The reason it is for me is that I'm using the PID output to set the duty cycle for a relay controlling a fridge, and the time window is 15 minutes. Therefore, every time I restarted the Arduino it would turn off the fridge for 15 minutes. I thought this might be useful to other people so I'm submitting this pull request.

@hildred
Copy link

hildred commented Feb 13, 2021

Thought provoking.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants