-
Notifications
You must be signed in to change notification settings - Fork 137
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
added new sensor type #997
Changes from 2 commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -6618,3 +6618,18 @@ VMADC: | |
- mixed_air_damper_percentage_sensor | ||
implements: | ||
- CONTROL | ||
|
||
MAZTM: | ||
description: "Zone temperature monitoring for MDF room with high & low temperature alarms, sensor failed alarm " | ||
is_abstract: true | ||
uses: | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Make alarms opt_uses |
||
- high_limit_zone_air_temperature_setpoint | ||
- low_limit_zone_air_temperature_setpoint | ||
- zone_air_temperature_sensor | ||
- high_zone_air_temperature_alarm | ||
- low_zone_air_temperature_alarm | ||
- failed_zone_air_temperature_alarm | ||
- failed_alarm | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Don't need failed_alarm here. These should typically go on general types or on |
||
implements: | ||
- OPERATIONAL | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -243,3 +243,10 @@ SENSOR_CO2M_VOCM: | |
- /SENSOR | ||
- CO2M | ||
- VOCM | ||
|
||
SENSOR_MAZTM: | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. It is not making sense to me to have an abstract type that contains setpoints on a sensor entity. This seems like just a zone temperature sensor to me, and the different setpoints should go on the AHU or whatever this sensor applies to. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Okay yeah this probably makes more sense |
||
description: "zone temperature monitoring with high, low alarm setpoints and unit failed alarms" | ||
is_canonical: true | ||
implements: | ||
- /SENSOR | ||
- MAZTM |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What does the MA stand for? Should be ZTC rather than ZTM since you have setpoints (controlling rather than monitoring).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I didnt think it was quite controlling though, the setpoints are to set the temperature at which the alarm is triggered?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If you are including the alarm points themselves there is no need to include the thresholds at which the alarms are triggered.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would suggest using the existing
ZTM
type if possible, there is nothing special about zone temp monitoring in an MDF room that would require a new type.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
okay closing, thank you