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

WeatherData HOM #73

Open
larissakuehn opened this issue Feb 21, 2024 · 1 comment
Open

WeatherData HOM #73

larissakuehn opened this issue Feb 21, 2024 · 1 comment
Assignees

Comments

@larissakuehn
Copy link
Contributor

These two models are used to calculate the radiation on the tilled surfaces of the HOM, based on data from the weather data reader (IBPSA.BoundaryConditions.WeatherData.ReaderTMY3):

  • BESMod.Systems.Demand.Building.Components.SunWithWeaBus
  • BESMod.Systems.Demand.Building.Components.RadOnTiltedSurf_LiuWeaBus

The first model includes a time shift for TMY instead of TRY data, which is not required with the updated version of the weather data converter, right?

In the latter model the irradiation on tilled surface is calculated on basis of the direct and diffuse irradiation on the horizontal surface. As the direct irradiation on a horizontal surface is not included in the weather data reader it is calculated from data. The models distinguishes between TRY and TMY data as an option. Currently in both cases the irradiance input for HDirHor is calculated by the following equation:
Input_HDirHor = HGloHor - HDiffHor
Unfurtunarly currently for further calculations the model now uses the following equation for TRY data, which is correct:
HDirHor = Input_HDirHor
but the following equation for TMY data:
HDirHor = Input_HDirHor * cos_tilled_z
The model assumes the input for TMY data to be HDirNor.

As we only want to use the TMY format in combination with the TMY3 weather data reader my suggestion is to delete the second option for TRY data and to use HDirNor as an input. This would be equivalent to the irradiance input for direct irradiance in the ROM models. For the first model, I would then delete the time shift and directly use the data.

Copy link

Branch issue-73-WeatherData_HOM created!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants