Skip to content

Clarify DefrostEIRTempModFac is an EIR, not an EIR modifier#11425

Merged
mitchute merged 1 commit intodevelopfrom
11380_defrost_doc
Feb 23, 2026
Merged

Clarify DefrostEIRTempModFac is an EIR, not an EIR modifier#11425
mitchute merged 1 commit intodevelopfrom
11380_defrost_doc

Conversation

@brianlball
Copy link
Contributor

Fixes #11380

Clarify in the Engineering Reference that DefrostEIRTempModFac represents a defrost energy input ratio (defrost power
/ rated capacity), not a modifier applied to a rated EIR. Other EIR modifier curves (e.g. EIRTempModFac) multiply
against rated EIR, but the defrost curve multiplies against rated capacity directly. The terminology was inconsistent
and confusing. Four locations updated in coils.tex:

  • Single-speed DX heating coil curve description
  • Single-speed defrost operation variable definition
  • Multi-speed defrost operation variable definition
  • Variable-speed defrost operation description

No code changes — documentation only.

Fixes #11380

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@brianlball
Copy link
Contributor Author

@JanghyunJK, @rraustad thoughts on this change?

@brianlball brianlball added the Documentation Related primarily on the LaTeX-based EnergyPlus documentation label Feb 19, 2026
Copy link
Collaborator

@rraustad rraustad left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good doc update.

@mitchute mitchute added the Defect Includes code to repair a defect in EnergyPlus label Feb 23, 2026
@mitchute mitchute merged commit 5e71a94 into develop Feb 23, 2026
11 of 13 checks passed
@mitchute mitchute deleted the 11380_defrost_doc branch February 23, 2026 22:41
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 Documentation Related primarily on the LaTeX-based EnergyPlus documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

"Defrost Operation" documentation

4 participants