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

Add heuristic approach to assign edge-based partition IDs #3813

Merged
merged 1 commit into from
Mar 13, 2017

Conversation

oxidase
Copy link
Contributor

@oxidase oxidase commented Mar 13, 2017

Issue

Add heuristic approach to assign edge-based partition IDs that minimize number of faked source/destination nodes that not actual source (have a row with all invalid edge weights in the cell storage) or destination (have a column with all invalid edge weights in the cell storage) nodes in the the following case: a single backward edge-based edge that goes backward into a boundary edge will be placed in the partition of the other node-based node partition.

This gives the following improvements in number of unconnected nodes ([warn] Node needs to either have incoming or outgoing edges in cell. Number of unconnected nodes is 2):

OSM before after
berlin-latest 26 0
bayern-latest 233 2
germany-latest 1852 43

Tasklist

  • review
  • adjust for comments

Requirements / Relations

#3205, #3731

that minimize number of faked source/destination nodes in
the edge case: a single backward edge will be placed
in other than node u partion
Copy link
Member

@TheMarex TheMarex left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is a good first step. 👍 I outlined a different approach in #3818 that would remove all unconnected nodes.

@TheMarex TheMarex merged commit c6e9cc8 into master Mar 13, 2017
@TheMarex TheMarex deleted the mld/ebg-paritioning branch March 13, 2017 22:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants