search

joesinger12 / TestTicketTransfer

0 stars 0 forks source link

VRF curve limit issue #86

Open joesinger12 opened 4 years ago

joesinger12 commented 4 years ago

While debugging a model for another project, I found that the VRF curves "CoilClgVRFClgQratio_fTwbTdbSI" and "CoilHtgVRFHtgQratio_fTwbTdbSI" have incorrect Max/MinOut values. This stems from the Curve workbook, so need to fix it there as well.

Coincidently, this issue was discovered by a user right after I found it: https://unmethours.com/question/43220/discrepancy-in-values-in-running-both-conventional-and-vrf-systems-in-2-different-locations/

Reported by: joesinger12

Original Ticket: cbecc-com/tickets/3193

joesinger12 commented 4 years ago

Original comment by: joesinger12

joesinger12 commented 4 years ago

Temporay fix by simply removing curve limits. For 2.0 final release, will review if there needs to be limits for these curves, and also commit updated curve workbook.

Revision: 6264 Author: davidreddy Date: Monday, June 29, 2020 9:33:33 PM Message: Ticket 3193: Remove curve limits for CoilClgVRFClgQratio_fTwbTdbSI and CoilHtgVRFHtgQratio_fTwbTdbSI, these were not correct

Modified : /branches/CBECC-Com_22-0-1_CECTest/RulesetDev/Rulesets/T24N/Rules/Library/Library_PerformanceCurves-T24N.rule

Original comment by: joesinger12

joesinger12 commented 4 years ago

Original comment by: joesinger12

joesinger12 commented 3 years ago

Original comment by: joesinger12

joesinger12 commented 3 years ago

Original comment by: joesinger12

joesinger12 commented 3 years ago

Original comment by: joesinger12

joesinger12 commented 3 years ago

Original comment by: joesinger12

joesinger12 commented 3 years ago

Original comment by: joesinger12

joesinger12 commented 3 years ago

Original comment by: joesinger12

joesinger12 commented 1 year ago

Original comment by: joesinger12