Closed curlyfro closed 5 months ago
This update encompasses a series of class renamings and inheritance adjustments across various models in the Amazon Bedrock provider. Additionally, there are significant reductions in the MaxTokens
settings for multiple chat settings classes, aligning them to a new standard of 4000 tokens. New classes have also been introduced, expanding the functionality and specificity of the provider's offerings.
File Path | Change Summary |
---|---|
.../MetaLlamaChatModel.cs |
Renamed MetaLlama2ChatModel to MetaLlamaChatModel . |
.../MistralModel.cs |
Renamed MistralInstructChatModel to MistralModel . |
.../Chat/Settings/...ChatSettings.cs |
Standardized MaxTokens to 4000 across various settings classes. |
.../Predefined/Anthropic.cs |
Added Claude3OpusModel , updated inheritance details. |
.../Predefined/Meta.cs |
Added new models, updated IDs, and changed inheritance for existing models. |
.../Predefined/Mistral.cs |
Renamed classes, updated their inheritance to MistralModel . |
🐰✨ Changes abound, code does hop, From fields so green, to mountain top. MaxTokens fall, like autumn leaves, New classes spring, as nature weaves. Oh, merry code, dance your new beat, With every line, a feat complete! 🌟 🐰💻
src/Providers/Amazon.Bedrock/src/Predefined/Mistral.cs (2)
`6-6`: Ensure the identifier for `Mistral7BInstruct` is consistent with other model naming conventions. --- `10-10`: The identifier for `Mistral8x7BInstruct` seems appropriate. Confirm that the versioning is correct.src/Providers/Amazon.Bedrock/src/Predefined/Meta.cs (5)
`7-7`: Confirm the identifier for `Llama2With13BModel` matches the expected format and versioning. --- `11-11`: Ensure the identifier for `Llama2With70BModel` is consistent with other model naming conventions. --- `15-15`: The identifier for `Llama2WithChat13BModel` seems appropriate. Confirm that the versioning is correct. --- `23-23`: Check if the identifier for `Llama3With8BInstructBModel` is correctly formatted and versioned. --- `27-27`: Ensure the identifier for `Llama3With70BInstructBModel` is consistent with other model naming conventions.src/Providers/Amazon.Bedrock/src/Predefined/Anthropic.cs (2)
`22-22`: Confirm the identifier for `Claude3HaikuModel` matches the expected format and versioning. --- `26-26`: Ensure the identifier for `Claude3OpusModel` is consistent with other model naming conventions.src/Providers/Amazon.Bedrock/src/Chat/Settings/MistralInstructChatSettings.cs (1)
`12-12`: The `MaxTokens` value has been standardized to 4000, aligning with the PR objectives. Confirm that this change is reflected across all relevant settings classes.src/Providers/Amazon.Bedrock/src/Chat/Settings/Ai21LabsJurassic2ChatSettings.cs (1)
`12-12`: The `MaxTokens` value has been standardized to 4000, aligning with the PR objectives. Confirm that this change is reflected across all relevant settings classes.src/Providers/Amazon.Bedrock/src/Chat/MetaLlamaChatModel.cs (1)
`11-11`: Ensure that the `MetaLlamaChatModel` class is correctly inheriting and implementing the necessary methods. Confirm that the identifiers and settings used are consistent with the PR objectives.src/Providers/Amazon.Bedrock/src/Chat/MistralModel.cs (1)
`11-11`: Ensure that the `MistralModel` class is correctly inheriting and implementing the necessary methods. Confirm that the identifiers and settings used are consistent with the PR objectives.
This is an automated message generated by Sweep AI.
added Claude3 Opus added Llama3 8B Instruct added Llama3 70B Instruct added Mistral Large fixed default tokens
Summary by CodeRabbit