-
Notifications
You must be signed in to change notification settings - Fork 14
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Metadata
for JRA55
#286
base: main
Are you sure you want to change the base?
Metadata
for JRA55
#286
Conversation
sounds good |
This should be ready to merge once the tests pass. However, we can probably merge #294 first and this one later. They should not conflict since this has nothing to do with the kernels but just with the organization of the data. However, this PR should be ready to review |
Another change I made is that |
Tests should pass so this is ready to merge / review |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #286 +/- ##
=====================================
Coverage 0.00% 0.00%
=====================================
Files 35 38 +3
Lines 2281 2227 -54
=====================================
+ Misses 2281 2227 -54 ☔ View full report in Codecov by Sentry. |
This PR is an initial proposal to generalize
ECCOMetadata
toMetadata
and rework the JRA55 module to useMetadata
. In this way, we can have different JRA55 versions (repeat year and multiple year) and we can define adownload_dataset
function to download the dataset independently of using JRA55 as we can do for ECCOThis PR also removes the ability to generate a
JRA55FieldTimeSeries
directly interpolated on the ocean grid, since we need to interpolate anyways when we compute fluxescloses #182