Skip to content

change to create a PR #132

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

Draft
wants to merge 33 commits into
base: main
Choose a base branch
from
Draft

change to create a PR #132

wants to merge 33 commits into from

Conversation

uurien
Copy link
Collaborator

@uurien uurien commented Apr 24, 2025

What does this PR do?

Motivation

Additional Notes

Describe how to test your changes

Checklist

  • The CHANGELOG.md has been updated
  • Unit tests have been updated and pass
  • If known, an appropriate milestone has been selected

@pr-commenter
Copy link

pr-commenter bot commented Apr 24, 2025

Benchmarks

Benchmark execution time: 2025-05-09 11:53:35

Comparing candidate commit d52d96a in PR branch ugaitz/add-benchmarks-testing with baseline commit cb05e58 in branch ugaitz/add-benchmarks.

Found 0 performance improvements and 3 performance regressions! Performance is the same for 2 metrics, 0 unstable metrics.

scenario:rewrite-iast+orchestrion-22

  • 🟥 cpu_user_time [+371.803ms; +385.538ms] or [+9.558%; +9.911%]
  • 🟥 execution_time [+392.876ms; +404.367ms] or [+9.186%; +9.455%]
  • 🟥 instructions [+1.2G instructions; +1.2G instructions] or [+10.899%; +11.041%]

@uurien uurien force-pushed the ugaitz/add-benchmarks-testing branch 4 times, most recently from f2aeec1 to 712e8fe Compare April 24, 2025 19:11
@uurien uurien force-pushed the ugaitz/add-benchmarks-testing branch from 6f2c88d to aa72003 Compare April 25, 2025 07:55
@uurien uurien force-pushed the ugaitz/add-benchmarks-testing branch from b31d128 to 09ab89f Compare April 25, 2025 08:54
@uurien uurien force-pushed the ugaitz/add-benchmarks-testing branch 3 times, most recently from f402638 to 45c046c Compare April 25, 2025 11:19
@uurien uurien force-pushed the ugaitz/add-benchmarks-testing branch 3 times, most recently from 1542706 to 2f8e272 Compare April 25, 2025 11:30
@uurien uurien force-pushed the ugaitz/add-benchmarks-testing branch from 2f8e272 to a5fe6da Compare April 25, 2025 11:36
@uurien uurien force-pushed the ugaitz/add-benchmarks-testing branch from 381da3c to 6fbb836 Compare May 8, 2025 13:30
@uurien uurien force-pushed the ugaitz/add-benchmarks branch 10 times, most recently from 2f359dc to 0c50e4f Compare May 9, 2025 07:17
@uurien uurien force-pushed the ugaitz/add-benchmarks branch from 0c50e4f to 2c4bf77 Compare May 9, 2025 07:19
@uurien uurien force-pushed the ugaitz/add-benchmarks branch from 4319097 to d9a7de2 Compare May 9, 2025 08:43
@uurien uurien force-pushed the ugaitz/add-benchmarks-testing branch from 6fbb836 to 74274cd Compare May 9, 2025 10:03
@uurien uurien force-pushed the ugaitz/add-benchmarks-testing branch from 74274cd to b59d462 Compare May 9, 2025 10:42
Force a perf regression

Increase tests

Fix

reduce the overhead

Reduce iterations

Testing

Restoring

MAJOR_VERSION
@uurien uurien force-pushed the ugaitz/add-benchmarks-testing branch from b59d462 to d52d96a Compare May 9, 2025 11:30
Base automatically changed from ugaitz/add-benchmarks to main May 16, 2025 07:26
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.

1 participant