AllenCell / AllenCell/vole-core

internals: setState api and efficient render updates

Offen
#90 3 Kommentare 0 Reaktionen 1 zugewiesene Person Beansprucht von @frasercl Auf GitHub ansehen
Tech Debt
Vorherrschende Sprache
TypeScript
Sterne
106
Forks
11
Ø Merge
3 T. 51 Min.
Gemergte PRs (30 T.)
12

Beschreibung

## Use Case

As a developer, the public api is too spread out over many functions that have partial side effects.
We would like to be able to update large chunks of state with only one function call.

## Solution

Copy the data structure from https://github.com/allen-cell-animated/website-3d-cell-viewer/issues/110 down into volume-viewer and export it.
Partially implement the react-like state update mechanism as discussed in Teams call in volume-viewer, starting only with VolumeOptions (our temporary name for the appearance and channel editing information).
We believe this can be implemented at low technical risk as an alternate code path that doesn't modify any of the other public api or internal Drawable classes.
After some nontrivial tbd amount of functionality is in, we can assess the impact on performance and code maintainability/readability.

The end goal which may require followup issues is to have all settings updated via this mechanism and remove the individual setters in the api. A requirement for the final goal will be to have good documentation of the whole data structure and a clear path forward for future modifications.

Beitragsleitfaden

Beitragsleitfaden öffnen

Bewertung

Dieses Issue wurde noch nicht bewertet.

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.