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.

Sign up for GitHub

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

Jump to bottom

Improved incremental layout #33

Open
1 of 2 tasks
ugurdogrusoz opened this issue Mar 2, 2021 · 1 comment
Open
1 of 2 tasks

Improved incremental layout #33

ugurdogrusoz opened this issue Mar 2, 2021 · 1 comment
Labels
enhancement Enhancement to existing feature

Comments

@ugurdogrusoz
Copy link
Contributor

ugurdogrusoz commented Mar 2, 2021 •
edited

We have two potential improvements for incremental layout:

  • 1. When tiled nodes are moved, their locations are used to determine the initial position of the tiled nodes as a whole but not individually. In below example I move a node to the previous row but it's always tiled the time when incremental layout is applied.
  • 2. Degree 1 nodes (or in general nodes that are reduced as part of trees) are not laid out incrementally. See how they change places with succeeding incremental layouts.
Visuall.Sample.App.-.Google.Chrome.2021-03-02.13-15-22.online-video-cutter.com.mp4
@ugurdogrusoz ugurdogrusoz added the enhancement Enhancement to existing feature label Mar 2, 2021
hasanbalci added a commit to iVis-at-Bilkent/cose-base that referenced this issue Mar 3, 2021
@hasanbalci
Apply tree reduction in incremental layout iVis-at-Bilkent/cytoscape.…
26c59e1 
…js-fcose#33

If it's a pure incremental layout (not an incremental layout applied after a randomized layout), we apply a different version of the tree reduction.
hasanbalci added a commit that referenced this issue Mar 3, 2021
@hasanbalci
Apply tree reduction in incremental layout #33
ff5209b 
Before we don't apply tree reduction in incremental layout, but now cose-base supports a new version of the tree reduction to be used in pure incremental layout (an incremental layout that is not applied after randomized layout)
@ugurdogrusoz
Copy link
Contributor Author

2 seems to be resolved!

Screen.Recording.2021-03-03.at.16.00.04.mov

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement Enhancement to existing feature
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@ugurdogrusoz