Skip to content

[charts-pro] add benchmark for zoomed in scatter chart #17756

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

Merged
merged 6 commits into from
May 9, 2025

Conversation

bernardobelchior
Copy link
Member

@bernardobelchior bernardobelchior commented May 8, 2025

Add benchmark for zoomed in scatter chart.

Used to compare with this performance improvement.

@bernardobelchior bernardobelchior added performance enhancement This is not a bug, nor a new feature component: charts This is the name of the generic UI component, not the React module! labels May 8, 2025
@mui-bot
Copy link

mui-bot commented May 8, 2025

Deploy preview: https://deploy-preview-17756--material-ui-x.netlify.app/

Generated by 🚫 dangerJS against 089ed61

Copy link

codspeed-hq bot commented May 8, 2025

CodSpeed Performance Report

Merging #17756 will not alter performance

Comparing bernardobelchior:scatter-zoom-in-bench (089ed61) with master (add0d27)

Summary

✅ 8 untouched benchmarks
🆕 1 new benchmarks

Benchmarks breakdown

Benchmark BASE HEAD Change
🆕 ScatterChartPro with big data amount and zoomed in N/A 7.5 s N/A

@bernardobelchior bernardobelchior force-pushed the scatter-zoom-in-bench branch from 146474b to cb6063f Compare May 9, 2025 11:12
@bernardobelchior bernardobelchior force-pushed the scatter-zoom-in-bench branch from 8830e91 to f27c07f Compare May 9, 2025 12:30
@JCQuintas JCQuintas merged commit 910366e into mui:master May 9, 2025
24 checks passed
@bernardobelchior bernardobelchior deleted the scatter-zoom-in-bench branch May 9, 2025 17:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component: charts This is the name of the generic UI component, not the React module! enhancement This is not a bug, nor a new feature performance
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants