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

added air source heat pump #1002

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

kevin-hereworks
Copy link
Contributor

No description provided.

@kevin-hereworks kevin-hereworks marked this pull request as ready for review April 4, 2023 14:38
@mschulze17 mschulze17 added the P2 Medium priority (default) label Apr 5, 2023
- CONTROL

CPM2X:
Copy link
Collaborator

Choose a reason for hiding this comment

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

I would expect these to have commands with them. The statuses are not very helpful on their own.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I believe these pumps are cycled automatically, for example: pump A used for 1 day, B used the next day etc

@@ -162,3 +162,16 @@ PMP_SS_VSC_NONCANONICAL_2:
- mixing_valve_percentage_command
- chilled_supply_water_temperature_setpoint
- chilled_supply_water_temperature_sensor


PMP_SS_REFPM_REFSM_OA_CPM2X:
Copy link
Collaborator

Choose a reason for hiding this comment

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

See CPM2X comment above.

@cstirdivant cstirdivant removed the request for review from tasodorff June 14, 2023 23:29
@@ -162,3 +162,16 @@ PMP_SS_VSC_NONCANONICAL_2:
- mixing_valve_percentage_command
- chilled_supply_water_temperature_setpoint
- chilled_supply_water_temperature_sensor


PMP_SS_REFPM_REFSM_OA_CPM2X:
Copy link
Collaborator

Choose a reason for hiding this comment

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

If this is an air-source heat pump it does not belong in the PMP file, this would be a CH or perhaps a new general type altogether.

@@ -566,6 +566,17 @@ CH_SS_SCHWTC_CDWFRM_CHWSWISOVM_CWSISOVM_CHWDPM_CHWFRM_CWDT_REFPM_IGM_DXSWC:
- IGM
- DXSWC

CH_SS_REFPM_REFSM_OA_CPM2X:
Copy link
Collaborator

Choose a reason for hiding this comment

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

  1. Is this heat pump air or water side (is the refrigerant cooling/heating water or air?)
  2. There is no temperature control on this type; what does the heat pump control to? I would expect to see some supply or return water temperature control.
  3. You've noted that there are two evaporator pumps, I am unfamilar with this type of configuration

Can you please email a diagram (schematic or BMS screenshot) to digitalbuildings-support@googlegroups.com ?

@cstirdivant cstirdivant added the ontology model Extensions or edits to the ontology model label Jan 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ontology model Extensions or edits to the ontology model P2 Medium priority (default)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants