Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Configurable node/edge attribute aggregation when simplifying graph or consolidating intersections #1155

Merged
merged 8 commits into from
Apr 1, 2024

Conversation

gboeing
Copy link
Owner

@gboeing gboeing commented Mar 30, 2024

This PR:

  • adds edge_attr_aggs param to the simplify_graph function to let users specify attribute aggregation functions when merging multiple edge segments together
  • adds node_attr_aggs param to the consolidate_intersections function to let users specify attribute aggregation functions when merging multiple nodes together

See also #1127 #1144 #1145 #1117 #1106

Copy link

codecov bot commented Mar 30, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 97.99%. Comparing base (d614c96) to head (ba66e0e).
Report is 6 commits behind head on v2.

Additional details and impacted files
@@           Coverage Diff           @@
##               v2    #1155   +/-   ##
=======================================
  Coverage   97.98%   97.99%           
=======================================
  Files          24       24           
  Lines        2436     2442    +6     
=======================================
+ Hits         2387     2393    +6     
  Misses         49       49           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@gboeing gboeing merged commit c71d702 into v2 Apr 1, 2024
7 checks passed
@gboeing gboeing deleted the simp branch April 1, 2024 23:40
@gboeing gboeing mentioned this pull request Apr 3, 2024
13 tasks
@gboeing
Copy link
Owner Author

gboeing commented May 3, 2024

The first pre-release OSMnx v2 beta has been released. Testers needed! See #1123 for details.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant