-
Notifications
You must be signed in to change notification settings - Fork 290
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
Create Client websocket.py #179
base: main
Are you sure you want to change the base?
Conversation
@calclavia thanks for pushing this draft, I'd love to see tests and happy to assist. |
any update on this? |
Hey, sorry I missed this thread. Will run some tests and update! |
Co-authored-by: Marcelo Trylesinski <[email protected]>
@Kludex I addressed your comments by fixing it. Also added a test and that new test passes. |
Btw, I'm not a maintainer here. I was just passing by. |
My bad - notifying @jalleyne |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
awesome - I've had a play with it locally and it works well. Once the requested changes are in I'm happy to merge
@jerome3o-anthropic Addressed comments! |
Adds a WebSocket client.
Motivation and Context
Currently, there's a WebSocket server but no client. Allows Python clients to connect to WS servers.
How Has This Been Tested?
This is a draft PR. If there's interest in this, then we can move on to adding a test.
Types of changes
Checklist