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.

Sign up for GitHub

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

Jump to bottom

Adding source code to produce the `ar/` version of this dataset. #46

Merged

dabhicusp merged 7 commits into main from ar

Aug 29, 2023

Collaborator

alxmrs commented Aug 15, 2023

No description provided.

alxmrs added 2 commits

August 15, 2023 09:31


          Adding source code to produce the ar/ version of this dataset.

4eae843


          Update a path.

d5883c4

alxmrs marked this pull request as ready for review

August 16, 2023 20:21

dabhicusp added 3 commits

August 22, 2023 15:55


          Pipeline code changed.

9b26adb


          Pipeline updated with the Pardo instead of map.

a7dc800


          Example updated.

ed2cccc

alxmrs commented

View reviewed changes

src/arco_era5/source_data.py Outdated Show resolved Hide resolved

alxmrs commented

View reviewed changes

src/arco_era5/source_data.py Outdated Show resolved Hide resolved

alxmrs commented

View reviewed changes

src/arco_era5/source_data.py Outdated Show resolved Hide resolved

alxmrs commented

View reviewed changes

src/netcdf_to_zarr.py Outdated Show resolved Hide resolved

alxmrs commented

View reviewed changes

setup.py Show resolved Hide resolved

alxmrs commented

View reviewed changes

src/netcdf_to_zarr.py Outdated Show resolved Hide resolved

dabhicusp added 2 commits

August 25, 2023 06:57


          Pipeline changes done.

113e552


          Example edited.

04e15da

Collaborator Author

alxmrs commented Aug 28, 2023

LGTM. However, I can't approve since I'm a co-author of this CL. We'll have to ask @DarshanSP19 :)

dabhicusp requested a review from DarshanSP19

August 29, 2023 04:05

DarshanSP19 approved these changes

View reviewed changes

dabhicusp merged commit db43c7f into main

3 checks passed

dabhicusp deleted the ar branch

August 29, 2023 16:47

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet