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

Refactoring of data layer framework #893

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

Refactoring of data layer framework #893

wants to merge 24 commits into from
+6,638 −5,839

Conversation

rubenthoms
Copy link
Collaborator

@rubenthoms rubenthoms commented Mar 20, 2025 •
edited
Loading

  • Refactoring of Data Layer framework
    • Each custom data layer implementation in a single file now
    • Custom data layer and setting implementations reusable with parameters to constructors in their respective registries
    • Extended Visualization Factory to support visualization-specific annotations, bounding box calculations, hover visualizations, custom data injection, and data accumulation
  • Respective adjustments to 2DViewer module
  • New utilities in lib
  • New components for selecting color scales
  • New preview component for color scale

@rubenthoms rubenthoms self-assigned this Mar 20, 2025
@rubenthoms rubenthoms force-pushed the improvements-to-layer-framework branch from f7becbe to d85ed74 Compare March 21, 2025 14:03
@rubenthoms rubenthoms marked this pull request as ready for review March 21, 2025 14:27
@rubenthoms rubenthoms requested a review from jorgenherje March 21, 2025 14:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jorgenherje jorgenherje Awaiting requested review from jorgenherje

At least 1 approving review is required to merge this pull request.

Assignees

@rubenthoms rubenthoms

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

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