Skip to content

update ollama usage docs#5854

Merged
ekzhu merged 8 commits intomainfrom
ekzhu-update-models-doc
Mar 6, 2025
Merged

update ollama usage docs#5854
ekzhu merged 8 commits intomainfrom
ekzhu-update-models-doc

Conversation

@ekzhu
Copy link
Copy Markdown
Contributor

@ekzhu ekzhu commented Mar 6, 2025

Update the outdated ollama usage doc.

@ekzhu ekzhu requested a review from peterychang March 6, 2025 19:24
@codecov
Copy link
Copy Markdown

codecov bot commented Mar 6, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 75.60%. Comparing base (2177076) to head (44ed945).
Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #5854   +/-   ##
=======================================
  Coverage   75.60%   75.60%           
=======================================
  Files         189      189           
  Lines       12734    12734           
=======================================
  Hits         9627     9627           
  Misses       3107     3107           
Flag Coverage Δ
unittests 75.60% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@peterychang
Copy link
Copy Markdown
Contributor

Should we mark the client as experimental/beta? Its seems pretty bare bones and untested for a full release?

Copy link
Copy Markdown
Contributor

@peterychang peterychang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe mark the client as experimental/beta? Otherwise LGTM

@ekzhu
Copy link
Copy Markdown
Contributor Author

ekzhu commented Mar 6, 2025

Should we mark the client as experimental/beta? It seems pretty bare bones and untested for a full release?

When is a full release ready? We can put a notice on it.

@peterychang
Copy link
Copy Markdown
Contributor

Should we mark the client as experimental/beta? It seems pretty bare bones and untested for a full release?

When is a full release ready? We can put a notice on it.

I'm not sure.. whenever we have time, really. Its usable now, just not fully featured

@ekzhu ekzhu merged commit 907dad4 into main Mar 6, 2025
55 checks passed
@ekzhu ekzhu deleted the ekzhu-update-models-doc branch March 6, 2025 20:53
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.

3 participants