Nick Hodgskin

> * `time_periodic` in `FieldSet` just noting that this would extend to `Field.time_periodic` and would result in `Grid.period` being dropped as well. Quite keen for this to go - should...

> * `postIterationCallbacks` functions in `pset.execute()` This was introduced in #764 . It looks like its main usage was as a hook for the `test_from_netcdf_memory_containment` test, would that be correct...

I think we should also remove `deferred_load` (opting for the `deferred_load = False` path in the code). This will be handled down the line by xarray, and we default to...

Ah yes. I agree. Let's drop `deferred_load=True` (not `deferred_load=False`). That would be much clearer.

Hi @rehamansoor, great to hear that you’re interested in this issue. Before the wall of text that is about to follow, please let me know if you have any questions...

Hi @rehamansoor . All good! We do have a few issues that would be better suited: In #1620 with have the subtask ` Remove erroneous calls to os.path in cleanup_remove_files()...

> * @erikvansebille I saw some reprs of the format `f"PType::{self.variables}"`. Does Parcels dynamically evaluate these in the C-code? Are these worth avoiding changes to? bump

> * Download the data (from some hosting provider) for realistic cases. Perhaps using [`pooch`](https://pypi.org/project/pooch/) rather than our current custom downloading mechanism. Since the examples we're dealing with are quite...

Here are the data files that we currently have in the `tests/test_data` folder. I want to move as many as possible to code only (i.e., idealised datasets that are created...

cc @michaeldenes . Feel free to add any additional thoughts that weren't included in our discusssion cc @erikvansebille let me know if you feel I'm missing any datasets here (note...