Skip to content

Conversation

jnnkhrtmnn
Copy link
Contributor

This PR makes it easier to integrate with other LLMs that are not OpenAI (eg AzureOpenAI or any other Local models, as long as they are recognized as LLM models in guidance-ai).

All tests passed. I did not commit any changes based on linting and formatting, as these will many files, even with the current configuration in the main branch.

Signed-off-by: Jannik hartmann <jannik.hartmann@outlook.de>
@jnnkhrtmnn jnnkhrtmnn force-pushed the integrate_additional_llms branch from 97d0916 to d942c96 Compare July 17, 2025 09:11
@jnnkhrtmnn
Copy link
Contributor Author

.. added sign-off to commit message

@amit-sharma
Copy link
Member

very useful, thanks for adding this @jnnkhrtmnn. Approving.

@all-contributors please add @jnnkhrtmnn for code.

Copy link
Contributor

@amit-sharma

I've put up a pull request to add @jnnkhrtmnn! 🎉

@amit-sharma amit-sharma merged commit 40fe922 into py-why:main Jul 20, 2025
4 checks passed
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.

2 participants