Skip to content

Conversation

@bjlittle
Copy link
Member

This PR removes the now defunct iris.experimental.concatenate module.

Reference #3515

Copy link
Contributor

@stephenworsley stephenworsley left a comment

Choose a reason for hiding this comment

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

Nice one.

@@ -0,0 +1,3 @@
* The :mod:`iris.experimental.concatenate` module has now been removed. Previously the experimental `concatenate`
functionality was moved to the :meth:`iris.cube.CubeList.concatenate` method as of v1.6.0, and calling the
:func:`iris.experimental.concatenate.concatenate` function raised an exception.
Copy link
Contributor

@abooton abooton Nov 15, 2019

Choose a reason for hiding this comment

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

Maybe slightly clearer - but is fine really:

  • The :mod:iris.experimental.concatenate module has now been removed. In v1.6.0 the experimental concatenate
    functionality was moved to the :meth:iris.cube.CubeList.concatenate method. Calling the :func:iris.experimental.concatenate.concatenate function has raised an exception since then.

Copy link
Member Author

Choose a reason for hiding this comment

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

@abooton I've made a further minor tweak to your excellent revision. Seems reasonable, take a peek...

Copy link
Contributor

Choose a reason for hiding this comment

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

Looks good.

@abooton abooton merged commit abbde31 into SciTools:master Nov 15, 2019
@bjlittle
Copy link
Member Author

@abooton Awesome, thanks again 😄

@bjlittle bjlittle deleted the remove-experimental-concatenate branch November 15, 2019 09:12
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.

4 participants