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

Bug in Tabula building area calcuation #662

Open
marcusfuchs opened this issue Nov 19, 2020 · 0 comments
Open

Bug in Tabula building area calcuation #662

marcusfuchs opened this issue Nov 19, 2020 · 0 comments

Comments

@marcusfuchs
Copy link
Contributor

By default the SingleFamilyHouse classes in teaser/logic/archetypebuildings/tabula/de/singlefamilyhouse.py and teaser/logic/archetypebuildings/tabula/dk/singlefamilyhouse.py contain 1 single zone. For a special use case, we want to add a second zone to this archetype. This exposed a bug in the area calculations performed in generate_archetype(). All areas are calculated similar to:

outer_wall.area = (
    self.facade_estimation_factors[self.building_age_group]["ow1"]
    * type_bldg_area
) / len(self._outer_wall_names_1)

For 1 zone, this leads to correct results. But for more than 1 zone the total outside area of the building is not distributed between the zones. Instead, each zone gets the full outside area assigned to it, so that 2 zones result in twice the outside area compared to a 1-zone model for the same building. This is because the equations use type_bldg_area when in fact they should use zone.area instead. The following leads to correct results:

outer_wall.area = (
    self.facade_estimation_factors[self.building_age_group]["ow1"]
    * zone.area
) / len(self._outer_wall_names_1)

As IMO this is a bug, I will branch of the master branch and submit a PR back to master.

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

1 participant