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

Evals #4

Merged
merged 5 commits into from
Dec 20, 2024
Merged

Evals #4

merged 5 commits into from
Dec 20, 2024

Conversation

Shuyib
Copy link
Owner

@Shuyib Shuyib commented Dec 20, 2024

This pull request includes several changes to the Makefile, README.md, and new test files to improve the project's testing capabilities and documentation. The most important changes include the addition of test commands in the Makefile, updates to the README.md to include testing guidelines and responsible AI practices, and the creation of unit and integration test files.

Makefile updates:

  • Added tests/*.py to the format target to ensure test files are also formatted.
  • Added a command to remove the tests/__pycache__ directory in the clean target.
  • Introduced a new test target to run tests using pytest.

Documentation updates:

  • Added a description of the tests directory and its contents in the README.md.
  • Included instructions for setting test-related environment variables in the README.md. [1] [2]
  • Added sections on responsible AI practices and testing guidelines in the README.md. [1] [2]

New test files:

  • Created tests/test_cases.py with unit tests for the function calling utilities, including tests for sending airtime, sending messages, and searching news.
  • Created tests/test_run.py with integration tests for the function calling system, including various scenarios and edge cases.

@Shuyib Shuyib merged commit 384d434 into main Dec 20, 2024
6 checks passed
@Shuyib Shuyib deleted the evals branch December 20, 2024 06:19
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.

1 participant