tudelft3d / bag3d

Software for generating a 3D version of the BAG dataset and more. Python + PostgreSQL + 3dfier
GNU General Public License v3.0
11 stars 11 forks source link

Add (relative) bulding heights #23

Closed balazsdukai closed 3 years ago

balazsdukai commented 5 years ago

Currently we only report the absolute height of the top of a building. For many applications it is more interesting to know the height of the building (top - ground). This could eventually replace the ground or roof absolute heights.

evetion commented 5 years ago

This would require more research. Experimenting with roof-50 - ground-50 will result in several buildings with negative heights and ignores towers on a larger, lower bases. Is this the multi-level issue in #13?

balazsdukai commented 5 years ago

@evetion no, this is not related to the multi-level issue, because the multi-level attribute would indicate if the roof geometry contains multiple height levels.

As you say, roof-50 - ground-50 easily results in negative heights, the reason for that is a bit convoluted to explain, but has to do with how many points are measured on the walls of the "real" building.

Nevertheless, the relative height would be determined by roof-X - mean ground, or even mean roof - mean ground. Thus possibly reporting only a single relative height value per building, and not 6.

balazsdukai commented 3 years ago

Repo closes because it has been superseded.