-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Add support for Cerebras Cloud SDK #189
Conversation
1e8bf7e
to
97e7257
Compare
763d592
to
8519fad
Compare
Note that there is a stale PR which seeks to accomplish the same thing: #109 This PR has been tested and is authored by Cerebras staff, and therefore should take precedence. |
@rohitprasad15 This PR is ready to review. Please advise for process for submission. |
66eeb1f
to
d239e38
Compare
@ksolo @rohitprasad15 - we would love to get a maintainer review on this. Thanks! |
5062310
to
5fdd02e
Compare
@hagayl-cerebras @kamilk-cerebras Sorry on the delay here. I can take a look on Monday (3/3) if @rohitprasad15 does not get ti it before then. |
Hi all, Just a friendly reminder about this pending PR! Thank you all in advance for your time. |
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.
This looks good. I'll need to manually test the provider by following the instructions in the readme. I'll have this done tonight and leave any additional notes from that. Provided that goes well we can get this merged soon (tomorrow).
5fdd02e
to
dccdb8f
Compare
Fixes #195