Skip to content

Conversation

@seddonym
Copy link
Collaborator

@seddonym seddonym commented Jan 10, 2025

  • Add tests to the benchmarking tests which assert on the values returned - helps check we're still testing the same thing.
  • Change the rust integration test to be for deep layers.

@codspeed-hq
Copy link

codspeed-hq bot commented Jan 10, 2025

CodSpeed Performance Report

Merging #170 will not alter performance

Comparing improve-benchmarking-tests (10de840) with master (eb1f883)

Summary

✅ 3 untouched benchmarks

@seddonym seddonym force-pushed the improve-benchmarking-tests branch from 32d9d85 to a3218c2 Compare January 10, 2025 17:05
This is because it returns some illegal imports.
@seddonym seddonym force-pushed the improve-benchmarking-tests branch from a3218c2 to 10de840 Compare January 10, 2025 17:06
@seddonym seddonym merged commit 9df4577 into master Jan 10, 2025
17 checks passed
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.

2 participants