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

huge refactor #115

Merged
merged 79 commits into from
Nov 10, 2015
Merged

huge refactor #115

merged 79 commits into from
Nov 10, 2015

Conversation

nikhan
Copy link
Contributor

@nikhan nikhan commented Nov 10, 2015

This is a huge refactor that migrates the visual graph from SVG to canvas. This provides monumental benefits to performance and scalability. The canvas implementation consists of several buffers that store an image of various elements (nodes, edges) that are all composed together to build the final visual representation of the graph. The canvas implementation can handle thousands of nodes per group -- and probably can be optimized further.

The data structures that govern how the model interacts with the views is now written with a Flux-like architecture in mind.

This PR introduces some regressions (in order of priority):

  • group labels cannot be set
  • source parameters cannot be set
  • there is no copy and paste
  • the group/ungroup buttons are not functional (although you can press g and u)
  • group translations/positions are wonky

@nikhan
Copy link
Contributor Author

nikhan commented Nov 10, 2015

Oh, and we should probably tag the current release...

@mikedewar
Copy link
Collaborator

made tag

@mikedewar
Copy link
Collaborator

well, it runs

if _, ok := s.blocks[c]; ok {
err := s.DeleteBlock(c)
if err != nil {
fmt.Println(err)
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

is this for debugging?

mikedewar added a commit that referenced this pull request Nov 10, 2015
@mikedewar mikedewar merged commit 8b59ffd into nytlabs:master Nov 10, 2015
@mikedewar
Copy link
Collaborator

🍏 📗 💚

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants