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

Duct cost multiplier - unconditioned area #634

Merged
merged 4 commits into from
Jul 8, 2021
Merged

Conversation

shorowit
Copy link
Contributor

@shorowit shorowit commented Jul 7, 2021

Resolves #[issue number here].

Pull Request Description

Converts "Duct Surface Area (ft^2)" cost multiplier to "Duct Unconditioned Surface Area (ft^2)". Provides consistency w/ ResStock-HPXML. Follow-up to #532.

Only buildings where the primary duct location is living space or finished basement are affected; they now return zero (instead of non-zero) values for the cost multiplier.

Checklist

Not all may apply:

  • Unit tests have been added or updated
  • All rake tasks have been run, and pass
  • Documentation has been modified appropriately
  • Any new options are added to project_testing
  • project_testing runs without any failures
  • No unexpected regression test changes
  • All tests are passing
  • The changelog has been updated appropriately
  • This branch is up-to-date with develop

For more information on how to perform these checklist items, see the documentation's Advanced Tutorial.

@shorowit shorowit self-assigned this Jul 7, 2021
@shorowit shorowit changed the title Use only unconditioned area in duct cost multiplier Duct cost multiplier - unconditioned area Jul 7, 2021
@shorowit
Copy link
Contributor Author

shorowit commented Jul 7, 2021

Verified that results perfectly match ResStock-HPXML for 30 project_national samples.

@shorowit shorowit marked this pull request as ready for review July 7, 2021 18:55
Copy link
Contributor

@joseph-robertson joseph-robertson left a comment

Choose a reason for hiding this comment

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

Seems to zero out the duct cost multiplier for some national project samples.

@shorowit shorowit requested a review from ejhw July 7, 2021 21:12
@shorowit shorowit merged commit d5cd76f into develop Jul 8, 2021
@shorowit shorowit deleted the duct_surface_area branch July 8, 2021 15:39
@lixiliu
Copy link
Contributor

lixiliu commented Jun 28, 2023

@shorowit - if you end up sealing ducts in conditioned space, would you expect there to be substantial energy saving?

I found an EUSS building (372868) in MN that has an envelop upgrade containing only duct sealing: 30% leakage, R-4. -> 10% leakage, R-8. The result saving is 1924 kWh, which is surprisingly high to me. For context, the heating system is ASHP, SEER 15, 8.5 HSPF of 12.61 kbtu/h cap, 6 deg F offset for heating at 68 deg F setpoint, -3h day & night.

But because of the ducts are in conditioned space, "Duct Unconditioned Surface Area (ft^2)"=0, leading to $0 upgrade cost. I understand the motivation for not air-sealing the ducts in conditioned space (and we could limit the apply_logic for ducts going forward), but am quite surprised by the energy saving.

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.

None yet

4 participants