Skip to content

chore(tests): add benchmark on medium-sized scenes too#449

Merged
jeertmans merged 9 commits into
mainfrom
more-benchmarks
May 18, 2026
Merged

chore(tests): add benchmark on medium-sized scenes too#449
jeertmans merged 9 commits into
mainfrom
more-benchmarks

Conversation

@jeertmans
Copy link
Copy Markdown
Owner

No description provided.

@github-actions github-actions Bot added python Changes Python code tests Changes tests benchmarks Changes benchmarks labels May 14, 2026
@codecov
Copy link
Copy Markdown

codecov Bot commented May 14, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 86.41%. Comparing base (366ce0f) to head (955d813).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #449   +/-   ##
=======================================
  Coverage   86.41%   86.41%           
=======================================
  Files          32       32           
  Lines        3165     3165           
=======================================
  Hits         2735     2735           
  Misses        430      430           

☔ 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.

@github-actions github-actions Bot added the ci Continuous integration (tests, lints, ...) label May 14, 2026
@codspeed-hq
Copy link
Copy Markdown

codspeed-hq Bot commented May 18, 2026

Merging this PR will degrade performance by 12.99%

⚠️ Different runtime environments detected

Some benchmarks with significant performance changes were compared across different runtime environments,
which may affect the accuracy of the results.

Open the report in CodSpeed to investigate

❌ 1 regressed benchmark
✅ 2 untouched benchmarks
🆕 12 new benchmarks
⏩ 7 skipped benchmarks1

Warning

Please fix the performance issues or acknowledge them on CodSpeed.

Performance Changes

Benchmark BASE HEAD Efficiency
🆕 test_compute_paths[medium-disconnect-exhaustive] N/A 12.5 s N/A
🆕 test_compute_paths[medium-disconnect-hybrid] N/A 30.9 s N/A
🆕 test_compute_paths[medium-no_disconnect-exhaustive] N/A 20.1 s N/A
🆕 test_compute_paths[small-disconnect-exhaustive] N/A 19.2 ms N/A
🆕 test_rays_intersect_any_triangle[small] N/A 30.8 ms N/A
🆕 test_compute_paths[small-disconnect-hybrid] N/A 143.3 ms N/A
🆕 test_transmitter_visibility[medium] N/A 7 s N/A
🆕 test_compute_paths[small-no_disconnect-exhaustive] N/A 12 ms N/A
🆕 test_transmitter_visibility[small] N/A 29.8 ms N/A
🆕 test_first_triangles_hit_by_rays[medium] N/A 11.2 s N/A
🆕 test_first_triangles_hit_by_rays[small] N/A 43.3 ms N/A
test_image_method 171.8 µs 197.4 µs -12.99%
🆕 test_rays_intersect_any_triangle[medium] N/A 8.5 s N/A

Tip

Investigate this regression by commenting @codspeedbot fix this regression on this PR, or directly use the CodSpeed MCP with your agent.


Comparing more-benchmarks (955d813) with main (a8e3340)

Open in CodSpeed

Footnotes

  1. 7 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports.

@jeertmans jeertmans merged commit f704eca into main May 18, 2026
51 of 54 checks passed
@jeertmans jeertmans deleted the more-benchmarks branch May 18, 2026 10:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

benchmarks Changes benchmarks ci Continuous integration (tests, lints, ...) python Changes Python code tests Changes tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant