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

Make CICE PIO compile optionally replaceable by NetCDF compile in RTs #2460

Open
LarissaReames-NOAA opened this issue Oct 10, 2024 · 0 comments · May be fixed by #2478
Open

Make CICE PIO compile optionally replaceable by NetCDF compile in RTs #2460

LarissaReames-NOAA opened this issue Oct 10, 2024 · 0 comments · May be fixed by #2478
Labels
enhancement New feature or request

Comments

@LarissaReames-NOAA
Copy link

LarissaReames-NOAA commented Oct 10, 2024

Description

Since #2145 was merged, CICE is set to compile using PIO in CICE/CMakeLists.txt. However, based on the investigation in #2320 it's clear that future very long (>90 day) or high-frequency output regression test forecasts (>500 output files) that use CICE will not be possible until that issue is addressed. We need a temporary workaround that enables a regression test to set CICE_IO=NetCDF in these situations.

Solution

A simple solution should modify CICE/CMakeLists.txt to only set CICE_PIO='PIO' if it is not already set. This, along with a modification to compile.sh to provide a CMAKE flag "-DCICE_IO=NetCDF" if an option is passed from a regression test, should provide this compile-time functionality for regression tests.

Related to

N/A

@LarissaReames-NOAA LarissaReames-NOAA added the enhancement New feature or request label Oct 10, 2024
@LarissaReames-NOAA LarissaReames-NOAA linked a pull request Oct 22, 2024 that will close this issue
14 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant