Skip to content

Conversation

@jeertmans
Copy link
Owner

Make benchmarks more reliable.

@github-actions github-actions bot added python Changes Python code tests Changes tests benchmarks Changes benchmarks labels Jul 29, 2025
@codecov
Copy link

codecov bot commented Jul 29, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 83.31%. Comparing base (d5d6efa) to head (3e2723c).
⚠️ Report is 45 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #301   +/-   ##
=======================================
  Coverage   83.31%   83.31%           
=======================================
  Files          32       32           
  Lines        3020     3020           
=======================================
  Hits         2516     2516           
  Misses        504      504           

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Make benchmarks more reliable.
@codspeed-hq
Copy link

codspeed-hq bot commented Jul 29, 2025

CodSpeed Performance Report

Merging #301 will degrade performances by 100%

Comparing cov-and-benches (3e2723c) with main (d5d6efa)

Summary

⚡ 3 improvements
❌ 1 (👁 1) regressions
🆕 2 new benchmarks
⁉️ 27 (👁 27) dropped benchmarks

Benchmarks breakdown

Benchmark BASE HEAD Change
👁 test_compute_paths_in_simple_street_canyon_scene[exhaustive-False-20000-0] 847.2 ms N/A N/A
👁 test_compute_paths_in_simple_street_canyon_scene[exhaustive-False-20000-1] 2 s N/A N/A
👁 test_compute_paths_in_simple_street_canyon_scene[exhaustive-False-20000-2] 2.5 s N/A N/A
👁 test_compute_paths_in_simple_street_canyon_scene[exhaustive-False-None-0] 809.2 ms N/A N/A
👁 test_compute_paths_in_simple_street_canyon_scene[exhaustive-False-None-1] 2.1 s N/A N/A
👁 test_compute_paths_in_simple_street_canyon_scene[exhaustive-False-None-2] 2.5 s N/A N/A
👁 test_compute_paths_in_simple_street_canyon_scene[exhaustive-True-20000-0] 4.6 ms N/A N/A
👁 test_compute_paths_in_simple_street_canyon_scene[exhaustive-True-20000-1] 2.3 s N/A N/A
👁 test_compute_paths_in_simple_street_canyon_scene[exhaustive-True-20000-2] 2.5 s N/A N/A
👁 test_compute_paths_in_simple_street_canyon_scene[exhaustive-True-None-0] 606.8 ms N/A N/A
👁 test_compute_paths_in_simple_street_canyon_scene[exhaustive-True-None-1] 2.4 s N/A N/A
👁 test_compute_paths_in_simple_street_canyon_scene[exhaustive-True-None-2] 2.6 s N/A N/A
🆕 test_compute_paths_in_simple_street_canyon_scene[exhaustive] N/A 29.5 ms N/A
👁 test_compute_paths_in_simple_street_canyon_scene[hybrid-False-20000-0] 8.1 s N/A N/A
👁 test_compute_paths_in_simple_street_canyon_scene[hybrid-False-20000-1] 8 s N/A N/A
👁 test_compute_paths_in_simple_street_canyon_scene[hybrid-False-20000-2] 10.1 s N/A N/A
👁 test_compute_paths_in_simple_street_canyon_scene[hybrid-False-None-0] 7.3 s N/A N/A
👁 test_compute_paths_in_simple_street_canyon_scene[hybrid-False-None-1] 9.4 s N/A N/A
👁 test_compute_paths_in_simple_street_canyon_scene[hybrid-False-None-2] 8.7 s N/A N/A
👁 test_compute_paths_in_simple_street_canyon_scene[hybrid-True-20000-0] 6 s N/A N/A
... ... ... ... ...

ℹ️ Only the first 20 benchmarks are displayed. Go to the app to view all benchmarks.

@jeertmans jeertmans merged commit 7495c2d into main Jul 29, 2025
63 of 64 checks passed
@jeertmans jeertmans deleted the cov-and-benches branch July 29, 2025 09:22
jeertmans added a commit that referenced this pull request Jul 29, 2025
Shortening benchmark names and adding tests for all ray-triangle tests.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

benchmarks Changes benchmarks python Changes Python code tests Changes tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant