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

Feil i polygon_centroid #5

Open
NKAmapper opened this issue Mar 23, 2021 · 3 comments
Open

Feil i polygon_centroid #5

NKAmapper opened this issue Mar 23, 2021 · 3 comments

Comments

@NKAmapper
Copy link
Owner

Om noen orker ta en titt:

Funksjonen polygon_centroid() i building_merge.py gir en del ganger rare resultater, med midtpunkt godt utenfor polygonet, selv for enkle kvadrater.
Hva kan være feil her?

Kunne ha brukt et ferdig bibliotek selvfølgelig, men målet her var å ha en lav terskel for å kjøre skriptet.

@noenandre
Copy link
Contributor

Du deler x og y på (3.0 * det) to ganger, både ved
x = x / (3.0 * det) y = y / (3.0 * det)
og
return (x / (3.0 * det), y / (3.0 * det) )
Nå implementerte jeg akkurat samme funksjonen i mitt pull-requests...

@NKAmapper
Copy link
Owner Author

Ja det har du visst rett i (må ha sneket seg inn etter testing), men også å ha korrigert for dette blir det fremdeles feil. Mange av punktene ligger et godt stykke utenfor bygningspolygonet. Kan plottes med add_node funksjonen.

@noenandre
Copy link
Contributor

Det var feil i min utgave i funksjonen også, det så jeg først etter at jeg kjørte scriptet for hele Oslo, og feilen ble verre jo mindre areal bygningen hadde. Så dette er nok en armetrisk feil.
Javascript-biblioteket Turf flyter først et polygon til et nytt referansesystem med origo i et det første punktet i polygonet,
og etter og implementert samme oppskrift forsvant feilen fra municipality_split.

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