Skip to content
This repository has been archived by the owner on Feb 17, 2023. It is now read-only.

BLD: Updated tested numpy version to 1.9 #193

Merged
merged 1 commit into from
Nov 11, 2016

Conversation

cpelley
Copy link

@cpelley cpelley commented Nov 10, 2016

Increase the minimum version of numpy that biggus is tested against with travis CI to 1.9
Matching the minimum requirements of iris.

This would facilitate making changes to biggus which would no longer require to be backward compatible with very old versions of numpy. In particular #176 would be made considerably simple by this change.

@cpelley
Copy link
Author

cpelley commented Nov 10, 2016

@marqh, I said I would ping you when I put this one up.

Cheers

@marqh
Copy link
Member

marqh commented Nov 11, 2016

this is a sensible change, we are way past supporting numpy 1.7

@cpelley
Copy link
Author

cpelley commented Nov 11, 2016

Thanks @marqh

@@ -5,7 +5,7 @@ python:
- "3.4"
- "3.5"
env:
- NUMPY="numpy==1.7"
- NUMPY="numpy==1.9"
Copy link
Member

Choose a reason for hiding this comment

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

Out of interest, what would happen if you unpinned NumPy entirely?

Copy link
Member

Choose a reason for hiding this comment

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

numpy is unpinned in one part of the test matrix

we test against 1 pinned version and latest, so currently this is 1.9 and 1.11

@DPeterK DPeterK merged commit 15fc0e9 into SciTools:master Nov 11, 2016
@DPeterK
Copy link
Member

DPeterK commented Nov 11, 2016

Thanks @cpelley

@cpelley cpelley deleted the MIN_NUMPY_VER_INC branch November 14, 2016 10:25
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants