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

Allow running benchmarks multiple times #719

Merged
merged 12 commits into from
Mar 13, 2024

Conversation

Ellpeck
Copy link
Member

@Ellpeck Ellpeck commented Mar 13, 2024

No description provided.

@Ellpeck Ellpeck linked an issue Mar 13, 2024 that may be closed by this pull request
Copy link

codecov bot commented Mar 13, 2024

Codecov Report

Attention: Patch coverage is 21.66667% with 47 lines in your changes are missing coverage. Please review.

Project coverage is 70.85%. Comparing base (84f42a6) to head (552db88).

Files Patch % Lines
.../util/summarizer/benchmark/second-phase/process.ts 19.23% 21 Missing ⚠️
src/util/summarizer/benchmark/summarizer.ts 20.00% 12 Missing ⚠️
...c/util/summarizer/benchmark/first-phase/process.ts 11.11% 8 Missing ⚠️
src/util/summarizer/benchmark/first-phase/input.ts 40.00% 6 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #719      +/-   ##
==========================================
- Coverage   71.13%   70.85%   -0.28%     
==========================================
  Files         217      217              
  Lines        7035     7075      +40     
  Branches     1096     1096              
==========================================
+ Hits         5004     5013       +9     
- Misses       1738     1769      +31     
  Partials      293      293              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@Ellpeck Ellpeck marked this pull request as ready for review March 13, 2024 16:27
Co-authored-by: Florian Sihler <florian.sihler@uni-ulm.de>
@EagleoutIce EagleoutIce merged commit 517dd18 into main Mar 13, 2024
21 checks passed
@EagleoutIce EagleoutIce deleted the 686-run-benchmarks-multiple-times branch March 13, 2024 17:10
@EagleoutIce
Copy link
Member

This pull request is included in v1.4.2 (see Release v1.4.2 (Dropping xmlparsedata, Benchmark Re-Runs, and Repl Fixes)).

1 similar comment
@EagleoutIce
Copy link
Member

This pull request is included in v1.4.2 (see Release v1.4.2 (Dropping xmlparsedata, Benchmark Re-Runs, and Repl Fixes)).

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.

Run Benchmarks Multiple Times
2 participants