Skip to content

Commit

Permalink
fix: remove completion_params from validation of llm scope
Browse files Browse the repository at this point in the history
  • Loading branch information
Yeuoly committed Jan 3, 2025
1 parent baf83c5 commit cc2ae5a
Showing 1 changed file with 1 addition and 4 deletions.
5 changes: 1 addition & 4 deletions internal/types/entities/plugin_entities/config.go
Original file line number Diff line number Diff line change
Expand Up @@ -349,10 +349,7 @@ func ValidateProviderConfigs(settings map[string]any, configs []ProviderConfig)
case string(MODEL_CONFIG_SCOPE_ALL):
// do nothing
case string(MODEL_CONFIG_SCOPE_LLM):
// completion_params
if _, ok := m["completion_params"]; !ok {
return errors.New("setting " + config_name + " is missing completion_params")
}
// do nothing
case string(MODEL_CONFIG_SCOPE_TEXT_EMBEDDING):
// do nothing
case string(MODEL_CONFIG_SCOPE_RERANK):
Expand Down

0 comments on commit cc2ae5a

Please sign in to comment.