Skip to content

Conversation

@xiaochuan-du
Copy link

@xiaochuan-du xiaochuan-du commented Apr 1, 2024

  1. Renamed the provider from claude to bedrock-claude for AWS Bedrock Claude to distinguish between AWS Bedrock Claude and the Anthropic Claude clients.
  2. Resolved the issue where the history display was incorrectly using bedrock-claude.

@drawal1
Copy link
Contributor

drawal1 commented Apr 1, 2024

@xiaochuan-du - take a look at pull request #751 . I have expanded on the bare bones aws support in the old bedrock.py. Also refactored design into providers vs models. Suggest you add your fixes there

@xiaochuan-du
Copy link
Author

Hey @drawal1 , #751 is closed. Are you planning to create another pull request for the update?

@drawal1
Copy link
Contributor

drawal1 commented Apr 2, 2024

@xiaochuan-du - yes, I found some issues while testing. I will resubmit today

@drawal1
Copy link
Contributor

drawal1 commented Apr 4, 2024 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants