Feat(ollama): Adding ollama for enabling local model agents #153
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
|
@arunabhcode is attempting to deploy a commit to the Sim Studio Team on Vercel. A member of the Team first needs to authorize it. |
arunabhcode
force-pushed
the
ollama-ticket
branch
from
becd6a4 to
3f74076
Compare
arunabhcode
force-pushed
the
ollama-ticket
branch
from
3f74076 to
7abc1f0
Compare
Contributor
Author
|
Start of this PR, initially just added the orchestration part. Will test and start to add the logic after. |
arunabhcode
force-pushed
the
ollama-ticket
branch
from
8730e85 to
7bd4aa7
Compare
arunabhcode
force-pushed
the
ollama-ticket
branch
from
73283e0 to
ac0ee95
Compare
arunabhcode
force-pushed
the
ollama-ticket
branch
from
ac0ee95 to
3b14342
Compare
arunabhcode
force-pushed
the
ollama-ticket
branch
from
3b14342 to
afbc843
Compare
waleedlatif1
force-pushed
the
main
branch
2 times, most recently
from
1bc12e1 to
fb5bd43
Compare
|
Deployment failed with the following error: |
arunabhcode
force-pushed
the
ollama-ticket
branch
from
9e6b603 to
06f48ce
Compare
waleedlatif1
previously approved these changes
Mar 28, 2025
Comment on lines
21
to
33
| // Evaluate options if it's a function and deduplicate based on values | ||
| const evaluatedOptions = useMemo(() => { | ||
| const rawOptions = typeof options === 'function' ? options() : options | ||
| const seen = new Set<string>() | ||
| return rawOptions.filter(option => { | ||
| const value = typeof option === 'string' ? option : option.id | ||
| if (seen.has(value)) return false | ||
| seen.add(value) | ||
| return true | ||
| }) | ||
| }, [options]) | ||
|
|
||
|
|
Collaborator
There was a problem hiding this comment.
@arunabhcode i like that, getModelProviders and getInitializedModelProviders
Collaborator
|
@arunabhcode tests are failing, but I left a review so once this is resolved we can merge it in |
…ent docker compose files and add a shell script to toggle between the deployment docker compose files add base ollama.ts implementation add latest attempt to fetch Ollama models dynamically fix ollama dynamic model fetching, models now being rendered on GUI fix package and package-lock.json to remove ollama dependency and add types.ts for ollama switch MODEL_PROVIDERS to getModelProviders make dynamic ollama model dropdown change using zustland store make dynamic ollama model changes to router and evaluator ts too
…ag to start_simstudio_docker.sh with ollama service
arunabhcode
force-pushed
the
ollama-ticket
branch
from
aaae88d to
c8eb525
Compare
arunabhcode
force-pushed
the
ollama-ticket
branch
from
54f25eb to
2e6051f
Compare
waleedlatif1
approved these changes
Mar 29, 2025
RadoBoiii
pushed a commit
to RadoBoiii/sim-fork
that referenced
this pull request
Apr 13, 2025
…oai#153) * feat(ollama): add ollama package dependency, add two separate deployment docker compose files and add a shell script to toggle between the deployment docker compose files add base ollama.ts implementation add latest attempt to fetch Ollama models dynamically fix ollama dynamic model fetching, models now being rendered on GUI fix package and package-lock.json to remove ollama dependency and add types.ts for ollama switch MODEL_PROVIDERS to getModelProviders make dynamic ollama model dropdown change using zustland store make dynamic ollama model changes to router and evaluator ts too * feat(ollama): fix evaluated options by de-duplicating it * feat(ollama): make README.md change to reflect local model workflow * feat(ollama): add base non-ollama docker compose file, add --local flag to start_simstudio_docker.sh with ollama service * feat(ollama): fix README.md local model instructions * feat(ollama): remove de-duplication logic and separate getModelProviders into two * fix non-local init and translate.ts * create combined docker-compose file and fix start_simstudio_docker script too * update package-lock.json * feat(ollama): fix README.md instructions and docker compose --------- Co-authored-by: Arunabh Sharma <arunabh.sharma@supernal.aero>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
This PR will address Ollama integration into sim-studio for enabling local model use.
Type of change
Please delete options that are not relevant.
Will fill the rest of this out at before converting to reviewable PR.
How Has This Been Tested?
Checklist:
Security Considerations:
Additional Information:
Any additional information, configuration or data that might be necessary to reproduce the issue or use the feature.