Skip to content

feat: Allow empty config path to run from overrides #206

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

Merged
merged 11 commits into from
Apr 24, 2025

Conversation

gmertes
Copy link
Member

@gmertes gmertes commented Apr 10, 2025

Description

Allows the user to pass no config to run, couple and retrieve to run completely from the overrides, useful for quick debugging.

Example:
anemoi-inference run input=mars checkpoint=mycheckpoint.ckpt

Type of Change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update

Code Compatibility

  • I have performed a self-review of my code

Code Performance and Testing

  • I have added tests that prove my fix is effective or that my feature works
  • I ran the complete Pytest test suite locally, and they pass

📚 Documentation preview 📚: https://anemoi-inference--206.org.readthedocs.build/en/206/

@github-project-automation github-project-automation bot moved this to Now In Progress in Anemoi-dev Apr 10, 2025
@github-actions github-actions bot added config bug Something isn't working documentation Improvements or additions to documentation enhancement New feature or request labels Apr 10, 2025
@gmertes gmertes removed bug Something isn't working documentation Improvements or additions to documentation labels Apr 10, 2025
@github-actions github-actions bot added bug Something isn't working documentation Improvements or additions to documentation labels Apr 10, 2025
@HCookie HCookie self-requested a review April 23, 2025 14:38
Copy link
Member

@HCookie HCookie left a comment

Choose a reason for hiding this comment

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

Looks good to me,
Can you add some comments to the docs to show this usage?

@gmertes
Copy link
Member Author

gmertes commented Apr 23, 2025

Done

@gmertes gmertes merged commit a5177d3 into main Apr 24, 2025
69 checks passed
@gmertes gmertes deleted the feat/run-from-overrides branch April 24, 2025 09:37
@github-project-automation github-project-automation bot moved this from Now In Progress to Done in Anemoi-dev Apr 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working config documentation Improvements or additions to documentation enhancement New feature or request
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants