FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

[ty] Skip parameter accumulation for object variadics by charliermarsh · Pull Request #24976 · astral-sh/ruff · GitHub

Repository navigation

[ty] Skip parameter accumulation for object variadics - #24976

Merged
charliermarsh merged 2 commits into
mainfrom
charlie/variadic-object-sink
May 3, 2026
Merged

charliermarsh merged 2 commits into
mainfrom
charlie/variadic-object-sink

Conversation

Copy link
Copy Markdown
Member

No description provided.

charliermarsh added performance Potential performance improvement ty The ty type checker labels May 3, 2026

astral-sh-bot Bot commented May 3, 2026 •
edited
Loading

Copy link
Copy Markdown

Typing conformance results

No changes detected ✅

Current numbers
The percentage of diagnostics emitted that were expected errors held steady at 89.34%. The percentage of expected errors that received a diagnostic held steady at 85.30%. The number of fully passing files held steady at 87/134.

astral-sh-bot Bot commented May 3, 2026 •
edited
Loading

Copy link
Copy Markdown

Memory usage report

Summary

Project Old New Diff Outcome
flake8 47.77MB 47.76MB -0.01% (6.36kB) ⬇️
sphinx 260.05MB 260.04MB -0.00% (7.41kB) ⬇️
trio 117.05MB 117.03MB -0.02% (18.74kB) ⬇️
prefect 679.04MB 679.02MB -0.00% (19.33kB) ⬇️

Significant changes

Click to expand detailed breakdown

flake8

Name Old New Diff Outcome
is_redundant_with_impl 82.95kB 81.18kB -2.13% (1.77kB) ⬇️
is_redundant_with_impl::interned_arguments 86.20kB 84.56kB -1.89% (1.63kB) ⬇️
infer_expression_types_impl 1.02MB 1.02MB -0.11% (1.12kB) ⬇️
UnionType 54.75kB 53.81kB -1.71% (960.00B) ⬇️
IntersectionType 45.38kB 44.73kB -1.45% (672.00B) ⬇️
infer_definition_types 1.87MB 1.87MB -0.01% (144.00B) ⬇️
infer_scope_types_impl 976.70kB 976.61kB -0.01% (96.00B) ⬇️

sphinx

Name Old New Diff Outcome
is_redundant_with_impl::interned_arguments 1.07MB 1.07MB -0.18% (1.98kB) ⬇️
UnionType 447.97kB 446.17kB -0.40% (1.80kB) ⬇️
is_redundant_with_impl 926.22kB 924.76kB -0.16% (1.46kB) ⬇️
infer_expression_types_impl 20.47MB 20.47MB -0.01% (1.18kB) ⬇️
infer_scope_types_impl 15.44MB 15.44MB -0.01% (840.00B) ⬇️
IntersectionType 463.27kB 463.11kB -0.04% (168.00B) ⬇️

trio

Name Old New Diff Outcome
is_redundant_with_impl 246.91kB 241.78kB -2.08% (5.13kB) ⬇️
is_redundant_with_impl::interned_arguments 288.15kB 283.68kB -1.55% (4.47kB) ⬇️
UnionType 146.77kB 142.73kB -2.75% (4.03kB) ⬇️
IntersectionType 133.02kB 130.89kB -1.60% (2.13kB) ⬇️
infer_expression_types_impl 6.96MB 6.96MB -0.02% (1.32kB) ⬇️
infer_scope_types_impl 4.73MB 4.73MB -0.02% (1.08kB) ⬇️
infer_definition_types 7.80MB 7.80MB -0.01% (588.00B) ⬇️

prefect

Name Old New Diff Outcome
infer_expression_types_impl 56.16MB 56.16MB -0.01% (7.29kB) ⬇️
is_redundant_with_impl 1.89MB 1.88MB -0.22% (4.22kB) ⬇️
is_redundant_with_impl::interned_arguments 2.13MB 2.13MB -0.19% (4.21kB) ⬇️
UnionType 888.56kB 886.16kB -0.27% (2.41kB) ⬇️
infer_definition_types 86.06MB 86.06MB -0.00% (888.00B) ⬇️
IntersectionType 707.84kB 707.52kB -0.05% (336.00B) ⬇️
infer_scope_types_impl 55.60MB 55.60MB -0.00% (12.00B) ⬇️

setup_rayon();

let mut code = "\
def accepts_objects(first: int, *args: object) -> None: ...

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Using *args: object makes the benchmark like 4x faster now, per #24974 (comment).

astral-sh-bot Bot commented May 3, 2026 •
edited
Loading

Copy link
Copy Markdown

ecosystem-analyzer results

No diagnostic changes detected ✅

Full report with detailed diff (timing results)

codspeed Bot commented May 3, 2026 •
edited
Loading

Copy link
Copy Markdown

Merging this PR will improve performance by ×4.1

⚡ 2 improved benchmarks
✅ 55 untouched benchmarks
⏩ 60 skipped benchmarks1

Performance Changes

Mode Benchmark BASE HEAD Efficiency
⚡ Simulation ty_micro[vararg_parameter_type_accumulation] 280.1 ms 68.6 ms ×4.1
⚡ Memory ty_micro[vararg_parameter_type_accumulation] 33.5 MB 14.6 MB ×2.3

Comparing charlie/variadic-object-sink (b965477) with main (5ee1d0c)

Footnotes

  1. 60 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. ↩

charliermarsh marked this pull request as ready for review May 3, 2026 00:41
charliermarsh merged commit e002c44 into main May 3, 2026
59 checks passed
astral-sh-bot Bot requested a review from oconnor663 May 3, 2026 00:41
charliermarsh deleted the charlie/variadic-object-sink branch May 3, 2026 00:41
carljm added a commit that referenced this pull request May 6, 2026
* main: (63 commits)
  Update uraimo/run-on-arch-action action to v3.1.0 (#25011)
  Update dependency astral-sh/uv to v0.11.10 (#25002)
  Update prek dependencies (#25003)
  Update taiki-e/install-action action to v2.75.25 (#25007)
  Update Rust crate mimalloc to v0.1.50 (#25006)
  Update Rust crate libc to v0.2.186 (#25005)
  Update maturin to v1.13.1 (#25009)
  Update CodSpeedHQ/action action to v4.15.0 (#25008)
  Update Rust crate jiff to v0.2.24 (#25004)
  Increase max value of `line-length` setting (#24962)
  [ty] Fix the Pyodide assets deployment in the ty playground. (#24994)
  [ty] Selectively promote a union of homogeneous fixed-length tuples to a single variadic tuple. (#24705)
  [ty] Cache results in desperate module resolution (#24977)
  [ty] implement proper handling of recursive types in `CycleDetector` (#24773)
  [ty] Skip parameter accumulation for object variadics (#24976)
  Add missing rust related files when detecting changes in CI (#24971)
  [ty] Expand support for narrowing within walruses (#24968)
  [ty] Fix missing visitor guard in `Callable` branch (#24964)
  [ty] Allow reference finding in stringified annotations (#24956)
  [ty] Unpack Union of TypedDict in various sites (#24958)
  ...
thejchap pushed a commit to thejchap/ruff that referenced this pull request May 23, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

performance Potential performance improvement ty The ty type checker

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants


Back | FazBrowse Home | New Git URL