Skip to content

[ PATCH ] OpenAI computer use#107

Open
AhmedShriefNeferdata wants to merge 1 commit intomainfrom
openai-computer-use
Open

[ PATCH ] OpenAI computer use#107
AhmedShriefNeferdata wants to merge 1 commit intomainfrom
openai-computer-use

Conversation

@AhmedShriefNeferdata
Copy link
Copy Markdown
Collaborator

No description provided.

Comment thread src/llm_models/tools.rs
Comment on lines +135 to +141
impl Default for OpenAIComputerUseConfig {
fn default() -> Self {
Self {
tool_type: OpenAIComputerUseToolType::Computer,
}
}
}
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

You don't need to implement this. Since you already have Default derived at OpenAIComputerUseToolType and have defined Computer as the default variant, all you need to do is derive Default for OpenAIComputerUseConfig.

Comment thread src/domain.rs
}

#[derive(Deserialize, Serialize, Debug, Clone, Eq, PartialEq)]
pub struct OpenAIComputerCall {
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

How are these used? They only seem to be defined but not actually used in code.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I think we will need to modify llms::openai::get_version_data:

fn get_version_data(

If the user is calling Responses API and attached the computer use tool, we will need to parse the response into your new types.

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.

2 participants