Patrick Peglar
Patrick Peglar
> self.assertEqual and self.assertRaisesRegex can be converted to pytest style Ok I have done that now + I think it looks pretty neat. https://github.com/SciTools/iris/pull/4933/commits/f6853deb1fa2695975fa049f24b3d40f92ff511e However, It may be appropriate for...
Meanwhile ... > Don't forget to re-target to `v3.3.x` Good spot. However, I think it's best to let #4976 go first, as that establishes the whatsnew section for 3.3.1. Rather...
Thanks @trexfeathers I have now **re-targetted and fixed, and I think will now be OK.** That includes fixing [the outstanding problem with the post-3.3 whatsnew](https://github.com/SciTools/iris/pull/4976#issuecomment-1257683528) : added an indent to...
Belated thanks @trexfeathers This was one of those occasions when I couldn't get a straight rebase to work for me. It is unavoidably tough when a rebase/squash interrupts the comment/response...
## API discussion The API (reference docs-build [here](https://pp-iris.readthedocs.io/en/nc_chunk_control/generated/api/iris/fileformats/netcdf.html#iris.fileformats.netcdf.ChunkControl)) is actually rather different from what I had expected to end up with. Here are some reasons.. I reasoned that if loading...
## Usage examples. This should really go in docs "somewhere" For now, I put usage examples in some stopgap [integration tests](https://github.com/SciTools/iris/blob/93b0917bd8152df163e5accaada75c6344e30c78/lib/iris/tests/integration/test_netcdf__chunk_control.py#L48-L90). An example that is working for me is, to...
As [noted on parent issue #3333](https://github.com/SciTools/iris/issues/3333#issuecomment-1293550863) , xarray already provides a (slightly less sophisticated) chunking control, and could be of use via #4994
> @bjlittle I really don't believe it makes sense to make iris-test-data importable. It just a big old data blob, and what's more it's only for developers. ok, maybe it's...
Also found we have another setting "Resources/doc_dir "that is still used [in this line](https://github.com/SciTools/iris/blob/0ff0dca3ce5b4de38a4ae02438bb00206e664460/lib/iris/tests/test_coding_standards.py#L31) Though it's not mentioned in site.cfg.template
Pure tests to start with -- proving that the test for `Cubelist._repr_html_()` **_will fail_**.