Skip to content

model reduction by residualization not implemented or not documented #1168

Description

@bbbccd

the documentation does not explain what exactly "matchdc" does. is it residualization or something else? It seems to not just be a gain adjustment.

Activity

  1. murrayrm commented on Jul 13, 2025

    @murrayrm
    Member

    From the model_reduction docstring:

     Two methods of state reduction are possible: 'truncate' removes the
     states marked for elimination, while 'matchdc' replaces the eliminated
     states with their equilibrium values (thereby keeping the input/output
     gain unchanged at zero frequency ["DC"]).
    
  2. bbbccd commented on Jul 13, 2025

    @bbbccd
    Author

    From the model_reduction docstring: ...

    ok that smells like the residualization described by skogestad, but i can't tell for sure (would be useful to mention explicitly in docs :D for those of us not as fluent in the possibilities and standard approaches)

    My bad again. I was reading docs here, which do not contain as descriptive documentation. It must have improved since then. Searching for the docstring you quoted i found that there is a "latest" url for readthedocs.

  3. murrayrm commented on Jul 13, 2025

    @murrayrm
    Member

    You should get a warning if you reference an older version of the documentation (see here).

    An open issue for anyone reading this and having some time to contribute: while 'matchdc' is described in model_reduction, it is not described in balanced reduction, where it is also used.

  4. bbbccd commented on Jul 13, 2025

    @bbbccd
    Author

    You should get a warning if you reference an older version of the documentation (see #1140 (comment)).

    sure enough - i refreshed the page and there is a tooltip that shows up for 4 seconds when the page is refreshed. sounds like a readthedocs problem, but the warning should probably not go away.

  5. self-assigned this
    on Jul 20, 2025
  6. added this to the 0.10.3 milestone on Jul 20, 2025
  7. murrayrm commented on Jul 20, 2025

    @murrayrm
    Member

    FYI, I've got a branch of small documentation changes and I have updated the documentation for balanced_reduction there, so this will get addressed for 0.10.3.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Type

No type

Projects

No projects

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions