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

feat(retrieve): Use input grib as target #88

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

Conversation

gmertes
Copy link
Member

@gmertes gmertes commented Dec 18, 2024

If the config contains an input grib, the retrieve command will use that are the target in the generated mars request.
Also make the default output stdout.

config.yaml:

input:
  grib: input.grib
checkpoint: model.ckpt
anemoi-inference retrieve config.yaml --date 20240101T00:00:00

will have target=input.grib

Target can still be overwritten by passing --extra target=anotherinput.grib

@codecov-commenter
Copy link

codecov-commenter commented Dec 18, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 98.03%. Comparing base (1187d78) to head (c262b56).

Additional details and impacted files
@@           Coverage Diff            @@
##           develop      #88   +/-   ##
========================================
  Coverage    98.03%   98.03%           
========================================
  Files            3        3           
  Lines           51       51           
========================================
  Hits            50       50           
  Misses           1        1           

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

Copy link
Collaborator

@b8raoult b8raoult left a comment

Choose a reason for hiding this comment

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

What is the purpose here?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Under Review
Development

Successfully merging this pull request may close these issues.

3 participants