cityjson / cityjson-qgis-plugin

A QGIS plugin that adds support for CityJSON files
Apache License 2.0
35 stars 8 forks source link

Change Altitude Clamping from Relative to Absolute #52

Open kannes opened 1 year ago

kannes commented 1 year ago

I was really puzzled for a while why my LoD2 buildings were floating above the terrain in 3D views. Then I realised that the 3D settings for the loaded CityJSON layers was set to Relative Altitude Clamping. For terrain and buildings to match, I had to change it to Absolute.

This seems like an oversight or bug to me. Surely the buildings should be positioned in space with their absolute 3D coordinates?

QGIS 3.26.1, CityJSON-loader 0.8.1

liberostelios commented 1 year ago

Hi @kannes,

Thank you for sharing this. 🙏

To be honest, it was never the plugin's main purpose to deal with the formatting aspect of the output layers, so this is indeed more like an oversight than a bug. But, I can see why this behaviour might cause some confusion. So I'll mark it as a bug and will fix this for the next release.

Cheers!

kannes commented 1 year ago

Awesome, thank you! I was super happy how it built great looking 3D geometries "just like that" so this will make it even better. :)