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

Feat: add node expand and prune feature to Graph front-end #1093

Open
wants to merge 53 commits into
base: main
Choose a base branch
from
Open

Feat: add node expand and prune feature to Graph front-end #1093

wants to merge 53 commits into from
+1,224 −422

Conversation

danielaskdd
Copy link
Collaborator

@danielaskdd danielaskdd commented Mar 15, 2025 •
edited
Loading

Description

Add node expansion and pruning feature to Graph front-end.

Related Issues

#975

Changes Made

  • Refactoring graph view and sigma instance lifetime behaviors to avoid flickering during node operations
  • Add mode expansion and pruning features on property pop up dialogue box
  • Added data reload button to graph labels
  • Fix layout animation function
  • Fix zoom auto fit bug after nodes layout changed
  • Rearrange buttons on action tool bar

Checklist

  • Changes tested locally
  • Code reviewed
  • Documentation updated (if necessary)
  • Unit tests added (if applicable)

@danielaskdd
Merge branch 'improve-property-tooltip' into feat-node-expand
eb11a3d
@danielaskdd
Add Node Expansion and Pruning Features
de01602
@danielaskdd
Fix linting
20c9765
@danielaskdd
Refactor node expansion logic with improved degree and size calculation
1ae65c9
@danielaskdd
Avoid graphics flickering during node operations
5decd03
@danielaskdd
Fix refresh layout button failure
413d201
@danielaskdd
Improve node layout by using polar coordinates for positioning expand…
fc4582b 
…ed nodes
@danielaskdd
Fix linting and upate webui assets
fdaf199
@danielaskdd
Refactor graph search to update search engin after node expand or prune
f4fceca
@danielaskdd
style: change double quotes to single quotes in ZoomControl.tsx
abda12f
@danielaskdd
feat: Improve graph zoom reset to fit all nodes with proper scaling
c5362d9 
- Add smarter node boundary calculation
- Include padding around graph nodes
- Calculate optimal scale ratio
- Add error handling and null checks
- Animate transition to new zoom level
@danielaskdd
Fix linting
1d4b024
@danielaskdd
Fix empty graph check problem of graph view load
4f4d7cb
@danielaskdd
Fix Graph TAB flickering issue in node expansion
97696dd
@danielaskdd
Remove reduntant search engine update functions
a138000
@danielaskdd
Optimize node spread animation with better spacing and randomized ang…
6555d5a 
…le offset

• Adjust spread factor calculation
• Add debug logging
@danielaskdd
Optimize node degree and size calculation for expanded graph nodes
b9dd53b
@danielaskdd
fix(graph): improve fit-to-screen calculation with proper padding and…
4ea0b85 
… zero size check
@danielaskdd
fix: improve graph zoom control and node expansion functionality
bfbcdcb 
• Fix container padding in zoom calculation
• Extract node size update into helper fn
• Update node sizes for empty expansions
@danielaskdd
feat: add react-error-boundary package to web UI dependencies
550b4d1
@danielaskdd
Improve graph state cleanup by fully resetting sigma instance on labe…
59ac94c 
…l change

• Modify reset logic to clear sigma instance
• Remove manual node deletion
• Update GraphLabels to use reset()
• Simplify state cleanup process
• Ensure complete graph refresh
@danielaskdd
Change zoom reset animation duration from 500ms to 1000ms in ZoomControl
55fc191
@danielaskdd
Update webui assets
537961d
@danielaskdd
style: Update colors and button styles in PropertiesView component
1b5c3d5
@danielaskdd
Update web assets
6c9270a
@danielaskdd
Simplify zoom control logic by using fixed ratio
ce6ad75
@danielaskdd
refactor: reorder icons order in control bool bar
64dfa65
@danielaskdd danielaskdd added the ui web ui label Mar 15, 2025
@danielaskdd danielaskdd reopened this Mar 16, 2025
@danielaskdd danielaskdd reopened this Mar 16, 2025
@danielaskdd
Merge branch 'main' into feat-node-expand
1c19290
@danielaskdd
Merge branch 'Fix-pipeline-batch' into feat-node-expand
c23cff2
@danielaskdd
Fix linting
58d519f
@danielaskdd
Update log message for in-memory DB persistence
5f27d42
@danielaskdd
Fix overflow issues in UI components
2753cbe 
- Set overflow-hidden for main container
- Add overflow-auto to documents tab
- Add overflow-hidden to graph and retrieval tabs
- Set overflow-hidden for SigmaContainer
- Set overflow-hidden for retrieval testing container
@danielaskdd
Update webui assets
3e3e963
@danielaskdd
Merge branch 'main' into feat-node-expand
1dd0cac
@danielaskdd
Merge branch 'main' into feat-node-expand
9e9c0bd
@danielaskdd
Enhance graph layout control with improved animations and stability
f231f2d 
- Added custom animation timer for smoother transitions
- Improved Force Directed layout parameters
- Added auto-stop feature for animations
@danielaskdd
Updated layout animation and force atlas translations
a40c9ef
@danielaskdd
Fix: Let tooltips show on top of layout tool bar
90c64b9 
- Removed PopoverPrimitive.Portal wrapper
- Added z-60 to TooltipContent
@danielaskdd
Remove refresh layout button and related code
e03e120
@danielaskdd
Optimized graph layout animation parameters, Smoothed overlapping ani…
971034f 
…mations

- Reduced animation duration to 300ms
- Decreased update interval to 200ms
- Extended auto-stop timeout to 3 seconds
@danielaskdd
Update webui assets
6f2761a
@danielaskdd
Optimize layout configurations for better node arrangement.
4ecb13d 
- Increased margin in Noverlap layout
- Adjusted Force layout iterations
- Enhanced Noverlap settings for spacing
- Standardized maxIterations across layouts
@danielaskdd
Added data reload button to graph labels component
3efcf46
@danielaskdd
Update webui assets
a2cdfcb
@danielaskdd danielaskdd requested a review from LarFii March 17, 2025 13:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@LarFii LarFii Awaiting requested review from LarFii

Assignees
No one assigned
Labels
ui web ui
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@danielaskdd