Skip to content

Commit

Permalink
removed extra comment
Browse files Browse the repository at this point in the history
  • Loading branch information
brainlid committed Feb 5, 2025
1 parent ad8609e commit dfa58ff
Showing 1 changed file with 0 additions and 2 deletions.
2 changes: 0 additions & 2 deletions test/chat_models/chat_open_ai_test.exs
Original file line number Diff line number Diff line change
Expand Up @@ -580,8 +580,6 @@ defmodule LangChain.ChatModels.ChatOpenAITest do
assert result == expected
end

# @mrluc I found the issue! It's with OpenAI's converting a Message into JSON when the LLM includes talking content AND a tool call. It isn't including the tool call content in the JSON that goes back to OpenAI. Should be an easy fix.

test "turns an assistant message with text and tool calls into expected JSON format" do
openai = ChatOpenAI.new!()

Expand Down

0 comments on commit dfa58ff

Please sign in to comment.