SciTools / iris-esmf-regrid

A collection of structured and unstructured ESMF regridding schemes for Iris.
https://iris-esmf-regrid.readthedocs.io/en/latest
BSD 3-Clause "New" or "Revised" License
19 stars 17 forks source link

Benchmarking overhaul and pin Flake8 <6 #220

Closed trexfeathers closed 1 year ago

trexfeathers commented 1 year ago

A detailed restructure of the benchmarking architecture in line with what has already been done on SciTools/iris. The original architecture included several experiments which we have since iterated and improved within Iris.

Closes #210.

codecov[bot] commented 1 year ago

Codecov Report

Merging #220 (6957388) into main (10cb843) will not change coverage. The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #220   +/-   ##
=======================================
  Coverage   99.36%   99.36%           
=======================================
  Files          28       28           
  Lines        2829     2829           
=======================================
  Hits         2811     2811           
  Misses         18       18           

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

trexfeathers commented 1 year ago

Flake8 failure seems possibly related to pycqa/flake8#1739, but I have not yet worked out what in our config is causing the problem.

trexfeathers commented 1 year ago

See also https://github.com/PyCQA/flake8/pull/1758

trexfeathers commented 1 year ago

Flake8 failure seems possibly related to PyCQA/flake8#1739, but I have not yet worked out what in our config is causing the problem.

Happens even without any config at all in .flake8, so I reckon somethings wrong. I'll pin it

22a6f63fe1ea4990953c0ceb174bca1fcaae96e0

trexfeathers commented 1 year ago

Flake8 failure seems possibly related to pycqa/flake8#1739, but I have not yet worked out what in our config is causing the problem.

Should be fixed once PyCQA/flake8-import-order#190 is in a release.