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

2.17.0 b5 #27

Closed
wants to merge 14 commits into from
Closed

2.17.0 b5 #27

wants to merge 14 commits into from

Conversation

teresanoviello
Copy link

  1. The motivation of the change and impact on code:
    This is the ecCodesMIA B5 version, also referred to as 'Reduction of Dynamic Allocation Frequency'.
    This is a pull request for a branch derived from ecCodes official repository, tag "2.17.0": so please, when merging take into account to not do it against the current ecCodes "develop" HEAD.
    The complete changes applied are described in the file "src/ChangesB5.txt".
    This code is not operational. Impacts on the code: all automated tests are passing, the complete output is in the file "ctest3.out".
    A full report of the ecCodesMIA project is accessible at https://gitlab.eumetsat.int/additional-data-services/eccodes-impact-analysis/-/blob/master/docs/officialdocs/ecCodesMIA_FinalReport_files/ecCodesMIA_FinalReport.md.

  2. The author of changes accepts the "ECMWF Contributors License Agreement".
    In particular:
    2.1. This code can be redistributed under Apache License 2.0;
    2.2. This code doesn't violate anyone IPR' rights;

3.Exception to PR requirements as described in "https://confluence.ecmwf.int/display/SUP/ECMWF+software+on+GitHub":
3.1. Although this code passes all the automated and regression tests, it is not intended as an operational version, as stated earlier.

@FussyDuck
Copy link

FussyDuck commented Jul 8, 2020

CLA assistant check
All committers have signed the CLA.

@teresanoviello
Copy link
Author

Closing this PR request.
I think the fastest way is to remove these PRs and I can just submit them again. I think it would work because:

  1. "[email protected]" is now in my GitHub account;
  2. I would be tidy and sign the new CLA with email "[email protected]"

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants