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

Additional water heater elements #421

Draft
wants to merge 5 commits into
base: master
Choose a base branch
from
Draft

Conversation

shorowit
Copy link
Contributor

Adds SealedCombustion, CondensingSystem, AtmosphericBurner, and PowerBurner elements for water heaters; these elements already exist for furnaces/boilers.

… `PowerBurner` elements for water heaters; same elements as exist for furnaces/boilers.
@shorowit shorowit added this to the v4.1 milestone Jul 15, 2024
@shorowit shorowit self-assigned this Jul 15, 2024
@shorowit
Copy link
Contributor Author

@jmaguire1 Can you look these over and provide feedback? I'm not sure if we should use, e.g., "DirectVent" or "PowerVent" instead.

@jmaguire1
Copy link

Yes, I never heard from anyone at GTI on this. It is a little confusing, but I think the way we'll want to do this:

CondensingSystem: If a tank WH, TRUE if UEF > 0.77 (or corresponding EF). If tankless, TRUE if EF (or UEF) > 0.94. Else False.

AtmosphericBurner: If tank WH, UEF < 0.64. FALSE for any tankless.
PowerBurner: The opposite of AtmosphericBurner, so TRUE if UEF >= 0.64 and for any tankless.
SealedCombustion: Same as "PowerBurner"

to rewrite the code for skinlossfrac here, I think this makes it:

if "AtmosphericBurner":
skinlossfrac = 0.63
if "Condensing":
skinlossfrac = 0.96
else:
skinlossfrac = 0.91.

What really matters here for whether you fall in the 0.91 category is if there's a flue damper, which I think is true pretty much any time "AtmosphericBurner" is false. Condensing vent a little colder, which is how we ended up with a slightly higher skinlossfrac here.

@shorowit
Copy link
Contributor Author

shorowit commented Aug 8, 2024

Two HPXML WG members lean towards switching to "vented" terminology.

@shorowit
Copy link
Contributor Author

shorowit commented Aug 8, 2024

A good resource from AO Smith: https://www.hotwater.com/info-center/water-heater-venting.html

@shorowit shorowit marked this pull request as draft September 10, 2024 21:44
@shorowit shorowit modified the milestones: v4.1, v5.0 Sep 23, 2024
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.

2 participants