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

[DataGrid] Conditional autoHeight experiment #9274

Closed
wants to merge 1 commit into from

Conversation

cherniavskii
Copy link
Member

@cherniavskii cherniavskii commented Jun 8, 2023

@cherniavskii cherniavskii added the component: data grid This is the name of the generic UI component, not the React module! label Jun 8, 2023
@mui-bot
Copy link

mui-bot commented Jun 8, 2023

Netlify deploy preview

Netlify deploy preview: https://deploy-preview-9274--material-ui-x.netlify.app/

Updated pages

These are the results for the performance tests:

Test case Unit Min Max Median Mean σ
Filter 100k rows ms 570.8 1,124.2 623.5 774.48 194.344
Sort 100k rows ms 518.7 1,085.5 726 831.46 193.791
Select 100k rows ms 184.2 396.7 222.9 251.8 75.795
Deselect 100k rows ms 135.7 270.2 195.8 201.98 54.401

Generated by 🚫 dangerJS against ca1b2ef

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

github-actions bot commented Sep 6, 2023

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

@michelengelen michelengelen changed the base branch from master to next November 6, 2023 14:27
@MBilalShafi MBilalShafi changed the base branch from next to master March 21, 2024 02:38
@marcpachecog
Copy link

Hi @cherniavskii, I hope you’re doing well! I was wondering if there are any updates on this issue. It's a common scenario, and many people would greatly appreciate a solution. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component: data grid 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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants