Skip to content

Add PerformanceStatsHistory to perf monitor overlay - #39202

Closed
ryancat wants to merge 1 commit into
react:mainfrom
ryancat:export-D48657874
Closed

Add PerformanceStatsHistory to perf monitor overlay#39202
ryancat wants to merge 1 commit into
react:mainfrom
ryancat:export-D48657874

Conversation

@ryancat

Copy link
Copy Markdown
Contributor

Summary:
This diff adds PerformanceStatsHistory.kt which is used to track performance stat in buckets with resolution (in ms) and size. This is to help group perf stats in a time series data structure that we can adapt to any visualization in the future.

Changelog:
[Internal] - Performance monitoring overlay improvements

Differential Revision: D48657874

@facebook-github-botfacebook-github-bot added CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. p: Facebook Partner: Facebook Partner labels Aug 30, 2023
@facebook-github-bot

Copy link
Copy Markdown
Contributor

This pull request was exported from Phabricator. Differential Revision: D48657874

@analysis-bot

analysis-bot commented Aug 30, 2023

Copy link
Copy Markdown
PlatformEngineArchSize (bytes)Diff
androidhermesarm64-v8a8,963,039+268
androidhermesarmeabi-v7an/a--
androidhermesx86n/a--
androidhermesx86_64n/a--
androidjscarm64-v8a9,555,744-268
androidjscarmeabi-v7an/a--
androidjscx86n/a--
androidjscx86_64n/a--

Base commit: a5fd0f1
Branch: main

@facebook-github-bot

Copy link
Copy Markdown
Contributor

This pull request was exported from Phabricator. Differential Revision: D48657874

Summary:
Pull Request resolved: react#39202
This diff adds `PerformanceStatsHistory.kt` which is used to track performance stat in buckets with resolution (in ms) and size. This is to help group perf stats in a time series data structure that we can adapt to any visualization in the future.
Changelog:
[Internal] - Performance monitoring overlay improvements
Reviewed By: rshest
Differential Revision: D48657874
fbshipit-source-id: 08e115f0387d167aa9a5710b0dcf79378d426e9a
@facebook-github-bot

Copy link
Copy Markdown
Contributor

This pull request was exported from Phabricator. Differential Revision: D48657874

@github-actions

Copy link
Copy Markdown
Warnings
⚠️

E2E tests for ci/circleci: test_e2e_android failed with errors. See the logs for details

⚠️

E2E tests for ci/circleci: test_e2e_android failed with errors. See the logs for details

Generated by 🚫 dangerJS against 2782f6b

@github-actions

Copy link
Copy Markdown

This pull request was successfully merged by Xin Chen in c99c653.

When will my fix make it into a release? | Upcoming Releases

@github-actionsgithub-actionsBot added the Merged This PR has been merged. label Aug 31, 2023
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA SignedThis label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.fb-exportedMergedThis PR has been merged.p: FacebookPartner: FacebookPartner

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@ryancat@facebook-github-bot@analysis-bot