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

Option lcdf64 = .false. do not dump restart file #858

Closed
TillRasmussen opened this issue Sep 1, 2023 · 2 comments · Fixed by #927
Closed

Option lcdf64 = .false. do not dump restart file #858

TillRasmussen opened this issue Sep 1, 2023 · 2 comments · Fixed by #927

Comments

@TillRasmussen
Copy link
Contributor

When running cice with lcdf64 = .false. netcdf files cannot exceed 2gb. I think that we should change the default to true and add a warning in case it is changed to false.
When the files are larger than 2gb it will not produce a warning. It will just not close the file probably and therefore it is not created

@phil-blain
Copy link
Member

Hi Till!

 When the files are larger than 2gb it will not produce a warning. It will just not close the file probably and therefore it is not created

Yes, we encountered that bug recently when updating one of our system to CICE6. I tracked it down and fixed it with that commit: phil-blain@9bee171.

In short, we do not check the return code of the NetCDF subroutine. This is the case at multiple places in the code... I fixed a similar one in phil-blain@9aa116c but a complete audit would be required.

These commits are not yet PR'd, but I plan on doing so.

@TillRasmussen
Copy link
Contributor Author

👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants