Skip to content
This repository was archived by the owner on Jun 2, 2025. It is now read-only.

Conversation

jacobbieker
Copy link
Member

Since ECMWF just interpolates to lower than 0.1, the 0.1 values should be right, so take those instead here

Pull Request

Description

Fixes potentially coarsen to actually take every other point if the source data is in 0.05 degree incerements

Fixes #

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce.
Please also list any relevant details for your test configuration

  • Yes

If your changes affect data processing, have you plotted any changes? i.e. have you done a quick sanity check?

  • Yes

Checklist:

  • My code follows OCF's coding style guidelines
  • I have performed a self-review of my own code
  • I have made corresponding changes to the documentation
  • I have added tests that prove my fix is effective or that my feature works
  • I have checked my code and corrected any misspellings

Since ECMWF just interpolates to lower than 0.1, the 0.1 values should be right, so take those instead here
@jacobbieker jacobbieker self-assigned this Mar 11, 2024
@jacobbieker jacobbieker merged commit 2961876 into main Mar 11, 2024
@jacobbieker jacobbieker deleted the jacob/coarsen_fix branch March 11, 2024 14:21
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.

1 participant