-
Notifications
You must be signed in to change notification settings - Fork 3.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
.Net: Anthropic - streaming #8560
.Net: Anthropic - streaming #8560
Conversation
Please fix the UnitTests. |
not sure what else is going on with this PR but I have the following notes: I wasn't able to get Vertex AI anthropic to work without the following changes:
I would suggest making these simple changes but these will likely impact Anthropic and Bedrock implementations of this. Perhaps some additional work needs to be done to make things more compatible (maybe let's pass the type of service and not try to determine it implicitly?) As for the failing unit tests I'm not sure what's going on there. |
Removed readonly modifiers from private fields _contentBlock and _delta to ensure proper JSON deserialization. This change allows modification of these fields during runtime, facilitating smoother data handling.
@RogerBarreto fixed |
17cfd02
into
microsoft:feature-connectors-anthropic
Motivation and Context
#5690
Description
Added streaming functionality and related tests.
@RogerBarreto @RogerBarret0
Contribution Checklist