Feature/sambanova integration

[rodrigo-92]

• Added sambanova client integration
• Added docs related to integration
• Added a patch to support arguments coming in dict format

---

Important

Add SambaNova Cloud integration for structured outputs with documentation and examples.

• Integration:
  • Add from_sambanova() function in client_sambanova.py to integrate SambaNova Cloud with Instructor.
  • Update __init__.py to include from_sambanova in available imports.
  • Add SAMBANOVA to Provider enum in utils.py.
• Documentation:
  • Add SambaNova integration details in docs/integrations/sambanova.md.
  • Update docs/index.md and docs/integrations/index.md to include SambaNova.
  • Add example usage in docs/examples/sambanova.md and examples/sambanova/sambanova_example.py.
• Functionality:
  • Modify parse_tools() in function_calls.py to handle dict arguments by converting them to JSON strings.

^{This description was created by for b4eb16a. It will automatically update as commits are pushed.}

[ellipsis-dev]

👍 Looks good to me! Reviewed everything up to b4eb16a in 1 minute and 12 seconds

More details

• Looked at 495 lines of code in 12 files
• Skipped 0 files when reviewing.
• Skipped posting 6 drafted comments based on config settings.

1. instructor/__init__.py:101

• Draft comment:
  Remove the unnecessary print statement.

• Reason this comment was not posted:
  Confidence changes required: 50%
  The print statement 'sambanova' in the init.py file is unnecessary and should be removed.

2. docs/blog/posts/open_source.md:239

• Draft comment:
  Assertions should include an error message for clarity and debugging purposes. This applies to the assertion on line 239 and similar assertions elsewhere in the code.
• Reason this comment was not posted:
  Confidence changes required: 80%
  The assertion statements in the code lack error messages, which is against the rule that assertions should always have an error message.

3. docs/examples/sambanova.md:54

• Draft comment:
  Assertions should include an error message for clarity and debugging purposes. This applies to the assertion on line 54 and similar assertions elsewhere in the code.
• Reason this comment was not posted:
  Confidence changes required: 80%
  The assertion statements in the code lack error messages, which is against the rule that assertions should always have an error message.

4. docs/integrations/sambanova.md:60

• Draft comment:
  Assertions should include an error message for clarity and debugging purposes. This applies to the assertion on line 60 and similar assertions elsewhere in the code.
• Reason this comment was not posted:
  Confidence changes required: 80%
  The assertion statements in the code lack error messages, which is against the rule that assertions should always have an error message.

5. examples/sambanova/sambanova_example.py:29

• Draft comment:
  Assertions should include an error message for clarity and debugging purposes. This applies to the assertion on line 29 and similar assertions elsewhere in the code.
• Reason this comment was not posted:
  Confidence changes required: 80%
  The assertion statements in the code lack error messages, which is against the rule that assertions should always have an error message.

6. instructor/__init__.py:101

• Draft comment:
  Remove the unnecessary print statement to clean up the code.
• Reason this comment was not posted:
  Confidence changes required: 80%
  The print statement on line 101 seems unnecessary and could be removed to clean up the code.

Workflow ID: wflow_l4ysQYQhLiPNl6xv

---

You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet mode, and more.

[rodrigo-92]

hey @jxnl could you help us with this integration please? Thanks in advance.