opik icon indicating copy to clipboard operation
opik copied to clipboard

Add Bedrock provider to LlmProvider

Open dhvll opened this issue 7 months ago • 2 comments

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

dhvll avatar May 29 '25 18:05 dhvll

HI @dhvll,

Thanks for your contribution.

Can you add tests for it? You can check the createAndGetCost test setup.

thiagohora avatar May 29 '25 20:05 thiagohora

Sure will add it

dhvll avatar May 29 '25 20:05 dhvll

@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.

vincentkoc avatar May 30 '25 08:05 vincentkoc