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 Ceres Solver #171

Draft
wants to merge 12 commits into
base: main
Choose a base branch
from
Draft

Conversation

simon-wacker
Copy link
Contributor

@simon-wacker simon-wacker commented Jan 10, 2024

Ceres Solver is a large scale non-linear optimization library. It's documented and versioned.

It comes with its own system dependencies that are partly added as external projects with CMake and you can install the ones that are not on Debian-base Linux distributions with

sudo apt-get install libatlas-base-dev # Use ATLAS for BLAS & LAPACK
sudo apt-get install libsuitesparse-dev # SuiteSparse (optional)

I updated the getting started wiki entry accordingly.

Note that I had to patch GFlags with https://github.com/ise-bipv/Windows-CalcEngine/pull/10/files#diff-eca6055ba39d4288773ba85b17d27e756d0eb1bff21c7456fa41de77797fa674 to add it as external project. This patch may become obsolete or may need to be updated in new versions of GFlags

To-Dos:


For the final setup and understanding cmake I took inspiration and learned from

@simon-wacker simon-wacker marked this pull request as draft January 10, 2024 13:53
@simon-wacker
Copy link
Contributor Author

@christoph-maurer I suppose that it works but I haven't checked it yet. To do so, we can make a dummy test in which we just include Ceres and solve some simple equation, or we can add a dummy of the new functionality in which we include Ceres and write a as of now failing test for the new functionality.

@christoph-maurer
Copy link
Contributor

@simon-wacker Thank you! A dummy with a simple equation would be great.

@RDmitchell
Copy link

Is there an issue in Windows Calc Engine that this pull request is associated with? If not, one should be made.

Is this something that the LBNL Windows team wants to add to Windows Calc Engine?

@christoph-maurer
Copy link
Contributor

Thanks for the reminder, @RDmitchell ! I have raised #172 . Does it solve your second question? This work is part of another research project and separate from EQWIN-P with its product data network buildingenvelopedata.org.

@simon-wacker simon-wacker force-pushed the add-ceres-solver branch 3 times, most recently from 4ddc9d7 to 6af4d5b Compare February 5, 2024 14:43
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.

3 participants