Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Revisit fixed variables treatment #360

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Revisit fixed variables treatment #360

wants to merge 5 commits into from

Commits on Jul 25, 2024

  1. Revisit fixed variables treatment

    * Change the behavior of MakeParameter, depending on the callback:
        - `SparseCallback`: remove the fixed variables and reduce problem's dimension in MadNLP
        - `DenseCallback`: keep the fixed variables frozen (as done previously)
    * Add a new `fixed_handler` if the problem has no fixed variable: `NoFixedVariable`.
    * Ensure we always interact to the nonlinear program `nlp` through the
      callback wrapper `cb` to avoid side effect.
    * Remove the function `get_index_constraint` and stores indexes directly
      in the callback wrapper `cb`.
    * Fix the computation of the bound's multipliers for fixed variables.
    frapac committed Jul 25, 2024
    Configuration menu
    Copy the full SHA
    39e1424 View commit details
    Browse the repository at this point in the history

Commits on Oct 3, 2024

  1. Configuration menu
    Copy the full SHA
    78e8383 View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    22e19ff View commit details
    Browse the repository at this point in the history
  3. Configuration menu
    Copy the full SHA
    95d682d View commit details
    Browse the repository at this point in the history
  4. Configuration menu
    Copy the full SHA
    5ac7e48 View commit details
    Browse the repository at this point in the history