Skip to content

Conversation

@m1dr
Copy link
Contributor

@m1dr m1dr commented Apr 6, 2021

Per regulation 92.1.8, longitudes are limited to the range 0 to 360 degrees. This PR adds support for calculating iDirectionIncrement when not given (i.e. resolutionAndComponentFlags bit 3 is 0) where longitudeOfFirstGridPoint and longitudeOfLastGridPoint span the prime meridian in GDTs 0 and 40 and siblings.

@jamesp
Copy link
Member

jamesp commented Apr 8, 2021

Hi @m1dr, we had a CI problem that's now been fixed, would you mind rebasing on master? The CI tests should then run properly.

@m1dr m1dr force-pushed the handle_longitude_mod branch from f854af6 to 01ef303 Compare April 8, 2021 11:41
@m1dr
Copy link
Contributor Author

m1dr commented Apr 8, 2021

Hi @jamesp, thanks for the heads up. Saw they'd failed for reasons unrelated to my changes but wasn't sure who to contact.

@lbdreyer
Copy link
Member

This is great! Thanks @m1dr! I'm going to merge, and then this will be available in the 0.17 release. We're hoping to get that out in the next couple of months

@lbdreyer lbdreyer merged commit 9cec4d6 into SciTools:master Apr 27, 2021
@m1dr m1dr deleted the handle_longitude_mod branch April 27, 2021 16:35
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