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

Cooler & Filter adaptive pressure adjustments are missing thier controls Any plans for these? #23

Closed
friskens opened this issue Nov 30, 2023 · 9 comments

Comments

@friskens
Copy link

Is your feature request related to a problem? Please describe.
No and yes, seems they are just an oversight on the developers side.

Describe the solution you'd like
A button and the associated alarms for the cooler accessory and for the adaptive pressure measurment device for filter changes.

Describe alternatives you've considered
Doing it myself, but serious lack of time due to too much HA work already.

Additional context
I have a Heru 160 T with cooler and also the filter degradation pressure detection device, due to cats, lots of cats.

@magicus
Copy link
Contributor

magicus commented Nov 30, 2023

Do you know the modbus addresses of these controls?

@friskens
Copy link
Author

alarms:
1x00032 Cooling alarm
Settings:
4x00068 Cooler type 0 1 0 = None, 1 = Water
4x00069 Cooler enabled 0 1 0 = no, 1 = yes

Then it's this mess for the filter and it seems also alot of other pressure sensors...
image

@magicus
Copy link
Contributor

magicus commented Dec 1, 2023

Sensor type 1 = 0-10V, 2 = Modbus sounds a bit scary. I'm just thinking that maybe your Heru is setup to have the sensor report as a voltage change instead, and what happens then if you switch to modbus? Frankly, I am personally quite wary of meddling with too much of the settings I don't understand.

Adding an alarm is probably simple and benign. Adding sensors can't really harm, but the pressure range integer might need some conversation, which is slightly non-trivial. But adding controls sounds scary...

@toringer
Copy link
Owner

toringer commented Dec 1, 2023

In general, I do not want to implement settings that on the IQC display has the warning "Warning. Only for qualified technicians".

I would be fine implementing these
alarms:

  • 1x00032 Cooling alarm

Settings:

  • 4x00068 Cooler type 0 1 0 = None, 1 = Water
  • 4x00069 Cooler enabled 0 1 0 = no, 1 = yes

For the filter settings, I am unsure.

Edit: In fact, the "Cooler type" setting would require a Select entity type. Currently this entity type is not use in the component, and therefore requires more work. I most likely will not prioritise this.

@magicus
Copy link
Contributor

magicus commented Dec 1, 2023

In general, I do not want to implement settings that on the IQC display has the warning "Warning. Only for qualified technicians".

Sounds like a good approach!

@friskens
Copy link
Author

Alright I ofc respect your hesitation of implementing the filter sensor, note however that those settings is either for a Modbus attached sensor indentifying as such or a specific other physical connection that measures 0-10V between two connectors, can't really mess any hardware up only make the unit act strange I guess... Much grateful for your cooler on off implementation looking forward too it this summer! Right now i have to set it to run manually...

@friskens friskens reopened this Dec 15, 2023
@friskens
Copy link
Author

Accidentally closed sigh

@friskens
Copy link
Author

@toringer so could at least cooler on/off and the alarm be sorted?

@toringer
Copy link
Owner

Switch and alarm added in v1.0.5

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