Skip to content

Add a scale parameter, to better handle low density pointclouds #169

Description

@Isaskar

So I've been trying for the past week to make Roofer work with the Swedish mapping agency Lantmäteriet's open LIDAR data. This is quite low density compared to what Roofer is designed around (only 1-2 points per m2) but it still has enough detail to be useful.

I spent ages trying to find parameter values that worked but I only got garbage results. That is until I finally had the idea of scaling my input data down, to simulate a higher density. Amazingly, this actually worked, and for the first time I got usable results. My testing indicates that a scale factor of 0.5-0.6 is optimal for the Lantmäteriet data set.

Now, I am of course able to do this by scaling my inputs down before use and then scaling the output inversely, but it would be much easier if Roofer could do this for me. How this is best implemented I don't know, but at least to me a scale parameter similar to what I'm already doing seems like a good idea.

For testing purposes, I will include some files here.

Here are some images to illustrate just how much of a difference this makes:

No scaling of input
Image

Input scaled by a factor of 0.55
Image

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions