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

[core] Use international locale format #3921

Merged
merged 1 commit into from
Feb 11, 2022

Conversation

oliviertassinari
Copy link
Member

@oliviertassinari oliviertassinari commented Feb 10, 2022

The intent is to update

Screenshot 2022-02-10 at 23 21 20

#3211

to show ### ar-SD instead of ### arSD. I'm trying to solve the root cause of #3831 (review).

@oliviertassinari oliviertassinari added the core Infrastructure work going on behind the scenes label Feb 10, 2022
@mui-bot
Copy link

mui-bot commented Feb 10, 2022

These are the results for the performance tests:

Test case Unit Min Max Median Mean σ
Filter 100k rows ms 216.7 380.4 287.5 298.56 58.243
Sort 100k rows ms 418.2 952.4 696.3 721.52 190.287
Select 100k rows ms 183.6 270.7 199.4 214.38 33.408
Deselect 100k rows ms 120.9 213 160.3 164.22 34.73

Generated by 🚫 dangerJS against c39df6c

@oliviertassinari oliviertassinari merged commit 4a91433 into mui:master Feb 11, 2022
@oliviertassinari oliviertassinari deleted the locale-format branch February 11, 2022 10:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
core Infrastructure work going on behind the scenes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants