-
Notifications
You must be signed in to change notification settings - Fork 316
move whatsnew contributions #3816
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
Changes from 1 commit
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
This file was deleted.
This file was deleted.
This file was deleted.
This file was deleted.
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -27,7 +27,9 @@ Features | |
| dependency on `matplotlib`_ at ``v2.x``. Now that ``Python2`` support has | ||
| been dropped, ``Iris`` is free to use the latest version of `matplotlib`_. | ||
|
|
||
| * `CF Ancillary Data`_ variables are now supported. | ||
| * `CF Ancillary Data`_ variables are now supported, and can be loaded from and | ||
| saved to NetCDF-CF files. Support for `Quality Flags`_ is also provided to | ||
| ensure they load and save with appropriate units. | ||
|
|
||
|
|
||
| Dependency Updates | ||
|
|
@@ -80,6 +82,10 @@ Bugs Fixed | |
| (previously would take the unit from a time coordinate, if present, even | ||
| though the coordinate's value had been changed via ``date2num``). | ||
|
|
||
| * Attributes of cell measures in NetCDF-CF files were being discarded during | ||
| loading. They are now available on the :class:`~iris.coords.CellMeasure` in | ||
| the loaded :class:`~iris.cube.Cube`. | ||
|
|
||
|
|
||
| Incompatible Changes | ||
| ==================== | ||
|
|
@@ -109,6 +115,11 @@ Incompatible Changes | |
| :func:`iris.experimental.concatenate.concatenate` function raised an | ||
| exception. | ||
|
|
||
| * When loading data from NetCDF-CF files, where a variable has no ``units`` | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. would it be useful to link to an issue or pr? |
||
| property, the corresponding Iris object will have ``units='unknown'``. | ||
| Prior to Iris ``3.0.0``, these cases defaulted to ``units='1'``. | ||
|
|
||
|
|
||
| Internal | ||
| ======== | ||
|
|
||
|
|
@@ -176,6 +187,10 @@ Documentation | |
| * Added a warning to the :func:`iris.analysis.cartography.project` function | ||
| regarding its behaviour on projections with non-rectangular boundaries. | ||
|
|
||
| * Added an explanation to the user guide of how ``Units`` are handled during | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. It would be useful to have a link to the section of documentation that is added. I assume it would be to this: Suggest adding a |
||
| cube arithmetic. | ||
|
|
||
| .. _Read the Docs: https://scitools-iris.readthedocs.io/en/latest/ | ||
| .. _matplotlib: https://matplotlib.org/ | ||
| .. _CF Ancillary Data: https://cfconventions.org/Data/cf-conventions/cf-conventions-1.8/cf-conventions.html#ancillary-data | ||
| .. _Quality Flags: https://cfconventions.org/Data/cf-conventions/cf-conventions-1.8/cf-conventions.html#flags | ||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
would it be useful to link to an issue or pr?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@tkknight Glad to do that, but for consistency that should also be the case for all other
whatsnewentries...There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think it would make the whats new page more useful - we could keep this discussion separate to this PR so we do not block it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@tkknight Cool.
For this PR I'll add the relevant links per
whatsnewentry mentioned here. Then I'll raise a separate PR to add relevant links to all the other entries, if possible... just to separate concerns. Thanks 👍