-
Notifications
You must be signed in to change notification settings - Fork 8
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
Horrible memory footprint... #11
Comments
I faced the same issue. Deployment crashed because OOM error. Took a while to figure this out. Would be great if this is fixed soon |
@leandermaerkisch The latest pydantic causes an unbounded memory leak. If you pin pydantic==2.10.6, it should fix this.... This auto-generated SDK really needs some work. |
That's great news, thanks a bunch @false-vacuum! |
@leandermaerkisch @false-vacuum we just reduced some of the sizes of Sorry about the inconvenience so far! |
Thanks for fixing this rapidly!
…On Fri, 4 Apr 2025 at 01:53, Deep Singhvi ***@***.***> wrote:
@leandermaerkisch <https://github.com/leandermaerkisch> @false-vacuum
<https://github.com/false-vacuum> we just reduced some of the sizes of
__init__.py to reduce the footprint by ~3x. We'll be making an update
later to use typedicts instead of pydantic models to further reduce the
memory pressure.
Sorry about the inconvenience so far!
—
Reply to this email directly, view it on GitHub
<#11 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AIZFPQ45KI6INDROBV7BB5L2XXJYTAVCNFSM6AAAAABYTJJDIKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDONZXGMYDENZXGA>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
[image: dsinghvi]*dsinghvi* left a comment (VapiAI/server-sdk-python#11)
<#11 (comment)>
@leandermaerkisch <https://github.com/leandermaerkisch> @false-vacuum
<https://github.com/false-vacuum> we just reduced some of the sizes of
__init__.py to reduce the footprint by ~3x. We'll be making an update
later to use typedicts instead of pydantic models to further reduce the
memory pressure.
Sorry about the inconvenience so far!
—
Reply to this email directly, view it on GitHub
<#11 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AIZFPQ45KI6INDROBV7BB5L2XXJYTAVCNFSM6AAAAABYTJJDIKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDONZXGMYDENZXGA>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
--
Best regards / Mit freundlichen Grüßen,
Leander Märkisch
|
Why does just importing this use 500+ mib of memory? That's insane.
Steps to reproduce
The text was updated successfully, but these errors were encountered: