Skip to content

Correct alias elimination docstring typo - #4745

Merged
ChrisRackauckas merged 1 commit into
SciML:masterfrom
ChrisRackauckas-Claude:agent/fix-alias-elimination-typo
Jul 13, 2026
Merged

Correct alias elimination docstring typo#4745
ChrisRackauckas merged 1 commit into
SciML:masterfrom
ChrisRackauckas-Claude:agent/fix-alias-elimination-typo

Conversation

@ChrisRackauckas-Claude

Copy link
Copy Markdown
Member

Summary

  • correct reprsenting to representing in the alias-elimination docstring
  • restore the repository-wide typos spelling check

Root cause

The typo entered in 0b01708ffb4c545b888f859bdcbc505350f723ef when the zero-variable elimination pass was added. It is already present on master, so unrelated pull requests such as #4744 inherit the spelling-check failure.

Validation

  • typos-cli 1.48.0: passed repository-wide
  • Runic 1.7.0 --check .: passed repository-wide
  • git diff --check: passed

Please ignore this PR until it has been reviewed by @ChrisRackauckas.

Co-Authored-By: Chris Rackauckas <accounts@chrisrackauckas.com>
@ChrisRackauckas-Claude

ChrisRackauckas-Claude commented Jul 13, 2026

Copy link
Copy Markdown
Member Author

Clean-master CI classification after independent local reproduction:

  • Both benchmark failures are the pre-existing MTK/MTKBase runner load-order bug fixed in draft Resolve benchmark MTKBase before runner startup #4747. Clean master exits 1 with a loaded-path mismatch and UndefVarError: analytically_integrated; the exact patched AirspeedVelocity run exits 0 and executes the mtkcompile benchmark.
  • Both Catalyst Modeling failures are the pre-existing Catalyst test dependency on non-public AffectSystem internals fixed in draft Test symbolic jump affects through public API Catalyst.jl#1509. The focused test and full symbolic_stoichiometry.jl pass locally on Julia 1.12.6.

These failures are independent of the docstring typo in this PR.

@ChrisRackauckas
ChrisRackauckas marked this pull request as ready for review July 13, 2026 19:55
@ChrisRackauckas
ChrisRackauckas merged commit c5b1d67 into SciML:master Jul 13, 2026
58 of 67 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