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.

Sign up for GitHub

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

Jump to bottom

WIP: [Utilities] maintain order of variables in default_copy_to #2495

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

WIP: [Utilities] maintain order of variables in default_copy_to #2495

wants to merge 5 commits into from

Conversation

odow
Copy link
Member

@odow odow commented May 3, 2024 •
edited

This PR is an investigation of what we would need to implement to close #2493. It is still very rough around the edges; I need to clean it up and profile for performance, but as a demo, the case from #2493 now works:

julia> main()
NAME
ROWS
 N  OBJ
COLUMNS
    x         OBJ       1
    y         OBJ       1
RHS
RANGES
BOUNDS
 FR bounds    x
 LO bounds    y         0
 PL bounds    y
ENDATA

TODOs

@odow
Copy link
Member Author

odow commented May 3, 2024

So this, somewhat surprisingly, didn't break any tests. I don't know if that is a good thing or a bad thing... I guess it's because we're completely agnostic to the ordering of columns, so we always use index_map.

@odow
Copy link
Member Author

odow commented May 4, 2024

Okay there are a few failures to look at in solver-tests.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Order of columns during copy_to
1 participant
@odow