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 to VTK #41

Merged
merged 1 commit into from
May 26, 2023
Merged

Add support to VTK #41

merged 1 commit into from
May 26, 2023

Conversation

pcafrica
Copy link
Contributor

As the title says :)

See also dealii/dealii#15266.

@@ -76,6 +76,31 @@ RUN cd /usr/src && \
cmake -GNinja .. -DCMAKE_INSTALL_PREFIX=/usr/ && \
ninja install && cd ../ && rm -rf build

# VTK
RUN cd /usr/src && \
Copy link
Member

Choose a reason for hiding this comment

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

For the jammy release, VTK 9.1 packages exists, see here.

Do you require a custom installation of VTK? If not, I would suggest to opt for the package on jammy. This will shorten the build time of the docker image.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Hi @marcfehling, thanks for pointing this out!

Unfortunately in Ubuntu 22.04 there is a clash between the dependencies of Trilinos and VTK, please see the discussion here.

Additionally, the custom build commands that I've provided result in a very lightweight installation of VTK, by getting rid of all the graphical modules that will likely never be used in deal.II.

Tested on my local machine and using 2 cores, the additional time for building the Docker associated with VTK is around 20 minutes.

Copy link
Member

Choose a reason for hiding this comment

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

Thanks for pointing it out. That's annoying! Looks like the difference between libtbb2 and libtbb is really just the version tbb 2020.3 vs tbb 2021.5 (probably for backwards compatibility?).

You found a working solution that works, and I guess 20 minutes extra for building the image is okay. Not ideal, but okay.

Copy link
Member

@luca-heltai luca-heltai left a comment

Choose a reason for hiding this comment

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

Thanks!

@luca-heltai luca-heltai merged commit 618d5b4 into dealii:master May 26, 2023
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.

None yet

3 participants