Skip to content

Conversation

@lbdreyer
Copy link
Member

@lbdreyer lbdreyer commented Feb 20, 2020

Companion Iris PR: SciTools/iris#3664

@lbdreyer
Copy link
Member Author

GRIB saving with eccodes 2.16 seems to be broken as is causing the tests to fail.

I suggest we pin back to eccodes 2.14 #193.
Once that's in, these tests should pass

self.assertEqual(cube.ndim, 3)
self.assertEqual(cube.coord_dims("time"), (0,))
cube = cube[0]

Copy link
Member Author

Choose a reason for hiding this comment

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

This cube loads in with a shape of (928, 744).

6 years ago, eccodes had a bug such that it could only save square data. Thus, in Iris this tests square off the data before saving it (see here). I have removed that as the bug has been fixed (many years ago!)

@pp-mo
Copy link
Member

pp-mo commented Feb 21, 2020

Thanks @lbdreyer : Very conscientious to have redistributed these so carefully !
👍 🥇
I'm satisfied it's all good here, but let's see if #193 will get us a green tick ...

@coveralls
Copy link

Coverage Status

Coverage increased (+3.02%) to 89.796% when pulling 9f14bde on lbdreyer:latest_grib into a09fb02 on SciTools:master.

@pp-mo pp-mo merged commit c4d152c into SciTools:master Feb 21, 2020
@lbdreyer lbdreyer deleted the latest_grib branch June 27, 2021 21:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants