-
Notifications
You must be signed in to change notification settings - Fork 26
Replace Hard-Coded Sensible Capacity Curves with ADP/BF #1707
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
Merged
Merged
Changes from all commits
Commits
Show all changes
70 commits
Select commit
Hold shift + click to select a range
698aaf4
Committing pseudo_code_heat_pump_sizing.txt to branch OS_ACCA_Man_S_u…
Sashadf1 1ac11a2
adding comment on apply_equipment_adjustments
Sashadf1 82e23b6
Merge remote-tracking branch 'origin/master' into acca_manual_s
Sashadf1 af484e8
adding explanatory comments
Sashadf1 49e998d
hvac_sizing.rb
Sashadf1 2aca92f
Committing comments for ADP/BF
Sashadf1 6861e46
Merge branch 'master' into adp_bf_method_design_shr
Sashadf1 af58345
Translating E+ coil bypass factor method to recalculate cool_cap_design
Sashadf1 c1a7827
calculating A_o_rated with correct unit conversions and methods in ps…
Sashadf1 2cdde89
renamed A_o from A_o rated because A_o is not a function of design vs…
Sashadf1 1e3ef9a
refactor add_cooling_system function variable input order convention …
Sashadf1 7905315
fixed typo in comments of CoilAoFactor method in psychrometrics.rb
Sashadf1 f9ed1b8
fixed typo in descriptive comments of psychrometrics.rb
Sashadf1 74bb5fa
removing pesky return 1.0 that shouldn't be there, preventing it from…
Sashadf1 0391906
Merge branch 'master' of https://github.com/NREL/OpenStudio-HPXML int…
yzhou601 85b2e27
work on implementing ADP/BF. coil a_o factor already calculated in hv…
Sashadf1 d37f8c1
using pre-existing coil ao factor stored in cooling additional proper…
Sashadf1 26a55b3
fixed unit conversion on cool_cap_rated so that Psychrometrics.rb doe…
Sashadf1 c721a01
iteration for ADP/BF
Sashadf1 f0d925c
more comments
Sashadf1 b177cb9
more work on iteration for design airflow
Sashadf1 1ca7c4a
finished writing code for ADP/BF implementation. ADP/BF method is a b…
Sashadf1 3f8f26d
confident in ADP/BF
Sashadf1 308c1f7
Fix error and minor refactoring.
shorowit 8f4b83d
Merge branch 'master' of https://github.com/NREL/OpenStudio-HPXML int…
shorowit a887009
Merge branch 'master' of https://github.com/NREL/OpenStudio-HPXML int…
shorowit 400a43f
Temporarily disable test to get CI results.
shorowit 315916f
Latest results.
487c702
defined test_calculate_SHR() in hvac_sizing_test.rb
Sashadf1 38cb971
added files after git restore HPXMLtoOpenStudio to recover HPXMLtoOpe…
Sashadf1 f660286
test_error.xml running without failure. haven't inspected annual or t…
Sashadf1 4d5ceca
remove unit test for Psychrometrics.CalculateSHR(), since test_error.…
Sashadf1 83d3094
hvac_sizing.rb, measure.xml, and docs updates after running RuboCop
Sashadf1 f598241
correct hvac_sizing and removing duplicated HPXMLtoOpenStudio directo…
Sashadf1 0b0cc26
Merge branch 'master' into adp_bf_method_design_shr
joseph-robertson 49fd105
Re-enable test, remove files.
shorowit 1edc822
Latest results.
14f36eb
removing puts statements before PR review
Sashadf1 2de64ea
last explanatory comment before marking adp_bf_method PR for review
Sashadf1 ba79c8a
adp_bf_method branch up to date w/ master, merge conflicts resolved m…
Sashadf1 599ec24
resolved merge conflicts in measure.xml
Sashadf1 776661a
manually resolved merge conflicts in results.csv
Sashadf1 d373f6e
Latest results.
c07cb17
refactor to rename hvac_sizings.Design_SHR object property to local v…
Sashadf1 547cf7f
removing :Design_SHR property from HVACSizingValues class definition,…
Sashadf1 1d3a43a
changing comments on get_nominal_speed method to match remote master
Sashadf1 2373097
updating comments on HVACSizingValues attributes to match master. No …
Sashadf1 8b94164
successful? merge of latest master into adp_bf_method_design_shr to u…
Sashadf1 a7d7e41
Merge branch 'master' of https://github.com/NREL/OpenStudio-HPXML int…
shorowit 50bb972
Latest results.
ce0994b
/ \ line extension + divisor in hvac_sizing.rb
Sashadf1 18f106b
descriptive comments on get_hvac_size_limits
Sashadf1 0a7b1f3
fixes to get_hvac_size_limits to ensure adp_bf_branch runs without error
Sashadf1 49372e7
Merge branch 'master' of https://github.com/NREL/OpenStudio-HPXML int…
shorowit 18cf413
Latest results.
178f5da
addressing first Jon W. comment re adp_bf review
Sashadf1 0692d10
concluded merge from diverging local and origin adp_bf_branches
Sashadf1 cb5e1d1
adding informative messages to design airflow calculation for central…
Sashadf1 40f3831
adding comments re a_o_rated to address Jon Winkler review comment 2
Sashadf1 9c683e7
running Rubocop
Sashadf1 5df10eb
pulling in master after addressing adp_bf review comments
Sashadf1 1c418dc
succesfully ran Rubocop
Sashadf1 a71a116
accepting incoming master to get adp branch up to date
Sashadf1 4779ada
changed variable name from hvac_cooling_shr to hvac_cooling_shr_rated…
Sashadf1 12fee34
measure.xml updated after running RuboCop. no offenses detected, meas…
Sashadf1 c6435fa
Merge branch 'master' of https://github.com/NREL/OpenStudio-HPXML int…
shorowit 50c9c7d
Unrelated code cleanup
shorowit 56a8713
Latest results.
5efcc61
Merge branch 'master' of https://github.com/NREL/OpenStudio-HPXML int…
shorowit d315784
Cleanup comments.
shorowit 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 hidden or 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 hidden or 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 hidden or 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 hidden or 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 hidden or 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.
Uh oh!
There was an error while loading. Please reload this page.