Michael Hahn
Michael Hahn
Ah I was looking for that there but missed it. Thanks! On Wed, Jul 17, 2024 at 6:50 PM Joseph Heck ***@***.***> wrote: > @mwildehahn https://discord.gg/T3nqMT6qbM > should do the...
``` from anthropic.types import ( MessageParam, TextBlockParam, ) from pydantic_ai.messages import ModelMessage, ModelRequest, SystemPromptPart from pydantic_ai.models.anthropic import AnthropicModel class AnthropicModelWithCache(AnthropicModel): async def _map_message( # type: ignore self, messages: list[ModelMessage] )...
I was testing this and noticed that when making requests, it will sometimes include this message to the responses API (i think from a tool call?): `{'id': '', 'summary': [{'text':...
it seems like here: ``` if thinking_parts: openai_messages.append( responses.ResponseReasoningItemParam( id=thinking_parts[0].signature or '', summary=[Summary(text=item.content, type='summary_text') for item in thinking_parts], type='reasoning', ) ) ``` we should not include if it has an...
Some of this was fixed with: https://github.com/openai/openai-python/issues/2311 but now seeing: - UserWarning: Handling of this event type is not yet implemented. Please report on our GitHub: ResponseReasoningSummaryTextDeltaEvent(delta='...', item_id='rs_680b0e42dd748191baf3259afcab72b40792f36bcb0feba3', output_index=0, summary_index=2,...
a couple things i found: - the response tool call from openai is `ResponseFunctionToolCall(arguments='', call_id='call_IstYYmboPGwj3IZpQBm1KOUr', name='...', type='function_call', id='fc_680b21b9319c8191b215aa9e4a73538004237e9105833371', status='in_progress')` specifically, they have a: - call_id - id Right now both...
Changes I had to get this to work with openai: https://github.com/pydantic/pydantic-ai/pull/1587
For more context, I'm using this in swift with openapi generated types so it natively tries to decode to an int and then fails.
It's sort of documented here: https://electric-sql.com/docs/api/http#shape-log > The values included in the shape log are strings formatted according to Postgres' display settings. But it could be helpful to have in...
can this get merged?