Skip to content

Fix handling of daily timestep inspection - #121

Merged
znichollscr merged 6 commits into
mainfrom
daily-timestep
Oct 1, 2025
Merged

znichollscr merged 6 commits into
mainfrom
daily-timestep

Conversation

@znichollscr

@znichollscr znichollscr commented Oct 1, 2025 •

Copy link
Copy Markdown
Contributor

Description

Checklist

Please confirm that this pull request has done the following:

  • Tests added
  • Documentation added (where applicable)
  • Changelog item added to changelog/

@znichollscr

Copy link
Copy Markdown
Contributor Author

@florence-bockting FYI: this is the solution to the issue we found. In short, if you don't load the data into memory using .compute before you subtract date times, then the result doesn't work properly (xarray doesn't realise the result is a date time delta). It must be some dask bug (it's always dask)

@znichollscr
znichollscr merged commit 790f941 into main Oct 1, 2025
12 checks passed
@znichollscr
znichollscr deleted the daily-timestep branch October 1, 2025 14:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant