Skip to content

Display serialization factor in summary of all experiments #272

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

Merged
merged 8 commits into from
Mar 7, 2023

Conversation

erichaagdev
Copy link
Member

@erichaagdev erichaagdev commented Jan 3, 2023

❄️ Winter Update PR Navigator ❄️

  1. Include realized build time savings in experiment summary #261
  2. Include performance characteristics and support -x for Gradle experiment 1  #262
  3. Always read project name from Build Scan data #263
  4. Gradle init script improvements #264
  5. Adopt Foojay Toolchains and Kotlin DSL plugins in build logic #265
  6. Add variable to query debug mode #266
  7. Modernize Java launch code #267
  8. Display serialization factor in summary of all experiments #272
  9. Display potential build time savings in experiment summaries #274
  10. Update the contract of command line tools #283

⛄ = you are here

Summary

ℹ️ This PR continues the implementation of #206. This felt like a good place to stop. More PRs will be coming.

This PR updates the summary of all experiments to display the serialization factor of the first build. For example:

image

Testing

To test this, I ran the following experiments and verified they displayed the serialization factor.

./01-validate-incremental-building.sh -r git@github.com:junit-team/junit5.git -t 'assemble' -s https://ge.solutions-team.gradle.com --debug
./01-validate-incremental-building.sh -r git@github.com:gradle/ge-solutions.git -p sample-projects/gradle/7.x/ge -t 'build' -x --debug
./02-validate-local-build-caching-same-location.sh -r git@github.com:junit-team/junit5.git -t 'assemble' -s https://ge.solutions-team.gradle.com --debug
./02-validate-local-build-caching-same-location.sh -r git@github.com:gradle/ge-solutions.git -p sample-projects/gradle/7.x/ge -t 'build' -x --debug
./03-validate-local-build-caching-different-locations.sh -r git@github.com:junit-team/junit5.git -t 'assemble' -s https://ge.solutions-team.gradle.com --debug
./03-validate-local-build-caching-different-locations.sh -r git@github.com:gradle/ge-solutions.git -p sample-projects/gradle/7.x/ge -t 'build' -x --debug
./04-validate-remote-build-caching-ci-ci.sh -1 https://ge.solutions-team.gradle.com/s/atkuswzf5tqbe -2 https://ge.solutions-team.gradle.com/s/gso4oyqjeri4e --debug
./05-validate-remote-build-caching-ci-local.sh -1 https://ge.solutions-team.gradle.com/s/us4sfslrqlfnc -s https://ge.solutions-team.gradle.com --debug
./01-validate-local-build-caching-same-location.sh -r git@github.com:xwiki/xwiki-commons.git -g 'package' -a '-DskipTests' -s https://ge.solutions-team.gradle.com --debug
./01-validate-local-build-caching-same-location.sh -r git@github.com:gradle/ge-solutions.git -p sample-projects/maven/3.8.x/ge -g 'package' -x --debug
./02-validate-local-build-caching-different-locations.sh -r git@github.com:xwiki/xwiki-commons.git -g 'package' -a '-DskipTests' -s https://ge.solutions-team.gradle.com --debug
./02-validate-local-build-caching-different-locations.sh -r git@github.com:gradle/ge-solutions.git -p sample-projects/maven/3.8.x/ge -g 'package' -x --debug
./03-validate-remote-build-caching-ci-ci.sh -1 https://ge.solutions-team.gradle.com/s/o7i5lmq4vdckm -2 https://ge.solutions-team.gradle.com/s/qqfvwh2rft6yg --debug
./04-validate-remote-build-caching-ci-local.sh -1 https://ge.solutions-team.gradle.com/s/ehiqsjd2wh5qs -s https://ge.solutions-team.gradle.com --debug

@runningcode runningcode removed their request for review March 2, 2023 09:03
@erichaagdev erichaagdev force-pushed the erichaagdev/modernize-java-launch-code branch from 7dc138b to c65b367 Compare March 6, 2023 00:39
@erichaagdev erichaagdev force-pushed the erichaagdev/display-serialization-factor branch from 0404f37 to e31a63d Compare March 6, 2023 00:44
Base automatically changed from erichaagdev/modernize-java-launch-code to main March 7, 2023 20:33
@erichaagdev erichaagdev merged commit 322d555 into main Mar 7, 2023
@erichaagdev erichaagdev deleted the erichaagdev/display-serialization-factor branch March 7, 2023 21:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants