Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A few minor comments raised in-line, but otherwise everything looks good. As for the CI jobs, I'm not sure why the 'Run test suite' jobs ran since this is a PR to lama-to-dask, regardless they aren't relevant yet so we can ignore the failures there. The relevant tests ('Test cf.Data' and 'Linting') do pass so that indicates everything is good test-wise, along with a local pass I have observed of course.

Feel free to merge once you have seen and responded as you wish to the in-line comments! Thanks.