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

Fix issue with scale rendering #373

Closed JackWilb closed 1 year ago

JackWilb commented 1 year ago

Does this PR close any open issues?

No

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

I noticed that the Carnegie dataset wasn't loading properly in this vis app. The problem was in the if statement guarding the color scales. I've fixed the issue and it renders without issue.

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

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

No

netlify[bot] commented 1 year ago

Deploy Preview for next-multilink ready!

Name Link
Latest commit 6e9fc65dcc457e2726e5d0c73d0eae3de4fd1d06
Latest deploy log https://app.netlify.com/sites/next-multilink/deploys/647a65356805d300086f1ebc
Deploy Preview https://deploy-preview-373--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.