NCAR / music-box

A box/column model using MICM chemistry
Apache License 2.0
14 stars 8 forks source link

Publish images for common platforms #135

Closed mattldawson closed 2 weeks ago

mattldawson commented 1 year ago

Make sure that published Docker images are provided for linux/amd64 and linux/arm64

K20shores commented 11 months ago

Building suitesparse takes too long and the github action to publish multiplatforms times out. We can install suitesparse-devel and suitesparse-static with dnf in the docker container. However, we will need to update the search paths for suitesparse in both camp and musicbox to do that. Doing this could speed up the build, but will bump up the version of suitesparse used and could cause compiling issues if the API has changed

K20shores commented 11 months ago

Install paths for suitesparse: https://packages.fedoraproject.org/pkgs/suitesparse/suitesparse-devel/fedora-37.html, which don't match the variables in cmake in either musicbox or camp

mattldawson commented 2 weeks ago

not needed after moving to python