-
Notifications
You must be signed in to change notification settings - Fork 633
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
quick implementation of most of ggraph/ggforce #1326
base: master
Are you sure you want to change the base?
Conversation
Was this PR ever considered? What's the situation now? |
@danton267 would you mind filing an issue requesting this feature? Having some |
@cpsievert I will happily work on some of the geoms |
@moutikabdessabour it'd be great if you could also look into converting |
@cpsievert currently implementing ggraph's |
This comment has been minimized.
This comment has been minimized.
I implemented the ggraph's Do you think I should go forward and implement all the examples found here?? https://www.data-imaginist.com/2017/ggraph-introduction-layouts/ Now all the hierarchical plots are supported. EDIT: I just noticed that you had some of the geoms implemented in this PR. so ggraph treemaps are supported in this PR too EDIT 2: Edit 3: Managed to implement I was able to implement all the @cpsievert how would I convert |
Hello @moutikabdessabour @cpsievert ! Thanks for your work - I am new to posting GitHub issues. Out of the ggforce package, is it possible to implement the |
Just want to put in my two cents that this would be an incredible addition if it somehow passes checks one day. |
TODO:
GeomMarkCircle
/GeomMarkEllipsis
/GeomMarkHull
GeomAxisHive
(path + text?). For example, see https://github.com/thomasp85/ggraph/blob/8d23845/R/geom_axis_hive.R#L123-L142key
attributes.geom2trace()
level).