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

Graph collectors? #3754

Open
oliviercailloux opened this issue Dec 26, 2019 · 4 comments
Open

Graph collectors? #3754

oliviercailloux opened this issue Dec 26, 2019 · 4 comments

Comments

@oliviercailloux
Copy link

Are there collectors that create and populate graphs, that I perhaps have missed? For example, can I easily stream the entries of a Map<K, K> and return a (say, directed) graph that would contain the corresponding edges?

If so, please document on the wiki.

If not, please consider adding a set of nice collectors, similar to those available for the guava collection classes!

@cpovirk
Copy link
Member

cpovirk commented Dec 26, 2019

We have nothing. This does sound like something we ought to have. Thanks.

@mitchellirvin
Copy link
Contributor

@cpovirk do you have a preference for which of the Graph types have collectors implemented or whether they're implemented in one PR or their own?

I'm initially thinking to start with adding graph/GraphCollectors and defining a basic collector for ImmutableGraph and moving forward from there.

Would you mind providing some guidance on any considerations you might have? I'm new to the codebase, definitely not sure of where the blindspots I have are yet.

@cpovirk
Copy link
Member

cpovirk commented Dec 27, 2019

Unfortunately, I know very little about common.graph. One of its owners will have to have a look eventually, but I wouldn't expect that to happen soon :(

@oliviercailloux
Copy link
Author

Related: Build Graph given a SuccessorsFunction and a set of nodes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants