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

fix #1947 #2074

Closed
wants to merge 3 commits into from
Closed

Conversation

quantresearch1
Copy link
Contributor

@jajhall Hello, I was not sure if you want you wanted to have a way to expose the solution status or if you just wanted to log them ? Let me know if this is not what you want, I am happy to change it.

@@ -537,8 +537,8 @@ highs_getColsEntries(Highs* h, HighsInt num_set_entries,
py::cast(value));
}

std::tuple<HighsStatus, HighsVarType>
highs_getColIntegrality(Highs* h, HighsInt col) {
std::tuple<HighsStatus, HighsVarType> highs_getColIntegrality(Highs* h,
Copy link
Contributor Author

@quantresearch1 quantresearch1 Dec 6, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks like I messed up the formatting... I believe VSCode might have done that automatically, let me check how to revert. Is there a developer guidelines about the formatting tools used in this project ? Apologies if obvious, I am a python developer and this is my first foray in C++ land

@jajhall
Copy link
Member

jajhall commented Dec 6, 2024

Primal/dual solution status is available from HighsInfo, and can't generally be pulled from the simplex primal/dual solution status, as simplex isn't necessarily used to solve problems - even for LP

We use version 14 of

clang-format -style=file -i foo.cpp/h/cc/hh

@jajhall jajhall closed this Dec 6, 2024
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