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

Squaring-Error #2472

Closed
hajo4 opened this issue Dec 11, 2014 · 4 comments · Fixed by #5999
Closed

Squaring-Error #2472

hajo4 opened this issue Dec 11, 2014 · 4 comments · Fixed by #5999
Labels
bug A bug - let's fix this!
Milestone

Comments

@hajo4
Copy link

hajo4 commented Dec 11, 2014

"Square an area" should align the corners of an area / building,
to beautify it to a more right-angled shape.
But sometimes this does not work right, and ends up with an unuseble shape,
e.g. one corner is lost (or 2 points moved to the same coordinates).

When mapping small villages, there are quite a lot of old buildings with odd shapes.
This is where I encounter the buggy behaviour of "square an area" more frequently,
but it can happen with simple 4-cornerd houses too.
(I just didn't bother to write a bugreport yet)

Here some examples from the village Rossdorf/Zeilhard :

Find all those places I marked with notes: http://overpass-turbo.eu/s/6tN

Note: since the report, there has been an automatic mass-edit,
that might have corrected the data that leads to the error,
see http://www.openstreetmap.org/changeset/27512311

@bhousel
Copy link
Member

bhousel commented Dec 11, 2014

Way/316801542 - 8-corner house gets distorted

yep:

orthagonalize issue

@bryceco
Copy link
Contributor

bryceco commented Dec 14, 2014

In Go Map!! I'm using the iD code for squaring, translated line-for-line to C. The first way (Way/316791604) fails because it contains a duplicated node that produces a NaN. The other examples work perfectly for me, with none of the distortions iD has, so it doesn't seem to be a fundamental algorithm bug.

@bhousel bhousel added the bug A bug - let's fix this! label Dec 18, 2014
@rivermont
Copy link
Contributor

I'm assuming this has been fixed @bhousel ?

@bhousel
Copy link
Member

bhousel commented Dec 6, 2018

I'm assuming this has been fixed @bhousel ?

I don't think anything has changed on this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug A bug - let's fix this!
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants