Skip to content

Conversation

connorjward
Copy link
Contributor

This adds some code from https://github.com/jorgensd/dolfinx-tutorial/blob/dokken/ngs/docker/Dockerfile#L19-L44 to build netgen from source since wheels are not available on ARM. I have checked on our mac runners and it at least runs.

I found that icepack would not build on ARM so I removed all the Firedrake apps from the ARM image. I don' t know if this a good idea or not.

@danshapero
Copy link
Contributor

Were you able to see what was making icepack fail to build on ARM? If it's some dependency that I can make optional then I'm happy to do that.

@connorjward
Copy link
Contributor Author

Were you able to see what was making icepack fail to build on ARM? If it's some dependency that I can make optional then I'm happy to do that.

It is failing to install rasterio, gmsh and pyroltrilinos.

I want to discuss in the meeting today what the purpose of the 'full-fat' firedrake container is. Does icepack actually use it? Looking at your website it looks as though you point people to firedrake-vanilla. I worry about committing to supporting the installation of lots of downstream packages when I don't know if anyone that actually uses the container for them.

@danshapero
Copy link
Contributor

The install docs on the icepack website are way out of date now. But I think you're right on both counts. I don't know of any icepack users who work off of the full-fat docker image. It's also an annoying maintenance burden for you all to have to support lots of downstream packages. For example, icepack really depends on rasterio but as per this issue they don't have ARM wheels.

@connorjward
Copy link
Contributor Author

The install docs on the icepack website are way out of date now. But I think you're right on both counts. I don't know of any icepack users who work off of the full-fat docker image. It's also an annoying maintenance burden for you all to have to support lots of downstream packages. For example, icepack really depends on rasterio but as per this issue they don't have ARM wheels.

From the meeting yesterday we concluded that we should drop all downstream packages from the container. Instead the container will just contain things like jax, pytorch and ngspetsc.

Thanks for the feedback!

@dham dham merged commit aff2559 into release Oct 21, 2025
7 checks passed
@dham dham deleted the connorjward/firedrake-docker-arm branch October 21, 2025 15:55
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