Perform topological sort to order components - #428

Open

cmacmackin wants to merge 31 commits into

cmacmackin/topological_sort_components

Commits on Sep 4, 2026

Switch transform method to using GuardedOptions
cmacmackin
authored
f0f2c6d
Added unit test for topological sorting of components
cmacmackin
authored
349ace7
Add ability to order components automatically
cmacmackin
authored
abbe01d
Ensure ordering of components accounts for pre-set state variables
cmacmackin
authored
ed2f019
Added support for sorting components based on section permissions
cmacmackin
authored
6bb13a5
Mention new sorting and remove references to component ordering
cmacmackin
authored
5adc68b
Check results are the same regardless of component order
cmacmackin
authored
e0b2437
Check only one component can make final write
cmacmackin
authored
55afb99
Fixed some incorrect access controls causing errors
cmacmackin
authored
e4faa42
Add better comments explaining component sorting process
cmacmackin
authored
646573f
Responded to some comments by @ZedThree
cmacmackin
authored
654bd19
Applied linter suggestions
cmacmackin
authored
3610219
Print meaningful error messages for circular dependencies
cmacmackin
authored
416468f
Addressed various comments on PR
cmacmackin
authored
bff5e68
external_apar: Use NamedComponent
bendudson
authored
2cdd49c
Improve documentation/description for component sorting algorithm
cmacmackin
authored
58590d7
Fix permissions causing examples to fail
cmacmackin
authored
ffd2ec8
Fix permission issue
cmacmackin
authored
7d7a756
Ensure electromagnetic only sets interior for velocity/momentum
cmacmackin
authored
74e2e19
Avoid debug 3D metrics build submitting empty reports to CodeCov
cmacmackin
authored
0ffde81
Fix test count expression
cmacmackin
authored
4572264

Commits on Sep 18, 2026

Merge branch 'master' into cmacmackin/topological_sort_components
bendudson
committed
7ed72f0
Comment unit tests to explain what they are doing
cmacmackin
committed
10f8836
Apply suggestions from review
cmacmackin
committed
a5a9b0d

Commits on Sep 29, 2026

Apply batched suggestions from code review

cmacmackin
and
mikekryjak
authored
f1bd541

Commits on Sep 30, 2026

Split Permissions::getVariablesWithPermission into two functions
cmacmackin
committed
b7242ec