Skip to content

Commit

Permalink
Added in linkchecks and coverage checks for gha ci (SciTools#432)
Browse files Browse the repository at this point in the history
* updated ci-tests

* added linkcheck exceptions

* fixed line lengths

* comments too?

* this is ridiculous

* fixed two links in index.rst

* missed a rebase spot

* missed a typo

* readded two links

* fixed some broken links

* fixed some broken links

* fixed some broken links

* fixed some broken links

* fixed some broken links

* fixed some broken links

---------

Co-authored-by: stephenworsley <[email protected]>
  • Loading branch information
ESadek-MO and stephenworsley authored Apr 19, 2024
1 parent 0870a9a commit a9d45b1
Show file tree
Hide file tree
Showing 5 changed files with 26 additions and 12 deletions.
7 changes: 6 additions & 1 deletion .github/workflows/ci-tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -36,13 +36,14 @@ jobs:
matrix:
os: ["ubuntu-latest"]
python-version: ["3.11"]
session: ["doctest"]
session: ["doctest", "linkcheck"]
iris-source : ["conda-forge"]
include:
- os: "ubuntu-latest"
python-version: "3.11"
session: "tests"
iris-source: "conda-forge"
coverage: "--coverage"
- os: "ubuntu-latest"
python-version: "3.11"
session: "tests"
Expand Down Expand Up @@ -129,3 +130,7 @@ jobs:
PY_VER: ${{ matrix.python-version }}
run: |
nox --session ${{ matrix.session }} -- --test-data-dir ${HOME}/iris-test-data/test_data
- name: "Upload coverage report"
uses: codecov/codecov-action@v4
if: ${{ matrix.coverage }}
8 changes: 8 additions & 0 deletions docs/conf.py
Original file line number Diff line number Diff line change
Expand Up @@ -105,6 +105,14 @@
# If true, `todo` and `todoList` produce output, else they produce nothing.
todo_include_todos = False

# url link checker. Some links registering as faulty, despite being functional.
# See https://www.sphinx-doc.org/en/1.2/config.html[...]
# [...]#options-for-the-linkcheck-builder

linkcheck_ignore = [
"https://github.com/Scitools/iris-grib/compare"
"/c4243ae..5c314e3#diff-cf46b46880cae59e82a91c7ab6bb81ba"
]

# -- Autodoc ------------------------------------------------------------------

Expand Down
4 changes: 2 additions & 2 deletions docs/index.rst
Original file line number Diff line number Diff line change
Expand Up @@ -214,7 +214,7 @@ Getting Started

To ensure all ``iris-grib`` dependencies, it is sufficient to have installed
:mod:`Iris <iris>` itself, and
`ecCodes <https://software.ecmwf.int/wiki/display/ECC/ecCodes+Home>`_ .
`ecCodes <https://confluence.ecmwf.int/display/ECC/ecCodes+Home>`_ .

The simplest way to install is with
`conda <https://conda.io/miniconda.html>`_ , using the
Expand All @@ -234,7 +234,7 @@ Development sources are hosted at `<https://github.com/SciTools/iris-grib>`_ .
Releases
========

For recent changes, see `Release Notes <ref/release_notes.html>`_ .
For recent changes, see :ref:`release_notes` .


Indices and tables
Expand Down
17 changes: 9 additions & 8 deletions docs/ref/release_notes.rst
Original file line number Diff line number Diff line change
@@ -1,3 +1,5 @@
.. _release_notes:

Release Notes
=============

Expand Down Expand Up @@ -128,7 +130,7 @@ Bugs Fixed
iris-grib to work with the latest versions of
`iris <https://scitools-iris.readthedocs.io/en/stable/>`_,
`cf-units <https://cf-units.readthedocs.io/en/latest/>`_,
`ecCodes <https://software.ecmwf.int/wiki/display/ECC/ecCodes+Home>`_ and
`ecCodes <https://confluence.ecmwf.int/display/ECC>`_ and
`cartopy <https://scitools.org.uk/cartopy/docs/latest/>`_, including casting
the usage of :meth:`cf_units.Unit.date2num` as float. setting and setting the
values of some missing keys using ``gribapi.GRIB_MISSING_LONG``.
Expand Down Expand Up @@ -247,15 +249,15 @@ Bugs Fixed
* `@pp-mo <https://github.com/pp-mo>`_ fixed loading of grid definition
template 3.90, "Space view perspective or orthographic grid", which was
**broken since Iris 2.3**. This now produces data with an iris
`Geostationary <https://scitools-iris.readthedocs.io/en/latest/generated/api/iris/coord_systems.html#iris.coord_systems.Geostationary>`_
`Geostationary <https://scitools-iris.readthedocs.io/en/stable/generated/api/iris.coord_systems.html#iris.coord_systems.Geostationary>`_
coordinate system. Prior to Iris 2.3, what is now the Iris 'Geostationary'
class was (incorrectly) named "VerticalPerspective" : When that was
`corrected in Iris 2.3 <https://github.com/SciTools/iris/pull/3406>`_ , it
broke the iris-grib loading, since the data was now incorrectly
assigned the "new-style" Iris
`VerticalPerspective <https://scitools-iris.readthedocs.io/en/latest/generated/api/iris/coord_systems.html#iris.coord_systems.VerticalPerspective>`_
`VerticalPerspective <https://scitools-iris.readthedocs.io/en/stable/generated/api/iris.coord_systems.html#iris.coord_systems.VerticalPerspective>`_
coordinate system, equivalent to the Cartopy
`NearsidePerspective <https://scitools.org.uk/cartopy/docs/latest/crs/projections.html#nearsideperspective>`_
`NearsidePerspective <https://scitools.org.uk/cartopy/docs/latest/reference/projections.html#nearsideperspective>`_
and Proj
`"nsper" <https://proj.org/operations/projections/nsper.html>`_ .
The plotting behaviour of this is now **the same again as before Iris 2.3** :
Expand All @@ -264,7 +266,7 @@ Bugs Fixed

* `@pp-mo <https://github.com/pp-mo>`_ fixed a problem where cubes were loading from GRIB 1 with a changed coordinate
system, since eccodes versions >= 1.19. This resulted from a change to eccodes, which now returns a different
'shapeOfTheEarth' parameter : see `eccodes issue ECC-811 <https://jira.ecmwf.int/browse/ECC-811>`_ . This resulted
'shapeOfTheEarth' parameter. This resulted
in a coordinate system with a different earth radius.
For backwards compatibilty, the earth radius has now been fixed to the same value as previously.
However, pending further investigation, this value may be technically incorrect and we may
Expand Down Expand Up @@ -293,8 +295,7 @@ Bugs Fixed
* `@pp-mo <https://github.com/pp-mo>`_ fixed a problem that caused very slow
loading, and possible memory overflows, with Dask versions >= 2.0.
**This requires Iris >= 2.4**, as a new minimum dependency.
( This problem was shared with UM file access in Iris : see
https://scitools.org.uk/iris/docs/v2.4.0/whatsnew/2.4.html#bugs-fixed ).
( This problem was shared with UM file access in Iris, fixed in Iris 2.4.
`(PR#190) <https://github.com/SciTools/iris-grib/pull/190>`_

* `@trexfeathers <https://github.com/trexfeathers>`_ fixed all the tests to
Expand Down Expand Up @@ -451,7 +452,7 @@ What's new in iris-grib v0.12
:Date: 25 Oct 2017

Updated to work with
`ecCodes <https://software.ecmwf.int/wiki/display/ECC/ecCodes+Home>`_ as its
`ecCodes <https://confluence.ecmwf.int/display/ECC>`_ as its
interface to GRIB files.
This is ECMWF's replacement for the older GRIB-API, which is now deprecated.

Expand Down
2 changes: 1 addition & 1 deletion iris_grib/__init__.py
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@
"""
Conversion of cubes to/from GRIB.
See: `ECMWF GRIB API <https://software.ecmwf.int/wiki/display/GRIB/Home>`_.
See: `ECMWF GRIB API <https://confluence.ecmwf.int/display/MTG2US/Migration+to+Grib+2+-+User+Space+Home>`_.
"""

Expand Down

0 comments on commit a9d45b1

Please sign in to comment.