🚇🩹 Fix wrong comparison in pr_benchmark workflow

[s-weigand]

In #1060 we just found that there is an error in the comparison of the last tag and the main branch in the pr_benchmark workflow. Instead of comparing the last tag to main, it compares it against the PR HEAD

Change summary

• 🚇🩹 Fixed release and main comparison using HEAD instead of main

Checklist

• ✔️ Passing the tests (mandatory for all PR's)
• 🚧 Added changes to changelog (mandatory for all PR's)

[github-actions]

👈 Launch a binder notebook on branch s-weigand/pyglotaran/fix/benchmark-workflow

[github-actions]

Benchmark is done. Checkout the benchmark result page.
Benchmark differences below 5% might be due to CI noise.

Benchmark diff v0.6.0 vs. main

Parametrized benchmark signatures:

BenchmarkOptimize.time_optimize(index_dependent, grouped, weight)

All benchmarks:

       before           after         ratio
     [6c3c390e]       [73ebe461]
     <v0.6.0>                   
       53.9±0.4ms       53.7±0.2ms     1.00  BenchmarkOptimize.time_optimize(False, False, False)
        58.6±30ms       57.9±0.7ms     0.99  BenchmarkOptimize.time_optimize(False, False, True)
       53.8±0.3ms       53.7±0.2ms     1.00  BenchmarkOptimize.time_optimize(False, True, False)
       57.2±0.3ms       57.8±0.5ms     1.01  BenchmarkOptimize.time_optimize(False, True, True)
       64.6±0.5ms       65.1±0.4ms     1.01  BenchmarkOptimize.time_optimize(True, False, False)
         148±50ms       68.0±0.3ms    ~0.46  BenchmarkOptimize.time_optimize(True, False, True)
       65.1±0.4ms       64.9±0.3ms     1.00  BenchmarkOptimize.time_optimize(True, True, False)
        68.5±30ms         171±60ms    ~2.49  BenchmarkOptimize.time_optimize(True, True, True)
             205M             201M     0.98  IntegrationTwoDatasets.peakmem_optimize
       1.72±0.07s       1.74±0.08s     1.01  IntegrationTwoDatasets.time_optimize

Benchmark diff main vs. PR

Parametrized benchmark signatures:

BenchmarkOptimize.time_optimize(index_dependent, grouped, weight)

All benchmarks:

       before           after         ratio
     [73ebe461]       [50612667]
       53.7±0.2ms       53.4±0.7ms     0.99  BenchmarkOptimize.time_optimize(False, False, False)
       57.9±0.7ms         58.5±4ms     1.01  BenchmarkOptimize.time_optimize(False, False, True)
       53.7±0.2ms       54.0±0.8ms     1.00  BenchmarkOptimize.time_optimize(False, True, False)
       57.8±0.5ms         59.3±3ms     1.03  BenchmarkOptimize.time_optimize(False, True, True)
       65.1±0.4ms       64.8±0.4ms     1.00  BenchmarkOptimize.time_optimize(True, False, False)
       68.0±0.3ms         71.5±4ms     1.05  BenchmarkOptimize.time_optimize(True, False, True)
       64.9±0.3ms       64.3±0.2ms     0.99  BenchmarkOptimize.time_optimize(True, True, False)
         171±60ms        67.8±20ms    ~0.40  BenchmarkOptimize.time_optimize(True, True, True)
             201M             204M     1.01  IntegrationTwoDatasets.peakmem_optimize
       1.74±0.08s       1.66±0.07s     0.95  IntegrationTwoDatasets.time_optimize

[codecov]

Codecov Report

Merging #1097 (5061266) into main (73ebe46) will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##            main   #1097   +/-   ##
=====================================
  Coverage   87.1%   87.1%           
=====================================
  Files        101     101           
  Lines       5339    5339           
  Branches     995     995           
=====================================
  Hits        4654    4654           
  Misses       530     530           
  Partials     155     155           

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 73ebe46...5061266. Read the comment docs.

[sonarqubecloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

No Coverage information
0.7% Duplication