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

Think through section floor area #156

Open
haneslinger opened this issue Feb 16, 2024 · 0 comments
Open

Think through section floor area #156

haneslinger opened this issue Feb 16, 2024 · 0 comments

Comments

@haneslinger
Copy link
Contributor

When generating an OSM for building, we check that the sections square footage totals to that of the building. If it does not, we error out.

currently we only test buildings with zero or one sections. This leaves us open to many untested and (and often error filled) cases in which a building has multiple sections, such as

  1. the sections overlap, and therefore, their square footage does not total to that of the building
  2. one or more of these sections do not specify their square footage
    3 one or more of the sections is excluded

This issue is further exacerbated by the fact that if they section is missing it square footage, we assume it is that of the buildings

We need to decide as a group if

  1. if a section square footage is missing, it's safe to assume that it's that of the buildings

  2. the buildingsyncgem should be able to handle multiple sections

  3. if so, flush out the logic for it the section square footage does not total to that of the buildings. (Ie, error out, exclude sections, ect)

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