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

Cmake and Eigen3 #15

Merged
merged 21 commits into from
Jul 24, 2020
Merged

Cmake and Eigen3 #15

merged 21 commits into from
Jul 24, 2020

Conversation

schulz0r
Copy link
Collaborator

Hello,
After we agreed that dmatrix and its dependencies are hard to integrate into a modern build system such as CMake, I removed dmatrix from this project. The PSOPT code has then been moved to the root directory and can now generate a project. Just create a directory build/, then do

cd build/
cmake ..

This will create a Makefile project. Check this site if you want to generate a Visual Studio project (cmake -G "Visual Studio 15 2017" ..) or others. Optionally, you can pass the option -DBUILD_EXAMPLES=ON in the command line. This will create the examples in the examples/ directory. So, here is what I usually do: cmake -G Xcode -DBUILD_EXAMPLES=ON ...

Don't forget to install Eigen3 on your system before generating the project. This should be available in every package manager out there.
When you are done, you will have a project file in the build/ directory. Open that with the IDE of your choice and hit the compile button. The generated project won't compile because dmatrix is missing. Now, the bugs must be resolved by changing all dmatrices with Eigen::Matrix and other things to do.
The Snopt-interface has not been altered until now, but feel free to ask me whenever it seems necessary.
I hope that this work will help you with modernizing PSOPT.

Edit: Sorry, the first PR did not address the eigen3 branch, so here I try again.

Best regards

Philipp Waxweiler

@vmbecerra vmbecerra merged commit 65d0508 into PSOPT:eigen3 Jul 24, 2020
@vmbecerra
Copy link
Contributor

vmbecerra commented Jul 24, 2020 via email

@schulz0r
Copy link
Collaborator Author

schulz0r commented Jul 24, 2020 via email

@vmbecerra
Copy link
Contributor

vmbecerra commented Jul 24, 2020 via email

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