Skip to content
This repository has been archived by the owner on Mar 4, 2020. It is now read-only.

docs: Add component perf charts to public docsite #2240

Merged
merged 3 commits into from
Jan 15, 2020

Conversation

miroslavstastny
Copy link
Member

@miroslavstastny miroslavstastny commented Jan 14, 2020

Enable per-component perf charts in production docsite.
Filter results to public builds only.

image

Depends on changes in GetPerfStats: miroslavstastny/stardust-stats-azure/pull/5

@@ -1,5 +1,5 @@
const config = {
getStatsUri: 'https://stardust.azurewebsites.net/api/GetPerfStats',
getStatsUri: 'https://fluent-ui-react-stats.azurewebsites.net/api/GetPerfStats',
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

API moved to fluent-ui

}
const query = process.env.NODE_ENV === 'production' ? '' : '?withPrivateBuilds=true'

fetch(`${config.getStatsUri}${query}`)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would like to use URL, but we can't as it's not supported by IE11 🙃

@DustyTheBot
Copy link
Collaborator

DustyTheBot commented Jan 14, 2020

Perf comparison

Status Scenario Fluent TPI Fabric TPI Ratio Iterations Ticks
🔧 Avatar.Fluent 0.54 0.36 1.5:1 2000 1087
🔧 Button.Fluent 1.19 0.16 7.44:1 1000 1191
🔧 Checkbox.Fluent 1.39 0.28 4.96:1 1000 1394
🔧 Dialog.Fluent 0.32 0.16 2:1 5000 1577
🔧 Dropdown.Fluent 3.17 0.35 9.06:1 1000 3170
🔧 Icon.Fluent 0.24 0.03 8:1 5000 1191
🔧 Image.Fluent 0.1 0.07 1.43:1 5000 522
🔧 Slider.Fluent 1.79 0.31 5.77:1 1000 1793
🦄 Text.Fluent 0.05 0.15 0.33:1 5000 263
🦄 Tooltip.Fluent 0.37 17.9 0.02:1 5000 1855

🔧 Needs work     🎯 On target     🦄 Amazing

Generated by 🚫 dangerJS

@miroslavstastny miroslavstastny merged commit 6e8ab70 into master Jan 15, 2020
@miroslavstastny miroslavstastny deleted the doc/component-perf-charts branch January 15, 2020 08:55
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants