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

Keep unions and their children together #6

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

djBirdman
Copy link
Contributor

Might not be the nicest layout (I'll try and work on improving that), but the ordering is correct. This also handles multiple unions per person, adding a unionyear in the data for ordering.
I also decided to compile the data file for the Stark family tree from Game of Thrones (not a fan, but a few other d3 tree examples have used parts of it). I suggest using this bigger tree to see what the d3-dag does by itself

@BenPortner
Copy link
Owner

Hi @djBirdman,

First off, sorry that it took me so long to get back to you! Unfortunately, js_family_tree is only a side project for me so I don't get to spend as much time on it as I would like. I hope you are still willing to work on this together!

Second off: Thank you so much for pouring so much time into issue #4! I had a look at your code and I can tell that it was a lot of work! I really appreciate your effort.

Now, my comments on the PR:

  • GoT data 1d5709f: Love it! Will use this for the main example on the start page. Right now I am working on a more detailed example section. Will make the change as soon as this is done.
  • Sort section 4ffcf81: Super impressive work! However, I feel like node and layer sorting algorithms should be part of d3-dag, rather than js_family_tree. We should discuss including a generalized version of your algorithm (with arbitrary fields to be used for ranking) in d3-dag. Could you open an issue in https://github.com/erikbrinkman/d3-dag to get the process started?

@djBirdman
Copy link
Contributor Author

I agree with your thoughts on sorting: I got carried away just trying to make something work without thinking about it properly.

Looking at existing issues in d3-dag, would the ordering discussed in erikbrinkman/d3-dag#93 and available in v0.11.4 of d3-dag be useful in this case? (I haven't nailed down a complete understanding of dag layout concepts!)

@BenPortner
Copy link
Owner

Hi @djBirdman,

I am not 100% sure but I think d3-dag#93 addresses a different issue (it concerns grid layout instead of sugiyama, I believe the ranking controls the order of the layers, not the order of nodes within the layers). However, I might be wrong. I believe it's best to open a new issue in d3-dag and make a careful inquiry. Describe the desired functionality in detail and link to your solution. Erik will surely be able to tell us if there is existing functionality to achieve the desired behavior. If not, I am sure he will appreciate your contribution 😃

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