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

google.call and llm.call x logfire #891

Closed
CharlesOural opened this issue Mar 6, 2025 · 4 comments
Closed

google.call and llm.call x logfire #891

CharlesOural opened this issue Mar 6, 2025 · 4 comments
Labels
bug Something isn't working mirascope

Comments

@CharlesOural
Copy link
Contributor

Description

When there is an error 503 server side or if there is a tenacity error it don't show in the logfire span.

Very difficult to provide a test code, because you need to emulate a serverside error from the provider.

But if I understand correctly if there is an error we need to do :

e._response = response
raise e

maybe check if this flow is properly done inside google.call and llm.call ?

Tell me if you need anything

Python, Mirascope & OS Versions, related packages (not required)

python 3.12
mirascope latest 1.20.1
@CharlesOural CharlesOural added the bug Something isn't working label Mar 6, 2025
@willbakst
Copy link
Contributor

Can you patch the client call with a side effect to reproduce?

@willbakst
Copy link
Contributor

On the tenacity side, that makes sense? Tenacity is catching the error and making another call? Or are you saying if tenacity fails completely and throws a RetryError that that isn't caught?

Without a minimal repro it's hard to know what's going on

@CharlesOural
Copy link
Contributor Author

Humm I think this was when we received a 503 from google for example, but it's normal that there is no response (the model crashed) so I think this was not on your end...

@willbakst
Copy link
Contributor

Feel free to reopen if it turns out it's something on our end :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working mirascope
Projects
None yet
Development

No branches or pull requests

2 participants