added new ahu type #1023
added new ahu type #1023
Conversation
| - failed_circulation_pump_alarm_1 | ||
| - failed_circulation_pump_alarm_2 | ||
| - circulation_pump_speed_percentage_setpoint | ||
| - low_supply_water_temperature_alarm |
There was a problem hiding this comment.
Not sure that the temperature point is relevant for the pump's speed control.
There was a problem hiding this comment.
Add more details to the descriptions so that they are different from each other and more specific. A device can control based on outdoor temperature and not use this summer/winter control schema. Make it clear that the device has different setpoints based on season.
| flexible_min: 255.9259259 | ||
| flexible_max: 308.1481481 | ||
|
|
||
| - season_mode: |
There was a problem hiding this comment.
I would change this to season_command as it has only two states, or season_status if this is a feedback point.
| @@ -3110,3 +3110,23 @@ literals: | |||
| - ultraviolet_irradiance_sensor: | |||
| fixed_min: 0.0 | |||
| flexible_max: 2000.0 | |||
|
|
|||
| - winter_outside_temperature_setpoint: | |||
There was a problem hiding this comment.
How can we have an outside air temperature setpoint? Not following this one.
| flexible_min: 255.9259259 | ||
| flexible_max: 322.037037 | ||
|
|
||
| - summer_outside_temperature_setpoint: |
| @@ -6618,3 +6623,69 @@ VMADC: | |||
| - mixed_air_damper_percentage_sensor | |||
| implements: | |||
| - CONTROL | |||
|
|
|||
| SEDPM: | |||
There was a problem hiding this comment.
What is the use case for these on the unit you're looking at? l would not think of these as required points but perhaps they could be added as optional on existing abstract types.
| implements: | ||
| - CONTROL | ||
|
|
||
| SEMC: |
There was a problem hiding this comment.
See comments in the fields section.
| implements: | ||
| - CONTROL | ||
|
|
||
| HWVPM: |
There was a problem hiding this comment.
If it has a command then it should have a sensor. See if you can use HWVM and flag your unit as missing the sensor if you don't see it.
| implements: | ||
| - MONITORING | ||
|
|
||
| HRCM: |
There was a problem hiding this comment.
The sensor is not very useful on it own. Do you have any of the other points in EHHRC?
| implements: | ||
| - MONITORING | ||
|
|
||
| WDPCA2X: |
There was a problem hiding this comment.
Can you provide a screenshot of where you are seeing this to digitalbuildings-support@googlegroups.com if possible?
No description provided.