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.

Sign up for GitHub

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

Jump to bottom

feat: compare lam statistics #203

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

feat: compare lam statistics #203

wants to merge 4 commits into from

Conversation

icedoom888
Copy link

@icedoom888 icedoom888 commented Feb 10, 2025 •
edited by github-actions bot
Loading

Description

Adding a new command to compare statistics withing a LAM region across 2 datasets.

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

Issue Number

Closes #202

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

Dependencies

  • I have ensured that the code is still pip-installable after the changes and runs
  • I have tested that new dependencies themselves are pip-installable.

prettytable
print_color
termcolor

Documentation

  • My code follows the style guidelines of this project
  • I have updated the documentation and docstrings to reflect the changes
  • I have added comments to my code, particularly in hard-to-understand areas

Added New command: compareLAM

📚 Documentation preview 📚: https://anemoi-datasets--203.org.readthedocs.build/en/203/

@FussyDuck
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@github-actions github-actions bot added contributor documentation Improvements or additions to documentation dependencies Pull requests that update a dependency file labels Feb 10, 2025
@icedoom888 icedoom888 self-assigned this Feb 10, 2025
@icedoom888 icedoom888 added the enhancement New feature or request label Feb 10, 2025
@icedoom888 icedoom888 changed the title (feat) compare lam statistics feat: compare lam statistics Feb 10, 2025
@icedoom888
Copy link
Author

All failed checks are related to issues and code not affected by this PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@theissenhelen theissenhelen Awaiting requested review from theissenhelen theissenhelen is a code owner

@JesperDramsch JesperDramsch Awaiting requested review from JesperDramsch JesperDramsch is a code owner

@gmertes gmertes Awaiting requested review from gmertes gmertes is a code owner

@b8raoult b8raoult Awaiting requested review from b8raoult b8raoult is a code owner

@floriankrb floriankrb Awaiting requested review from floriankrb floriankrb is a code owner

@anaprietonem anaprietonem Awaiting requested review from anaprietonem anaprietonem is a code owner

@HCookie HCookie Awaiting requested review from HCookie HCookie is a code owner

@JPXKQX JPXKQX Awaiting requested review from JPXKQX JPXKQX is a code owner

@mchantry mchantry Awaiting requested review from mchantry mchantry is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

@icedoom888 icedoom888

Labels
contributor dependencies Pull requests that update a dependency file documentation Improvements or additions to documentation enhancement New feature or request
Projects
Anemoi-dev
Status: Under Review
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Feature Request: Overlap Dataset Statistic Comparison
2 participants
@icedoom888 @FussyDuck