Skip to content

fix: global has geopotential_height_on_pressure_levels too#46

Merged
gadomski merged 2 commits intomainfrom
issues/45-geopotential-height-again
Jan 28, 2026
Merged

fix: global has geopotential_height_on_pressure_levels too#46
gadomski merged 2 commits intomainfrom
issues/45-geopotential-height-again

Conversation

@gadomski
Copy link
Contributor

Related issues

Description

I only had geopotential_height_on_pressure_levels for uk, we needed it for global too. cc @ghidalgo3, FYI a minimum-reproducibe example is here: https://github.com/stactools-packages/met-office-deterministic/compare/issues/45-geopotential-height-again?expand=1#diff-fcd8900fc8d14bab79e38362aa07a61c1bf1ce3cea05ccd0aab72662fbcf9f1dR92-R95

Checklist

  • Tests pass (run scripts/test)
  • Examples have been updated to reflect changes, if applicable

@gadomski gadomski self-assigned this Jan 28, 2026
@gadomski gadomski merged commit ac8331f into main Jan 28, 2026
7 checks passed
@gadomski gadomski deleted the issues/45-geopotential-height-again branch January 28, 2026 16:07
gadomski added a commit that referenced this pull request Jan 28, 2026
🤖 I have created a release *beep* *boop*
---


##
[0.4.2](v0.4.1...v0.4.2)
(2026-01-28)


### Bug Fixes

* global has geopotential_height_on_pressure_levels too
([#46](#46))
([ac8331f](ac8331f))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).
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.

KeyError: 'geopotential_height_on_pressure_levels' still exists with 0.4.1

1 participant