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.

Sign up for GitHub

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

Jump to bottom

Cluster buildings update #44

Closed
wants to merge 6 commits into from
Closed

Cluster buildings update #44

wants to merge 6 commits into from

Conversation

Margherita-Capitani
Copy link
Contributor

With this Pull Request the Building Clustering rule is updated, the main differences are:

  • The Geopandas package is used
  • The clustering process is updated. All the “polygon” elements in the geoJSON file are selected, the centroid for each building is calculated, and the k.means method is used by exploiting the centroids of the poylgons.
  • A feature is added to obtain a dataframe that counts for each cluster the building types, on the osm data tag.

The rule has been tested and works correctly locally.

@davide-f davide-f force-pushed the Cluster_buildings_update branch 5 times, most recently from 294116e to 72a7bcc Compare June 18, 2024 12:45
@davide-f davide-f force-pushed the Cluster_buildings_update branch from 72a7bcc to 86477a3 Compare June 18, 2024 12:51
@davide-f
Copy link
Member

davide-f commented Jul 4, 2024

Given the other PR, can we close this? @Margherita-Capitani

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Margherita-Capitani @davide-f