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

Add support for properly disabling the DLONG support #16

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

imciner2
Copy link
Member

@imciner2 imciner2 commented Feb 3, 2021

This fixes the compile error on systems where QDLDL_int is a 32-bit integer instead of a 64-bit integer. The problem is that the DLONG flag was being always defined in the code, and that then the qdldl interface code was always calling the long version of the amd_order instead of the one for the size of the integer. I have tested this on a 32-bit Beaglebone Black and my 64-bit Fedora machine and it compiles on both. (I can't run the tests on the 32-bit machine because I don't have Python >3.7).

Note that this will depend on PR osqp/qdldl#37 being merged into QDLDL and then being included here. In the mean time, it can be tested with the proposed QDLDL fix by running the following commands inside the qdldl submodule

git remote add imremote https://github.com/imciner2/qdldl.git
git fetch imremote
git checkout im/confheader

Fixes #15

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.

Build fails on i386, armv7 architectures
1 participant