[Feature Request]
HBDrew opened this issue · 12 comments
Is your feature request related to a problem? Please describe.
Not a problem, just an improvement for cleaning. It would be awesome to have a relay to switch the pump directions for cleaning. This would allow for backing the liquid out of the lines both before and after cleaning.
Describe the solution you'd like
An option to control a single relay to reverse all pumps for cleaning.
Alternative*
I have considered a separate physical switch to perform this and it may be the easiest solution, but certainly isn’t as fun! :-)
Hey, this sounds for a good addition.
To understand the technical needed details: the inversion/switching of the pump direction is just the activation of the relay over an additional GPIO (let it call the inverter pin for now) for the duration of the inversion?
I can imagine one of two flows:
- before cleaning, the inverter relay could be activated, then the pumps would activate a short period, the inversion stops and the regular cleaning begins and after that the inversion once again activates and the remaining fluid is flown in the opposite direction again.
- There is a separate option called inverse cleaning (or similar) the user can click instead of cleaning, this would work like the cleaning but activated the inverter Relais before and deactivates after the process.
Currently leaning towards option 2 since one may want to go a cleaning or inverse step multiple times if needed.
Hey @HBDrew, do you have any setup / software with which you could test the feature?
The current implementation will ask before starting the cleaning (if reversion option is activated) if to revert the cleaning. If prompted with yes, the cleaning will start reverted, otherwise it will start normally. Users not having this option enabled will not be prompted with this question. This was done to not clutter the UI with multiple clean buttons. In theory and code perspective it does work, but I can't validate it against physical hardware, since I have no bidirectional pumps nor a two current way setup. Anyway, I will release this feature with the next update, but you can already test it now if you switch to the "dev" branch after pulling the latest git status with git pull
, if you want to check it out.
Hey, @HBDrew, were you able you test it? I would close this issue, unless you have any issues with the current status, but then you can always reopen the issue as well.
Can you send me the error (or a picture) and the config file with it? Did you set all config over the program (GUI) or did you temper in the config file? Doing over the GUI, usually you won't be able to enter bad values, when entering values via an editor in the config, I can only check them for reason at program start and they may be faulty.
Well, I think I found the Issue. Please update with the terminal (git pull) to the latest (1.27.1) version. This should then work properly (hopefully). If it was the error, I was able to recreate on one of my hardwares.
Well, I think I found the Issue. Please update with the terminal (git pull) to the latest (1.27.1) version. This should then work properly (hopefully). If it was the error, I was able to recreate on one of my hardwares.
@HBDrew Can you give me feedback if this works, when you tested it? :)