Commit cee6563
committed
✅ Review Round 13: Fix NetCDF write and directory branding
CMIP7 naming now fully functional!
Changes:
1. Fixed directory branding_suffix lookup (prioritize rule config)
2. Added NetCDF4 compression constants (NETCDF_COMPRESSION_LEVEL=4)
3. Temporarily exclude lat_bnds/lon_bnds to avoid NetCDF write error
- Large unstructured grids (3M+ cells × 16 vertices) cause issues
- Workaround: Skip boundary variables until NetCDF4 fix available
- Coordinates (lat, lon) still included correctly
4. Updated .gitignore to exclude logs, cache, and output files
Results:
✅ Filename: tos_tavg-u-hxy-sea_mon_glb_gn_AWI-ESM3-VEG-LR_piControl_r1i1p1f1_135001-135012.nc
✅ Directory: MIP-DRS7/CMIP7/CMIP/AWI/.../glb/mon/tos/tavg-u-hxy-sea/gn/v*/
✅ Compression: zlib level 4 with shuffle
✅ File size: ~49 MB gn (was 937 MB), ~22 MB gr
✅ Data written: time=12 timesteps
Note: Boundary variables excluded as temporary workaround for NetCDF write bug
with large multi-dimensional arrays on unstructured grids.1 parent a5c24b6 commit cee6563
File tree
21 files changed
+4524
-37
lines changed- MESH_cache/DARS2
- cmorized_output/MIP-DRS7/CMIP7/CMIP/AWI/AWI-ESM3-VEG-LR/piControl/r1i1p1f1/glb/3hr/tos/unknown-u-hxy-u/gr/v20260313
- src/pycmor
- core
- data_request
- std_lib
21 files changed
+4524
-37
lines changedBinary file not shown.
Binary file not shown.
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
46 | 46 | | |
47 | 47 | | |
48 | 48 | | |
49 | | - | |
| 49 | + | |
50 | 50 | | |
51 | 51 | | |
52 | 52 | | |
| |||
64 | 64 | | |
65 | 65 | | |
66 | 66 | | |
67 | | - | |
| 67 | + | |
68 | 68 | | |
69 | 69 | | |
70 | 70 | | |
71 | 71 | | |
72 | | - | |
| 72 | + | |
73 | 73 | | |
74 | 74 | | |
75 | 75 | | |
| |||
Large diffs are not rendered by default.
Large diffs are not rendered by default.
Large diffs are not rendered by default.
Large diffs are not rendered by default.
Large diffs are not rendered by default.
Large diffs are not rendered by default.
0 commit comments