Skip to content

Conversation

@AlexanderRichert-NOAA
Copy link
Contributor

@AlexanderRichert-NOAA AlexanderRichert-NOAA commented Dec 29, 2025

This PR:

  • enables oneapi support for ufs-weather-model, and corrects the error logic when a supported compiler is not found
  • sets CMAKE_MODULE_PATH
  • sets an upper limit on bacio version, as ufsatm needs bacio_4
  • add cxx dependency (fails without it)

@AlexanderRichert-NOAA
Copy link
Contributor Author

@spackbot fix style

@spackbot-app
Copy link

spackbot-app bot commented Dec 29, 2025

Let me see if I can fix that for you!

@spackbot-app
Copy link

spackbot-app bot commented Dec 29, 2025

I was able to run spack style --fix for you!

spack style --fix
repos/spack_repo/builtin/packages/ufs_weather_model/package.py
�[1;34m==> �[0mrunning flake8
�[1;34m==> �[0mrunning isort
�[1;34m==> �[0mrunning black
�[1;34m==> �[0mstyle checks passed
Keep in mind that I cannot fix your flake8 or mypy errors, so if you have any you'll need to fix them and update the pull request. If I was able to push to your branch, if you make further changes you will need to pull from your updated branch before pushing again.

I wasn't able to make any further changes, but please see the message above for remaining issues you can fix locally!

@AlexanderRichert-NOAA AlexanderRichert-NOAA changed the title ufs-weather-model: Support oneAPI compiler ufs-weather-model: misc. updates Dec 29, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant