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

perf: Optimize memory for plot tooltips #1876

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

Conversation

marek-mihok
Copy link
Contributor

@marek-mihok marek-mihok commented Mar 7, 2023

Introduces the unpackByIdx function for h2o-wave package which unlike unpack function unpacks only the part of data on the specific index which has a performance benefit for displaying plot tooltip.

@marek-mihok marek-mihok requested review from lo5 and mturoci as code owners March 7, 2023 12:56
@mturoci mturoci changed the title feat: Optimize memory performance for plot tooltips perf: Optimize memory for plot tooltips Mar 7, 2023
Copy link
Collaborator

@mturoci mturoci left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @marek-mihok.

ts/index.ts Outdated Show resolved Hide resolved
ui/vite.config.js Outdated Show resolved Hide resolved
ts/index.ts Outdated Show resolved Hide resolved
ts/index.ts Outdated Show resolved Hide resolved
ts/index.ts Outdated Show resolved Hide resolved
ts/index.ts Outdated Show resolved Hide resolved
ts/package.json Outdated Show resolved Hide resolved
ui/src/plot.tsx Outdated Show resolved Hide resolved
ts/index.ts Outdated Show resolved Hide resolved
ts/index.ts Outdated Show resolved Hide resolved
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 this pull request may close these issues.

2 participants