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

Additional unit tests #406

Merged
merged 16 commits into from
Nov 21, 2024
Merged

Additional unit tests #406

merged 16 commits into from
Nov 21, 2024

Conversation

finozzifa
Copy link
Contributor

@finozzifa finozzifa commented Nov 12, 2024

Changes proposed in this Pull Request

The pull request proposes the following additional unit tests:

  • test/test_creation.py:
    • test_name
    • test_module
    • test_crs
    • test_shape
    • test_extent
    • test_bounds
    • test_transform
    • test_transform_r

Checklist

  • Code changes are sufficiently documented; i.e. new functions contain docstrings and further explanations may be given in doc.
  • Unit tests for new features were added (if applicable).
  • Newly introduced dependencies are added to environment.yaml, environment_docs.yaml and setup.py (if applicable).
  • A note for the release notes doc/release_notes.rst of the upcoming release is included.
  • I consent to the release of this PR's code under the MIT license.

@finozzifa finozzifa marked this pull request as ready for review November 20, 2024 14:50
Copy link
Member

@lkstrp lkstrp left a comment

Choose a reason for hiding this comment

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

Only some Atlite instead of atlite left.

Thank you @finozzifa !

atlite/cutout.py Outdated Show resolved Hide resolved
atlite/cutout.py Outdated Show resolved Hide resolved
atlite/cutout.py Outdated Show resolved Hide resolved
@lkstrp lkstrp merged commit 1f994fd into PyPSA:master Nov 21, 2024
13 of 19 checks passed
@finozzifa finozzifa deleted the extra_unit_tests branch November 22, 2024 09:18
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.

2 participants