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

Default clustering in the demo #104

Open
ugurdogrusoz opened this issue Aug 4, 2020 · 3 comments · Fixed by #106
Open

Default clustering in the demo #104

ugurdogrusoz opened this issue Aug 4, 2020 · 3 comments · Fixed by #106
Labels
enhancement Enhancement to existing feature

Comments

@ugurdogrusoz
Copy link
Contributor

Enhance the demo (choose an appropriate one of the demos) by taking an arbitrary flat graph and generating a hierarchically clustered version, where clusters are represented by compound nodes. This should be useful in providing a default mechanism to handle complexity of large graphs. For clustering, we could try using this algorithm or something similar in Cytoscape.js core.
For loading an arbitrary graph we could use the import GraphML facility as in this demo.

@ugurdogrusoz
Copy link
Contributor Author

ugurdogrusoz commented Aug 25, 2020

  • 1. When I upload some graphml files, I see some disconnected nodes (corresponding to empty clusters). We should remove those during clustering.

  • 2. "markov" -> "Markov"

  • 3. Limit Markov clustering depth to be greater than 0

  • 4. Let's add a checkbox with title "Apply Markov clustering on load" at the end and apply Markov clustering automatically if this is check during uploading from a file.

  • 5. Let's keep the colors in the UI to a mininum.

@ugurdogrusoz
Copy link
Contributor Author

1 Those disconnected nodes are gone but I see some flickering on load (when apply Markov on load is checked). Try checking this option, then load circle.graphml in the samples. Note that the graph is first rendered in the middle of the canvas and then rendered again to the lower left and moved to the center with animation.

3 I can still make the depth 0 or negative.

msalihaltun added a commit that referenced this issue Aug 27, 2020
…ed order of collapse and layout operations
@ugurdogrusoz
Copy link
Contributor Author

On load, there is still flickering as described in previous comment. Let's leave this for later.

@ugurdogrusoz ugurdogrusoz removed their assignment Aug 27, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Enhancement to existing feature
Projects
None yet
2 participants