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

RPM Control PID: Sinlge PWM output for both raise and lower #661

Closed
floriske opened this issue May 28, 2019 · 4 comments
Closed

RPM Control PID: Sinlge PWM output for both raise and lower #661

floriske opened this issue May 28, 2019 · 4 comments

Comments

@floriske
Copy link

Mycodo Issue Report:

  • Specific Mycodo Version: 7.5.3

Problem Description

I have 2 PWM controlled fans of which I'm monitoring- and want to control the RPM.

If I want to do this with a PID controller I have to create 2 PWM outputs in order to be able to do this because the raise and lower outputs aren't allowed to be the same.

Is it possible to make it possbile to use the same output in a future version?

@kizniche
Copy link
Owner

I initially made this sanity check to prevent issues with PIDs, but your use case makes a good argument for removing this and allowing users to make that decision. I'll test removing this check and will see about implementing it in the next release.

@floriske
Copy link
Author

Thank you very much!

@kizniche
Copy link
Owner

v7.5.4 Released. Feel free to reopen this if there are still issues related to this.

@kizniche
Copy link
Owner

Also, for historical reference, this is related to #660

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

2 participants