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

Update encoder_processor_decoder.py #53

Merged

Conversation

gabrieloks
Copy link
Contributor

There were recent changes in anemoi-models to introduce the remapper into anemoi-training ecmwf/anemoi-training#17. These changes introduced an additional level of indexing:

from model to internal_model and model

from data to internal_data and data

These changes were not taken into account in the bounding feature PR. In this PR, we fix this.

@codecov-commenter
Copy link

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 99.84%. Comparing base (53dc8ed) to head (57841b6).
Report is 3 commits behind head on develop.

Additional details and impacted files
@@           Coverage Diff            @@
##           develop      #53   +/-   ##
========================================
  Coverage    99.84%   99.84%           
========================================
  Files           23       23           
  Lines         1277     1277           
========================================
  Hits          1275     1275           
  Misses           2        2           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@JesperDramsch JesperDramsch merged commit bb1b105 into develop Sep 30, 2024
105 of 106 checks passed
@JesperDramsch JesperDramsch deleted the QUICK-FIX]-data-indices-in-the-bounding-initialisation branch September 30, 2024 14:52
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.

3 participants