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

When I lower the set-point, PID goes to min #31

Open
lukepighetti opened this issue Oct 18, 2022 · 6 comments
Open

When I lower the set-point, PID goes to min #31

lukepighetti opened this issue Oct 18, 2022 · 6 comments

Comments

@lukepighetti
Copy link

Temp was 275f, set point was 275f. P is 3 and I is 0.01

Changed set point to 250f. It immediately dropped to the minimum. I expected it to smoothly transition.

Is this a tuning problem or a bug?

Screen Shot 2022-10-18 at 12 57 32 PM

@soloam
Copy link
Owner

soloam commented Oct 18, 2022

What version of the PID are you running? Is it the latest?

@soloam
Copy link
Owner

soloam commented Oct 18, 2022

Normally that bump is because you might have a very high "p". Give me your config, aren't you using the "d"?

@Pukinn
Copy link

Pukinn commented Dec 15, 2022

I have the same issue. But it's not because of a high Kp. It seems the controller resets after changing the set_point. The attributes p, i, d are going to 0. If a high Kp were the reason, I think p should go to a negative value.
Also it goes to min value, if I increase the set_point (invert == false).

I use the latest version 1.1.5

@soloam
Copy link
Owner

soloam commented Dec 15, 2022 via email

@Pukinn
Copy link

Pukinn commented Dec 15, 2022

In which scenario this could be desirable? After a change of the set point I expect a correction because of the derivative part in the next cycle.
But I increase the set_point and the correction goes down to 0 until the next update of the source.

I think the controller should only reset if the reset is manually triggered.

@soloam
Copy link
Owner

soloam commented Dec 15, 2022

I agree with that! The reason to do this is to avoid sudden spikes up in the pid, but this is making the spike down! I'll fix it!

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

No branches or pull requests

3 participants