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

Python: metkitlib integration #75

Merged
merged 1 commit into from
Mar 28, 2025
Merged

Conversation

tmi
Copy link
Contributor

@tmi tmi commented Mar 28, 2025

1/ Moves pymetkit into python/pymetkit, for standard organization. However, pyproject.toml remains rooted until ci is adapted
2/ Adds a dependency of metkitlib to pymetkit, so that the libmetkit.so is brought in automatically

I've tested as follows in a clean venv:

uv pip install --prerelease=allow -e .
python -c 'import pymetkit._version; print(pymetkit._version.__version__)' # simple test that it can be imported
pytest ./python/pymetkit/tests

and all looked good

Note that the --prerelease=allow param is needed until all of eccodes, eckit and metkit are released (in the sense of develop->main merge) and then wheels published (in the sense of running corresponding github action from main)

1/ Moves pymetkit into python/pymetkit, for standard organization
   However, pyproject.toml remains rooted until ci is adapted
2/ Adds a dependency of metkitlib to pymetkit, so that the libmetkit.so
   is brought in automatically
@tmi tmi force-pushed the feat/metkitlibIntegration branch from 19a25a6 to a51d901 Compare March 28, 2025 13:59
@simondsmart simondsmart merged commit e6c1c6c into develop Mar 28, 2025
4 checks passed
@simondsmart simondsmart deleted the feat/metkitlibIntegration branch March 28, 2025 19:27
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