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

Consolidate neighbor count logic #710

Closed
kmcginnes opened this issue Dec 10, 2024 · 0 comments · Fixed by #744
Closed

Consolidate neighbor count logic #710

kmcginnes opened this issue Dec 10, 2024 · 0 comments · Fixed by #744
Assignees

Comments

@kmcginnes
Copy link
Collaborator

kmcginnes commented Dec 10, 2024

The logic around discovering and collating neighbor information is spread out across the code base. On my prior attempts to make graph reloading from IDs work this was a major bottleneck.

The task is to pull out all the logic around neighbor calculations to a single place and integrate it with display entities for just the graph canvas and table. This allows the gathering of the information to be deferred and ensures consistent behavior.

It may even end up with better performance.

@kmcginnes kmcginnes added this to the Release 1.13.0 milestone Dec 10, 2024
@kmcginnes kmcginnes self-assigned this Dec 10, 2024
@kmcginnes kmcginnes changed the title [Task] Consolidate neighbor count logic Consolidate neighbor count logic Jan 14, 2025
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 a pull request may close this issue.

1 participant