Peter Hill
Peter Hill
My understanding is that netCDF files cannot contain HDF5 object references of any kind in order to avoid cyclic graphs. This means that definitions of datatypes (and dimensions as well)...
LGTM! I think your xarray tests are failing because, somewhat ironically, they actually work and you expect them to fail! :smile: Just to note that nc-complex supports a wider variety...
> Sure, we can think about aligning h5netcdf's legacyapi to what netcdf4-python supports. This would be neat, but much less important than your work here to ensure that currently released...
LGTM! Thanks @kmuehlbauer !
You might want to report which `nc-config` it's trying to run. Currently the Windows builds are failing with `FileNotFoundError: [WinError 2] The system cannot find the file specified` which doesn't...
I've done something similar for my nc-complex package: [CI workflow](https://github.com/PlasmaFAIR/nc-complex/blob/main/.github/workflows/build_wheels.yml), [cibuildwheel config in the `pyproject.toml`](https://github.com/PlasmaFAIR/nc-complex/blob/4dc9bbd793729d22c810e7eec4bb0cdc7383866f/pyproject.toml#L27-L46), [example workflow run](https://github.com/PlasmaFAIR/nc-complex/actions/runs/6469586788/job/17563939909) I used `manylinux_2_28` in order to get the more recent HDF5, and...
I suggest using `manylinux_2_28` with the system HDF5 (which is [1.10](https://rhel.pkgs.org/8/epel-x86_64/hdf5-1.10.5-4.el8.x86_64.rpm.html)) and netcdf-C from source (because the system version is [only 4.7.0](https://rhel.pkgs.org/8/epel-x86_64/netcdf-4.7.0-3.el8.x86_64.rpm.html)). I would definitely try avoiding building HDF5 from...
> The issue is that ford ignores the cpp directives as it uses fypp as preprocessor. One option could be to add a wrapper script that runs both preprocessors appropriately,...
My worry here is the tracking storing a *lot* of data if it's storing the full `Field3D` on every single change. Should it also be a compile-time option so that...
> It could be, but I don't think it is needed. I would personally keep it enabled for production runs, as it has a runtime switch, that is disabled by...