Skip to content

Decompose Bench class into focused helper classes #687

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.

Sign up for GitHub

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

Merged
blooop merged 3 commits into from
Dec 26, 2025

Formatting

685cb99
Select commit
Loading
Failed to load commit list.
Merged

Decompose Bench class into focused helper classes #687

Formatting
685cb99
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch succeeded Dec 26, 2025 in 0s

93.90% of diff hit (target 89.10%)

View this Pull Request on Codecov

93.90% of diff hit (target 89.10%)

Annotations

Check warning on line 124 in bencher/bencher.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

bencher/bencher.py#L124 

Added line #L124 was not covered by tests

Check warning on line 129 in bencher/bencher.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

bencher/bencher.py#L129 

Added line #L129 was not covered by tests

Check warning on line 585 in bencher/bencher.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

bencher/bencher.py#L585 

Added line #L585 was not covered by tests

Check warning on line 136 in bencher/result_collector.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

bencher/result_collector.py#L134-L136 

Added lines #L134 - L136 were not covered by tests

Check warning on line 241 in bencher/result_collector.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

bencher/result_collector.py#L237-L241 

Added lines #L237 - L241 were not covered by tests

Check warning on line 248 in bencher/result_collector.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

bencher/result_collector.py#L248 

Added line #L248 was not covered by tests

Check warning on line 332 in bencher/result_collector.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

bencher/result_collector.py#L330-L332 

Added lines #L330 - L332 were not covered by tests

Check warning on line 357 in bencher/result_collector.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

bencher/result_collector.py#L357 

Added line #L357 was not covered by tests

Check warning on line 102 in bencher/sweep_executor.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

bencher/sweep_executor.py#L102 

Added line #L102 was not covered by tests
View more details on Codecov