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
Added support for CGAL library. #13644
Conversation
Ouhh, interesting. I will be on the lookout for this :)! |
/rebuild |
4250ee4
to
80d17b5
Compare
51016fc
to
20ee402
Compare
Co-authored-by: Luca Heltai <luca.heltai@gmail.com>
20ee402
to
31c5f13
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
One small suggestion, but other than that this LGTM. Thanks @fdrmrc and @luca-heltai !
Maybe you also want to consider amending https://github.com/dealii/dealii/blob/master/doc/readme.html and https://github.com/dealii/dealii/blob/master/doc/users/cmake_dealii.html as well? |
btw, the failure in windows is related to uploading artifacts, not to the actual build. |
To have at least the docker compiled with CGAL, I've opened |
Co-authored-by: Jean-Paul Pelteret <jppelteret@gmail.com>
We would like to port to deal.II the support for the CGAL library, that we are using for mesh generation, mesh adaptation, and mesh intersections in github.com/luca-heltai/fsi-suite.
@fdrmrc is the main author, and he'll be pushing many more PR once this is accepted.
I'm not sure if we want to have all of the features we are currently supporting before the release, but I think it would be nice to have this in 9.4, so that people with some CGAL experience could already benefit from it.