Skip to content

Updated HVAC general type FAN_SS_VSC_ZSPM_ZTM and opt uses for telemetry fields DFSS and DFVSC - #1740

Merged
shambersafai merged 5 commits into
google:masterfrom
kevinqnguyen001:1350-fan-onboarding
Sep 16, 2026
Merged

shambersafai merged 5 commits into
google:masterfrom
kevinqnguyen001:1350-fan-onboarding

Conversation

@kevinqnguyen001

Copy link
Copy Markdown
Contributor

Added failed_zone_air_static_pressure_alarm to the telemetry fields folder. New general type FAN_SS_VSC_ZSPM_ZTM. Added failed_zone_air_static_pressure_alarm to ZSPM, local_override_alarm to DFVSC, schedule_run_command to DFSS as opt_uses.

@google-cla

google-cla Bot commented Sep 14, 2026

Copy link
Copy Markdown

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

- failed_zone_air_temperature_alarm
- high_zone_air_temperature_alarm
- low_zone_air_temperature_alarm
- schedule_run_command

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I am not sure that it makes the most sense to add this one here. What is the full canonical type of your device? It might make more sense to add this to one of the other abstracts on it.


FAN_SS_VSC_ZSPM_ZTM:
guid: "d02f772c-7c0a-4221-9ccd-02756c65b158"
description: "Fan with start/stop status, open-loop speed control, zone static pressure monitorinig, and zone temperature monitoring."

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

typo on monitoring

@shambersafai
shambersafai merged commit 77ab184 into google:master Sep 16, 2026
10 of 11 checks passed
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

Successfully merging this pull request may close these issues.

2 participants