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

kiva autocalculated deep ground depth is negative #7200

Closed
3 tasks
mldichter opened this issue Mar 5, 2019 · 1 comment · Fixed by #9328
Closed
3 tasks

kiva autocalculated deep ground depth is negative #7200

mldichter opened this issue Mar 5, 2019 · 1 comment · Fixed by #9328
Assignees
Labels
Defect Includes code to repair a defect in EnergyPlus NotIDDChange Code does not impact IDD (can be merged after IO freeze)

Comments

@mldichter
Copy link

Issue overview

I took the example files for the kiva objects, ZoneCoupledKivaSlab.idf and ZoneCoupledKivaBasement.idf, and changed the Site:Location, SizingPeriod:DesignDay objects, and weather file for each of the CA climate zones. There are some strange errors for the "autocalculated deep ground depth" in some of the simulations. Obviously, something is wrong. Here is a google drive link to the files.

ZoneCoupledKivaSlab.idf

Foundation:Kiva="SLAB FOUNDATION", the autocalculated deep ground depth (1.328 m) is shallower than foundation construction elements (1.500 m)

Foundation:Kiva="SLAB FOUNDATION", the autocalculated deep ground depth (-0.919 m) is shallower than foundation construction elements (1.500 m)

ZoneCoupledKivaBasement.idf

Foundation:Kiva="BASEMENT FOUNDATION", the autocalculated deep ground depth (1.328 m) is shallower than foundation construction elements (3.425 m)

Foundation:Kiva="BASEMENT FOUNDATION", the autocalculated deep ground depth (3.883 m) is shallower than foundation construction elements (3.425 m)

Foundation:Kiva="BASEMENT FOUNDATION", the autocalculated deep ground depth (-0.919 m) is shallower than foundation construction elements (3.425 m)

Details

Some additional details for this issue (if relevant):

Checklist

Add to this list or remove from it as applicable. This is a simple templated set of guidelines.

  • Defect file added (list location of defect file here)
  • Ticket added to Pivotal for defect (development team task)
  • Pull request created (the pull request will have additional tasks related to reviewing changes that fix this defect)
@nealkruis
Copy link
Member

This is intended behavior. See my response on Unmet Hours.

The warning message should be changed, and we should add a warning if both autocalculate and a deep ground temperature are selected (see comments in Unmet Hours).

@matthew-larson matthew-larson self-assigned this Feb 21, 2022
@matthew-larson matthew-larson added Defect Includes code to repair a defect in EnergyPlus NotIDDChange Code does not impact IDD (can be merged after IO freeze) labels Mar 2, 2022
@matthew-larson matthew-larson added this to the EnergyPlus 22.1 milestone Mar 2, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Defect Includes code to repair a defect in EnergyPlus NotIDDChange Code does not impact IDD (can be merged after IO freeze)
Projects
None yet
3 participants