Skip to content

Merge master to mesh data model#4093

Merged
trexfeathers merged 44 commits intoSciTools:mesh-data-modelfrom
bjlittle:merge-master-to-mesh-data-model
Apr 9, 2021
Merged

Merge master to mesh data model#4093
trexfeathers merged 44 commits intoSciTools:mesh-data-modelfrom
bjlittle:merge-master-to-mesh-data-model

Conversation

@bjlittle
Copy link
Member

@bjlittle bjlittle commented Apr 9, 2021

🚀 Pull Request

Description

This PR merges the latest changes from master to the mesh-data-model feature branch.


Consult Iris pull request check list

pp-mo and others added 30 commits February 8, 2021 11:20
Co-authored-by: stephen.worsley <stephen.worsley@metoffice.gov.uk>
…ols#3989)

* Added text to state the Python version used to build the docs.

* Added footer template that includes the Python version used to build.

* added new line

* Review actions

* added whatsnew
* 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 <bill.james.little@gmail.com>

* add jamesp to whatsnew + minor tweak

Co-authored-by: James Penn <james@jamespenn.co.uk>
* update cop maps example

* comment tweaks

* minor comment tweak + whatsnew

* reinstate whatsnew addition

* remove duplicate whatsnew
* Extra tests; fix for array attributes.

* Docstring for CubeSummary, and remove some unused parts.

* Fix section name capitalisation, in line with existing cube summary.

* Handle array differences; quote strings in extras and if 'awkward'-printing.

* Ensure scalar string coord 'content' prints on one line.
* update intersphinx mapping and matplotlib urls

* use matplotlib intersphinx where possible

* review actions

* review actions
* update readme badges

* pimp twitter badge
* Stopped using deprecated aliases of builtin types.
This is required to avoid warnings starting with NumPy 1.20.0.

* Update lib/iris/tests/test_cell.py

Co-authored-by: Bill Little <bill.little@metoffice.gov.uk>

* Update lib/iris/tests/test_cell.py

Co-authored-by: Bill Little <bill.little@metoffice.gov.uk>

* Updated whatsnew.

Co-authored-by: Bill Little <bill.little@metoffice.gov.uk>
* baseline

* removed debug comments

* reverted

* remove line

* Testing

* testing extensions

* testing rtd_version

* fixed if

* removed line

* tidy up

* tidy comments

* debug of pre-existing rtd variables

* added reminder

* testing

* testing still

* updated comments

* added whatsnew

* expanded the if conditiion

* review actions

* Update layout.html

Remove alternative banner that used the RestructuredText notation.

* review actions
* cirrus-ci conditional tasks

* use bc for bash arithmetic

* revert back to sed

* use expr

* reword

* minor documentation changes

* review actions
* implement metadata string

* review actions
@bjlittle bjlittle force-pushed the merge-master-to-mesh-data-model branch from 776ffbc to 8f6bb44 Compare April 9, 2021 16:22
Copy link
Contributor

@trexfeathers trexfeathers left a comment

Choose a reason for hiding this comment

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

This PR has every expected commit since branch creation, and no unexpected commits (the only extra one being the conflict-resolving 8f6bb44).

The diff matches this comparison of master's state at the last merge-back (#4022) with the final master commit in this PR.

@trexfeathers
Copy link
Contributor

Confident that the CI delays will not be masking any failures, and low risk given this is a feature branch. Merging now.

@trexfeathers trexfeathers merged commit bbeb543 into SciTools:mesh-data-model Apr 9, 2021
@bjlittle bjlittle deleted the merge-master-to-mesh-data-model branch April 9, 2021 18:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants