Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Run documents in batch when the prompt has tool calls #784

Merged
merged 1 commit into from
Jan 22, 2025

Conversation

andresgutgon
Copy link
Contributor

@andresgutgon andresgutgon commented Jan 20, 2025

What?

We implemented tool calling mocking in our playground. But we also need to handle tool calling when the running of the prompt is in batch

TODO

  • Create a document for gen tool responses in development workspace
  • Implement AI service to auto-generate tool responses based on tool calls
  • Handle tool calling when running in batch from the playground
  • Handle tool calling when running an evaluation in batch
  • Provision missing environment variables in production I'll postpone from now. I did a default variable if the env is not there.
  • Create a document for gen tool responses in the production workspace. Copy from dev

Next PR

  • Tests for run batch service when tool calls done here

@andresgutgon andresgutgon added the 🚧 wip Work in progress label Jan 20, 2025
@andresgutgon andresgutgon force-pushed the fix/run-batch-with-tool-calls branch from 4e1610e to 23ce1dd Compare January 20, 2025 14:33
)
}

const toolCallResponses = response.object as ToolCallResponse[]
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Validate that the returned object is like an array of ToolCallResponse. AI could hallucinate

@andresgutgon andresgutgon force-pushed the fix/run-batch-with-tool-calls branch 2 times, most recently from c5d36c4 to 38445e8 Compare January 22, 2025 15:10
@andresgutgon andresgutgon force-pushed the fix/run-batch-with-tool-calls branch from 38445e8 to 43337e7 Compare January 22, 2025 15:15
@andresgutgon andresgutgon removed the 🚧 wip Work in progress label Jan 22, 2025
@andresgutgon andresgutgon merged commit 1849d44 into main Jan 22, 2025
4 checks passed
@andresgutgon andresgutgon deleted the fix/run-batch-with-tool-calls branch January 22, 2025 16:15
@github-actions github-actions bot locked and limited conversation to collaborators Jan 22, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants