-
Notifications
You must be signed in to change notification settings - Fork 0
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
use internal python api instead of requests #1
Comments
This is good. This plugin will help me a lot |
@dimontau did you test this already? in my dev environment the api request (as done in cbpi3) doesnt work and i am not sure why. could be the dev enironemnt, could be a missing authentification, could be anything. If you could test the current implementation on your real Pi that would be great, if not no problem. what do you use the GPIO buttons for? Because when i change the logic to use internal api calls i need to implement each possible button function individually and make them selectable, so i dont want to miss your use case. |
I haven't tested yet. I can try at the end of the week. I want to use to manually turn actors on and off. Buttons without fixation, that is, one press turns it on, and the next one turns it off. and I want to connect buttons via PCF8574 |
I mainly deal with whiskey grain mashes. Could you help me with the fermentation plugin. I miss the periodic inclusion of mixing. How it's done in cbpi4-BM-PID-SmartBoilWithPump. So that Agitator turns on after a period for a while. And then I have to do it manually. |
for periodic mixing you just make a PWM actor, set the frequency to something small and then set the power to the amount of time you want your actor to be on in percent power (not 100% power) for example a PWMActor with frequency 0.1 will toggle 2 times in 10 seconds. If you put the power to 60% it will turn off after 6 seconds and then 4 seconds after that it will turn on again and repeats. This is how i did it in cbpi3 as well btw. |
the api should be called python internally and not via http request. I will start working on this at the end of this week.
The text was updated successfully, but these errors were encountered: