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

adding lines in vertical stacked bar chart complex combo chart (#15551) #16639

Merged
merged 1 commit into from
Jan 27, 2021

Conversation

kubkon
Copy link
Contributor

@kubkon kubkon commented Jan 26, 2021

  • adding lines in the vertical stacked bar chart

  • Change files

  • fix build errors

  • fix build erros

  • when only the line is highlighted we should show callout to that

  • fix build issues

  • making the callout sort according to the rects

  • updating snapshots

Co-authored-by: v-sivsar v-sivsar@microsoft.com

@kubkon kubkon requested a review from Raghurk as a code owner January 26, 2021 14:39
@kubkon kubkon added this to To do in Version 8 cherry-picks via automation Jan 26, 2021
@kubkon kubkon moved this from To do to In progress in Version 8 cherry-picks Jan 26, 2021
@codesandbox-ci
Copy link

codesandbox-ci bot commented Jan 26, 2021

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit 8c73eb3:

Sandbox Source
Fluent UI Button Configuration
codesandbox-react-template Configuration
codesandbox-react-northstar-template Configuration

@fabricteam
Copy link
Collaborator

fabricteam commented Jan 26, 2021

Perf Analysis

No significant results to display.

All results

Scenario Render type Master Ticks PR Ticks Iterations Status
Avatar mount 846 855 5000
BaseButtonCompat mount 910 905 5000
Breadcrumb mount 43805 43812 5000
Checkbox mount 1488 1495 5000
CheckboxBase mount 1244 1260 5000
ChoiceGroup mount 4728 4754 5000
ComboBox mount 970 981 1000
CommandBar mount 10206 10203 1000
ContextualMenu mount 6077 6098 1000
DefaultButtonCompat mount 1102 1122 5000
DetailsRow mount 3641 3693 5000
DetailsRowFast mount 3625 3684 5000
DetailsRowNoStyles mount 3433 3433 5000
Dialog mount 1432 1465 1000
DocumentCardTitle mount 1843 1842 1000
Dropdown mount 3337 3308 5000
FocusTrapZone mount 1849 1832 5000
FocusZone mount 1938 1859 5000
IconButtonCompat mount 1760 1749 5000
Label mount 342 351 5000
Layer mount 1773 1784 5000
Link mount 459 457 5000
MakeStyles mount 2173 2227 50000
MenuButtonCompat mount 1451 1453 5000
MessageBar mount 2020 1993 5000
Nav mount 3300 3277 1000
OverflowSet mount 1069 1047 5000
Panel mount 1412 1442 1000
Persona mount 860 858 1000
Pivot mount 1391 1398 1000
PrimaryButtonCompat mount 1281 1267 5000
Rating mount 7542 7473 5000
SearchBox mount 1303 1339 5000
Shimmer mount 2641 2494 5000
Slider mount 1945 1894 5000
SpinButton mount 4954 4946 5000
Spinner mount 429 415 5000
SplitButtonCompat mount 3108 3138 5000
Stack mount 523 500 5000
StackWithIntrinsicChildren mount 1557 1540 5000
StackWithTextChildren mount 4449 4462 5000
SwatchColorPicker mount 10279 10265 5000
Tabs mount 1390 1405 1000
TagPicker mount 2840 2852 5000
TeachingBubble mount 11764 11654 5000
Text mount 419 416 5000
TextField mount 1358 1364 5000
ThemeProvider mount 2201 2155 5000
ThemeProvider virtual-rerender 641 662 5000
Toggle mount 787 808 5000
button mount 686 699 5000
buttonNative mount 112 101 5000

Perf Analysis (Fluent)

Perf comparison
Status Scenario Fluent TPI Fabric TPI Ratio Iterations Ticks
🦄 Avatar.Fluent 0.18 0.51 0.35:1 2000 356
🦄 Button.Fluent 0.11 0.2 0.55:1 5000 568
🔧 Checkbox.Fluent 0.65 0.35 1.86:1 1000 653
🎯 Dialog.Fluent 0.17 0.22 0.77:1 5000 835
🔧 Dropdown.Fluent 3.06 0.4 7.65:1 1000 3062
🔧 Icon.Fluent 0.14 0.06 2.33:1 5000 707
🦄 Image.Fluent 0.08 0.12 0.67:1 5000 405
🔧 Slider.Fluent 1.63 0.44 3.7:1 1000 1626
🔧 Text.Fluent 0.07 0.03 2.33:1 5000 362
🦄 Tooltip.Fluent 0.12 0.89 0.13:1 5000 590

🔧 Needs work     🎯 On target     🦄 Amazing

Perf tests with no regressions
Scenario Current PR Ticks Baseline Ticks Ratio
AttachmentMinimalPerf.default 178 163 1.09:1
ListMinimalPerf.default 529 500 1.06:1
ChatDuplicateMessagesPerf.default 394 377 1.05:1
ChatWithPopoverPerf.default 458 435 1.05:1
FlexMinimalPerf.default 326 310 1.05:1
TableMinimalPerf.default 452 432 1.05:1
VideoMinimalPerf.default 664 635 1.05:1
AccordionMinimalPerf.default 170 164 1.04:1
ButtonUseCssPerf.default 857 824 1.04:1
ChatMinimalPerf.default 647 621 1.04:1
RadioGroupMinimalPerf.default 467 451 1.04:1
AnimationMinimalPerf.default 428 417 1.03:1
LayoutMinimalPerf.default 435 424 1.03:1
MenuMinimalPerf.default 906 881 1.03:1
TextAreaMinimalPerf.default 503 486 1.03:1
TooltipMinimalPerf.default 858 837 1.03:1
Icon.Fluent 707 689 1.03:1
DropdownManyItemsPerf.default 748 736 1.02:1
HeaderMinimalPerf.default 393 385 1.02:1
InputMinimalPerf.default 1351 1323 1.02:1
LabelMinimalPerf.default 441 433 1.02:1
ReactionMinimalPerf.default 425 415 1.02:1
SliderMinimalPerf.default 1624 1597 1.02:1
Avatar.Fluent 356 350 1.02:1
Checkbox.Fluent 653 643 1.02:1
BoxMinimalPerf.default 379 376 1.01:1
ButtonMinimalPerf.default 183 182 1.01:1
ButtonOverridesMissPerf.default 1759 1743 1.01:1
DatepickerMinimalPerf.default 49681 49105 1.01:1
DividerMinimalPerf.default 408 404 1.01:1
FormMinimalPerf.default 429 425 1.01:1
TableManyItemsPerf.default 2079 2061 1.01:1
ToolbarMinimalPerf.default 968 963 1.01:1
TreeMinimalPerf.default 799 792 1.01:1
Slider.Fluent 1626 1616 1.01:1
AvatarMinimalPerf.default 214 215 1:1
ButtonSlotsPerf.default 600 598 1:1
CheckboxMinimalPerf.default 2906 2911 1:1
DropdownMinimalPerf.default 3047 3033 1:1
HeaderSlotsPerf.default 795 798 1:1
ItemLayoutMinimalPerf.default 1255 1249 1:1
LoaderMinimalPerf.default 742 739 1:1
PopupMinimalPerf.default 721 723 1:1
SplitButtonMinimalPerf.default 3793 3793 1:1
StatusMinimalPerf.default 759 757 1:1
IconMinimalPerf.default 690 688 1:1
Dropdown.Fluent 3062 3059 1:1
Tooltip.Fluent 590 589 1:1
AlertMinimalPerf.default 298 302 0.99:1
AttachmentSlotsPerf.default 1215 1228 0.99:1
ButtonUseCssNestingPerf.default 1108 1114 0.99:1
EmbedMinimalPerf.default 4215 4243 0.99:1
ListNestedPerf.default 591 598 0.99:1
PortalMinimalPerf.default 170 171 0.99:1
ProviderMergeThemesPerf.default 1659 1668 0.99:1
ProviderMinimalPerf.default 993 1007 0.99:1
TextMinimalPerf.default 373 377 0.99:1
Dialog.Fluent 835 841 0.99:1
CardMinimalPerf.default 568 581 0.98:1
DialogMinimalPerf.default 814 831 0.98:1
MenuButtonMinimalPerf.default 1601 1627 0.98:1
SkeletonMinimalPerf.default 387 395 0.98:1
CustomToolbarPrototype.default 3824 3901 0.98:1
ListWith60ListItems.default 601 621 0.97:1
RefMinimalPerf.default 236 244 0.97:1
TreeWith60ListItems.default 184 189 0.97:1
Button.Fluent 568 585 0.97:1
Image.Fluent 405 417 0.97:1
Text.Fluent 362 375 0.97:1
GridMinimalPerf.default 357 375 0.95:1
ListCommonPerf.default 638 670 0.95:1
CarouselMinimalPerf.default 511 546 0.94:1
ImageMinimalPerf.default 381 409 0.93:1
SegmentMinimalPerf.default 357 390 0.92:1
RosterPerf.default 1158 1294 0.89:1

@size-auditor
Copy link

size-auditor bot commented Jan 26, 2021

Asset size changes

⚠️ Insufficient baseline data to detect size changes

Unable to find bundle size details for Baseline commit: cc0450e

Possible causes

  • The baseline build cc0450e is broken
  • The Size Auditor run for the baseline build cc0450e was not triggered

Recommendations

  • Please merge your branch for this Pull request with the latest master build and commit your changes once again

@ecraig12345
Copy link
Member

I suspect you're missing snapshot updates. You'll need to merge with master (which has an update-snapshots fix) then:

yarn
yarn buildto react-examples
yarn workspace @fluentui/react-charting update-snapshots
# maybe, not sure if this affects example snapshots
yarn workspace @fluentui/react-examples update-snapshots

Copy link
Member

@ecraig12345 ecraig12345 left a comment

Choose a reason for hiding this comment

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

Once the snapshots are fixed you can have an admin merge this (I think @layershifter should be able to do that if you don't want to wait for US time)

…soft#15551)

* adding lines in the vertical stacked bar chart

* Change files

* fix build errors

* fix build erros

* when only the line is highlighted we should show callout to that

* fix build issues

* making the callout sort according to the rects

* updating snapshots

Co-authored-by: v-sivsar <v-sivsar@microsoft.com>
@Hotell Hotell merged commit f5914d8 into microsoft:master Jan 27, 2021
Version 8 cherry-picks automation moved this from In progress to Done Jan 27, 2021
@kubkon kubkon deleted the cherry-15551 branch January 27, 2021 12:57
@msft-fluent-ui-bot
Copy link
Collaborator

🎉@fluentui/react-charting@v5.0.0-beta.44 has been released which incorporates this pull request.:tada:

Handy links:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

None yet

5 participants