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.

Sign up for GitHub

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

Jump to bottom

Write unit tests for the rest of the LLM classes #15

Open
sestinj opened this issue Sep 4, 2023 · 3 comments
Open

Write unit tests for the rest of the LLM classes #15

sestinj opened this issue Sep 4, 2023 · 3 comments
Labels
good first issue Good for newcomers

Comments

@sestinj
Copy link
Contributor

sestinj commented Sep 4, 2023 •
edited

We already have a framework setup to test subclasses of BaseLLM in llm.test.ts, but are missing some of the LLM providers found in core/llm/llms.

This task is mostly a matter of copying the patterns seen above in the file, running the tests, and modifying as needed to make sure that the tests can pass for each.
@sestinj sestinj added the good first issue Good for newcomers label Sep 4, 2023
@khareyash05
Copy link

Hey @sestinj I would love to take this one up. If this is open. Thanks

@sestinj
Copy link
Contributor Author

sestinj commented Mar 18, 2024

@khareyash05 Sure that would be great! Is there anything you'd want me to clarify to help you get started?

@khareyash05
Copy link

@khareyash05 Sure that would be great! Is there anything you'd want me to clarify to help you get started?

I will let you know if i need something. Thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
good first issue Good for newcomers
Projects
Continue Contribution Ideas
Status: Good First Issues (Code)
Milestone
No milestone
Development

No branches or pull requests
2 participants
@sestinj @khareyash05