Graph clustering #823
Merged
Graph clustering #823
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
@Manon3005 would also like to add #835 to this PR if possible |
DiegoVinasco
requested changes
Aug 27, 2024
There was a problem hiding this comment.
The PR is in good shape. There's just a few things that need to be modified before the PR is accepted, please see my comments.
Once those are done we should follow the PR submission checklist to make sure everything is ready. Then we can merge the PR!
- Comments
- Checklist
- PR/MR
Manon3005
force-pushed
the
graph-clustering
branch
from
3939ece
to
d8b5fff
Compare
DiegoVinasco
approved these changes
Aug 27, 2024
- added dropdown menu for nodes functions - added queries for retrieving nodes descendants - added clusterisation by neighbors - added initial clusterisation by type
- added clusterisation by zoom - added graph configuration menu - added interactions with the 3D scene
- added link direction - updated mouseover event - added tip - updated cluster node label
- display of the number of nodes clustered in a cluster - distinction between links between the same nodes (path link instead of line link) display of link direction no longer works: the problem will be corrected in a future commit
- updated node color selection
- fixed a bug in the method changeVisibilityDescendants
…e demo ud-graphclustering
Manon3005
force-pushed
the
graph-clustering
branch
from
7adca36
to
829f2a9
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
#824
#825
#826
#827
#828
#829
#830
#831