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

Improve subgraph support #23

Closed
wants to merge 1 commit into from
Closed

Improve subgraph support #23

wants to merge 1 commit into from

Conversation

phyrwork
Copy link

Add ability to add subgraphs alongside ungrouped vertices

Create graphs that look like this (created with this revision)

Requires Vertex::group to be nullable to get this effect (I don't see any reason why a Vertex cannot be groupless

Until then implementation will continue to function as usual (either all subgraphs or no subgraphs)

Add ability to add subgraphs alongside ungrouped vertices
@makasim
Copy link

makasim commented Jul 17, 2018

What could be done to get this merged? I need the feature too.

@clue clue added this to the v0.4.0 milestone Oct 9, 2018
@clue
Copy link
Member

clue commented Oct 9, 2018

@phyrwork Thank you for working on this and filing this PR! I agree that this feature makes perfect sense 👍

That being said, I think this is something that will need a bigger changeset as it is somewhat related to #8 and #13 and also graphp/graph#114. There are plans to remove the "group" property altogether, at which point it makes perfect sense to assign custom "subgraph" attributes. As such, I would like to postpone this for now and have just assigned this to the v0.4.0 milestone.

What do you think about this? 👍

@clue
Copy link
Member

clue commented Oct 4, 2019

I'm having to close this for now as it hasn't received any input in a while and it's unlikely this will get traction any time soon. Please come back with more details if you think this is still relevant and we can reopen this 👍

Thank you for your effort nonetheless, keep it up! 👍 For a follow-up, see also #23.

@clue clue closed this Oct 4, 2019
@clue clue removed this from the v0.4.0 milestone Oct 4, 2019
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.

None yet

3 participants