Skip to content
This repository has been archived by the owner on Dec 20, 2024. It is now read-only.

Fix/Inference mode for Imputers #75

Open
wants to merge 6 commits into
base: develop
Choose a base branch
from

Conversation

JesperDramsch
Copy link
Member

@JesperDramsch JesperDramsch commented Nov 8, 2024

This resets the nan_locations in the BaseImputer during non-training runs of the code.

Specifically, this enables changes in the dataset after training.

Closes ecmwf/anemoi-core#34

@codecov-commenter
Copy link

codecov-commenter commented Nov 8, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 99.85%. Comparing base (489a241) to head (f1516ed).

Additional details and impacted files
@@           Coverage Diff            @@
##           develop      ecmwf/anemoi-models#75   +/-   ##
========================================
  Coverage    99.85%   99.85%           
========================================
  Files           23       23           
  Lines         1399     1426   +27     
========================================
+ Hits          1397     1424   +27     
  Misses           2        2           

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

@HCookie HCookie self-requested a review November 8, 2024 12:58
@JesperDramsch JesperDramsch self-assigned this Nov 8, 2024
@JesperDramsch JesperDramsch added the enhancement New feature or request label Nov 8, 2024
@HCookie
Copy link
Member

HCookie commented Nov 11, 2024

Once a changelog entry is added, I see no issue with approving and merging this.

sahahner
sahahner previously approved these changes Nov 11, 2024
Copy link
Member

@sahahner sahahner left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Imputer should have an inference mode
4 participants