NatLabRockies / NatLabRockies/EnergyPlus

Consolidate soil/ground properties into a single object

Open
#9,701 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

NewFeatureRequest
Dominant language
C++
Stars
1.6k
Forks
490
Avg merge
6d 21h
Merged PRs (30d)
22

Description

Issue overview

Objects that ask for soil/ground properties (e.g., conductivity, specific heat, etc.) include:

  • Site:GroundTemperature:Undisturbed:FiniteDifference
  • Site:GroundTemperature:Undisturbed:KusudaAchenbach
  • Site:GroundTemperature:Undisturbed:Xing
  • GroundHeatExchanger:System
  • Foundation:Kiva:Settings
  • PipingSystem:Underground:Domain

... and maybe others.

A new object (Site:GroundSoilProperties?) where these properties can be defined once would be nice.

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)

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by reviewing the listed ground-temperature, heat-exchanger, foundation, and piping-system objects and comparing the soil or ground properties they currently request. Determine whether other objects also duplicate these properties and define the scope of a shared object such as Site:GroundSoilProperties. Done means the agreed properties can be defined once and used consistently by the applicable objects.

Written by the indexing model from the issue text.

Assessment

Tech stack
cpp
Domain
backend
Issue type
Feature
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
20/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.