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

Independently allow cubic spline interpolation option in PerformancePrecisionTradeoffs #9107

Open
3 tasks
shorowit opened this issue Sep 29, 2021 · 1 comment
Open
3 tasks
Labels
NewFeatureRequest This "issue" is a new feature request, not a defect report

Comments

@shorowit
Copy link
Contributor

Issue overview

E+ 9.6 offers a new cubic spline interpolation option for PerformancePrecisionTradeoffs. In trying to use it, I discovered that it can only be accessed by selecting Mode06 or higher. It'd be nice if this input could be independently offered so that you aren't required to use everything else included in Mode06. As far as I'm aware, every other option in this object can be independently selected so that a user has more flexibility than the preset modes.

Details

Some additional details for this issue (if relevant):

  • Platform (Operating system, version)
  • Version of EnergyPlus (if using an intermediate build, include SHA)
  • Unmethours link or helpdesk ticket number

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)
@JasonGlazer JasonGlazer added the NewFeatureRequest This "issue" is a new feature request, not a defect report label Mar 16, 2022
@JasonGlazer JasonGlazer changed the title Independently allow cubic spline interpolation option Independently allow cubic spline interpolation option in PerformancePrecisionTradeoffs Mar 16, 2022
@mjwitte
Copy link
Contributor

mjwitte commented Mar 16, 2022

At the same time, it would be nice to use a consistent format for the warnings from PerformancePrecisiontTradeoffs. Here's an example of some of the current warnings:

    ** Warning ** Due to PerformancePrecisionTradeoffs Override Mode, internal variable MaxZoneTempDiff will be set to 1.0 .
    ** Warning ** Due to PerformancePrecisionTradeoffs Override Mode, internal variable MaxAllowedDelTemp will be set to 0.1 .
    ** Warning ** PerformancePrecisionTradeoffs: Coil Direct Solution simulation is selected.
    ** Warning ** Due to the use of the fast simulation mode, the time step for simulation and averaging window of sizing is overwritten to one hour. Original user inputs for averaging window and timestep are no longer used.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
NewFeatureRequest This "issue" is a new feature request, not a defect report
Projects
None yet
Development

No branches or pull requests

3 participants