Skip to content

Conversation

@MuggleJinx
Copy link
Collaborator

Description

Close #1976.

@lightaime
Copy link
Member

I am not quite sure about solving this issue by hardcoding a hidden prompt is a good idea or not

@fengju0213
Copy link
Collaborator

actually,in chatagent ,self.content in FunctionCallingMessage is set empty
image

@fengju0213 fengju0213 self-requested a review March 27, 2025 12:49
@MuggleJinx
Copy link
Collaborator Author

MuggleJinx commented Mar 28, 2025

actually,in chatagent ,self.content in FunctionCallingMessage is set empty

image

Thanks! Will double check this one.

@fengju0213 fengju0213 requested review from raywhoelse and zjrwtx March 28, 2025 07:44
@MuggleJinx
Copy link
Collaborator Author

The thing is, I cannot recover this error locally. Claude 3.7 can perform tool calling. Have you guys run into same error?

@fengju0213
Copy link
Collaborator

The thing is, I cannot recover this error locally. Claude 3.7 can perform tool calling. Have you guys run into same error?

i will try it

@fengju0213
Copy link
Collaborator

The thing is, I cannot recover this error locally. Claude 3.7 can perform tool calling. Have you guys run into same error?

hi @MuggleJinx, are you using the official API?

@MuggleJinx
Copy link
Collaborator Author

The thing is, I cannot recover this error locally. Claude 3.7 can perform tool calling. Have you guys run into same error?

hi @MuggleJinx, are you using the official API?

Yes.

@Wendong-Fan
Copy link
Member

thanks @MuggleJinx and sorry for the late reply, I created another PR: #2582, feel free to help reviewing the code

@MuggleJinx MuggleJinx closed this Jul 1, 2025
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.

[Feature Request] set default content in FunctionCallingMessage::to_openai_assistant_message

7 participants