Skip to content

Conversation

rcomer
Copy link
Member

@rcomer rcomer commented Sep 8, 2025

🚀 Pull Request

Description

Fixes #2076

The problem was that merge tries to add the cell measure to the same dimension that it had on the input cubes. This is wrong, because the new dimension[s] are added to the start. So we need to add an offset in the same way that is already done for coordinates. The problem was not picked up by the tests because the output cube there was square (2 by 2) and there was no check for which dimension the cell measure was on. By adding a third cube in the test input, we reproduce the error shown in the issue.

Also made equivalent change for ancillary variables.

Full disclosure: the list comprehensions and most of the what's new entry were written by Copilot.


Consult Iris pull request check list


Add any of the below labels to trigger actions on this PR:

  • benchmark_this Request that this pull request be benchmarked to check if it introduces performance shifts

Copy link

codecov bot commented Sep 8, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 90.29%. Comparing base (005b51a) to head (64f80f2).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #6688   +/-   ##
=======================================
  Coverage   90.29%   90.29%           
=======================================
  Files          91       91           
  Lines       24465    24467    +2     
  Branches     4568     4568           
=======================================
+ Hits        22090    22092    +2     
  Misses       1607     1607           
  Partials      768      768           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@rcomer rcomer force-pushed the merge-cell_measures branch from c2590ea to 64f80f2 Compare September 9, 2025 08:12
@rcomer rcomer added Type: Bug Feature: Merge/Concatenate benchmark_this Request that this pull request be benchmarked to check if it introduces performance shifts labels Sep 9, 2025
Copy link
Contributor

github-actions bot commented Sep 9, 2025

⏱️ Performance Benchmark Report: 1e8e4ad

Performance shifts

Full benchmark results

Benchmarks that have stayed the same:

| Change   | Before [005b51ac]    | After [1e8e4ad7]    |   Ratio | Benchmark (Parameter)                                                                       |
|----------|----------------------|---------------------|---------|---------------------------------------------------------------------------------------------|
|          | 20.7±0.2ms           | 20.9±0.2ms          |    1.01 | aggregate_collapse.Aggregation.time_aggregated_by_COUNT(False)                              |
|          | 52.7±1ms             | 52.7±0.7ms          |    1    | aggregate_collapse.Aggregation.time_aggregated_by_COUNT(True)                               |
|          | 34.3±0.1ms           | 34.4±0.4ms          |    1    | aggregate_collapse.Aggregation.time_aggregated_by_FAST_PERCENTILE(False)                    |
|          | 165±2ms              | 168±1ms             |    1.02 | aggregate_collapse.Aggregation.time_aggregated_by_FAST_PERCENTILE(True)                     |
|          | 22.7±0.2ms           | 23.1±0.4ms          |    1.02 | aggregate_collapse.Aggregation.time_aggregated_by_GMEAN(False)                              |
|          | 31.4±0.5ms           | 31.5±0.2ms          |    1    | aggregate_collapse.Aggregation.time_aggregated_by_GMEAN(True)                               |
|          | 23.0±0.2ms           | 23.2±0.4ms          |    1.01 | aggregate_collapse.Aggregation.time_aggregated_by_HMEAN(False)                              |
|          | 31.7±0.6ms           | 31.5±0.4ms          |    0.99 | aggregate_collapse.Aggregation.time_aggregated_by_HMEAN(True)                               |
|          | 20.6±0.2ms           | 21.0±0.4ms          |    1.02 | aggregate_collapse.Aggregation.time_aggregated_by_MAX(False)                                |
|          | 45.2±0.9ms           | 44.7±0.6ms          |    0.99 | aggregate_collapse.Aggregation.time_aggregated_by_MAX(True)                                 |
|          | 125±0.6ms            | 126±0.6ms           |    1.01 | aggregate_collapse.Aggregation.time_aggregated_by_MAX_RUN(False)                            |
|          | 126±0.8ms            | 127±1ms             |    1.01 | aggregate_collapse.Aggregation.time_aggregated_by_MAX_RUN(True)                             |
|          | 21.5±0.2ms           | 21.9±0.2ms          |    1.02 | aggregate_collapse.Aggregation.time_aggregated_by_MEAN(False)                               |
|          | 47.8±0.6ms           | 48.5±0.9ms          |    1.01 | aggregate_collapse.Aggregation.time_aggregated_by_MEAN(True)                                |
|          | 23.0±0.3ms           | 23.1±0.1ms          |    1.01 | aggregate_collapse.Aggregation.time_aggregated_by_MEDIAN(False)                             |
|          | 56.8±0.5ms           | 57.8±0.8ms          |    1.02 | aggregate_collapse.Aggregation.time_aggregated_by_MEDIAN(True)                              |
|          | 20.6±0.08ms          | 20.9±0.1ms          |    1.01 | aggregate_collapse.Aggregation.time_aggregated_by_MIN(False)                                |
|          | 44.3±0.5ms           | 46.0±1ms            |    1.04 | aggregate_collapse.Aggregation.time_aggregated_by_MIN(True)                                 |
|          | 1.07±0.01s           | 1.08±0.01s          |    1    | aggregate_collapse.Aggregation.time_aggregated_by_PEAK(False)                               |
|          | 1.07±0.01s           | 1.07±0.01s          |    1    | aggregate_collapse.Aggregation.time_aggregated_by_PEAK(True)                                |
|          | 212±4ms              | 214±4ms             |    1.01 | aggregate_collapse.Aggregation.time_aggregated_by_PERCENTILE(False)                         |
|          | 345±5ms              | 342±10ms            |    0.99 | aggregate_collapse.Aggregation.time_aggregated_by_PERCENTILE(True)                          |
|          | 21.9±0.2ms           | 21.8±0.3ms          |    1    | aggregate_collapse.Aggregation.time_aggregated_by_PROPORTION(False)                         |
|          | 29.7±0.3ms           | 30.3±0.6ms          |    1.02 | aggregate_collapse.Aggregation.time_aggregated_by_PROPORTION(True)                          |
|          | 21.8±0.07ms          | 22.2±0.3ms          |    1.02 | aggregate_collapse.Aggregation.time_aggregated_by_RMS(False)                                |
|          | 59.5±1ms             | 59.6±2ms            |    1    | aggregate_collapse.Aggregation.time_aggregated_by_RMS(True)                                 |
|          | 22.6±0.2ms           | 23.0±0.2ms          |    1.02 | aggregate_collapse.Aggregation.time_aggregated_by_STD_DEV(False)                            |
|          | 61.8±0.8ms           | 61.9±1ms            |    1    | aggregate_collapse.Aggregation.time_aggregated_by_STD_DEV(True)                             |
|          | 22.6±0.2ms           | 22.5±0.5ms          |    1    | aggregate_collapse.Aggregation.time_aggregated_by_VARIANCE(False)                           |
|          | 58.1±0.6ms           | 58.0±1ms            |    1    | aggregate_collapse.Aggregation.time_aggregated_by_VARIANCE(True)                            |
|          | 7.79±0.1ms           | 7.88±0.1ms          |    1.01 | aggregate_collapse.Aggregation.time_collapsed_by_COUNT(False)                               |
|          | 21.5±0.2ms           | 22.0±0.3ms          |    1.02 | aggregate_collapse.Aggregation.time_collapsed_by_COUNT(True)                                |
|          | 19.6±0.2ms           | 19.7±0.1ms          |    1    | aggregate_collapse.Aggregation.time_collapsed_by_FAST_PERCENTILE(False)                     |
|          | 123±2ms              | 125±3ms             |    1.02 | aggregate_collapse.Aggregation.time_collapsed_by_FAST_PERCENTILE(True)                      |
|          | 8.13±0.09ms          | 8.22±0.1ms          |    1.01 | aggregate_collapse.Aggregation.time_collapsed_by_GMEAN(False)                               |
|          | 20.3±0.4ms           | 20.4±0.9ms          |    1.01 | aggregate_collapse.Aggregation.time_collapsed_by_GMEAN(True)                                |
|          | 7.99±0.1ms           | 8.04±0.1ms          |    1.01 | aggregate_collapse.Aggregation.time_collapsed_by_HMEAN(False)                               |
|          | 19.8±0.4ms           | 20.3±0.7ms          |    1.03 | aggregate_collapse.Aggregation.time_collapsed_by_HMEAN(True)                                |
|          | 7.66±0.07ms          | 7.75±0.04ms         |    1.01 | aggregate_collapse.Aggregation.time_collapsed_by_MAX(False)                                 |
|          | 20.3±0.2ms           | 20.8±0.2ms          |    1.03 | aggregate_collapse.Aggregation.time_collapsed_by_MAX(True)                                  |
|          | 24.3±0.3ms           | 24.2±0.2ms          |    1    | aggregate_collapse.Aggregation.time_collapsed_by_MAX_RUN(False)                             |
|          | 34.2±0.4ms           | 35.0±0.7ms          |    1.02 | aggregate_collapse.Aggregation.time_collapsed_by_MAX_RUN(True)                              |
|          | 8.06±0.08ms          | 7.89±0.09ms         |    0.98 | aggregate_collapse.Aggregation.time_collapsed_by_MEAN(False)                                |
|          | 20.8±0.3ms           | 20.9±0.2ms          |    1.01 | aggregate_collapse.Aggregation.time_collapsed_by_MEAN(True)                                 |
|          | 8.98±0.1ms           | 9.00±0.1ms          |    1    | aggregate_collapse.Aggregation.time_collapsed_by_MEDIAN(False)                              |
|          | 22.9±0.6ms           | 22.8±0.4ms          |    1    | aggregate_collapse.Aggregation.time_collapsed_by_MEDIAN(True)                               |
|          | 7.71±0.1ms           | 7.77±0.1ms          |    1.01 | aggregate_collapse.Aggregation.time_collapsed_by_MIN(False)                                 |
|          | 20.5±0.4ms           | 20.3±0.2ms          |    0.99 | aggregate_collapse.Aggregation.time_collapsed_by_MIN(True)                                  |
|          | 529±5ms              | 530±4ms             |    1    | aggregate_collapse.Aggregation.time_collapsed_by_PEAK(False)                                |
|          | 536±4ms              | 537±5ms             |    1    | aggregate_collapse.Aggregation.time_collapsed_by_PEAK(True)                                 |
|          | 44.1±0.7ms           | 44.5±0.4ms          |    1.01 | aggregate_collapse.Aggregation.time_collapsed_by_PERCENTILE(False)                          |
|          | 134±1ms              | 133±1ms             |    0.99 | aggregate_collapse.Aggregation.time_collapsed_by_PERCENTILE(True)                           |
|          | 8.09±0.07ms          | 8.07±0.1ms          |    1    | aggregate_collapse.Aggregation.time_collapsed_by_PROPORTION(False)                          |
|          | 20.2±0.3ms           | 19.8±0.6ms          |    0.98 | aggregate_collapse.Aggregation.time_collapsed_by_PROPORTION(True)                           |
|          | 8.19±0.08ms          | 7.86±0.08ms         |    0.96 | aggregate_collapse.Aggregation.time_collapsed_by_RMS(False)                                 |
|          | 22.7±0.3ms           | 22.9±0.5ms          |    1.01 | aggregate_collapse.Aggregation.time_collapsed_by_RMS(True)                                  |
|          | 8.33±0.05ms          | 8.10±0.04ms         |    0.97 | aggregate_collapse.Aggregation.time_collapsed_by_STD_DEV(False)                             |
|          | 22.3±0.4ms           | 22.0±0.4ms          |    0.99 | aggregate_collapse.Aggregation.time_collapsed_by_STD_DEV(True)                              |
|          | 8.31±0.08ms          | 8.13±0.08ms         |    0.98 | aggregate_collapse.Aggregation.time_collapsed_by_VARIANCE(False)                            |
|          | 21.6±0.1ms           | 21.8±0.4ms          |    1.01 | aggregate_collapse.Aggregation.time_collapsed_by_VARIANCE(True)                             |
|          | 22.1±0.4ms           | 22.4±0.2ms          |    1.01 | aggregate_collapse.WeightedAggregation.time_w_aggregated_by_MEAN(False)                     |
|          | 82.8±1ms             | 82.9±0.9ms          |    1    | aggregate_collapse.WeightedAggregation.time_w_aggregated_by_MEAN(True)                      |
|          | 22.0±0.3ms           | 22.7±0.2ms          |    1.03 | aggregate_collapse.WeightedAggregation.time_w_aggregated_by_RMS(False)                      |
|          | 94.3±2ms             | 94.1±1ms            |    1    | aggregate_collapse.WeightedAggregation.time_w_aggregated_by_RMS(True)                       |
|          | 21.1±0.2ms           | 21.5±0.3ms          |    1.02 | aggregate_collapse.WeightedAggregation.time_w_aggregated_by_SUM(False)                      |
|          | 55.1±0.7ms           | 55.7±0.9ms          |    1.01 | aggregate_collapse.WeightedAggregation.time_w_aggregated_by_SUM(True)                       |
|          | 8.38±0.09ms          | 8.28±0.05ms         |    0.99 | aggregate_collapse.WeightedAggregation.time_w_collapsed_by_MEAN(False)                      |
|          | 26.9±0.5ms           | 26.6±0.8ms          |    0.99 | aggregate_collapse.WeightedAggregation.time_w_collapsed_by_MEAN(True)                       |
|          | 8.24±0.2ms           | 8.23±0.1ms          |    1    | aggregate_collapse.WeightedAggregation.time_w_collapsed_by_RMS(False)                       |
|          | 28.3±0.3ms           | 28.5±0.6ms          |    1.01 | aggregate_collapse.WeightedAggregation.time_w_collapsed_by_RMS(True)                        |
|          | 7.86±0.1ms           | 7.88±0.08ms         |    1    | aggregate_collapse.WeightedAggregation.time_w_collapsed_by_SUM(False)                       |
|          | 22.7±0.6ms           | 23.3±0.7ms          |    1.03 | aggregate_collapse.WeightedAggregation.time_w_collapsed_by_SUM(True)                        |
|          | 222±3ms              | 222±4ms             |    1    | aggregate_collapse.WeightedAggregation.time_w_collapsed_by_WPERCENTILE(False)               |
|          | 283±4ms              | 282±4ms             |    1    | aggregate_collapse.WeightedAggregation.time_w_collapsed_by_WPERCENTILE(True)                |
|          | 1.10±0.02ms          | 1.08±0ms            |    0.98 | cube.CubeCreation.time_create(False, 'construct')                                           |
|          | 389±6μs              | 382±3μs             |    0.98 | cube.CubeCreation.time_create(False, 'instantiate')                                         |
|          | 1.91±0.02ms          | 1.96±0.02ms         |    1.03 | cube.CubeCreation.time_create(True, 'construct')                                            |
|          | 1.39±0.02ms          | 1.38±0.02ms         |    0.99 | cube.CubeCreation.time_create(True, 'instantiate')                                          |
|          | 88.0±0.9ms           | 88.5±1ms            |    1.01 | cube.CubeEquality.time_equality(False, False, 'all_equal')                                  |
|          | 24.8±1ms             | 25.4±1ms            |    1.02 | cube.CubeEquality.time_equality(False, False, 'coord_inequality')                           |
|          | 98.6±1ms             | 102±1ms             |    1.03 | cube.CubeEquality.time_equality(False, False, 'data_inequality')                            |
|          | 16.5±0.5μs           | 16.1±0.1μs          |    0.97 | cube.CubeEquality.time_equality(False, False, 'metadata_inequality')                        |
|          | 90.1±2ms             | 91.0±2ms            |    1.01 | cube.CubeEquality.time_equality(False, True, 'all_equal')                                   |
|          | 27.7±0.7ms           | 29.0±1ms            |    1.05 | cube.CubeEquality.time_equality(False, True, 'coord_inequality')                            |
|          | 102±1ms              | 104±1ms             |    1.02 | cube.CubeEquality.time_equality(False, True, 'data_inequality')                             |
|          | 16.4±0.5μs           | 16.5±0.2μs          |    1.01 | cube.CubeEquality.time_equality(False, True, 'metadata_inequality')                         |
|          | 170±3ms              | 173±2ms             |    1.02 | cube.CubeEquality.time_equality(True, False, 'all_equal')                                   |
|          | 65.4±1ms             | 67.7±0.6ms          |    1.03 | cube.CubeEquality.time_equality(True, False, 'coord_inequality')                            |
|          | 200±2ms              | 201±3ms             |    1.01 | cube.CubeEquality.time_equality(True, False, 'data_inequality')                             |
|          | 51.0±0.4μs           | 50.5±0.4μs          |    0.99 | cube.CubeEquality.time_equality(True, False, 'metadata_inequality')                         |
|          | 244±4ms              | 245±4ms             |    1.01 | cube.CubeEquality.time_equality(True, True, 'all_equal')                                    |
|          | 140±3ms              | 142±2ms             |    1.02 | cube.CubeEquality.time_equality(True, True, 'coord_inequality')                             |
|          | 276±4ms              | 279±3ms             |    1.01 | cube.CubeEquality.time_equality(True, True, 'data_inequality')                              |
|          | 361±4μs              | 363±4μs             |    1.01 | cube.CubeEquality.time_equality(True, True, 'metadata_inequality')                          |
|          | 798±10μs             | 796±4μs             |    1    | import_iris.Iris.time__concatenate                                                          |
|          | 184±2μs              | 184±2μs             |    1    | import_iris.Iris.time__constraints                                                          |
|          | 112±0.6μs            | 114±0.7μs           |    1.02 | import_iris.Iris.time__data_manager                                                         |
|          | 91.1±0.5μs           | 91.1±0.3μs          |    1    | import_iris.Iris.time__deprecation                                                          |
|          | 162±0.7μs            | 163±1μs             |    1.01 | import_iris.Iris.time__lazy_data                                                            |
|          | 893±4μs              | 897±3μs             |    1.01 | import_iris.Iris.time__merge                                                                |
|          | 73.4±0.6μs           | 74.0±0.2μs          |    1.01 | import_iris.Iris.time__representation                                                       |
|          | 613±4μs              | 613±6μs             |    1    | import_iris.Iris.time_analysis                                                              |
|          | 138±1μs              | 144±2μs             |    1.04 | import_iris.Iris.time_analysis__area_weighted                                               |
|          | 106±1μs              | 107±0.9μs           |    1.01 | import_iris.Iris.time_analysis__grid_angles                                                 |
|          | 244±2μs              | 249±2μs             |    1.02 | import_iris.Iris.time_analysis__interpolation                                               |
|          | 193±2μs              | 192±3μs             |    0.99 | import_iris.Iris.time_analysis__regrid                                                      |
|          | 110±1μs              | 110±0.7μs           |    1    | import_iris.Iris.time_analysis__scipy_interpolate                                           |
|          | 138±2μs              | 138±2μs             |    1    | import_iris.Iris.time_analysis_calculus                                                     |
|          | 320±2μs              | 329±3μs             |    1.03 | import_iris.Iris.time_analysis_cartography                                                  |
|          | 91.6±1μs             | 90.6±1μs            |    0.99 | import_iris.Iris.time_analysis_geomerty                                                     |
|          | 211±2μs              | 217±3μs             |    1.03 | import_iris.Iris.time_analysis_maths                                                        |
|          | 93.5±0.4μs           | 95.1±0.5μs          |    1.02 | import_iris.Iris.time_analysis_stats                                                        |
|          | 172±2μs              | 171±1μs             |    0.99 | import_iris.Iris.time_analysis_trajectory                                                   |
|          | 318±5μs              | 318±1μs             |    1    | import_iris.Iris.time_aux_factory                                                           |
|          | 79.4±0.7μs           | 80.2±0.4μs          |    1.01 | import_iris.Iris.time_common                                                                |
|          | 159±1μs              | 162±4μs             |    1.02 | import_iris.Iris.time_common_lenient                                                        |
|          | 1.34±0.01ms          | 1.33±0.01ms         |    0.99 | import_iris.Iris.time_common_metadata                                                       |
|          | 166±2μs              | 168±1μs             |    1.01 | import_iris.Iris.time_common_mixin                                                          |
|          | 1.16±0ms             | 1.16±0ms            |    1.01 | import_iris.Iris.time_common_resolve                                                        |
|          | 197±1μs              | 199±1μs             |    1.01 | import_iris.Iris.time_config                                                                |
|          | 129±1μs              | 129±3μs             |    1    | import_iris.Iris.time_coord_categorisation                                                  |
|          | 379±1μs              | 384±10μs            |    1.01 | import_iris.Iris.time_coord_systems                                                         |
|          | 758±4μs              | 758±3μs             |    1    | import_iris.Iris.time_coords                                                                |
|          | 684±6μs              | 696±6μs             |    1.02 | import_iris.Iris.time_cube                                                                  |
|          | 244±2μs              | 243±6μs             |    1    | import_iris.Iris.time_exceptions                                                            |
|          | 73.4±0.8μs           | 74.0±0.6μs          |    1.01 | import_iris.Iris.time_experimental                                                          |
|          | 181±3μs              | 181±3μs             |    1    | import_iris.Iris.time_fileformats                                                           |
|          | 263±6μs              | 261±4μs             |    0.99 | import_iris.Iris.time_fileformats__ff                                                       |
|          | 2.54±0.01ms          | 2.59±0.03ms         |    1.02 | import_iris.Iris.time_fileformats__ff_cross_references                                      |
|          | 75.9±1μs             | 75.8±0.4μs          |    1    | import_iris.Iris.time_fileformats__pp_lbproc_pairs                                          |
|          | 111±0.7μs            | 113±0.8μs           |    1.02 | import_iris.Iris.time_fileformats_abf                                                       |
|          | 443±7μs              | 440±5μs             |    0.99 | import_iris.Iris.time_fileformats_cf                                                        |
|          | 4.69±0.04ms          | 4.72±0.07ms         |    1.01 | import_iris.Iris.time_fileformats_dot                                                       |
|          | 70.1±0.4μs           | 71.5±1μs            |    1.02 | import_iris.Iris.time_fileformats_name                                                      |
|          | 250±2μs              | 256±6μs             |    1.03 | import_iris.Iris.time_fileformats_name_loaders                                              |
|          | 113±0.8μs            | 115±1μs             |    1.02 | import_iris.Iris.time_fileformats_netcdf                                                    |
|          | 118±0.5μs            | 123±1μs             |    1.04 | import_iris.Iris.time_fileformats_nimrod                                                    |
|          | 205±0.6μs            | 211±1μs             |    1.03 | import_iris.Iris.time_fileformats_nimrod_load_rules                                         |
|          | 793±6μs              | 805±10μs            |    1.02 | import_iris.Iris.time_fileformats_pp                                                        |
|          | 180±2μs              | 187±2μs             |    1.04 | import_iris.Iris.time_fileformats_pp_load_rules                                             |
|          | 135±3μs              | 140±3μs             |    1.04 | import_iris.Iris.time_fileformats_pp_save_rules                                             |
|          | 536±4μs              | 546±3μs             |    1.02 | import_iris.Iris.time_fileformats_rules                                                     |
|          | 217±1μs              | 226±3μs             |    1.04 | import_iris.Iris.time_fileformats_structured_array_identification                           |
|          | 79.7±0.3μs           | 80.3±0.5μs          |    1.01 | import_iris.Iris.time_fileformats_um                                                        |
|          | 156±2μs              | 162±1μs             |    1.04 | import_iris.Iris.time_fileformats_um__fast_load                                             |
|          | 135±0.7μs            | 138±0.8μs           |    1.02 | import_iris.Iris.time_fileformats_um__fast_load_structured_fields                           |
|          | 71.4±0.3μs           | 72.6±0.5μs          |    1.02 | import_iris.Iris.time_fileformats_um__ff_replacement                                        |
|          | 77.4±0.4μs           | 79.4±0.4μs          |    1.03 | import_iris.Iris.time_fileformats_um__optimal_array_structuring                             |
|          | 941±7μs              | 952±9μs             |    1.01 | import_iris.Iris.time_fileformats_um_cf_map                                                 |
|          | 136±2μs              | 138±2μs             |    1.01 | import_iris.Iris.time_io                                                                    |
|          | 174±3μs              | 173±1μs             |    1    | import_iris.Iris.time_io_format_picker                                                      |
|          | 210±1μs              | 212±0.7μs           |    1.01 | import_iris.Iris.time_iris                                                                  |
|          | 124±1μs              | 126±1μs             |    1.02 | import_iris.Iris.time_iterate                                                               |
|          | 8.00±0.05ms          | 8.03±0.03ms         |    1    | import_iris.Iris.time_palette                                                               |
|          | 1.76±0.02ms          | 1.79±0.01ms         |    1.01 | import_iris.Iris.time_plot                                                                  |
|          | 217±2μs              | 217±1μs             |    1    | import_iris.Iris.time_quickplot                                                             |
|          | 2.13±0.01ms          | 2.17±0.02ms         |    1.02 | import_iris.Iris.time_std_names                                                             |
|          | 1.83±0.02ms          | 1.82±0.01ms         |    1    | import_iris.Iris.time_symbols                                                               |
|          | 92.4±0.8ms           | 93.0±0.8ms          |    1.01 | import_iris.Iris.time_tests                                                                 |
|          | 249±0.6μs            | 251±0.3μs           |    1.01 | import_iris.Iris.time_third_party_cartopy                                                   |
|          | 5.02±0.05ms          | 5.09±0.02ms         |    1.01 | import_iris.Iris.time_third_party_cf_units                                                  |
|          | 116±0.6μs            | 116±0.7μs           |    1    | import_iris.Iris.time_third_party_cftime                                                    |
|          | 2.67±0.02ms          | 2.70±0.01ms         |    1.01 | import_iris.Iris.time_third_party_matplotlib                                                |
|          | 1.35±0.01ms          | 1.32±0.01ms         |    0.98 | import_iris.Iris.time_third_party_numpy                                                     |
|          | 167±1μs              | 169±1μs             |    1.02 | import_iris.Iris.time_third_party_scipy                                                     |
|          | 98.5±2μs             | 98.6±2μs            |    1    | import_iris.Iris.time_time                                                                  |
|          | 373±3μs              | 388±9μs             |    1.04 | import_iris.Iris.time_util                                                                  |
|          | 71.1±0.7μs           | 70.0±0.8μs          |    0.99 | iterate.IZip.time_izip                                                                      |
|          | 9.97±0.2ms           | 10.0±0.04ms         |    1    | load.LoadAndRealise.time_load((1280, 960, 5), False, 'FF')                                  |
|          | 16.6±0.2ms           | 15.7±0.7ms          |    0.94 | load.LoadAndRealise.time_load((1280, 960, 5), False, 'NetCDF')                              |
|          | 9.97±0.1ms           | 10.1±0.1ms          |    1.01 | load.LoadAndRealise.time_load((1280, 960, 5), False, 'PP')                                  |
|          | 10.1±0.09ms          | 9.92±0.06ms         |    0.99 | load.LoadAndRealise.time_load((1280, 960, 5), True, 'FF')                                   |
|          | 13.2±0.04ms          | 13.3±0.08ms         |    1.01 | load.LoadAndRealise.time_load((1280, 960, 5), True, 'NetCDF')                               |
|          | 9.86±0.1ms           | 10.2±0.08ms         |    1.03 | load.LoadAndRealise.time_load((1280, 960, 5), True, 'PP')                                   |
|          | 1.50±0.01s           | 1.51±0.02s          |    1    | load.LoadAndRealise.time_load((2, 2, 1000), False, 'FF')                                    |
|          | 12.4±0.2ms           | 12.3±0.1ms          |    0.99 | load.LoadAndRealise.time_load((2, 2, 1000), False, 'NetCDF')                                |
|          | 1.53±0.01s           | 1.53±0.01s          |    1    | load.LoadAndRealise.time_load((2, 2, 1000), False, 'PP')                                    |
|          | 1.50±0s              | 1.51±0.02s          |    1.01 | load.LoadAndRealise.time_load((2, 2, 1000), True, 'FF')                                     |
|          | 12.2±0.09ms          | 12.3±0.1ms          |    1.01 | load.LoadAndRealise.time_load((2, 2, 1000), True, 'NetCDF')                                 |
|          | 1.53±0.01s           | 1.52±0s             |    1    | load.LoadAndRealise.time_load((2, 2, 1000), True, 'PP')                                     |
|          | 5.35±0.05ms          | 5.34±0.03ms         |    1    | load.LoadAndRealise.time_load((50, 50, 2), False, 'FF')                                     |
|          | 11.9±0.1ms           | 12.1±0.1ms          |    1.02 | load.LoadAndRealise.time_load((50, 50, 2), False, 'NetCDF')                                 |
|          | 5.32±0.03ms          | 5.33±0.03ms         |    1    | load.LoadAndRealise.time_load((50, 50, 2), False, 'PP')                                     |
|          | 5.31±0.05ms          | 5.32±0.02ms         |    1    | load.LoadAndRealise.time_load((50, 50, 2), True, 'FF')                                      |
|          | 11.8±0.05ms          | 11.9±0.1ms          |    1.01 | load.LoadAndRealise.time_load((50, 50, 2), True, 'NetCDF')                                  |
|          | 5.30±0.05ms          | 5.33±0.03ms         |    1    | load.LoadAndRealise.time_load((50, 50, 2), True, 'PP')                                      |
|          | 23.3±1ms             | 22.6±1ms            |    0.97 | load.LoadAndRealise.time_realise((1280, 960, 5), False, 'FF')                               |
|          | 23.4±0.2ms           | 23.7±0.4ms          |    1.02 | load.LoadAndRealise.time_realise((1280, 960, 5), False, 'NetCDF')                           |
|          | 12.1±1ms             | 12.3±1ms            |    1.02 | load.LoadAndRealise.time_realise((1280, 960, 5), False, 'PP')                               |
|          | 28.1±1ms             | 28.3±0.9ms          |    1.01 | load.LoadAndRealise.time_realise((1280, 960, 5), True, 'FF')                                |
|          | 71.2±3ms             | 71.4±3ms            |    1    | load.LoadAndRealise.time_realise((1280, 960, 5), True, 'NetCDF')                            |
|          | 27.9±1ms             | 27.1±1ms            |    0.97 | load.LoadAndRealise.time_realise((1280, 960, 5), True, 'PP')                                |
|          | 598±4ms              | 603±3ms             |    1.01 | load.LoadAndRealise.time_realise((2, 2, 1000), False, 'FF')                                 |
|          | 3.41±0.1ms           | 3.45±0.1ms          |    1.01 | load.LoadAndRealise.time_realise((2, 2, 1000), False, 'NetCDF')                             |
|          | 610±2ms              | 601±5ms             |    0.99 | load.LoadAndRealise.time_realise((2, 2, 1000), False, 'PP')                                 |
|          | 610±2ms              | 615±3ms             |    1.01 | load.LoadAndRealise.time_realise((2, 2, 1000), True, 'FF')                                  |
|          | 3.54±0.1ms           | 3.51±0.08ms         |    0.99 | load.LoadAndRealise.time_realise((2, 2, 1000), True, 'NetCDF')                              |
|          | 607±4ms              | 611±4ms             |    1.01 | load.LoadAndRealise.time_realise((2, 2, 1000), True, 'PP')                                  |
|          | 2.10±0.05ms          | 2.05±0.07ms         |    0.98 | load.LoadAndRealise.time_realise((50, 50, 2), False, 'FF')                                  |
|          | 3.30±0.06ms          | 3.36±0.1ms          |    1.02 | load.LoadAndRealise.time_realise((50, 50, 2), False, 'NetCDF')                              |
|          | 2.04±0.06ms          | 2.11±0.08ms         |    1.04 | load.LoadAndRealise.time_realise((50, 50, 2), False, 'PP')                                  |
|          | 2.12±0.05ms          | 2.08±0.06ms         |    0.98 | load.LoadAndRealise.time_realise((50, 50, 2), True, 'FF')                                   |
|          | 3.39±0.08ms          | 3.42±0.1ms          |    1.01 | load.LoadAndRealise.time_realise((50, 50, 2), True, 'NetCDF')                               |
|          | 2.09±0.07ms          | 2.07±0.05ms         |    0.99 | load.LoadAndRealise.time_realise((50, 50, 2), True, 'PP')                                   |
|          | 341±2ms              | 341±1ms             |    1    | load.ManyCubes.time_many_cube_load                                                          |
|          | 87.1±2ms             | 86.6±0.6ms          |    0.99 | load.ManyVars.time_many_var_load                                                            |
|          | 9.99±0.09ms          | 10.0±0.1ms          |    1.01 | load.STASHConstraint.time_stash_constraint((1280, 960, 5), 'FF')                            |
|          | 10.2±0.1ms           | 10.2±0.06ms         |    1    | load.STASHConstraint.time_stash_constraint((1280, 960, 5), 'PP')                            |
|          | 1.52±0.02s           | 1.53±0.01s          |    1    | load.STASHConstraint.time_stash_constraint((2, 2, 1000), 'FF')                              |
|          | 1.54±0.01s           | 1.54±0.01s          |    1    | load.STASHConstraint.time_stash_constraint((2, 2, 1000), 'PP')                              |
|          | 5.30±0.05ms          | 5.31±0.02ms         |    1    | load.STASHConstraint.time_stash_constraint((2, 2, 2), 'FF')                                 |
|          | 5.26±0.03ms          | 5.39±0.07ms         |    1.03 | load.STASHConstraint.time_stash_constraint((2, 2, 2), 'PP')                                 |
|          | 8.91±0.09ms          | 9.01±0.09ms         |    1.01 | load.StructuredFF.time_structured_load((1280, 960, 5), False)                               |
|          | 5.63±0.02ms          | 5.61±0.03ms         |    1    | load.StructuredFF.time_structured_load((1280, 960, 5), True)                                |
|          | 1.50±0.01s           | 1.49±0.01s          |    1    | load.StructuredFF.time_structured_load((2, 2, 1000), False)                                 |
|          | 423±4ms              | 423±6ms             |    1    | load.StructuredFF.time_structured_load((2, 2, 1000), True)                                  |
|          | 4.35±0.01ms          | 4.41±0.01ms         |    1.01 | load.StructuredFF.time_structured_load((2, 2, 2), False)                                    |
|          | 4.26±0.01ms          | 4.25±0.02ms         |    1    | load.StructuredFF.time_structured_load((2, 2, 2), True)                                     |
|          | 153±1ms              | 155±2ms             |    1.01 | load.TimeConstraint.time_time_constraint(20, 'FF')                                          |
|          | 15.0±0.2ms           | 15.5±0.2ms          |    1.03 | load.TimeConstraint.time_time_constraint(20, 'NetCDF')                                      |
|          | 155±1ms              | 158±4ms             |    1.02 | load.TimeConstraint.time_time_constraint(20, 'PP')                                          |
|          | 30.6±0.07ms          | 31.0±0.2ms          |    1.01 | load.TimeConstraint.time_time_constraint(3, 'FF')                                           |
|          | 14.8±0.2ms           | 14.8±0.1ms          |    1    | load.TimeConstraint.time_time_constraint(3, 'NetCDF')                                       |
|          | 31.2±0.5ms           | 31.2±0.5ms          |    1    | load.TimeConstraint.time_time_constraint(3, 'PP')                                           |
|          | 18.7±0.3ms           | 19.0±0.4ms          |    1.02 | load.ugrid.BasicLoading.time_load_file(1)                                                   |
|          | 49.4±0.7ms           | 49.2±0.8ms          |    0.99 | load.ugrid.BasicLoading.time_load_file(200000)                                              |
|          | 8.85±0.2ms           | 8.94±0.2ms          |    1.01 | load.ugrid.BasicLoading.time_load_mesh(1)                                                   |
|          | 17.0±0.5ms           | 17.0±0.3ms          |    1    | load.ugrid.BasicLoading.time_load_mesh(200000)                                              |
|          | 19.2±0.2ms           | 18.8±0.4ms          |    0.98 | load.ugrid.BasicLoadingTime.time_load_file(1)                                               |
|          | 19.3±0.5ms           | 19.0±0.5ms          |    0.98 | load.ugrid.BasicLoadingTime.time_load_file(200000)                                          |
|          | 8.87±0.2ms           | 8.82±0.2ms          |    0.99 | load.ugrid.BasicLoadingTime.time_load_mesh(1)                                               |
|          | 11.9±0.4ms           | 11.8±0.4ms          |    1    | load.ugrid.BasicLoadingTime.time_load_mesh(200000)                                          |
|          | 21.2±0.4ms           | 20.7±0.3ms          |    0.98 | load.ugrid.Callback.time_load_file_callback(1)                                              |
|          | 60.4±0.6ms           | 59.8±0.8ms          |    0.99 | load.ugrid.Callback.time_load_file_callback(200000)                                         |
|          | 20.7±0.3ms           | 21.0±0.4ms          |    1.01 | load.ugrid.CallbackTime.time_load_file_callback(1)                                          |
|          | 22.3±0.6ms           | 22.9±0.4ms          |    1.03 | load.ugrid.CallbackTime.time_load_file_callback(200000)                                     |
|          | 3.45±0.1ms           | 3.62±0.1ms          |    1.05 | load.ugrid.DataRealisation.time_realise_data(10000)                                         |
|          | 4.85±0.4ms           | 5.74±0.7ms          |    1.18 | load.ugrid.DataRealisation.time_realise_data(200000)                                        |
|          | 38.1±0.9ms           | 39.3±1ms            |    1.03 | load.ugrid.DataRealisationTime.time_realise_data(10000)                                     |
|          | 793±6ms              | 784±10ms            |    0.99 | load.ugrid.DataRealisationTime.time_realise_data(200000)                                    |
|          | 1.37±0.03s           | 1.36±0.04s          |    0.99 | merge_concat.Concatenate.time_concatenate(False)                                            |
|          | 426±6ms              | 429±5ms             |    1.01 | merge_concat.Concatenate.time_concatenate(True)                                             |
|          | 2.42±0G              | 2.42±0G             |    1    | merge_concat.Concatenate.tracemalloc_concatenate(False)                                     |
|          | 110±5M               | 110±4M              |    1    | merge_concat.Concatenate.tracemalloc_concatenate(True)                                      |
|          | 40.8±6ms             | 40.1±1ms            |    0.98 | merge_concat.Merge.time_merge                                                               |
|          | 126±0.03M            | 126±0.03M           |    1    | merge_concat.Merge.tracemalloc_merge                                                        |
|          | 352±2ns              | 355±2ns             |    1.01 | mesh.utils.regions_combine.CombineRegionsComputeRealData.time_compute_data(50)              |
|          | 197±1ms              | 199±2ms             |    1.01 | mesh.utils.regions_combine.CombineRegionsComputeRealData.time_compute_data(500)             |
|          | 771±0.5k             | 771±0.5k            |    1    | mesh.utils.regions_combine.CombineRegionsComputeRealData.tracemalloc_compute_data(50)       |
|          | 60.2±0M              | 60.2±0M             |    1    | mesh.utils.regions_combine.CombineRegionsComputeRealData.tracemalloc_compute_data(500)      |
|          | 18.9±0.4ms           | 18.8±0.2ms          |    0.99 | mesh.utils.regions_combine.CombineRegionsCreateCube.time_create_combined_cube(50)           |
|          | 22.5±0.4ms           | 21.8±0.4ms          |    0.97 | mesh.utils.regions_combine.CombineRegionsCreateCube.time_create_combined_cube(500)          |
|          | 1.27±0.04M           | 1.27±0.04M          |    1    | mesh.utils.regions_combine.CombineRegionsCreateCube.tracemalloc_create_combined_cube(50)    |
|          | 25±0.04M             | 25±0.04M            |    1    | mesh.utils.regions_combine.CombineRegionsCreateCube.tracemalloc_create_combined_cube(500)   |
|          | 119±0.8ms            | 119±0.8ms           |    0.99 | mesh.utils.regions_combine.CombineRegionsFileStreamedCalc.time_stream_file2file(50)         |
|          | 580±4ms              | 580±3ms             |    1    | mesh.utils.regions_combine.CombineRegionsFileStreamedCalc.time_stream_file2file(500)        |
|          | 1.48±0.03M           | 1.46±0.1M           |    0.99 | mesh.utils.regions_combine.CombineRegionsFileStreamedCalc.tracemalloc_stream_file2file(50)  |
|          | 96.5±0.1M            | 96.5±0.03M          |    1    | mesh.utils.regions_combine.CombineRegionsFileStreamedCalc.tracemalloc_stream_file2file(500) |
|          | 77.6±0.9ms           | 77.9±2ms            |    1    | mesh.utils.regions_combine.CombineRegionsSaveData.time_save(50)                             |
|          | 532±5ms              | 536±5ms             |    1.01 | mesh.utils.regions_combine.CombineRegionsSaveData.time_save(500)                            |
|          | 1.43±0.02M           | 1.41±0.03M          |    0.99 | mesh.utils.regions_combine.CombineRegionsSaveData.tracemalloc_save(50)                      |
|          | 96.5±0.04M           | 96.5±0.02M          |    1    | mesh.utils.regions_combine.CombineRegionsSaveData.tracemalloc_save(500)                     |
|          | 2.1752849999999997   | 2.1752849999999997  |    1    | mesh.utils.regions_combine.CombineRegionsSaveData.track_filesize_saved(50)                  |
|          | 216.01528499999998   | 216.01528499999998  |    1    | mesh.utils.regions_combine.CombineRegionsSaveData.track_filesize_saved(500)                 |
|          | 6.50±0.02ms          | 6.55±0.04ms         |    1.01 | plot.AuxSort.time_aux_sort                                                                  |
|          | 79.3±2ms             | 78.3±1ms            |    0.99 | regridding.CurvilinearRegridding.time_regrid_pic                                            |
|          | 136±3M               | 136±3M              |    1    | regridding.CurvilinearRegridding.tracemalloc_regrid_pic                                     |
|          | 106±6ms              | 104±6ms             |    0.98 | regridding.HorizontalChunkedRegridding.time_regrid_area_w                                   |
|          | 61.5±7ms             | 61.5±5ms            |    1    | regridding.HorizontalChunkedRegridding.time_regrid_area_w_new_grid                          |
|          | 107±0.09M            | 107±0.07M           |    1    | regridding.HorizontalChunkedRegridding.tracemalloc_regrid_area_w                            |
|          | 147±0.04M            | 147±0.04M           |    1    | regridding.HorizontalChunkedRegridding.tracemalloc_regrid_area_w_new_grid                   |
|          | 4.68±0.04ms          | 4.74±0.05ms         |    1.01 | save.NetcdfSave.time_netcdf_save_cube(50, False)                                            |
|          | 82.6±1ms             | 86.1±2ms            |    1.04 | save.NetcdfSave.time_netcdf_save_cube(50, True)                                             |
|          | 41.2±0.7ms           | 42.3±0.4ms          |    1.03 | save.NetcdfSave.time_netcdf_save_cube(600, False)                                           |
|          | 467±5ms              | 472±5ms             |    1.01 | save.NetcdfSave.time_netcdf_save_cube(600, True)                                            |
|          | 87.1±0.6ns           | 86.5±0.5ns          |    0.99 | save.NetcdfSave.time_netcdf_save_mesh(50, False)                                            |
|          | 61.9±2ms             | 63.7±0.8ms          |    1.03 | save.NetcdfSave.time_netcdf_save_mesh(50, True)                                             |
|          | 87.3±0.3ns           | 86.7±0.4ns          |    0.99 | save.NetcdfSave.time_netcdf_save_mesh(600, False)                                           |
|          | 411±3ms              | 409±3ms             |    0.99 | save.NetcdfSave.time_netcdf_save_mesh(600, True)                                            |
|          | 31.8±0.3k            | 31.7±0.4k           |    1    | save.NetcdfSave.tracemalloc_netcdf_save(50, False)                                          |
|          | 1.81±0.2M            | 1.85±0.1M           |    1.02 | save.NetcdfSave.tracemalloc_netcdf_save(50, True)                                           |
|          | 31.8±0.5k            | 31.8±0.3k           |    1    | save.NetcdfSave.tracemalloc_netcdf_save(600, False)                                         |
|          | 225±20M              | 242±20M             |    1.08 | save.NetcdfSave.tracemalloc_netcdf_save(600, True)                                          |
|          | 39.0±0.4ms           | 39.0±0.8ms          |    1    | stats.PearsonR.time_lazy                                                                    |
|          | 9.33±0.2ms           | 9.09±0.3ms          |    0.97 | stats.PearsonR.time_real                                                                    |
|          | 29.2±1M              | 29.4±1M             |    1.01 | stats.PearsonR.tracemalloc_lazy                                                             |
|          | 18.3±0.01M           | 18.3±0.01M          |    1    | stats.PearsonR.tracemalloc_real                                                             |
|          | 25.2±0.5ms           | 24.5±0.7ms          |    0.97 | trajectory.TrajectoryInterpolation.time_trajectory_linear                                   |
|          | 61.9±0.9ms           | 60.9±0.5ms          |    0.98 | trajectory.TrajectoryInterpolation.time_trajectory_nearest                                  |
|          | 17.6±0.02M           | 17.6±0.02M          |    1    | trajectory.TrajectoryInterpolation.tracemalloc_trajectory_linear                            |
|          | 7.75±0.02M           | 7.75±0.02M          |    1    | trajectory.TrajectoryInterpolation.tracemalloc_trajectory_nearest                           |

Generated by GHA run 17577661483

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
benchmark_this Request that this pull request be benchmarked to check if it introduces performance shifts Feature: Merge/Concatenate Type: Bug
Projects
Status: No status
Development

Successfully merging this pull request may close these issues.

Unable to merge cubes with cell measures
1 participant