Evaluate feature PR for potential merge

[snova-rodrigom]

Is your feature request related to a problem? Please describe.
Not related to a problem but a request. We have a PR here that hasn't been evaluated yet and it's about integrating SambaNova. Could you please help us with that?

Describe the solution you'd like
Please, review PR above and let us know how we could help to merge it.

Describe alternatives you've considered
None

Additional context
We're trying to integrate SambaNova Cloud endpoints into Instructor, so devs could also have that possibility.

[ivanleomk]

@snova-rodrigom sorry missed the PR, let me get on reviewing it now

[ivanleomk]

Hey @snova-rodrigom just looked at the PR and it seems like under the hood, it's using the OpenAI client to make the calls.

I don't think we should define a separate Samba Nova class in this instance since that just adds an additional layer of abstraction that will be difficult to maintain. People should just use the from_openai class in that case because that has a much richer layer of functionality too.

Would you be open instead to having a integrations page entry for samba nova?

[snova-rodrigom]

hi @ivanleomk , yeah that'd be great. We may have a proper SDK soon, could you give me more details about the integrations page?