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

Question: Why does BuildingElement not have any geometry? #262

Open
filipGG opened this issue Apr 5, 2024 · 2 comments
Open

Question: Why does BuildingElement not have any geometry? #262

filipGG opened this issue Apr 5, 2024 · 2 comments

Comments

@filipGG
Copy link

filipGG commented Apr 5, 2024

In my case I would like to be able to represent a wall as either a series of lines or as a polygon for thick walls.

I could store it as a custom property, but in my mind having actual geometry would make more sense.
What do you think?

@ogiel
Copy link

ogiel commented Apr 8, 2024

I agree that this would make sense to have (and something to want). Just checked and it is indeed not possible to create a geometry twin and give it a relationship to a building element (no relationships available).

Maybe make a PR that contains relationship between building element and geometry equally to relationship between architecture and geometry, and then create building element + geometry twin pairs for each element.

@filipGG
Copy link
Author

filipGG commented Apr 8, 2024

Seems like I don't have permission to open a PR, but I agree and my suggestion would be to add this relationship to BuildingElement:

{
  "@type": "Relationship",
  "description": {
    "en": "A GeoJSON Geometry representing the position or extent of the BuildingElement."
  },
  "displayName": {
    "en": "geometry"
  },
  "maxMultiplicity": 1,
  "name": "geometry",
  "target": "dtmi:org:w3id:rec:Geometry;1",
  "writable": true
}

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