-
Notifications
You must be signed in to change notification settings - Fork 80
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
Modeling VRFs and Heat Pumps #590
Merged
Merged
Changes from 7 commits
Commits
Show all changes
34 commits
Select commit
Hold shift + click to select a range
80edee0
Merge pull request #4 from BrickSchema/master
jbkoh 36e645b
Merge pull request #5 from mapped/sync-with-upstream-230928
connorjcantrell dfe0985
Merge remote-tracking branch 'upstream/master'
connorjcantrell 4d42a16
Merge pull request #6 from mapped/sync-with-upstream-231114
connorjcantrell b9cccb8
Merge branch 'master' of https://github.com/BrickSchema/Brick into up…
connorjcantrell f66670a
Merge pull request #7 from mapped/update-master
connorjcantrell a74f788
Add VRF, Heat Pump, and Chiller equipment
connorjcantrell 256aec5
Add VRF System
connorjcantrell b6decd3
Add definitions
connorjcantrell 8fc21e8
Add Condensing Unit definitions
connorjcantrell 6f4cb1b
Add commas
connorjcantrell 7eb9bd6
Add Expansion Valves
connorjcantrell 4c5e4d5
Update Electronic Expansion Valve name
connorjcantrell 2c8a2a2
Move expansion valves to new metering device class
connorjcantrell 1e1fe92
Add refrigeration system
connorjcantrell 3afe698
Add metering device definitions
connorjcantrell 5a10c2c
Add connectedTo relationship
connorjcantrell 8e73f53
Rename metering device to throttling device
connorjcantrell cbcc5d4
Add ground, air, water source condensing and packaged units
connorjcantrell 192c141
Add alias to Duct FCU
connorjcantrell 135c7c2
Add horizontal tag to Duct FCU
connorjcantrell c3d3f22
Rename throttling device to refrigerant metering device
connorjcantrell 56f6ad1
Add capillary tube metering device
connorjcantrell fcfa489
Add VRF, Heat Pump, and Chiller equipment
connorjcantrell 636e03e
Add Expansion Valves
connorjcantrell f313094
Update Electronic Expansion Valve name
connorjcantrell d15d70e
Move expansion valves to new metering device class
connorjcantrell 720b806
Rename metering device to throttling device
connorjcantrell bbe898a
Merge branch 'master' into add-vrf
connorjcantrell c2950c4
Remove comma
connorjcantrell f1a83a6
Fix format
connorjcantrell bc0c789
Wrap definition
connorjcantrell 6a03d2a
Update obsolete terms
connorjcantrell 8510dd9
Remove Packaged GSHP
connorjcantrell File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
@gtfierro @jbkoh How could we model a heat pump which is a split system? These definitions work well for a packaged unit, such as the illustrated "Air_Source_Heat_Pump":
However, a split system would have both a Condensing Unit and an FCU. This, too, is a Heat Pump:
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.
In some sense, a split system is more of a "Heat Pump System", where as a packaged unit is a "Heat Pump Equipment"
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.
We do have "Heat_Pump_Condensing_Unit", perhaps "Heat_Pump" class could be modified to "Heat_Pump_Packaged_Unit" for clarification.
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 think the distinction between a packaged and split system just the location/relationships. Similar to how an RTU is just an AHU that hasLocation roof.
Package System > feeds > Zone
Split System Condenser Unit > feeds > Evaporator_Unit (do we have this?) > feeds > Zone
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.
@MatthewSteen In the split-system example above, this would be achieved by using the Heat_Pump_Condensing_Unit > feeds > FCU
In this case, do you agree that Heat_Pump should be "Packaged Heat Pump"?
https://www.trane.com/residential/en/products/packaged-systems/heat-pump-packaged-systems/
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.
Renamed "Heat Pump" to "Packaged_Heat_Pump" and added air/water/ground source to all condensing unit classes. In doing so, we can represent the same level of detail for both Packaged and Split System variations.
cbcc5d4