gephi / gephi/gephi-lite

Cache merged dynamic and static data

Open
#262 0 comments 0 reactions 0 assignees View on GitHub
performance
Dominant language
TypeScript
Stars
351
Forks
48
Avg merge
10h 21m
Merged PRs (30d)
1

Description

Right now, dynamic edges and nodes data seem to be properly cached, but components have to merge them with the static data themselves, using `mergeStaticDynamicData`.

This helper function is not free, and is called very often in various places, such as:
- [Node.tsx](https://github.com/gephi/gephi-lite/blob/41a67b523610a7936ed2a8f003bf8605f3f8f164/packages/gephi-lite/src/components/data/Node.tsx#L35-L48) and [Edge.tsx](https://github.com/gephi/gephi-lite/blob/41a67b523610a7936ed2a8f003bf8605f3f8f164/packages/gephi-lite/src/components/data/Edge.tsx#L55-L78)
- [RangeFilter.tsx](https://github.com/gephi/gephi-lite/blob/6469a7bc11b5fca854b527de7ce1ffc9d01ea427/packages/gephi-lite/src/components/GraphFilters/RangeFilter.tsx#L54-L60) and [TermsFilter.tsx](https://github.com/gephi/gephi-lite/blob/9bfcbe0ad6f2ea2ba7a3504054d69470641e6030/packages/gephi-lite/src/components/GraphFilters/TermsFilter.tsx#L21-L27)
- [Selection.tsx](https://github.com/gephi/gephi-lite/blob/1e4e890192590044e151fe300a2b3cd36d4469b5/packages/gephi-lite/src/views/graphPage/Selection.tsx#L218-L223) (which itself then calls Node.tsx and/or Edge.tsx)

We probably could cache the merged items data, to prevent having to reproduce these merged collections all the time.

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.