Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Mergeback v3.0.x release feature branch #4276

Merged
merged 46 commits into from
Aug 10, 2021

Conversation

bjlittle
Copy link
Member

@bjlittle bjlittle commented Aug 9, 2021

🚀 Pull Request

Description

This PR merges the v3.0.x release feature branch back to main.

Do not squash commit this PR, please enable and perform a merge commit.

Closes #4264


Consult Iris pull request check list

bjlittle and others added 30 commits October 2, 2020 10:36
* Add release highlights and pin rc version

* review actions
* Add failing test of plotting

* Implement fix to pass test

* Update idiff to ignore irrelevant hyphens in path

* Update imagerepo (following docs)

* Update after review by @trexfeathers

* Add whatsnew entries

* Move whatsnew entries into correct file
* Minor phrasing change in 'Release candidate'.

* Before release deprecations.

* Whatsnew highlights section.
* Updated CF saver version in User Guide and docstring

* Remove references to CF version of the loader in docstrings

* Added whatsnew

* Pin cftime<1.3.0
* migrate from travis-ci to cirrus-ci

* added whatsnew entries
* Use Title Case Capitalisation for Documentation

* add whatsnew enter
* requirements pip to conda

* use pip install over develop

* default PY_VER to python versions
* update links

* added s to http
* drop stickler references in docs

* remove sticker from common links
* update docs for travis-ci to cirrus-ci

* add 'travis-ci' reference locally to whatsnew

* update whatsnew comment
* docs for nox

* add titles, notices and additional detail

* review actions
* test coverage for __init__ and __call__

* test coverage for metadata resolve and coverage

* partial test coverage for metadata mapping

* python 3.6 workaround for deepcopy of mock.sentinel

* test coverage for Resolve._free_mapping

* test coverage for Resolve convenience methods

* add test stub for Resolve._metadata_mapping

* fix Test__tgt_cube_position

* test coverage for shape

* test coverage for _as_compatible_cubes

* test coverage for Resolve._metadata_mapping

* test coverage for Resolve._prepare_common_dim_payload

* test coverage for Resolve._prepare_common_aux_payload

* test coverage for Resolve._prepare_points_and_bounds

* test coverage for Resolve._create_prepared_item

* test coverage for Resolve._prepare_local_payload_dim

* test coverage for Resolve._prepare_local_payload_aux

* test coverage for Resolve._prepare_local_payload_scalar + docs URL skip

* test coverage for Resolve._prepare_local_payload

* test coverage for Resolve._metadata_prepare

* added docs URL linkcheck skip

* test coverage for Resolve._prepare_factory_payload

* test coverage for Resolve._get_prepared_item

* review actions

* test coverage for Resolve.cube
* promote unknown to dimensionless units in aux factories

* patch aux factories to promote unknown to dimensionless units for formula terms

* add whatnew PR for entry
* update intersphinx mapping and matplotlib urls (SciTools#4003)

* update intersphinx mapping and matplotlib urls

* use matplotlib intersphinx where possible

* review actions

* review actions

* cirrus-ci compute credits (SciTools#4007)

* cirrus-ci conditional tasks (SciTools#4019)

* cirrus-ci conditional tasks

* use bc for bash arithmetic

* revert back to sed

* use expr

* reword

* minor documentation changes

* review actions

* prepare v3.0.2 release

* Fix test_incompatible_dimensions test (SciTools#3977)

* test_incompatible_dimensions used a ragged array for the test, which has been deprecated in numpy, and now fails if dtype is anything other than object.  This test appears to be checking that the addition of a [2x4] masked array to a [2x3] masked cube should raise a ValueError. This commit fixes the creation of `data3` object to be a [2x4] non-ragged array.

* Added entry to what's new

* Added name to core developer list :)

* Update latest.rst

Fixed space in PR macro call

* update whatsnew v3.0.2

Co-authored-by: James Penn <[email protected]>
* Modified pyke rule

* Tests added

* Black and whatsnew

* Include PR number

* Remove latest.rst

* Add what's new
* mpl 3.4.1 updates (SciTools#4087)

* replace most recent hashes (SciTools#4112)

* Corrected plot_anomaly_log_colouring for new Matplotlib linscale rules. (SciTools#4115)

* Cartopy 0.19 updates (SciTools#4128)

* Use assertArrayAllClose for sqrt test (SciTools#4118)

* using AllClose for sqrt test

* Omitting the checksum from test cml

* use ArrayAllClose (rebase reset it?)

* Iris py38 (SciTools#3976)

* support for py38

* update CI and noxfile

* enforce alphabetical xml element attribute order

* full tests for py38 + fix docs-tests

* add whatsnew entry

* update doc-strings + review actions

* Alternate xml handling routine (#29)

* all xml tests pass for nox tests-3.8

* restored docstrings

* move sort_xml_attrs

* make sort_xml_attrs a classmethod

* update sort_xml_attr doc-string

Co-authored-by: Bill Little <[email protected]>

* add jamesp to whatsnew + minor tweak

Co-authored-by: James Penn <[email protected]>

* reinstate black and nox links

* Linkcheck update (SciTools#4104)

* Updated links

* Added login remark

* Removed extra space

* change to kick cirrus

* kick cirrus

* test verbose on cirrus

* Removed test settings.

Co-authored-by: Bill Little <[email protected]>
Co-authored-by: Martin Yeo <[email protected]>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: James Penn <[email protected]>
Co-authored-by: James Penn <[email protected]>
Co-authored-by: tkknight <[email protected]>
* fix intersection out of bounds point

* fix intersection out of bounds point take 2

* add referencing comment
* add cirrus-ci docker support

* wip

* wip

* wip

* wip

* wip

* add pip

* revert docker

* fix add_weekday (#32)

* fix add_weekday

* fix black link

* fix link

Co-authored-by: Ruth Comer <[email protected]>
* Use 'meta' in da.from_array to stop it sampling netcdf variables, which is quite slow.

* Fix PR number.

* Fix test.

* Review changes.

* Update docs/iris/src/whatsnew/3.0.2.rst

Co-authored-by: lbdreyer <[email protected]>
bjlittle and others added 16 commits May 27, 2021 16:04
* fix coord with cube arithmetic

* add coord arithmetic test coverage

* add a whatsnew entry
* Remove workaround when dask-wrapping PP data, obsoleted by SciTools#4135.

* Remove old slice testing

* Add whats new

* move whitespace?

* missing line

Co-authored-by: lbdreyer <[email protected]>
SciTools#4220)

* In cube.intersection, find split cells using a tolerant equality check

* remove unused var

* Add bounds check to tests; add what's new

* Fix whats new; update version number
Update release date in `3.0.3.rst`
* Widen cube printout for long ancil or cell-measure names.

* Adjust result for fixed cube-units printout.

* Added whatsnew.

* Include newest whatsnew in index.

* Review: fix whatsnew structure.

* Fix initial sections display.
* Fix whatsnew for SciTools#4233.

* Move 'empty slicings' whatsnew entry to 3.0.2 section.
* test changes

* add awol init

* rename system test cases

* fix PartialDateTime tests

* fix typo

* add whatsnew
@bjlittle bjlittle added this to the v3.1.0 milestone Aug 9, 2021
@bjlittle bjlittle changed the title Merge back v3p0px nox lock Mergeback v3.0.x release feature branch Aug 9, 2021
@trexfeathers trexfeathers mentioned this pull request Aug 9, 2021
Copy link
Contributor

@abooton abooton left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've reviewed the mergeback by doing it independently myself and comparing the results. It looks good to me. I just had a couple of questions, not - necessarily things that need fixing, just some differences I didn't quite understand. Looks good though - it is not an easy changeset to merge and review!

Copy link
Contributor

@abooton abooton left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looking good to me.

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.

Mergeback v3.0.x branch into main
9 participants