Skip to content

Conversation

@CalebMartinUiPath
Copy link
Contributor

No description provided.

Copy link
Collaborator

@radu-mocanu radu-mocanu left a comment

Choose a reason for hiding this comment

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

looks good,
we can merge it after prereq PR's are in

we need to also bump the version

"type": "string",
"description": query_setting.description
if query_setting and query_setting.description
else "The query to create a deeprag off of",

Choose a reason for hiding this comment

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

can probably improve this description

@CalebMartinUiPath CalebMartinUiPath marked this pull request as ready for review January 29, 2026 08:07
output_schema=output_model.model_json_schema(),
example_calls=[], # Examples cannot be provided for internal tools
)
async def batch_transform_tool_fn(**kwargs: Any) -> dict[str, Any]:

Choose a reason for hiding this comment

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

can the args not be typed?

Copy link
Contributor Author

@CalebMartinUiPath CalebMartinUiPath Jan 30, 2026

Choose a reason for hiding this comment

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

the problem with typing the args is that then I would have to duplicate the function. 1 for static which doesn't expect a query and 1 for dynamic which needs a query. which I could do idc

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