SciTools / iris

A powerful, format-agnostic, and community-driven Python package for analysing and visualising Earth science data
https://scitools-iris.readthedocs.io/en/stable/
BSD 3-Clause "New" or "Revised" License
635 stars 284 forks source link

Performance Shift(s): `3820ae21` #4625

Closed github-actions[bot] closed 2 years ago

github-actions[bot] commented 2 years ago

Benchmark comparison has identified performance shifts at commit 3820ae21 (#4618). Please review the report below and take corrective/congratulatory action as appropriate :slightly_smiling_face:

Performance shift report ``` before after ratio [35d97ed9] [3820ae21] - 2.60±0.8μs 2.00±0.2μs 0.77 experimental.ugrid.MeshLazy.time_return(6) - 0.359375 0.1640625 0.46 netcdf_save.NetcdfSave.track_addedmem_netcdf_save(1, False) - 0.09375 0.05859375 0.62 regions_combine.CombineRegionsCreateCube.track_addedmem_create_combined_cube(4) before after ratio [35d97ed9] [3820ae21] + 0.0 0.1328125 n/a regions_combine.CombineRegionsComputeRealData.track_addedmem_compute_data(4) ```

Generated by GHA run 1954280682

trexfeathers commented 2 years ago

The time_return benchmarks need to go - they're useless.

The track_addedmem benchmarks are solved by #4621.