θΏ™ζ˜―indexlocζδΎ›ηš„ζœεŠ‘οΌŒδΈθ¦θΎ“ε…₯任何密码
Skip to content

Conversation

@shatfield4
Copy link
Collaborator

Pull Request Type

  • ✨ feat
  • πŸ› fix
  • ♻️ refactor
  • πŸ’„ style
  • πŸ”¨ chore
  • πŸ“ docs

Relevant Issues

resolves #xxx

What is in this change?

  • Implement useProviderEndpointAutoDiscovery hook for LocalAI embedding provider and LLM provider
  • Auto detects URL to help with friction of LocalAI setup

Additional Information

Developer Validations

  • I ran yarn lint from the root of the repo & committed changes
  • Relevant documentation has been updated
  • I have tested my code functionality
  • Docker build succeeds locally

@shatfield4 shatfield4 added the PR:needs review Needs review by core team label Aug 10, 2024
Copy link
Member

@timothycarambat timothycarambat left a comment

Choose a reason for hiding this comment

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

Lets add the port to defaults - otherwise the default URLs will never resolve automatically.

@timothycarambat timothycarambat merged commit dfb951e into master Aug 12, 2024
@timothycarambat timothycarambat deleted the localai-auto-url branch August 12, 2024 23:12
cabwds pushed a commit to cabwds/anything-llm that referenced this pull request Jul 3, 2025
* implement auto url detection for localai

* add ports to localai common urls
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

PR:needs review Needs review by core team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants