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

Enhance Ollama Model Integration and Type Definitions #809

Closed
wants to merge 26 commits into from

Conversation

TarekS93
Copy link

This pull request introduces improvements to the Ollama model integration and refines type definitions for better type safety and clarity. The changes are primarily focused on the constants.ts and types.ts files.

Key Changes:

  1. In constants.ts:

    • Updated imports to include OllamaApiResponse and OllamaModel types.
    • Modified the getOllamaModels function to use the newly defined types.
    • Improved type safety by specifying the response type as OllamaApiResponse.
    • Updated the model mapping to use the OllamaModel type instead of any.
  2. In types.ts:

    • Made the OllamaModel interface exportable by adding the export keyword.
    • This change allows the interface to be imported and used in other parts of the application, enhancing type consistency across the project.

These updates improve the overall type safety of the application, particularly in handling Ollama model data. They also enhance code readability and maintainability by using more specific types instead of any.

The changes are minimal but impactful, ensuring better integration with the Ollama API and providing a more robust type system for future development.

coleam00 and others added 26 commits October 13, 2024 13:53
Add provider filtering on model list
Added Google Generative AI (gemini) integration
let the ollama models be auto generated from ollama api
added download button to downloade generated code in a zip file
I have two accounts and one that commited is yunatamos
Sync with stackblitz upstream
docs: fix grammar in readme

just cleaned up some awkward phrasing + fixed a few typos :)

- kofi
@TarekS93 TarekS93 closed this Oct 22, 2024
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.

9 participants