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

Use E+ 22.2 Incident solar multiplier #1175

Merged
merged 33 commits into from Oct 18, 2023
Merged

Conversation

shorowit
Copy link
Contributor

@shorowit shorowit commented Sep 12, 2022

Pull Request Description

Closes #1051.

Checklist

PR Author: Check these when they're done. Not all may apply. strikethrough and check any that do not apply.

PR Reviewer: Verify each has been completed.

  • Schematron validator (EPvalidator.xml) has been updated
  • Sample files have been added/updated (via tasks.rb)
  • Unit tests have been added/updated (e.g., HPXMLtoOpenStudio/tests)
  • Documentation has been updated
  • Changelog has been updated
  • openstudio tasks.rb update_measures has been run
  • No unexpected changes to simulation results of sample files

@shorowit shorowit self-assigned this Sep 12, 2022
Base automatically changed from kiva-error-bugfix to master September 13, 2022 00:15
@shorowit shorowit linked an issue Sep 13, 2022 that may be closed by this pull request
… incident-solar-multiplier

# Conflicts:
#	HPXMLtoOpenStudio/measure.xml
@shorowit shorowit changed the base branch from master to os350 September 27, 2022 01:00
shorowit and others added 5 commits September 26, 2022 19:03
… incident-solar-multiplier

# Conflicts:
#	HPXMLtoOpenStudio/measure.xml
… incident-solar-multiplier

# Conflicts:
#	BuildResidentialHPXML/measure.xml
#	BuildResidentialScheduleFile/measure.xml
#	HPXMLtoOpenStudio/measure.xml
#	workflow/tests/base_results/results.csv
#	workflow/tests/base_results/results_bills.csv
@shorowit
Copy link
Contributor Author

Component load residuals are much higher when using this object. Need to investigate by looking at window heat transfer output variables. Maybe one of them is not properly updating based on this object?

Base automatically changed from os350 to master October 25, 2022 18:36
shorowit and others added 5 commits November 7, 2022 09:14
…o incident-solar-multiplier

# Conflicts:
#	BuildResidentialHPXML/measure.xml
#	BuildResidentialScheduleFile/measure.xml
#	HPXMLtoOpenStudio/measure.xml
#	workflow/tests/base_results/results.csv
#	workflow/tests/base_results/results_bills.csv
…o incident-solar-multiplier

# Conflicts:
#	BuildResidentialHPXML/measure.xml
#	HPXMLtoOpenStudio/measure.xml
#	workflow/tests/base_results/results.csv
#	workflow/tests/base_results/results_bills.csv
shorowit and others added 4 commits May 4, 2023 12:00
…o incident-solar-multiplier

# Conflicts:
#	BuildResidentialHPXML/measure.xml
#	BuildResidentialScheduleFile/measure.xml
#	HPXMLtoOpenStudio/measure.rb
#	HPXMLtoOpenStudio/measure.xml
#	HPXMLtoOpenStudio/resources/constructions.rb
#	HPXMLtoOpenStudio/tests/test_enclosure.rb
#	ReportSimulationOutput/measure.xml
#	workflow/tests/base_results/results.csv
#	workflow/tests/base_results/results_bills.csv
#	workflow/tests/hpxml_translator_test.rb
@shorowit
Copy link
Contributor Author

shorowit commented May 5, 2023

I spent a little time today investing further. Documenting what I found...

From looking at hourly output, it seems like this IncidentSolarMultiplier (ISM) improves the accuracy of the window heat transfer relative to our current approach (of using a shading surface and modifying the view factor to adjust the ground reflectance). If you compare a home w/ no shading to a home w/ 100% shading, the ISM results (red line) perfectly matches the no shading results (blue line) for all hours of the year where there is no sun. Meanwhile, the current approach (orange line) is consistently offset from the ISM results for every hour of the year. Not really sure why that is -- maybe the window convection is somehow being changed in the current approach?

image

So that's a good sign, but other results are strange. The difference in window heat transfer between the ISM approach and current approach is ~3.5 MBtu. Our reported window component loads also show a net change of approximately 3.5 MBtu (see below). But the heating/cooling delivered loads (and thus energy use) change by far more than that. Thus, the sum of component loads is nowhere close to the delivered loads, and the energy use is suspicious.

image

So:

  1. Need to track down why the delivered loads change so much more than the component loads. Should probably run with an ideal air system to further simplify the models.
  2. Ideally would track down why the original approach is offset from the ISM approach for every hour of the year.

@shorowit
Copy link
Contributor Author

shorowit commented May 6, 2023

Seems like the ISM has a bug: NREL/EnergyPlus#10001

@shorowit
Copy link
Contributor Author

shorowit commented May 9, 2023

It also seems like the original approach may be excluding radiation exchange between the window and the sky, which is why it's offset for every hour of the year. According to the transmittance schedule documentation: "Shading devices are assumed to be opaque to long-wave radiation no matter what the solar transmittance value is."

@shorowit shorowit added the bug Something isn't working label May 9, 2023
…o incident-solar-multiplier

# Conflicts:
#	BuildResidentialHPXML/measure.xml
#	HPXMLtoOpenStudio/measure.xml
#	workflow/tests/base_results/results.csv
#	workflow/tests/base_results/results_bills.csv
@shorowit
Copy link
Contributor Author

The E+ bug should be fixed for the next release, which should finally allow us to start using this new object.

@shorowit
Copy link
Contributor Author

shorowit commented Aug 16, 2023

@shorowit shorowit added this to the 1.7.0 milestone Sep 19, 2023
…o incident-solar-multiplier

# Conflicts:
#	BuildResidentialHPXML/measure.xml
#	BuildResidentialScheduleFile/measure.xml
#	HPXMLtoOpenStudio/measure.xml
#	ReportSimulationOutput/measure.xml
#	workflow/tests/base_results/results.csv
#	workflow/tests/base_results/results_bills.csv
#	workflow/tests/hpxml_translator_test.rb
… incident-solar-multiplier

# Conflicts:
#	BuildResidentialHPXML/measure.xml
#	HPXMLtoOpenStudio/measure.xml
#	workflow/tests/base_results/results.csv
#	workflow/tests/base_results/results_bills.csv
@shorowit shorowit changed the base branch from master to os370 September 19, 2023 22:23
GitHub Action and others added 9 commits September 19, 2023 23:39
… incident-solar-multiplier

# Conflicts:
#	BuildResidentialHPXML/measure.xml
#	HPXMLtoOpenStudio/measure.xml
#	ReportSimulationOutput/measure.xml
#	workflow/tests/base_results/results.csv
#	workflow/tests/base_results/results_bills.csv
… incident-solar-multiplier

# Conflicts:
#	HPXMLtoOpenStudio/measure.xml
#	ReportSimulationOutput/measure.xml
#	workflow/tests/base_results/results.csv
#	workflow/tests/base_results/results_bills.csv
@shorowit shorowit marked this pull request as ready for review October 6, 2023 23:33
shorowit and others added 3 commits October 18, 2023 11:47
… incident-solar-multiplier

# Conflicts:
#	BuildResidentialHPXML/measure.xml
#	HPXMLtoOpenStudio/measure.xml
#	ReportSimulationOutput/measure.xml
#	workflow/tests/base_results/results.csv
#	workflow/tests/base_results/results_bills.csv
@shorowit shorowit merged commit 6e68f8a into os370 Oct 18, 2023
@shorowit shorowit deleted the incident-solar-multiplier branch October 21, 2023 19:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working performance refactor
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Use E+ simple shading object
1 participant