-
Notifications
You must be signed in to change notification settings - Fork 28
V0.2.0 release changelog, pyproject.toml update, codecov fix #148
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
Conversation
@@ -47,9 +47,7 @@ test = [ | |||
mace = ["mace-torch>=0.3.11"] | |||
mattersim = ["mattersim>=0.1.2"] | |||
metatensor = ["metatensor-torch >=0.7,<0.8", "metatrain[pet] >=2025.4"] | |||
orb = [ | |||
"orb-models@git+https://github.com/orbital-materials/orb-models#egg=637a98d49cfb494e2491a457d9bbd28311fecf21", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Was there a reason why the Orb version pinned? Otherwise look good.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
it was pinned because they released parts of the v3 code around 2 weeks before cutting v3 release and sharing the new models. It was being pinned to the repo so that we could write the model around the new orb tooling in the knowledge v3 was coming.
@@ -47,7 +47,7 @@ jobs: | |||
--ignore=tests/models/test_metatensor.py | |||
|
|||
- name: Upload coverage to Codecov | |||
uses: codecov/codecov-action@v5 | |||
uses: codecov/codecov-action@v5.4.0 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
v5.4.2 is out so we can revert to v5 to get later releases
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great, I can sneak this in to a future PR
Summary
Include a summary of major changes in bullet points:
Checklist
Work-in-progress pull requests are encouraged, but please enable the draft status on your PR.
Before a pull request can be merged, the following items must be checked:
Run ruff on your code.
Note that the CI system will run all the above checks. But it will be much more
efficient if you already fix most errors prior to submitting the PR. It is highly
recommended that you use the pre-commit hook provided in the repository. Simply run
pre-commit install
and a check will be run prior to allowing commits.