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

Pulls of insulation and some others result in inaccurate values due to volume quantity type or density #78

Open
enarhi opened this issue Apr 2, 2021 · 3 comments
Assignees
Labels
type:bug Error or unexpected behaviour

Comments

@enarhi
Copy link
Member

enarhi commented Apr 2, 2021

Description:

See https://buildingtransparency.org/ec3/epds/ec37yaqu for reference. This pulls as being per m3, but is really per m2 with a thickness applied to make R_SI = 1 m2K/W. In this case, that thickness is 34mm, not the 1000mm that the pulled per volume value implies.

See also https://buildingtransparency.org/ec3/industry-epds/92f158ad940142f9af73d28980bd9004 for reference for issues when pulling Opening material values, as density is not provided and results in being pulled as 1 leading to inaccurate calcs as QuantityType is per mass.

Also occurs for gyp where the unit is per 1000ft2 as per https://buildingtransparency.org/ec3/epds/ec34aqja, QuantityType is incorrectly pulled as per volume.

@enarhi enarhi added the type:bug Error or unexpected behaviour label Apr 2, 2021
@enarhi
Copy link
Member Author

enarhi commented Apr 2, 2021

Temporary workaround shown below for reference:
image

@enarhi enarhi changed the title Pulls of insulation result in inaccurate values due to volume quantity type Pulls of insulation and some others result in inaccurate values due to volume quantity type or density Apr 5, 2021
@enarhi
Copy link
Member Author

enarhi commented Apr 5, 2021

Similar issues occur for https://buildingtransparency.org/ec3/industry-epds/c7b32918b09f47beac8563e5e0c3a13a, where the density is brought in wrong since it assumes density is per t/m3 but is actually kg/m3.

@michaelhoehn
Copy link
Contributor

I'll take a closer look at this in 7.1

@michaelhoehn michaelhoehn added this to the BHoM 7.1 β MVP milestone Jan 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type:bug Error or unexpected behaviour
Projects
None yet
Development

No branches or pull requests

2 participants