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

[charts] Test pro components without zooming #15469

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

JCQuintas
Copy link
Member

@JCQuintas JCQuintas added test component: charts This is the name of the generic UI component, not the React module! labels Nov 18, 2024
@JCQuintas JCQuintas self-assigned this Nov 18, 2024
@mui-bot
Copy link

mui-bot commented Nov 18, 2024

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

Generated by 🚫 dangerJS against 9c68cd5

Copy link

codspeed-hq bot commented Nov 18, 2024

CodSpeed Performance Report

Merging #15469 will degrade performances by 72.9%

Comparing JCQuintas:test-pro-components-without-zooming (9c68cd5) with master (e151681)

Summary

❌ 4 regressions
✅ 2 untouched benchmarks

🆕 3 new benchmarks

⚠️ Please fix the performance issues or acknowledge them on CodSpeed.

Benchmarks breakdown

Benchmark master JCQuintas:test-pro-components-without-zooming Change
BarChartPro with big data amount 360.5 ms 1,330 ms -72.9%
🆕 BarChartPro with big data amount zoomed N/A 150.5 ms N/A
LineChart with big data amount 854.9 ms 945.8 ms -9.61%
LineChartPro with big data amount 540.9 ms 939.1 ms -42.4%
🆕 LineChartPro with big data amount zoomed N/A 166.7 ms N/A
ScatterChartPro with big data amount 185.3 ms 569.6 ms -67.47%
🆕 ScatterChartPro with big data amount zoomed N/A 142.7 ms N/A

@github-actions github-actions bot added the PR: out-of-date The pull request has merge conflicts and can't be merged label Dec 23, 2024
Copy link

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

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! PR: out-of-date The pull request has merge conflicts and can't be merged test
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants