Skip to content
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

Minor Improvements 0 #151

Merged
merged 11 commits into from
Nov 13, 2024
Merged

Minor Improvements 0 #151

merged 11 commits into from
Nov 13, 2024

Conversation

mariusaurus
Copy link
Contributor

Earth2Studio Pull Request

Description

This commits adds multiple small enhancements:

it adds the 2m dew point temperature as 'd2m' to the lexica of arco, base, cds, gefs, gfs, ifs and wb2. In some SFNO checkpoints that variable was called 'd2', which is now corrected when initialising the model.
when downloading data from arco users can optionally specify the time after which the download gets cancelled. Before, that time was hard-coded. The hard-coded value is now the default value in case no specific time is passed to the data source function.
when writing a data source output to file using the function from utils.py, the user can now define of which type the data shall be.
for ensemble scores, a routine was added computing the mse and the variance of the forecasts. These can be averaged over multiple forecasts for computing a skill-spread ratio over all forecasts. That was not possible with the original skill-spread ratio.
a new data loader was added, which accesses xarray-data-array stored in monthly files located in yearly directories. A test for this loader has been added as well

Checklist

  • I am familiar with the Contributing Guidelines.
  • New or existing tests cover these changes.
  • The documentation is up to date with these changes.
  • The CHANGELOG.md is up to date with these changes.
  • An issue is linked to this pull request.

Dependencies

@dallasfoster
Copy link
Collaborator

/blossom-ci

@dallasfoster
Copy link
Collaborator

/blossom-ci

@dallasfoster
Copy link
Collaborator

/blossom-ci

@dallasfoster
Copy link
Collaborator

/blossom-ci

1 similar comment
@dallasfoster
Copy link
Collaborator

/blossom-ci

@dallasfoster dallasfoster merged commit 3f99399 into NVIDIA:main Nov 13, 2024
1 check passed
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.

2 participants