Skip to content

Conversation

@PicoCentauri
Copy link
Contributor

@PicoCentauri PicoCentauri commented Oct 6, 2025

Fixes #201

Contributor (creator of pull-request) checklist

  • Tests updated (for new features and bugfixes)?
    - [ ] Documentation updated (for new features)?
  • Issue referenced (for PRs that solve an issue)?

Reviewer checklist

  • CHANGELOG updated with public API or any other important changes?

📚 Documentation preview 📚: https://torch-pme--203.org.readthedocs.build/en/203/

@PicoCentauri PicoCentauri requested a review from E-Rum October 6, 2025 10:25
Copy link
Contributor

@E-Rum E-Rum 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! Also a good catch that params should be copied for each test case!

@PicoCentauri PicoCentauri merged commit 52eda7b into main Oct 7, 2025
14 checks passed
@PicoCentauri PicoCentauri deleted the fix-tests branch October 7, 2025 12:16
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.

Broken tests in tests/calculators/test_workflow.py

3 participants