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

Fix e2b websocket issue #177

Merged
merged 3 commits into from
Jul 30, 2024
Merged

Fix e2b websocket issue #177

merged 3 commits into from
Jul 30, 2024

Conversation

humpydonkey
Copy link
Collaborator

@humpydonkey humpydonkey commented Jul 22, 2024

  1. Upgrade e2b libs to a newer version
  2. Fix backward incompatible changes introduced from the lib upgrade
  3. Add a embedding LRU cache to reduce chat latency
  4. A set of stability improvements:
    • Remove check liveness call to improve stability
    • Bump pydantic to 2.7.4 to include a bug fix caused by the new e2b base image;
    • Rewrite save_video() using cv2 (get rid of moviepy);
    • Add retry for common e2b exceptions
    • Add more loggings

Remove health/liveness checking code
…ng cv2 (get rid of moviepy); Add retry for common e2b exceptions
Copy link
Member

@dillonalaird dillonalaird left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@humpydonkey humpydonkey merged commit 9f53e01 into main Jul 30, 2024
8 checks passed
@humpydonkey humpydonkey deleted the fix-e2b-websocket-issue branch July 30, 2024 06:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants