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

[Bug]: Unnecessary recursive call #8223

Open
theGitNoob opened this issue Feb 3, 2025 · 0 comments · May be fixed by #8225
Open

[Bug]: Unnecessary recursive call #8223

theGitNoob opened this issue Feb 3, 2025 · 0 comments · May be fixed by #8225
Labels
bug Something isn't working

Comments

@theGitNoob
Copy link
Contributor

What happened?

I was debugging a call to the method: litellm.get_model_info("together_ai/meta-llama/Meta-Llama-3.1-70B-Instruct-Turbo") when I noticed that several recursive calls were being executed unnecessarily. The problem is that this method is every time a together model would be used.
They are happening in this order:
get_model_info->get_supported_openai_params->litellm.TogetherAIConfig().get_supported_openai_params(model=model)->get_model_info

Image
Image
Image

Relevant log output

Are you a ML Ops Team?

No

What LiteLLM version are you on ?

v1.60.0

Twitter / LinkedIn details

No response

@theGitNoob theGitNoob added the bug Something isn't working label Feb 3, 2025
@theGitNoob theGitNoob linked a pull request Feb 4, 2025 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant