Explicitly close `xr.Dataset` in `VarHandler.cmorize()` - Githubissues

E3SM-Project / e3sm_to_cmip

Tools to CMORize E3SM output

https://e3sm-to-cmip.readthedocs.io/en/latest/

MIT License

7 stars 7 forks source link

Explicitly close `xr.Dataset` in `VarHandler.cmorize()` #244

Closed tomvothecoder closed 9 months ago

tomvothecoder commented 9 months ago

Description

By explicitly closing the multi-file xr.Dataset object, we avoid any possibility of resource locking with .nc files

This is an attempted fix for https://github.com/E3SM-Project/zppy/issues/549.

Checklist

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] My changes generate no new warnings
[x] Any dependent changes have been merged and published in downstream modules

If applicable:

[ ] New and existing unit tests pass with my changes (locally and CI/CD build)
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have noted that this is a breaking change for a major release (fix or feature that would cause existing functionality to not work as expected)