Skip to content

Upgrade ComponentArrays QA to SciMLTesting 2.4 - #394

Merged
ChrisRackauckas merged 3 commits into
SciML:mainfrom
ChrisRackauckas-Claude:agent/scimltesting-24-componentarrays
Jul 22, 2026
Merged

Upgrade ComponentArrays QA to SciMLTesting 2.4#394
ChrisRackauckas merged 3 commits into
SciML:mainfrom
ChrisRackauckas-Claude:agent/scimltesting-24-componentarrays

Conversation

@ChrisRackauckas-Claude

Copy link
Copy Markdown
Member

Summary

  • require SciMLTesting 2.4 in every ComponentArrays test environment
  • use strict rendered-docs, explicit-import, and reexport QA defaults
  • improve definition-site public API documentation and test the external AbstractAxis extension contract

Verification

  • strict QA: 17/17 passed with SciMLTesting 2.4.0
  • rendered Documenter build and doctests passed
  • grouped core tests passed; 9 existing broadcasting cases remain marked broken
  • Runic check passed for all modified source and test files

Ignore until reviewed by @ChrisRackauckas.

Co-Authored-By: Chris Rackauckas <accounts@chrisrackauckas.com>
Co-Authored-By: Chris Rackauckas <accounts@chrisrackauckas.com>
Co-Authored-By: Chris Rackauckas <accounts@chrisrackauckas.com>
Comment thread src/linear_algebra.jl
)
F = LinearAlgebra.qr(getdata(A), pivot; kwargs...)
return _rewrap_qr(F, getaxes(A))
return LinearAlgebra.qr!(copy(A), pivot; kwargs...)

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

why copy?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

oh because it's qr not qr!

@ChrisRackauckas
ChrisRackauckas marked this pull request as ready for review July 22, 2026 10:51
@ChrisRackauckas
ChrisRackauckas merged commit 2af4add into SciML:main Jul 22, 2026
20 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