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

Engine benchmark visualization tool can compare two bench runs #6198

Merged
merged 4 commits into from
Apr 5, 2023

Conversation

Akirathan
Copy link
Member

@Akirathan Akirathan commented Apr 4, 2023

Pull Request Description

Add --compare <bench-run-id-1> <bench-run-id-2> option to bench_download.py script, that prints the difference of benchmark scores for two benchmark runs from GH as a table.

Important Notes

I tried to add an option to compare two or more branches and visualize them in the graphs, but gave that up after some struggling.

Checklist

Please ensure that the following checklist has been satisfied before submitting the PR:

  • The documentation has been updated, if necessary.
  • Screenshots/screencasts have been attached, if there are any visual changes. For interactive or animated visual changes, a screencast is preferred.
  • All code follows the
    Scala,
    Java,
    and
    Rust
    style guides. In case you are using a language not listed above, follow the Rust style guide.
  • All code has been tested:
    • Unit tests have been written where possible.
    • If GUI codebase was changed, the GUI was tested when built using ./run ide build.

@Akirathan Akirathan requested a review from 4e6 as a code owner April 4, 2023 15:54
@Akirathan Akirathan linked an issue Apr 4, 2023 that may be closed by this pull request
@Akirathan Akirathan self-assigned this Apr 4, 2023
@Akirathan Akirathan added the CI: No changelog needed Do not require a changelog entry for this PR. label Apr 4, 2023
@Akirathan Akirathan added the CI: Ready to merge This PR is eligible for automatic merge label Apr 5, 2023
@mergify mergify bot merged commit c7da5f8 into develop Apr 5, 2023
@mergify mergify bot deleted the wip/akirathan/bench-compare-branches-6110 branch April 5, 2023 19:03
Procrat added a commit that referenced this pull request Apr 6, 2023
* develop:
  Project Sharing (#6077)
  Adjust `{Table|Column}.parse` to use `Value_Type` (#6213)
  Add cloud endpoints for frontend (#6002)
  Implement `Table.union` for Database backend (#6204)
  Batch insert suggestions (#6189)
  Formatter fix to not fail when encountering an invalid symlink. (#6172)
  Suspended atom fields are evaluated only once (#6151)
  Text.to_display_text is (shortened) identity (#6174)
  Engine benchmark visualization tool can compare two bench runs (#6198)
  Add PRIVATE so function hidden from Component Browser and other tidying... (#6207)
  Hotfix for #6203. (#6210)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI: No changelog needed Do not require a changelog entry for this PR. CI: Ready to merge This PR is eligible for automatic merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add ability to compare branches to bench_download tool
2 participants