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

Improve caching of TVP Nodes #145

Open
ckipp01 opened this issue Jun 3, 2021 · 0 comments
Open

Improve caching of TVP Nodes #145

ckipp01 opened this issue Jun 3, 2021 · 0 comments
Labels
enhancement New feature or request TVP

Comments

@ckipp01
Copy link
Member

ckipp01 commented Jun 3, 2021

Task

A large improvement that can be made currently with the TVP implementation is that when a build target is compiled, the root node for that target is replaced. This comes in via the metals/treeViewDidChange. Now even if you have a bunch of stuff underneath that node expanded and nothing really changes in the nodes, the entire node chain is replaced.

Instead we should take a snapshot of that node and everything underneath it. We'll then need to do the retrieval again and then compare the trees to see what needs to be updated instead of just replacing the whole thing.

What this currently means is that if you have the tree open and expanded and you save, the tree will collapse since everything underneath was replaced.

Search terms

TVP caching.

@ckipp01 ckipp01 added task TVP enhancement New feature or request and removed task labels Jun 3, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request TVP
Projects
None yet
Development

No branches or pull requests

1 participant