Skip to content

Add MUSICA library as a build option #1309

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

Open
wants to merge 11 commits into
base: develop
Choose a base branch
from

Conversation

mattldawson
Copy link
Collaborator

@mattldawson mattldawson commented May 8, 2025

This PR adds a build option to include the MUSICA-Fortran library in the atmosphere model build. This is intended as a first step towards being able to use MUSICA chemistry components (MICM, TUV-x) in MPAS-A.

To include the MUSICA-Fortran library in the build, MUSICA-Fortran and its dependencies must be installed locally. Include MUSICA in the MPAS-A build like this:

make gnu CORE=atmosphere MUSICA=true

Omitting the MUSICA flag does not attempt to link against the MUSICA libraries.

I'm happy to make any changes. I did my best to follow existing patterns in the codebase, but I'm brand-new to MPAS-A.

@mgduda @rkumar-ncar @marybarthbrock @pfisterggp @K20shores @boulderdaze

Copy link

@boulderdaze boulderdaze left a comment

Choose a reason for hiding this comment

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

I reviewed it and it looks good to me, but I also just started learning about the MPAS way to creating makefiles by reviewing this PR, so I'd love to see comments or feedbacks from others who are more familiar with MPAS.

@mgduda mgduda self-requested a review May 21, 2025 22:16
@mgduda mgduda added Atmosphere Chemistry Changes specific to chemistry in MPAS labels May 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Atmosphere Chemistry Changes specific to chemistry in MPAS
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants