Nick Hodgskin

Results 204 comments of Nick Hodgskin

A bit of a comparison I whipped up (not 1to1 since the projects aren't the same sizes, but hopefully somewhat insightful) ![output](https://github.com/user-attachments/assets/01552c42-e371-4c99-bdf9-68235f2401b4) ![output](https://github.com/user-attachments/assets/29c58c92-b40a-421d-8610-4c467471ed62)

> What are the types of overly-complex syntax that can/need to be changed? I don't think there's a single answer, and I think there are lots of ways that we...

With v4 of Parcels this is no longer relevant - we have a lot more separation of responsibilities in the codebase.

Oh yes. I thought we had a property `Field.time` but it looks like its just lon and lat. I'll put up a patch

As @michaeldenes mentioned this is an enhancement not a bug :))

> ```python > from pathlib import Path > > @pytest.mark.parametrize("with_timestamps", [False]) > def test(with_timestamps): > TEST_DATA = Path(__file__).resolve().parent / "test_data" > filenames = { > "lon": TEST_DATA / "mask_nemo_cross_180lon.nc", >...

Only relevant to v3 (will be free in v4). Closing

Hi @lucvandenbulcke. This bug has now been patched in #1604. If you'd like to use these changes, please follow the development installation instructions outlined in the ["Installation for developers" section...

To be clear, I don't think think this bug is really encountered in production due to how the TimeConverter class is used. Regardless, I think this would be good to...

TimeConverter is primarily used in the Field/FieldSet/Grid classes (with usage outside those classes still highly related to those classes). With v4 and the reliance on xarray, this would no longer...