Skip to content

[feature/#719] Add Benchmark Module and baseline profile generator#1011

Open
l2hyunwoo wants to merge 2 commits intoDroidKaigi:mainfrom
l2hyunwoo:feature/#719
Open

[feature/#719] Add Benchmark Module and baseline profile generator#1011
l2hyunwoo wants to merge 2 commits intoDroidKaigi:mainfrom
l2hyunwoo:feature/#719

Conversation

@l2hyunwoo
Copy link
Contributor

@l2hyunwoo l2hyunwoo commented Aug 29, 2023

Issue

Overview (Required)

  • Add Benchmark Module and baseline profile generator
  • There's an issue

@l2hyunwoo l2hyunwoo requested a review from a team as a code owner August 29, 2023 12:36
@l2hyunwoo l2hyunwoo requested a review from RyuNen344 August 29, 2023 12:36
@l2hyunwoo l2hyunwoo marked this pull request as draft August 29, 2023 12:36
@github-actions
Copy link

github-actions bot commented Aug 29, 2023 •
edited
Loading

Test Results

216 tests   216 ✔️  7m 11s ⏱️
  11 suites      0 💤
  11 files        0 ❌

Results for commit cefc3c5.

♻️ This comment has been updated with latest results.

@takahirom takahirom self-requested a review August 29, 2023 15:28
takahirom
takahirom reviewed Aug 29, 2023
View reviewed changes
app-android/src/devRelease/generated/baselineProfiles/baseline-prof.txt
Copy link
Member

@takahirom takahirom Aug 29, 2023

Choose a reason for hiding this comment

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

Sorry I'm going to create this by myself without committing so could you remove this?

Copy link
Contributor Author

@l2hyunwoo l2hyunwoo Aug 29, 2023

Choose a reason for hiding this comment

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

It's deleted

@takahirom
Copy link
Member

takahirom commented Aug 29, 2023

Let me squash this PR not to enter the baseline prof file 🙏

@l2hyunwoo l2hyunwoo marked this pull request as ready for review August 29, 2023 15:57
@DroidKaigi DroidKaigi deleted a comment from github-actions bot Aug 29, 2023
@takahirom
Copy link
Member

takahirom commented Aug 30, 2023

@l2hyunwoo Do you have any idea why this CI takes 1 hour? 👀

@l2hyunwoo
Copy link
Contributor Author

l2hyunwoo commented Aug 31, 2023

@takahirom It's hard to find the reason why it takes much longer than others..... Reasonable reason that I can think is there's no build cache related to baseline-profile. So if build cache is created, build time will restore I suppose.

+) One more I think is there're procedures that related to prod variant, kind of strange things....(I thought it just assembleDev..) It could increase build time much more
스크린샷 2023-08-31 오후 11 13 32

@takahirom
Copy link
Member

takahirom commented Sep 2, 2023

@l2hyunwoo
Well, you can try to use increased build time for the GitHub Action. 🙏

@takahirom takahirom mentioned this pull request Sep 16, 2023
Open
@l2hyunwoo
[feature/DroidKaigi#719] Add baselineprofile module
3cef2be 
[feature/DroidKaigi#719] Add baseline-prof when variant is dev


[feature/DroidKaigi#719] Delete baseline-prof


[feature/DroidKaigi#719] Fix local build
@github-actions
Copy link

github-actions bot commented May 27, 2024

Hi @l2hyunwoo! Codes seem to be unformatted. To resolve this issue, please run ./gradlew detekt --auto-correct and fix the results of ./gradlew lintDebug.. Thank you for your contribution.

@github-actions github-actions bot had a problem deploying to deploygate-distribution May 27, 2024 04:12 Failure
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@takahirom takahirom takahirom left review comments

@RyuNen344 RyuNen344 Awaiting requested review from RyuNen344 RyuNen344 is a code owner automatically assigned from DroidKaigi/conferenceapp2023reviewer

At least 0 approving reviews are required to merge this pull request.

Assignees

No one assigned

Labels

awesome

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Add benchmark module and setup Baseline Profile generator

2 participants

@l2hyunwoo @takahirom