dify icon indicating copy to clipboard operation
dify copied to clipboard

feat: add support for Bedrock LLAMA3

Open longzhihun opened this issue 1 year ago • 1 comments

Description

Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.

Fixes # (issue)

Type of Change

Please delete options that are not relevant.

  • [x] New feature (non-breaking change which adds functionality)

How Has This Been Tested?

  1. Create Chat bot
  2. Select LLAMA3 as llm provider
  3. chat, find any error in chat process

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

  • [ ] TODO

Suggested Checklist:

  • [x] I have performed a self-review of my own code
  • [x] I have commented my code, particularly in hard-to-understand areas
  • [x] My changes generate no new warnings
  • [ ] I ran dev/reformat(backend) and cd web && npx lint-staged(frontend) to appease the lint gods
  • [ ] optional I have made corresponding changes to the documentation
  • [ ] optional I have added tests that prove my fix is effective or that my feature works
  • [ ] optional New and existing unit tests pass locally with my changes

longzhihun avatar Apr 26 '24 09:04 longzhihun

Hi, thanks for the pr. lgtm. can you place the meta models together in _position.yaml as well?

done, please review.

longzhihun avatar Apr 26 '24 22:04 longzhihun