Skip to content

use the cached reorder_parameters in a few more places#4690

Merged
AayushSabharwal merged 1 commit into
SciML:masterfrom
KristofferC:kc/reorder_parameters_default
Jul 1, 2026
Merged

use the cached reorder_parameters in a few more places#4690
AayushSabharwal merged 1 commit into
SciML:masterfrom
KristofferC:kc/reorder_parameters_default

Conversation

@KristofferC

Copy link
Copy Markdown
Contributor

Only the 1 arg version uses the cached reorder_parameters. Follow up from #4685. On a large model this goes from

 12.897235 seconds (192.77 M allocations: 7.152 GiB, 23.57% gc time)

to

 11.219255 seconds (173.77 M allocations: 6.507 GiB, 19.11% gc time)

@AayushSabharwal AayushSabharwal merged commit 079e236 into SciML:master Jul 1, 2026
69 of 89 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