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

Various additional value constraints #402

Merged
merged 7 commits into from
Apr 18, 2024
Merged

Conversation

shorowit
Copy link
Contributor

@shorowit shorowit commented Feb 7, 2024

Closes #385.

New constraints:

  • AirInfiltrationMeasurement/EffectiveLeakageArea > 0 (Note: BuildingAirLeakage is already > 0)
  • AirInfiltrationMeasurement/InfiltrationVolume > 0
  • AirInfiltrationMeasurement/InfiltrationHeight > 0
  • DuctLeakage/Value >= 0
  • GeothermalLoop/LoopFlow > 0
  • GeothermalLoop/BoreholesOrTrenches/Length > 0
  • GeothermalLoop/BoreholesOrTrenches/Spacing > 0
  • GeothermalLoop/BoreholesOrTrenches/Diameter > 0
  • GeothermalLoop/Grout/Conductivity > 0
  • GeothermalLoop/Pipe/Conductivity > 0
  • GeothermalLoop/Pipe/Diameter > 0
  • GeothermalLoop/Pipe/ShankSpacing > 0
  • Soil/Conductivity > 0
  • CoolingSystem/SensibleHeatFraction > 0.5
  • HeatPump/CoolingSensibleHeatFraction > 0.5

@shorowit shorowit self-assigned this Feb 7, 2024
@shorowit shorowit requested a review from nmerket February 7, 2024 17:26
@shorowit shorowit added this to the v4.0-rc4 milestone Feb 20, 2024
@shorowit shorowit added the merge label Mar 14, 2024
@nmerket
Copy link
Contributor

nmerket commented Mar 20, 2024

@nmerket nmerket merged commit fdce274 into master Apr 18, 2024
2 checks passed
@nmerket nmerket deleted the additional_value_constraints branch April 18, 2024 15:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add more value constraints
2 participants