Skip to content
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

Provide option in attrs.define to allow users to exclude parameters set to default value from repr #1276

Open
wants to merge 26 commits into
base: main
Choose a base branch
from

Merge branch 'main' into repr_optionally_exclude_param_defaults

9f1963b
Select commit
Loading
Failed to load commit list.
Open

Provide option in attrs.define to allow users to exclude parameters set to default value from repr #1276

Merge branch 'main' into repr_optionally_exclude_param_defaults
9f1963b
Select commit
Loading
Failed to load commit list.
CodSpeed HQ / CodSpeed Performance Analysis completed Jul 29, 2024

Performance Regression: -28.49%

Summary

❌ 3 regressions
✅ 5 untouched benchmarks

⚠️ Please fix the performance issues or acknowledge them on CodSpeed.

Benchmarks breakdown

Benchmark main RNKuhns:repr_optionally_exclude_param_defaults Change
❌ test_create_frozen_class 1.7 s 2.3 s -25.32%
❌ test_create_simple_class 1.5 s 2.1 s -28.06%
❌ test_create_simple_class_make_class 1.5 s 2 s -28.49%
View more details on CodSpeed HQ