Skip to content

Conversation

@bblay
Copy link

@bblay bblay commented Sep 24, 2025

Adds cell statistics core functionality to cset with a new operator and plotter.

To run: From the temporary folder delme, run run_cell_statistics.py. It reads the cached retrieval data from RES' test_lfric config.

A comparison of RES and CSET output:
res
cset

todo:

  • HTML styling.
  • Select the "all" time point by default, not just the first in the list (or make it first, even better).
  • Axis labels, legend, etc.
  • Why do the two plots above look similar/identical but the y axes don't match?
  • The git commits were made with `--no-verify'.
  • I didn't get as far as considering rose. I was just running from VDI.
  • In the operator there's a variable called y_axis. It's not clear to me if the code this controls is still necessary.

Further thoughts:

  • Should we make the operator return a CubeList? It currently returns a nested dict, five levels deep, because there's so many controls on the page. This led to the new pickle read and write operators, which helped speed up development.
  • It would be nice if the read_cubes operator allowed multiple vars to be loaded so we don't have to load the files once per var. That would allow us to add a control for the variable, and would increase the depth of the nested dict.

- brought more code in from res
- think i need to remove the outer cubes loop
- need to plug into the histogram plotting, possibly with a new histo plot just for this data
…w plot function, which probably needs to call the existing histogram plot function more than once - we'll see.
- one var not working
- lots of skipped plots due to no data or nan in the min/max
- needs a bespoke plot and html to bring it all together sensibly
@bblay bblay changed the title Bb cell stats cell stats Sep 24, 2025
…olish

something's messed around with code formatting
…for both threshold and time point. We need to

 - move the index into the subfolder so it doesn't get overwritten
 - ? move the cell attribute and time grouping from the recipe runner into the code so it's all on the one page together
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