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.

Sign up for GitHub

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

Jump to bottom

fix(perf): support performance api #3341

Closed
wants to merge 50 commits into from
Closed

fix(perf): support performance api #3341

wants to merge 50 commits into from

Conversation

etkmao
Copy link
Collaborator

@etkmao etkmao commented Jun 30, 2023

Pre-PR Checklist

  • I added/updated relevant documentation.
  • I followed the Convention Commit guideline with maximum 72 characters to submit commit message.
  • I squashed the repeated code commits.
  • I signed the [CLA].
  • I added/updated test cases to check the change I am making.
  • All existing and new tests are passing.

etkmao and others added 30 commits June 8, 2023 11:04
fix(perf): support performance api
d92e048
fix(perf): support performance api
2fb52f5
fix(perf): support performance api
49129b8
fix(perf): support performance api
97120ad
fix(perf): support performance api
55adca2
fix(perf): support performance api
ad14473
@ozonelmy
feat(ios): add bridge time point performance
760bf8b
fix(perf): support performance api
6935879
fix(perf): support performance api
a824bba
Merge branch 'v3.0-dev' into performance_api
e99bf53
fix(perf): support performance api
5b5b7f1
Merge branch 'v3.0-dev' into performance_api
17cd64c
@ozonelmy
feat(ios): remove old ios performance logger
0654974
@ozonelmy
feat(ios): add performance point for engine start/end
118f443
fix(perf): support performance api
fc6e169
@ozonelmy
feat(ios): engine init point and execute source point and resource point
148ec7d
Merge branch 'v3.0-dev' into performance_api
aae5494
fix(perf): support performance api
68a82bb
fix(perf): support performance api
2984f6e
Merge branch 'v3.0-dev' into performance_api
d960294
@ozonelmy
feat(ios): add first frame naviation timing point
d3ea62c
Merge branch 'v3.0-dev' into performance_api
62b0f62 
# Conflicts:
#	framework/ios/base/executors/HippyJSExecutor.mm
fix(perf): support performance api
41e39d1
fix(perf): support performance api
7d44234
fix(perf): support performance api
cc6fc6a
@siguangli2018
feat(android): add performace impl
6fb5162
fix(perf): support performance api
558fd8a
Merge branch 'v3.0-dev' into performance_api
4620839
@ozonelmy
feat(ios): fix conflicts & add performance api
0ff0e7b
@ozonelmy
feat(ios): add performance api
78fb529
@github-actions github-actions bot added module: footstone module: ios size: xl Denotes a PR that changes 1000+ lines labels Jun 30, 2023
@hippy-service
Copy link

hippy-service bot commented Jun 30, 2023

Hi, @etkmao. Thanks for your PR! 👏

🏷️ You can leave a comment in this PR with #help tag when you need help (e.g. some status checks run failed due to internal issue), admin team members will help asap.

@etkmao etkmao changed the title Performance api fix(perf): support performance api Jun 30, 2023
ozonelmy
ozonelmy previously approved these changes Jul 3, 2023
View reviewed changes
@etkmao @zealotchen0
fix(jsc): fix jsc crash on ios12
d9ded26
@etkmao
Merge branch 'main' into performance_api
13ef5f9 
# Conflicts:
#	framework/ios/base/executors/HippyJSExecutor.mm
@etkmao etkmao requested a review from zealotchen0 as a code owner July 3, 2023 08:58
@etkmao etkmao requested review from lvfen and skindhu as code owners July 4, 2023 02:50
@hippy-service
Copy link

hippy-service bot commented Sep 9, 2023

Pay attention 🛎️ !!
There has been no activity on this PR for 2 months, so I will label it stalled.
It will be automatically closed in 30 days if no more activity. Feel free to leave a comment if you have any questions.

@hippy-service hippy-service bot added the stale: 60d No interaction in the last 60 days label Sep 9, 2023
@hippy-service Hippy Service
Copy link

hippy-service bot commented Nov 9, 2023

Sorry, closing this PR because it has stalled for over 4 months.
Feel free to reopen if this PR is still relevant, or to ping the collaborator if you have any questions.

@hippy-service hippy-service bot added the stale: closed Closed due to stale label Nov 9, 2023
@hippy-service hippy-service bot closed this Nov 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@skindhu skindhu skindhu approved these changes

@lvfen lvfen lvfen approved these changes

@ozonelmy ozonelmy ozonelmy left review comments

@ilikethese ilikethese Awaiting requested review from ilikethese ilikethese is a code owner

@siguangli siguangli Awaiting requested review from siguangli siguangli is a code owner

@zealotchen0 zealotchen0 Awaiting requested review from zealotchen0 zealotchen0 is a code owner

Assignees
No one assigned
Labels
build: xcode doc: pages driver: js framework: android framework: ios framework: voltron gh: workflow module: footstone module: ios module: vfs renderer: native size: xl Denotes a PR that changes 1000+ lines stale: closed Closed due to stale stale: 60d No interaction in the last 60 days
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
8 participants
@etkmao @skindhu @ozonelmy @lvfen @siguangli2018 @gguoyu @zealotchen0 @henryjin0511