Tom Vo
Tom Vo
The current structure involves adding a new `_parameter.py` file and sub-classing `CoreParameter`. One file per class introduces more overhead that isn't needed in this case because the sub-classes aren't that...
Related to: https://github.com/E3SM-Project/e3sm_diags/pull/606#discussion_r918383099 https://github.com/E3SM-Project/e3sm_diags/pull/606#discussion_r918384214
This can be caused by different reasons, including: * For loops repeating logger line (not a bug) * Logger objects having two handlers that repeat the same line sequentially Related...
The `exclude` option is getting pretty lengthy and becomes a hassle to update when contributors add new directories. https://github.com/E3SM-Project/e3sm_diags/blob/a776ece46f9b0cb2154011fb39373445610a4590/.pre-commit-config.yaml#L1 Instead of excluding directories from pre-commit checks, we should specify the...
I noticed that (1) `mypy e3sm_diags` and (2) `pre-commit run mypy --all-files` produce different results. I learned that (2) runs in an isolated virtualenv without dependencies installed, so we must...
### What happened? An attendee at the E3SM tutorial workshop ran into a `VersionMismatch` error with `esmf=8.4.2` and `esmpy=8.6.0` after running the `import` cell containing `xcdat` ([source notebook](https://github.com/xCDAT/xcdat-validation/blob/main/demos/2024-e3sm-workshop/xcdat_practicum_notebook.ipynb)). This notebook...
## Description - Closes #663 ## Checklist - [ ] My code follows the style guidelines of this project - [ ] I have performed a self-review of my own...
### What happened? This bug was caught while working on #664. `unify_plev()` is supposed to sort the Z axis in ascending order. However, this operation is not performed in-place/in-memory on...
### Describe your documentation update This means `conda create` and `conda install` should run as fast as `mamba`, removing the need to install `mamba` through MambaForge or as a package...
### What happened? In PR #815: > This PR validates the replication of the slice flag feature added in PR #750 to fix #759. > > I re-ran `ex1.py` and...