Support {graph:'huge'} for label propagation

knutwalker commented 5 years ago

This is not yet full huge support for LPA but allows for the huge graph to be used in case it isn't actually huge but can be used in places where a simple Graph is sufficient.

Loading node weights and properties has been added to the huge graph
LPA lifted its restriction to require the HeavyGraph and falls back to empty NodeWeights in case the graph does not support NodeWeights itself (GraphView should be the only one). Since the node weight loading is added to the 'old' huge loading (without #771 ), I propose to add this first, so that I can rebase 771 and we can have the actual huge LPA later on with a smaller changeset.

mneedham commented 5 years ago

@knutwalker is this now ready for review?

knutwalker commented 5 years ago

@mneedham yea, sure. I've added all follow-ups that aren't ready yet to another branch, so this one is independent and can be reviewed/merged.

mneedham commented 5 years ago

@knutwalker this all looks good. Think we created some conflicts from merging the huge loading stuff earlier

neo4j-contrib / neo4j-graph-algorithms

Support {graph:'huge'} for label propagation #842