multinet-app / multilink

A node-link visualization built to work with the multinet ecosystem.
https://multinet.app
Apache License 2.0
4 stars 2 forks source link

Use force directed layout for categorical variables #317

Closed JackWilb closed 2 years ago

JackWilb commented 2 years ago

Does this PR close any open issues?

No

Give a longer description of what this PR addresses and why it's needed

This PR makes the categorical layout use the force directed layout for jittering the nodes of one group. This means that the nodes are no longer overlapping each other when rendering in this manner.

Provide pictures/videos of the behavior before and after these changes (optional)

I find this so satisfying: https://user-images.githubusercontent.com/36867477/177649981-fe40bf4a-1be2-4852-959d-bd809b4facd3.mp4

Are there any additional TODOs before this PR is ready to go?

TODOs:

netlify[bot] commented 2 years ago

Deploy Preview for next-multilink ready!

Name Link
Latest commit f4342476b52442ab211e967e71b4eadbe9fc550c
Latest deploy log https://app.netlify.com/sites/next-multilink/deploys/62c605ed03f2110008caf06a
Deploy Preview https://deploy-preview-317--next-multilink.netlify.app/
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

alexsb commented 2 years ago

This is awesome! One comment: the labels switch off automatically when switching to categorical layout. Is that intentional? It might not be necessary in this case.