ecmwf / anemoi-inference

Apache License 2.0
14 stars 4 forks source link

bug fix #37

Closed dietervdb-meteo closed 1 month ago

dietervdb-meteo commented 1 month ago
  1. In run.py GribFileInput was called with one runner argument too much.
  2. In runner.py the attributes dynamic_forcings_sources_that_bring_new_fields and dynamic_forcings_sources_that_modify_grids of the Runner class were undefined, but called in the function add_dynamic_forcings_to_input_tensor.

This PR fixes the above two issues, although the solution to 2. is only a temporary one.

FussyDuck commented 1 month ago

CLA assistant check
All committers have signed the CLA.

dietervdb-meteo commented 1 month ago

In the meanwhile the main repository for this branch was updated. The bugs reported above seem to survive, although in somewhat altered form. Will see if I can fix them, if so might make new pull request.

dietervdb-meteo commented 1 month ago

Issue 2. is solved by the new version of the code. Issue 1. survived, be it with the argument renamed from runner to context. It is fixed in the new PR #38