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.

Sign up for GitHub

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

Jump to bottom

Cubeprinter #63

Closed
wants to merge 17 commits into from
Closed

Cubeprinter #63

wants to merge 17 commits into from

Conversation

pp-mo
Copy link
Owner

@pp-mo pp-mo commented Feb 8, 2021 •
edited
Loading

Here's a preview of what comes after SciTools#3988

This adds the new cube printout support

  • make a CubePrinter from a CubeSummary
  • the 'Table' function that supports the minim output formatting needs

Beyond this, I shall be :

  1. adding proper unit-organised testing
  2. reworking the actual cube methods, and fixing the existing tests which this breaks (much existing code for that already prototyped, but now needs re-targetting)
      • which includes providing a temporary fix / workaround to keep the _repr_html_ working.

tkknight and others added 12 commits February 9, 2021 12:14
@tkknight
Added text to state the Python version used to build the docs. (SciTo…
c51dab2 
…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
@bjlittle @jamesp
Iris py38 (SciTools#3976)
8fb33bb 
* 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]>
@bjlittle
normalise version to implicit development release number (SciTools#3991)
1549bae
@rcomer
Gallery: update COP maps example (SciTools#3934)
9317133 
* update cop maps example

* comment tweaks

* minor comment tweak + whatsnew

* reinstate whatsnew addition

* remove duplicate whatsnew
@rcomer
don't support mpl v1.2 (SciTools#3941)
e378eb8
@pp-mo
Cubesummary tidy (SciTools#3988)
e3c1905 
* 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.
@pp-mo
Extra tests; fix for array attributes.
6cc41bc
@pp-mo
Rename _representation to _representation/cube_summary.
7495f6a
@pp-mo
Cube printout code, rough testing only.
95542b8
@pp-mo
Adding proper tests.
47c8de8
@pp-mo
Proper unit tests; partial - more to come.
6d4e07e
@pp-mo
Remove spurious import.
f350d18
@pp-mo pp-mo closed this Mar 18, 2022
@pp-mo pp-mo deleted the cubeprinter branch March 18, 2022 15:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@pp-mo @bjlittle @tkknight @rcomer