-
Notifications
You must be signed in to change notification settings - Fork 195
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
Addresses #4362, add support of HeatPump:PlantLoop:EIR:Heating to model air-to-water heat pump #4368
Merged
Merged
Addresses #4362, add support of HeatPump:PlantLoop:EIR:Heating to model air-to-water heat pump #4368
Changes from all commits
Commits
Show all changes
48 commits
Select commit
Hold shift + click to select a range
58f3c74
Update idd.
joseph-robertson 9e164fc
Stub model files.
joseph-robertson a5bcaac
Stub ft files.
joseph-robertson c289db7
Update other model files.
joseph-robertson e7ddb21
Update other ft files.
joseph-robertson a6a5978
Apply clang format.
joseph-robertson 1822217
Fill out model and ft files.
joseph-robertson c9bedf9
Minor idd updates.
joseph-robertson 1de1171
Missing metods.
joseph-robertson 92718da
Fill out model tests for hp heating.
joseph-robertson 95e1961
Apply clang format.
joseph-robertson 2b30cbe
Fix hp heating model test file.
joseph-robertson dd37073
Fill out hp cooling model tests.
joseph-robertson 9695a50
Missing object types in idd.
joseph-robertson 0cab715
Fix object lists in proposed idd.
joseph-robertson 902f24f
Update var names in ft test.
joseph-robertson b88c745
Fill out curve fields in ctors.
joseph-robertson 7257aaa
Typos in proposed idd curve objects.
joseph-robertson af9e61f
Merge branch 'develop' into issue-4362
jmarrec f92dfa3
Remove coments that only existed in ProposedE+.idd and aren't true here
jmarrec 1b30d9a
Switch model load and source.
joseph-robertson 2d93697
Adjust curves in model files.
joseph-robertson 5f3c3c0
Add valid condenser type values.
joseph-robertson 09cf5c8
Update model tests.
joseph-robertson 809d58f
Improve ft test.
joseph-robertson aa09bae
Clean up types.
joseph-robertson 29fda9f
Typo in heating cpp.
joseph-robertson 0f97e07
Fix typos in tests.
joseph-robertson e74e228
Include model in test files.
joseph-robertson 4d5fbf1
Clean up ft test.
joseph-robertson f88cb0c
Use curve concrete model objects for model tests.
joseph-robertson 894ffcc
Merge branch 'develop' into issue-4362
joseph-robertson c7d2885
Add ft test for plant eq op schemes.
joseph-robertson 4d46d02
Initialize a var.
joseph-robertson fae6167
Update ft for water vs air source nodes.
joseph-robertson c5747e6
Update ft test for node name changes.
joseph-robertson 8a5d7bc
Fix typos.
joseph-robertson 8e22d2e
Fix formatting in ft files.
joseph-robertson 5a27cd1
More formatting in ft files.
joseph-robertson e5ee8a2
Fix expected node names in ft test.
joseph-robertson ea37e88
Merge remote-tracking branch 'origin/develop' into issue-4362
jmarrec b929379
Enforce condenserType in addToNode / removeFromSecondaryPlantLoop / s…
jmarrec 7eb1714
Add More convenience functions to get the nodes (it's confusing other…
jmarrec 12aac9c
Add a model GTest for enforcing condenserType + convenience functions…
jmarrec 9e499d1
Adjust FT test
jmarrec 3d1f597
Name getters/setters like the IDD field, like GenerateClass.rb would …
jmarrec 2e98e01
Merge branch 'develop' into issue-4362
joseph-robertson 2f7221d
Merge branch 'develop' into issue-4362
joseph-robertson 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
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
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.
The same should be applied to the actual E+ idd. Did you do a PR to NREL/EnergyPlus or opened an issue?
Anyways, good catch 👍
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 did not open an issue or PR in NREL/EnergyPlus.
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.
Filled a PR at NREL/EnergyPlus#8982