mui-x icon indicating copy to clipboard operation
mui-x copied to clipboard

[charts] Filter item outside the drawing area for perf

Open alexfauquette opened this issue 1 year ago • 2 comments

I noticed that zooming on 5% of the data takes as much time to render than a full zoom out. The more we zoom the faster it should render since less elements are to plot

For Bar charts, zoning on 5% of a 1,000 bars (only shows 50 bars out of the 1,000) makes the rendering time move from 100ms to 28ms

alexfauquette avatar Aug 21 '24 13:08 alexfauquette

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

Generated by :no_entry_sign: dangerJS against db09b3dc25f1912fac15fd3c6da402437c86ae3d

mui-bot avatar Aug 21 '24 14:08 mui-bot

CodSpeed Performance Report

Merging #14281 will not alter performance

Comparing alexfauquette:filter-with-zoom (db09b3d) with master (7c4c924)

Summary

✅ 3 untouched benchmarks

🆕 3 new benchmarks

Benchmarks breakdown

Benchmark master alexfauquette:filter-with-zoom Change
🆕 BarChartPro with big data amount N/A 351.8 ms N/A
🆕 LineChartPro with big data amount N/A 534.5 ms N/A
🆕 ScatterChartPro with big data amount N/A 151.6 ms N/A

codspeed-hq[bot] avatar Aug 21 '24 14:08 codspeed-hq[bot]

This pull request has conflicts, please resolve those before we can evaluate the pull request.

github-actions[bot] avatar Nov 07 '24 15:11 github-actions[bot]

This pull request has conflicts, please resolve those before we can evaluate the pull request.

github-actions[bot] avatar Nov 11 '24 09:11 github-actions[bot]