opik
opik copied to clipboard
Add Bedrock provider to LlmProvider
Details
- Added 'BEDROCK' to the LlmProvider enum.
- Updated CostService to include 'bedrock' in the cost calculation mappings.
- Modified model_prices_and_context_window.json to include pricing details for new models and providers, enhancing the overall model support.
Issues
/fix #1836 Resolves /claim #1836
Testing
Documentation
HI @dhvll,
Thanks for your contribution.
Can you add tests for it? You can check the createAndGetCost test setup.
Sure will add it
@dhvll do you need help with this PR? also as per the bounty rules we need videos of working PRs, please refrain from raising PRs for untested and not working solutions.