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

Model Tester: An model testing CLI #1393

Closed
wants to merge 13 commits into from

Conversation

Abellegese
Copy link
Contributor

@Abellegese Abellegese commented Nov 20, 2024

Thank you for taking your time to contribute to Ersilia, just a few checks before we proceed

  • Have you followed the guidelines in our Contribution Guide
  • Have you written new tests for your core changes, as applicable?
  • Have you successfully ran tests with your changes locally?

Description

Model Tester: An model testing CLI

@miquelduranfrigola
Copy link
Member

FYI, Copilot-generated summary:

This pull request includes significant updates to the ersilia/cli/commands/test.py file and changes to the pyproject.toml file to enhance the functionality and dependencies of the project. The most important changes include the addition of new command-line options for the test_cmd function and the inclusion of new dependencies in the project configuration.

Enhancements to the test_cmd function:

  • ersilia/cli/commands/test.py: Added new command-line options --env, --type, --level, and --dir to the test_cmd function to provide more flexibility in specifying the environment, type, level, and directory for testing models.
  • ersilia/cli/commands/test.py: Removed unused imports to clean up the code and reduce potential dependencies.

Updates to project dependencies:

  • pyproject.toml: Added rich as an optional dependency to enhance the project's functionality.
  • pyproject.toml: Included nox in the test extras to support additional testing capabilities.

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.

3 participants