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

whatsnew entry for dependencies #3831

Merged
merged 1 commit into from
Sep 9, 2020

Conversation

bjlittle
Copy link
Member

@bjlittle bjlittle commented Sep 8, 2020

This PR adds the whatsnew entry for #3812.

It also addresses several minor documentation issues that have unfortunately crept in 🤔

We need to fix the sphinx logger issue #3789

@@ -1198,7 +1198,7 @@ def _check(item):

class DimCoordMetadata(CoordMetadata):
"""
Metadata container for a :class:`~iris.coords.DimCoord"
Metadata container for a :class:`~iris.coords.DimCoord`
Copy link
Member Author

Choose a reason for hiding this comment

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

@tkknight This was just wrong, and causing a documentation warning during sphinx build time.


* Iris now requires `Cartopy <https://github.com/SciTools/cartopy>`_ >= 0.18 in
order to remain compatible with the latest version of
`Matplotlib <https://github.com/matplotlib/matplotlib>`_.
order to remain compatible with the latest version of `Matplotlib`_.
Copy link
Member Author

Choose a reason for hiding this comment

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

@tkknight There ended up being two matplotlib references...

@@ -22,6 +22,4 @@
ax.coastlines(resolution="50m")
ax.gridlines()

plt.tight_layout()

Copy link
Member Author

Choose a reason for hiding this comment

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

@tkknight The plt.tight_layout() was causing a traceback within matplotlib, resulting in the plot not being rendered in the documentation i.e., see here.

@tkknight
Copy link
Contributor

tkknight commented Sep 9, 2020

Looks good @bjlittle.

@tkknight tkknight merged commit c471bcc into SciTools:master Sep 9, 2020
@bjlittle bjlittle deleted the whatsnew-requirements branch September 9, 2020 09:52
tkknight added a commit to tkknight/iris that referenced this pull request Sep 9, 2020
* master:
  Fix sphinx logger (SciTools#3832)
  whatsnew entry for dependencies (SciTools#3831)
  Added link to scitools.org.uk on the main index. (SciTools#3830)
tkknight added a commit to tkknight/iris that referenced this pull request Sep 9, 2020
* master:
  Fix sphinx logger (SciTools#3832)
  whatsnew entry for dependencies (SciTools#3831)
  Added link to scitools.org.uk on the main index. (SciTools#3830)
tkknight added a commit to tkknight/iris that referenced this pull request Sep 11, 2020
* master: (36 commits)
  Rework whatsnew into new scheme. (SciTools#3834)
  Lazy regridding with Linear, Nearest, and AreaWeighted (SciTools#3701)
  Iris readme minimal (SciTools#3833)
  Fix sphinx logger (SciTools#3832)
  whatsnew entry for dependencies (SciTools#3831)
  Added link to scitools.org.uk on the main index. (SciTools#3830)
  Updated Iris install instructions (SciTools#3817)
  Delete labeler.yml
  Delete label.yml
  Create label.yml
  Delete labeler.yml
  Update labeler.yml
  Update labeler.yml
  Add template collapsible markdown section (SciTools#3823)
  Requirements re-haul (SciTools#3812)
  Automate pull request labels (SciTools#3819)
  move whatsnew contributions (SciTools#3816)
  Parse with packaging version (SciTools#3815)
  Add "What's New" entries for unpinning Cartopy, Matplotlib, Proj (SciTools#3811)
  tidy issue templates (SciTools#3814)
  ...
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.

2 participants