opensearch-project / flow-framework

OpenSearch plugin that enables builders to innovate AI apps on OpenSearch
Apache License 2.0
32 stars 35 forks source link

[FEATURE] Ml-commons plugin introducing a new optional field "config" in MLTooSpec #878

Closed jngz-es closed 3 weeks ago

jngz-es commented 1 month ago

Is your feature request related to a problem?

Ml-commons is introducing a new field config in MLToolSpec as showed in the pr https://github.com/opensearch-project/ml-commons/pull/2977. With the change, we allow cx to configure some static parameters for tools that stay unchanged/persist during the tool execution. It will fix a couple of bugs mentioned in the pr.

What solution would you like?

The pr https://github.com/opensearch-project/ml-commons/pull/2977.

What alternatives have you considered?

N/A.

Do you have any additional context?

It's an optional field.

dbwiddis commented 1 month ago

This should be a simple change to the ToolStep class adding the new optional parameter.